rns-nativecall 0.4.8 → 0.5.0

package/android/src/main/java/com/rnsnativecall/CallHeadlessTask.kt

@@ -18,33 +18,45 @@ class CallHeadlessTask : HeadlessJsTaskService() {
     override fun onCreate() {
         super.onCreate()
         val channelId = "call_service"
+        val notificationId = 1 // Constant ID for the foreground service
+
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
-            val channel = NotificationChannel(channelId, "Call Service", NotificationManager.IMPORTANCE_LOW)
+            val channel = NotificationChannel(
+                channelId, 
+                "Call Connection Service", 
+                NotificationManager.IMPORTANCE_LOW
+            )
             val manager = getSystemService(NotificationManager::class.java)
             manager.createNotificationChannel(channel)
         }
 
+        // IMPORTANT: Must have a setSmallIcon or the service will crash on modern Android
         val notification: Notification = NotificationCompat.Builder(this, channelId)
-            .setContentTitle("incoming...")
+            .setContentTitle("Incoming call...")
+            .setContentText("Establishing connection")
+            .setSmallIcon(android.R.drawable.sym_action_call) // Built-in icon as fallback
             .setPriority(NotificationCompat.PRIORITY_LOW)
+            .setCategory(NotificationCompat.CATEGORY_SERVICE)
             .build()
 
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.UPSIDE_DOWN_CAKE) {
-            startForeground(1, notification, ServiceInfo.FOREGROUND_SERVICE_TYPE_PHONE_CALL)
+            // Android 14+ requires the FOREGROUND_SERVICE_TYPE_PHONE_CALL type
+            startForeground(notificationId, notification, ServiceInfo.FOREGROUND_SERVICE_TYPE_PHONE_CALL)
         } else {
-            startForeground(1, notification)
+            startForeground(notificationId, notification)
         }
     }
 
     override fun getTaskConfig(intent: Intent?): HeadlessJsTaskConfig? {
         val extras = intent?.extras
         return if (extras != null) {
+            // Linear retry helps if the JS bridge is busy or mid-reload
             val retryPolicy = LinearCountingRetryPolicy(3, 1000)
             HeadlessJsTaskConfig(
                 "ColdStartCallTask",
                 Arguments.fromBundle(extras),
-                60000, 
-                true,
+                60000, // Timeout after 60s
+                true,  // Allow task to run in foreground
                 retryPolicy
             )
         } else null

package/android/src/main/java/com/rnsnativecall/CallMessagingService.kt

@@ -11,6 +11,7 @@ import android.app.NotificationChannel
 import android.app.PendingIntent
 import android.os.Build
 import androidx.core.app.NotificationCompat
+import androidx.core.content.ContextCompat
 import com.google.firebase.messaging.FirebaseMessagingService
 import com.google.firebase.messaging.RemoteMessage
 import com.facebook.react.HeadlessJsTaskService
@@ -39,33 +40,28 @@ class CallMessagingService : FirebaseMessagingService() {
         }
         
         if (isAppInForeground(context)) {
-            // When in foreground, we just tell JS to show the in-app UI
             CallModule.sendEventToJS("onCallReceived", data)
         } else {
-            // 1. Show the Notification IMMEDIATELY
-            // Do not wait for HeadlessJS. Android requires immediate feedback for background starts.
+            // 1. Show the Full Screen Notification IMMEDIATELY
             NativeCallManager.handleIncomingPush(context, data)
 
-            // 2. Start HeadlessJS Task to sync Javascript state (WebRTC initialization, etc.)
+            // 2. Start HeadlessJS Task as a Foreground Service
             try {
                 val headlessIntent = Intent(context, CallHeadlessTask::class.java).apply {
-                    putExtras(Bundle().apply {
-                        data.forEach { (k, v) -> putString(k, v) }
-                    })
+                    val bundle = Bundle()
+                    data.forEach { (k, v) -> bundle.putString(k, v) }
+                    putExtras(bundle)
                 }
                 
-                // Use a standard startService. If CallHeadlessTask is a HeadlessJsTaskService, 
-                // it handles its own foreground/background lifecycle.
-                context.startService(headlessIntent)
+                // CRITICAL: Must use startForegroundService for background starts on Android 8+
+                ContextCompat.startForegroundService(context, headlessIntent)
                 HeadlessJsTaskService.acquireWakeLockNow(context)
             } catch (e: Exception) { 
                 e.printStackTrace() 
             }
 
-            // 3. Backup Safety: Ensure if JS fails, the notification is eventually handled
+            // 3. Backup Safety
             val showNotificationRunnable = Runnable {
-                // If the notification was somehow canceled or not showing, re-trigger
-                // (Usually not needed if handleIncomingPush is called above, but kept for your logic)
                 if (!isAppInForeground(context)) {
                     NativeCallManager.handleIncomingPush(context, data)
                 }

package/android/src/main/java/com/rnsnativecall/NativeCallManager.kt

@@ -77,10 +77,11 @@ object NativeCallManager {
         val caller = Person.Builder()
             .setName(name)
             .setImportant(true)
+            .setIcon(context.applicationInfo.icon)
             .build()
 
         val builder = NotificationCompat.Builder(context, CALL_CHANNEL_ID)
-            .setSmallIcon(context.applicationInfo.icon)
+            //.setSmallIcon(context.applicationInfo.icon)
             .setContentTitle("Incoming $callType call")
             .setContentText(name)
             .setPriority(NotificationCompat.PRIORITY_MAX)
@@ -93,8 +94,8 @@ object NativeCallManager {
             .setStyle(
                 NotificationCompat.CallStyle.forIncomingCall(
                     caller,
-                    rejectPendingIntent,
                     answerPendingIntent
+                    rejectPendingIntent,
                 )
             )
    

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "rns-nativecall",
-    "version": "0.4.8",
+    "version": "0.5.0",
     "description": "RNS nativecall component with native Android/iOS for handling native call ui, when app is not open or open.",
     "main": "index.js",
     "types": "index.d.ts",