-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Fix broken links #5447
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix broken links #5447
Conversation
|
| ## Configuring Transports in Kilo Code | ||
|
|
||
| For detailed information on configuring STDIO and SSE transports in Kilo Code, including example configurations, see the [Understanding Transport Types](/features/mcp/using-mcp-in-kilo-code#understanding-transport-types) section in the Using MCP in Kilo Code guide. | ||
| For detailed information on configuring STDIO and SSE transports in Kilo Code, including example configurations, see the [Understanding Transport Types](using-mcp-in-kilo-code#understanding-transport-types) section in the Using MCP in Kilo Code guide. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CRITICAL: Broken link - filename mismatch
The relative link using-mcp-in-kilo-code doesn't match the actual file using-in-kilo-code.md. This will result in a 404 error.
| For detailed information on configuring STDIO and SSE transports in Kilo Code, including example configurations, see the [Understanding Transport Types](using-mcp-in-kilo-code#understanding-transport-types) section in the Using MCP in Kilo Code guide. | |
| For detailed information on configuring STDIO and SSE transports in Kilo Code, including example configurations, see the [Understanding Transport Types](using-in-kilo-code#understanding-transport-types) section in the Using MCP in Kilo Code guide. |
Code Review SummaryStatus: No Issues Found | Recommendation: Merge OverviewThis PR updates broken MCP documentation links from the old
All link targets have been verified to exist and the changes are consistent with the documentation structure. Files Reviewed (7 files)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request fixes broken documentation links that were orphaned after the documentation restructuring in PR #5401. The changes update all references to MCP documentation pages to use the correct paths and filenames.
Changes:
- Updated absolute path references from
/features/mcp/*to/docs/automate/mcp/*for cross-directory links - Updated relative path references within the
/automate/mcp/directory to use the new filenameusing-in-kilo-code(renamed fromusing-mcp-in-kilo-code) - Ensured all link changes follow the documentation style guide (absolute paths starting with
/docs/for cross-directory links, relative paths for within-directory links)
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| apps/kilocode-docs/pages/getting-started/faq.md | Updated MCP overview link to use correct absolute path |
| apps/kilocode-docs/pages/customize/workflows.md | Updated MCP integrations link to use correct absolute path |
| apps/kilocode-docs/pages/contributing/index.md | Updated MCP Servers documentation link to use correct absolute path |
| apps/kilocode-docs/pages/code-with-ai/agents/free-and-budget-models.md | Updated MCP usage guide link to use correct absolute path and new filename |
| apps/kilocode-docs/pages/automate/mcp/what-is-mcp.md | Updated internal MCP documentation links to use relative paths with new filenames |
| apps/kilocode-docs/pages/automate/mcp/using-in-kilo-code.md | Updated transport documentation links to use relative paths |
| apps/kilocode-docs/pages/automate/mcp/server-transports.md | Updated link to transport types section with correct relative path |
| apps/kilocode-docs/pages/automate/mcp/overview.md | Updated all MCP documentation section links to use relative paths with new filenames |
|
@tris We did a big refactor on the docs so this was fixed meanwhile but thanks a lot regardless! 🚀 |
|
@kevinvandijk I noticed that the 404s/redirects were fixed, but the links actually weren't if you look closely. (reopen please? :) |
Just correcting some orphaned links from #5401:
/features/mcp/overview->/docs/automate/mcp/overview/features/mcp/using-mcp-in-kilo-code->/docs/automate/mcp/using-in-kilo-code/features/mcp/what-is-mcp->/docs/automate/mcp/what-is-mcp/features/mcp/server-transports->/docs/automate/mcp/server-transports/features/mcp/mcp-vs-api->/docs/automate/mcp/mcp-vs-api