feat(mcp): granular feature selection with plugin/MCP decoupling#442
Open
skoob13 wants to merge 5 commits into
Open
feat(mcp): granular feature selection with plugin/MCP decoupling#442skoob13 wants to merge 5 commits into
skoob13 wants to merge 5 commits into
Conversation
🧙 Wizard CIRun the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands: Test all apps:
Test all apps in a directory:
Test an individual app:
Show more apps
Results will be posted here when complete. |
VojtechBartos
approved these changes
May 7, 2026
Member
VojtechBartos
left a comment
There was a problem hiding this comment.
Thanks for fixing the features passing 🤟
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.
Problem
Pain points in the MCP install flow:
promptswas gone server-side, and 6 newer features (web_analytics, signals, tracing, sdk_doctor, batch_exports, customer_analytics) weren't surfaced.Recommended installation
Configurable installation
Changes
--featuresflag still bypasses to the legacy yes/no path.Claude CodeandCodex.addServernow callsclaude mcp add --transport http …/codex mcp add … --url … --bearer-token-env-var …directly. Routing:Select editor to installand each editor shows whether it'll be installed as(plugin)or(MCP).getSupportedClients. It's the only client still using themcp-remotestdio wrapper since Claude Desktop has no native HTTP MCP support.tool-definitions*.jsonregistry. Removedprompts; addedweb_analytics,customer_analytics,signals(Data & Analytics),sdk_doctor,visual_review,tracing(Development Tools),batch_exports(Data Management). 41 features total. Skippedfeedbacksince the server marks italways_available.mcp completeevent now carriesmcp_features_selected: 'all' | string[]so we can see which features users actually opt into.isAllFeaturesSelectedhelper from insidebuildMCPUrl. URLs with all features selected omit the?features=param entirely.How did you test this code?
addServer/isServerInstalledpaths for Codex, and the existing Claude Code plugin methods are unaffected.pnpm try mcp addfor the all, customize, and no thanks branches still owed before merge — including verification thatclaude mcp list/codex mcp listshow the expected entry per mode.Publish to changelog?
no