expo-beacon 0.7.1 → 0.7.2

package/android/src/main/AndroidManifest.xml

@@ -48,13 +48,17 @@
             android:foregroundServiceType="connectedDevice"
             android:exported="false" />
 
-        <!-- Restart monitoring after boot -->
+        <!-- Restart monitoring after boot.
+             android:exported="true" is required for the system to deliver BOOT_COMPLETED.
+             No android:permission — the incorrect RECEIVE_BOOT_COMPLETED attribute was
+             restricting incoming broadcasts instead of restricting who can receive them. -->
         <receiver
             android:name="expo.modules.beacon.BootReceiver"
-            android:exported="true"
-            android:permission="android.permission.RECEIVE_BOOT_COMPLETED">
+            android:exported="true">
             <intent-filter>
                 <action android:name="android.intent.action.BOOT_COMPLETED" />
+                <!-- Self-scheduled retry when BT isn't ready at boot -->
+                <action android:name="expo.modules.beacon.ACTION_RETRY_MONITORING" />
             </intent-filter>
         </receiver>
     </application>

package/android/src/main/java/expo/modules/beacon/BeaconForegroundService.kt

@@ -112,10 +112,17 @@ class BeaconForegroundService : Service(), BeaconConsumer {
                 startForeground(FOREGROUND_NOTIF_ID, buildForegroundNotification())
             }
         } catch (e: Exception) {
-            // SecurityException on Android 14+ if BT permissions missing,
-            // or other platform-specific issues. Stop gracefully instead of crashing.
-            Log.e(TAG, "startForeground failed — stopping service", e)
-            sendErrorBroadcast(null, "SERVICE_START_FAILED", "startForeground failed — stopping service: ${e.message}")
+            // SecurityException on Android 14+ if BT runtime permissions weren't yet granted,
+            // or ForegroundServiceStartNotAllowedException on Android 12+ / Android 17 beta
+            // if the service start window was missed (e.g. BT not yet initialized at boot).
+            val retryCount = intent?.getIntExtra(EXTRA_RETRY_COUNT, 0) ?: 0
+            Log.e(TAG, "startForeground failed (retry=$retryCount) — stopping service", e)
+            sendErrorBroadcast(null, "SERVICE_START_FAILED", "startForeground failed (retry=$retryCount): ${e.message}")
+            // Schedule a retry so monitoring can recover without user interaction, capped to
+            // avoid infinite crash loops. Only retry if monitoring should still be active.
+            if (retryCount < MAX_STARTFOREGROUND_RETRIES && isMonitoringActive(this)) {
+                scheduleServiceRetry(retryCount + 1)
+            }
             stopSelf()
             return START_NOT_STICKY
         }
@@ -129,6 +136,43 @@ class BeaconForegroundService : Service(), BeaconConsumer {
         return START_STICKY
     }
 
+    /**
+     * Schedules a one-shot alarm to restart this service via startForegroundService().
+     * Used when startForeground() fails transiently (e.g. BT not yet ready at boot).
+     */
+    private fun scheduleServiceRetry(retryCount: Int) {
+        val alarmManager = getSystemService(AlarmManager::class.java) ?: return
+        val retryIntent = Intent(this, BeaconForegroundService::class.java)
+            .putExtra(EXTRA_RETRY_COUNT, retryCount)
+        val pendingIntent = PendingIntent.getService(
+            this,
+            RETRY_SERVICE_REQUEST_CODE,
+            retryIntent,
+            PendingIntent.FLAG_UPDATE_CURRENT or PendingIntent.FLAG_IMMUTABLE
+        )
+        alarmManager.setExactAndAllowWhileIdle(
+            AlarmManager.ELAPSED_REALTIME_WAKEUP,
+            SystemClock.elapsedRealtime() + RETRY_DELAY_MS,
+            pendingIntent
+        )
+        Log.w(TAG, "startForeground retry $retryCount scheduled in ${RETRY_DELAY_MS}ms")
+    }
+
+    /**
+     * Called by the system on Android 14+ (API 34) if a foreground service with a time-limited
+     * type exceeds its allowed duration. connectedDevice has no documented time limit as of
+     * Android 17, but this override ensures we handle a future limit gracefully rather than
+     * receiving an ANR.
+     */
+    @Suppress("NewApi")
+    override fun onTimeout(startId: Int, fgsType: Int) {
+        Log.w(TAG, "BeaconForegroundService.onTimeout(startId=$startId, fgsType=$fgsType) — scheduling restart")
+        if (isMonitoringActive(this)) {
+            scheduleServiceRetry(0)
+        }
+        stopSelf()
+    }
+
     override fun onBeaconServiceConnect() {
         serviceConnected = true
         // Read max distance, exit distance, and min RSSI from options prefs
@@ -736,6 +780,10 @@ class BeaconForegroundService : Service(), BeaconConsumer {
         const val DISTANCE_JUMP_FACTOR = 5.0
 
         private const val PREF_IS_MONITORING = "expo.beacon.is_monitoring"
+        private const val EXTRA_RETRY_COUNT = "retryCount"
+        private const val MAX_STARTFOREGROUND_RETRIES = 3
+        private const val RETRY_DELAY_MS = 10_000L
+        private const val RETRY_SERVICE_REQUEST_CODE = 0x42454143 // "BEAC"
         @Volatile private var activeService: BeaconForegroundService? = null
 
         fun start(context: Context) {

package/android/src/main/java/expo/modules/beacon/BootReceiver.kt

@@ -1,17 +1,89 @@
 package expo.modules.beacon
 
+import android.app.ActivityManager
+import android.app.AlarmManager
+import android.app.PendingIntent
 import android.content.BroadcastReceiver
 import android.content.Context
 import android.content.Intent
+import android.os.Build
+import android.os.SystemClock
+import android.util.Log
+
+private const val ACTION_RETRY_MONITORING = "expo.modules.beacon.ACTION_RETRY_MONITORING"
+private const val RETRY_DELAY_MS = 10_000L
+private const val RETRY_REQUEST_CODE = 0x424F4F54 // "BOOT"
 
 /**
  * Restarts beacon monitoring after device reboot if it was active before shutdown.
+ *
+ * Also handles ACTION_RETRY_MONITORING — a self-scheduled alarm used to retry the
+ * service start if Bluetooth is not yet fully initialized when BOOT_COMPLETED fires
+ * (observed on Android 17 beta where startForegroundService() throws SecurityException
+ * or ForegroundServiceStartNotAllowedException shortly after boot).
  */
 class BootReceiver : BroadcastReceiver() {
     override fun onReceive(context: Context, intent: Intent) {
-        if (intent.action == Intent.ACTION_BOOT_COMPLETED) {
-            if (BeaconForegroundService.isMonitoringActive(context)) {
-                BeaconForegroundService.start(context)
+        when (intent.action) {
+            Intent.ACTION_BOOT_COMPLETED -> {
+                logMemoryKillDiagnostics(context)
+                if (BeaconForegroundService.isMonitoringActive(context)) {
+                    tryStartService(context)
+                }
+            }
+            ACTION_RETRY_MONITORING -> {
+                if (BeaconForegroundService.isMonitoringActive(context)) {
+                    tryStartService(context)
+                }
+            }
+        }
+    }
+
+    private fun tryStartService(context: Context) {
+        try {
+            BeaconForegroundService.start(context)
+            Log.d(TAG, "BootReceiver: BeaconForegroundService started successfully")
+        } catch (e: SecurityException) {
+            // ForegroundServiceStartNotAllowedException extends SecurityException.
+            // Can occur on Android 17 beta if Bluetooth is not yet fully initialized at boot.
+            Log.e(TAG, "BootReceiver: Failed to start service (SecurityException) — retrying in ${RETRY_DELAY_MS}ms", e)
+            scheduleRetry(context)
+        } catch (e: Exception) {
+            Log.e(TAG, "BootReceiver: Failed to start service — retrying in ${RETRY_DELAY_MS}ms", e)
+            scheduleRetry(context)
+        }
+    }
+
+    private fun scheduleRetry(context: Context) {
+        val alarmManager = context.getSystemService(AlarmManager::class.java) ?: return
+        val retryIntent = Intent(context, BootReceiver::class.java).apply {
+            action = ACTION_RETRY_MONITORING
+        }
+        val pendingIntent = PendingIntent.getBroadcast(
+            context,
+            RETRY_REQUEST_CODE,
+            retryIntent,
+            PendingIntent.FLAG_UPDATE_CURRENT or PendingIntent.FLAG_IMMUTABLE
+        )
+        alarmManager.setExactAndAllowWhileIdle(
+            AlarmManager.ELAPSED_REALTIME_WAKEUP,
+            SystemClock.elapsedRealtime() + RETRY_DELAY_MS,
+            pendingIntent
+        )
+    }
+
+    /**
+     * Logs if the previous process was killed by Android 17's app memory limits
+     * (ApplicationExitInfo.description contains "MemoryLimiter") so the cause is
+     * visible in Logcat when the service restarts after being killed.
+     */
+    private fun logMemoryKillDiagnostics(context: Context) {
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R) {
+            val am = context.getSystemService(ActivityManager::class.java)
+            am?.getHistoricalProcessExitReasons(null, 0, 5)?.forEach { info ->
+                if (info.description?.contains("MemoryLimiter") == true) {
+                    Log.w(TAG, "BootReceiver: previous process killed by Android 17 memory limits: ${info.description}")
+                }
             }
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-beacon",
-  "version": "0.7.1",
+  "version": "0.7.2",
   "description": "Expo module for scanning, pairing, and monitoring iBeacons on Android and iOS",
   "main": "build/index.js",
   "types": "build/index.d.ts",