mnfst-render 0.4.3 → 0.4.5

package/manifest.render.mjs

@@ -238,6 +238,11 @@ function resolveConfig() {
     localeSubstitutionExclude: Array.isArray(pre.localeSubstitutionExclude)
       ? pre.localeSubstitutionExclude.map(String)
       : [],
+    /** Explicit locale-neutral paths to render in addition to those discovered automatically.
+     *  Each entry is expanded to all locale variants (e.g. "legal/privacy" → "cs/legal/privacy", ...) */
+    paths: Array.isArray(pre.paths)
+      ? pre.paths.map((p) => String(p).replace(/^\/+|\/+$/g, '')).filter(Boolean)
+      : [],
     dryRun: !!cli.dryRun,
     debugPrerender: !!(cli.debugPrerender ?? pre.debugPrerender),
     pipelineTimeout: Math.max(3000, Number(pre.pipelineTimeout) || 25000),
@@ -1127,32 +1132,71 @@ function loadAllLocaleContentData(manifest, rootDir, locales) {
   const data = manifest?.data;
   if (!data || typeof data !== 'object') return new Map();
 
-  const csvFiles = [];
-  const seen = new Set();
-  const addCsv = (ref) => {
-    if (typeof ref !== 'string' || !ref.endsWith('.csv')) return;
-    const p = join(rootDir, ref.startsWith('/') ? ref.slice(1) : ref);
-    if (!seen.has(p)) { seen.add(p); csvFiles.push(p); }
-  };
+  // Lazy-load js-yaml for parsing per-locale YAML files
+  let jsYaml = null;
+  try { jsYaml = require('js-yaml'); } catch { /* yaml not available; YAML locale files will be skipped */ }
+
+  // Deep-merge source into target (for combining multiple data sources per locale)
+  function deepMerge(target, source) {
+    if (!source || typeof source !== 'object' || Array.isArray(source)) return;
+    for (const key of Object.keys(source)) {
+      if (source[key] && typeof source[key] === 'object' && !Array.isArray(source[key])) {
+        target[key] = (target[key] && typeof target[key] === 'object') ? target[key] : {};
+        deepMerge(target[key], source[key]);
+      } else {
+        target[key] = source[key];
+      }
+    }
+  }
+
+  const result = new Map();
+  for (const locale of locales) result.set(locale, {});
 
   for (const [, value] of Object.entries(data)) {
-    if (typeof value === 'string') { addCsv(value); continue; }
-    if (value && typeof value === 'object' && !Array.isArray(value)) {
+    if (typeof value === 'string') {
+      // Single CSV with locale columns (all locales in one file)
+      if (value.endsWith('.csv')) {
+        const csvPath = join(rootDir, value.startsWith('/') ? value.slice(1) : value);
+        for (const locale of locales) {
+          deepMerge(result.get(locale), parseCsvToKeyValue(csvPath, locale));
+        }
+      }
+    } else if (value && typeof value === 'object' && !Array.isArray(value)) {
       if (value.locales) {
+        // { locales: "/path/to/multi-locale.csv" } format
         const refs = Array.isArray(value.locales) ? value.locales : [value.locales];
-        refs.forEach(addCsv);
+        for (const ref of refs) {
+          if (typeof ref !== 'string' || !ref.endsWith('.csv')) continue;
+          const csvPath = join(rootDir, ref.startsWith('/') ? ref.slice(1) : ref);
+          for (const locale of locales) {
+            deepMerge(result.get(locale), parseCsvToKeyValue(csvPath, locale));
+          }
+        }
+      } else {
+        // Per-locale files: { "en": "/data/content.en.yaml", "fr": "/data/content.fr.yaml", ... }
+        for (const [localeKey, filePath] of Object.entries(value)) {
+          if (!locales.includes(localeKey) || typeof filePath !== 'string') continue;
+          const fullPath = join(rootDir, filePath.startsWith('/') ? filePath.slice(1) : filePath);
+          if (!existsSync(fullPath)) continue;
+          let localeData = null;
+          try {
+            const raw = readFileSync(fullPath, 'utf8');
+            if ((filePath.endsWith('.yaml') || filePath.endsWith('.yml')) && jsYaml) {
+              localeData = jsYaml.load(raw);
+            } else if (filePath.endsWith('.json')) {
+              localeData = JSON.parse(raw);
+            } else if (filePath.endsWith('.csv')) {
+              localeData = parseCsvToKeyValue(fullPath, localeKey);
+            }
+          } catch { /* ignore parse errors for individual locale files */ }
+          if (localeData && typeof localeData === 'object') {
+            deepMerge(result.get(localeKey), localeData);
+          }
+        }
       }
     }
   }
 
-  const result = new Map();
-  for (const locale of locales) {
-    const merged = {};
-    for (const csvPath of csvFiles) {
-      Object.assign(merged, parseCsvToKeyValue(csvPath, locale));
-    }
-    result.set(locale, merged);
-  }
   return result;
 }
 
@@ -1599,6 +1643,14 @@ async function runPrerender(config) {
 
   const defaultLocale = locales[0] ?? null;
   const routeSegments = discoverRoutes(manifest, config.root);
+  // Merge any explicitly configured paths (manifest.prerender.paths) into the discovered segments.
+  // These are treated as locale-neutral and get full locale-expansion like all other discovered paths.
+  if (config.paths && config.paths.length > 0) {
+    const segSet = new Set(routeSegments);
+    for (const p of config.paths) {
+      if (!segSet.has(p)) { routeSegments.push(p); segSet.add(p); }
+    }
+  }
   const localeSet = new Set(locales.map((l) => String(l).toLowerCase()));
   const localeNeutralSegments = routeSegments.filter((seg) => {
     if (!seg) return true;
@@ -2101,6 +2153,48 @@ async function runPrerender(config) {
         pushDebug({ path: displayPath, stage: 'post-xfor-mark', metrics: afterMark });
       }
 
+      // For static x-for clones that contain data-hydrate elements, inject the loop-scope
+      // variable as x-data on the clone element itself. This ensures that after the loop
+      // template is removed, data-hydrate bindings referencing loop variables (e.g.
+      // plan?.price?.[currency]?.[frequency]) continue to work at runtime via the injected scope.
+      // The parent Alpine scope (e.g. <main x-data="{ currency, frequency }") remains accessible.
+      await page.evaluate(() => {
+        const loopVarRx = /^\s*(?:\(\s*([A-Za-z_$][\w$]*)(?:\s*,\s*([A-Za-z_$][\w$]*))?\s*\)|([A-Za-z_$][\w$]*))\s+in\s+/;
+        document.querySelectorAll('template[x-for][data-prerender-static-generated="1"]').forEach((tpl) => {
+          if (tpl.hasAttribute('data-prerender-hydrate') || tpl.closest('[data-prerender-hydrate]')) return;
+          const xFor = (tpl.getAttribute('x-for') || '').trim();
+          const m = xFor.match(loopVarRx);
+          const itemVar = m ? (m[1] || m[3] || '') : '';
+          if (!itemVar) return;
+          const first = tpl.content && tpl.content.firstElementChild;
+          if (!first) return;
+          const tag = first.tagName;
+          const cls = first.getAttribute('class') || '';
+          let n = tpl.nextElementSibling;
+          while (n && n.tagName === tag && (n.getAttribute('class') || '') === cls) {
+            // Only process clones that contain data-hydrate descendants
+            if (
+              !n.hasAttribute('x-data') &&
+              (n.hasAttribute('data-prerender-hydrate') || n.querySelector('[data-prerender-hydrate]'))
+            ) {
+              try {
+                const A = window.Alpine;
+                if (A && typeof A.$data === 'function') {
+                  const scope = A.$data(n);
+                  if (scope && Object.prototype.hasOwnProperty.call(scope, itemVar)) {
+                    const raw = scope[itemVar];
+                    // Serialize only own-enumerable properties to avoid circular refs / proxies
+                    const snapshot = JSON.parse(JSON.stringify(raw));
+                    n.setAttribute('x-data', JSON.stringify({ [itemVar]: snapshot }));
+                  }
+                }
+              } catch { /* serialisation failed — leave binding as-is */ }
+            }
+            n = n.nextElementSibling;
+          }
+        });
+      });
+
       // Strip loop-scope bindings from x-for clones while <template> nodes still exist.
       // (If we remove static templates first, querySelectorAll('template[x-for]') misses them and clones
       // keep x-text/x-bind referencing card/item — Alpine then mutates or errors on the static HTML.)
@@ -2112,6 +2206,8 @@ async function runPrerender(config) {
         const stripLoopBindings = (el, itemVar, indexVar) => {
           const nodes = [el, ...Array.from(el.querySelectorAll('*'))];
           for (const node of nodes) {
+            // Skip elements inside data-hydrate islands — their bindings must remain live
+            if (node.hasAttribute('data-prerender-hydrate') || node.closest('[data-prerender-hydrate]')) continue;
             const attrs = node.attributes ? Array.from(node.attributes) : [];
             for (const attr of attrs) {
               if (!bindingAttrRegex.test(attr.name)) continue;

package/package.json

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