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

feat(tui): add missing toggle and transcript keybinds#603

Closed
Olusammytee wants to merge 2 commits intoKilo-Org:devfrom
Olusammytee:feat/126-missing-toggle-keybinds
Closed

feat(tui): add missing toggle and transcript keybinds#603
Olusammytee wants to merge 2 commits intoKilo-Org:devfrom
Olusammytee:feat/126-missing-toggle-keybinds

Conversation

@Olusammytee
Copy link
Copy Markdown

Summary

  • add missing keybind config entries for MCP toggle, console toggle, diff-wrap toggle, timestamps toggle, and session transcript copy
  • wire those keybinds to existing TUI commands in the app and session command palette
  • keep defaults set to
    one to avoid introducing new shortcut collisions

Fixes Kilo-Org/kilocode#6331

@github-actions
Copy link
Copy Markdown
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented Feb 22, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

This PR adds missing keybind config entries for 5 TUI command palette items (MCP toggle, console toggle, diff-wrap toggle, timestamps toggle, session copy) and wires them to the corresponding commands. All new keybinds default to "none" to avoid shortcut collisions.

The changes follow the established pattern consistently — every keybind name matches its config schema key, and the auto-generated SDK types are properly updated.

Files Reviewed (5 files)
  • packages/opencode/src/config/config.ts - 5 new keybind schema entries
  • packages/opencode/src/cli/cmd/tui/app.tsx - 3 keybind properties added
  • packages/opencode/src/cli/cmd/tui/routes/session/index.tsx - 2 keybind properties added
  • packages/sdk/js/src/gen/types.gen.ts - auto-generated type updates
  • packages/sdk/js/src/v2/gen/types.gen.ts - auto-generated type updates

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds missing keybind configuration entries for several toggle actions and wires them to existing TUI commands. The changes address feature request #126 by enabling users to configure keyboard shortcuts for frequently used toggle functions, improving the CLI workflow efficiency.

Changes:

  • Added 5 new keybind configuration entries (mcp_toggle, console_toggle, diffwrap_toggle, timestamps_toggle, session_copy) to the config schema with default value of "none" to avoid shortcut collisions
  • Wired these keybinds to their corresponding command palette entries in the app-level and session-level command registrations

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
packages/opencode/src/config/config.ts Added 5 new keybind config entries (mcp_toggle, console_toggle, diffwrap_toggle, timestamps_toggle, session_copy) to the Keybinds schema with "none" defaults
packages/opencode/src/cli/cmd/tui/app.tsx Wired keybinds for mcp_toggle, console_toggle, and diffwrap_toggle to their respective command palette entries
packages/opencode/src/cli/cmd/tui/routes/session/index.tsx Wired keybinds for timestamps_toggle and session_copy to their respective command palette entries

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Olusammytee
Copy link
Copy Markdown
Author

Pushed follow-up commit a7d0f9c to align SDK keybind typings with the new keybind entries (mcp_toggle, console_toggle, diffwrap_toggle, timestamps_toggle, session_copy). This should resolve the typecheck failure.

@markijbema markijbema closed this Feb 25, 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.

3 participants