expo-beacon 0.5.3 → 0.5.5

package/ios/ExpoBeaconModule.swift

@@ -251,10 +251,12 @@ public class ExpoBeaconModule: Module {
 
             var eddystones = self.loadPairedEddystonesRaw()
             eddystones.removeAll { ($0["identifier"] as? String) == identifier }
+            // Normalize hex to lowercase — parseEddystoneFrame produces lowercase,
+            // so stored values must match for monitoring comparisons.
             eddystones.append([
                 "identifier": identifier,
-                "namespace": namespace,
-                "instance": instance
+                "namespace": namespace.lowercased(),
+                "instance": instance.lowercased()
             ])
             self.defaults.set(eddystones, forKey: PAIRED_EDDYSTONES_KEY)
             self.cachedPairedEddystones = nil
@@ -307,11 +309,17 @@ public class ExpoBeaconModule: Module {
                 self.defaults.removeObject(forKey: EXIT_DISTANCE_KEY)
             }
             self.defaults.set(true, forKey: IS_MONITORING_KEY)
-            self.requestLocationPermission(requireAlways: true) { granted in
+            self.requestLocationPermission { granted in
                 guard granted else {
-                    promise.reject("PERMISSION_DENIED", "Always location permission required for background monitoring")
+                    promise.reject("PERMISSION_DENIED", "Location permission required for monitoring")
                     return
                 }
+                // Request Always authorization non-blockingly for background support.
+                // On iOS 13+ requestAlwaysAuthorization() from WhenInUse may be a
+                // no-op if the user already made their choice — don't block on it.
+                if self.locationManager.authorizationStatus != .authorizedAlways {
+                    self.locationManager.requestAlwaysAuthorization()
+                }
                 self.requestNotificationPermission()
                 self.startRegionMonitoring()
                 promise.resolve(nil)
@@ -662,27 +670,14 @@ public class ExpoBeaconModule: Module {
             guard let identifier = paired["identifier"] as? String,
                   let pns = paired["namespace"] as? String,
                   let pinst = paired["instance"] as? String,
-                  pns == ns && pinst == inst else { continue }
+                  pns.lowercased() == ns && pinst.lowercased() == inst else { continue }
 
             eddystoneLatestSeen[identifier] = Date()
             eddystoneMissCounters[identifier] = 0
 
-            // Throttle distance events
-            let now = Date()
-            if let lastEmit = eddystoneLastDistanceEmit[identifier],
-               now.timeIntervalSince(lastEmit) < DISTANCE_EVENT_THROTTLE_INTERVAL {
-                break
-            }
-            eddystoneLastDistanceEmit[identifier] = now
-
-            sendEvent("onEddystoneDistance", [
-                "identifier": identifier,
-                "namespace": ns,
-                "instance": inst,
-                "distance": distance
-            ])
-
-            // Distance-driven enter/exit with hysteresis
+            // Distance-driven enter/exit with hysteresis — evaluated on every
+            // BLE callback (not throttled) so the hysteresis counters advance
+            // reliably regardless of advertisement rate.
             let maxDist = self.defaults.object(forKey: MAX_DISTANCE_KEY) as? Double
             let exitDist = self.defaults.object(forKey: EXIT_DISTANCE_KEY) as? Double
             let action = evaluateDistanceHysteresis(
@@ -716,6 +711,22 @@ public class ExpoBeaconModule: Module {
             case .none:
                 break
             }
+
+            // Throttle distance events — enter/exit above is evaluated on every
+            // callback, but distance events are rate-limited to avoid flooding JS.
+            let now = Date()
+            if let lastEmit = eddystoneLastDistanceEmit[identifier],
+               now.timeIntervalSince(lastEmit) < DISTANCE_EVENT_THROTTLE_INTERVAL {
+                break
+            }
+            eddystoneLastDistanceEmit[identifier] = now
+
+            sendEvent("onEddystoneDistance", [
+                "identifier": identifier,
+                "namespace": ns,
+                "instance": inst,
+                "distance": distance
+            ])
             break
         }
     }
@@ -1213,9 +1224,12 @@ private class BluetoothDelegate: NSObject, CBCentralManagerDelegate {
         case .poweredOn:
             module?.ensureBleScanRunning()
         case .unauthorized:
+            print("[ExpoBeacon] Bluetooth authorization denied — Eddystone scanning/monitoring unavailable. " +
+                  "Ensure NSBluetoothAlwaysUsageDescription is set in Info.plist.")
             module?.eddystoneScanPromise?.reject("BLUETOOTH_UNAUTHORIZED", "Bluetooth permission denied")
             module?.eddystoneScanPromise = nil
         case .poweredOff:
+            print("[ExpoBeacon] Bluetooth is powered off — Eddystone scanning/monitoring unavailable.")
             module?.eddystoneScanPromise?.reject("BLUETOOTH_OFF", "Bluetooth is powered off")
             module?.eddystoneScanPromise = nil
         default:

package/package.json

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