fix(android): fixes Java IndexOutOfBoundsException exception

Author: patlux

apps/expo-example-app/app.json

@@ -13,12 +13,13 @@
     "updates": {
       "fallbackToCacheTimeout": 0
     },
-    "assetBundlePatterns": [
-      "**/*"
-    ],
+    "assetBundlePatterns": ["**/*"],
     "ios": {
       "supportsTablet": true,
-      "bundleIdentifier": "com.patwoz.expoexampleapp"
+      "bundleIdentifier": "com.patwoz.expoexampleapp",
+      "infoPlist": {
+        "NSBluetoothAlwaysUsageDescription": "Your reason to use bluetooth"
+      }
     },
     "android": {
       "adaptiveIcon": {

apps/expo-example-app/ios/ExpoExampleApp.xcodeproj/project.pbxproj

@@ -3,7 +3,7 @@
 	archiveVersion = 1;
 	classes = {
 	};
-	objectVersion = 46;
+	objectVersion = 54;
 	objects = {
 
 /* Begin PBXBuildFile section */
@@ -27,7 +27,7 @@
 		13B07FB71A68108700A75B9A /* main.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; name = main.m; path = ExpoExampleApp/main.m; sourceTree = "<group>"; };
 		32B9975796F647FABCCD77EB /* noop-file.swift */ = {isa = PBXFileReference; explicitFileType = undefined; fileEncoding = 4; includeInIndex = 0; lastKnownFileType = sourcecode.swift; name = "noop-file.swift"; path = "ExpoExampleApp/noop-file.swift"; sourceTree = "<group>"; };
 		58EEBF8E8E6FB1BC6CAF49B5 /* libPods-ExpoExampleApp.a */ = {isa = PBXFileReference; explicitFileType = archive.ar; includeInIndex = 0; path = "libPods-ExpoExampleApp.a"; sourceTree = BUILT_PRODUCTS_DIR; };
-		6451BEBE8E39D32B566384D5 /* PrivacyInfo.xcprivacy */ = {isa = PBXFileReference; includeInIndex = 1; name = PrivacyInfo.xcprivacy; path = ExpoExampleApp/PrivacyInfo.xcprivacy; sourceTree = "<group>"; };
+		6451BEBE8E39D32B566384D5 /* PrivacyInfo.xcprivacy */ = {isa = PBXFileReference; includeInIndex = 1; lastKnownFileType = text.xml; name = PrivacyInfo.xcprivacy; path = ExpoExampleApp/PrivacyInfo.xcprivacy; sourceTree = "<group>"; };
 		6C2E3173556A471DD304B334 /* Pods-ExpoExampleApp.debug.xcconfig */ = {isa = PBXFileReference; includeInIndex = 1; lastKnownFileType = text.xcconfig; name = "Pods-ExpoExampleApp.debug.xcconfig"; path = "Target Support Files/Pods-ExpoExampleApp/Pods-ExpoExampleApp.debug.xcconfig"; sourceTree = "<group>"; };
 		7A4D352CD337FB3A3BF06240 /* Pods-ExpoExampleApp.release.xcconfig */ = {isa = PBXFileReference; includeInIndex = 1; lastKnownFileType = text.xcconfig; name = "Pods-ExpoExampleApp.release.xcconfig"; path = "Target Support Files/Pods-ExpoExampleApp/Pods-ExpoExampleApp.release.xcconfig"; sourceTree = "<group>"; };
 		AA286B85B6C04FC6940260E9 /* SplashScreen.storyboard */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = file.storyboard; name = SplashScreen.storyboard; path = ExpoExampleApp/SplashScreen.storyboard; sourceTree = "<group>"; };
@@ -341,14 +341,18 @@
 				CLANG_ENABLE_MODULES = YES;
 				CODE_SIGN_ENTITLEMENTS = ExpoExampleApp/ExpoExampleApp.entitlements;
 				CURRENT_PROJECT_VERSION = 1;
+				DEVELOPMENT_TEAM = UB663H59T6;
 				ENABLE_BITCODE = NO;
 				GCC_PREPROCESSOR_DEFINITIONS = (
 					"$(inherited)",
 					"FB_SONARKIT_ENABLED=1",
 				);
 				INFOPLIST_FILE = ExpoExampleApp/Info.plist;
 				IPHONEOS_DEPLOYMENT_TARGET = 15.1;
-				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks";
+				LD_RUNPATH_SEARCH_PATHS = (
+					"$(inherited)",
+					"@executable_path/Frameworks",
+				);
 				MARKETING_VERSION = 1.0;
 				OTHER_LDFLAGS = (
 					"$(inherited)",
@@ -374,9 +378,13 @@
 				CLANG_ENABLE_MODULES = YES;
 				CODE_SIGN_ENTITLEMENTS = ExpoExampleApp/ExpoExampleApp.entitlements;
 				CURRENT_PROJECT_VERSION = 1;
+				DEVELOPMENT_TEAM = UB663H59T6;
 				INFOPLIST_FILE = ExpoExampleApp/Info.plist;
 				IPHONEOS_DEPLOYMENT_TARGET = 15.1;
-				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks";
+				LD_RUNPATH_SEARCH_PATHS = (
+					"$(inherited)",
+					"@executable_path/Frameworks",
+				);
 				MARKETING_VERSION = 1.0;
 				OTHER_LDFLAGS = (
 					"$(inherited)",
@@ -441,14 +449,14 @@
 				GCC_WARN_UNUSED_FUNCTION = YES;
 				GCC_WARN_UNUSED_VARIABLE = YES;
 				IPHONEOS_DEPLOYMENT_TARGET = 15.1;
-				LD_RUNPATH_SEARCH_PATHS = "/usr/lib/swift $(inherited)";
+				LD_RUNPATH_SEARCH_PATHS = (
+					/usr/lib/swift,
+					"$(inherited)",
+				);
 				LIBRARY_SEARCH_PATHS = "$(SDKROOT)/usr/lib/swift\"$(inherited)\"";
 				MTL_ENABLE_DEBUG_INFO = YES;
 				ONLY_ACTIVE_ARCH = YES;
-				OTHER_LDFLAGS = (
-					"$(inherited)",
-					" ",
-				);
+				OTHER_LDFLAGS = "$(inherited)  ";
 				REACT_NATIVE_PATH = "${PODS_ROOT}/../../../../node_modules/react-native";
 				SDKROOT = iphoneos;
 				SWIFT_ACTIVE_COMPILATION_CONDITIONS = "$(inherited) DEBUG";
@@ -497,13 +505,13 @@
 				GCC_WARN_UNUSED_FUNCTION = YES;
 				GCC_WARN_UNUSED_VARIABLE = YES;
 				IPHONEOS_DEPLOYMENT_TARGET = 15.1;
-				LD_RUNPATH_SEARCH_PATHS = "/usr/lib/swift $(inherited)";
-				LIBRARY_SEARCH_PATHS = "$(SDKROOT)/usr/lib/swift\"$(inherited)\"";
-				MTL_ENABLE_DEBUG_INFO = NO;
-				OTHER_LDFLAGS = (
+				LD_RUNPATH_SEARCH_PATHS = (
+					/usr/lib/swift,
 					"$(inherited)",
-					" ",
 				);
+				LIBRARY_SEARCH_PATHS = "$(SDKROOT)/usr/lib/swift\"$(inherited)\"";
+				MTL_ENABLE_DEBUG_INFO = NO;
+				OTHER_LDFLAGS = "$(inherited)  ";
 				REACT_NATIVE_PATH = "${PODS_ROOT}/../../../../node_modules/react-native";
 				SDKROOT = iphoneos;
 				USE_HERMES = true;

apps/expo-example-app/ios/Podfile.lock

@@ -1,4 +1,26 @@
 PODS:
+  - BluetoothStateManager (2.0.1):
+    - DoubleConversion
+    - glog
+    - hermes-engine
+    - NitroModules
+    - RCT-Folly (= 2024.11.18.00)
+    - RCTRequired
+    - RCTTypeSafety
+    - React-Core
+    - React-debug
+    - React-Fabric
+    - React-featureflags
+    - React-graphics
+    - React-ImageManager
+    - React-NativeModulesApple
+    - React-RCTFabric
+    - React-rendererdebug
+    - React-utils
+    - ReactCodegen
+    - ReactCommon/turbomodule/bridging
+    - ReactCommon/turbomodule/core
+    - Yoga
   - boost (1.84.0)
   - DoubleConversion (1.1.6)
   - EXConstants (17.0.8):
@@ -44,6 +66,29 @@ PODS:
   - hermes-engine (0.77.0):
     - hermes-engine/Pre-built (= 0.77.0)
   - hermes-engine/Pre-built (0.77.0)
+  - NitroModules (0.25.2):
+    - DoubleConversion
+    - glog
+    - hermes-engine
+    - RCT-Folly (= 2024.11.18.00)
+    - RCTRequired
+    - RCTTypeSafety
+    - React-callinvoker
+    - React-Core
+    - React-debug
+    - React-Fabric
+    - React-featureflags
+    - React-graphics
+    - React-ImageManager
+    - React-jsi
+    - React-NativeModulesApple
+    - React-RCTFabric
+    - React-rendererdebug
+    - React-utils
+    - ReactCodegen
+    - ReactCommon/turbomodule/bridging
+    - ReactCommon/turbomodule/core
+    - Yoga
   - RCT-Folly (2024.11.18.00):
     - boost
     - DoubleConversion
@@ -1539,6 +1584,7 @@ PODS:
   - Yoga (0.0.0)
 
 DEPENDENCIES:
+  - BluetoothStateManager (from `../../../node_modules/react-native-bluetooth-state-manager`)
   - boost (from `../../../node_modules/react-native/third-party-podspecs/boost.podspec`)
   - DoubleConversion (from `../../../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec`)
   - EXConstants (from `../../../node_modules/expo-constants/ios`)
@@ -1553,6 +1599,7 @@ DEPENDENCIES:
   - fmt (from `../../../node_modules/react-native/third-party-podspecs/fmt.podspec`)
   - glog (from `../../../node_modules/react-native/third-party-podspecs/glog.podspec`)
   - hermes-engine (from `../../../node_modules/react-native/sdks/hermes-engine/hermes-engine.podspec`)
+  - NitroModules (from `../../../node_modules/react-native-nitro-modules`)
   - RCT-Folly (from `../../../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec`)
   - RCT-Folly/Fabric (from `../../../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec`)
   - RCTDeprecation (from `../../../node_modules/react-native/ReactApple/Libraries/RCTFoundation/RCTDeprecation`)
@@ -1620,6 +1667,8 @@ SPEC REPOS:
     - SocketRocket
 
 EXTERNAL SOURCES:
+  BluetoothStateManager:
+    :path: "../../../node_modules/react-native-bluetooth-state-manager"
   boost:
     :podspec: "../../../node_modules/react-native/third-party-podspecs/boost.podspec"
   DoubleConversion:
@@ -1649,6 +1698,8 @@ EXTERNAL SOURCES:
   hermes-engine:
     :podspec: "../../../node_modules/react-native/sdks/hermes-engine/hermes-engine.podspec"
     :tag: hermes-2024-11-25-RNv0.77.0-d4f25d534ab744866448b36ca3bf3d97c08e638c
+  NitroModules:
+    :path: "../../../node_modules/react-native-nitro-modules"
   RCT-Folly:
     :podspec: "../../../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec"
   RCTDeprecation:
@@ -1769,6 +1820,7 @@ EXTERNAL SOURCES:
     :path: "../../../node_modules/react-native/ReactCommon/yoga"
 
 SPEC CHECKSUMS:
+  BluetoothStateManager: 598d565ceff2ad482220799c1c82b980a2ddae17
   boost: 7e761d76ca2ce687f7cc98e698152abd03a18f90
   DoubleConversion: cb417026b2400c8f53ae97020b2be961b59470cb
   EXConstants: a1f35b9aabbb3c6791f8e67722579b1ffcdd3f18
@@ -1783,6 +1835,7 @@ SPEC CHECKSUMS:
   fmt: a40bb5bd0294ea969aaaba240a927bd33d878cdd
   glog: eb93e2f488219332457c3c4eafd2738ddc7e80b8
   hermes-engine: 1f783c3d53940aed0d2c84586f0b7a85ab7827ef
+  NitroModules: 584c0d7f68712f46353a8ab687d68bf1604f6c13
   RCT-Folly: 36fe2295e44b10d831836cc0d1daec5f8abcf809
   RCTDeprecation: f5c19ebdb8804b53ed029123eb69914356192fc8
   RCTRequired: 6ae6cebe470486e0e0ce89c1c0eabb998e7c51f4

apps/expo-example-app/package.json

@@ -9,11 +9,15 @@
   "dependencies": {
     "expo": "~52.0.46",
     "react": "18.3.1",
-    "react-native": "0.76.9"
+    "react-native": "0.76.9",
+    "react-native-nitro-modules": "*",
+    "react-native-bluetooth-state-manager": "*"
   },
   "prettier": {
     "semi": false,
     "singleQuote": true,
     "trailingComma": "all"
-  }
+  },
+  "devDependencies": {}
 }
+

apps/expo-example-app/src/app/App.tsx

@@ -1,6 +1,11 @@
-import { SafeAreaView, StatusBar, Text } from 'react-native'
+import { useState } from 'react'
+import { Pressable, SafeAreaView, StatusBar, Text } from 'react-native'
+import { useBluetoothState } from 'react-native-bluetooth-state-manager'
 
 export const App = () => {
+  const [items, setItems] = useState<number[]>([...Array(5)].map((_, i) => i))
+  console.log({ items })
+
   return (
     <>
       <StatusBar barStyle="dark-content" />
@@ -12,9 +17,43 @@ export const App = () => {
         }}
       >
         <Text>Example Expo App</Text>
+        {items.map((_, index) => {
+          return (
+            <Pressable
+              key={index}
+              onPress={() => {
+                setItems((prev) => {
+                  const next = [
+                    ...prev.slice(0, index),
+                    ...prev.slice(index + 1),
+                  ]
+                  return next
+                })
+              }}
+            >
+              <Text>{index}.</Text>
+              <Listener />
+            </Pressable>
+          )
+        })}
+        <Pressable
+          onPress={() => {
+            setItems((prev) => {
+              return [...prev, prev.length]
+            })
+          }}
+        >
+          <Text>Add</Text>
+        </Pressable>
       </SafeAreaView>
     </>
   )
 }
 
+const Listener = () => {
+  const bleState = useBluetoothState()
+  console.log({ bleState })
+  return <Text>{bleState}</Text>
+}
+
 export default App

bun.lock

@@ -12,6 +12,7 @@
         "react": "~18.3.1",
         "react-dom": "~18.3.1",
         "react-native": "0.76.3",
+        "react-native-nitro-modules": "^0.25.2",
         "react-native-svg": "~15.8.0",
         "react-native-svg-transformer": "~1.5.0",
         "react-native-web": "~0.19.13",
@@ -71,6 +72,8 @@
         "expo": "~52.0.46",
         "react": "18.3.1",
         "react-native": "0.76.9",
+        "react-native-bluetooth-state-manager": "*",
+        "react-native-nitro-modules": "*",
       },
     },
     "packages/react-native-bluetooth-state-manager": {

package.json

@@ -41,6 +41,7 @@
     "react": "~18.3.1",
     "react-dom": "~18.3.1",
     "react-native": "0.76.3",
+    "react-native-nitro-modules": "^0.25.2",
     "react-native-svg": "~15.8.0",
     "react-native-svg-transformer": "~1.5.0",
     "react-native-web": "~0.19.13"

packages/react-native-bluetooth-state-manager/android/src/main/java/com/margelo/nitro/bluetoothstatemanager/BluetoothStateManager.kt

@@ -13,6 +13,7 @@ import com.facebook.react.bridge.BaseActivityEventListener
 import com.facebook.react.bridge.ReactApplicationContext
 import com.margelo.nitro.NitroModules
 import com.margelo.nitro.core.Promise
+import java.util.UUID
 
 class BluetoothStateManager : HybridBluetoothStateManagerSpec() {
   override val memorySize: Long
@@ -36,7 +37,7 @@ class BluetoothStateManager : HybridBluetoothStateManagerSpec() {
       when (intent?.action) {
         BluetoothAdapter.ACTION_STATE_CHANGED -> {
           val state = intent.getIntExtra(BluetoothAdapter.EXTRA_STATE, BluetoothAdapter.ERROR)
-          for (listener in listeners) {
+          for (listener in listeners.values) {
             listener(fromBluetoothState(state))
           }
         }
@@ -82,21 +83,22 @@ class BluetoothStateManager : HybridBluetoothStateManagerSpec() {
     }
   }
 
-  private var listeners = mutableListOf<(state: BluetoothState) -> Unit>()
+  private val listeners = mutableMapOf<Any, (BluetoothState) -> Unit>()
 
-  override fun addListener(callback: (state: BluetoothState) -> Unit): Double {
-    if (this.listeners.size == 0) {
+  override fun addListener(callback: (state: BluetoothState) -> Unit): String {
+    if (this.listeners.isEmpty()) {
       this.startListenForBluetoothStateChange()
     }
 
-    this.listeners.add(callback)
-    return this.listeners.size.toDouble() - 1
+    val key = UUID.randomUUID().toString()
+    this.listeners[key] = callback
+    return key
   }
 
-  override fun removeListener(index: Double) {
-    this.listeners.removeAt(index.toInt())
-    if (this.listeners.size == 0) {
-      this.stopListenForBluetoothStateChange()
+  override fun removeListener(callbackRef: String) {
+    this.listeners.remove(callbackRef)
+    if (this.listeners.isEmpty()) {
+      stopListenForBluetoothStateChange()
     }
   }
 

packages/react-native-bluetooth-state-manager/ios/HybridBluetoothStateManager.swift

@@ -51,16 +51,17 @@ class Listener {
 }
 
 class HybridBluetoothStateManager: HybridBluetoothStateManagerSpec {
-    var listeners: [Listener] = []
+    var listeners: [String: Listener] = [:]
 
-    func addListener(callback: @escaping (BluetoothState) -> Void) throws -> Double {
+    func addListener(callback: @escaping (BluetoothState) -> Void) throws -> String {
         let listener = Listener(callback: callback)
-        listeners.append(listener)
-        return Double(listeners.count) - 1.0
+        let key = UUID().uuidString
+        listeners[key] = listener
+        return key
     }
 
-    func removeListener(index: Double) throws {
-        listeners.remove(at: Int(index))
+    func removeListener(callbackRef: String) throws {
+        listeners.removeValue(forKey: callbackRef)
     }
 
     var centralManager: CBCentralManager!
@@ -69,7 +70,7 @@ class HybridBluetoothStateManager: HybridBluetoothStateManagerSpec {
     override init() {
         super.init()
         self.bManager = CentralManager.init { state in
-            for listener in self.listeners {
+            for listener in self.listeners.values {
                 listener.callback(state)
             }
         }