⚡️ Liquid-only TOC title/intro rendering (#60573)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: heiskr

src/article-api/lib/get-all-toc-items.ts

@@ -1,10 +1,13 @@
 import type { Context, Page } from '@/types'
 import type { LinkData } from '@/article-api/transformers/types'
 import { resolvePath } from './resolve-path'
+import { renderLiquid } from '@/content-render/liquid/index'
 
 interface PageWithChildren extends Page {
   children?: string[]
   category?: string[]
+  rawTitle: string
+  rawIntro?: string
 }
 
 interface TocItem extends LinkData {
@@ -26,14 +29,11 @@ export async function getAllTocItems(
   page: Page,
   context: Context,
   options: {
-    recurse?: boolean
-    renderIntros?: boolean
     /** Only recurse into children whose resolved path starts with this prefix.
      *  Prevents cross-product traversal (e.g. /en/rest listing /enterprise-admin). */
     basePath?: string
   } = {},
 ): Promise<TocItem[]> {
-  const { recurse = true, renderIntros = true } = options
   const pageWithChildren = page as PageWithChildren
   const languageCode = page.languageCode || 'en'
 
@@ -70,19 +70,15 @@ export async function getAllTocItems(
       )
       const href = childPermalink ? childPermalink.href : childHref
 
-      const title = await childPage.renderTitle(context, { unwrap: true })
-
-      let intro = ''
-      if (renderIntros && childPage.intro) {
-        intro = await childPage.renderProp('intro', context, { textOnly: true })
-      }
+      const title = await renderPropFast(childPage, 'title', context)
+      const intro = await renderPropFast(childPage, 'intro', context)
 
       const category = childPage.category || []
 
       // Only recurse if the child is within the same product section
       const withinSection = href.startsWith(basePath)
       const childTocItems =
-        recurse && withinSection && childPage.children && childPage.children.length > 0
+        withinSection && childPage.children && childPage.children.length > 0
           ? await getAllTocItems(childPage, context, { ...options, basePath })
           : []
 
@@ -139,3 +135,38 @@ export function flattenTocItems(
   recurse(tocItems)
   return result
 }
+
+/**
+ * Check whether a string contains markdown link syntax that would need
+ * processing by the unified pipeline (e.g. link rewriting, AUTOTITLE).
+ *
+ * Use this to short-circuit expensive rendering when the text is
+ * Liquid-only and contains no markdown that needs transformation.
+ */
+function hasMarkdownLinks(text: string): boolean {
+  return text.includes('[') && text.includes('](/')
+}
+
+const RAW_PROP_MAP = {
+  title: 'rawTitle',
+  intro: 'rawIntro',
+} as const
+
+/**
+ * Fast-path rendering for page properties. Renders Liquid only, skipping
+ * the full unified pipeline. Falls back to page.renderProp() when the
+ * Liquid output contains markdown links that need rewriting.
+ */
+async function renderPropFast(
+  page: PageWithChildren,
+  prop: keyof typeof RAW_PROP_MAP,
+  context: Context,
+): Promise<string> {
+  const raw = page[RAW_PROP_MAP[prop]]
+  if (!raw) return ''
+  const rendered = await renderLiquid(raw, context)
+  if (hasMarkdownLinks(rendered)) {
+    return page.renderProp(prop, context, { textOnly: true })
+  }
+  return rendered.trim()
+}

src/article-api/transformers/bespoke-landing-transformer.ts

@@ -95,10 +95,7 @@ export class BespokeLandingTransformer implements PageTransformer {
     // Note: For bespoke-landing pages, the site shows ALL articles regardless of includedCategories
     // (includedCategories only filters for discovery-landing pages)
     if (bespokePage.children && bespokePage.children.length > 0) {
-      const tocItems = await getAllTocItems(page, context, {
-        recurse: true,
-        renderIntros: true,
-      })
+      const tocItems = await getAllTocItems(page, context)
 
       // Flatten to get all leaf articles (excludeParents: true means only get articles, not category pages)
       const allArticles = flattenTocItems(tocItems, { excludeParents: true })

src/article-api/transformers/discovery-landing-transformer.ts

@@ -132,10 +132,7 @@ export class DiscoveryLandingTransformer implements PageTransformer {
     // recursion (e.g. /rest listing /enterprise-admin children that
     // point outside the /rest hierarchy).
     if (discoveryPage.children && discoveryPage.children.length > 0) {
-      const tocItems = await getAllTocItems(page, context, {
-        recurse: true,
-        renderIntros: true,
-      })
+      const tocItems = await getAllTocItems(page, context)
 
       // Flatten to get all leaf articles (excludeParents: true means only get articles, not category pages)
       let allArticles = flattenTocItems(tocItems, { excludeParents: true })