Skip to content

fix(docs): align open markdown links with llms routing#7529

Merged
mhartington merged 2 commits intomainfrom
aman/open-markdown-v6-fix
Feb 19, 2026
Merged

fix(docs): align open markdown links with llms routing#7529
mhartington merged 2 commits intomainfrom
aman/open-markdown-v6-fix

Conversation

@AmanVarshney01
Copy link
Member

@AmanVarshney01 AmanVarshney01 commented Feb 19, 2026

Summary

  • switch Open menu to use explicit markdown URLs (page.url + .mdx), matching Fumadocs scaffold behavior
  • update v7 and v6 docs pages to pass markdownUrl into ViewOptions
  • make /llms.mdx/[[...slug]] resolve v6-prefixed slugs and generate v6 static params with v6 prefix

Why

  • fixes mismatch between Open-menu markdown links and app routing
  • ensures v6 markdown URLs can be resolved via llms route behavior

Summary by CodeRabbit

  • New Features

    • The copy-to-clipboard button for page markdown is now always available.
  • Bug Fixes

    • Markdown view options now consistently use the correct markdown link so copy/view actions work reliably.
  • Refactor

    • Improved page resolution and routing generation for v6 content to ensure consistent access and pre-rendering.

@vercel
Copy link

vercel bot commented Feb 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Feb 19, 2026 9:22am
docs Ready Ready Preview, Comment Feb 19, 2026 9:22am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

This pull request changes the ViewOptions prop from pageUrl to an explicit markdownUrl, moving markdown URL construction to callers, and adds a resolvePage() helper in the LLMs route to unify page resolution and namespace v6 slugs consistently.

Changes

Cohort / File(s) Summary
ViewOptions Component
apps/docs/src/components/page-actions.tsx
Renamed prop pageUrlmarkdownUrl; removed internal derivation of markdown URL; updated JSDoc and memo dependencies to use markdownUrl.
Docs Page Call Sites
apps/docs/src/app/docs/(default)/[[...slug]]/page.tsx, apps/docs/src/app/docs/v6/[[...slug]]/page.tsx
Uncommented and enabled LLMCopyButton; updated ViewOptions usage to pass markdownUrl={${page.url}.mdx} (replacing pageUrl={page.url}).
LLMs Route Handler
apps/docs/src/app/llms.mdx/[[...slug]]/route.ts
Added resolvePage(slug) to centralize page lookup across v7 and v6 (handles "v6" prefix). Prefixed v6 static params with ["v6", ...] for consistent pre-render namespacing.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: aligning markdown link handling in the docs with the llms routing system by switching to explicit markdown URLs.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

🍈 Lychee Link Check Report

3660 links: ✅ 2983 OK | 🚫 11 errors | 🔀 0 redirects | 👻 664 excluded

❌ Errors

./apps/docs/content/docs.v6/(index)/prisma-orm/add-to-existing-project/mongodb.mdx

./apps/docs/content/docs.v6/(index)/prisma-orm/quickstart/mongodb.mdx

./apps/docs/content/docs.v6/guides/solid-start.mdx

./apps/docs/content/docs.v6/orm/prisma-client/setup-and-configuration/databases-connections/index.mdx

./apps/docs/content/docs/(index)/prisma-orm/add-to-existing-project/mongodb.mdx

./apps/docs/content/docs/(index)/prisma-orm/quickstart/mongodb.mdx

./apps/docs/content/docs/guides/frameworks/solid-start.mdx


Full Statistics Table
Status Count
✅ Successful 2983
🔀 Redirected 0
👻 Excluded 664
🚫 Errors 11
⛔ Unsupported 2
⏳ Timeouts 0
❓ Unknown 0

coderabbitai[bot]
coderabbitai bot previously approved these changes Feb 19, 2026
@mhartington mhartington merged commit f832022 into main Feb 19, 2026
7 of 11 checks passed
@mhartington mhartington deleted the aman/open-markdown-v6-fix branch February 19, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments