@qusaieilouti99/call-manager 0.1.63 → 0.1.65

package/android/src/main/java/com/margelo/nitro/qusaieilouti99/callmanager/CallEngine.kt

@@ -48,13 +48,11 @@ object CallEngine {
     private const val FOREGROUND_NOTIF_ID = 1001
 
     // Core context - initialized once and maintained
-    // Improved context management with thread safety
     @Volatile
     private var appContext: Context? = null
     private val isInitialized = AtomicBoolean(false)
     private val initializationLock = Any()
 
-
     // Audio & Media
     private var ringtone: android.media.Ringtone? = null
     private var ringbackPlayer: MediaPlayer? = null
@@ -87,7 +85,7 @@ object CallEngine {
         fun onLockScreenBypassChanged(shouldBypass: Boolean)
     }
 
-    // --- INITIALIZATION with better error handling ---
+    // --- INITIALIZATION - Fixed for better context management ---
     fun initialize(context: Context) {
         synchronized(initializationLock) {
             if (isInitialized.compareAndSet(false, true)) {
@@ -143,38 +141,79 @@ object CallEngine {
         }
     }
 
-    // --- Audio Focus Management (Simplified) ---
+    // --- FIXED Audio Focus Management ---
     private val audioFocusChangeListener = AudioManager.OnAudioFocusChangeListener { focusChange ->
         Log.d(TAG, "Audio focus changed: $focusChange")
         when (focusChange) {
-            AudioManager.AUDIOFOCUS_LOSS,
+            AudioManager.AUDIOFOCUS_LOSS -> {
+                Log.d(TAG, "Permanent audio focus loss - another app took focus")
+                hasAudioFocus = false
+                isSystemCallActive = true
+                holdSystemCalls()
+            }
             AudioManager.AUDIOFOCUS_LOSS_TRANSIENT -> {
+                Log.d(TAG, "Transient audio focus loss - temporary interruption")
                 hasAudioFocus = false
                 isSystemCallActive = true
                 holdSystemCalls()
             }
+            AudioManager.AUDIOFOCUS_LOSS_TRANSIENT_CAN_DUCK -> {
+                Log.d(TAG, "Audio focus loss with ducking - lowering volume but keeping active")
+                // Don't hold the call for ducking, just lower volume
+                hasAudioFocus = false
+            }
             AudioManager.AUDIOFOCUS_GAIN -> {
+                Log.d(TAG, "Audio focus gained")
                 hasAudioFocus = true
                 isSystemCallActive = false
+                // Delay resuming to avoid rapid hold/unhold cycles
                 Handler(Looper.getMainLooper()).postDelayed({
                     resumeSystemHeldCalls()
-                }, 1000)
+                }, 500)  // Reduced from 1000ms
+            }
+            AudioManager.AUDIOFOCUS_GAIN_TRANSIENT -> {
+                Log.d(TAG, "Transient audio focus gained")
+                hasAudioFocus = true
             }
         }
         updateForegroundNotification()
     }
 
     private fun holdSystemCalls() {
-        activeCalls.values.filter { it.state == CallState.ACTIVE }.forEach { call ->
-            if (!call.wasHeldBySystem) {
+        val callsToHold = activeCalls.values.filter {
+            it.state == CallState.ACTIVE && !it.wasHeldBySystem
+        }
+
+        if (callsToHold.isEmpty()) {
+            Log.d(TAG, "No active calls to hold due to audio focus loss")
+            return
+        }
+
+        Log.d(TAG, "Holding ${callsToHold.size} calls due to audio focus loss")
+        callsToHold.forEach { call ->
+            // Add a small delay to prevent holding immediately after answering
+            val timeSinceAnswer = System.currentTimeMillis() - call.timestamp
+            if (timeSinceAnswer > 2000) {  // Only hold if call has been active for 2+ seconds
                 holdCallInternal(call.callId, heldBySystem = true)
+            } else {
+                Log.d(TAG, "Skipping hold for recently answered call: ${call.callId}")
             }
         }
         stopRingback()
     }
 
     private fun resumeSystemHeldCalls() {
-        activeCalls.values.filter { it.state == CallState.HELD && it.wasHeldBySystem }.forEach { call ->
+        val callsToResume = activeCalls.values.filter {
+            it.state == CallState.HELD && it.wasHeldBySystem
+        }
+
+        if (callsToResume.isEmpty()) {
+            Log.d(TAG, "No system-held calls to resume")
+            return
+        }
+
+        Log.d(TAG, "Resuming ${callsToResume.size} system-held calls")
+        callsToResume.forEach { call ->
             unholdCallInternal(call.callId, resumedBySystem = true)
         }
     }
@@ -192,12 +231,13 @@ object CallEngine {
                             .setContentType(AudioAttributes.CONTENT_TYPE_SPEECH)
                             .build()
                     )
-                    .setOnAudioFocusChangeListener(audioFocusChangeListener)
+                    .setOnAudioFocusChangeListener(audioFocusChangeListener, Handler(Looper.getMainLooper()))
+                    .setAcceptsDelayedFocusGain(true)  // Added this
                     .build()
             }
             val result = audioManager?.requestAudioFocus(audioFocusRequest!!)
             hasAudioFocus = result == AudioManager.AUDIOFOCUS_REQUEST_GRANTED
-            Log.d(TAG, "Audio focus request result: $result")
+            Log.d(TAG, "Audio focus request result: $result (granted: $hasAudioFocus)")
             hasAudioFocus
         } else {
             @Suppress("DEPRECATION")
@@ -207,7 +247,7 @@ object CallEngine {
                 AudioManager.AUDIOFOCUS_GAIN
             )
             hasAudioFocus = result == AudioManager.AUDIOFOCUS_REQUEST_GRANTED
-            Log.d(TAG, "Audio focus request result (legacy): $result")
+            Log.d(TAG, "Audio focus request result (legacy): $result (granted: $hasAudioFocus)")
             hasAudioFocus
         }
     }
@@ -345,7 +385,7 @@ object CallEngine {
 
         try {
             telecomManager.addNewIncomingCall(phoneAccountHandle, extras)
-            startForegroundService() // Fixed: Always start foreground service
+            startForegroundService()
             Log.d(TAG, "Successfully reported incoming call to TelecomManager for $callId")
         } catch (e: SecurityException) {
             Log.e(TAG, "SecurityException: Failed to report incoming call. Check MANAGE_OWN_CALLS permission: ${e.message}", e)
@@ -395,7 +435,7 @@ object CallEngine {
         val phoneAccountHandle = getPhoneAccountHandle()
         val addressUri = Uri.fromParts(PhoneAccount.SCHEME_TEL, targetName, null)
 
-        // 1) build a bundle of ONLY your own keys
+        // Build a bundle of ONLY your own keys
         val outgoingExtras = Bundle().apply {
             putString(MyConnectionService.EXTRA_CALL_ID, callId)
             putString(MyConnectionService.EXTRA_CALL_TYPE, callType)
@@ -404,7 +444,7 @@ object CallEngine {
             metadata?.let { putString("metadata", it) }
         }
 
-        // 2) wrap under the single Telecom‐honored key
+        // Wrap under the single Telecom-honored key
         val extras = Bundle().apply {
             putParcelable(TelecomManager.EXTRA_PHONE_ACCOUNT_HANDLE, phoneAccountHandle)
             putBundle(TelecomManager.EXTRA_OUTGOING_CALL_EXTRAS, outgoingExtras)
@@ -414,8 +454,15 @@ object CallEngine {
         try {
             telecomManager.placeCall(addressUri, extras)
             startForegroundService()
-            requestAudioFocus()
-            startRingback()
+
+            // FIXED: Request audio focus BEFORE starting ringback
+            val audioFocusGranted = requestAudioFocus()
+            if (audioFocusGranted) {
+                startRingback()
+            } else {
+                Log.w(TAG, "Audio focus not granted for outgoing call, skipping ringback")
+            }
+
             bringAppToForeground()
             keepScreenAwake(true)
             setInitialAudioRoute(callType)
@@ -464,13 +511,13 @@ object CallEngine {
         registerPhoneAccount()
         requestAudioFocus()
         bringAppToForeground()
-        startForegroundService() // Fixed: Start foreground service for JS-initiated calls
+        startForegroundService()
         keepScreenAwake(true)
         setInitialAudioRoute(callType)
         updateLockScreenBypass()
 
-        // Emit call answered event with metadata
-        emitCallAnsweredWithMetadata(callId)
+        // Emit outgoing call answered event with metadata for JS-initiated calls
+        emitOutgoingCallAnsweredWithMetadata(callId)
     }
 
     // --- Call Answer Management ---
@@ -484,6 +531,7 @@ object CallEngine {
         coreCallAnswered(callId, isLocalAnswer = true)
     }
 
+    // FIXED: Core Call Answered Method with proper audio focus handling
     private fun coreCallAnswered(callId: String, isLocalAnswer: Boolean) {
         val context = requireContext()
         Log.d(TAG, "coreCallAnswered: $callId, isLocalAnswer: $isLocalAnswer")
@@ -494,13 +542,27 @@ object CallEngine {
             return
         }
 
+        // FIXED: Request audio focus FIRST, before stopping media
+        val audioFocusGranted = requestAudioFocus()
+        if (!audioFocusGranted) {
+            Log.w(TAG, "Failed to get audio focus for call $callId, but continuing...")
+            // Don't fail the call, but warn about audio issues
+        }
+
+        // Stop media AFTER getting audio focus
         stopRingtone()
         stopRingback()
         cancelIncomingCallUI()
-        requestAudioFocus()
 
-        activeCalls[callId] = callInfo.copy(state = CallState.ACTIVE)
-        currentCallId = callId
+        // FIXED: Only set call to ACTIVE if we have audio focus OR if it's a remote answer
+        if (audioFocusGranted || !isLocalAnswer) {
+            activeCalls[callId] = callInfo.copy(state = CallState.ACTIVE)
+            currentCallId = callId
+            Log.d(TAG, "Call $callId set to ACTIVE state")
+        } else {
+            Log.w(TAG, "Call $callId not set to ACTIVE due to audio focus failure")
+            return
+        }
 
         if (!canMakeMultipleCalls) {
             activeCalls.filter { it.key != callId }.values.forEach { otherCall ->
@@ -511,18 +573,25 @@ object CallEngine {
         }
 
         bringAppToForeground()
-        startForegroundService() // Fixed: Ensure foreground service is running
+        startForegroundService()
         keepScreenAwake(true)
         resetAudioMode()
         updateLockScreenBypass()
         updateForegroundNotification()
 
-        // Emit with metadata
-        emitCallAnsweredWithMetadata(callId)
+        // FIXED: Emit different events based on call direction
+        if (isLocalAnswer) {
+            // This is for incoming calls - user answered locally
+            emitCallAnsweredWithMetadata(callId)
+        } else {
+            // This is for outgoing calls - remote party answered
+            emitOutgoingCallAnsweredWithMetadata(callId)
+        }
 
         Log.d(TAG, "Call $callId successfully answered and UI cleaned up")
     }
 
+    // For incoming calls (local answer)
     private fun emitCallAnsweredWithMetadata(callId: String) {
         val callInfo = activeCalls[callId] ?: return
         val metadata = callMetadata[callId]
@@ -543,6 +612,27 @@ object CallEngine {
         })
     }
 
+    // For outgoing calls (remote answer)
+    private fun emitOutgoingCallAnsweredWithMetadata(callId: String) {
+        val callInfo = activeCalls[callId] ?: return
+        val metadata = callMetadata[callId]
+
+        emitEvent(CallEventType.OUTGOING_CALL_ANSWERED, JSONObject().apply {
+            put("callId", callId)
+            put("callType", callInfo.callType)
+            put("displayName", callInfo.displayName)
+            callInfo.pictureUrl?.let { put("pictureUrl", it) }
+            metadata?.let {
+                try {
+                    put("metadata", JSONObject(it))
+                } catch (e: Exception) {
+                    Log.w(TAG, "Invalid metadata JSON for callId: $callId", e)
+                    put("metadata", it) // fallback to string
+                }
+            }
+        })
+    }
+
     // --- Call Control Methods ---
     fun holdCall(callId: String) {
         holdCallInternal(callId, heldBySystem = false)
@@ -601,7 +691,7 @@ object CallEngine {
             return
         }
 
-        // Fixed: Simplified audio focus check to prevent UNHELD FAILED
+        // FIXED: Simplified audio focus check to prevent UNHELD FAILED
         if (!hasAudioFocus && !resumedBySystem && !requestAudioFocus()) {
             Log.w(TAG, "Failed to get audio focus for unhold - but continuing anyway")
             // Don't emit UNHELD FAILED - just continue

package/lib/typescript/src/CallEventType.d.ts

@@ -1,2 +1,2 @@
-export type CallEventType = 'AUDIO_DEVICES_CHANGED' | 'AUDIO_ROUTE_CHANGED' | 'CALL_HELD' | 'CALL_UNHELD' | 'CALL_UNHOLD_FAILED' | 'CALL_MUTED' | 'CALL_UNMUTED' | 'CALL_ANSWERED' | 'CALL_REJECTED' | 'CALL_ENDED' | 'DTMF_TONE';
+export type CallEventType = 'AUDIO_DEVICES_CHANGED' | 'AUDIO_ROUTE_CHANGED' | 'CALL_HELD' | 'CALL_UNHELD' | 'CALL_UNHOLD_FAILED' | 'CALL_MUTED' | 'CALL_UNMUTED' | 'CALL_ANSWERED' | 'OUTGOING_CALL_ANSWERED' | 'CALL_REJECTED' | 'CALL_ENDED' | 'DTMF_TONE';
 //# sourceMappingURL=CallEventType.d.ts.map

package/lib/typescript/src/CallEventType.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"CallEventType.d.ts","sourceRoot":"","sources":["../../../src/CallEventType.ts"],"names":[],"mappings":"AACA,MAAM,MAAM,aAAa,GACvB,uBAAuB,GACrB,qBAAqB,GACrB,WAAW,GACX,aAAa,GACb,oBAAoB,GACpB,YAAY,GACZ,cAAc,GACd,eAAe,GACf,eAAe,GACf,YAAY,GACZ,WAAW,CAAC"}
+{"version":3,"file":"CallEventType.d.ts","sourceRoot":"","sources":["../../../src/CallEventType.ts"],"names":[],"mappings":"AACA,MAAM,MAAM,aAAa,GACvB,uBAAuB,GACrB,qBAAqB,GACrB,WAAW,GACX,aAAa,GACb,oBAAoB,GACpB,YAAY,GACZ,cAAc,GACd,eAAe,GACf,wBAAwB,GACxB,eAAe,GACf,YAAY,GACZ,WAAW,CAAC"}

package/nitrogen/generated/android/c++/JCallEventType.hpp

@@ -49,6 +49,7 @@ namespace margelo::nitro::qusaieilouti99_callmanager {
       static const auto fieldCALL_MUTED = clazz->getStaticField<JCallEventType>("CALL_MUTED");
       static const auto fieldCALL_UNMUTED = clazz->getStaticField<JCallEventType>("CALL_UNMUTED");
       static const auto fieldCALL_ANSWERED = clazz->getStaticField<JCallEventType>("CALL_ANSWERED");
+      static const auto fieldOUTGOING_CALL_ANSWERED = clazz->getStaticField<JCallEventType>("OUTGOING_CALL_ANSWERED");
       static const auto fieldCALL_REJECTED = clazz->getStaticField<JCallEventType>("CALL_REJECTED");
       static const auto fieldCALL_ENDED = clazz->getStaticField<JCallEventType>("CALL_ENDED");
       static const auto fieldDTMF_TONE = clazz->getStaticField<JCallEventType>("DTMF_TONE");
@@ -70,6 +71,8 @@ namespace margelo::nitro::qusaieilouti99_callmanager {
           return clazz->getStaticFieldValue(fieldCALL_UNMUTED);
         case CallEventType::CALL_ANSWERED:
           return clazz->getStaticFieldValue(fieldCALL_ANSWERED);
+        case CallEventType::OUTGOING_CALL_ANSWERED:
+          return clazz->getStaticFieldValue(fieldOUTGOING_CALL_ANSWERED);
         case CallEventType::CALL_REJECTED:
           return clazz->getStaticFieldValue(fieldCALL_REJECTED);
         case CallEventType::CALL_ENDED:

package/nitrogen/generated/android/kotlin/com/margelo/nitro/qusaieilouti99/callmanager/CallEventType.kt

@@ -24,6 +24,7 @@ enum class CallEventType {
   CALL_MUTED,
   CALL_UNMUTED,
   CALL_ANSWERED,
+  OUTGOING_CALL_ANSWERED,
   CALL_REJECTED,
   CALL_ENDED,
   DTMF_TONE;

package/nitrogen/generated/ios/swift/CallEventType.swift

@@ -33,6 +33,8 @@ public extension CallEventType {
         self = .callUnmuted
       case "CALL_ANSWERED":
         self = .callAnswered
+      case "OUTGOING_CALL_ANSWERED":
+        self = .outgoingCallAnswered
       case "CALL_REJECTED":
         self = .callRejected
       case "CALL_ENDED":
@@ -65,6 +67,8 @@ public extension CallEventType {
         return "CALL_UNMUTED"
       case .callAnswered:
         return "CALL_ANSWERED"
+      case .outgoingCallAnswered:
+        return "OUTGOING_CALL_ANSWERED"
       case .callRejected:
         return "CALL_REJECTED"
       case .callEnded:

package/nitrogen/generated/shared/c++/CallEventType.hpp

@@ -37,9 +37,10 @@ namespace margelo::nitro::qusaieilouti99_callmanager {
     CALL_MUTED      SWIFT_NAME(callMuted) = 5,
     CALL_UNMUTED      SWIFT_NAME(callUnmuted) = 6,
     CALL_ANSWERED      SWIFT_NAME(callAnswered) = 7,
-    CALL_REJECTED      SWIFT_NAME(callRejected) = 8,
-    CALL_ENDED      SWIFT_NAME(callEnded) = 9,
-    DTMF_TONE      SWIFT_NAME(dtmfTone) = 10,
+    OUTGOING_CALL_ANSWERED      SWIFT_NAME(outgoingCallAnswered) = 8,
+    CALL_REJECTED      SWIFT_NAME(callRejected) = 9,
+    CALL_ENDED      SWIFT_NAME(callEnded) = 10,
+    DTMF_TONE      SWIFT_NAME(dtmfTone) = 11,
   } CLOSED_ENUM;
 
 } // namespace margelo::nitro::qusaieilouti99_callmanager
@@ -62,6 +63,7 @@ namespace margelo::nitro {
         case hashString("CALL_MUTED"): return CallEventType::CALL_MUTED;
         case hashString("CALL_UNMUTED"): return CallEventType::CALL_UNMUTED;
         case hashString("CALL_ANSWERED"): return CallEventType::CALL_ANSWERED;
+        case hashString("OUTGOING_CALL_ANSWERED"): return CallEventType::OUTGOING_CALL_ANSWERED;
         case hashString("CALL_REJECTED"): return CallEventType::CALL_REJECTED;
         case hashString("CALL_ENDED"): return CallEventType::CALL_ENDED;
         case hashString("DTMF_TONE"): return CallEventType::DTMF_TONE;
@@ -79,6 +81,7 @@ namespace margelo::nitro {
         case CallEventType::CALL_MUTED: return JSIConverter<std::string>::toJSI(runtime, "CALL_MUTED");
         case CallEventType::CALL_UNMUTED: return JSIConverter<std::string>::toJSI(runtime, "CALL_UNMUTED");
         case CallEventType::CALL_ANSWERED: return JSIConverter<std::string>::toJSI(runtime, "CALL_ANSWERED");
+        case CallEventType::OUTGOING_CALL_ANSWERED: return JSIConverter<std::string>::toJSI(runtime, "OUTGOING_CALL_ANSWERED");
         case CallEventType::CALL_REJECTED: return JSIConverter<std::string>::toJSI(runtime, "CALL_REJECTED");
         case CallEventType::CALL_ENDED: return JSIConverter<std::string>::toJSI(runtime, "CALL_ENDED");
         case CallEventType::DTMF_TONE: return JSIConverter<std::string>::toJSI(runtime, "DTMF_TONE");
@@ -101,6 +104,7 @@ namespace margelo::nitro {
         case hashString("CALL_MUTED"):
         case hashString("CALL_UNMUTED"):
         case hashString("CALL_ANSWERED"):
+        case hashString("OUTGOING_CALL_ANSWERED"):
         case hashString("CALL_REJECTED"):
         case hashString("CALL_ENDED"):
         case hashString("DTMF_TONE"):

package/package.json

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

package/src/CallEventType.ts

@@ -8,6 +8,7 @@ export type CallEventType =
   | 'CALL_MUTED'
   | 'CALL_UNMUTED'
   | 'CALL_ANSWERED'
+  | 'OUTGOING_CALL_ANSWERED'
   | 'CALL_REJECTED'
   | 'CALL_ENDED'
   | 'DTMF_TONE'; // ADD THIS LINE IF YOU NEED DTMF EVENTS