mnfst-render 0.5.10 → 0.5.11

package/manifest.render.mjs

@@ -1271,6 +1271,49 @@ function buildSubstitutionPairs(defaultLocaleData, targetLocaleData) {
   return pairs;
 }
 
+/**
+/**
+ * Prefix internal navigation links with the target locale so that prerendered
+ * MPA pages link directly to the correct locale variant (e.g. /platform →
+ * /fr/platform on the French page).  Without this, users must rely on runtime
+ * JS interception (`installMpaStickyLocaleLinks`) which may not be ready by
+ * the time they click — causing navigation to fall back to English.
+ *
+ * Only rewrites `<a href="...">` where the href is a root-relative path that
+ * doesn't already carry a locale prefix and isn't an excluded route.
+ */
+function prefixLocaleInternalLinks(html, locale, locales, localeRouteExclude) {
+  if (!locale || !locales || !locales.length) return html;
+  const localeSet = new Set(locales);
+  const excludeSet = new Set(localeRouteExclude || []);
+
+  // Match <a ... href="..." ...>  — capture the href value
+  return html.replace(
+    /(<a\b[^>]*\shref=["'])(\/?[^"'#][^"']*)(["'][^>]*>)/gi,
+    (full, prefix, href, suffix) => {
+      // Only process root-relative paths
+      if (!href.startsWith('/')) return full;
+      // Skip external protocols embedded as relative (shouldn't happen but guard)
+      if (/^\/\//.test(href)) return full;
+
+      const withoutSlash = href.replace(/^\//, '');
+      const firstSeg = withoutSlash.split('/')[0].split('#')[0].split('?')[0];
+
+      // Already has a locale prefix
+      if (localeSet.has(firstSeg)) return full;
+
+      // Skip asset-like paths
+      if (/\.(css|js|json|svg|png|jpg|jpeg|gif|ico|woff2?|ttf|eot|pdf|xml|txt)$/i.test(href)) return full;
+
+      // Respect localeRouteExclude — these routes stay locale-neutral
+      if (excludeSet.has(firstSeg)) return full;
+
+      // Prefix with locale
+      return `${prefix}/${locale}${href}${suffix}`;
+    }
+  );
+}
+
 /**
  * Apply locale text substitution to rendered HTML.
  * Replaces content in text nodes (between > and <) and in key attributes:
@@ -1334,6 +1377,12 @@ function generateLocaleVariantHtml({
   // Apply locale text substitution
   html = applyLocaleSubstitution(html, substitutionPairs);
 
+  // Prefix internal <a> links with the locale so MPA navigation stays in-locale
+  // without relying on runtime JS interception.
+  if (targetLocale && targetLocale !== defaultLocale) {
+    html = prefixLocaleInternalLinks(html, targetLocale, locales, config.localeRouteExclude);
+  }
+
   // Standard Node.js post-processing (same sequence as processPath)
   html = stripDevOnlyContent(html);
   html = stripInjectedPluginScripts(html);
@@ -1778,7 +1827,13 @@ async function runPrerender(config) {
         console.error('  npm i -D puppeteer-core @sparticuz/chromium');
         process.exit(1);
       }
-      return await puppeteer.default.launch({ headless: true });
+      return await puppeteer.default.launch({
+        headless: true,
+        args: [
+          '--no-sandbox',
+          '--disable-setuid-sandbox',
+        ],
+      });
     }
   }
   let browser = await launchBrowser();
@@ -2398,6 +2453,18 @@ async function runPrerender(config) {
               const src = source[name];
               const cur = name in currentAttrs ? currentAttrs[name] : null;
               if (src !== cur) {
+                // If the source value is null (attribute didn't exist originally)
+                // but a reactive Alpine binding controls this attribute, skip the
+                // restoration.  Alpine will re-evaluate the binding on init and
+                // set the correct value; nulling it in the contract would flash
+                // the element unstyled until Alpine + async data loads catch up.
+                // The baked value IS the correct initial render.
+                if (src === null) {
+                  const hasBinding =
+                    (name === 'style' && (':style' in source || 'x-bind:style' in source)) ||
+                    (name === 'class' && (':class' in source || 'x-bind:class' in source));
+                  if (hasBinding) continue;
+                }
                 attrsOut[name] = src; // may be null (means "remove this attribute")
                 dirty = true;
               }
@@ -2798,6 +2865,13 @@ async function runPrerender(config) {
       html = stripEmptyInlineMaskStyles(html);
       html = stripResolvedXIconDirectives(html);
       html = markPrerenderedManifestComponents(html);
+
+      // Prefix internal <a> links with the locale for non-default locales so
+      // MPA navigation stays in-locale without relying on runtime JS.
+      if (currentLocale && currentLocale !== defaultLocale) {
+        html = prefixLocaleInternalLinks(html, currentLocale, locales, config.localeRouteExclude);
+      }
+
       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.5.10",
+  "version": "0.5.11",
   "description": "Render Manifest sites to static HTML for SEO",
   "type": "module",
   "bin": {