mnfst-render 0.2.7 → 0.2.9

package/manifest.render.mjs

@@ -23,6 +23,180 @@ async function importFromProject(moduleName) {
   }
 }
 
+/**
+ * localechange reloads localized sources asynchronously; waitForNetworkIdle can finish before
+ * Alpine.store('data') updates. Without this, x-for / $modify('items') lists often snapshot empty.
+ */
+async function waitForAlpineDataStoresSettled(page, timeoutMs = 12000) {
+  const interval = 100;
+  const deadline = Date.now() + timeoutMs;
+  while (Date.now() < deadline) {
+    const ok = await page
+      .evaluate(() => {
+        try {
+          const Alpine = window.Alpine;
+          if (!Alpine?.store) return true;
+          const d = Alpine.store('data');
+          if (!d) return true;
+          if (d._localeChanging) return false;
+          for (const k of Object.keys(d)) {
+            if (!k.startsWith('_') || !k.endsWith('_state')) continue;
+            const s = d[k];
+            if (s && typeof s === 'object' && s.loading) return false;
+          }
+          return true;
+        } catch {
+          return true;
+        }
+      })
+      .catch(() => true);
+    if (ok) return;
+    await new Promise((r) => setTimeout(r, interval));
+  }
+}
+
+async function flushAlpineEffects(page) {
+  await page
+    .evaluate(() => {
+      return new Promise((resolve) => {
+        try {
+          if (typeof Alpine !== 'undefined' && typeof Alpine.nextTick === 'function') {
+            Alpine.nextTick(() => {
+              if (typeof Alpine.nextTick === 'function') Alpine.nextTick(resolve);
+              else resolve();
+            });
+          } else {
+            queueMicrotask(resolve);
+          }
+        } catch {
+          resolve();
+        }
+      });
+    })
+    .catch(() => {});
+}
+
+/**
+ * After locale + route sync: run component swapping explicitly, then wait until Alpine data stores
+ * are settled. We call ManifestComponentsSwapping.processAll() directly because swapping only
+ * subscribes to manifest:route-change if window.ManifestRouting already exists at initialize() time;
+ * when the router loads after components, that listener is never registered and no second
+ * manifest:components-processed fires.
+ */
+async function waitForManifestPrerenderPipeline(page, { allLocales, currentLocale, timeoutMs }) {
+  const result = await page
+    .evaluate(
+      async ({ localeList, loc, ms }) => {
+        const sleep = (n) => new Promise((r) => setTimeout(r, n));
+        const deadline = Date.now() + ms;
+
+        const checkData = () => {
+          try {
+            const Alpine = window.Alpine;
+            if (!Alpine?.store) return true;
+            const d = Alpine.store('data');
+            if (!d) return true;
+            if (d._localeChanging) return false;
+            for (const k of Object.keys(d)) {
+              if (!k.startsWith('_') || !k.endsWith('_state')) continue;
+              const s = d[k];
+              if (s && typeof s === 'object' && s.loading) return false;
+            }
+            return true;
+          } catch {
+            return true;
+          }
+        };
+
+        try {
+          const locales = Array.isArray(localeList) ? localeList : [];
+          if (loc && typeof loc === 'string') {
+            try {
+              document.documentElement.lang = loc;
+            } catch {
+              /* no-op */
+            }
+          }
+          const store = typeof Alpine !== 'undefined' && Alpine.store ? Alpine.store('locale') : null;
+          if (store) {
+            if (!Array.isArray(store.available) || store.available.length === 0) {
+              store.available = locales.slice();
+            } else {
+              store.available = Array.from(new Set([...store.available, ...locales]));
+            }
+            if (loc && typeof loc === 'string') {
+              store.current = loc;
+            }
+          }
+
+          if (loc && typeof loc === 'string') {
+            window.dispatchEvent(new CustomEvent('localechange', { detail: { locale: loc } }));
+          }
+
+          const rawRoute = window.ManifestRoutingNavigation?.getCurrentRoute?.() ?? window.location.pathname;
+          const clean = String(rawRoute || '/').replace(/^\/+|\/+$/g, '');
+          const parts = clean ? clean.split('/') : [];
+          const logical =
+            parts.length > 0 && locales.includes(parts[0])
+              ? '/' + parts.slice(1).join('/')
+              : clean
+                ? '/' + clean
+                : '/';
+          const to = logical === '//' ? '/' : logical;
+          const normalizedPath =
+            typeof to === 'string' && to !== '/' ? to.replace(/^\/|\/$/g, '') : '/';
+
+          window.dispatchEvent(
+            new CustomEvent('manifest:route-change', {
+              detail: {
+                from: to,
+                to,
+                normalizedPath,
+              },
+            })
+          );
+          window.dispatchEvent(new PopStateEvent('popstate'));
+
+          if (window.ManifestComponentsSwapping?.processAll) {
+            try {
+              await window.ManifestComponentsSwapping.processAll(normalizedPath);
+            } catch (e) {
+              return { ok: false, reason: 'processAll-error', message: String(e?.message || e) };
+            }
+          }
+
+          while (Date.now() < deadline) {
+            if (checkData()) {
+              return { ok: true };
+            }
+            await sleep(50);
+          }
+          return {
+            ok: false,
+            reason: 'timeout',
+            dataOk: checkData(),
+            hadSwapping: !!window.ManifestComponentsSwapping?.processAll,
+          };
+        } catch (err) {
+          return { ok: false, reason: 'error', message: String(err?.message || err) };
+        }
+      },
+      {
+        localeList: allLocales,
+        loc: currentLocale,
+        ms: timeoutMs,
+      }
+    )
+    .catch((e) => ({ ok: false, reason: 'evaluate', message: String(e) }));
+
+  if (!result?.ok) {
+    const parts = [`prerender: pipeline wait incomplete (${result?.reason ?? 'unknown'})`];
+    if (result?.dataOk === false) parts.push('data still loading');
+    if (result?.message) parts.push(result.message);
+    process.stdout.write(`${parts.join('; ')}\n`);
+  }
+}
+
 // --- Config ------------------------------------------------------------------
 
 function parseArgs() {
@@ -88,6 +262,7 @@ function resolveConfig() {
     concurrency: Math.max(1, cli.concurrency ?? pre.concurrency ?? 6),
     dryRun: !!cli.dryRun,
     debugPrerender: !!(cli.debugPrerender ?? pre.debugPrerender),
+    pipelineTimeout: Math.max(3000, Number(pre.pipelineTimeout) || 25000),
   };
 }
 
@@ -1273,6 +1448,25 @@ async function runPrerender(config) {
 
     const page = await browser.newPage();
     try {
+      // Align <html lang> with the URL being prerendered before any app script runs.
+      // initializeDataSourcesPlugin picks locale from document.documentElement.lang first; a mismatch
+      // (e.g. headless default vs /en/...) leaves $x.* empty while x-route sections still render.
+      await page.evaluateOnNewDocument((locale) => {
+        const apply = () => {
+          try {
+            if (locale && typeof locale === 'string') document.documentElement.lang = locale;
+          } catch {
+            /* no-op */
+          }
+        };
+        if (typeof document !== 'undefined') {
+          if (document.readyState === 'loading') {
+            document.addEventListener('DOMContentLoaded', apply, { once: true });
+          }
+          apply();
+        }
+      }, currentLocale);
+
       pushDebug({ path: displayPath, stage: 'start' });
       await page.goto(url, {
         waitUntil: 'domcontentloaded',
@@ -1313,6 +1507,34 @@ async function runPrerender(config) {
         });
       }).catch(() => { });
 
+      await page.waitForNetworkIdle({ idleTime: 1500, timeout: 10000 }).catch(() => { });
+
+      await page.evaluate(() => {
+        return new Promise((resolve) => {
+          const observer = new MutationObserver(() => {
+            clearTimeout(stable);
+            stable = setTimeout(resolve, 800);
+          });
+          observer.observe(document.documentElement, { childList: true, subtree: true, characterData: true });
+          let stable = setTimeout(() => {
+            observer.disconnect();
+            resolve();
+          }, 800);
+        });
+      }).catch(() => { });
+
+      // Locale + route sync, then wait for manifest:components-processed (swapping) and settled data stores
+      // before snapshot. Listener is installed inside the page before dispatch (see waitForManifestPrerenderPipeline).
+      await waitForManifestPrerenderPipeline(page, {
+        allLocales: locales,
+        currentLocale,
+        timeoutMs: config.pipelineTimeout,
+      });
+
+      await page.waitForNetworkIdle({ idleTime: 800, timeout: 12000 }).catch(() => { });
+      await waitForAlpineDataStoresSettled(page, 10000);
+      await flushAlpineEffects(page);
+
       if (config.debugPrerender) {
         const before = await page.evaluate(() => {
           const templates = Array.from(document.querySelectorAll('template[x-for]'));
@@ -1334,71 +1556,102 @@ async function runPrerender(config) {
               cloneCount,
             };
           });
+
+          const listDiagnostics = {
+            htmlLang: '',
+            localeCurrent: null,
+            dataLocaleChanging: null,
+            dataStates: {},
+            topLevelArrayLengths: {},
+            nestedContentCards: null,
+            emptyStaticXFors: [],
+          };
+
+          try {
+            listDiagnostics.htmlLang = document.documentElement.lang || '';
+            const Alpine = window.Alpine;
+            if (Alpine?.store) {
+              const loc = Alpine.store('locale');
+              listDiagnostics.localeCurrent = loc?.current ?? null;
+              const d = Alpine.store('data');
+              if (d) {
+                listDiagnostics.dataLocaleChanging = !!d._localeChanging;
+                for (const k of Object.keys(d)) {
+                  if (k.startsWith('_') && k.endsWith('_state')) {
+                    const short = k.slice(1, -'_state'.length);
+                    const s = d[k];
+                    if (s && typeof s === 'object') {
+                      listDiagnostics.dataStates[short] = {
+                        loading: !!s.loading,
+                        ready: !!s.ready,
+                        hasError: s.error != null,
+                      };
+                    }
+                  } else if (!k.startsWith('_') && Array.isArray(d[k])) {
+                    listDiagnostics.topLevelArrayLengths[k] = d[k].length;
+                  }
+                }
+                try {
+                  const cards = d.content?.home?.differentiators?.cards;
+                  if (Array.isArray(cards)) listDiagnostics.nestedContentCards = cards.length;
+                  else if (cards && typeof cards === 'object') listDiagnostics.nestedContentCards = Object.keys(cards).length;
+                  else listDiagnostics.nestedContentCards = cards == null ? null : 'non-iterable';
+                } catch {
+                  listDiagnostics.nestedContentCards = 'error';
+                }
+              }
+            }
+          } catch (e) {
+            listDiagnostics.probeError = String(e?.message || e);
+          }
+
+          for (const tpl of templates) {
+            if (tpl.getAttribute('data-prerender-collapsed') === '1') continue;
+            const first = tpl.content?.firstElementChild;
+            const tag = first ? first.tagName : null;
+            const cls = first ? (first.getAttribute('class') || '') : '';
+            let cloneCount = 0;
+            let next = tpl.nextElementSibling;
+            while (next && (!tag || next.tagName === tag)) {
+              if (tag && (next.getAttribute('class') || '') !== cls) break;
+              cloneCount++;
+              next = next.nextElementSibling;
+            }
+            if (cloneCount > 0) continue;
+            const routeAnc = tpl.closest('[x-route]');
+            let hiddenReason = null;
+            let el = tpl.parentElement;
+            while (el) {
+              if (el.hasAttribute('hidden')) {
+                hiddenReason = 'ancestor-hidden';
+                break;
+              }
+              const st = el.getAttribute('style') || '';
+              if (/\bdisplay\s*:\s*none\b/i.test(st)) {
+                hiddenReason = 'ancestor-display-none';
+                break;
+              }
+              el = el.parentElement;
+            }
+            const itemsHost = tpl.closest('[items]');
+            listDiagnostics.emptyStaticXFors.push({
+              xFor: (tpl.getAttribute('x-for') || '').slice(0, 160),
+              nearestXRoute: routeAnc ? (routeAnc.getAttribute('x-route') || '').slice(0, 100) : null,
+              hiddenReason,
+              hostItemsAttr: itemsHost ? (itemsHost.getAttribute('items') || '').slice(0, 120) : null,
+            });
+          }
+
           return {
             templateCount: templates.length,
             nonCollapsedTemplateCount: templates.filter((t) => t.getAttribute('data-prerender-collapsed') !== '1').length,
             entries,
+            listDiagnostics,
           };
         }).catch(() => null);
         pushDebug({ path: displayPath, stage: 'post-dom-settle', metrics: before });
       }
 
-      await page.waitForNetworkIdle({ idleTime: 1500, timeout: 10000 }).catch(() => { });
-
-      await page.evaluate(() => {
-        return new Promise((resolve) => {
-          const observer = new MutationObserver(() => {
-            clearTimeout(stable);
-            stable = setTimeout(resolve, 800);
-          });
-          observer.observe(document.documentElement, { childList: true, subtree: true, characterData: true });
-          let stable = setTimeout(() => {
-            observer.disconnect();
-            resolve();
-          }, 800);
-        });
-      }).catch(() => { });
-
-      // Ensure $route-dependent expressions are recalculated after locale/data stores settle.
-      // This helps localized dynamic pages (e.g. /ko/articles/slug) compute prev/next links correctly.
-      await page.evaluate(({ allLocales, currentLocale }) => {
-        try {
-          const localeList = Array.isArray(allLocales) ? allLocales : [];
-          const store = (typeof Alpine !== 'undefined' && Alpine.store) ? Alpine.store('locale') : null;
-          if (store) {
-            if (!Array.isArray(store.available) || store.available.length === 0) {
-              store.available = localeList.slice();
-            } else {
-              const merged = Array.from(new Set([...store.available, ...localeList]));
-              store.available = merged;
-            }
-            if (currentLocale && typeof currentLocale === 'string') {
-              store.current = currentLocale;
-            }
-          }
-
-          const rawRoute = window.ManifestRoutingNavigation?.getCurrentRoute?.() ?? window.location.pathname;
-          const clean = String(rawRoute || '/').replace(/^\/+|\/+$/g, '');
-          const parts = clean ? clean.split('/') : [];
-          const logical = parts.length > 0 && localeList.includes(parts[0])
-            ? '/' + parts.slice(1).join('/')
-            : (clean ? '/' + clean : '/');
-          const to = logical === '//' ? '/' : logical;
-
-          window.dispatchEvent(new CustomEvent('manifest:route-change', {
-            detail: {
-              from: to,
-              to,
-              normalizedPath: (typeof to === 'string' && to !== '/') ? to.replace(/^\/|\/$/g, '') : '/'
-            }
-          }));
-          window.dispatchEvent(new PopStateEvent('popstate'));
-        } catch {
-          // no-op
-        }
-      }, { allLocales: locales, currentLocale }).catch(() => { });
-      await new Promise((resolve) => setTimeout(resolve, 60));
-
       // Optional extra delay so in-page async (e.g. fetch() in x-init for client logos) can complete before snapshot.
       if (config.waitAfterIdle > 0) {
         await new Promise((r) => setTimeout(r, config.waitAfterIdle));

package/package.json

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