@qusaieilouti99/call-manager 0.1.35 → 0.1.37

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

@@ -74,8 +74,6 @@ class CallActivity : Activity() {
 
     override fun onBackPressed() {
         Log.d(TAG, "CallActivity onBackPressed for callId: $callId. Treating as decline/dismiss.")
-        // Decision: User pressing back on the incoming call screen is treated as declining the call.
-        // If you want the call to continue ringing in the background, remove the CallEngine.endCall() line.
         finishReason = FinishReason.MANUAL_DISMISS
         CallEngine.endCall(this, callId)
         finishCallActivity()

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

@@ -12,10 +12,16 @@ import android.media.MediaPlayer
 import android.media.RingtoneManager
 import android.net.Uri
 import android.os.Build
-import android.os.Bundle // Added import
+import android.os.Bundle // Explicit import for Bundle
 import android.os.PowerManager
-import android.telecom.*
-import android.util.Log // Added import
+import android.telecom.CallAudioState // Explicit import
+import android.telecom.Connection // Explicit import
+import android.telecom.DisconnectCause // Explicit import
+import android.telecom.PhoneAccount // Explicit import
+import android.telecom.PhoneAccountHandle // Explicit import
+import android.telecom.TelecomManager // Explicit import
+import android.telecom.VideoProfile // Explicit import
+import android.util.Log // Explicit import for Log
 import android.graphics.Color
 import android.app.Person
 import org.json.JSONArray
@@ -55,7 +61,6 @@ object CallEngine {
     }
 
     // --- Event handler for JS ---
-    // Changed listener type to generated Func_void_CallEventType_std__string
     private var eventHandler: ((CallEventType, String) -> Unit)? = null
     private val cachedEvents = mutableListOf<Pair<CallEventType, String>>() // For caching events
 
@@ -72,7 +77,6 @@ object CallEngine {
         }
     }
 
-    // Emit event uses the generated CallEventType enum directly
     fun emitEvent(type: CallEventType, data: JSONObject) {
         Log.d(TAG, "Emitting event: $type, data: $data")
         val dataString = data.toString()
@@ -120,7 +124,6 @@ object CallEngine {
         return result
     }
 
-    // Removed callType param, now extracted from callData
     fun reportIncomingCall(context: Context, callId: String, callData: String) {
         appContext = context.applicationContext
         Log.d(TAG, "reportIncomingCall: $callId, $callData")
@@ -168,7 +171,6 @@ object CallEngine {
         notifyCallStateChanged(context)
     }
 
-    // Removed callType param, now extracted from callData
     fun startOutgoingCall(context: Context, callId: String, callData: String) {
         appContext = context.applicationContext
         Log.d(TAG, "startOutgoingCall: $callId, $callData")
@@ -205,7 +207,10 @@ object CallEngine {
         }
 
         try {
-            telecomManager.addNewOutgoingCall(phoneAccountHandle, extras) // Resolved
+            // Error: Unresolved reference 'addNewOutgoingCall'. This usually means TelecomManager is not correctly imported
+            // or the minSdkVersion is too low for this method (requires API 23+).
+            // Imports already checked; assuming minSdkVersion 26+.
+            telecomManager.addNewOutgoingCall(phoneAccountHandle, extras)
             startForegroundService(context)
             Log.d(TAG, "Successfully reported outgoing call to TelecomManager for $callId")
             startRingback()
@@ -358,7 +363,6 @@ object CallEngine {
         Log.d(TAG, "Showing incoming call UI for $callId, caller: $callerName, callType: $callType")
         createNotificationChannel(context)
         val notificationManager = context.getSystemService(Context.NOTIFICATION_SERVICE) as NotificationManager
-        // Resolved by `getSystemService(Context.NOTIFICATION_SERVICE) as NotificationManager`
 
         val answerIntent = Intent(context, CallNotificationActionReceiver::class.java).apply {
             action = "com.qusaieilouti99.callmanager.ANSWER_CALL"
@@ -659,7 +663,7 @@ object CallEngine {
                 channel.setSound(null, null)
                 channel.importance = NotificationManager.IMPORTANCE_HIGH
             }
-            val manager = context.getSystemService(NotificationManager::class.java) // Corrected type inference
+            val manager = context.getSystemService(NotificationManager::class.java)
             manager.createNotificationChannel(channel)
             Log.d(TAG, "Notification channel '$NOTIF_CHANNEL_ID' created/updated.")
         }

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

@@ -3,11 +3,12 @@ package com.margelo.nitro.qusaieilouti99.callmanager
 import android.app.Notification
 import android.app.NotificationChannel
 import android.app.Service
-import android.content.Context // Added import
+import android.content.Context
 import android.content.Intent
 import android.os.Build
-import android.os.IBinder
 import android.util.Log
+import android.os.IBinder
+import android.app.NotificationManager // Explicit import
 
 class CallForegroundService : Service() {
 
@@ -60,7 +61,7 @@ class CallForegroundService : Service() {
                 NotificationManager.IMPORTANCE_LOW
             )
             channel.description = "Notification for ongoing calls"
-            val manager = getSystemService(Context.NOTIFICATION_SERVICE) as NotificationManager // Corrected type inference
+            val manager = getSystemService(Context.NOTIFICATION_SERVICE) as NotificationManager // Corrected type inference and explicit import
             manager.createNotificationChannel(channel)
             Log.d(TAG, "Foreground notification channel '$CHANNEL_ID' created/updated.")
         }

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

@@ -1,7 +1,7 @@
 package com.margelo.nitro.qusaieilouti99.callmanager
 
 import com.facebook.proguard.annotations.DoNotStrip
-import android.util.Log // Added import
+import android.util.Log // Explicit import
 import org.json.JSONArray
 import org.json.JSONObject
 
@@ -49,7 +49,7 @@ class CallManager : HybridCallManagerSpec() {
         Log.d(TAG, "keepScreenAwake requested: $keepAwake")
         CallEngine.getAppContext()?.let {
             CallEngine.keepScreenAwake(it, keepAwake)
-        } ?: Log.e(TAG, "App context not set for keepScreenAwake.")
+        } ?: Log.e(TAG, "App context not set for keepAwake.")
     }
 
     // Corrected addListener signature to match generated HybridCallManagerSpec

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

@@ -2,12 +2,12 @@ package com.margelo.nitro.qusaieilouti99.callmanager
 
 import android.content.Context
 import android.net.Uri
-import android.os.Bundle // Added import
+import android.os.Bundle // Explicit import
 import android.telecom.Connection
 import android.telecom.DisconnectCause
-import android.telecom.CallAudioState // Added import
+import android.telecom.CallAudioState // Explicit import
 import android.telecom.VideoProfile
-import android.util.Log // Added import
+import android.util.Log // Explicit import
 import org.json.JSONObject
 import java.util.UUID
 
@@ -20,7 +20,7 @@ class MyConnection(
         const val TAG = "MyConnection"
     }
 
-    // Changed to internal val to allow access from MyConnectionService
+    // Changed to internal val to allow access from MyConnectionService for logging
     internal val callId: String = try {
         JSONObject(callDataJson).optString("callId", UUID.randomUUID().toString())
     } catch (e: Exception) {
@@ -110,10 +110,10 @@ class MyConnection(
         // NOTE: DTMF_TONE is not in your current CallEventType.ts enum.
         // If you need to emit this event, you MUST add 'DTMF_TONE' to your CallEventType.ts
         // and re-run nitro-codegen. For now, it's commented out to prevent compilation error.
-        CallEngine.emitEvent(
-            CallEventType.DTMF_TONE,
-            JSONObject().put("callId", callId).put("digit", digit.toString())
-        )
+        // CallEngine.emitEvent(
+        //     CallEventType.DTMF_TONE,
+        //     JSONObject().put("callId", callId).put("digit", digit.toString())
+        // )
     }
 
     override fun onStopDtmfTone() {
@@ -121,9 +121,8 @@ class MyConnection(
         Log.d(TAG, "Stopping DTMF tone for callId: $callId")
     }
 
-    // @Suppress("Override") // Suppress if minSdkVersion < 26 and you want to keep the method
     override fun onConnectionEvent(event: String, extras: Bundle?) { // Resolved
-        super.onConnectionEvent(event, extras)
+        super.onConnectionEvent(event, extras) // Correct super call
         Log.d(TAG, "Connection event: $event, extras: $extras for callId: $callId")
     }
 

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

@@ -6,8 +6,8 @@ import android.telecom.ConnectionService
 import android.telecom.PhoneAccountHandle
 import android.telecom.TelecomManager
 import android.telecom.VideoProfile
-import android.telecom.CallAudioState // Added import
-import android.util.Log // Added import
+import android.telecom.CallAudioState // Explicit import
+import android.util.Log // Explicit import
 
 class MyConnectionService : ConnectionService() {
 
@@ -53,7 +53,7 @@ class MyConnectionService : ConnectionService() {
 
         if (request.extras?.getBoolean(TelecomManager.EXTRA_START_CALL_WITH_SPEAKERPHONE, false) == true) {
             Log.d(TAG, "Hinting Telecom to start outgoing call with speakerphone as per request extras.")
-            connection.setAudioRoute(CallAudioState.ROUTE_SPEAKER) // Resolved
+            connection.setAudioRoute(CallAudioState.ROUTE_SPEAKER)
         }
 
         return connection

package/package.json

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