mnfst-render 0.3.2 → 0.3.4

package/manifest.render.mjs

@@ -904,6 +904,7 @@ function stripPrerenderDynamicBindings(html) {
   return html.replace(/<(\w+)([^>]*)>/g, (match, tagName, attrsStr) => {
     if (tagName.toLowerCase() === 'script') return match;
     const isAnchor = tagName.toLowerCase() === 'a';
+    const isImg = tagName.toLowerCase() === 'img';
     let workAttrs = attrsStr;
     workAttrs = workAttrs.replace(/\s+:style=(?:"([^"]*)"|'([^']*)')/gi, (sub, d, s) => {
       const val = (d !== undefined ? d : s) || '';
@@ -919,8 +920,9 @@ function stripPrerenderDynamicBindings(html) {
     let m;
     while ((m = bindingRegex.exec(workAttrs)) !== null) {
       const attrName = (m[1] || '').toLowerCase();
-      // Keep href on anchors so prerendered static navigation stays valid.
-      if (attrName === 'class' || attrName === 'style' || (isAnchor && attrName === 'href')) continue;
+      // Keep href on anchors and src on images: :href / :src often reference x-for iterators (e.g.
+      // article?.banner). Stripping the baked literal leaves only :src/:href and breaks static HTML.
+      if (attrName === 'class' || attrName === 'style' || (isAnchor && attrName === 'href') || (isImg && attrName === 'src')) continue;
       const val = (m[2] !== undefined ? m[2] : m[3]) || '';
       if (val.indexOf('$x') === -1) toStrip.add(attrName);
     }
@@ -937,6 +939,18 @@ function stripPrerenderDynamicBindings(html) {
   });
 }
 
+// Drop :src / x-bind:src when img already has a baked src= (x-for / iterator expressions break hydrate).
+function stripRedundantImgSrcBindings(html) {
+  return html.replace(/<img\b([^>]*)>/gi, (full, attrs) => {
+    const srcM = attrs.match(/\ssrc=(["'])([\s\S]*?)\1/i);
+    if (!srcM || !String(srcM[2] || '').trim()) return full;
+    if (!/\s:src\s*=|\sx-bind:src\s*=/i.test(attrs)) return full;
+    let next = attrs.replace(/\s:src=(?:"[^"]*"|'[^']*')/gi, '');
+    next = next.replace(/\sx-bind:src=(?:"[^"]*"|'[^']*')/gi, '');
+    return `<img${next}>`;
+  });
+}
+
 // Remove empty inline mask-image styles emitted before data resolves
 // (e.g. style="mask-image: url()"), while keeping any :style/x-bind:style bindings.
 function stripEmptyInlineMaskStyles(html) {
@@ -1797,7 +1811,8 @@ async function runPrerender(config) {
       // keep x-text/x-bind referencing card/item — Alpine then mutates or errors on the static HTML.)
       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:|@)/;
+        // Include x-init: expanded clones still had x-init="getDescription(article)" etc.; Alpine then throws (article undefined).
+        const bindingAttrRegex = /^(?:x-bind:|:|x-text|x-html|x-show|x-if|x-model|x-effect|x-init|x-on:|@)/;
         const hasVar = (expr, varName) => varName && new RegExp(`\\b${varName}\\b`).test(expr || '');
         const stripLoopBindings = (el, itemVar, indexVar) => {
           const nodes = [el, ...Array.from(el.querySelectorAll('*'))];
@@ -1891,7 +1906,7 @@ async function runPrerender(config) {
       // outside their template scope. These throw Alpine errors in live static hosting.
       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 bindingAttrRegex = /^(?:x-bind:|:|x-text|x-html|x-show|x-if|x-model|x-effect|x-init|x-on:|@)/;
         const hasVar = (expr, varName) => varName && new RegExp(`\\b${varName}\\b`).test(expr || '');
         const elementReferencesLoopScope = (el, itemVar, indexVar) => {
           if (!el) return false;
@@ -1992,6 +2007,7 @@ async function runPrerender(config) {
       const xData = { manifest, content };
       html = resolveHeadXBindings(html, xData);
       html = stripPrerenderDynamicBindings(html);
+      html = stripRedundantImgSrcBindings(html);
       html = stripEmptyInlineMaskStyles(html);
       html = rewriteHtmlAssetPaths(html, fileSegments.length);
       const liveBase = config.liveUrl.replace(/\/$/, '');

package/package.json

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