Repository navigation
fix(tui): keep MCP dialog open after OAuth - #51079
BowTiedSwan wants to merge 1 commit into
Conversation
|
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: |
|
The description already includes |
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |
Issue for this PR
Closes #51060
Type of change
What does this PR do?
Return to the MCP list after successful authentication, with the same server selected. The sign-in dialog previously had no completion callback, so it used the default behavior of closing the dialog.
Uses the existing integration callback and MCP status updates so users can authenticate the next server without reopening
/mcp.How did you verify your code works?
packages/tui:bun test --timeout 30000 test/cli/tui/dialog-mcp.test.tsx test/cli/tui/dialog-integration.test.tsx test/cli/tui/dialog-select.test.tsx— 48 passed.bun run check— repository-wide lint and all 35 typecheck tasks passed with Bun 1.4.2.git diff --checkpassed.To confirm manually, authorize an MCP server from
/mcp, then select another server from the restored list.Screenshots / recordings
Real TUI component frames with fixture-controlled OAuth completion; no external account authorization was used.
Checklist