Repository navigation
docs: fix dead VersionFox and MCP documentation links - #11117
Conversation
- VersionFox homepage vfox.lhan.me no longer resolves; point glossary at vfox.dev - MCP root URL returns bare 308; link the stable getting-started intro page
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated vfox references to ChangesDocumentation Link Updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis is a docs-only fix replacing three broken external links with their working replacements. No runtime or logic changes are made.
Confidence Score: 5/5Safe to merge — purely swaps three dead external URLs for working ones with no code or behavior changes. All three changes are straightforward URL replacements in documentation files. The new targets (vfox.dev and modelcontextprotocol.io/docs/getting-started/intro) are verified to return 200 per the PR description, and no logic, configuration, or runtime behavior is touched. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "docs: fix remaining stale VersionFox lin..." | Re-trigger Greptile |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/glossary.md`:
- Line 87: Update the stale VersionFox URL in the dummy plugin README from
https://vfox.lhan.me/ to https://vfox.dev/, preserving the surrounding
documentation and link text.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Central YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: f01e2234-7922-466e-a465-7a28621292b0
📒 Files selected for processing (2)
docs/glossary.mddocs/mcp.md
|
Good catch @coderabbitai — fixed the remaining stale |
|
Tip For best results, initiate chat on the files or code changes.
If you found this review helpful, would you consider giving us a shout-out on X? Thank you for using CodeRabbit! |
|
Thanks CodeRabbit — addressed in the follow-up commit on this branch (crates/vfox/plugins/dummy/README.md: VersionFox homepage updated to the current site). Full PR now covers glossary + dummy plugin README + MCP intro link. CI remains green. |
Summary
vfox.lhan.meDNS NXDOMAIN → vfox.dev).https://modelcontextprotocol.io/returns HTTP 308; point at the stable intro page that resolves 200.Motivation
Readers hitting these external links from the docs currently get hard failures (DNS or redirect-only roots without a comfortable landing).
Verification
Docs-only; no runtime change.
Agent-Owner:
sera· Platform: hermes · Claim-TTL: 24hClaim: sera
Summary by CodeRabbit