mnfst-render 0.5.6 → 0.5.7

package/manifest.render.mjs

@@ -150,11 +150,11 @@ async function waitForManifestRenderReady(page, { allLocales, currentLocale, tim
     )
     .catch((e) => ({ ok: false, reason: 'evaluate', message: String(e) }));
 
-  if (!result?.ok) {
-    const parts = [`prerender: render-ready wait incomplete (${result?.reason ?? 'unknown'})`];
-    if (result?.message) parts.push(result.message);
-    process.stdout.write(`${parts.join('; ')}\n`);
-  }
+  // Note: render-ready wait timeouts are silently tolerated.  Earlier versions
+  // logged a warning per path, but it fires on essentially every route in
+  // projects whose data plugins don't dispatch `manifest:render-ready` (i.e.
+  // most of them), drowning the terminal in noise.  The fallback timeout is
+  // intentional and benign — the DOM is still captured.
 }
 
 // --- Config ------------------------------------------------------------------
@@ -246,7 +246,11 @@ function resolveConfig() {
       : [],
     dryRun: !!cli.dryRun,
     debugPrerender: !!cli.debugPrerender,
-    pipelineTimeout: 25000,
+    // Cap on the manifest:render-ready wait.  When the data plugin dispatches
+    // the event, we resolve immediately; when it doesn't (most projects), we
+    // fall back to the timeout.  Lowered from 25000 — the earlier settle
+    // waits already give plugins enough time to finish.
+    pipelineTimeout: 6000,
   };
 }
 
@@ -2024,16 +2028,21 @@ async function runPrerender(config) {
         timeout: Math.min(timeout, 30000),
       });
 
+      // Belt-and-suspenders settle waits.  Each one is bounded short because
+      // the authoritative `manifest:render-ready` wait below is what we really
+      // rely on.  These earlier waits exist as fallbacks for projects whose
+      // data plugins don't dispatch the modern signal — but they shouldn't
+      // dominate per-path runtime when they DO time out.
       await Promise.race([
         page.evaluate(() => {
           return new Promise((resolve) => {
             const done = () => resolve();
-            const t = setTimeout(done, 6000);
+            const t = setTimeout(done, 1500); // was 6000
             window.addEventListener(
               'manifest:routing-ready',
               () => {
                 clearTimeout(t);
-                setTimeout(done, 2000);
+                setTimeout(done, 500); // was 2000
               },
               { once: true }
             );
@@ -2042,13 +2051,13 @@ async function runPrerender(config) {
         new Promise((_, rej) => setTimeout(() => rej(new Error('ready timeout')), timeout)),
       ]).catch(() => { });
 
-      // Ensure manifest.min.js (dynamic loader) has run and injected plugin scripts before snapshot.
-      // Static output still runs the loader and Alpine; we just capture the DOM after they've set up.
+      // Ensure the dynamic loader has injected at least one plugin script.
+      // In practice this happens within ~100ms; the 1500ms cap is generous.
       await page.evaluate(() => {
         return new Promise((resolve) => {
           const check = () => document.querySelectorAll('script[src*="manifest"]').length >= 2;
           if (check()) return resolve();
-          const deadline = Date.now() + 5000;
+          const deadline = Date.now() + 1500; // was 5000
           const t = setInterval(() => {
             if (check() || Date.now() >= deadline) {
               clearInterval(t);
@@ -2058,19 +2067,21 @@ async function runPrerender(config) {
         });
       }).catch(() => { });
 
-      await page.waitForNetworkIdle({ idleTime: 1500, timeout: 10000 }).catch(() => { });
+      // Network idle: shorter idle window + shorter cap.  Most plugin scripts
+      // and data fetches complete in under 2s on a healthy local server.
+      await page.waitForNetworkIdle({ idleTime: 800, timeout: 4000 }).catch(() => { });
 
+      // DOM stability: 300ms quiet window is plenty after networkIdle has
+      // already drained pending fetches.
       await page.evaluate(() => {
         return new Promise((resolve) => {
           const observer = new MutationObserver(() => {
             clearTimeout(stable);
-            stable = setTimeout(resolve, 800);
+            stable = setTimeout(finish, 300); // was 800
           });
           observer.observe(document.documentElement, { childList: true, subtree: true, characterData: true });
-          let stable = setTimeout(() => {
-            observer.disconnect();
-            resolve();
-          }, 800);
+          const finish = () => { observer.disconnect(); resolve(); };
+          let stable = setTimeout(finish, 300); // was 800
         });
       }).catch(() => { });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mnfst-render",
-  "version": "0.5.6",
+  "version": "0.5.7",
   "description": "Render Manifest sites to static HTML for SEO",
   "type": "module",
   "bin": {