feat: implement MCP server search functionality and registry integration#2197
feat: implement MCP server search functionality and registry integration#2197vvnsrzn wants to merge 2 commits intohuggingface:mainfrom
Conversation
There was a problem hiding this comment.
💡 Codex Review
chat-ui/src/lib/components/mcp/AddServerForm.svelte
Lines 104 to 105 in e121362
Required registry headers can still be removed by clearing both inputs: validate() ignores rows where key/value are both empty, and this filter then drops those rows entirely on submit. In practice, a user can bypass the “non-removable” UI for mandatory auth headers and save a broken server config that fails health checks/connection.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| {#if !header.description} | ||
| <button |
There was a problem hiding this comment.
Represent required headers explicitly, not via description
The form treats description as the required/locked signal (!header.description controls whether a header is removable), but registry header descriptions are optional. That means a header marked required by the registry but lacking a description is shown as removable/optional, so the UI can silently violate required-auth metadata.
Useful? React with 👍 / 👎.
…dd required field
MCP Search
Adds an MCP Registry search tab to the "Add Server" modal, allowing users to discover and add servers from the official registry in one click.
When selecting a server, the name, URL, and required authentication headers are automatically pre-filled with description hints — and mandatory headers cannot be removed.
Existing custom servers can now be edited directly from the server list
A new /api/mcp/registry endpoint proxies the MCP registry with a 5-minute cache.