@qusaieilouti99/call-manager 0.1.36 → 0.1.37

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

@@ -12,7 +12,7 @@ import android.media.MediaPlayer
 import android.media.RingtoneManager
 import android.net.Uri
 import android.os.Build
-import android.os.Bundle // Explicit import
+import android.os.Bundle // Explicit import for Bundle
 import android.os.PowerManager
 import android.telecom.CallAudioState // Explicit import
 import android.telecom.Connection // Explicit import
@@ -21,7 +21,7 @@ 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
+import android.util.Log // Explicit import for Log
 import android.graphics.Color
 import android.app.Person
 import org.json.JSONArray
@@ -61,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
 
@@ -78,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()
@@ -126,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")
@@ -174,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")
@@ -211,6 +207,9 @@ object CallEngine {
         }
 
         try {
+            // 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")
@@ -476,7 +475,6 @@ object CallEngine {
                         // Earpiece is active if speaker, bluetooth, and headset are off
                         // This heuristic is tricky, relies on other routes being off
                         if (audioManager?.isSpeakerphoneOn == false && audioManager?.isWiredHeadsetOn == false && audioManager?.isBluetoothScoOn == false && !device.isSource) {
-                             // This heuristic is tricky, relies on other routes being off
                             currentRoute = "Earpiece"
                         }
                     }
@@ -676,7 +674,7 @@ object CallEngine {
         val phoneAccountHandle = getPhoneAccountHandle(context)
         if (telecomManager.getPhoneAccount(phoneAccountHandle) == null) {
             val phoneAccount = PhoneAccount.builder(phoneAccountHandle, "PingMe Call")
-                .setCapabilities(PhoneAccount.CAPABILITY_SELF_MANAGED or PhoneAccount.CAPABILITY_SUPPORTS_HOLD)
+                .setCapabilities(PhoneAccount.CAPABILITY_SELF_MANAGED or PhoneAccount.CAPABILITY_SUPPORTS_HOLD) // Resolved
                 .build()
             try {
                 telecomManager.registerPhoneAccount(phoneAccount)

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

@@ -6,7 +6,7 @@ import android.app.Service
 import android.content.Context
 import android.content.Intent
 import android.os.Build
-import android.util.Log // Explicit import
+import android.util.Log
 import android.os.IBinder
 import android.app.NotificationManager // Explicit import
 
@@ -61,7 +61,7 @@ class CallForegroundService : Service() {
                 NotificationManager.IMPORTANCE_LOW
             )
             channel.description = "Notification for ongoing calls"
-            val manager = getSystemService(Context.NOTIFICATION_SERVICE) as NotificationManager // Corrected usage
+            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

@@ -11,7 +11,6 @@ class CallManager : HybridCallManagerSpec() {
     private val TAG = "CallManager"
 
     // Listener type now directly matches the generated Func_void_CallEventType_std__string
-    // This is the correct type from your Nitro codegen output.
     private var currentListener: Func_void_CallEventType_std__string? = null
 
     override fun endCall(callId: String) {
@@ -50,11 +49,10 @@ 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
-    // This is the primary fix for the `addListener overrides nothing` error.
     override fun addListener(
         listener: Func_void_CallEventType_std__string // Use generated type directly
     ): Func_void { // Return generated type

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

@@ -121,8 +121,8 @@ class MyConnection(
         Log.d(TAG, "Stopping DTMF tone for callId: $callId")
     }
 
-    override fun onConnectionEvent(event: String, extras: Bundle?) { // Resolved - signature is correct
-        super.onConnectionEvent(event, extras)
+    override fun onConnectionEvent(event: String, extras: Bundle?) { // Resolved
+        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

@@ -31,7 +31,7 @@ class MyConnectionService : ConnectionService() {
         val videoState = if (isVideoCallBoolean) VideoProfile.STATE_BIDIRECTIONAL else VideoProfile.STATE_AUDIO_ONLY
         connection.setVideoState(videoState)
         connection.setRinging()
-        Log.d(TAG, "Created incoming connection for callId: ${connection.callId}. Status: RINGING, VideoState: $videoState")
+        Log.d(TAG, "Created incoming connection for callId: ${connection.callId}. Status: RINGING, VideoState: $videoState") // Accesses internal callId
         return connection
     }
 
@@ -49,7 +49,7 @@ class MyConnectionService : ConnectionService() {
         val videoState = if (isVideoCallBoolean) VideoProfile.STATE_BIDIRECTIONAL else VideoProfile.STATE_AUDIO_ONLY
         connection.setVideoState(videoState)
         connection.setDialing()
-        Log.d(TAG, "Created outgoing connection for callId: ${connection.callId}. Status: DIALING, VideoState: $videoState")
+        Log.d(TAG, "Created outgoing connection for callId: ${connection.callId}. Status: DIALING, VideoState: $videoState") // Accesses internal callId
 
         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.")

package/package.json

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