Let Ctrl+Enter insert a newline in the TUI input box

Author: alexylon

CHANGELOG.md

@@ -6,6 +6,7 @@ All notable changes to Sofos are documented in this file.
 
 ### Fixed
 
+- **Ctrl+Enter now inserts a newline in the TUI input box, matching Shift+Enter and Alt+Enter.** Earlier the keystroke was silently swallowed by the textarea router, even though the placeholder text and the dispatch comments already documented it as a fallback newline binding for terminals that do not deliver Shift+Enter distinctly.
 - **Setting `compaction_preserve_recent` to `0` no longer crashes the next compaction.** The split-point lookup used to index one past the end of the message list and panic. It now clamps to the last valid index, so a zero-preserve configuration just compacts everything older than the very last message.
 - **The summary call's token usage is now counted toward the session total.** When auto-compaction got a usable response from the model but the summary was too short to apply (fell back to plain trimming), sofos used to discard the response without billing it; the spend now lands on the session counters in every Ok path.
 - **The auto-compaction summary call no longer fights the prompt cache.** The one-shot summary used to share the OpenAI prompt-cache shard with the regular session, so the two distinct prefixes evicted each other on every compaction. The summary call now uses a `"<session>-summary"` shard of its own.

src/repl/tui/app.rs

@@ -226,8 +226,11 @@ impl App {
     /// Route a key event into the textarea (idle) or picker (overlay).
     pub fn handle_textarea_input(&mut self, key: KeyEvent) {
         let input = key_to_input(key);
-        // Ignore Enter here — the caller handles submission.
-        if matches!(input.key, Key::Enter) && !input.shift && !input.alt {
+        // Ignore plain Enter here — the caller handles submission.
+        // Modified Enter (Shift/Alt/Ctrl) falls through so the
+        // textarea inserts a newline, matching the fallback bindings
+        // documented at the input dispatch site.
+        if matches!(input.key, Key::Enter) && !input.shift && !input.alt && !input.ctrl {
             return;
         }
         self.textarea.input(input);
@@ -353,6 +356,31 @@ mod tests {
         assert_eq!(a.textarea.lines(), &["hello", ""]);
     }
 
+    #[test]
+    fn ctrl_enter_inserts_newline_through_textarea_handler() {
+        // Ctrl+Enter is documented at the input dispatch site as a
+        // fallback newline binding for terminals that don't deliver
+        // Shift+Enter distinctly. The early-return used to drop the
+        // event because only Shift and Alt were checked.
+        let mut a = app();
+        a.textarea.insert_str("hello");
+        let ctrl_enter = KeyEvent::new(KeyCode::Enter, KeyModifiers::CONTROL);
+        a.handle_textarea_input(ctrl_enter);
+        assert_eq!(a.textarea.lines(), &["hello", ""]);
+    }
+
+    #[test]
+    fn alt_enter_inserts_newline_through_textarea_handler() {
+        // Mirror of the Shift+Enter test; pinned so a future tweak
+        // to the early-return guard does not silently regress the
+        // documented Alt+Enter fallback.
+        let mut a = app();
+        a.textarea.insert_str("hello");
+        let alt_enter = KeyEvent::new(KeyCode::Enter, KeyModifiers::ALT);
+        a.handle_textarea_input(alt_enter);
+        assert_eq!(a.textarea.lines(), &["hello", ""]);
+    }
+
     #[test]
     fn is_safe_mode_reads_from_status_snapshot() {
         use crate::repl::tui::event::{Mode, StatusSnapshot};