react-native-image-stitcher 0.14.1 → 0.15.0

package/ios/Sources/RNImageStitcher/StitcherFrameHostObject.h

@@ -1,60 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// StitcherFrameHostObject.h — Obj-C facade for the v0.8.0
-// `StitcherFrame` JSI host object.  Header is intentionally
-// Obj-C-only (no `<jsi/jsi.h>` import) so this can land in the
-// public CocoaPods umbrella without breaking `use_frameworks!` hosts
-// (same rationale as `KeyframeGateBridge.h`).
-//
-// The C++ JSI host object class lives in the .mm; this facade
-// exposes only what cross-module callers need:
-//
-//   - Factory `+ fromARFrame:pose:` that the AR worklet runtime
-//     calls per ARFrame to construct a host object backed by the
-//     current AR session's frame.
-//   - Opaque accessor `- (void *)jsiHostObjectPtr` returning the
-//     `std::shared_ptr<facebook::jsi::HostObject> *` (boxed) that
-//     the worklet runtime hands to `jsi::Object::createFromHostObject`.
-//
-// Lifetime: the Obj-C wrapper holds the C++ shared_ptr; ARC frees
-// the wrapper when nothing references it.  Worklet runtime
-// invalidates the underlying ARFrame retain when the dispatch
-// returns; after invalidation, JSI access throws.
-
-#pragma once
-
-#import <Foundation/Foundation.h>
-#import <ARKit/ARKit.h>
-
-@class RNSARFramePose;
-
-NS_ASSUME_NONNULL_BEGIN
-
-NS_SWIFT_NAME(StitcherFrameHostObject)
-@interface StitcherFrameHostObject : NSObject
-
-/// Construct a host object backed by the supplied ARFrame + pose.
-/// Retains the ARFrame for the host object's lifetime — caller can
-/// safely release their reference.
-///
-/// Thread: safe to call from the ARSession delegate queue; the
-/// resulting host object's JSI access must happen on the worklet
-/// runtime's thread (separate queue).
-+ (instancetype)fromARFrame:(ARFrame *)arFrame pose:(RNSARFramePose *)pose;
-
-/// Mark the host object's underlying ARFrame as no longer accessible.
-/// Subsequent JSI property reads return `undefined` or throw,
-/// depending on the property.  Idempotent.
-- (void)invalidate;
-
-/// Opaque pointer to a `std::shared_ptr<facebook::jsi::HostObject>`.
-/// The worklet runtime (Obj-C++ context with JSI available) casts
-/// this back via `*reinterpret_cast<std::shared_ptr<facebook::jsi::HostObject>*>(ptr)`
-/// to hand to `jsi::Object::createFromHostObject`.
-///
-/// Returns `NULL` if the host object has been invalidated.
-- (nullable void *)jsiHostObjectPtr;
-
-@end
-
-NS_ASSUME_NONNULL_END

package/ios/Sources/RNImageStitcher/StitcherFrameHostObject.mm

@@ -1,214 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// StitcherFrameHostObject.mm — iOS-specific wrapper for the shared
-// `retailens::StitcherFrameJsiHostObject` (defined in
-// `cpp/stitcher_frame_jsi.{hpp,cpp}`).
-//
-// Owns:
-//   - The Obj-C facade callable from Swift / other Obj-C / .mm files.
-//   - The iOS-specific `PixelBufferReader` impl (wraps a
-//     `CVPixelBufferRef` from `ARFrame.capturedImage`; lock / memcpy
-//     / unlock pattern).
-//   - The Obj-C → C++ extraction logic that builds a
-//     `retailens::StitcherFrameData` from an `ARFrame` + the lib's
-//     `RNSARFramePose`.
-//
-// Does NOT own:
-//   - The JSI `get` / `getPropertyNames` dispatch.  That lives in
-//     `cpp/stitcher_frame_jsi.cpp` and is identical to the Android
-//     implementation (DRY across platforms).
-
-#import "StitcherFrameHostObject.h"
-
-#import <Foundation/Foundation.h>
-#import <CoreVideo/CVPixelBuffer.h>
-#import <CoreMedia/CoreMedia.h>
-#import <os/log.h>
-
-#include <jsi/jsi.h>
-
-#include <algorithm>
-#include <cstring>
-#include <memory>
-#include <string>
-#include <utility>
-
-#include "stitcher_frame_data.hpp"
-#include "stitcher_frame_jsi.hpp"
-
-using namespace facebook;
-
-// Forward-declare the Swift `RNSARFramePose` Obj-C surface we need.
-// This matches the pattern in `KeyframeGateFrameProcessor.mm`
-// (forward-declaring `IncrementalStitcher`) — avoids depending on
-// the autogenerated `RNImageStitcher-Swift.h`, which is created at
-// build time and not always available to .mm files in this pod.
-//
-// MUST stay in sync with `RNSARSession.swift::RNSARFramePose` —
-// adding a new field there means adding it here too.
-@class RNSARFramePose;
-@interface RNSARFramePose : NSObject
-@property (nonatomic, readonly) double tx;
-@property (nonatomic, readonly) double ty;
-@property (nonatomic, readonly) double tz;
-@property (nonatomic, readonly) double qx;
-@property (nonatomic, readonly) double qy;
-@property (nonatomic, readonly) double qz;
-@property (nonatomic, readonly) double qw;
-@property (nonatomic, readonly) NSInteger imageWidth;
-@property (nonatomic, readonly) NSInteger imageHeight;
-@property (nonatomic, readonly) double timestampMs;
-@end
-
-#pragma mark - iOS PixelBufferReader
-
-namespace {
-
-/// iOS-specific `retailens::PixelBufferReader` impl.  See the base
-/// class docstring for the general contract (thread-affinity,
-/// invalidation semantics, Y-plane-only constraint).  This subclass
-/// adds:
-///   - `CVPixelBuffer` lock/memcpy/unlock per copyTo
-///   - `CFBridgingRetain` of the parent `ARFrame` so ARKit's
-///     pool can't reclaim the underlying buffer mid-read
-class IOSPixelBufferReader : public retailens::PixelBufferReader {
- public:
-  explicit IOSPixelBufferReader(ARFrame* arFrame) {
-    // Retain the ARFrame for our lifetime.  CFBridgingRetain hands
-    // ARC ownership to our void*.  Released in destructor.
-    _retainedFrame = (void*)CFBridgingRetain(arFrame);
-    CVPixelBufferRef pixelBuffer = arFrame.capturedImage;
-    if (pixelBuffer != NULL) {
-      _bytesPerRow = CVPixelBufferGetBytesPerRow(pixelBuffer);
-      _height = CVPixelBufferGetHeight(pixelBuffer);
-    }
-  }
-
-  ~IOSPixelBufferReader() override {
-    // Transfer ownership back to ARC, which then releases.
-    if (_retainedFrame != nullptr) {
-      ARFrame* frame = CFBridgingRelease(_retainedFrame);
-      (void)frame;
-      _retainedFrame = nullptr;
-    }
-  }
-
-  std::size_t byteSize() const override {
-    return _bytesPerRow * _height;
-  }
-
-  std::size_t copyTo(uint8_t* dst, std::size_t maxBytes) override {
-    if (_retainedFrame == nullptr) return 0;
-    ARFrame* frame = (__bridge ARFrame*)_retainedFrame;
-    CVPixelBufferRef pixelBuffer = frame.capturedImage;
-    if (pixelBuffer == NULL) return 0;
-
-    CVPixelBufferLockBaseAddress(pixelBuffer, kCVPixelBufferLock_ReadOnly);
-    const uint8_t* src = (const uint8_t*)CVPixelBufferGetBaseAddress(pixelBuffer);
-    std::size_t toCopy = std::min<std::size_t>(byteSize(), maxBytes);
-    if (src != nullptr && toCopy > 0) {
-      std::memcpy(dst, src, toCopy);
-    } else {
-      toCopy = 0;
-    }
-    CVPixelBufferUnlockBaseAddress(pixelBuffer, kCVPixelBufferLock_ReadOnly);
-    return toCopy;
-  }
-
- private:
-  void* _retainedFrame = nullptr;   // CFBridgingRetain'd ARFrame
-  std::size_t _bytesPerRow = 0;
-  std::size_t _height = 0;
-};
-
-}  // anonymous namespace
-
-#pragma mark - Obj-C facade
-
-@implementation StitcherFrameHostObject {
-  std::shared_ptr<retailens::StitcherFrameJsiHostObject> _hostObject;
-}
-
-+ (instancetype)fromARFrame:(ARFrame*)arFrame pose:(RNSARFramePose*)pose {
-  StitcherFrameHostObject* obj = [[self alloc] init];
-
-  retailens::StitcherFrameData data;
-  data.source = "ar";
-  data.width = static_cast<int32_t>(pose.imageWidth);
-  data.height = static_cast<int32_t>(pose.imageHeight);
-  // ARKit's `kCVPixelFormatType_420YpCbCr8BiPlanarFullRange` (NV12)
-  // is reported as "yuv".  Other formats (rare in ARKit; possible if
-  // ARWorldTrackingConfiguration.videoFormat is overridden to BGRA)
-  // → "unknown" + os_log warning so worklets that gate on
-  // `pixelFormat === 'yuv'` can be debugged without a screen recording.
-  OSType pf = CVPixelBufferGetPixelFormatType(arFrame.capturedImage);
-  if (pf == kCVPixelFormatType_420YpCbCr8BiPlanarFullRange ||
-      pf == kCVPixelFormatType_420YpCbCr8BiPlanarVideoRange) {
-    data.pixelFormat = "yuv";
-  } else {
-    data.pixelFormat = "unknown";
-    os_log_error(OS_LOG_DEFAULT,
-        "[StitcherFrame] unexpected ARKit pixel format 0x%x; "
-        "worklet receives pixelFormat='unknown' and toArrayBuffer() "
-        "bytes are first-plane only (layout undefined for unknown "
-        "formats).  See StitcherFrame.ts docstring.", (unsigned int)pf);
-  }
-  // ARKit doesn't have a `Frame.orientation` per se; pose carries
-  // the imageWidth >= imageHeight discriminator the lib uses
-  // elsewhere (`isLandscape`).  v0.8.0 ships a coarse mapping;
-  // worklets that need exact UI orientation can read it from
-  // device-orientation sensors.
-  data.orientation =
-      (pose.imageWidth >= pose.imageHeight) ? "landscape-right" : "portrait";
-  // `ARFrame.timestamp` is CFAbsoluteTime (seconds since epoch).
-  // Convert to ns to match vc Frame.timestamp.
-  data.timestampNs = arFrame.timestamp * 1e9;
-
-  data.qx = pose.qx;
-  data.qy = pose.qy;
-  data.qz = pose.qz;
-  data.qw = pose.qw;
-  data.tx = pose.tx;
-  data.ty = pose.ty;
-  data.tz = pose.tz;
-  data.hasTranslation = true;   // AR mode always has translation
-
-  switch (arFrame.camera.trackingState) {
-    case ARTrackingStateNotAvailable:
-      data.arTrackingState = "notAvailable";
-      break;
-    case ARTrackingStateLimited:
-      data.arTrackingState = "limited";
-      break;
-    case ARTrackingStateNormal:
-      data.arTrackingState = "normal";
-      break;
-  }
-
-  data.pixelReader = std::make_shared<IOSPixelBufferReader>(arFrame);
-
-  // Use the static factory (private ctor enforces shared_ptr
-  // ownership — required for `shared_from_this()` inside the JSI
-  // `toArrayBuffer` lambda).
-  obj->_hostObject =
-      retailens::StitcherFrameJsiHostObject::create(std::move(data));
-  return obj;
-}
-
-- (void)invalidate {
-  if (_hostObject) {
-    _hostObject->invalidate();
-  }
-}
-
-- (void*)jsiHostObjectPtr {
-  if (!_hostObject) return NULL;
-  // Box a heap-allocated copy of the shared_ptr to the abstract
-  // `jsi::HostObject` base.  Caller (worklet runtime) does:
-  //   auto sp = static_cast<std::shared_ptr<jsi::HostObject>*>(ptr);
-  //   auto jsObj = jsi::Object::createFromHostObject(rt, *sp);
-  //   delete sp;
-  return new std::shared_ptr<jsi::HostObject>(_hostObject);
-}
-
-@end

package/ios/Sources/RNImageStitcher/StitcherJsiInstaller.h

@@ -1,42 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// StitcherJsiInstaller.h — RN module that installs the
-// `globalThis.__stitcherProxy` JSI host object on the main JS
-// runtime.  Called once at lib boot from the TS layer
-// (`src/index.ts` or the `useFrameProcessor` hook) via a
-// synchronous JS bridge call.
-//
-// The proxy exposes two host functions:
-//
-//   __stitcherProxy.install(workletFn)  →  string ID
-//   __stitcherProxy.uninstall(id)       →  undefined
-//
-// `install` wraps the worklet function into a
-// `RNWorklet::WorkletInvoker` and stores it in the C++
-// `retailens::StitcherWorkletRegistry` singleton (in
-// `cpp/stitcher_worklet_registry.{hpp,cpp}`).  The AR worklet
-// runtime's per-frame dispatch reads from that registry to fan
-// out invocations.
-//
-// Why a RN module (not a vanilla NSObject installable):
-//   - Hosts can't reliably reach into the JSI runtime from JS
-//     without a native sync method to broker the install.
-//   - `RCT_EXPORT_BLOCKING_SYNCHRONOUS_METHOD` is the documented
-//     pattern for "JS calls a native method synchronously to
-//     install JSI bindings on the main runtime".  vision-camera
-//     uses the same pattern (`VisionCameraInstaller.mm`).
-//   - In RN's bridgeless mode the legacy `RCTCxxBridge.runtime`
-//     accessor still works (vc has a comment to migrate but it
-//     hasn't been needed yet — same applies to us).
-
-#pragma once
-
-#import <Foundation/Foundation.h>
-#import <React/RCTBridgeModule.h>
-
-NS_ASSUME_NONNULL_BEGIN
-
-@interface StitcherJsiInstaller : NSObject <RCTBridgeModule>
-@end
-
-NS_ASSUME_NONNULL_END

package/ios/Sources/RNImageStitcher/StitcherJsiInstaller.mm

@@ -1,160 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-//
-// StitcherJsiInstaller.mm — implementation.  Installs
-// `globalThis.__stitcherProxy` on the main JS runtime.
-//
-// ## Why a host object rather than two globalThis functions
-//
-// We could install `__stitcherProxy_install` + `__stitcherProxy_uninstall`
-// directly on `globalThis`.  Wrapping them in a host object is
-// slightly more code but:
-//   - Namespaces the proxy under a single global property
-//     (easier to feature-detect; one `if (globalThis.__stitcherProxy)`
-//     instead of two).
-//   - Matches vc's pattern (`global.VisionCameraProxy`), so future
-//     readers recognise the shape.
-//   - Keeps room to grow (e.g., add `__stitcherProxy.snapshot()` for
-//     diagnostics) without polluting globalThis further.
-
-#import "StitcherJsiInstaller.h"
-
-#import <Foundation/Foundation.h>
-#import <React/RCTBridge.h>
-#import <React/RCTBridge+Private.h>
-#import <React/RCTUtils.h>
-#import <ReactCommon/CallInvoker.h>
-// `RCTCxxBridge` (and its bridgeless-mode `RCTBridgeProxy` forwarder)
-// exposes `-jsCallInvoker` returning `std::shared_ptr<CallInvoker>`,
-// but the property declaration lives in `<ReactCommon/RCTTurboModule.h>`
-// which isn't on our pod's HEADER_SEARCH_PATHS (worklets-core gets it
-// via its own ReactCommon dep).  Rather than enlarging our pod's
-// dependency surface, forward-declare the property in an anonymous
-// category — the runtime dispatches to RN's actual implementation.
-// Pattern matches `WKTJsiWorkletContext.cpp`'s approach to keep the
-// pod self-contained.
-@interface RCTCxxBridge ()
-@property (nonatomic, readonly) std::shared_ptr<facebook::react::CallInvoker> jsCallInvoker;
-@end
-#import <os/log.h>
-
-#include <jsi/jsi.h>
-
-#include "stitcher_proxy_jsi.hpp"
-// v0.11.1 — worklets-core JsiWorkletContext.  We initialize the
-// SINGLETON default instance here so that other contexts in this
-// library that use the 2-arg `JsiWorkletContext(name, workletInvoker)`
-// constructor inherit a working `_jsCallInvoker` (and thus their
-// `runOnJS` / `Worklets.createRunOnJS` callbacks actually route back
-// to the main JS thread).  Specifically: `RNSARWorkletRuntime`'s AR-
-// side worklet context (see `RNSARWorkletRuntime.mm:155`) uses the
-// 2-arg ctor; pre-v0.11.1 that left its inherited `_jsCallInvoker`
-// nullptr, and `invokeOnJsThread` silently no-op'd (see
-// `WKTJsiWorkletContext.cpp:124-131`).  Test 2 of the v0.11.0
-// manual-verification checklist surfaced this as "AR-mode host
-// worklets register but their runOnJS callbacks never fire."
-#include "WKTJsiWorkletContext.h"
-
-using namespace facebook;
-
-// The host object class + install logic moved to shared C++ in
-// `cpp/stitcher_proxy_jsi.{hpp,cpp}` (v0.8.0 Phase 4b.ii).  The
-// Android JNI installer reuses the same `install` / `uninstall` /
-// `count` host functions verbatim — the JSI dispatch is identical
-// across platforms (matches the StitcherFrame host object's design).
-
-#pragma mark - RN module
-
-@implementation StitcherJsiInstaller
-
-// RN injects `_bridge` at module init (legacy bridge → RCTBridge*;
-// bridgeless / new arch → RCTBridgeProxy*, which forwards `runtime`
-// access via NSProxy `forwardInvocation:`).  Using the injected
-// `_bridge` instead of `[RCTBridge currentBridge]` is the
-// bridgeless-compatible idiom — `currentBridge` is nil under new
-// arch.  Pattern lifted from `react-native-worklets-core/ios/Worklets.mm`.
-@synthesize bridge = _bridge;
-
-RCT_EXPORT_MODULE()
-
-+ (BOOL)requiresMainQueueSetup {
-  return YES;
-}
-
-- (void)setBridge:(RCTBridge*)bridge {
-  _bridge = bridge;
-}
-
-// Synchronous install method.  JS calls this once at lib bootstrap
-// to install the global proxy on the main JS runtime.  Returns
-// `@YES` on success or `@NO` if the JSI runtime wasn't reachable
-// (remote debug mode pre-Hermes; bridge not yet ready; etc.).
-//
-// `RCT_EXPORT_BLOCKING_SYNCHRONOUS_METHOD` is the documented
-// pattern for "run native code synchronously on the JS thread to
-// install JSI bindings."  Same pattern worklets-core + vision-camera
-// use for their installs.
-//
-// **Bridgeless mode:** `_bridge` is an `RCTBridgeProxy` (NSProxy
-// subclass) that forwards `-runtime` / `-jsCallInvoker` invocations
-// to the underlying RCTHost-backed runtime.  The `(RCTCxxBridge*)`
-// cast is a no-op at runtime (NSProxy ignores static type) but
-// keeps the Obj-C compiler happy about property access.
-RCT_EXPORT_BLOCKING_SYNCHRONOUS_METHOD(install) {
-  if (_bridge == nil) {
-    os_log_error(OS_LOG_DEFAULT,
-        "[StitcherJsiInstaller] _bridge is nil; the module was "
-        "instantiated without bridge injection.  Cannot install "
-        "__stitcherProxy.");
-    return @NO;
-  }
-
-  RCTCxxBridge* cxxBridge = (RCTCxxBridge*)_bridge;
-  if (cxxBridge.runtime == nullptr) {
-    os_log_error(OS_LOG_DEFAULT,
-        "[StitcherJsiInstaller] _bridge.runtime is nullptr; the JS "
-        "runtime hasn't been initialized yet OR remote debugger is "
-        "attached.  Cannot install __stitcherProxy.");
-    return @NO;
-  }
-
-  jsi::Runtime& runtime = *(jsi::Runtime*)cxxBridge.runtime;
-  retailens::installStitcherProxy(runtime);
-
-  // v0.11.1 — initialize the singleton default JsiWorkletContext so
-  // that downstream 2-arg ctors (RNSARWorkletRuntime) inherit a
-  // working `_jsCallInvoker`.  Without this, AR-mode host worklets'
-  // `runOnJS` / `Worklets.createRunOnJS` callbacks silently no-op
-  // (`WKTJsiWorkletContext.cpp:124-131` early-returns when
-  // `_jsCallInvoker == nullptr`).  See file-top comment for the full
-  // diagnosis (Test 2 of v0.11.0 manual-verification checklist).
-  //
-  // Idempotent at the worklets-core level: re-initialization is
-  // tolerated; the default instance is a process-scope singleton
-  // and we're called once per JS-runtime bootstrap.  In bridgeless
-  // mode `cxxBridge.jsCallInvoker` is forwarded via RCTBridgeProxy
-  // to the underlying RCTHost's `CallInvoker` (same forwarding
-  // pattern as `cxxBridge.runtime` above).
-  auto jsCallInvoker = cxxBridge.jsCallInvoker;
-  if (jsCallInvoker == nullptr) {
-    os_log_error(OS_LOG_DEFAULT,
-        "[StitcherJsiInstaller] cxxBridge.jsCallInvoker is nullptr; "
-        "AR-mode host worklets' runOnJS will not fire.  Proxy installed "
-        "but worklet-bridging is impaired.");
-    // Proxy is still installed; only the runOnJS path is impaired.
-    // Return @YES so JS callers don't fall back to the JS-side registry.
-    return @YES;
-  }
-  auto jsInvokerAdapter =
-      [jsCallInvoker](std::function<void()>&& fp) {
-        jsCallInvoker->invokeAsync(std::move(fp));
-      };
-  RNWorklet::JsiWorkletContext::getDefaultInstance()->initialize(
-      "stitcher.default", &runtime, jsInvokerAdapter);
-
-  os_log_info(OS_LOG_DEFAULT,
-      "[StitcherJsiInstaller] installed globalThis.__stitcherProxy "
-      "AND initialized default JsiWorkletContext on main JS runtime.");
-  return @YES;
-}
-
-@end

package/src/stitching/IncrementalStitcherView.tsx

@@ -1,198 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-/**
- * IncrementalStitcherView — live preview component for the panorama
- * engine.  Renders the latest snapshot JPEG written by the native
- * side, with confidence + hint overlays.
- *
- * Why <Image> + cache-bust query string instead of a custom native
- * view: per the design doc's open question, the JPEG-write approach
- * is V1; if perf measurements show we're hitting RN's image cache
- * too hard, swap in an `Animated.Image` or a native UIView with
- * an in-memory bitmap.  Until then, the simple path keeps the
- * cross-platform surface tiny.
- */
-
-import React, { useMemo } from 'react';
-import {
-  ActivityIndicator,
-  Image,
-  StyleSheet,
-  Text,
-  View,
-  type ViewStyle,
-} from 'react-native';
-import type { IncrementalState } from './incremental';
-import type { IncrementalHint } from './useIncrementalStitcher';
-
-
-export interface IncrementalStitcherViewProps {
-  /** Latest engine state — typically `useIncrementalStitcher().state`. */
-  state: IncrementalState | null;
-  /**
-   * Active hint to surface as a banner overlay.  Pass
-   * `useIncrementalStitcher().hint` directly; the view picks the
-   * right wording.
-   */
-  hint: IncrementalHint;
-  /**
-   * Confidence ring colour driver — typically
-   * `useIncrementalStitcher().confidenceLevel`.
-   */
-  confidenceLevel?: 'high' | 'medium' | null;
-  /** Outer container style (size, position).  Required: the view
-   *  has no intrinsic size since the panorama dimensions vary. */
-  style?: ViewStyle;
-  /**
-   * Optional override for the spinner shown before the first frame
-   * is accepted.  Default is a subtle "Pan to begin" caption.
-   */
-  emptyText?: string;
-}
-
-
-function hintMessage(hint: IncrementalHint): string | null {
-  switch (hint) {
-    case 'slow-down':         return 'Slow down — alignment lost';
-    case 'scene-uniform':     return 'Pan to a textured area';
-    case 'alignment-lost':    return 'Slow down — re-acquiring alignment';
-    case 'tracking-poor':     return 'Hold steady — AR re-acquiring';
-    default:                  return null;
-  }
-}
-
-
-export function IncrementalStitcherView({
-  state,
-  hint,
-  confidenceLevel,
-  style,
-  emptyText = 'Pan to begin capturing',
-}: IncrementalStitcherViewProps): React.JSX.Element {
-  // Cache-bust the panorama URI.  The native side rotates through
-  // 4 filenames so the path itself changes between snapshots, plus
-  // we tag with acceptedCount as belt-and-suspenders since RN's
-  // image cache on iOS sometimes ignores file:// query strings.
-  const imageUri = useMemo(() => {
-    if (!state?.panoramaPath) return null;
-    return `file://${state.panoramaPath}?v=${state.acceptedCount}`;
-  }, [state?.panoramaPath, state?.acceptedCount]);
-
-  // Use the panorama's NATURAL aspect ratio so the strip widens as
-  // the user pans across.  Falls back to 4:3 (a single frame's
-  // shape) before any snapshot has been written.  Without this the
-  // PiP was forced into a 3:1 letterbox, cropping the actual
-  // panorama to a thin slice across the middle.
-  const naturalAspect = state?.width && state?.height && state.height > 0
-    ? state.width / state.height
-    : 4 / 3;
-
-  const ringColor = confidenceLevel === 'high'
-    ? '#1aaf5d'
-    : confidenceLevel === 'medium'
-      ? '#e6b800'
-      : 'rgba(255,255,255,0.35)';
-
-  const message = hintMessage(hint);
-
-  return (
-    <View style={[styles.container, { aspectRatio: naturalAspect }, style]}>
-      {imageUri ? (
-        // `contain` so the FULL panorama is visible inside the
-        // strip, not cropped to a slice.  Background fills the
-        // letterbox edges.  Key={acceptedCount} forces RN to
-        // remount the Image component each accept — the surest
-        // way to defeat the native image cache on file:// URIs.
-        <Image
-          key={state?.acceptedCount ?? 0}
-          source={{ uri: imageUri }}
-          style={StyleSheet.absoluteFill}
-          resizeMode="contain"
-          fadeDuration={0}
-        />
-      ) : (
-        <View style={styles.empty}>
-          <ActivityIndicator color="#fff" />
-          <Text style={styles.emptyText}>{emptyText}</Text>
-        </View>
-      )}
-
-      {/* Confidence ring — subtle border that picks up colour for
-          medium-confidence accepts.  Always visible (white-translucent
-          when no confidence signal) so the operator can see exactly
-          where the live preview is on screen. */}
-      <View
-        pointerEvents="none"
-        style={[styles.ring, { borderColor: ringColor }]}
-      />
-
-      {message ? (
-        <View pointerEvents="none" style={styles.hintBanner}>
-          <Text style={styles.hintText}>{message}</Text>
-        </View>
-      ) : null}
-
-      {state ? (
-        <View pointerEvents="none" style={styles.counterPill}>
-          <Text style={styles.counterText}>
-            {state.acceptedCount} frame{state.acceptedCount === 1 ? '' : 's'}
-          </Text>
-        </View>
-      ) : null}
-    </View>
-  );
-}
-
-
-const styles = StyleSheet.create({
-  container: {
-    backgroundColor: 'rgba(0,0,0,0.7)',
-    overflow: 'hidden',
-    borderRadius: 8,
-  },
-  empty: {
-    ...StyleSheet.absoluteFillObject,
-    alignItems: 'center',
-    justifyContent: 'center',
-    gap: 6,
-  },
-  emptyText: {
-    color: '#fff',
-    fontSize: 12,
-    opacity: 0.85,
-  },
-  ring: {
-    ...StyleSheet.absoluteFillObject,
-    borderRadius: 8,
-    borderWidth: 2,
-  },
-  hintBanner: {
-    position: 'absolute',
-    left: 8,
-    right: 8,
-    bottom: 8,
-    paddingVertical: 4,
-    paddingHorizontal: 8,
-    backgroundColor: 'rgba(220, 53, 69, 0.92)',
-    borderRadius: 6,
-  },
-  hintText: {
-    color: '#fff',
-    fontSize: 11,
-    textAlign: 'center',
-    fontWeight: '500',
-  },
-  counterPill: {
-    position: 'absolute',
-    top: 6,
-    right: 6,
-    paddingVertical: 2,
-    paddingHorizontal: 8,
-    backgroundColor: 'rgba(0, 0, 0, 0.55)',
-    borderRadius: 10,
-  },
-  counterText: {
-    color: '#fff',
-    fontSize: 10,
-    fontWeight: '500',
-  },
-});