Skip to content
This repository was archived by the owner on Feb 25, 2026. It is now read-only.

feat: add missing keybinds from ctrl+p menu and mcp toggles#127

Closed
pschiel wants to merge 2 commits intoKilo-Org:devfrom
pschiel:missing-keybinds
Closed

feat: add missing keybinds from ctrl+p menu and mcp toggles#127
pschiel wants to merge 2 commits intoKilo-Org:devfrom
pschiel:missing-keybinds

Conversation

@pschiel
Copy link
Contributor

@pschiel pschiel commented Feb 4, 2026

Fixes Kilo-Org/kilocode#6331
Fixes #172

adds keybinds for:

console_toggle - to toggle console
timestamps_toggle - to toggle timestamps display
thinking_toggle - to toggle thinking blocks
diffwrap_toggle - to toggle diff wrap
session_copy - copy session transcript
mcp_toggle - toggles mcp
mcp_toggle_1 .. mcp_toggle_10 - toggles mcp slot 1 .. 10

(Note: SDK generation unclear how this is handled in your CI, part of PR or not)

@pschiel
Copy link
Contributor Author

pschiel commented Feb 20, 2026

removed thinking_toggle as display_thinking toggle was added

@kilo-code-bot
Copy link
Contributor

kilo-code-bot bot commented Feb 20, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

This PR adds keybind support to several existing TUI commands and introduces new MCP slot toggle commands. The changes are clean and well-structured:

  • New keybind definitions in config.ts for console_toggle, mcp_toggle, mcp_toggle_1mcp_toggle_10, timestamps_toggle, diffwrap_toggle, and session_copy — all defaulting to "none"
  • MCP slot toggle commands (1–10) in app.tsx that map keybinds to alphabetically-sorted MCP servers, with proper error handling and user feedback via toasts
  • Keybind wiring for existing commands (Toggle MCPs, Toggle console, Toggle timestamps, Copy session transcript, Toggle diff wrapping)
  • SDK types regenerated to match the new config schema

The MCP slot toggle implementation correctly follows the existing pattern from dialog-mcp.tsx (toggle → refresh status → update store), and the alphabetical sort provides deterministic slot assignment.

Files Reviewed (4 files)
  • packages/opencode/src/cli/cmd/tui/app.tsx - MCP slot toggle commands, keybind wiring for mcp_toggle and console_toggle
  • packages/opencode/src/cli/cmd/tui/routes/session/index.tsx - keybind wiring for timestamps_toggle, diffwrap_toggle, session_copy; new diff wrap toggle command
  • packages/opencode/src/config/config.ts - 15 new keybind schema definitions
  • packages/sdk/js/src/v2/gen/types.gen.ts - Auto-generated SDK types (skipped detailed review)

@markijbema
Copy link
Contributor

Hi! Thank you for taking the time to contribute to this project—we really appreciate it. 🙏

We are currently working on re-platforming the core of our VS Code and JetBrains extensions to be based on our new Kilo CLI, with a complete rebuild based on OpenCode as our new foundation, and the moment has come to promote this repository to become the main repository. To do that, we moved the code from this repository to the kilocode repository.

This unfortunately means we cannot merge this branch here anymore. Please add https://github.com/Kilo-Org/kilocode.git as a remote, and push your branch there and create a new PR in https://github.com/Kilo-Org/kilocode . We unfortunately cannot do this for you as then the PR would not be in your name anymore. If you need any help, feel free to ask on our Discord in #kilo-dev-contributors

Sorry for the inconvenience and thank you for contributing to Kilo!

@markijbema markijbema closed this Feb 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: add missing toggle keybinds (from ctrl+p menu, and MCPs) [VS Code extension] [FEATURE]: Toggle Thinking

2 participants