Skip to content

agent: Improve the UX around interacting with MCP servers #32622

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 22 commits into from
Jun 18, 2025
Merged

agent: Improve the UX around interacting with MCP servers #32622

merged 22 commits into from
Jun 18, 2025

Conversation

danilo-leal
Copy link
Contributor

@danilo-leal danilo-leal commented Jun 12, 2025 •
edited
Loading

Still a work in progress! Todos before merging:

  • Allow to delete (not just turn off) an MCP server from the panel's settings view
  • Also uninstall the extension upon deleting the server (check if the extension just provides MCPs)
  • Resolve repository URL again
  • Add a button to open the configuration modal from the panel's settings view
  • Improve modal UX to install and configure a non-extension MCP

Release Notes:

  • agent: Added support to delete and configure MCP servers from the panel's settings view.
  • agent: Added ability to see the source of an MCP server, whether it is from an extension or custom added.

@danilo-leal @bennetbo
Work in progress
53e8210 
Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jun 12, 2025
bennetbo and others added 21 commits June 13, 2025 09:42
@bennetbo
WIP modal
f763964
@bennetbo
Cleanup
07cb8be
@bennetbo
WIP Unify dialog
3dca8ea
@bennetbo
WIP repository URL
1702a52
@bennetbo @benbrandt
WIP
da10387 
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@bennetbo @benbrandt
Remove open logs
15fc398 
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@bennetbo @benbrandt
Handle multiple context servers from extension
baae4d7 
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@bennetbo @benbrandt
Merge branch 'main' into ag-mcp-controls
df50e16 
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@bennetbo @benbrandt
Clippy
3f2bcd1 
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@bennetbo @benbrandt
Resolve repository url for context servers coming from extensions
8bc293a 
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@bennetbo @benbrandt
WIP
ae1b78b 
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
@danilo-leal
Display source of the MCP in the settings item
e88f4fc
@danilo-leal
Clean up
7f15e35
@danilo-leal
Fix clippy
804f184
@danilo-leal
Remove unused dependency
e53f305
@danilo-leal
Disable the modal's configure button while connection is waiting
75ba296
@danilo-leal
Merge branch 'main' into ag-mcp-controls
147015b
@bennetbo @danilo-leal
Also uninstall extension when removing context server
f164c69 
Co-authored-by: Danilo Leal <daniloleal09@gmail.com>
@bennetbo @danilo-leal
Remove dbg!
132bcec 
Co-authored-by: Danilo Leal <daniloleal09@gmail.com>
@bennetbo @danilo-leal
Fix issue with remaining comma in settings
fe5414a 
Co-authored-by: Danilo Leal <daniloleal09@gmail.com>
@bennetbo @danilo-leal
Show toast when uninstalling extension is not possible
f611a7b 
Co-authored-by: Danilo Leal <daniloleal09@gmail.com>
@bennetbo bennetbo enabled auto-merge (squash) June 18, 2025 22:36
@bennetbo bennetbo merged commit 804b91a into main Jun 18, 2025
20 checks passed
@bennetbo bennetbo deleted the ag-mcp-controls branch June 18, 2025 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@danilo-leal @bennetbo