fix(mcp): let users select both web and claude code together.#667
Open
gewenyu99 wants to merge 3 commits into
Open
fix(mcp): let users select both web and claude code together.#667gewenyu99 wants to merge 3 commits into
gewenyu99 wants to merge 3 commits into
Conversation
The client picker is a multi-select but flagged the Claude Desktop/Web connector 'exclusive', so picking it silently cleared every other client (and vice-versa) with no indication why. The exclusivity was only there to avoid a mixed post-pick flow — but both phases already install the full selection. Drop the flag; route a mixed pick through the feature picker (doInstall opens the connector page too) and reserve the connector-only screen for a connector-only selection. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
🧙 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. |
…r steps
When both a local editor and the browser connector are picked, they no
longer install at once. Local editors install through the feature picker,
then the connector opens as a deliberate next step. The Working screen names
what's installing, and the connector handoff confirms what just installed
("MCP installed for Cursor") and frames the browser step as what's next.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- Mixed pick installs local editors first, then the browser connector opens as a deliberate step that waits for the user to finish (no auto-advance). - Working/connector/Done screens name what's installing and what's next; the Done summary shows before the tutorial. - Auth-wait screen surfaces the esc-to-cancel hint instead of looking like a hang; a 'Signed in' confirmation step follows auth before the tutorial. - Looser spacing on the install picker and connector screens. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
gewenyu99
changed the title
gewenyu99
commented
Jun 15, 2026
Collaborator
Author
There was a problem hiding this comment.
I knowww, it's huge nowwww.
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.
Why
Right now, you can only select Claude Web vs Claude Code.
Not a technical constraint — an accidental regression from #442.
mode="multi"picker. Mixed picks worked: the post-pick logic (anyNeedsFeatures = clientNames.some(name => !info?.finish)) routed any selection containing a local editor through the feature picker, and connector-only selections installed directly. Selecting the web connector alongside a local editor was fine.Phase.Connectorscreen. As part of that it addedexclusive: Boolean(c.finish)to the picker and replacedanyNeedsFeatureswithisConnector → Phase.Connector, which doesn't handle a mix. The commit comment frames the connector as "connector-only … show its own screen" — a code convenience to avoid threading a mixed selection through the new screen, not a real conflict. (The PR also noted the manual TUI walkthrough was "still owed before merge.")So a multi-select silently became single-select for the connector: picking Claude Desktop/Web cleared every other client (and vice-versa), with no label or disabled state explaining why.
Fix
Screen.Recording.2026-06-15.at.5.03.29.PM.mov
Both post-pick phases already
doInstallthe full selection (Phase.FeatureSelectwith features,Phase.Connectorwith[]), so a mixed install needs no new plumbing — the connector just ignores the feature list. So:exclusivefrom the picker option (free multi-select again).Phase.FeatureSelect(installs local editors with chosen features and opens the connector page).Phase.Connectorfor a connector-only selection.Effectively restores the pre-#442 "mixed picks route through the feature picker" behavior, adapted to the new phase structure.