[drishtyraghav0001]

What kind of change does this PR introduce?

Bugfix

Issue Number:

• Closes 🐛 Bug: Headings with inline code or formatting produce truncated anchor IDs, breaking TOC and internal links #2576

Screenshots/videos:

n/a, heading ids only. checked in the built html that the draft-06 release notes heading id now matches its TOC link

If relevant, did you update the documentation?

no

Summary

slugifyMarkdownHeadline was dropping every child that wasn't a plain string, so headings with inline code, bold, italic or links got cut-off ids and the TOC links didn't scroll anywhere. it now pulls text out of nested children recursively (strings, numbers, arrays, react elements, md nodes). the [#custom-id] fragment still works the same.

the two draft-06 release notes pages had hand written TOC links pointing at the old truncated ids, so I updated those 7 links to the new ids. I scanned all headings under pages/ and 19 ids changed, all headings with formatting in them. no other links in the repo pointed at the old ids.

added tests in cypress/components/slugifyMarkdownHeadline.cy.tsx and one in Headlines.cy.tsx. lint, typecheck, build and the full component suite pass locally.

Does this PR introduce a breaking change?

no. external deep links to the old truncated ids on those 19 headings would stop matching, but they were broken from the TOC anyway

Checklist

• Read, understood, and followed the contributing guidelines.

[github-actions]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name	Status	Preview	Last Commit
website	✅ Ready (View Log)	Visit Preview	fe5b1de

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (92f9c18) to head (fe5b1de).

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #2579   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           31        31           
  Lines          719       719           
  Branches       223       223           
=========================================
  Hits           719       719           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[drishtyraghav0001]

@playlikeboy quick q. ids for quoted inline code keep the quotes now (...-with-"additionalproperties"), same as plain text headings already do. your example had them stripped, happy to change that if you prefer. I also updated the TOC links in the draft-06 release notes that pointed at the old cut off ids. does this look right to you?

[playlikeboy]

Looks great! Keeping quotes is definitely the right call since it stays consistent with how plain-text headings are slugified across the rest of the site. Tested locally and on the Cloudflare preview — all the TOC links scroll to the right headings now. Thanks for putting this together!

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

[shivaminfo1]

Thanks @drishtyraghav0001, nice cleanup of the heading slug logic and solid test coverage. LGTM!

Posted from the wrong account by mistake.

[vtushar06]

Thanks @drishtyraghav0001, nice cleanup of the heading slug logic and solid test coverage. LGTM!