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

package/dist/components/VideoRecorder.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"VideoRecorder.d.ts","sourceRoot":"","sources":["../../src/components/VideoRecorder.tsx"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAmD,MAAM,OAAO,CAAC;AAexE,OAAO,EAAE,WAAW,EAAE,mBAAmB,EAAE,iBAAiB,EAAmC,MAAM,oCAAoC,CAAC;AAE1I,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,WAAW,CAAC;IACpB,QAAQ,CAAC,EAAE,iBAAiB,CAAC;IAC7B,sEAAsE;IACtE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,sCAAsC;IACtC,YAAY,CAAC,EAAE,mBAAmB,EAAE,CAAC;IACrC,0CAA0C;IAC1C,UAAU,CAAC,EAAE,eAAe,EAAE,CAAC;IAC/B,wCAAwC;IACxC,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,kDAAkD;IAClD,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,wCAAwC;IACxC,UAAU,EAAE,CAAC,SAAS,EAAE,oBAAoB,KAAK,IAAI,CAAC;IACtD,iCAAiC;IACjC,QAAQ,EAAE,MAAM,IAAI,CAAC;IACrB,gDAAgD;IAChD,iBAAiB,CAAC,EAAE,MAAM,OAAO,CAAC,eAAe,EAAE,CAAC,CAAC;CACtD;AAED,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,oBAAoB,EAAE,OAAO,CAAC;IAC9B,YAAY,EAAE,MAAM,CAAC;IACrB,mBAAmB,EAAE,MAAM,EAAE,CAAC;IAC9B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AA2CD,eAAO,MAAM,aAAa,EAAE,KAAK,CAAC,EAAE,CAAC,kBAAkB,CAwhCtD,CAAC;AA4PF,eAAe,aAAa,CAAC"}
+{"version":3,"file":"VideoRecorder.d.ts","sourceRoot":"","sources":["../../src/components/VideoRecorder.tsx"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,KAAmD,MAAM,OAAO,CAAC;AAexE,OAAO,EAAE,WAAW,EAAE,mBAAmB,EAAE,iBAAiB,EAAmC,MAAM,oCAAoC,CAAC;AAE1I,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,WAAW,CAAC;IACpB,QAAQ,CAAC,EAAE,iBAAiB,CAAC;IAC7B,sEAAsE;IACtE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,sCAAsC;IACtC,YAAY,CAAC,EAAE,mBAAmB,EAAE,CAAC;IACrC,0CAA0C;IAC1C,UAAU,CAAC,EAAE,eAAe,EAAE,CAAC;IAC/B,wCAAwC;IACxC,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,kDAAkD;IAClD,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,wCAAwC;IACxC,UAAU,EAAE,CAAC,SAAS,EAAE,oBAAoB,KAAK,IAAI,CAAC;IACtD,iCAAiC;IACjC,QAAQ,EAAE,MAAM,IAAI,CAAC;IACrB,gDAAgD;IAChD,iBAAiB,CAAC,EAAE,MAAM,OAAO,CAAC,eAAe,EAAE,CAAC,CAAC;CACtD;AAED,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,MAAM,EAAE,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,oBAAoB,EAAE,OAAO,CAAC;IAC9B,YAAY,EAAE,MAAM,CAAC;IACrB,mBAAmB,EAAE,MAAM,EAAE,CAAC;IAC9B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AA2CD,eAAO,MAAM,aAAa,EAAE,KAAK,CAAC,EAAE,CAAC,kBAAkB,CA2lCtD,CAAC;AA4PF,eAAe,aAAa,CAAC"}

package/dist/components/VideoRecorder.js

@@ -102,12 +102,20 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
     const [guidanceText, setGuidanceText] = (0, react_1.useState)(null);
     const [hasPermission, setHasPermission] = (0, react_1.useState)(false);
     const [isCameraReady, setIsCameraReady] = (0, react_1.useState)(false);
+    // Controlled via stopRecording — set to false before calling onComplete so
+    // VisionCamera tears down the native session and any in-flight takePhoto()
+    // calls fail gracefully instead of crashing with IllegalViewOperationException
+    // when the parent unmounts the <Camera> view underneath them.
+    const [isCameraActive, setIsCameraActive] = (0, react_1.useState)(true);
     const cameraRef = (0, react_1.useRef)(null);
     const { hasPermission: cameraPermission, requestPermission } = (0, react_native_vision_camera_1.useCameraPermission)();
     // Prefer wide-angle (main) front camera — some Android devices expose
     // multiple front sensors and the default may lack autofocus.
+    // Only request wide-angle — ultra-wide opens a multi-physical-camera session
+    // that is slow to release, causing CameraViewModule.findCameraView to throw
+    // IllegalViewOperationException when the component unmounts after recording.
     const device = (0, react_native_vision_camera_1.useCameraDevice)('front', {
-        physicalDevices: ['wide-angle-camera', 'ultra-wide-angle-camera'],
+        physicalDevices: ['wide-angle-camera'],
     });
     const [deviceReady, setDeviceReady] = (0, react_1.useState)(!!device);
     const fadeAnim = (0, react_1.useRef)(new react_native_1.Animated.Value(0)).current;
@@ -121,9 +129,19 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
     const videoRecordingRef = (0, react_1.useRef)(null);
     const isRecordingRef = (0, react_1.useRef)(false);
     const recordingTimeoutRef = (0, react_1.useRef)(null);
+    // Tracks whether a takePhoto() call is currently dispatched to the native
+    // bridge.  We must wait for it to resolve before unmounting the <Camera>
+    // view, otherwise CameraViewModule.findCameraView crashes on Android.
+    const captureInFlightRef = (0, react_1.useRef)(false);
+    const isMountedRef = (0, react_1.useRef)(true);
     // Trigger autofocus on center — some Android devices (e.g. Ulefone Armor)
     // don't auto-focus without an explicit programmatic trigger.
     const triggerFocus = (0, react_1.useCallback)(() => {
+        // Guard: focus() dispatches to native findCameraView which crashes
+        // if the view is gone.  The JS try/catch does NOT help — the exception
+        // happens on the Android UI thread Looper, not in the JS promise chain.
+        if (!isMountedRef.current)
+            return;
         try {
             const { width: w, height: h } = react_native_1.Dimensions.get('window');
             cameraRef.current?.focus({ x: w / 2, y: h / 2 });
@@ -132,6 +150,30 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
             // focus() not supported or camera not ready — safe to ignore
         }
     }, []);
+    // ── Unmount cleanup ──────────────────────────────────────────────────
+    // Prevents scenario 2: unexpected unmount (back button, navigation,
+    // phone call) while timers are still scheduled.  Without this, pending
+    // setTimeout callbacks fire after unmount, dispatch takePhoto() to the
+    // native bridge, and crash with IllegalViewOperationException.
+    (0, react_1.useEffect)(() => {
+        isMountedRef.current = true;
+        return () => {
+            isMountedRef.current = false;
+            isRecordingRef.current = false;
+            if (frameCaptureInterval.current) {
+                clearTimeout(frameCaptureInterval.current);
+                frameCaptureInterval.current = null;
+            }
+            if (recordingTimeoutRef.current) {
+                clearTimeout(recordingTimeoutRef.current);
+                recordingTimeoutRef.current = null;
+            }
+            if (frameInterval.current) {
+                clearTimeout(frameInterval.current);
+                frameInterval.current = null;
+            }
+        };
+    }, []);
     const minDurationMs = 8000;
     const totalDuration = duration || Math.max(minDurationMs, challenges.reduce((sum, c) => sum + c.duration_ms, 0) + 2000);
     (0, react_1.useEffect)(() => {
@@ -307,6 +349,7 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
         setChallengeProgress(0);
         setOverallProgress(0);
         recordingStartTime.current = 0;
+        setIsCameraActive(true);
         setPhase('countdown');
         setCountdown(3);
     }, []);
@@ -314,6 +357,30 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
         setPhase('loading');
         react_native_1.Alert.alert('Recording Error', 'Failed to record video. Please try again.', [{ text: 'OK', onPress: onCancel }]);
     }, [onCancel]);
+    // Wait for any in-flight takePhoto() to finish, then deactivate the camera
+    // and deliver the result.  This prevents the IllegalViewOperationException
+    // crash: the native view stays alive while the native bridge processes the
+    // last queued takePhoto(), and only THEN do we trigger the parent unmount
+    // via onComplete.
+    const safeComplete = (0, react_1.useCallback)((result) => {
+        const drain = () => {
+            if (captureInFlightRef.current) {
+                // A takePhoto() is still on the native bridge — check again shortly
+                setTimeout(drain, 30);
+                return;
+            }
+            // No in-flight capture — safe to tear down
+            setIsCameraActive(false);
+            // Small delay to let VisionCamera process the isActive=false prop change
+            // and release the native camera session before the parent unmounts the view.
+            setTimeout(() => {
+                if (isMountedRef.current) {
+                    onComplete(result);
+                }
+            }, 100);
+        };
+        drain();
+    }, [onComplete]);
     // Safety timeout: if the camera session never fires onInitialized within
     // 10 seconds, treat it as a hardware failure and send the user back.
     (0, react_1.useEffect)(() => {
@@ -380,12 +447,7 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
             };
             biometric_identity_sdk_core_1.logger.info('Video recording completed:', result.frames.length, 'frames,', (actualDuration / 1000).toFixed(1) + 's', 'challenges:', currentCompletedChallenges.length);
             isRecordingRef.current = false;
-            // Delay onComplete so Vision Camera's native session (especially
-            // multi-physical-camera sessions opened via physicalDevices) has time
-            // to fully close before the parent unmounts the <Camera> view.
-            // Without this delay, CameraViewModule.findCameraView throws
-            // IllegalViewOperationException on Android 13 (SDK 33).
-            setTimeout(() => onComplete(result), 250);
+            safeComplete(result);
         }
         catch (error) {
             biometric_identity_sdk_core_1.logger.error('Error processing video:', error);
@@ -393,7 +455,7 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
             setOverallProgress(0);
             handleRecordingError(error);
         }
-    }, [frames, completedChallenges, challenges, sessionId, onComplete, resetAndRetry, handleRecordingError, strings, minDurationMs, phase]);
+    }, [frames, completedChallenges, challenges, sessionId, safeComplete, resetAndRetry, handleRecordingError, strings, minDurationMs, phase]);
     const startFrameCapture = (0, react_1.useCallback)(() => {
         if (cameraRef.current && device) {
             biometric_identity_sdk_core_1.logger.info('Starting serial frame capture');
@@ -441,9 +503,13 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
             // finishes, preventing overlapping takePhoto() calls that cause the
             // camera to throw "busy" errors and kill the capture loop.
             const captureNextFrame = async () => {
-                if (!isRecordingRef.current || framesRef.current.length >= MAX_FRAMES) {
+                // Bail if recording stopped, component unmounted, or we have enough frames.
+                // Checking isMountedRef prevents dispatching takePhoto() to a native view
+                // that is about to be removed from NativeViewHierarchyManager.
+                if (!isRecordingRef.current || !isMountedRef.current || framesRef.current.length >= MAX_FRAMES) {
                     return;
                 }
+                captureInFlightRef.current = true;
                 try {
                     const photo = await cameraRef.current?.takePhoto({
                         flash: 'off',
@@ -486,12 +552,14 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
                     biometric_identity_sdk_core_1.logger.warn('Frame capture error', consecutiveErrors, '/', maxConsecutiveErrors, error?.message);
                     if (consecutiveErrors >= maxConsecutiveErrors) {
                         biometric_identity_sdk_core_1.logger.error('Too many consecutive capture errors, stopping');
+                        captureInFlightRef.current = false;
                         return;
                     }
                 }
+                captureInFlightRef.current = false;
                 // Schedule next capture only after this one completes (serial chain).
                 // Small delay lets the camera hardware reset between captures.
-                if (isRecordingRef.current && framesRef.current.length < MAX_FRAMES) {
+                if (isRecordingRef.current && isMountedRef.current && framesRef.current.length < MAX_FRAMES) {
                     frameCaptureInterval.current = setTimeout(captureNextFrame, 66);
                 }
             };
@@ -536,7 +604,7 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
                         sessionId,
                     };
                     setPhase('processing');
-                    setTimeout(() => onComplete(result), 250);
+                    safeComplete(result);
                 }
                 else {
                     biometric_identity_sdk_core_1.logger.info('Stopping recording with frames (fallback):', currentFrames.length);
@@ -549,7 +617,7 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
                         sessionId,
                     };
                     setPhase('processing');
-                    setTimeout(() => onComplete(result), 250);
+                    safeComplete(result);
                 }
             }
         }
@@ -580,11 +648,9 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
                 challengesCompleted: currentCompletedChallenges,
                 sessionId,
             };
-            setTimeout(() => {
-                onComplete(result);
-            }, 250);
+            safeComplete(result);
         }
-    }, [frames, completedChallenges, challenges, sessionId, onComplete, minDurationMs, resetAndRetry, strings]);
+    }, [frames, completedChallenges, challenges, sessionId, onComplete, safeComplete, minDurationMs, resetAndRetry, strings]);
     const runChallenge = (0, react_1.useCallback)((index) => {
         const currentCompletedChallenges = completedChallengesRef.current.length > 0 ? completedChallengesRef.current : completedChallenges;
         biometric_identity_sdk_core_1.logger.info('VideoRecorder: runChallenge called', {
@@ -626,7 +692,7 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
                         sessionId,
                     };
                     setPhase('processing');
-                    setTimeout(() => onComplete(result), 250);
+                    safeComplete(result);
                 }
             }
             return;
@@ -678,7 +744,7 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
                 runChallenge(index + 1);
             });
         }, challenge.duration_ms);
-    }, [challenges, fadeAnim, progressAnim, animateArrow, totalDuration, minDurationMs, stopRecording]);
+    }, [challenges, fadeAnim, progressAnim, animateArrow, totalDuration, minDurationMs, stopRecording, safeComplete]);
     const startRecording = (0, react_1.useCallback)(async () => {
         setPhase('recording');
         recordingStartTime.current = Date.now();
@@ -807,7 +873,7 @@ const VideoRecorder = ({ theme, language, duration, instructions, challenges: pr
     }
     return (react_1.default.createElement(react_native_1.View, { style: styles.container },
         react_1.default.createElement(react_native_1.View, { style: styles.cameraContainer },
-            react_1.default.createElement(react_native_vision_camera_1.Camera, { ref: cameraRef, style: react_native_1.StyleSheet.absoluteFill, device: device, isActive: true, video: true, photo: true, audio: false, onInitialized: () => setIsCameraReady(true), onError: (error) => {
+            react_1.default.createElement(react_native_vision_camera_1.Camera, { ref: cameraRef, style: react_native_1.StyleSheet.absoluteFill, device: device, isActive: isCameraActive, video: true, photo: true, audio: false, onInitialized: () => setIsCameraReady(true), onError: (error) => {
                     biometric_identity_sdk_core_1.logger.error('Camera hardware error:', error);
                     handleRecordingError(error);
                 } }),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hexar/biometric-identity-sdk-react-native",
-  "version": "1.22.0",
+  "version": "1.24.0",
   "description": "React Native wrapper for Biometric Identity SDK",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/components/VideoRecorder.tsx

@@ -131,12 +131,20 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
   const [guidanceText, setGuidanceText] = useState<string | null>(null);
   const [hasPermission, setHasPermission] = useState(false);
   const [isCameraReady, setIsCameraReady] = useState(false);
+  // Controlled via stopRecording — set to false before calling onComplete so
+  // VisionCamera tears down the native session and any in-flight takePhoto()
+  // calls fail gracefully instead of crashing with IllegalViewOperationException
+  // when the parent unmounts the <Camera> view underneath them.
+  const [isCameraActive, setIsCameraActive] = useState(true);
   const cameraRef = useRef<Camera>(null);
   const { hasPermission: cameraPermission, requestPermission } = useCameraPermission();
   // Prefer wide-angle (main) front camera — some Android devices expose
   // multiple front sensors and the default may lack autofocus.
+  // Only request wide-angle — ultra-wide opens a multi-physical-camera session
+  // that is slow to release, causing CameraViewModule.findCameraView to throw
+  // IllegalViewOperationException when the component unmounts after recording.
   const device = useCameraDevice('front', {
-    physicalDevices: ['wide-angle-camera', 'ultra-wide-angle-camera'],
+    physicalDevices: ['wide-angle-camera'],
   });
   const [deviceReady, setDeviceReady] = useState(!!device);
   
@@ -152,9 +160,18 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
   const videoRecordingRef = useRef<any>(null);
   const isRecordingRef = useRef<boolean>(false);
   const recordingTimeoutRef = useRef<NodeJS.Timeout | null>(null);
+  // Tracks whether a takePhoto() call is currently dispatched to the native
+  // bridge.  We must wait for it to resolve before unmounting the <Camera>
+  // view, otherwise CameraViewModule.findCameraView crashes on Android.
+  const captureInFlightRef = useRef<boolean>(false);
+  const isMountedRef = useRef<boolean>(true);
   // Trigger autofocus on center — some Android devices (e.g. Ulefone Armor)
   // don't auto-focus without an explicit programmatic trigger.
   const triggerFocus = useCallback(() => {
+    // Guard: focus() dispatches to native findCameraView which crashes
+    // if the view is gone.  The JS try/catch does NOT help — the exception
+    // happens on the Android UI thread Looper, not in the JS promise chain.
+    if (!isMountedRef.current) return;
     try {
       const { width: w, height: h } = Dimensions.get('window');
       cameraRef.current?.focus({ x: w / 2, y: h / 2 });
@@ -163,6 +180,31 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
     }
   }, []);
 
+  // ── Unmount cleanup ──────────────────────────────────────────────────
+  // Prevents scenario 2: unexpected unmount (back button, navigation,
+  // phone call) while timers are still scheduled.  Without this, pending
+  // setTimeout callbacks fire after unmount, dispatch takePhoto() to the
+  // native bridge, and crash with IllegalViewOperationException.
+  useEffect(() => {
+    isMountedRef.current = true;
+    return () => {
+      isMountedRef.current = false;
+      isRecordingRef.current = false;
+      if (frameCaptureInterval.current) {
+        clearTimeout(frameCaptureInterval.current);
+        frameCaptureInterval.current = null;
+      }
+      if (recordingTimeoutRef.current) {
+        clearTimeout(recordingTimeoutRef.current);
+        recordingTimeoutRef.current = null;
+      }
+      if (frameInterval.current) {
+        clearTimeout(frameInterval.current);
+        frameInterval.current = null;
+      }
+    };
+  }, []);
+
   const minDurationMs = 8000;
   const totalDuration = duration || Math.max(
     minDurationMs,
@@ -356,6 +398,7 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
     setChallengeProgress(0);
     setOverallProgress(0);
     recordingStartTime.current = 0;
+    setIsCameraActive(true);
     setPhase('countdown');
     setCountdown(3);
   }, []);
@@ -369,6 +412,31 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
     );
   }, [onCancel]);
 
+  // Wait for any in-flight takePhoto() to finish, then deactivate the camera
+  // and deliver the result.  This prevents the IllegalViewOperationException
+  // crash: the native view stays alive while the native bridge processes the
+  // last queued takePhoto(), and only THEN do we trigger the parent unmount
+  // via onComplete.
+  const safeComplete = useCallback((result: VideoRecordingResult) => {
+    const drain = () => {
+      if (captureInFlightRef.current) {
+        // A takePhoto() is still on the native bridge — check again shortly
+        setTimeout(drain, 30);
+        return;
+      }
+      // No in-flight capture — safe to tear down
+      setIsCameraActive(false);
+      // Small delay to let VisionCamera process the isActive=false prop change
+      // and release the native camera session before the parent unmounts the view.
+      setTimeout(() => {
+        if (isMountedRef.current) {
+          onComplete(result);
+        }
+      }, 100);
+    };
+    drain();
+  }, [onComplete]);
+
   // Safety timeout: if the camera session never fires onInitialized within
   // 10 seconds, treat it as a hardware failure and send the user back.
   useEffect(() => {
@@ -450,19 +518,14 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
       logger.info('Video recording completed:', result.frames.length, 'frames,', (actualDuration / 1000).toFixed(1) + 's', 'challenges:', currentCompletedChallenges.length);
 
       isRecordingRef.current = false;
-      // Delay onComplete so Vision Camera's native session (especially
-      // multi-physical-camera sessions opened via physicalDevices) has time
-      // to fully close before the parent unmounts the <Camera> view.
-      // Without this delay, CameraViewModule.findCameraView throws
-      // IllegalViewOperationException on Android 13 (SDK 33).
-      setTimeout(() => onComplete(result), 250);
+      safeComplete(result);
     } catch (error) {
       logger.error('Error processing video:', error);
       setPhase('recording');
       setOverallProgress(0);
       handleRecordingError(error);
     }
-  }, [frames, completedChallenges, challenges, sessionId, onComplete, resetAndRetry, handleRecordingError, strings, minDurationMs, phase]);
+  }, [frames, completedChallenges, challenges, sessionId, safeComplete, resetAndRetry, handleRecordingError, strings, minDurationMs, phase]);
 
   const startFrameCapture = useCallback(() => {
     if (cameraRef.current && device) {
@@ -522,10 +585,14 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
       // finishes, preventing overlapping takePhoto() calls that cause the
       // camera to throw "busy" errors and kill the capture loop.
       const captureNextFrame = async () => {
-        if (!isRecordingRef.current || framesRef.current.length >= MAX_FRAMES) {
+        // Bail if recording stopped, component unmounted, or we have enough frames.
+        // Checking isMountedRef prevents dispatching takePhoto() to a native view
+        // that is about to be removed from NativeViewHierarchyManager.
+        if (!isRecordingRef.current || !isMountedRef.current || framesRef.current.length >= MAX_FRAMES) {
           return;
         }
 
+        captureInFlightRef.current = true;
         try {
           const photo = await cameraRef.current?.takePhoto({
             flash: 'off',
@@ -565,13 +632,15 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
           logger.warn('Frame capture error', consecutiveErrors, '/', maxConsecutiveErrors, error?.message);
           if (consecutiveErrors >= maxConsecutiveErrors) {
             logger.error('Too many consecutive capture errors, stopping');
+            captureInFlightRef.current = false;
             return;
           }
         }
+        captureInFlightRef.current = false;
 
         // Schedule next capture only after this one completes (serial chain).
         // Small delay lets the camera hardware reset between captures.
-        if (isRecordingRef.current && framesRef.current.length < MAX_FRAMES) {
+        if (isRecordingRef.current && isMountedRef.current && framesRef.current.length < MAX_FRAMES) {
           frameCaptureInterval.current = setTimeout(captureNextFrame, 66);
         }
       };
@@ -622,7 +691,7 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
             sessionId,
           };
           setPhase('processing');
-          setTimeout(() => onComplete(result), 250);
+          safeComplete(result);
         } else {
           logger.info('Stopping recording with frames (fallback):', currentFrames.length);
           const result: VideoRecordingResult = {
@@ -634,7 +703,7 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
             sessionId,
           };
           setPhase('processing');
-          setTimeout(() => onComplete(result), 250);
+          safeComplete(result);
         }
       }
     } else {
@@ -672,11 +741,9 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
         sessionId,
       };
       
-      setTimeout(() => {
-        onComplete(result);
-      }, 250);
+      safeComplete(result);
     }
-  }, [frames, completedChallenges, challenges, sessionId, onComplete, minDurationMs, resetAndRetry, strings]);
+  }, [frames, completedChallenges, challenges, sessionId, onComplete, safeComplete, minDurationMs, resetAndRetry, strings]);
 
   const runChallenge = useCallback((index: number) => {
     const currentCompletedChallenges = completedChallengesRef.current.length > 0 ? completedChallengesRef.current : completedChallenges;
@@ -723,7 +790,7 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
             sessionId,
           };
           setPhase('processing');
-          setTimeout(() => onComplete(result), 250);
+          safeComplete(result);
         }
       }
       return;
@@ -784,7 +851,7 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
         runChallenge(index + 1);
       });
     }, challenge.duration_ms);
-  }, [challenges, fadeAnim, progressAnim, animateArrow, totalDuration, minDurationMs, stopRecording]);
+  }, [challenges, fadeAnim, progressAnim, animateArrow, totalDuration, minDurationMs, stopRecording, safeComplete]);
 
   const startRecording = useCallback(async () => {
     setPhase('recording');
@@ -978,7 +1045,7 @@ export const VideoRecorder: React.FC<VideoRecorderProps> = ({
           ref={cameraRef}
           style={StyleSheet.absoluteFill}
           device={device}
-          isActive={true}
+          isActive={isCameraActive}
           video={true}
           photo={true}
           audio={false}