@ape-egg/vibe 1.8.0 → 1.9.0

package/runtime/pre-compiled-manifest.js

@@ -134,6 +134,14 @@ const detectHyperspeed = async () => {
   if (hyperspeedDetectionAttempted) return hyperspeedData;
   hyperspeedDetectionAttempted = true;
 
+  // Runtime-mode pages still have [vibe-fouc] / .vibe-fouc on their vibe root
+  // at this point — the compiler strips it at build time, and the runtime only
+  // clears it after hydration (PHASE_READY). Vibe can latch to any element, so
+  // search the whole document. If any fouc marker is still here, we're in
+  // runtime mode and no hyperspeed manifest will exist — skip the network
+  // fetches and avoid the 404 devtools noise.
+  const skipNetwork = !!document.querySelector("[vibe-fouc], .vibe-fouc");
+
   try {
     let pagePath = window.location.pathname;
 
@@ -185,21 +193,38 @@ const detectHyperspeed = async () => {
       );
     }
 
-    // Try each possible path
-    for (const manifestPath of possiblePaths) {
-      try {
-        const module = await import(manifestPath);
-        hyperspeedData = {
-          manifest: module.default,
-          path: manifestPath,
-        };
-        return hyperspeedData;
-      } catch (e) {
-        // Try next path
-        continue;
+    if (!skipNetwork) {
+      // Fully-runtime dynamic import. Hidden behind `new Function` so any
+      // bundler's static-analysis can't read into it — there's nothing we
+      // could or should tell it about these manifest paths, which are decided
+      // at runtime by searching a list.
+      const dynamicImport = new Function('p', 'return import(p)');
+      // Try each possible path
+      for (const manifestPath of possiblePaths) {
+        try {
+          const module = await dynamicImport(manifestPath);
+          hyperspeedData = {
+            manifest: module.default,
+            path: manifestPath,
+          };
+          return hyperspeedData;
+        } catch (e) {
+          // Try next path
+          continue;
+        }
       }
     }
 
+    // When skipping network, yield a macrotask so module-graph timing matches
+    // the old behavior where `await import()` on a missing manifest resolved
+    // via a network 404 (macrotask), not a microtask. Without this yield, the
+    // vibe module-graph resolves too fast and the post-boot microtask fires
+    // before sibling `<script type="module">` tags (e.g. component scripts)
+    // have had a chance to register their state.
+    if (skipNetwork) {
+      await new Promise((resolve) => setTimeout(resolve, 0));
+    }
+
     // No manifest found
     return null;
   } catch {
@@ -468,7 +493,7 @@ export const restoreMarkersFromManifest = (
             if (text.startsWith("if")) {
               if (!startComment) {
                 const expression = childTree.meta?.expression;
-                const expectedText = expression ? 'if ' + expression : null;
+                const expectedText = expression ? "if " + expression : null;
                 if (expectedText && text === expectedText) {
                   startComment = comment;
                   conditionalDepth = 1;