Skip to content

fix(tui): fix broken /mcp toggling#16431

Merged
nexxeln merged 2 commits intoanomalyco:devfrom
natewill:fix/toggle-mcp-fix
Mar 8, 2026
Merged

fix(tui): fix broken /mcp toggling#16431
nexxeln merged 2 commits intoanomalyco:devfrom
natewill:fix/toggle-mcp-fix

Conversation

@natewill
Copy link
Contributor

@natewill natewill commented Mar 7, 2026

Issue for this PR

Closes #16358

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

The/mcp command uses the space bar to toggle between mcps.
Within certain terminals (ex. VS Code Integrated Terminal), toggling is completely broken because hitting the space bar types a space into the search bar instead of toggling. this PR fixes that.

How did you verify your code works?

Tested in iTerm2, Kitty, and VS Code Integrated Terminal. The space key now toggles MCPs instead of entering a space key into the search bar

I verified by running bun test test/keybind.test.ts from packages/opencode (all tests passed).

Screenshots / recordings

Before Fix:

before.mov

After Fix:

after.mov

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@natewill natewill changed the title fix(opencode): restore space toggle in mcp tui fix(tui): restore space toggle in mcp tui Mar 7, 2026
@natewill natewill changed the title fix(tui): restore space toggle in mcp tui fix(tui): fix toggling in /mcp within the TUI Mar 7, 2026
@natewill
Copy link
Contributor Author

natewill commented Mar 7, 2026

@nexxeln this is similar to another one you merged earlier #16443

@natewill natewill changed the title fix(tui): fix toggling in /mcp within the TUI fix(tui): fix broken /mcp toggling Mar 7, 2026
@nexxeln nexxeln merged commit fdad35a into anomalyco:dev Mar 8, 2026
6 of 8 checks passed
tallpizza pushed a commit to tallpizza/ontology that referenced this pull request Mar 9, 2026
Co-authored-by: Shoubhit Dash <shoubhit2005@gmail.com>
(cherry picked from commit fdad35a)
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.

Unable to Enable MCP in TUI

2 participants