@qusaieilouti99/call-manager 0.1.199 → 0.1.201

package/ios/AudioManager.swift

@@ -247,4 +247,11 @@ class AudioManager {
             return "Earpiece"
         }
         switch output.portType {
-        case
+        case .bluetoothHFP, .bluetoothA2DP, .bluetoothLE: return "Bluetooth"
+        case .builtInSpeaker: return "Speaker"
+        case .builtInReceiver: return "Earpiece"
+        case .headphones, .headsetMic: return "Headset"
+        default: return "Earpiece"
+        }
+    }
+}

package/ios/CallEngine.swift

@@ -315,7 +315,13 @@ class CallEngine {
 
     private func updateOverallIdleTimerDisabledState() {
         let shouldDisable = manualIdleTimerDisabled || hasActiveCalls()
+
         DispatchQueue.main.async {
+            // Only modify UI if app is in foreground/active state
+            guard UIApplication.shared.applicationState == .active else {
+                self.logger.info("Skipping idle timer update - app not active")
+                return
+            }
             UIApplication.shared.isIdleTimerDisabled = shouldDisable
         }
     }

package/ios/CallManager.swift

@@ -3,6 +3,9 @@ import NitroModules
 import OSLog
 import UIKit
 
+/// The public-facing bridge class that implements the `HybridCallManagerSpec` protocol.
+/// Its responsibility is to translate calls from the JavaScript layer into commands
+/// for the `CallEngine` singleton. It acts as a thin, safe interface to the native call logic.
 public class CallManager: HybridCallManagerSpec {
     private let logger = Logger(
         subsystem: "com.qusaieilouti99.callmanager",
@@ -53,6 +56,7 @@ public class CallManager: HybridCallManagerSpec {
             self.logger.debug("🎯 event \(event.stringValue), payload.len=\(payload.count)")
             listener(event, payload)
         }
+        // Return a closure that will be called by the JS layer to unsubscribe.
         return {
             self.logger.info("🎯 removeListener ▶ js → native")
             CallEngine.shared.setEventHandler(nil)
@@ -150,10 +154,12 @@ public class CallManager: HybridCallManagerSpec {
     }
 
     public func requestOverlayPermissionAndroid() throws -> Bool {
+        // This is an Android-specific method, so we provide a no-op success case for iOS.
         return true
     }
 
     public func hasOverlayPermissionAndroid() throws -> Bool {
+        // This is an Android-specific method, so we provide a no-op success case for iOS.
         return true
     }
 }

package/ios/VoIPTokenManager.swift

@@ -2,7 +2,12 @@ import Foundation
 import PushKit
 import OSLog
 
+/// Manages the VoIP push token lifecycle using Apple's PushKit framework.
+/// This class is a singleton responsible for registering for VoIP pushes, receiving the token,
+/// and handling incoming push payloads that represent new calls.
 class VoIPTokenManager: NSObject, PKPushRegistryDelegate {
+    // MARK: - Singleton and Properties
+
     static let shared = VoIPTokenManager()
     private let logger = Logger(
         subsystem: "com.qusaieilouti99.callmanager",
@@ -14,39 +19,49 @@ class VoIPTokenManager: NSObject, PKPushRegistryDelegate {
 
     private override init() {
         super.init()
-        logger.info("VoIPTokenManager init")
+        logger.info("VoIPTokenManager singleton created.")
     }
 
+    // MARK: - Public Methods
+
+    /// Configures the PushKit registry. This should be called once when the app's call engine is initialized.
     func setupPushKit() {
         guard pushRegistry == nil else {
-            logger.info("PushKit already setup")
+            logger.info("PushKit already set up. Ignoring.")
             return
         }
         pushRegistry = PKPushRegistry(queue: .main)
         pushRegistry?.delegate = self
         pushRegistry?.desiredPushTypes = [.voIP]
-        logger.info("PushKit registry configured")
+        logger.info("PushKit registry configured.")
     }
 
+    /// Registers a listener (typically from the JS bridge) to receive the VoIP token.
     func registerTokenListener(_ listener: @escaping (String) -> Void) {
-        logger.info("registerTokenListener")
+        logger.info("Registering VoIP token listener.")
         tokenListener = listener
+        // If the token has already been received, provide it to the new listener immediately.
         if let t = cachedToken {
-            logger.info("returning cached token")
+            logger.info("Providing cached VoIP token to new listener.")
             listener(t)
         }
     }
 
+    /// Unregisters the token listener.
     func unregisterTokenListener() {
-        logger.info("unregisterTokenListener")
+        logger.info("Unregistering VoIP token listener.")
         tokenListener = nil
     }
 
+    // MARK: - PKPushRegistryDelegate
+
+    /// Called by the system when a new VoIP push token is available or has been updated.
     func pushRegistry(
         _ registry: PKPushRegistry,
         didUpdate pushCredentials: PKPushCredentials,
         for type: PKPushType
     ) {
+        // Convert the token data to a hex string for transmission.
         let token = pushCredentials.token
             .map { String(format: "%02.2hhx", $0) }
             .joined()
@@ -55,24 +70,42 @@ class VoIPTokenManager: NSObject, PKPushRegistryDelegate {
         tokenListener?(token)
     }
 
+    /// Called by the system when the VoIP push token has been invalidated.
     func pushRegistry(_ registry: PKPushRegistry, didInvalidatePushTokenFor type: PKPushType) {
         logger.warning("VoIP token invalidated.")
         cachedToken = nil
         tokenListener?("")
     }
 
-    // MARK: - PKPushRegistryDelegate
+    /// This is the entry point for an incoming VoIP call when the app is in the background or killed.
+    /// It is CRITICAL that the `completion` handler is called in all code paths to avoid the
+    /// system terminating the app.
     func pushRegistry(
         _ registry: PKPushRegistry,
         didReceiveIncomingPushWith payload: PKPushPayload,
         for type: PKPushType,
         completion: @escaping () -> Void
     ) {
-
         logger.info("🔔 Received VoIP Push")
 
+        // Initialize if not already initialized
+        CallEngine.shared.initialize()
+
+        // *** THIS IS THE WATCHDOG TIMER ***
+        // It creates a failsafe that will call the completion handler after 10 seconds
+        // if our main logic hangs or fails silently, preventing the OS from killing the app.
+        let completionTimeout = DispatchWorkItem {
+            self.logger.error("⚠️ VoIP push handling timed out. Forcing completion.")
+            completion()
+        }
+
+        DispatchQueue.main.asyncAfter(deadline: .now() + 10, execute: completionTimeout)
+
+        // The payload is expected to have a specific structure.
+        // Using a key like "custom_payload" or "data" is common.
         guard let info = payload.dictionaryPayload["custom_payload"] as? [AnyHashable: Any] else {
             logger.error("❌ Invalid payload format or missing 'custom_payload' dictionary.")
+            completionTimeout.cancel() // Defuse the timer
             completion()
             return
         }
@@ -85,6 +118,7 @@ class VoIPTokenManager: NSObject, PKPushRegistryDelegate {
                 .error(
                     "❌ Missing required fields in VoIP payload: callId, callType, or name."
                 )
+            completionTimeout.cancel() // Defuse the timer
             completion()
             return
         }
@@ -93,7 +127,9 @@ class VoIPTokenManager: NSObject, PKPushRegistryDelegate {
         let metadata = info["metadata"] as? String
 
         logger.info("📞 Reporting incoming call from VoIP push: \(callId)")
-        // *** FIX: Updated to match the corrected CallEngine signature ***
+
+        // Delegate the call reporting to the CallEngine.
+        // The CallEngine is self-initializing, making this call safe.
         CallEngine.shared.reportIncomingCall(
             callId: callId,
             callType: callType,
@@ -102,6 +138,8 @@ class VoIPTokenManager: NSObject, PKPushRegistryDelegate {
             metadata: metadata
         ) { success in
             self.logger.info("📞 CallKit report from VoIP push completed. Success: \(success)")
+            // This is the crucial call to satisfy the PushKit watchdog.
+            completionTimeout.cancel() // Defuse the timer
             completion()
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qusaieilouti99/call-manager",
-  "version": "0.1.199",
+  "version": "0.1.201",
   "description": "Call manager",
   "main": "./lib/module/index.js",
   "types": "./lib/typescript/src/index.d.ts",