Skip to content

Comments

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

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

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
Contributor

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 #126

Copilot AI review requested due to automatic review settings February 22, 2026 18:33
@github-actions
Copy link
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.

@kiloconnect
Copy link
Contributor

kiloconnect 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
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
Contributor 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant