yaver-feedback-react-native 0.7.6 → 0.7.7

package/dist/FeedbackModal.js

@@ -105,18 +105,27 @@ const FeedbackModal = () => {
             await fn(client);
         }
         catch (err) {
-            const msg = err instanceof Error ? err.message : String(err);
-            // 401 / 403 "invalid token" — the cached session is stale
-            // (common after a Convex deployment migration). Sign out so
-            // the next interaction surfaces the login sheet, then bubble
-            // a readable error up to the user.
-            const authFailed = /\b(401|403)\b.*invalid token|invalid token|unauthor/i.test(msg);
+            const msg = (err instanceof Error ? err.message : String(err)) || '';
+            // Avoid unbounded `.*` in regex — on RN 0.81 / Hermes rope
+            // strings plus a background SSE reconnect, that pattern has
+            // reliably SIGSEGV'd Hermes's string-view flattening path.
+            // Split into short, literal-only alternations.
+            const lower = msg.toLowerCase();
+            const authFailed = lower.indexOf('invalid token') >= 0 ||
+                lower.indexOf('unauthor') >= 0 ||
+                lower.indexOf(' 401') >= 0 ||
+                lower.indexOf(' 403') >= 0;
             if (authFailed) {
                 await YaverFeedback_1.YaverFeedback.signOut();
                 YaverFeedback_1.YaverFeedback.showLogin();
                 throw new Error('Session expired — please sign in again.');
             }
-            const transient = /Network request failed|timeout|ECONNREFUSED|Failed to fetch|fetch failed|aborted/i.test(msg);
+            const transient = lower.indexOf('network request failed') >= 0 ||
+                lower.indexOf('econnrefused') >= 0 ||
+                lower.indexOf('failed to fetch') >= 0 ||
+                lower.indexOf('fetch failed') >= 0 ||
+                lower.indexOf('aborted') >= 0 ||
+                lower.indexOf('timeout') >= 0;
             if (!transient)
                 throw err;
             const ok = await YaverFeedback_1.YaverFeedback.reconnect();

package/dist/MachinePickerScreen.js

@@ -94,15 +94,33 @@ const YaverMachinePickerScreen = ({ token, currentDeviceId, onPick, onCancel, })
             : device.needsAuth
                 ? '#f59e0b'
                 : '#22c55e';
+        // Derive a single short status phrase the user can act on.
+        let statusLine = device.platform;
+        if (!device.isOnline) {
+            statusLine = 'Offline — start `yaver serve` on the Mac';
+        }
+        else if (device.needsAuth) {
+            statusLine =
+                'Needs pairing — open the Yaver app to adopt this machine';
+        }
+        else if (device.runnerDown) {
+            statusLine = 'Runner down — restart the coding agent on the Mac';
+        }
+        else {
+            // Happy-path subtitle: platform + optional host/share hint.
+            statusLine = device.platform;
+            if (device.isGuest && device.hostEmail) {
+                statusLine = `${statusLine} • ${device.hostEmail}`;
+            }
+            else if (device.accessScope === 'shared-scoped') {
+                statusLine = `${statusLine} • paylaşılan`;
+            }
+        }
         return (<react_native_1.TouchableOpacity key={device.deviceId} style={[styles.deviceRow, selected && styles.deviceSelected]} onPress={() => handlePick(device)}>
         <react_native_1.View style={[styles.health, { backgroundColor: healthColor }]}/>
         <react_native_1.View style={{ flex: 1 }}>
           <react_native_1.Text style={styles.deviceName}>{device.name || device.deviceId}</react_native_1.Text>
-          <react_native_1.Text style={styles.deviceMeta}>
-            {device.platform}
-            {device.isGuest && device.hostEmail ? `  •  ${device.hostEmail}` : ''}
-            {device.accessScope === 'shared-scoped' ? '  •  paylaşılan' : ''}
-          </react_native_1.Text>
+          <react_native_1.Text style={styles.deviceMeta}>{statusLine}</react_native_1.Text>
         </react_native_1.View>
         {selected && <react_native_1.Text style={styles.selectedBadge}>seçili</react_native_1.Text>}
       </react_native_1.TouchableOpacity>);

package/dist/P2PClient.js

@@ -13,21 +13,28 @@ const react_native_1 = require("react-native");
  * 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)) {
+    // Plain .indexOf instead of regex — short literal tests sidestep
+    // a Hermes rope-flatten SIGSEGV we saw on RN 0.81 / iOS 18.3.1
+    // when the same body was already being processed by a concurrent
+    // SSE reconnect loop.
+    const lower = (body || '').toLowerCase();
+    const hasRefused = lower.indexOf('connection refused') >= 0 ||
+        lower.indexOf('econnrefused') >= 0;
+    const hasLoopback = lower.indexOf('127.0.0.1') >= 0 || lower.indexOf('localhost') >= 0;
+    if (hasRefused && hasLoopback) {
         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)) {
+    if (lower.indexOf('no dev server') >= 0 ||
+        lower.indexOf('not running') >= 0) {
         return 'No dev server running on your machine. Start Metro first.';
     }
-    if (status === 403)
+    if (status === 401 || 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)
+    }
+    if (status >= 500) {
         return 'Agent hit an internal error while reloading. Check `yaver logs`.';
+    }
     return `Reload failed (${status}).`;
 }
 /**

package/dist/YaverFeedback.js

@@ -171,19 +171,18 @@ class YaverFeedback {
                     });
                 }
             });
-            // 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: BlackBox.start() is intentionally NOT auto-called here.
+            // An earlier version (0.7.6) did auto-start it, and when the
+            // agent was in bootstrap / needs-auth mode — which can happen
+            // any time after `yaver serve` restarts before the user pairs —
+            // the SSE channel retried with exponential backoff on 401s,
+            // producing a tight loop of string concatenation + JSON parse
+            // that tripped a Hermes rope-string SIGSEGV on iOS 18.3.1
+            // during any other JS-thread regex work (e.g. react-native-
+            // view-shot's internal string handling during Screenshot &
+            // Fix). Host apps call BlackBox.start() explicitly once they
+            // know the agent URL + token are valid (SFMG does this inside
+            // its YaverFeedbackWidget after auth).
         }
         // 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.6",
+  "version": "0.7.7",
   "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

@@ -116,18 +116,29 @@ export const FeedbackModal: React.FC = () => {
       try {
         await fn(client);
       } catch (err) {
-        const msg = err instanceof Error ? err.message : String(err);
-        // 401 / 403 "invalid token" — the cached session is stale
-        // (common after a Convex deployment migration). Sign out so
-        // the next interaction surfaces the login sheet, then bubble
-        // a readable error up to the user.
-        const authFailed = /\b(401|403)\b.*invalid token|invalid token|unauthor/i.test(msg);
+        const msg = (err instanceof Error ? err.message : String(err)) || '';
+        // Avoid unbounded `.*` in regex — on RN 0.81 / Hermes rope
+        // strings plus a background SSE reconnect, that pattern has
+        // reliably SIGSEGV'd Hermes's string-view flattening path.
+        // Split into short, literal-only alternations.
+        const lower = msg.toLowerCase();
+        const authFailed =
+          lower.indexOf('invalid token') >= 0 ||
+          lower.indexOf('unauthor') >= 0 ||
+          lower.indexOf(' 401') >= 0 ||
+          lower.indexOf(' 403') >= 0;
         if (authFailed) {
           await YaverFeedback.signOut();
           YaverFeedback.showLogin();
           throw new Error('Session expired — please sign in again.');
         }
-        const transient = /Network request failed|timeout|ECONNREFUSED|Failed to fetch|fetch failed|aborted/i.test(msg);
+        const transient =
+          lower.indexOf('network request failed') >= 0 ||
+          lower.indexOf('econnrefused') >= 0 ||
+          lower.indexOf('failed to fetch') >= 0 ||
+          lower.indexOf('fetch failed') >= 0 ||
+          lower.indexOf('aborted') >= 0 ||
+          lower.indexOf('timeout') >= 0;
         if (!transient) throw err;
         const ok = await YaverFeedback.reconnect();
         if (!ok) throw err;

package/src/MachinePickerScreen.tsx

@@ -87,6 +87,24 @@ export const YaverMachinePickerScreen: React.FC<YaverMachinePickerProps> = ({
       : device.needsAuth
         ? '#f59e0b'
         : '#22c55e';
+    // Derive a single short status phrase the user can act on.
+    let statusLine = device.platform;
+    if (!device.isOnline) {
+      statusLine = 'Offline — start `yaver serve` on the Mac';
+    } else if (device.needsAuth) {
+      statusLine =
+        'Needs pairing — open the Yaver app to adopt this machine';
+    } else if (device.runnerDown) {
+      statusLine = 'Runner down — restart the coding agent on the Mac';
+    } else {
+      // Happy-path subtitle: platform + optional host/share hint.
+      statusLine = device.platform;
+      if (device.isGuest && device.hostEmail) {
+        statusLine = `${statusLine} • ${device.hostEmail}`;
+      } else if (device.accessScope === 'shared-scoped') {
+        statusLine = `${statusLine} • paylaşılan`;
+      }
+    }
     return (
       <TouchableOpacity
         key={device.deviceId}
@@ -96,11 +114,7 @@ export const YaverMachinePickerScreen: React.FC<YaverMachinePickerProps> = ({
         <View style={[styles.health, { backgroundColor: healthColor }]} />
         <View style={{ flex: 1 }}>
           <Text style={styles.deviceName}>{device.name || device.deviceId}</Text>
-          <Text style={styles.deviceMeta}>
-            {device.platform}
-            {device.isGuest && device.hostEmail ? `  •  ${device.hostEmail}` : ''}
-            {device.accessScope === 'shared-scoped' ? '  •  paylaşılan' : ''}
-          </Text>
+          <Text style={styles.deviceMeta}>{statusLine}</Text>
         </View>
         {selected && <Text style={styles.selectedBadge}>seçili</Text>}
       </TouchableOpacity>

package/src/P2PClient.ts

@@ -18,22 +18,34 @@ export interface FeedbackEvent {
  * 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)
-  ) {
+  // Plain .indexOf instead of regex — short literal tests sidestep
+  // a Hermes rope-flatten SIGSEGV we saw on RN 0.81 / iOS 18.3.1
+  // when the same body was already being processed by a concurrent
+  // SSE reconnect loop.
+  const lower = (body || '').toLowerCase();
+  const hasRefused =
+    lower.indexOf('connection refused') >= 0 ||
+    lower.indexOf('econnrefused') >= 0;
+  const hasLoopback =
+    lower.indexOf('127.0.0.1') >= 0 || lower.indexOf('localhost') >= 0;
+  if (hasRefused && hasLoopback) {
     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)) {
+  if (
+    lower.indexOf('no dev server') >= 0 ||
+    lower.indexOf('not running') >= 0
+  ) {
     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`.';
+  if (status === 401 || status === 403) {
+    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}).`;
 }
 

package/src/YaverFeedback.ts

@@ -185,18 +185,18 @@ export class YaverFeedback {
           });
         }
       });
-      // 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: BlackBox.start() is intentionally NOT auto-called here.
+      // An earlier version (0.7.6) did auto-start it, and when the
+      // agent was in bootstrap / needs-auth mode — which can happen
+      // any time after `yaver serve` restarts before the user pairs —
+      // the SSE channel retried with exponential backoff on 401s,
+      // producing a tight loop of string concatenation + JSON parse
+      // that tripped a Hermes rope-string SIGSEGV on iOS 18.3.1
+      // during any other JS-thread regex work (e.g. react-native-
+      // view-shot's internal string handling during Screenshot &
+      // Fix). Host apps call BlackBox.start() explicitly once they
+      // know the agent URL + token are valid (SFMG does this inside
+      // its YaverFeedbackWidget after auth).
     }
 
     // NOTE: We intentionally do NOT hook ErrorUtils.setGlobalHandler().