react-native-web-serial-api 0.0.2 → 0.0.3

package/android/src/main/java/dev/webserialapi/NativeUsbSerialModule.java

@@ -657,11 +657,17 @@ public class NativeUsbSerialModule extends NativeUsbSerialSpec {
             Intent intent = new Intent(ACTION_USB_PERMISSION);
             intent.putExtra(EXTRA_REQUEST_CODE, requestCode);
 
+            intent.setPackage(getReactApplicationContext().getPackageName());
+
+            int flags = android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.S
+                    ? PendingIntent.FLAG_MUTABLE
+                    : 0;
+
             PendingIntent permissionIntent = PendingIntent.getBroadcast(
                 getReactApplicationContext(),
                 requestCode,
                 intent,
-                PendingIntent.FLAG_IMMUTABLE
+                flags
             );
             usbManager.requestPermission(driver.getDevice(), permissionIntent);
         } catch (Exception e) {

package/example/src/screens/DevicesScreen.tsx

@@ -135,23 +135,24 @@ export function DevicesScreen({onSelect}: Props) {
     [onSelect],
   );
 
-  // Tap on an unpermitted row: request Android USB permission. On grant, the
-  // library emits "connect" -> refresh() runs -> the row becomes permitted.
-  const grantPermission = React.useCallback(async (row: DeviceRow) => {
-    setError(null);
-    const usb = nativeUsb();
-    if (!usb) {
-      return;
-    }
-    try {
-      await usb.requestPermission(row.deviceId);
-    } catch (e: any) {
-      setError(e?.message ?? String(e));
-    } finally {
-      // Always refresh after permission attempt (fixes stale state)
-      refresh(); 
-    }
-  }, [refresh]);
+  // Tap on an unpermitted row: request Android USB permission. On grant, 
+  // refresh immediately.
+  const grantPermission = React.useCallback(
+    async (row: DeviceRow) => {
+      setError(null);
+      const usb = nativeUsb();
+      if (!usb) {
+        return;
+      }
+      try {
+        await usb.requestPermission(row.deviceId);
+      } catch (e: any) {
+        setError(e?.message ?? String(e));
+      }
+      refresh();
+    },
+    [refresh],
+  );
 
   const requestNew = React.useCallback(async () => {
     setError(null);
@@ -171,7 +172,10 @@ export function DevicesScreen({onSelect}: Props) {
         refresh();
       }
     };
-    const subscription = AppState.addEventListener('change', handleAppStateChange);
+    const subscription = AppState.addEventListener(
+      'change',
+      handleAppStateChange,
+    );
     return () => {
       subscription.remove();
     };

package/example/src/screens/TerminalScreen.tsx

@@ -249,6 +249,14 @@ export function TerminalScreen({port, settings, onBack}: Props) {
   }, [teardown]);
 
   const connect = React.useCallback(async () => {
+    // Idempotent: ignore overlapping triggers. On re-attach the device may emit
+    // several "connect" events (USB attach + permission-grant), and port.open()
+    // itself awaits the permission dialog — so guard against re-entrancy that
+    // would otherwise cause a double-open ("port already open") and leave us
+    // stuck in the failed/closed state.
+    if (connectedRef.current !== 'False') {
+      return;
+    }
     connectedRef.current = 'Pending';
     setConnected('Pending');
     try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-web-serial-api",
-  "version": "0.0.2",
+  "version": "0.0.3",
   "description": "W3C Web Serial API (navigator.serial) for React Native on Android, backed by a USB-serial TurboModule (built on mik3y/usb-serial-for-android).",
   "main": "lib/commonjs/index.js",
   "react-native": "lib/commonjs/index.js",