yaver-feedback-react-native 0.9.9 → 0.9.10

package/app.plugin.js

@@ -227,7 +227,9 @@ function withYaverAppDelegateHook(config) {
     // Existing consumers may already have the hot-reload hook from an older
     // SDK. Still apply newer, independently-versioned native contracts.
     if (contents.includes("YaverHotReload")) {
-      config.modResults.contents = patchDogfoodAppShortcut(contents);
+      config.modResults.contents = patchDogfoodAppShortcut(
+        patchHotReloadBootConfirmation(contents)
+      );
       return config;
     }
 
@@ -273,20 +275,14 @@ function withYaverAppDelegateHook(config) {
       name: Notification.Name("YaverHotReloadBundle"),
       object: nil
     )
-    // Crash-revert safety net: clear the boot-attempt counter once
-    // RN renders its first frame, OR after 10 s of uptime — whichever
-    // fires first. If neither fires (bundle crashes before render),
-    // YaverHotReload.bundleURL() will eventually revert to the
-    // TestFlight-installed bundle after 3 failed boots. See
-    // YaverHotReload.swift for the full state machine.
+    // Crash-revert safety net: only a real first React frame confirms boot.
+    // Uptime is not success: JS can fail while the native process remains
+    // alive on its launch screen indefinitely.
     NotificationCenter.default.addObserver(
       forName: NSNotification.Name(rawValue: "RCTContentDidAppearNotification"),
       object: nil,
       queue: .main
     ) { _ in YaverHotReload.markBootSuccessful() }
-    DispatchQueue.main.asyncAfter(deadline: .now() + 10) {
-      YaverHotReload.markBootSuccessful()
-    }
   }
 
   @objc private func yaverHandleHotReload(_ notification: Notification) {
@@ -374,6 +370,23 @@ function withYaverAppDelegateHook(config) {
   });
 }
 
+/** Upgrade an already-generated AppDelegate from the pre-0.9.10 boot guard.
+ * Older plugins treated ten seconds of process uptime as a successful React
+ * boot. A JS exception can leave that process alive on the native splash, so
+ * the timer reset the crash counter forever and prevented bundle rollback. */
+function patchHotReloadBootConfirmation(contents) {
+  let patched = contents;
+  patched = patched.replace(
+    /    \/\/ Crash-revert safety net: clear the boot-attempt counter once\n    \/\/ RN renders its first frame, OR after 10 s of uptime — whichever\n    \/\/ fires first\. If neither fires \(bundle crashes before render\),\n    \/\/ YaverHotReload\.bundleURL\(\) will eventually revert to the\n    \/\/ TestFlight-installed bundle after 3 failed boots\. See\n    \/\/ YaverHotReload\.swift for the full state machine\./,
+    `    // Crash-revert safety net: only a real first React frame confirms boot.\n    // Uptime is not success: JS can fail while the native process remains\n    // alive on its launch screen indefinitely.`
+  );
+  patched = patched.replace(
+    /    DispatchQueue\.main\.asyncAfter\(deadline: \.now\(\) \+ 10\) \{\n      YaverHotReload\.markBootSuccessful\(\)\n    \}\n?/g,
+    ""
+  );
+  return patched;
+}
+
 /** Wire the dynamic iOS Home Screen shortcut into the SDK native module.
  * The shortcut itself is created at runtime only after backend ACL success;
  * this hook merely consumes a user-selected action on warm/cold launch. */
@@ -820,6 +833,7 @@ const yaverFeedbackPlugin = createRunOncePlugin(
 // config-plugin seam catches template drift before a consumer discovers it in
 // an archive build.
 yaverFeedbackPlugin.__test = {
+  patchHotReloadBootConfirmation,
   patchDogfoodAppShortcut,
   patchDogfoodSceneDelegate,
   findAppDelegateClassClose,

package/dist/DogfoodRuntime.d.ts

@@ -32,9 +32,14 @@ export interface DogfoodLaneOption {
 /** One framework-to-lane matrix for Yaver and third-party consumers. */
 export declare function dogfoodLaneOptions(framework: string, capabilities?: {
     nativeRuntimeAvailable?: boolean;
+    browserRuntimeAvailable?: boolean;
     selfDevelopment?: boolean;
 }): DogfoodLaneOption[];
-export declare function defaultDogfoodLane(framework: string): DogfoodLane;
+export declare function defaultDogfoodLane(framework: string, capabilities?: {
+    nativeRuntimeAvailable?: boolean;
+    browserRuntimeAvailable?: boolean;
+    selfDevelopment?: boolean;
+}): DogfoodLane;
 export interface DogfoodLogLine {
     text: string;
     at: number;

package/dist/DogfoodRuntime.js

@@ -21,7 +21,10 @@ exports.runtimeLogLinesFromDevEvent = runtimeLogLinesFromDevEvent;
 function dogfoodLaneOptions(framework, capabilities = {}) {
     const normalized = String(framework || '').trim().toLowerCase();
     const reactNative = normalized === 'expo' || normalized === 'react-native';
-    const browserCapable = reactNative || [
+    // The framework matrix is the safe default. A positively detected browser
+    // target may add an exception (for example SwiftWasm/Tokamak), but a missing
+    // browser binary must not incorrectly remove RN/Flutter's browser build lane.
+    const browserCapable = capabilities.browserRuntimeAvailable === true || reactNative || [
         'flutter', 'web', 'next', 'nextjs', 'vite', 'remix', 'svelte', 'vue', 'angular',
     ].includes(normalized);
     const nativeAvailable = capabilities.nativeRuntimeAvailable === true;
@@ -40,8 +43,11 @@ function dogfoodLaneOptions(framework, capabilities = {}) {
         },
     ];
 }
-function defaultDogfoodLane(framework) {
-    return dogfoodLaneOptions(framework).find((option) => option.default && option.supported)?.lane || 'browser';
+function defaultDogfoodLane(framework, capabilities = {}) {
+    const options = dogfoodLaneOptions(framework, capabilities);
+    return options.find((option) => option.default && option.supported)?.lane
+        || options.find((option) => option.supported)?.lane
+        || 'browser';
 }
 class DogfoodRuntimeError extends Error {
     constructor(failure) {

package/dist/DogfoodSessionUi.d.ts

@@ -40,6 +40,7 @@ export declare const DogfoodLanePicker: React.FC<{
  * Logs; Hermes/WebRTC use the same lifecycle and failure/remedy treatment. */
 export declare const DogfoodLiveConsole: React.FC<{
     lane: DogfoodLane;
+    sourceLabel?: string;
     phase: DogfoodPhase;
     message: string;
     logs: readonly DogfoodLogLine[];

package/dist/DogfoodSessionUi.js

@@ -91,15 +91,17 @@ function runtimeTone(phase, colors) {
 }
 /** Shared second-stage live console. Browser lane deliberately names Browser
  * Logs; Hermes/WebRTC use the same lifecycle and failure/remedy treatment. */
-const DogfoodLiveConsole = ({ lane, phase, message, logs, failure, maxLines = 80, colors: colorOverrides, renderText }) => {
+const DogfoodLiveConsole = ({ lane, sourceLabel, phase, message, logs, failure, maxLines = 80, colors: colorOverrides, renderText }) => {
     const colors = resolvedColors(colorOverrides);
     const text = logs.slice(-maxLines).map((line) => line.text).join('\n');
     const title = lane === 'browser' ? 'Browser Logs' : lane === 'hermes' ? 'Hermes Logs' : 'WebRTC Logs';
-    return (<react_native_1.View style={[styles.console, { backgroundColor: colors.console, borderColor: colors.border }]} accessibilityLabel={title}>
+    const accessibleTitle = sourceLabel ? `${title} · ${sourceLabel}` : title;
+    return (<react_native_1.View style={[styles.console, { backgroundColor: colors.console, borderColor: colors.border }]} accessibilityLabel={accessibleTitle}>
       <react_native_1.View style={styles.consoleHeader}>
         <react_native_1.View style={[styles.statusDot, { backgroundColor: runtimeTone(phase, colors) }]}/>
         <react_native_1.Text style={[styles.consoleTitle, { color: colors.text }]}>{title}</react_native_1.Text>
       </react_native_1.View>
+      {sourceLabel ? <react_native_1.Text style={[styles.consoleSource, { color: colors.muted }]}>Source · {sourceLabel}</react_native_1.Text> : null}
       <react_native_1.Text style={[styles.consoleStatus, { color: colors.muted }]}>{message}</react_native_1.Text>
       {text ? (renderText ? renderText(text) : <react_native_1.Text selectable style={[styles.consoleText, { color: colors.text }]}>{text}</react_native_1.Text>) : (<react_native_1.Text style={[styles.consoleEmpty, { color: colors.muted }]}>Waiting for the first line from the remote PC…</react_native_1.Text>)}
       {failure ? (<react_native_1.View style={[styles.failure, { borderColor: colors.blocked }]}>
@@ -126,6 +128,7 @@ const styles = react_native_1.StyleSheet.create({
     console: { width: '100%', maxHeight: 320, overflow: 'hidden', marginTop: 10, borderWidth: 1, borderRadius: 10, padding: 11, gap: 7 },
     consoleHeader: { flexDirection: 'row', alignItems: 'center', gap: 7 },
     consoleTitle: { fontSize: 12, fontWeight: '800' },
+    consoleSource: { fontSize: 10, lineHeight: 14 },
     consoleStatus: { fontSize: 11, lineHeight: 16 },
     consoleText: { fontFamily: 'monospace', fontSize: 10, lineHeight: 15 },
     consoleEmpty: { fontSize: 10, fontStyle: 'italic' },