Skip to content

Commit 9e8c9b5

Browse files
committed
refactor(keyboard): remove unused Ctrl+M sendMessage binding
Removed the sendMessage action and Ctrl+M keybinding since it was not implemented and did nothing. Ctrl+M remains in NON_REBINDABLE to prevent user confusion (it maps to Enter in ASCII).
1 parent 8da4a29 commit 9e8c9b5

2 files changed

Lines changed: 1 addition & 7 deletions

File tree

src-rust/crates/core/src/keybindings.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,6 @@ pub const NON_REBINDABLE: &[&str] = &["ctrl+c", "ctrl+d", "ctrl+m"];
151151
/// - **Alt+←/Alt+→**: Navigate to previous/next message in transcript
152152
/// - **Ctrl+. (Ctrl+>)**: Jump to next error/issue in messages
153153
/// - **Ctrl+Shift+.**: Jump to previous error/issue
154-
/// - **Ctrl+M**: Send message (alternative to Enter)
155154
/// - **Shift+Tab**: Reverse indent/unindent in input (cycle permission mode)
156155
/// - **Ctrl+H**: Delete character before cursor (Chat context, Emacs-style)
157156
/// - **Alt+H**: Open help (alternative to F1)
@@ -172,7 +171,6 @@ pub fn default_bindings() -> Vec<ParsedBinding> {
172171
// ========== CHAT / INPUT CONTEXT ==========
173172
// Message submission
174173
("enter", "submit", KeyContext::Chat),
175-
("ctrl+m", "sendMessage", KeyContext::Chat),
176174

177175
// Newline insertion (Shift+Enter / Ctrl+J for multi-line composing)
178176
("shift+enter", "newline", KeyContext::Chat),
@@ -849,7 +847,7 @@ mod tests {
849847

850848
// Check Phase 1 keybinding actions exist
851849
assert!(actions.contains(&"clearLine".to_string()), "clearLine action not found");
852-
assert!(actions.contains(&"sendMessage".to_string()), "sendMessage action not found");
850+
assert!(actions.contains(&"submit".to_string()), "submit action not found");
853851
assert!(actions.contains(&"jumpToNextError".to_string()), "jumpToNextError action not found");
854852
assert!(actions.contains(&"jumpToPreviousError".to_string()), "jumpToPreviousError action not found");
855853
assert!(actions.contains(&"previousMessage".to_string()), "previousMessage action not found");

src-rust/crates/tui/src/app.rs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4898,10 +4898,6 @@ impl App {
48984898
}
48994899
false
49004900
}
4901-
"sendMessage" => {
4902-
// Ctrl+M: Send message (alternative to Enter)
4903-
!self.is_streaming
4904-
}
49054901
"newline" => {
49064902
// Shift+Enter: insert a literal newline into the prompt.
49074903
if !self.is_streaming {

0 commit comments

Comments
 (0)