mnfst 0.5.218 → 0.5.220

package/lib/manifest.markdown.js

@@ -516,9 +516,11 @@ async function initializeMarkdownPlugin() {
                         markdownSource = normalizeContent(newContent);
                     }
 
-                    // Skip if no content
+                    // Skip if no content. Baked content stays visible: a reactive
+                    // expression can evaluate empty at boot (data still loading),
+                    // and hiding then would blank prerendered output.
                     if (!markdownSource || markdownSource.trim() === '') {
-                        element.style.opacity = '0';
+                        if (!hasContent) element.style.opacity = '0';
                         return;
                     }
 
@@ -607,11 +609,13 @@ async function initializeMarkdownPlugin() {
                 return;
             }
 
-            // Prerendered MPA pages with baked content are already correct — skip
-            // the effect (MPA navigation is full page loads).
-            if (hasBakedContent) {
-                return;
-            }
+            // Prerendered MPA pages with baked content are already correct on
+            // FIRST paint, but the effect still has to register: a reactive
+            // expression (x-markdown="note") must keep re-rendering when its
+            // value changes. File-backed markdown never reaches here — the
+            // prerender strips x-markdown from those (data-mnfst-md-src), so
+            // registering the effect cannot re-fetch baked articles.
+            // updateContent already no-ops when the render equals the baked HTML.
 
             // Classify the source: real expressions evaluate; bare markdown/paths
             // are quoted so Alpine doesn't choke on them.
@@ -641,15 +645,21 @@ async function initializeMarkdownPlugin() {
 
             // Track last processed content to prevent unnecessary re-renders
             let lastProcessedContent = null;
+            // Baked output is protected from empty-value hides only until the
+            // first live render replaces it; after that, clearing the value
+            // hides the element like it always did.
+            let bakedOnly = hasBakedContent;
 
             effect(() => {
                 getMarkdownContent(async (pathOrContent) => {
                     const mdDone = mdTrack();
                     try {
-                    // Reset visibility if content is empty/undefined
+                    // Reset visibility if content is empty/undefined. Baked
+                    // content stays visible: a reactive expression can evaluate
+                    // empty at boot (data still loading), and hiding then would
+                    // blank prerendered output.
                     if (!pathOrContent || pathOrContent === undefined || pathOrContent === '') {
-                        el.style.opacity = '0';
-                        hasContent = false;
+                        if (!bakedOnly) { el.style.opacity = '0'; hasContent = false; }
                         return;
                     }
 
@@ -722,8 +732,7 @@ async function initializeMarkdownPlugin() {
                     // Ensure we have a string (e.g. $route('path')?.content can be a proxy while loading)
                     const contentStr = typeof markdownContent === 'string' ? markdownContent : '';
                     if (!contentStr || contentStr.trim() === '') {
-                        el.style.opacity = '0';
-                        hasContent = false;
+                        if (!bakedOnly) { el.style.opacity = '0'; hasContent = false; }
                         return;
                     }
 
@@ -766,6 +775,7 @@ async function initializeMarkdownPlugin() {
 
                     // Show content with fade-in
                     hasContent = true;
+                    bakedOnly = false;
                     el.style.opacity = '1';
 
                     // Extract headings for anchor links