feat(tui): add MCP OAuth authentication in dialog-mcp#11925
feat(tui): add MCP OAuth authentication in dialog-mcp#11925dmae97 wants to merge 1 commit intoanomalyco:devfrom
Conversation
- Add 'a' keybind to trigger OAuth authentication for MCP servers - Show needs_auth/needs_client_registration status with visual indicators - Display error messages for failed MCP connections - Add toast notifications for auth flow feedback fix(provider): add missing error checks in OAuth authorize - Add OauthMissing error when auth provider not found - Add OauthMethodNotFound error when method index invalid fix(mcp): improve OAuth flow reliability - Track serverUrl in auth storage to invalidate on URL change - Pass mcpName to waitForCallback for proper cancellation - Fix cancelPending to find auth by mcpName instead of oauthState - Remove unnecessary Bun.sleep in auth.ts
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
|
The following comment was made by an LLM, it may be inaccurate: Based on the search results, I found potentially related PRs to PR #11925: Related PRs Found:
Most relevant: PR #11477 may be worth checking if there's overlap in OAuth UI implementation approach. |
Summary
akeybind to trigger OAuth authentication for MCP servers in TUIneeds_auth/needs_client_registrationstatus with visual indicatorsBug Fixes
Changes
src/cli/cmd/auth.tssrc/provider/auth.tssrc/mcp/auth.tssrc/mcp/index.tssrc/mcp/oauth-callback.tssrc/mcp/oauth-provider.tssrc/cli/cmd/tui/component/dialog-mcp.tsx