feat(web): add configurable composer focus shortcut - #10910
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (6)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe PR adds the ChangesComposer focus keybinding
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This adds an optional composer-focus shortcut without changing default bindings. The configured command is accepted, resolved across platforms, and focuses the composer as intended, with no current merge-blocking risk identified. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 5 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
What Changed
Add
composer.focusto Settings → Keybindings → Add keybinding on web and desktop. Users choose the shortcut; there is no default. The existing chat shortcut handler focuses the composer at the end of its draft, including when a terminal has focus.Why
There was no configurable shortcut to return to the composer. This reuses the existing command list, settings editor, and composer focus method without adding another event listener.
Validation
UI Changes
Checklist
Model: GPT-6. Harness: Codex in T3 Code.
Summary by CodeRabbit
New Features
composer.focusto the available keybinding commands. No shortcut is assigned by default; users can configure one in Settings → Keybindings.Documentation
Tests