@hexar/biometric-identity-sdk-react-native 1.24.0 → 1.25.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hexar/biometric-identity-sdk-react-native",
-  "version": "1.24.0",
+  "version": "1.25.0",
   "description": "React Native wrapper for Biometric Identity SDK",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -8,7 +8,8 @@
   "scripts": {
     "build": "tsc && cp -r src/assets dist/",
     "dev": "tsc --watch",
-    "clean": "rm -rf dist"
+    "clean": "rm -rf dist",
+    "postinstall": "node scripts/postinstall.js"
   },
   "peerDependencies": {
     "@hexar/biometric-identity-sdk-core": ">=1.8.0",

package/patches/react-native-vision-camera+4.7.3.patch

@@ -0,0 +1,139 @@
+diff --git a/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/core/utils/runOnUiThread.kt b/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/core/utils/runOnUiThread.kt
+index e96e78e..9b9ddfc 100644
+--- a/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/core/utils/runOnUiThread.kt
++++ b/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/core/utils/runOnUiThread.kt
+@@ -13,9 +13,13 @@ suspend inline fun <T> runOnUiThreadAndWait(crossinline function: () -> T): T {
+ 
+   return suspendCancellableCoroutine { continuation ->
+     UiThreadUtil.runOnUiThread {
+-      if (continuation.isCancelled) throw CancellationException()
+-      val result = function()
+-      continuation.resume(result)
++      try {
++        if (continuation.isCancelled) throw CancellationException()
++        val result = function()
++        continuation.resume(result)
++      } catch (e: Throwable) {
++        continuation.resumeWithException(e)
++      }
+     }
+   }
+ }
+diff --git a/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/react/CameraViewModule.kt b/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/react/CameraViewModule.kt
+index 059f677..54ead5a 100644
+--- a/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/react/CameraViewModule.kt
++++ b/node_modules/react-native-vision-camera/android/src/main/java/com/mrousavy/camera/react/CameraViewModule.kt
+@@ -91,7 +91,12 @@ class CameraViewModule(reactContext: ReactApplicationContext) : ReactContextBase
+   @ReactMethod
+   fun takePhoto(viewTag: Int, options: ReadableMap, promise: Promise) {
+     backgroundCoroutineScope.launch {
+-      val view = findCameraView(viewTag)
++      val view = try {
++        findCameraView(viewTag)
++      } catch (e: Throwable) {
++        promise.reject("camera/view-not-found", "Camera view has been removed", e)
++        return@launch
++      }
+       withPromise(promise) {
+         view.takePhoto(options)
+       }
+@@ -101,7 +106,12 @@ class CameraViewModule(reactContext: ReactApplicationContext) : ReactContextBase
+   @ReactMethod
+   fun takeSnapshot(viewTag: Int, jsOptions: ReadableMap, promise: Promise) {
+     backgroundCoroutineScope.launch {
+-      val view = findCameraView(viewTag)
++      val view = try {
++        findCameraView(viewTag)
++      } catch (e: Throwable) {
++        promise.reject("camera/view-not-found", "Camera view has been removed", e)
++        return@launch
++      }
+       runOnUiThread {
+         try {
+           val options = TakeSnapshotOptions.fromJSValue(reactApplicationContext, jsOptions)
+@@ -118,7 +128,13 @@ class CameraViewModule(reactContext: ReactApplicationContext) : ReactContextBase
+   @ReactMethod
+   fun startRecording(viewTag: Int, jsOptions: ReadableMap, onRecordCallback: Callback) {
+     backgroundCoroutineScope.launch {
+-      val view = findCameraView(viewTag)
++      val view = try {
++        findCameraView(viewTag)
++      } catch (e: Throwable) {
++        val map = makeErrorMap("camera/view-not-found", "Camera view has been removed", e)
++        onRecordCallback(null, map)
++        return@launch
++      }
+       try {
+         val options = RecordVideoOptions.fromJSValue(reactApplicationContext, jsOptions)
+         view.startRecording(options, onRecordCallback)
+@@ -136,8 +152,13 @@ class CameraViewModule(reactContext: ReactApplicationContext) : ReactContextBase
+   @ReactMethod
+   fun pauseRecording(viewTag: Int, promise: Promise) {
+     backgroundCoroutineScope.launch {
++      val view = try {
++        findCameraView(viewTag)
++      } catch (e: Throwable) {
++        promise.reject("camera/view-not-found", "Camera view has been removed", e)
++        return@launch
++      }
+       withPromise(promise) {
+-        val view = findCameraView(viewTag)
+         view.pauseRecording()
+         return@withPromise null
+       }
+@@ -147,7 +168,12 @@ class CameraViewModule(reactContext: ReactApplicationContext) : ReactContextBase
+   @ReactMethod
+   fun resumeRecording(viewTag: Int, promise: Promise) {
+     backgroundCoroutineScope.launch {
+-      val view = findCameraView(viewTag)
++      val view = try {
++        findCameraView(viewTag)
++      } catch (e: Throwable) {
++        promise.reject("camera/view-not-found", "Camera view has been removed", e)
++        return@launch
++      }
+       withPromise(promise) {
+         view.resumeRecording()
+         return@withPromise null
+@@ -158,7 +184,12 @@ class CameraViewModule(reactContext: ReactApplicationContext) : ReactContextBase
+   @ReactMethod
+   fun stopRecording(viewTag: Int, promise: Promise) {
+     backgroundCoroutineScope.launch {
+-      val view = findCameraView(viewTag)
++      val view = try {
++        findCameraView(viewTag)
++      } catch (e: Throwable) {
++        promise.reject("camera/view-not-found", "Camera view has been removed", e)
++        return@launch
++      }
+       withPromise(promise) {
+         view.stopRecording()
+         return@withPromise null
+@@ -169,7 +200,12 @@ class CameraViewModule(reactContext: ReactApplicationContext) : ReactContextBase
+   @ReactMethod
+   fun cancelRecording(viewTag: Int, promise: Promise) {
+     backgroundCoroutineScope.launch {
+-      val view = findCameraView(viewTag)
++      val view = try {
++        findCameraView(viewTag)
++      } catch (e: Throwable) {
++        promise.reject("camera/view-not-found", "Camera view has been removed", e)
++        return@launch
++      }
+       withPromise(promise) {
+         view.cancelRecording()
+         return@withPromise null
+@@ -180,7 +216,12 @@ class CameraViewModule(reactContext: ReactApplicationContext) : ReactContextBase
+   @ReactMethod
+   fun focus(viewTag: Int, point: ReadableMap, promise: Promise) {
+     backgroundCoroutineScope.launch {
+-      val view = findCameraView(viewTag)
++      val view = try {
++        findCameraView(viewTag)
++      } catch (e: Throwable) {
++        promise.reject("camera/view-not-found", "Camera view has been removed", e)
++        return@launch
++      }
+       withPromise(promise) {
+         view.focus(point)
+         return@withPromise null

package/scripts/postinstall.js

@@ -0,0 +1,168 @@
+#!/usr/bin/env node
+
+/**
+ * Postinstall script for @hexar/biometric-identity-sdk-react-native
+ *
+ * Patches react-native-vision-camera to prevent IllegalViewOperationException
+ * crashes on Android when CameraViewModule.findCameraView is called after
+ * the React Native view has been unmounted.
+ *
+ * The patch makes two changes:
+ * 1. runOnUiThreadAndWait: wraps the UI-thread lambda in try/catch so
+ *    exceptions route back to the coroutine via resumeWithException instead
+ *    of crashing the main thread Looper.
+ * 2. CameraViewModule: wraps every findCameraView call in try/catch so
+ *    the exception becomes a Promise rejection (catchable from JS) instead
+ *    of an uncaught native crash.
+ */
+
+const { execSync } = require('child_process');
+const fs = require('fs');
+const path = require('path');
+
+// Walk up from this package to find the consumer's node_modules root.
+// This script runs from:
+//   <consumer>/node_modules/@hexar/biometric-identity-sdk-react-native/scripts/
+// We need:
+//   <consumer>/node_modules/react-native-vision-camera/
+function findConsumerRoot() {
+  let dir = __dirname;
+  // Walk up until we find a node_modules parent that contains react-native-vision-camera
+  for (let i = 0; i < 10; i++) {
+    dir = path.dirname(dir);
+    const candidate = path.join(dir, 'react-native-vision-camera');
+    if (fs.existsSync(candidate)) {
+      return path.dirname(dir); // consumer project root
+    }
+    // Also check if we're at a node_modules boundary
+    if (path.basename(dir) === 'node_modules') {
+      return path.dirname(dir); // consumer project root
+    }
+  }
+  return null;
+}
+
+function applyPatch() {
+  const consumerRoot = findConsumerRoot();
+  if (!consumerRoot) {
+    // Not installed in a consumer project (e.g. local dev) — skip silently
+    return;
+  }
+
+  const visionCameraDir = path.join(consumerRoot, 'node_modules', 'react-native-vision-camera');
+  if (!fs.existsSync(visionCameraDir)) {
+    // vision-camera not installed — nothing to patch
+    return;
+  }
+
+  // Target files to patch
+  const runOnUiThreadFile = path.join(
+    visionCameraDir,
+    'android/src/main/java/com/mrousavy/camera/core/utils/runOnUiThread.kt'
+  );
+  const cameraViewModuleFile = path.join(
+    visionCameraDir,
+    'android/src/main/java/com/mrousavy/camera/react/CameraViewModule.kt'
+  );
+
+  if (!fs.existsSync(runOnUiThreadFile)) {
+    // Different version layout — skip
+    return;
+  }
+
+  // Check if already patched (idempotent)
+  const runOnUiThreadContent = fs.readFileSync(runOnUiThreadFile, 'utf8');
+  if (runOnUiThreadContent.includes('resumeWithException')) {
+    // Already patched
+    return;
+  }
+
+  console.log('[biometric-sdk] Patching react-native-vision-camera to fix Android IllegalViewOperationException crash...');
+
+  // Patch 1: runOnUiThread.kt — wrap UI thread lambda in try/catch
+  const patchedRunOnUiThread = runOnUiThreadContent.replace(
+    `  return suspendCancellableCoroutine { continuation ->
+    UiThreadUtil.runOnUiThread {
+      if (continuation.isCancelled) throw CancellationException()
+      val result = function()
+      continuation.resume(result)
+    }
+  }`,
+    `  return suspendCancellableCoroutine { continuation ->
+    UiThreadUtil.runOnUiThread {
+      try {
+        if (continuation.isCancelled) throw CancellationException()
+        val result = function()
+        continuation.resume(result)
+      } catch (e: Throwable) {
+        continuation.resumeWithException(e)
+      }
+    }
+  }`
+  );
+
+  if (patchedRunOnUiThread !== runOnUiThreadContent) {
+    fs.writeFileSync(runOnUiThreadFile, patchedRunOnUiThread, 'utf8');
+    console.log('[biometric-sdk]   ✓ Patched runOnUiThread.kt');
+  }
+
+  // Patch 2: CameraViewModule.kt — wrap findCameraView in try/catch for all methods
+  let moduleContent = fs.readFileSync(cameraViewModuleFile, 'utf8');
+  const original = moduleContent;
+
+  // Helper: replace "val view = findCameraView(viewTag)" with try/catch version
+  // that rejects the promise. Handle both promise and callback patterns.
+
+  // Pattern for promise-based methods (takePhoto, takeSnapshot, pauseRecording,
+  // resumeRecording, stopRecording, cancelRecording, focus)
+  // Match: "val view = findCameraView(viewTag)" inside a launch block that has a promise
+  const promiseMethods = [
+    'takePhoto', 'takeSnapshot', 'resumeRecording',
+    'stopRecording', 'cancelRecording', 'focus'
+  ];
+
+  for (const method of promiseMethods) {
+    const pattern = new RegExp(
+      `(fun ${method}\\(viewTag: Int,.*?promise: Promise\\).*?backgroundCoroutineScope\\.launch \\{\\n)(\\s*val view = findCameraView\\(viewTag\\))`,
+      's'
+    );
+    moduleContent = moduleContent.replace(pattern, (match, before, viewLine) => {
+      const indent = viewLine.match(/^(\s*)/)[1];
+      return `${before}${indent}val view = try {\n${indent}  findCameraView(viewTag)\n${indent}} catch (e: Throwable) {\n${indent}  promise.reject("camera/view-not-found", "Camera view has been removed", e)\n${indent}  return@launch\n${indent}}`;
+    });
+  }
+
+  // pauseRecording has findCameraView inside withPromise — different pattern
+  moduleContent = moduleContent.replace(
+    /fun pauseRecording\(viewTag: Int, promise: Promise\).*?backgroundCoroutineScope\.launch \{\n(\s*)withPromise\(promise\) \{\n(\s*)val view = findCameraView\(viewTag\)/s,
+    (match, indent1, indent2) => {
+      return match.replace(
+        `${indent1}withPromise(promise) {\n${indent2}val view = findCameraView(viewTag)`,
+        `${indent1}val view = try {\n${indent1}  findCameraView(viewTag)\n${indent1}} catch (e: Throwable) {\n${indent1}  promise.reject("camera/view-not-found", "Camera view has been removed", e)\n${indent1}  return@launch\n${indent1}}\n${indent1}withPromise(promise) {`
+      );
+    }
+  );
+
+  // startRecording uses Callback, not Promise
+  moduleContent = moduleContent.replace(
+    /(fun startRecording\(viewTag: Int,.*?onRecordCallback: Callback\).*?backgroundCoroutineScope\.launch \{\n)(\s*val view = findCameraView\(viewTag\))/s,
+    (match, before, viewLine) => {
+      const indent = viewLine.match(/^(\s*)/)[1];
+      return `${before}${indent}val view = try {\n${indent}  findCameraView(viewTag)\n${indent}} catch (e: Throwable) {\n${indent}  val map = makeErrorMap("camera/view-not-found", "Camera view has been removed", e)\n${indent}  onRecordCallback(null, map)\n${indent}  return@launch\n${indent}}`;
+    }
+  );
+
+  if (moduleContent !== original) {
+    fs.writeFileSync(cameraViewModuleFile, moduleContent, 'utf8');
+    console.log('[biometric-sdk]   ✓ Patched CameraViewModule.kt');
+  }
+
+  console.log('[biometric-sdk] Done.');
+}
+
+try {
+  applyPatch();
+} catch (e) {
+  // Never fail the install — the patch is a safety improvement, not a requirement
+  console.warn('[biometric-sdk] Warning: could not apply vision-camera patch:', e.message);
+}