Skip to content

[docs-agent] Prefix /docs/ on wallet links breaking lychee#1268

Closed
JackReacher0807 wants to merge 2 commits intomainfrom
docs/prefix-docs-on-wallet-links
Closed

[docs-agent] Prefix /docs/ on wallet links breaking lychee#1268
JackReacher0807 wants to merge 2 commits intomainfrom
docs/prefix-docs-on-wallet-links

Conversation

@JackReacher0807
Copy link
Copy Markdown
Collaborator

Summary

Many internal links currently start with /wallets/... when they should start with /docs/wallets/.... They render correctly on the live site thanks to a fallback in the URL handling, but the lychee link checker (run on every PR) treats them as broken and fails.

This PR prefixes /docs/ to every offending wallet link (27 instances across 6 MDX files):

  • content/wallets/pages/third-party/signers/openfort.mdx
  • content/wallets/pages/third-party/signers/privy.mdx
  • content/wallets/pages/third-party/signers/turnkey.mdx
  • content/wallets/wallet-integrations/privy/signer-migration-overview.mdx
  • content/wallets/wallet-integrations/privy/jwt-auth-migration.mdx
  • content/wallets/wallet-integrations/privy/react-migration.mdx

It also updates content/wallets/CONTRIBUTING.md so the contributor guidance reflects the correct format. The previous wording told writers to use /wallets/... as the relative-link form, which is how this kept getting reintroduced (most recently in #1230, before that re-creeping into the third-party signer pages after the link-normalization PR #1068).

Verification: grep -rnE '\]\(/wallets[/)#]' content/ now returns no matches.

Linear

DOCS-68 — https://linear.app/alchemyapi/issue/DOCS-68/prepend-docs-to-wallet-links-breaking-lychee-link-checker

Requested by

@dslovinsky (via Slack thread)

Many internal links currently start with /wallets/... when they should start with /docs/wallets/.... They work on the live site due to fallback formatting, but lychee fails on every PR.

Fixes 27 link instances across 6 wallets MDX files (third-party signer pages and the Privy signer-migration guides) and updates content/wallets/CONTRIBUTING.md so future contributors see the correct format.

Refs DOCS-68

Requested-by: @dslovinsky
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

🔗 Preview Mode

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Apr 27, 2026, 4:35 PM

@SahilAujla
Copy link
Copy Markdown
Collaborator

closing as it's a duplicate of #1267

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

🔍 Link Check

Status: ❌ Failed

Summary

Status Count
🔍 Total 8012
✅ Successful 6868
⏳ Timeouts 0
🔀 Redirected 460
👻 Excluded 676
❓ Unknown 0
🚫 Errors 7
⛔ Unsupported 1
Broken links (7) — click to expand

Errors per input

Errors in ./content/tutorials/snapshots/avalanche.mdx

Errors in ./content/wallets/wallet-integrations/privy/jwt-auth-migration.mdx

Errors in ./content/api-reference/base/base-api-faq.mdx

Errors in ./content/wallets/wallet-integrations/privy/signer-migration-overview.mdx

Errors in ./content/tutorials/snapshots/aptos.mdx

Errors in ./content/api-reference/arbitrum-nova/arbitrum-nova-deprecation-notice.mdx

Errors in ./content/tutorials/getting-started/developer-best-practices/choosing-a-web3-network.mdx

View workflow run

dslovinsky
dslovinsky previously approved these changes Apr 27, 2026
@github-actions github-actions Bot dismissed dslovinsky’s stale review April 27, 2026 16:35

@dslovinsky you are listed as the originator of this docs request (via the Requested-by trailer on a docs-agent commit). Per the docs-agent self-review policy, the originator can't approve their own request. Please ask another team member to review.

@JackReacher0807
Copy link
Copy Markdown
Collaborator Author

Closing this PR. Its 27 link-fix changes overlapped with #1267 (now merged), leaving only the content/wallets/CONTRIBUTING.md update as net-new. Reopened that single change as #1269 so the contributor-guideline edit lands as a clean, single-purpose PR.

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.

3 participants