mnfst-render 0.1.2 → 0.1.4

package/manifest.render.mjs

@@ -384,8 +384,10 @@ function stripPrerenderDynamicBindings(html) {
     const bindingRegex = /(?:^|\s)(?::|x-bind:)(\w+)=(?:"([^"]*)"|'([^']*)')/g;
     let m;
     while ((m = bindingRegex.exec(attrsStr)) !== null) {
+      const attrName = (m[1] || '').toLowerCase();
+      if (attrName === 'class' || attrName === 'style') continue;
       const val = (m[2] !== undefined ? m[2] : m[3]) || '';
-      if (val.indexOf('$x') === -1) toStrip.add(m[1].toLowerCase());
+      if (val.indexOf('$x') === -1) toStrip.add(attrName);
     }
     if (toStrip.size === 0) return match;
     let newAttrs = attrsStr;
@@ -951,6 +953,20 @@ async function runPrerender(config) {
       await page.evaluate(() => {
         const loopVarRegex = /^\s*(?:\(\s*([A-Za-z_$][\w$]*)(?:\s*,\s*([A-Za-z_$][\w$]*))?\s*\)|([A-Za-z_$][\w$]*))\s+in\s+/;
         const bindingAttrRegex = /^(?:x-bind:|:|x-text|x-html|x-show|x-if|x-model|x-effect|x-on:|@)/;
+        const hasVar = (expr, varName) => varName && new RegExp(`\\b${varName}\\b`).test(expr || '');
+        const elementReferencesLoopScope = (el, itemVar, indexVar) => {
+          if (!el) return false;
+          const nodes = [el, ...Array.from(el.querySelectorAll('*'))];
+          for (const node of nodes) {
+            const attrs = node.attributes ? Array.from(node.attributes) : [];
+            for (const attr of attrs) {
+              if (!bindingAttrRegex.test(attr.name)) continue;
+              const expr = attr.value || '';
+              if (hasVar(expr, itemVar) || hasVar(expr, indexVar)) return true;
+            }
+          }
+          return false;
+        };
 
         document.querySelectorAll('template[x-for]').forEach((tpl) => {
           const xFor = (tpl.getAttribute('x-for') || '').trim();
@@ -968,19 +984,7 @@ async function runPrerender(config) {
             const sameTag = next.tagName === tag;
             if (!sameTag) break;
 
-            let referencesLoopScope = false;
-            const attrNodes = next.attributes ? Array.from(next.attributes) : [];
-            for (const attr of attrNodes) {
-              if (!bindingAttrRegex.test(attr.name)) continue;
-              const expr = attr.value || '';
-              if (
-                (itemVar && new RegExp(`\\b${itemVar}\\b`).test(expr)) ||
-                (indexVar && new RegExp(`\\b${indexVar}\\b`).test(expr))
-              ) {
-                referencesLoopScope = true;
-                break;
-              }
-            }
+            const referencesLoopScope = elementReferencesLoopScope(next, itemVar, indexVar);
 
             const toRemove = next;
             next = next.nextElementSibling;

package/package.json

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