docs(perps): fix broken relative links in knowledge docs - #74
Open
ZayanKhan-12 wants to merge 1 commit into
Open
docs(perps): fix broken relative links in knowledge docs#74ZayanKhan-12 wants to merge 1 commit into
ZayanKhan-12 wants to merge 1 commit into
Conversation
All relative links in domains/perps/knowledge were broken, from two causes dating to the import from metamask-mobile docs/perps: - Links to perps-connection-architecture.md and perps-screens.md kept the original filenames, but the files were imported with the perps- prefix stripped. Rewritten to ./connection-architecture.md and ./screens.md. - Links to perps-sentry-reference.md, perps-metametrics-reference.md, perps-ab-testing.md, and hyperliquid/ point at files that were never imported. Rewritten to their canonical locations in MetaMask/metamask-mobile docs/perps. The backticked docs/perps paths in review-antipatterns.md are left as-is: they are consumer-repo paths that resolve inside a metamask-mobile checkout. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
2 participants
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.
Description
Every relative markdown link in
domains/perps/knowledge/(23 links acrossarchitecture.mdandfeature-flags.md) was broken, from two causes dating to the import of these docs from metamask-mobile'sdocs/perps/:1. Files renamed on import, links not rewritten (11 links)
The docs were imported with the
perps-prefix stripped from filenames, but intra-doc links kept the old names:./perps-connection-architecture.md→./connection-architecture.md(7×)./perps-screens.md→./screens.md(4×)2. Referenced files never imported (12 links)
These targets don't exist anywhere in this repo — they live only in
MetaMask/metamask-mobileunderdocs/perps/(verified via the GitHub contents API). Rewritten to their canonical locations:./perps-sentry-reference.md→ [metamask-mobile blob URL] (3×)./perps-metametrics-reference.md→ [metamask-mobile blob URL] (4×)./perps-ab-testing.md→ [metamask-mobile blob URL] (2×)./hyperliquid/→ [metamask-mobile tree URL] (3×)Not touched: the backticked
docs/perps/...references inreview-antipatterns.md, which appear to be intentional consumer-repo paths that resolve inside a metamask-mobile checkout.Happy to switch the external links to repo-relative
docs/perps/...paths instead if that convention is preferred for skills consumed inside metamask-mobile.Verification
MetaMask/metamask-mobiledocs/perps/node --test test/*.test.mjs— 36/36 pass🤖 Generated with Claude Code