Skip to content

feat(mcp): add HTTP transport UI fields to MCP server config#1

Merged
Shooksie merged 1 commit intomainfrom
feat/mcp-transport-ui
Apr 1, 2026
Merged

feat(mcp): add HTTP transport UI fields to MCP server config#1
Shooksie merged 1 commit intomainfrom
feat/mcp-transport-ui

Conversation

@Shooksie
Copy link
Copy Markdown
Contributor

@Shooksie Shooksie commented Apr 1, 2026

Summary

  • Adds McpTransport type and McpServerConfig interface (src/types/mcp.ts)
  • Adds McpServerForm component with transport type selector (stdio / HTTP), URL field, optional bearer token field, and live validation
  • Adds McpServerList component for managing the server list (add / edit / remove)
  • Integrates McpServerList into App.tsx below the existing panel grid

Why

Issue #96 shipped HTTP/remote MCP transport support in ao-cli, but ao-desktop had no UI to configure it. This follow-on surfaces the transport selector so users can connect to remote MCP servers (HTTP/SSE) in addition to local stdio processes.

Transport fields

Transport Fields
stdio Command (executable + args)
http URL (validated as valid URL), Bearer Token (optional)

Test plan

  • pnpm build (tsc + vite) passes — 0 type errors, 0 build errors
  • Manual: open app, click "Add server", toggle between stdio / HTTP, verify correct fields appear
  • Manual: submit with empty required fields — verify per-field error messages
  • Manual: submit valid stdio config — verify entry appears in list with "stdio" badge
  • Manual: submit valid HTTP config — verify entry appears in list with "HTTP" badge and URL shown
  • Manual: click Edit — verify form pre-populates; save updates the entry
  • Manual: click Remove — verify entry removed from list

🤖 Generated with Claude Code

Add transport type selector (stdio/http) and URL/token fields to MCP
server connection config (#96 follow-on). Ships McpServerConfig type,
McpServerForm with live validation, and McpServerList with add/edit/remove
flows. Build passes (tsc + vite).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Shooksie Shooksie merged commit 8d57bd5 into main Apr 1, 2026
1 check 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.

1 participant