[Docs] Fix broken cross-page links in cookbook and Ascend NPU docs#29518
Open
ajinkyajawale14499 wants to merge 1 commit into
Open
[Docs] Fix broken cross-page links in cookbook and Ascend NPU docs#29518ajinkyajawale14499 wants to merge 1 commit into
ajinkyajawale14499 wants to merge 1 commit into
Conversation
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates several documentation files by replacing relative file-based markdown links with clean, absolute-like path links without file extensions. There are no review comments, and the changes look correct and consistent.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Four internal documentation links used relative paths with .md/.mdx file extensions. Under Mintlify these 404: routes are extensionless and the rest of the docs/cookbook reference pages via absolute /docs and /cookbook routes (docs.json nav carries no file extensions). Convert the four links to the canonical site-route form: - cookbook/autoregressive/Jina/Jina-reranker-m0.mdx -> DeepSeek-V3.2 - cookbook/autoregressive/DeepSeek/DeepSeek-V4.mdx -> native_api anchor - docs/hardware-platforms/ascend-npus/diffusion/disaggregation.mdx -> NPU install guide + sglang-diffusion disaggregation guide All four target pages and the referenced anchor were verified to exist. Signed-off-by: ajinkyajawale14499 <ajinkyajawale14499@gmail.com>
5d85f7a to
8dcc4b8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Four internal documentation links in the cookbook and Ascend NPU docs use relative paths with
.md/.mdxfile extensions. Under Mintlify these resolve to 404s: site routes are extensionless, and the rest of the docs/cookbook reference pages via absolute/docsand/cookbookroutes (docs.jsonnavigation carries no file extensions). This fixes the broken cross-page navigation.This is the same class of fix as the recently merged #29261 ("[Docs] Fix broken links in cookbook").
Modifications
Converted four links to the canonical site-route form (extensionless, absolute):
docs_new/cookbook/autoregressive/Jina/Jina-reranker-m0.mdx../DeepSeek/DeepSeek-V3_2.md/cookbook/autoregressive/DeepSeek/DeepSeek-V3_2docs_new/cookbook/autoregressive/DeepSeek/DeepSeek-V4.mdx../../../docs/basic_usage/native_api.mdx#capture-.../docs/basic_usage/native_api#capture-...docs_new/docs/hardware-platforms/ascend-npus/diffusion/disaggregation.mdx../ascend_npu.mdx/docs/hardware-platforms/ascend-npus/ascend_npu../../../sglang-diffusion/disaggregation.mdx/docs/sglang-diffusion/disaggregationAll four target pages were verified to exist, and the
#capture-expert-selection-distribution-in-moe-modelsanchor is a real heading innative_api.mdx. The chosen route form matches how every other page links these same targets.Accuracy Tests
N/A — documentation-only change, no code paths affected.
Speed Tests and Profiling
N/A — documentation-only change.
Checklist
Disclosure: this contribution was prepared with AI assistance (Claude Code). The broken links were found by scanning
docs_newfor links whose Mintlify route does not resolve, and each fix was verified locally against the actual docs tree anddocs.jsonnav convention.CI States
Latest PR Test (Base): ✅ Run #28306043601
Latest PR Test (Extra): ❌ Run #28306043551