Skip to content

Studio: open the MCP dialog to the server list#6100

Merged
Imagineer99 merged 2 commits into
unslothai:mainfrom
oobabooga:studio/mcp-manage-servers-discoverable
Jun 8, 2026
Merged

Studio: open the MCP dialog to the server list#6100
Imagineer99 merged 2 commits into
unslothai:mainfrom
oobabooga:studio/mcp-manage-servers-discoverable

Conversation

@oobabooga

Copy link
Copy Markdown
Contributor

Currently the "Add custom MCP" button opens the dialog to add a new server; to reach the dialog to manage servers (edit, remove, etc), it's necessary to first open that dialog and then click Cancel.

This is unintuitive and a UX gap, see for instance #6042.

Proposed fix

Replace "Add custom MCP" with "Manage MCP servers", which opens the dialog with the list of servers. To add a new server, the user simply clicks "Add server" in that dialog.

Closes #6042

Maybe makes #6030 unnecessary.

image

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the openToCreate prop from ChatMcpServersDialog and renames the "Add custom MCP" option to "Manage MCP servers". It also updates the dialog's open effect to reset to the list view by default. Feedback was provided to reset the view state when the dialog is closed rather than when it is opened, which prevents a brief visual flash of the previous state.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread studio/frontend/src/features/chat/chat-mcp-servers-dialog.tsx

@Imagineer99 Imagineer99 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and working as expected.

Image Image

@Imagineer99 Imagineer99 merged commit ebf28e7 into unslothai:main Jun 8, 2026
27 of 29 checks passed
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.

How is one supposed to remove or alter the settings for an MCP server?

2 participants