mnfst-render 0.3.5 → 0.3.7

package/manifest.render.mjs

@@ -956,6 +956,21 @@ function stripRedundantImgSrcBindings(html) {
  * prerender-baked markup (stripped :style, expanded lists, etc.). Tag opens with data-pre-rendered
  * are skipped by manifest.components.processor — required for static prerender output to hydrate correctly.
  */
+// Prerender inlined Iconify SVG under <i x-icon="iterator.icon">; clear x-icon value so Alpine does not evaluate
+// loop/item expressions while the attribute remains for CSS (e.g. inline layout that keys off [x-icon]).
+function stripResolvedXIconDirectives(html) {
+  return html.replace(/<i\b([^>]*)>([\s\S]*?)<\/i>/gi, (full, attrs, inner) => {
+    if (!/\sx-icon\s*=/i.test(attrs)) return full;
+    if (!/<svg\b/i.test(inner) || !/\bdata-icon\s*=/i.test(inner)) return full;
+    const cleaned = attrs
+      .replace(/\s+x-icon\s*=\s*"[^"]*"/gi, ' x-icon=""')
+      .replace(/\s+x-icon\s*=\s*'[^']*'/gi, ' x-icon=""')
+      .trim();
+    const sp = cleaned ? ' ' : '';
+    return `<i${sp}${cleaned}>${inner}</i>`;
+  });
+}
+
 function markPrerenderedManifestComponents(html) {
   return html.replace(/<(x-[a-z][\w-]*)([^>]*)>/gi, (full, tag, attrs) => {
     const a = attrs || '';
@@ -1826,7 +1841,7 @@ 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+/;
         // 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 bindingAttrRegex = /^(?:x-bind:|:|x-text|x-html|x-show|x-if|x-model|x-effect|x-init|x-icon|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('*'))];
@@ -1847,6 +1862,10 @@ async function runPrerender(config) {
                   node.removeAttribute(name);
                   continue;
                 }
+                if (name === 'x-icon') {
+                  node.setAttribute('x-icon', '');
+                  continue;
+                }
                 let boundAttr = '';
                 if (name.startsWith(':')) boundAttr = name.slice(1);
                 else if (name.startsWith('x-bind:')) boundAttr = name.slice('x-bind:'.length);
@@ -1920,7 +1939,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-init|x-on:|@)/;
+        const bindingAttrRegex = /^(?:x-bind:|:|x-text|x-html|x-show|x-if|x-model|x-effect|x-init|x-icon|x-on:|@)/;
         const hasVar = (expr, varName) => varName && new RegExp(`\\b${varName}\\b`).test(expr || '');
         const elementReferencesLoopScope = (el, itemVar, indexVar) => {
           if (!el) return false;
@@ -2023,6 +2042,7 @@ async function runPrerender(config) {
       html = stripPrerenderDynamicBindings(html);
       html = stripRedundantImgSrcBindings(html);
       html = stripEmptyInlineMaskStyles(html);
+      html = stripResolvedXIconDirectives(html);
       html = rewriteHtmlAssetPaths(html, fileSegments.length);
       const liveBase = config.liveUrl.replace(/\/$/, '');
       const canonicalHreflang = buildCanonicalAndHreflang(is404 ? '' : pathSeg, locales, defaultLocale, liveBase);

package/package.json

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