Merged
Conversation
Reduce 202 → 162 lines (-19.8%) by condensing verbose intro and Important note, removing duplicate microsoftdocs HTTP example, consolidating mcp add code block, and merging External Resources into Related Documentation. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
pelikhan
approved these changes
Mar 25, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR streamlines the “Using Custom MCP Servers” guide by removing redundant explanations and consolidating examples while keeping the core configuration guidance intact.
Changes:
- Condensed the introductory and IMPORTANT sections to reduce repetition and improve readability.
- Removed duplicate/verbose examples (notably the repeated HTTP example) and tightened CLI usage snippets.
- Merged the “External Resources” links into the existing “Related Documentation” section.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## Adding MCP Servers from the Registry | ||
|
|
||
| The easiest way to add MCP servers is using the GitHub MCP registry with the `gh aw mcp add` command: | ||
| Use `gh aw mcp add` to browse and add servers from the GitHub MCP registry (default: `https://api.mcp.github.com/v0`): |
There was a problem hiding this comment.
The default MCP registry URL here is documented as https://api.mcp.github.com/v0, but the CLI/codebase defaults to https://api.mcp.github.com/v0.1 (see pkg/constants/constants.go and pkg/cli/mcp_add.go). Please update the docs to match the actual default to avoid users pointing at the wrong registry endpoint.
Suggested change
| Use `gh aw mcp add` to browse and add servers from the GitHub MCP registry (default: `https://api.mcp.github.com/v0`): | |
| Use `gh aw mcp add` to browse and add servers from the GitHub MCP registry (default: `https://api.mcp.github.com/v0.1`): |
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.
Reduces
docs/src/content/docs/guides/mcps.mdfrom 202 to 162 lines (-19.8%) while preserving all essential technical content.What was removed
afe outputs→safe outputsmicrosoftdocsserver appeared in both the "Manually Configuring" section and the "HTTP MCP Servers" section; removed the duplicate from the latter["*"]..." paragraph: Integrated into the code comment in the MCP Tool Filtering sectionScreenshots
Screenshots could not be captured: Playwright runs with
--network host(Docker host perspective) but the agent container's bridge IP (172.30.0.20) is on a separate subnet not routable from the Docker host. Thecurlcheck confirms the page returns HTTP 200.References: