yaver-feedback-react-native 0.9.0 → 0.9.2

package/dist/P2PClient.js

@@ -35,7 +35,16 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.P2PClient = void 0;
 exports.resolveAppIdentity = resolveAppIdentity;
+exports.resolveReportIdentity = resolveReportIdentity;
 const react_native_1 = require("react-native");
+const endpoints_1 = require("./_core/endpoints");
+const reloadActions_1 = require("./reloadActions");
+function unrefTimer(timer) {
+    const maybeNodeTimer = timer;
+    if (typeof maybeNodeTimer.unref === 'function') {
+        maybeNodeTimer.unref();
+    }
+}
 /**
  * Try to resolve `{projectName, bundleId}` for the running app so the
  * agent can map the reload request to a MobileProject in its scan
@@ -84,6 +93,140 @@ function resolveAppIdentity(opts) {
         out.projectPath = projectPath;
     return out;
 }
+/**
+ * Is this JS running inside Yaver's own container as a pushed Hermes guest?
+ *
+ * The YaverInfo native module is registered only by Yaver's app, so its
+ * presence means the ambient runtime describes Yaver rather than the app
+ * whose code is executing. Mirrors the checks in YaverFeedback.ts,
+ * ShakeDetector.ts and QuickActionIcon.tsx.
+ */
+function isInsideYaverContainer() {
+    try {
+        const { NativeModules } = require('react-native');
+        return !!NativeModules?.YaverInfo;
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Read the guest project Yaver's host shell pinned for this bundle.
+ *
+ * When an app's JS is pushed into Yaver's container as a Hermes guest, the
+ * ambient runtime describes the HOST — `expo-constants` and `SettingsManager`
+ * both answer `Yaver` / `io.yaver.mobile`. Auto-resolution would therefore
+ * file every guest report against Yaver's own repo, and the fix task would
+ * edit the SDK instead of the app under test.
+ *
+ * Yaver already knows the right answer: picking a project in the Hot Reload
+ * tab calls `YaverInfo.setInheritedGuestProject(name, path)`. Only the name
+ * is used here — the agent resolves the path itself and deliberately ignores
+ * client-supplied ones on feedback reports.
+ */
+function resolveInheritedGuestProjectName() {
+    try {
+        const { NativeModules } = require('react-native');
+        const name = String(NativeModules?.YaverInfo?.inheritedGuestProjectName || '').trim();
+        return name || undefined;
+    }
+    catch {
+        // Not inside Yaver's container, or react-native unavailable (unit tests).
+        return undefined;
+    }
+}
+/**
+ * Build the app-identity half of a feedback report's metadata.
+ *
+ * `resolveAppIdentity()` has always fed /vibing/execute and /dev/reload-app,
+ * so the agent could route a "vibe on THIS app" request to the right repo —
+ * but nothing fed /feedback. Reports arrived with no identity at all, so the
+ * agent's fix router fell through to its own working directory and edited
+ * whichever repo it happened to be sitting in. This closes that gap by
+ * reusing the same resolver for the feedback path.
+ *
+ * Precedence, most to least trustworthy:
+ *   1. What the host app declared (`FeedbackConfig.projectName`/`bundleId`).
+ *      An app knows its own identity; nothing should override it.
+ *   2. The guest project Yaver's host shell pinned, when running as a Hermes
+ *      guest. Ambient lookups describe Yaver there, not the guest.
+ *   3. Ambient expo-constants / native modules — correct for a standalone
+ *      build, which is the common case.
+ *
+ * Every lookup is best-effort: a bare RN app with no expo-constants still
+ * produces a report, just one the agent resolves by its own means.
+ */
+function resolveReportIdentity(opts) {
+    let projectName = (opts?.projectName || '').trim() || undefined;
+    let bundleId = (opts?.bundleId || '').trim() || undefined;
+    let version;
+    let buildNumber;
+    if (isInsideYaverContainer()) {
+        // Every ambient lookup answers for the HOST here. Because the agent
+        // routes on bundle id first, letting Yaver's id through would resolve
+        // straight to Yaver's own repo and the guest's name would never be
+        // consulted — so nothing ambient is trusted in the container. The
+        // version is Yaver's too, and reporting it would only mislabel which
+        // build the report came from.
+        projectName = projectName || resolveInheritedGuestProjectName();
+    }
+    else {
+        const ambient = resolveAppIdentity({ projectName, bundleId });
+        projectName = ambient.projectName;
+        bundleId = ambient.bundleId;
+        try {
+            const Constants = require('expo-constants').default ?? require('expo-constants');
+            const cfg = Constants?.expoConfig ?? Constants?.manifest ?? {};
+            version = Constants?.nativeAppVersion || cfg?.version;
+            buildNumber =
+                Constants?.nativeBuildVersion ||
+                    cfg?.ios?.buildNumber ||
+                    (cfg?.android?.versionCode != null ? String(cfg.android.versionCode) : undefined);
+        }
+        catch {
+            // expo-constants not installed (bare RN). Version is cosmetic — the
+            // router keys off appName/bundleId, both of which survive without it.
+        }
+    }
+    const app = {};
+    if (bundleId)
+        app.bundleId = bundleId;
+    if (version)
+        app.version = version;
+    if (buildNumber)
+        app.buildNumber = buildNumber;
+    if (!projectName && !bundleId) {
+        return { app };
+    }
+    const project = { surface: opts?.surface ?? 'mobile' };
+    if (projectName) {
+        project.projectName = projectName;
+        project.appName = projectName;
+    }
+    // Note: projectPath is intentionally omitted — the agent ignores
+    // client-supplied paths on feedback reports and resolves them itself.
+    if (bundleId)
+        project.bundleId = bundleId;
+    if (opts?.surfaces)
+        project.surfaces = opts.surfaces;
+    if (opts?.stack)
+        project.stack = opts.stack;
+    if (opts?.stacks)
+        project.stacks = opts.stacks;
+    if (opts?.testSurfaces)
+        project.testSurfaces = opts.testSurfaces;
+    if (opts?.feedbackSdk)
+        project.feedbackSdk = opts.feedbackSdk;
+    if (opts?.feedbackTransport)
+        project.feedbackTransport = opts.feedbackTransport;
+    if (opts?.voiceCapabilities)
+        project.voiceCapabilities = opts.voiceCapabilities;
+    if (opts?.sttProvider)
+        project.sttProvider = opts.sttProvider;
+    if (opts?.ttsProvider)
+        project.ttsProvider = opts.ttsProvider;
+    return { appName: projectName, app, project };
+}
 /**
  * Translate a raw Go-agent error into something a user can act on.
  *
@@ -327,19 +470,24 @@ class P2PClient {
     }
     /** Health check — returns true if the agent is reachable. */
     async health() {
+        let timeoutId = null;
         try {
             const controller = new AbortController();
-            const timeoutId = setTimeout(() => controller.abort(), 3000);
+            timeoutId = setTimeout(() => controller.abort(), 3000);
+            unrefTimer(timeoutId);
             const response = await fetch(`${this.baseUrl}/health`, {
                 method: 'GET',
                 signal: controller.signal,
             });
-            clearTimeout(timeoutId);
             return response.ok;
         }
         catch {
             return false;
         }
+        finally {
+            if (timeoutId)
+                clearTimeout(timeoutId);
+        }
     }
     /** Get agent info (hostname, version, platform). */
     async info() {
@@ -488,6 +636,78 @@ class P2PClient {
      * via the BlackBox command channel.
      * @param mode - "dev" for hot reload, "bundle" for native bundle rebuild
      */
+    /**
+     * Read the dev server's state so the overlay can decide WHICH reload
+     * actions to offer, and disable the rest with a reason.
+     *
+     * Returns null when the machine cannot be reached at all — which the
+     * caller must render as "not connected", never as "no dev server".
+     * Those are two different problems with two different fixes.
+     */
+    async getDevServerStatus() {
+        try {
+            const resp = await fetch(`${this.baseUrl}${endpoints_1.AGENT_ENDPOINTS.devStatus}`, {
+                headers: this.authHeaders(),
+            });
+            if (!resp.ok)
+                return null;
+            const data = await resp.json().catch(() => ({}));
+            return {
+                running: data.running === true,
+                building: data.building === true,
+                framework: typeof data.framework === 'string' ? data.framework : undefined,
+            };
+        }
+        catch {
+            return null;
+        }
+    }
+    /**
+     * Trigger a reload with an EXPLICIT fast/full mode — no bundle fallback.
+     *
+     * `reloadApp('dev')` silently falls through to a bundle rebuild when the
+     * dev server is down, which is right for a one-button UX and wrong the
+     * moment the user picks between two named actions: someone who pressed
+     * "Full Reload" must not get a 60-second bundle rebuild without being
+     * told. So this method reports the failure instead, with a named cause.
+     *
+     * Auth: the SAME bearer used for the feedback POST. `/dev/reload` is
+     * registered under `authSDKOrGuest` in desktop/agent/httpserver.go and is
+     * already in the `guest-reload` SDK-token scope list — nothing widens.
+     */
+    async reloadWithMode(mode, snapshot) {
+        if (mode === 'bundle')
+            return this.reloadApp('bundle');
+        let resp;
+        try {
+            resp = await fetch(`${this.baseUrl}${reloadActions_1.RELOAD_PATH}`, {
+                method: 'POST',
+                headers: this.authHeaders({ 'Content-Type': 'application/json' }),
+                body: JSON.stringify({ mode }),
+            });
+        }
+        catch (err) {
+            throw new Error((0, reloadActions_1.describeReloadFailure)(0, err instanceof Error ? err.message : '', snapshot));
+        }
+        if (!resp.ok) {
+            const text = await resp.text().catch(() => '');
+            throw new Error((0, reloadActions_1.describeReloadFailure)(resp.status, text, snapshot));
+        }
+        const payload = await resp.json().catch(() => ({}));
+        const nativeChangesDetected = payload.nativeChangesDetected === true;
+        return {
+            ok: true,
+            mode: 'dev',
+            acknowledged: true,
+            nativeChangesDetected,
+            changeClass: typeof payload.changeClass === 'string' ? payload.changeClass : undefined,
+            message: nativeChangesDetected
+                ? 'Reload accepted, but native files changed — a rebuild is required.'
+                : mode === 'full'
+                    ? 'Full reload requested.'
+                    : 'Hot reload requested.',
+        };
+    }
     async reloadApp(mode = 'bundle', opts) {
         // Default path: always rebuild a fresh Hermes bundle.
         //

package/dist/ShakeDetector.js

@@ -84,6 +84,8 @@ class ShakeDetector {
      * the `ShakeEvent` name a handful of third-party shake libraries emit.
      */
     subscribeDevMenu(onShake) {
+        if (typeof react_native_1.DeviceEventEmitter?.addListener !== 'function')
+            return;
         const eventName = react_native_1.Platform.OS === 'ios' ? 'shakeEvent' : 'ShakeEvent';
         this.devMenuSub = react_native_1.DeviceEventEmitter.addListener(eventName, () => {
             this.fire(onShake);

package/dist/YaverFeedback.d.ts

@@ -1,5 +1,6 @@
 import { FeedbackConfig, CapturedError } from './types';
 import { P2PClient } from './P2PClient';
+import { CaptureStoreScreenshotsOptions, CaptureStoreScreenshotsResult } from './storeShots';
 import { QuickIconColorPreset } from './preferences';
 /**
  * Main entry point for the Yaver Feedback SDK.
@@ -15,7 +16,26 @@ export declare class YaverFeedback {
      * If no `agentUrl` is provided, the SDK will attempt auto-discovery
      * via `YaverDiscovery` on the first `startReport()` call.
      */
+    /** The runtime lane, when running on web inside a Yaver preview WebView. */
+    static detectWebLane(): 'browser' | 'webrtc' | null;
+    /**
+     * Mount a draggable DOM floating "Y" icon for the BROWSER lane (RN-web). It
+     * lives in the WebView's document.body (position:fixed, max z-index), so it
+     * renders ON TOP of the page inside the WebView and is never occluded by the
+     * fullScreen preview modal — the one occlusion-proof affordance on iOS.
+     * Tap opens the SDK's existing report flow (the RN FeedbackModal renders in
+     * the same WebView DOM, also un-occluded). No-op off-web or off-lane, and
+     * idempotent (a re-init reuses the existing node).
+     */
+    static mountBrowserLaneIcon(): void;
     static init(cfg: FeedbackConfig): void;
+    /**
+     * Opt-in global crash handler. This wraps the existing React Native
+     * ErrorUtils handler and still calls it, so Sentry/Crashlytics/Bugsnag can
+     * remain the system of record. The SDK only uploads a crash report and,
+     * when configured, asks the agent to create a fix task.
+     */
+    static installCrashHandler(): void;
     /**
      * Run agent discovery in the background.
      * Called automatically from init() when no agentUrl is provided.
@@ -46,6 +66,29 @@ export declare class YaverFeedback {
     static setAuthToken(token: string): Promise<void>;
     /** Returns true once the SDK has a session token it can use. */
     static isAuthed(): boolean;
+    /**
+     * On-device App Store screenshot capture (Engine 2). Walks the routes
+     * the host hands us, screenshots each, and uploads to the agent which
+     * runs the App Store Connect backend. agentUrl / authToken / relay
+     * password default to the SDK's resolved session; the host only has to
+     * supply a navigation ref + the ordered route list.
+     *
+     *   YaverFeedback.captureStoreScreenshots({
+     *     app: 'sfmg',
+     *     navigationRef,
+     *     routes: ['/(tabs)/dashboard', '/(tabs)/messages', '/(tabs)/clients'],
+     *     submit: true,
+     *   })
+     */
+    static captureStoreScreenshots(opts: Omit<CaptureStoreScreenshotsOptions, 'agentUrl' | 'authToken' | 'relayPassword'> & Partial<Pick<CaptureStoreScreenshotsOptions, 'agentUrl' | 'authToken' | 'relayPassword'>>): Promise<CaptureStoreScreenshotsResult>;
+    /**
+     * Let the mobile app / CLI kick THIS device into self-capturing. Registers
+     * a BlackBox command listener that fires captureStoreScreenshots when the
+     * agent pushes a `capture_store_shots` command (its `data` may override
+     * `submit` / `locale`). Returns an unsubscribe fn. Call once after init,
+     * passing the app's navigation ref + the routes to walk.
+     */
+    static enableStoreShotsOnCommand(opts: Omit<CaptureStoreScreenshotsOptions, 'agentUrl' | 'authToken' | 'relayPassword'>): () => void;
     /**
      * Request the embedded FeedbackModal to show the login screen. Works by
      * emitting an event the modal listens for — avoids forcing the host app
@@ -112,6 +155,46 @@ export declare class YaverFeedback {
      * - All methods become active
      */
     static setEnabled(value: boolean): void;
+    /**
+     * Turn shake-to-report on or off without tearing the SDK down.
+     *
+     * `trigger` is read only inside init(), so before this the only ways to
+     * stop listening for a shake were setEnabled(false) — which also kills the
+     * flight recorder and the agent command channel — or a re-init, which used
+     * to stack a duplicate command handler. Neither is what "turn the shake
+     * catcher off" should cost.
+     *
+     * Persists onto config.disableShakeGesture, so a later setEnabled(true)
+     * honours it rather than resurrecting the listener.
+     */
+    static setShakeEnabled(value: boolean): void;
+    /** Whether the shake listener is currently armed. */
+    static isShakeEnabled(): boolean;
+    /**
+     * Bring the shake listener in line with the current config. Idempotent —
+     * safe to call whenever `enabled`, `trigger`, or `disableShakeGesture`
+     * moves.
+     */
+    private static syncShakeDetector;
+    private static stopShakeDetector;
+    /** Cancel a pending BlackBox auto-start retry chain. */
+    private static cancelBlackBoxAutoStart;
+    /**
+     * Start BlackBox as soon as we have BOTH an agentUrl and a token, polling
+     * until then.
+     *
+     * Both conditions are mandatory and deliberate: starting without a token
+     * makes the SSE channel 401 and retry with backoff, which is the tight
+     * string-concat + JSON-parse loop that used to SIGSEGV Hermes on iOS 18.3.1
+     * during Screenshot & Fix. This only re-checks that same guard over time.
+     *
+     * Bounded at ~60s: long enough for AsyncStorage hydration plus a Convex
+     * discovery round trip on a cold, off-LAN start, short enough that a device
+     * whose user never signs in stops polling. Giving up here costs nothing —
+     * setAuthToken() and startReport() both reschedule, so signing in later
+     * still brings the channel up.
+     */
+    private static scheduleBlackBoxAutoStart;
     /** Returns whether the SDK is currently enabled. */
     static isEnabled(): boolean;
     /** Returns the current config, or null if not initialized. */
@@ -150,6 +233,21 @@ export declare class YaverFeedback {
      * // wrapper, and the chain stays intact.
      */
     static wrapErrorHandler(next?: ((error: Error, isFatal?: boolean) => void) | null): (error: Error, isFatal?: boolean) => void;
+    /**
+     * Upload a crash-aware feedback bundle. Apps can call this from their own
+     * error boundary/global handler. If crashReporting.autoFix is true, the SDK
+     * also triggers the agent's feedback-fix task; reload delivery still happens
+     * through the existing BlackBox command stream.
+     */
+    static reportCrash(error: Error, opts?: {
+        isFatal?: boolean;
+        source?: 'manual' | 'global-handler';
+        metadata?: Record<string, unknown>;
+        autoFix?: boolean;
+    }): Promise<{
+        reportId?: string;
+        taskId?: string;
+    }>;
     /**
      * Returns the P2P client instance.
      * Available after init if agentUrl is set, or after first successful discovery.