mnfst-render 0.2.4 → 0.2.6

package/manifest.render.mjs

@@ -1385,6 +1385,25 @@ async function runPrerender(config) {
           const forceCollapse = explicit || inferred;
           if (!forceCollapse) {
             tpl.removeAttribute('data-prerender-collapsed');
+            tpl.removeAttribute('data-prerender-static-generated');
+            // Static mode: if prerender produced concrete siblings, mark template for removal later.
+            const first = tpl.content?.firstElementChild;
+            if (first) {
+              const tag = first.tagName;
+              const cls = first.getAttribute('class') || '';
+              let next = tpl.nextElementSibling;
+              let generatedCount = 0;
+              while (next) {
+                if (next.tagName !== tag) break;
+                const sameClass = (next.getAttribute('class') || '') === cls;
+                if (!sameClass) break;
+                generatedCount++;
+                next = next.nextElementSibling;
+              }
+              if (generatedCount > 0) {
+                tpl.setAttribute('data-prerender-static-generated', '1');
+              }
+            }
             return; // keep prerendered list for SEO
           }
           tpl.setAttribute('data-prerender-collapsed', '1');
@@ -1405,6 +1424,14 @@ async function runPrerender(config) {
         });
       });
 
+      // Remove static x-for templates once static clones are generated.
+      // This prevents Alpine from rendering duplicate lists at runtime.
+      await page.evaluate(() => {
+        document.querySelectorAll('template[x-for][data-prerender-static-generated="1"]').forEach((tpl) => {
+          tpl.remove();
+        });
+      });
+
       // Remove orphan x-for clones that still reference loop-scope vars (e.g. image/index)
       // outside their template scope. These throw Alpine errors in live static hosting.
       await page.evaluate(() => {
@@ -1467,7 +1494,35 @@ async function runPrerender(config) {
               if (!bindingAttrRegex.test(attr.name)) continue;
               const expr = attr.value || '';
               if (hasVar(expr, itemVar) || hasVar(expr, indexVar)) {
-                node.removeAttribute(attr.name);
+                const name = attr.name;
+                // Remove text/html bindings only when static content already exists.
+                if (name === 'x-text' || name === 'x-html') {
+                  if ((node.textContent || '').trim() || (node.innerHTML || '').trim()) {
+                    node.removeAttribute(name);
+                  }
+                  continue;
+                }
+
+                // Remove x-show/x-if if they reference loop vars; cloned node is now static.
+                if (name === 'x-show' || name === 'x-if') {
+                  node.removeAttribute(name);
+                  continue;
+                }
+
+                // For :attr / x-bind:attr, only remove binding if a concrete attr is present.
+                let boundAttr = '';
+                if (name.startsWith(':')) boundAttr = name.slice(1);
+                else if (name.startsWith('x-bind:')) boundAttr = name.slice('x-bind:'.length);
+                if (boundAttr) {
+                  const concrete = node.getAttribute(boundAttr);
+                  if (concrete != null && String(concrete).trim() !== '') {
+                    node.removeAttribute(name);
+                  }
+                  continue;
+                }
+
+                // Event/other loop-scoped bindings are unsafe on static clones.
+                node.removeAttribute(name);
               }
             }
           }

package/package.json

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