Add MCP App form deferral opt-out#2921
Open
connor4312 wants to merge 1 commit into
Open
Conversation
Allow clients to keep MCP App views enabled while making form-backed write tools execute directly when explicitly configured. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a user-controlled feature flag to bypass MCP App form deferral while preserving UI metadata and result views.
Changes:
- Adds and documents
mcp_apps_disable_form_deferral. - Applies the opt-out across form-backed issue and pull-request tools.
- Tests feature resolution, HTTP headers, insiders mode, and deferral behavior.
Show a summary per file
| File | Description |
|---|---|
pkg/http/server_test.go |
Tests HTTP-header and insiders behavior. |
pkg/github/ui_capability.go |
Adds the form-deferral opt-out check. |
pkg/github/ui_capability_test.go |
Tests deferral flag combinations. |
pkg/github/feature_flags.go |
Defines and allows the new flag. |
pkg/github/feature_flags_test.go |
Tests feature resolution behavior. |
docs/server-configuration.md |
Documents remote and local configuration. |
Review details
- Files reviewed: 6/6 changed files
- Comments generated: 0
- Review effort level: Medium
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.
MCP Apps in the GitHub MCP Server are useful, but having issue and pull request tools open a form instead of performing the requested operation can be disruptive, especially in autopilot workflows where autonomous task completion is expected.
This adds the user-controllable
mcp_apps_disable_form_deferralfeature flag. When enabled alongsideremote_mcp_ui_apps, form-backed write tools execute directly while MCP App result views and UI metadata remain enabled. HTTP clients can opt in per request throughX-MCP-Features.Testing
script/lintscript/testissue_writeto its interactive formissue_writeMCP App UI metadata