mnfst-render 0.5.2 → 0.5.3

package/manifest.render.mjs

@@ -172,6 +172,7 @@ function parseArgs() {
     if (args[i] === '--wait' && args[i + 1]) { out.wait = parseInt(args[++i], 10); continue; }
     if (args[i] === '--wait-after-idle' && args[i + 1]) { out.waitAfterIdle = parseInt(args[++i], 10); continue; }
     if (args[i] === '--concurrency' && args[i + 1]) { out.concurrency = parseInt(args[++i], 10); continue; }
+    if (args[i] === '--retries' && args[i + 1]) { out.retries = parseInt(args[++i], 10); continue; }
     if (args[i] === '--dry-run') { out.dryRun = true; continue; }
     if (args[i] === '--debug-prerender') { out.debugPrerender = true; continue; }
   }
@@ -231,6 +232,7 @@ function resolveConfig() {
     wait: cli.wait ?? pre.wait ?? null,
     waitAfterIdle: 0,
     concurrency: Math.max(1, cli.concurrency ?? pre.concurrency ?? Math.max(4, cpus().length - 1)),
+    retries: Math.max(0, cli.retries ?? pre.retries ?? 2),
     localeSubstitution: true,
     localeSubstitutionExclude: [],
     /** Explicit locale-neutral paths to render in addition to those discovered automatically.
@@ -1751,8 +1753,9 @@ async function runPrerender(config) {
     browser = await puppeteer.default.launch({ headless: true });
   }
 
-  const timeout = config.wait ?? 15000;
+  const timeout = config.wait ?? 30000;
   const concurrency = config.concurrency;
+  const maxRetries = config.retries ?? 2;
   const pathTotal = pathList.length;
   const failedPaths = [];
   const debugRows = [];
@@ -2721,19 +2724,34 @@ async function runPrerender(config) {
     }
   }
 
-  // Phase 1: Puppeteer — render base paths, cache raw DOM for substitution
+  // Phase 1: Puppeteer — render base paths, cache raw DOM for substitution.
+  // Any failures (e.g. transient navigation timeouts) are retried up to
+  // `maxRetries` times with a short backoff before being reported as fatal.
   try {
     let index = 0;
     async function worker() {
       while (true) {
         const i = index++;
         if (i >= puppeteerPaths.length) return;
-        await processPath(puppeteerPaths[i], i, {
-          onRawHtml: (seg, html) => {
-            // Cache raw DOM snapshot for locale variant generation (NOT_FOUND_PATH excluded)
-            if (seg !== NOT_FOUND_PATH) baseHtmlCache.set(seg || '', html);
-          },
-        });
+        const pathSeg = puppeteerPaths[i];
+        let attempt = 0;
+        while (true) {
+          const failureCountBefore = failedPaths.length;
+          await processPath(pathSeg, i, {
+            onRawHtml: (seg, html) => {
+              // Cache raw DOM snapshot for locale variant generation (NOT_FOUND_PATH excluded)
+              if (seg !== NOT_FOUND_PATH) baseHtmlCache.set(seg || '', html);
+            },
+          });
+          if (failedPaths.length === failureCountBefore) break; // success
+          if (attempt >= maxRetries) break; // out of retries — leave the failure recorded
+          // Pop the failure record and retry after a short backoff.
+          failedPaths.pop();
+          attempt++;
+          const displayPath = pathSeg === '' ? '/' : (pathSeg === NOT_FOUND_PATH ? '/__prerender_404__' : '/' + pathSeg);
+          process.stderr.write(`prerender: retrying ${displayPath} (attempt ${attempt + 1}/${maxRetries + 1})\n`);
+          await new Promise((r) => setTimeout(r, 500 * attempt));
+        }
       }
     }
     await Promise.all(

package/package.json

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