expo-beacon 0.6.11 → 0.6.13

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

@@ -288,9 +288,10 @@ class BeaconForegroundService : Service(), BeaconConsumer {
                 missCounters.remove(region.uniqueId)
             }
             if (wasEntered) {
-                cancelTimeout(region.uniqueId)
                 sendBeaconBroadcast(region, "exit", -1.0)
                 showEnterExitNotification(region, "exit")
+                // OS-level exit safety net — start the timeout clock.
+                scheduleTimeoutIfConfigured(region)
             }
         }
 
@@ -329,13 +330,15 @@ class BeaconForegroundService : Service(), BeaconConsumer {
                         enteredRegions.add(region.uniqueId)
                         sendBeaconBroadcast(region, "enter", beacon.distance, beacon.rssi)
                         showEnterExitNotification(region, "enter")
-                        scheduleTimeoutIfConfigured(region)
+                        // Beacon returned — cancel any running timeout timer.
+                        cancelTimeout(region.uniqueId)
                     }
                     HysteresisAction.EXIT -> {
-                        cancelTimeout(region.uniqueId)
                         enteredRegions.remove(region.uniqueId)
                         sendBeaconBroadcast(region, "exit", beacon.distance, beacon.rssi)
                         showEnterExitNotification(region, "exit")
+                        // Beacon left — start the timeout clock.
+                        scheduleTimeoutIfConfigured(region)
                     }
                     HysteresisAction.NONE -> {}
                 }
@@ -348,18 +351,14 @@ class BeaconForegroundService : Service(), BeaconConsumer {
                 missCounters[region.uniqueId] = count
 
                 if (enteredRegions.contains(region.uniqueId) && count >= EXIT_MISS_THRESHOLD) {
-                    // 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
                     exitCounters[region.uniqueId] = 0
                     sendBeaconBroadcast(region, "exit", -1.0)
                     showEnterExitNotification(region, "exit")
+                    // Beacon disappeared — start the timeout clock.
+                    scheduleTimeoutIfConfigured(region)
                 }
             }
         }
@@ -459,14 +458,10 @@ class BeaconForegroundService : Service(), BeaconConsumer {
 
     private fun scheduleTimeoutIfConfigured(region: Region) {
         val seconds = beaconTimeouts[region.uniqueId] ?: return
-        // Cancel any existing timer (shouldn't happen, but be safe)
+        // Cancel any existing timer so each exit resets the clock.
         cancelTimeout(region.uniqueId)
         val runnable = Runnable {
             timeoutRunnables.remove(region.uniqueId)
-            // 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

package/ios/ExpoBeaconModule.swift

@@ -848,9 +848,10 @@ public class ExpoBeaconModule: Module {
                         "rssi": beaconRssi
                     ])
                     postBeaconNotification(identifier: identifier, eventType: "enter")
-                    scheduleEddystoneTimeout(identifier: identifier, namespace: ns, instance: inst)
-                case .exit:
+                    // Beacon returned — cancel any running timeout timer.
                     cancelEddystoneTimeout(identifier: identifier)
+                case .exit:
+                    smoothedDistances.removeValue(forKey: identifier)
                     sendLoggedEvent("onEddystoneExit", [
                         "identifier": identifier,
                         "namespace": ns,
@@ -860,6 +861,8 @@ public class ExpoBeaconModule: Module {
                         "rssi": beaconRssi
                     ])
                     postBeaconNotification(identifier: identifier, eventType: "exit")
+                    // Beacon left — start the timeout clock.
+                    scheduleEddystoneTimeout(identifier: identifier, namespace: ns, instance: inst)
                 case .none:
                     break
                 }
@@ -1078,7 +1081,7 @@ public class ExpoBeaconModule: Module {
                 eddystoneEnterCounters[identifier] = 0
                 eddystoneExitCounters[identifier] = 0
                 eddystoneLatestSeen.removeValue(forKey: identifier)
-                // Do NOT cancel the timeout here — same reason as iBeacon miss-based exit.
+                smoothedDistances.removeValue(forKey: identifier)
 
                 let ns = paired["namespace"] as? String ?? ""
                 let inst = paired["instance"] as? String ?? ""
@@ -1091,16 +1094,17 @@ public class ExpoBeaconModule: Module {
                 ]
                 sendLoggedEvent("onEddystoneExit", params)
                 postBeaconNotification(identifier: identifier, eventType: "exit")
+                // Beacon disappeared — start the timeout clock.
+                scheduleEddystoneTimeout(identifier: identifier, namespace: ns, instance: inst)
             }
         }
     }
 
     // MARK: - Timeout timer helpers
 
-    private func scheduleBeaconTimeout(identifier: String, beacon: CLBeacon) {
-        // Cancel any existing timer for this identifier (shouldn't happen, but be safe)
-        beaconTimeoutTimers[identifier]?.cancel()
-        beaconTimeoutTimers.removeValue(forKey: identifier)
+    private func scheduleBeaconTimeout(identifier: String, beacon: CLBeacon? = nil, region: CLBeaconRegion? = nil) {
+        // Cancel any existing timer so each exit resets the clock.
+        cancelBeaconTimeout(identifier: identifier)
 
         let paired = loadPairedBeaconsRaw().first { ($0["identifier"] as? String) == identifier }
         guard let seconds = paired?["timeoutSeconds"] as? Int, seconds > 0 else { return }
@@ -1108,11 +1112,7 @@ public class ExpoBeaconModule: Module {
         let work = DispatchWorkItem { [weak self] in
             guard let self = self else { return }
             self.beaconTimeoutTimers.removeValue(forKey: identifier)
-            // 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))
+            self.sendLoggedEvent("onBeaconTimeout", self.makeBeaconEventParams(identifier: identifier, beacon: beacon, region: region))
         }
         beaconTimeoutTimers[identifier] = work
         DispatchQueue.main.asyncAfter(deadline: .now() + .seconds(seconds), execute: work)
@@ -1123,8 +1123,8 @@ public class ExpoBeaconModule: Module {
     }
 
     private func scheduleEddystoneTimeout(identifier: String, namespace: String, instance: String) {
-        eddystoneTimeoutTimers[identifier]?.cancel()
-        eddystoneTimeoutTimers.removeValue(forKey: identifier)
+        // Cancel any existing timer so each exit resets the clock.
+        cancelEddystoneTimeout(identifier: identifier)
 
         let paired = loadPairedEddystonesRaw().first { ($0["identifier"] as? String) == identifier }
         guard let seconds = paired?["timeoutSeconds"] as? Int, seconds > 0 else { return }
@@ -1132,7 +1132,6 @@ public class ExpoBeaconModule: Module {
         let work = DispatchWorkItem { [weak self] in
             guard let self = self else { return }
             self.eddystoneTimeoutTimers.removeValue(forKey: identifier)
-            // Fire unconditionally — same reason as iBeacon timeout.
             self.sendLoggedEvent("onEddystoneTimeout", [
                 "identifier": identifier,
                 "namespace": namespace,
@@ -1197,16 +1196,21 @@ public class ExpoBeaconModule: Module {
     // MARK: - Distance smoothing + enter/exit hysteresis
 
     /// Apply exponential moving average (EMA) smoothing to a raw distance reading.
-    /// Returns nil if the reading is a jump outlier (raw differs from smoothed by > DISTANCE_JUMP_FACTOR).
+    /// If the reading is a large jump (> DISTANCE_JUMP_FACTOR), resets the EMA to the new
+    /// value instead of rejecting it — this ensures distance events keep flowing when the
+    /// user moves away from a beacon, rather than freezing because the EMA is stuck at the
+    /// old close-range value and every new far-range reading is rejected.
     private func smoothDistance(identifier: String, rawDistance: Double) -> Double? {
         guard let prev = smoothedDistances[identifier] else {
             smoothedDistances[identifier] = rawDistance
             return rawDistance
         }
-        // Jump guard: if the raw value is wildly different, treat as outlier
+        // Jump guard: if the raw value is wildly different, reset EMA to the new reading
+        // so the hysteresis pipeline keeps receiving data and can fire the exit event.
         let ratio = prev > 0.001 ? rawDistance / prev : rawDistance
         if ratio > Self.DISTANCE_JUMP_FACTOR || (ratio > 0 && ratio < 1.0 / Self.DISTANCE_JUMP_FACTOR) {
-            return nil
+            smoothedDistances[identifier] = rawDistance
+            return rawDistance
         }
         let smoothed = Self.DISTANCE_EMA_ALPHA * rawDistance + (1 - Self.DISTANCE_EMA_ALPHA) * prev
         smoothedDistances[identifier] = smoothed
@@ -1354,11 +1358,14 @@ public class ExpoBeaconModule: Module {
                 case .enter:
                     sendLoggedEvent("onBeaconEnter", makeBeaconEventParams(identifier: identifier, beacon: beacon, event: "enter"))
                     postBeaconNotification(identifier: identifier, eventType: "enter")
-                    scheduleBeaconTimeout(identifier: identifier, beacon: beacon)
-                case .exit:
+                    // Beacon returned — cancel any running timeout timer.
                     cancelBeaconTimeout(identifier: identifier)
+                case .exit:
+                    smoothedDistances.removeValue(forKey: identifier)
                     sendLoggedEvent("onBeaconExit", makeBeaconEventParams(identifier: identifier, beacon: beacon, event: "exit"))
                     postBeaconNotification(identifier: identifier, eventType: "exit")
+                    // Beacon left — start the timeout clock.
+                    scheduleBeaconTimeout(identifier: identifier, beacon: beacon)
                 case .none:
                     break
                 }
@@ -1379,16 +1386,14 @@ public class ExpoBeaconModule: Module {
                     missCounters[identifier] = 0
                     enterCounters[identifier] = 0
                     exitCounters[identifier] = 0
-                    // 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.
+                    smoothedDistances.removeValue(forKey: identifier)
 
                     // Look up region info for the exit event payload
                     let region = monitoredRegions.first { $0.identifier == identifier }
                     sendLoggedEvent("onBeaconExit", makeBeaconEventParams(identifier: identifier, region: region, event: "exit"))
                     postBeaconNotification(identifier: identifier, eventType: "exit")
+                    // Beacon disappeared — start the timeout clock.
+                    scheduleBeaconTimeout(identifier: identifier, region: region)
                 }
             }
             return
@@ -1429,10 +1434,12 @@ public class ExpoBeaconModule: Module {
         enterCounters.removeValue(forKey: identifier)
         exitCounters.removeValue(forKey: identifier)
         missCounters.removeValue(forKey: identifier)
+        smoothedDistances.removeValue(forKey: identifier)
         if wasEntered {
-            cancelBeaconTimeout(identifier: identifier)
             sendLoggedEvent("onBeaconExit", makeBeaconEventParams(identifier: identifier, region: beaconRegion, event: "exit"))
             postBeaconNotification(identifier: identifier, eventType: "exit")
+            // OS-level exit safety net — start the timeout clock.
+            scheduleBeaconTimeout(identifier: identifier, region: beaconRegion)
         }
     }
 

package/package.json

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