Skip to content

feat: persist MCP install preferences - #1772

Merged
RhysSullivan merged 2 commits into
UsefulSoftwareCo:mainfrom
baggiiiie:fix/persist-install-pref
Aug 28, 2026
Merged

feat: persist MCP install preferences#1772
RhysSullivan merged 2 commits into
UsefulSoftwareCo:mainfrom
baggiiiie:fix/persist-install-pref

Conversation

@baggiiiie

Copy link
Copy Markdown
Contributor

previously, when user clicks advanced dropdown under the command for connecting agents, the options do not persist after page refresh (web) or desktop app restart:

before.mp4

this PR adds local storage to persist the options

after.mp4

note

i understand the options in advanced is only for building the visible command, this PR is changing for a better user experience when adding agents. tho i also wonder, if these options should become a server/daemon level config, as when im experimenting with the newly added ?search_tools=true, it was awkward to update all of the mcp configs i have for different agents

baggiiiie and others added 2 commits August 27, 2026 16:46
localStorage is per-origin, not per-account, so one key carried an org's
transport and elicitation choices into every other org and every other
user signed in through the same browser. The rendered install command
differs per org, so the shared preference was wrong rather than merely
surprising.

Key the preferences by the active org slug, with one shared bucket for
hosts that are not org-scoped (local, desktop). Nothing is migrated: the
old global key is simply left unread, so those users fall back to
defaults once.

Switching orgs reloads that org's preferences during render rather than
in an effect, so the save effect cannot write the previous org's choices
under the new org's key.
@RhysSullivan
RhysSullivan merged commit 597ce90 into UsefulSoftwareCo:main Aug 28, 2026
75 of 76 checks passed
This was referenced Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants