@mui/internal-docs-infra 0.12.1-canary.25 → 0.12.1-canary.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-docs-infra",
-  "version": "0.12.1-canary.25",
+  "version": "0.12.1-canary.26",
   "author": "MUI Team",
   "description": "MUI Infra - internal documentation creation tools.",
   "license": "MIT",
@@ -804,5 +804,5 @@
   "bin": {
     "docs-infra": "./cli/index.mjs"
   },
-  "gitSha": "cbb48ed31d032fb3e0f5b02d9eaf72fcd5828376"
+  "gitSha": "c0efb218192569943f75b79890cd21de4d80ee81"
 }

package/pipeline/syncPageIndex/metadataToMarkdown.mjs

@@ -1134,10 +1134,26 @@ export function metadataToMarkdown(data, options = {}) {
   return `${lines.join('\n').trimEnd()}\n`;
 }
 
+// During a build every child page re-parses the same parent index markdown, so cache
+// the parsed metadata by content. Consumers only read the result.
+const markdownToMetadataCache = new Map();
+const MARKDOWN_TO_METADATA_CACHE_LIMIT = 50;
+
 /**
  * Parses markdown content and extracts page metadata using unified
  */
 export async function markdownToMetadata(markdown) {
+  let cached = markdownToMetadataCache.get(markdown);
+  if (cached === undefined) {
+    cached = await parseMarkdownToMetadata(markdown);
+    if (markdownToMetadataCache.size >= MARKDOWN_TO_METADATA_CACHE_LIMIT) {
+      markdownToMetadataCache.clear();
+    }
+    markdownToMetadataCache.set(markdown, cached);
+  }
+  return cached;
+}
+async function parseMarkdownToMetadata(markdown) {
   const tree = unified().use(remarkParse).parse(markdown);
   let title = null;
   let description;

package/pipeline/syncPageIndex/syncPageIndex.mjs

@@ -282,56 +282,27 @@ export async function syncPageIndex(options) {
     }
   };
 
-  // Step 3: Check if any of our metadata items need updating
-  let needsUpdate = false;
-  for (const metaItem of metadataArray) {
-    const existingPageIndex = existingPages.findIndex(p => p.slug === metaItem.slug);
-    if (existingPageIndex >= 0) {
-      const existingPage = existingPages[existingPageIndex];
-      // Compare metadata - if different, we need to update
-      const existingPageJson = JSON.stringify(existingPage);
-      const newPageJson = JSON.stringify(metaItem);
-      if (existingPageJson !== newPageJson) {
-        needsUpdate = true;
-        break;
-      }
-    } else {
-      // Page doesn't exist, we need to add it
-      needsUpdate = true;
-      break;
-    }
-  }
-  if (!needsUpdate) {
+  // Step 3: Determine whether this update would change the rendered index. A field-level
+  // comparison of the raw metadata false-positives on every call (it carries volatile AST
+  // positions and fields that mergeMetadataPages fills in from the existing markdown), so
+  // render the exact markdown the writer would produce and compare it to what's on disk.
+  const relativeIndexPath = baseDir ? relative(resolve(baseDir), indexPath) : undefined;
+  const {
+    finalMarkdown: renderedMarkdown
+  } = await renderIndexPages(existingMarkdown, existingPages, metadataArray, {
+    indexTitle,
+    indexWrapperComponent,
+    preserveExistingTitleAndSlug,
+    relativeIndexPath
+  });
+  if (existingContent === renderedMarkdown) {
     await saveExistingIndexCache();
 
-    // All pages are already up-to-date, no need to acquire lock or write the index.
+    // Index is already up-to-date, no need to acquire lock or write it.
     return;
   }
-
-  // If errorIfOutOfDate is true, the writer would refuse to touch the file. The coarse
-  // per-page comparison above compares the raw in-memory metadata, which contains fields
-  // that never reach the persisted index (volatile AST node positions, or metadata that
-  // mergeMetadataPages fills in from the existing markdown). Those produce false positives.
-  // Determine whether the index is genuinely out of date by rendering the exact markdown
-  // the writer would produce and comparing it to what is committed on disk. Only fail when
-  // the rendered output actually differs. This is validation only - it never writes, so it
-  // works from the pre-lock read (existingMarkdown/existingPages) rather than re-reading
-  // under a lock like the writer does.
   if (errorIfOutOfDate) {
-    const relativeIndexPath = baseDir ? relative(resolve(baseDir), indexPath) : undefined;
-    const {
-      finalMarkdown
-    } = await renderIndexPages(existingMarkdown, existingPages, metadataArray, {
-      indexTitle,
-      indexWrapperComponent,
-      preserveExistingTitleAndSlug,
-      relativeIndexPath
-    });
-    if (existingContent !== finalMarkdown) {
-      throw new Error(`Index file is out of date: ${relativeIndexPath ?? indexPath}\n` + `Please run the validation command (or next build) locally and commit the updated index files.`);
-    }
-    await saveExistingIndexCache();
-    return;
+    throw new Error(`Index file is out of date: ${relativeIndexPath ?? indexPath}\n` + `Please run the validation command (or next build) locally and commit the updated index files.`);
   }
 
   // Step 4: Ensure the file exists before locking (proper-lockfile requires an existing file)
@@ -380,7 +351,6 @@ export async function syncPageIndex(options) {
     // Re-merge with the latest content re-read under the lock, passing the COMPLETE list
     // of pages. The returned normalized metadata is reused below to populate the cache
     // without re-parsing what we just wrote.
-    const relativeIndexPath = baseDir ? relative(resolve(baseDir), indexPath) : undefined;
     const {
       allPages,
       merged,