expo-beacon 0.6.9 → 0.6.11

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

@@ -348,7 +348,12 @@ class BeaconForegroundService : Service(), BeaconConsumer {
                 missCounters[region.uniqueId] = count
 
                 if (enteredRegions.contains(region.uniqueId) && count >= EXIT_MISS_THRESHOLD) {
-                    cancelTimeout(region.uniqueId)
+                    // Do NOT cancel the timeout here. A miss-based exit is triggered by BLE
+                    // scan gaps (unreliable signal disappearance), not a confirmed physical
+                    // departure. Cancelling the timeout here would prevent it from ever firing
+                    // when the configured timeout (e.g. 25 s) exceeds the miss window (~21 s).
+                    // The timeout runnable fires unconditionally; distance-based exits still
+                    // call cancelTimeout() reliably when the beacon moves out of range.
                     enteredRegions.remove(region.uniqueId)
                     missCounters[region.uniqueId] = 0
                     enterCounters[region.uniqueId] = 0
@@ -458,10 +463,11 @@ class BeaconForegroundService : Service(), BeaconConsumer {
         cancelTimeout(region.uniqueId)
         val runnable = Runnable {
             timeoutRunnables.remove(region.uniqueId)
-            // Only fire if the beacon is still in range
-            if (enteredRegions.contains(region.uniqueId)) {
-                sendBeaconBroadcast(region, "timeout", -1.0)
-            }
+            // Fire unconditionally. A miss-based exit may have cleared enteredRegions before
+            // the timer elapsed (BLE gaps can cause false exits at ~21 s), but the beacon
+            // may still be physically present. Distance-based exits call cancelTimeout() so
+            // this runnable is never queued when the beacon has genuinely moved away.
+            sendBeaconBroadcast(region, "timeout", -1.0)
         }
         timeoutRunnables[region.uniqueId] = runnable
         timeoutHandler.postDelayed(runnable, seconds * 1000L)

package/ios/ExpoBeaconModule.swift

@@ -1078,7 +1078,7 @@ public class ExpoBeaconModule: Module {
                 eddystoneEnterCounters[identifier] = 0
                 eddystoneExitCounters[identifier] = 0
                 eddystoneLatestSeen.removeValue(forKey: identifier)
-                cancelEddystoneTimeout(identifier: identifier)
+                // Do NOT cancel the timeout here — same reason as iBeacon miss-based exit.
 
                 let ns = paired["namespace"] as? String ?? ""
                 let inst = paired["instance"] as? String ?? ""
@@ -1108,8 +1108,10 @@ public class ExpoBeaconModule: Module {
         let work = DispatchWorkItem { [weak self] in
             guard let self = self else { return }
             self.beaconTimeoutTimers.removeValue(forKey: identifier)
-            // Only fire if the beacon is still in range
-            guard self.enteredRegions.contains(identifier) else { return }
+            // Fire unconditionally. A miss-based exit may have cleared enteredRegions before
+            // the timer elapsed (ranging gaps can cause false exits), but the beacon may still
+            // be physically present. Distance-based exits call cancelBeaconTimeout() so this
+            // work item is cancelled before it runs on genuine out-of-range departures.
             self.sendLoggedEvent("onBeaconTimeout", self.makeBeaconEventParams(identifier: identifier, beacon: beacon))
         }
         beaconTimeoutTimers[identifier] = work
@@ -1130,7 +1132,7 @@ public class ExpoBeaconModule: Module {
         let work = DispatchWorkItem { [weak self] in
             guard let self = self else { return }
             self.eddystoneTimeoutTimers.removeValue(forKey: identifier)
-            guard self.eddystoneEnteredRegions.contains(identifier) else { return }
+            // Fire unconditionally — same reason as iBeacon timeout.
             self.sendLoggedEvent("onEddystoneTimeout", [
                 "identifier": identifier,
                 "namespace": namespace,
@@ -1377,7 +1379,11 @@ public class ExpoBeaconModule: Module {
                     missCounters[identifier] = 0
                     enterCounters[identifier] = 0
                     exitCounters[identifier] = 0
-                    cancelBeaconTimeout(identifier: identifier)
+                    // Do NOT cancel the timeout here. A miss-based exit is triggered by
+                    // ranging gaps (e.g. accuracy == -1), not a confirmed physical departure.
+                    // Cancelling here prevents the timeout from firing when it is longer than
+                    // the miss window. Distance-based exits call cancelBeaconTimeout() so the
+                    // timer is still cancelled on genuine out-of-range events.
 
                     // Look up region info for the exit event payload
                     let region = monitoredRegions.first { $0.identifier == identifier }

package/package.json

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