[docs] Add permission handling guidance to MCP documentation#19
Closed
github-actions[bot] wants to merge 1 commit intomainfrom
Closed
[docs] Add permission handling guidance to MCP documentation#19github-actions[bot] wants to merge 1 commit intomainfrom
github-actions[bot] wants to merge 1 commit intomainfrom
Conversation
- Add prominent warning at top of MCP overview about deny-by-default permissions - Add new 'Permission Handling' section with approve-all and custom handler examples - Update all MCP code examples to include :on-permission-request copilot/approve-all - Updated both doc/mcp/overview.md and doc/mcp/debugging.md - Updated CHANGELOG.md with documentation improvements This addresses documentation gap where MCP examples would fail due to permission denials without guidance on the deny-by-default model introduced in PR #18. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
github-actions
Bot
added
automation
documentation
Contributor
Author
|
This pull request was automatically closed because it expired on 2026-02-20T12:52:45.308Z.
|
auto-merge was automatically disabled
February 20, 2026 13:44
Pull request was closed
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
1 participant
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.
Documentation Updates - 2026-02-19
This PR updates the MCP documentation to address a critical gap: users configuring MCP servers need to understand the deny-by-default permission model introduced in PR #18.
Features Documented
approve-allconvenience handler usage in MCP contexts (from Fix permission denial keyword types and pin examples to gpt-5.2 #18)Changes Made
doc/mcp/overview.md:approve-allusage example:on-permission-request copilot/approve-alldoc/mcp/debugging.md::on-permission-request copilot/approve-allCHANGELOG.md:Merged PRs Referenced
Validation
The documentation follows the style guide:
copilot,h)Notes
This documentation change is critical for users setting up MCP servers, as they will encounter permission denials without guidance on the new deny-by-default model. All MCP examples now demonstrate the recommended pattern.