mnfst-render 0.2.8 → 0.2.9

package/manifest.render.mjs

@@ -77,133 +77,109 @@ async function flushAlpineEffects(page) {
 }
 
 /**
- * After locale + route sync, wait until (1) Manifest fires manifest:components-processed for that pass
- * (component swapping finished) and (2) no Alpine data _*_state is still loading / _localeChanging.
- * Listener is registered before dispatch so the initial page load event is ignored.
+ * 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(
-      ({ localeList, loc, ms }) => {
-        return new Promise((resolve) => {
-          const deadline = Date.now() + ms;
-          let dispatched = false;
-          let sawComponents = false;
-          let poll = 0;
+      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;
+          }
+        };
 
-          const checkData = () => {
+        try {
+          const locales = Array.isArray(localeList) ? localeList : [];
+          if (loc && typeof loc === 'string') {
             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;
+              document.documentElement.lang = loc;
             } catch {
-              return true;
+              /* no-op */
             }
-          };
-
-          const skipComponentsWait = () =>
-            !window.__manifestComponentsInitialized || !window.ManifestComponentsSwapping;
-
-          const cleanup = () => {
-            if (poll) clearInterval(poll);
-            window.removeEventListener('manifest:components-processed', onComponents);
-          };
-
-          const tryFinish = () => {
-            if (!dispatched) return;
-            if (skipComponentsWait()) {
-              sawComponents = true;
+          }
+          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 (sawComponents && checkData()) {
-              cleanup();
-              resolve({ ok: true });
+            if (loc && typeof loc === 'string') {
+              store.current = loc;
             }
-          };
+          }
 
-          function onComponents() {
-            if (!dispatched) return;
-            sawComponents = true;
-            tryFinish();
+          if (loc && typeof loc === 'string') {
+            window.dispatchEvent(new CustomEvent('localechange', { detail: { locale: loc } }));
           }
 
-          window.addEventListener('manifest:components-processed', onComponents);
-
-          poll = setInterval(() => {
-            if (Date.now() > deadline) {
-              cleanup();
-              resolve({
-                ok: false,
-                reason: 'timeout',
-                sawComponents,
-                dataOk: checkData(),
-                skippedComponents: skipComponentsWait(),
-              });
-              return;
-            }
-            tryFinish();
-          }, 50);
+          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'));
 
-          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 (window.ManifestComponentsSwapping?.processAll) {
+            try {
+              await window.ManifestComponentsSwapping.processAll(normalizedPath);
+            } catch (e) {
+              return { ok: false, reason: 'processAll-error', message: String(e?.message || e) };
             }
+          }
 
-            if (loc && typeof loc === 'string') {
-              window.dispatchEvent(new CustomEvent('localechange', { detail: { locale: loc } }));
+          while (Date.now() < deadline) {
+            if (checkData()) {
+              return { ok: true };
             }
-
-            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;
-
-            dispatched = true;
-            window.dispatchEvent(
-              new CustomEvent('manifest:route-change', {
-                detail: {
-                  from: to,
-                  to,
-                  normalizedPath: typeof to === 'string' && to !== '/' ? to.replace(/^\/|\/$/g, '') : '/',
-                },
-              })
-            );
-            window.dispatchEvent(new PopStateEvent('popstate'));
-            tryFinish();
-          } catch (err) {
-            cleanup();
-            resolve({ ok: false, reason: 'error', message: String(err?.message || err) });
+            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,
@@ -216,7 +192,7 @@ async function waitForManifestPrerenderPipeline(page, { allLocales, currentLocal
   if (!result?.ok) {
     const parts = [`prerender: pipeline wait incomplete (${result?.reason ?? 'unknown'})`];
     if (result?.dataOk === false) parts.push('data still loading');
-    if (result?.sawComponents === false && !result?.skippedComponents) parts.push('no manifest:components-processed');
+    if (result?.message) parts.push(result.message);
     process.stdout.write(`${parts.join('; ')}\n`);
   }
 }

package/package.json

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