fix: prevent sendMessage() auto-reconnect on intentional background disconnect

Add isIntentionallyDisconnecting flag to ACPClientWrapper. Set in
disconnect(), cleared in connect(). Guards the sendMessage() reconnect
loop so backgrounding the app (which calls disconnectAll()) no longer
creates a second WebSocket connection racing AgentManager's reconnect
on foreground — which was preventing push notifications from firing.

Also adds scenePhase handler in AptoveApp: background closes all
WebSocket connections (with UIBackgroundTask to complete the close
frame), active reconnects after 0.5s delay.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: saltukalakus

Aptove/AptoveApp.swift

@@ -1,5 +1,6 @@
 import SwiftUI
 import UserNotifications
+import UIKit
 
 /// AppDelegate for handling push notification registration
 class AppDelegate: NSObject, UIApplicationDelegate, UNUserNotificationCenterDelegate {
@@ -72,14 +73,26 @@ struct AptoveApp: App {
                     .onChange(of: scenePhase) { _, newPhase in
                         switch newPhase {
                         case .active:
-                            print("🌅 [push-dbg] App became ACTIVE — reconnecting agents")
-                            agentManager.autoConnectAllAgents()
+                            print("🌅 [push-dbg] App became ACTIVE — reconnecting agents (0.5s delay for in-flight disconnect)")
+                            // Small delay so any in-flight background disconnect completes before
+                            // we reconnect, preventing two simultaneous connections on the bridge.
+                            Task {
+                                try? await Task.sleep(nanoseconds: 500_000_000)
+                                agentManager.autoConnectAllAgents()
+                            }
                         case .inactive:
                             print("🌄 [push-dbg] App became INACTIVE (transitioning)")
                         case .background:
                             print("🌙 [push-dbg] App entered BACKGROUND — closing WebSocket so bridge detects disconnect and pushes")
+                            // Request background execution time so iOS doesn't suspend the process
+                            // before the WebSocket close frame is delivered to the bridge.
+                            var bgTaskId = UIBackgroundTaskIdentifier.invalid
+                            bgTaskId = UIApplication.shared.beginBackgroundTask(withName: "ws-background-disconnect") {
+                                UIApplication.shared.endBackgroundTask(bgTaskId)
+                            }
                             Task {
                                 await agentManager.disconnectAll()
+                                UIApplication.shared.endBackgroundTask(bgTaskId)
                             }
                         @unknown default:
                             break

Aptove/Services/ACPClientWrapper.swift

@@ -190,6 +190,8 @@ class ACPClientWrapper: ObservableObject {
     }
 
     func connect(existingSessionId: String?) async {
+        // An intentional disconnect is over — allow sendMessage() to reconnect again.
+        isIntentionallyDisconnecting = false
         print("🔌 ACPClientWrapper.connect(): Starting connection flow...")
         if let sessionId = existingSessionId {
             print("🔌 ACPClientWrapper.connect(): Will try to load session: \(sessionId)")
@@ -427,6 +429,8 @@ class ACPClientWrapper: ObservableObject {
     }
 
     func disconnect() async {
+        // Signal in-flight sendMessage() Tasks to skip their reconnect loop.
+        isIntentionallyDisconnecting = true
         // Cancel the transport observer first so an intentional disconnect
         // does not trigger the unexpected-disconnect callback.
         transportObserverTask?.cancel()
@@ -585,6 +589,10 @@ class ACPClientWrapper: ObservableObject {
     var onUnexpectedDisconnect: (() -> Void)?
     private var transportObserverTask: Task<Void, Never>?
 
+    /// Set to true by `disconnect()` to suppress the sendMessage() internal reconnect
+    /// when the disconnect is intentional (e.g. app backgrounding). Cleared by `connect()`.
+    private var isIntentionallyDisconnecting = false
+
     /// Called when another device sends a user message to the same session.
     /// The string is the plain text of the remote message.
     var onRemoteUserMessage: ((String) -> Void)?
@@ -723,6 +731,17 @@ class ACPClientWrapper: ObservableObject {
                     return
                 }
 
+                // Skip reconnect if disconnect() was called intentionally (e.g. app backgrounding).
+                // AgentManager will reconnect via autoConnectAllAgents() when the app foregrounds.
+                if await MainActor.run(body: { self.isIntentionallyDisconnecting }) {
+                    print("🔄 Skipping auto-reconnect — intentional disconnect in progress")
+                    await MainActor.run {
+                        self.connectionState = .disconnected
+                        onComplete(nil, error.localizedDescription)
+                    }
+                    return
+                }
+
                 // If the agent says the session doesn't exist, clear it
                 // so the reconnect creates a fresh session instead of
                 // retrying the same stale session ID in a loop.