yaver-feedback-react-native 0.7.4 → 0.7.6

package/dist/FeedbackModal.js

@@ -58,9 +58,22 @@ const FeedbackModal = () => {
                 setAction('idle');
             }
         });
+        // Agent streams build / compile progress through the BlackBox
+        // SSE command channel as `command: "status"`; YaverFeedback re-emits
+        // it as `yaverFeedback:status`. Show the most recent message in the
+        // toast so a multi-second rebuild feels like "working" instead of
+        // "stuck".
+        const statusSub = react_native_1.DeviceEventEmitter.addListener('yaverFeedback:status', (payload) => {
+            if (!mountedRef.current)
+                return;
+            const msg = payload?.message || payload?.phase || '';
+            if (msg)
+                setToast(msg);
+        });
         return () => {
             mountedRef.current = false;
             sub.remove();
+            statusSub.remove();
         };
     }, []);
     const closeSoon = (0, react_1.useCallback)((delayMs = 1200) => {

package/dist/P2PClient.js

@@ -2,6 +2,34 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.P2PClient = void 0;
 const react_native_1 = require("react-native");
+/**
+ * Translate a raw Go-agent error into something a user can act on.
+ *
+ * The agent surfaces Go's low-level error text verbatim inside JSON,
+ * e.g. `Get "http://127.0.0.1:8081/reload": dial tcp 127.0.0.1:8081:
+ * connect: connection refused`. That's accurate but unreadable for a
+ * phone user and — more importantly — leads them to think the SDK is
+ * misconfigured. The real cause is almost always "no dev server
+ * running on the host machine".
+ */
+function friendlyReloadError(status, body) {
+    const lower = body.toLowerCase();
+    if (/connection refused|econnrefused/.test(lower) &&
+        /127\.0\.0\.1|localhost/.test(lower)) {
+        return ('No dev server running on your machine. ' +
+            'Start Metro with `yaver dev start` or use "Screenshot & Fix" instead.');
+    }
+    if (/no dev server/.test(lower) || /not running/.test(lower)) {
+        return 'No dev server running on your machine. Start Metro first.';
+    }
+    if (status === 403)
+        return 'Agent rejected the session — please sign in again.';
+    if (status === 401)
+        return 'Agent rejected the session — please sign in again.';
+    if (status >= 500)
+        return 'Agent hit an internal error while reloading. Check `yaver logs`.';
+    return `Reload failed (${status}).`;
+}
 /**
  * Lightweight P2P HTTP client for communicating with a Yaver agent.
  *
@@ -180,39 +208,46 @@ class P2PClient {
      * via the BlackBox command channel.
      * @param mode - "dev" for hot reload, "bundle" for native bundle rebuild
      */
-    async reloadApp(mode = 'dev') {
-        // Primary path: /dev/reload — same endpoint the Yaver mobile app uses.
-        // Triggers Metro/Expo HMR synchronously and emits an SSE `reload` event
-        // on /dev/events that the FeedbackModal can subscribe to for progress.
+    async reloadApp(mode = 'bundle') {
+        // Default path: always rebuild a fresh Hermes bundle.
         //
-        // Only fall back to /dev/reload-app (the BlackBox-SSE-broadcast path)
-        // when the primary path reports "no dev server running" — that mode is
-        // really for the mobile app remotely kicking a third-party app, not
-        // for the app kicking itself.
-        const primary = await fetch(`${this.baseUrl}/dev/reload`, {
-            method: 'POST',
-            headers: { Authorization: `Bearer ${this.authToken}` },
-        });
-        if (primary.ok) {
-            return primary.json().catch(() => ({ ok: true }));
-        }
-        if (primary.status >= 500 || primary.status === 404 || mode === 'bundle') {
-            const fallback = await fetch(`${this.baseUrl}/dev/reload-app`, {
+        // Rationale: the SDK's common caller is a phone user who's not
+        // sitting at their Mac — they're doing vibe coding with an AI agent
+        // editing files on the Mac remotely, or they installed the app via
+        // TestFlight and there's no Metro running at all. A Metro-based
+        // reload would either fail (Metro offline) or — worse — serve a
+        // stale bundle because Metro can be slow to re-index fresh edits.
+        // Bundle mode always produces the correct bytecode from the current
+        // filesystem state, taking ~30–60 s, and hits /dev/reload-app which
+        // uses BlackBox SSE to push the fresh bundle URL to the device.
+        //
+        // Callers who know they want Metro HMR pass `mode='dev'` explicitly;
+        // we still honour that. Everything else defaults to bundle.
+        if (mode === 'dev') {
+            const primary = await fetch(`${this.baseUrl}/dev/reload`, {
                 method: 'POST',
-                headers: {
-                    Authorization: `Bearer ${this.authToken}`,
-                    'Content-Type': 'application/json',
-                },
-                body: JSON.stringify({ mode }),
+                headers: { Authorization: `Bearer ${this.authToken}` },
             });
-            if (!fallback.ok) {
-                const text = await fallback.text().catch(() => '');
-                throw new Error(`[P2PClient] Reload failed (${fallback.status}): ${text}`);
+            if (primary.ok) {
+                return primary.json().catch(() => ({ ok: true }));
             }
-            return fallback.json().catch(() => ({ ok: true }));
+            // Dev mode failed — fall through to bundle rebuild below rather
+            // than surfacing the raw error, so the user never has to know
+            // Metro wasn't running.
+        }
+        const res = await fetch(`${this.baseUrl}/dev/reload-app`, {
+            method: 'POST',
+            headers: {
+                Authorization: `Bearer ${this.authToken}`,
+                'Content-Type': 'application/json',
+            },
+            body: JSON.stringify({ mode: 'bundle' }),
+        });
+        if (!res.ok) {
+            const text = await res.text().catch(() => '');
+            throw new Error(friendlyReloadError(res.status, text));
         }
-        const text = await primary.text().catch(() => '');
-        throw new Error(`[P2PClient] Reload failed (${primary.status}): ${text}`);
+        return res.json().catch(() => ({ ok: true }));
     }
     /**
      * Open a vibing session on the connected agent. Vibing is the Yaver

package/dist/YaverFeedback.js

@@ -126,9 +126,13 @@ class YaverFeedback {
                 }
             });
         }
-        // Wire up BlackBox command handlers for reload signals from the agent.
-        // This enables the vibe coder to trigger reload from the Yaver mobile app
-        // and have the third-party app (with this SDK) automatically reload.
+        // Wire up BlackBox command handlers for reload + status signals from
+        // the agent. Opens an SSE channel the agent uses to:
+        //   - broadcast reload_bundle so the phone picks up a fresh Hermes
+        //     bundle after a vibe-coding edit;
+        //   - stream build/compile progress ("Compiling bundle…", "Pushing
+        //     assets…", "Done") so the SDK can surface it like a normal
+        //     "Working…" spinner instead of a silent 60-second freeze.
         if (enabled) {
             BlackBox_1.BlackBox.onCommand((cmd) => {
                 if (cmd.command === 'reload') {
@@ -149,7 +153,37 @@ class YaverFeedback {
                         YaverFeedback.defaultReloadBundle(bundleUrl, assetsUrl);
                     }
                 }
+                else if (cmd.command === 'status') {
+                    // Pipe agent progress pings to the UI. The FeedbackModal
+                    // subscribes to this event and renders the message while a
+                    // reload / build is in flight.
+                    const message = typeof cmd.data?.message === 'string'
+                        ? cmd.data.message
+                        : '';
+                    const phase = typeof cmd.data?.phase === 'string'
+                        ? cmd.data.phase
+                        : '';
+                    const { DeviceEventEmitter } = require('react-native');
+                    DeviceEventEmitter.emit('yaverFeedback:status', {
+                        message,
+                        phase,
+                        at: Date.now(),
+                    });
+                }
             });
+            // Auto-open the command-stream SSE channel so `status` +
+            // `reload_bundle` messages from the agent actually reach us.
+            // Host apps can still call BlackBox.start(...) themselves to
+            // customise deviceId / appName, but they don't have to.
+            if (!BlackBox_1.BlackBox.isStreaming) {
+                try {
+                    BlackBox_1.BlackBox.start();
+                }
+                catch {
+                    // BlackBox.start throws if agentUrl/token aren't ready yet;
+                    // a later setAuthToken() / discoverAgent() tick will retry.
+                }
+            }
         }
         // NOTE: We intentionally do NOT hook ErrorUtils.setGlobalHandler().
         // Sentry, Crashlytics, Bugsnag, and other tools all compete for that

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "yaver-feedback-react-native",
-  "version": "0.7.4",
+  "version": "0.7.6",
   "description": "Visual feedback SDK for Yaver — bug reports, screen recording, voice annotations, and local-first developer workflows",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/FeedbackModal.tsx

@@ -67,9 +67,23 @@ export const FeedbackModal: React.FC = () => {
         setAction('idle');
       }
     });
+    // Agent streams build / compile progress through the BlackBox
+    // SSE command channel as `command: "status"`; YaverFeedback re-emits
+    // it as `yaverFeedback:status`. Show the most recent message in the
+    // toast so a multi-second rebuild feels like "working" instead of
+    // "stuck".
+    const statusSub = DeviceEventEmitter.addListener(
+      'yaverFeedback:status',
+      (payload: { message?: string; phase?: string }) => {
+        if (!mountedRef.current) return;
+        const msg = payload?.message || payload?.phase || '';
+        if (msg) setToast(msg);
+      },
+    );
     return () => {
       mountedRef.current = false;
       sub.remove();
+      statusSub.remove();
     };
   }, []);
 

package/src/P2PClient.ts

@@ -7,6 +7,36 @@ export interface FeedbackEvent {
   data: any;
 }
 
+/**
+ * Translate a raw Go-agent error into something a user can act on.
+ *
+ * The agent surfaces Go's low-level error text verbatim inside JSON,
+ * e.g. `Get "http://127.0.0.1:8081/reload": dial tcp 127.0.0.1:8081:
+ * connect: connection refused`. That's accurate but unreadable for a
+ * phone user and — more importantly — leads them to think the SDK is
+ * misconfigured. The real cause is almost always "no dev server
+ * running on the host machine".
+ */
+function friendlyReloadError(status: number, body: string): string {
+  const lower = body.toLowerCase();
+  if (
+    /connection refused|econnrefused/.test(lower) &&
+    /127\.0\.0\.1|localhost/.test(lower)
+  ) {
+    return (
+      'No dev server running on your machine. ' +
+      'Start Metro with `yaver dev start` or use "Screenshot & Fix" instead.'
+    );
+  }
+  if (/no dev server/.test(lower) || /not running/.test(lower)) {
+    return 'No dev server running on your machine. Start Metro first.';
+  }
+  if (status === 403) return 'Agent rejected the session — please sign in again.';
+  if (status === 401) return 'Agent rejected the session — please sign in again.';
+  if (status >= 500) return 'Agent hit an internal error while reloading. Check `yaver logs`.';
+  return `Reload failed (${status}).`;
+}
+
 /**
  * Lightweight P2P HTTP client for communicating with a Yaver agent.
  *
@@ -214,39 +244,47 @@ export class P2PClient {
    * via the BlackBox command channel.
    * @param mode - "dev" for hot reload, "bundle" for native bundle rebuild
    */
-  async reloadApp(mode: 'dev' | 'bundle' = 'dev'): Promise<{ ok: boolean }> {
-    // Primary path: /dev/reload — same endpoint the Yaver mobile app uses.
-    // Triggers Metro/Expo HMR synchronously and emits an SSE `reload` event
-    // on /dev/events that the FeedbackModal can subscribe to for progress.
+  async reloadApp(mode: 'dev' | 'bundle' = 'bundle'): Promise<{ ok: boolean }> {
+    // Default path: always rebuild a fresh Hermes bundle.
     //
-    // Only fall back to /dev/reload-app (the BlackBox-SSE-broadcast path)
-    // when the primary path reports "no dev server running" — that mode is
-    // really for the mobile app remotely kicking a third-party app, not
-    // for the app kicking itself.
-    const primary = await fetch(`${this.baseUrl}/dev/reload`, {
-      method: 'POST',
-      headers: { Authorization: `Bearer ${this.authToken}` },
-    });
-    if (primary.ok) {
-      return primary.json().catch(() => ({ ok: true }));
-    }
-    if (primary.status >= 500 || primary.status === 404 || mode === 'bundle') {
-      const fallback = await fetch(`${this.baseUrl}/dev/reload-app`, {
+    // Rationale: the SDK's common caller is a phone user who's not
+    // sitting at their Mac — they're doing vibe coding with an AI agent
+    // editing files on the Mac remotely, or they installed the app via
+    // TestFlight and there's no Metro running at all. A Metro-based
+    // reload would either fail (Metro offline) or — worse — serve a
+    // stale bundle because Metro can be slow to re-index fresh edits.
+    // Bundle mode always produces the correct bytecode from the current
+    // filesystem state, taking ~30–60 s, and hits /dev/reload-app which
+    // uses BlackBox SSE to push the fresh bundle URL to the device.
+    //
+    // Callers who know they want Metro HMR pass `mode='dev'` explicitly;
+    // we still honour that. Everything else defaults to bundle.
+    if (mode === 'dev') {
+      const primary = await fetch(`${this.baseUrl}/dev/reload`, {
         method: 'POST',
-        headers: {
-          Authorization: `Bearer ${this.authToken}`,
-          'Content-Type': 'application/json',
-        },
-        body: JSON.stringify({ mode }),
+        headers: { Authorization: `Bearer ${this.authToken}` },
       });
-      if (!fallback.ok) {
-        const text = await fallback.text().catch(() => '');
-        throw new Error(`[P2PClient] Reload failed (${fallback.status}): ${text}`);
+      if (primary.ok) {
+        return primary.json().catch(() => ({ ok: true }));
       }
-      return fallback.json().catch(() => ({ ok: true }));
+      // Dev mode failed — fall through to bundle rebuild below rather
+      // than surfacing the raw error, so the user never has to know
+      // Metro wasn't running.
+    }
+
+    const res = await fetch(`${this.baseUrl}/dev/reload-app`, {
+      method: 'POST',
+      headers: {
+        Authorization: `Bearer ${this.authToken}`,
+        'Content-Type': 'application/json',
+      },
+      body: JSON.stringify({ mode: 'bundle' }),
+    });
+    if (!res.ok) {
+      const text = await res.text().catch(() => '');
+      throw new Error(friendlyReloadError(res.status, text));
     }
-    const text = await primary.text().catch(() => '');
-    throw new Error(`[P2PClient] Reload failed (${primary.status}): ${text}`);
+    return res.json().catch(() => ({ ok: true }));
   }
 
   /**

package/src/YaverFeedback.ts

@@ -142,9 +142,13 @@ export class YaverFeedback {
       });
     }
 
-    // Wire up BlackBox command handlers for reload signals from the agent.
-    // This enables the vibe coder to trigger reload from the Yaver mobile app
-    // and have the third-party app (with this SDK) automatically reload.
+    // Wire up BlackBox command handlers for reload + status signals from
+    // the agent. Opens an SSE channel the agent uses to:
+    //   - broadcast reload_bundle so the phone picks up a fresh Hermes
+    //     bundle after a vibe-coding edit;
+    //   - stream build/compile progress ("Compiling bundle…", "Pushing
+    //     assets…", "Done") so the SDK can surface it like a normal
+    //     "Working…" spinner instead of a silent 60-second freeze.
     if (enabled) {
       BlackBox.onCommand((cmd) => {
         if (cmd.command === 'reload') {
@@ -161,8 +165,38 @@ export class YaverFeedback {
           } else {
             YaverFeedback.defaultReloadBundle(bundleUrl, assetsUrl);
           }
+        } else if (cmd.command === 'status') {
+          // Pipe agent progress pings to the UI. The FeedbackModal
+          // subscribes to this event and renders the message while a
+          // reload / build is in flight.
+          const message =
+            typeof cmd.data?.message === 'string'
+              ? (cmd.data.message as string)
+              : '';
+          const phase =
+            typeof cmd.data?.phase === 'string'
+              ? (cmd.data.phase as string)
+              : '';
+          const { DeviceEventEmitter } = require('react-native');
+          DeviceEventEmitter.emit('yaverFeedback:status', {
+            message,
+            phase,
+            at: Date.now(),
+          });
         }
       });
+      // Auto-open the command-stream SSE channel so `status` +
+      // `reload_bundle` messages from the agent actually reach us.
+      // Host apps can still call BlackBox.start(...) themselves to
+      // customise deviceId / appName, but they don't have to.
+      if (!BlackBox.isStreaming) {
+        try {
+          BlackBox.start();
+        } catch {
+          // BlackBox.start throws if agentUrl/token aren't ready yet;
+          // a later setAuthToken() / discoverAgent() tick will retry.
+        }
+      }
     }
 
     // NOTE: We intentionally do NOT hook ErrorUtils.setGlobalHandler().