Skip to content

Add session note history to the Capture input #49

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ScottMorris merged 3 commits into from
Apr 2, 2026
Merged

Add session note history to the Capture input #49

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
917e52a
fix: add session note history to the capture input
ScottMorris Apr 2, 2026
89327ce
docs: document capture history shortcuts
ScottMorris Apr 2, 2026
5ad53a5
fix: polish capture history and note rendering
ScottMorris Apr 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,7 @@ The TUI starts in **Insert mode**:
- **Up/Down** arrows navigate the thread list; the thread list auto-scrolls to keep selection visible; **Enter** on empty input expands/collapses branches
- The **Capture** pane grows as you add new lines, so multi-line drafts stay visible without taking extra space up front
- Press `Ctrl+J` in the **Capture** pane to insert a new line without submitting
- Press `Ctrl+Up` and `Ctrl+Down` in the **Capture** pane to browse plain-text note history from the current session and return to the in-flight draft at the bottom of the stack
- Mouse-wheel scrolling follows the hovered pane: `Threads`, `Status`, and `Help` each scroll independently
- Left-click in the thread list selects the clicked thread or branch
- The **Status** pane follows the selected thread or branch for inspection
Expand Down
3 changes: 2 additions & 1 deletion SPEC.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ The TUI provides a three-pane interface:

| Mode | Description |
|---|---|
| Insert | Text input active. Enter submits, `Ctrl+J` inserts a new line, and Esc switches to Normal. |
| Insert | Text input active. Enter submits, `Ctrl+J` inserts a new line, `Ctrl+Up` and `Ctrl+Down` browse session note history, and Esc switches to Normal. |
| Normal | Keyboard navigation. `j`/`k`/Up/Down to move through threads and branches, `Enter` to expand or collapse the selected thread, `r` to resume the selected item and make it active, `p` to park a selected branch, `d` to mark the selected item done, `Shift+A` to archive the selected item, `Ctrl+Z` to suspend `flo`, `i` to insert, `?` for help, `a` for about, `q` to quit. |
| Help | Help overlay. Esc or `?` to dismiss. |
| About | About overlay with app info. Esc, `q`, or Enter to dismiss. |
Expand All @@ -131,6 +131,7 @@ The thread list supports navigating both threads and their branches:
- The thread list auto-scrolls to keep the selected row visible when the list exceeds the available height
- **Enter** (on empty input in Insert, or in Normal mode) toggles expand/collapse for the selected thread's branches
- **Ctrl+J** inserts a new line in the Capture pane without submitting the current input
- **Ctrl+Up** and **Ctrl+Down** browse plain-text note history from the current TUI session and can restore the current in-flight draft
- Mouse-wheel scrolling follows the hovered pane, so `Threads`, `Status`, and `Help` scroll independently
- Left-click in the thread list selects the clicked thread or branch
- The **Status** pane follows the selected item for inspection
Expand Down
163 changes: 138 additions & 25 deletions crates/liminal-flow-tui/src/app.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ use crossterm::terminal::{
use ratatui::backend::CrosstermBackend;
use ratatui::Terminal;
use rusqlite::Connection;
use tui_textarea::TextArea;
use tui_textarea::{CursorMove, TextArea};

use crate::input::{self, InputResult};
use crate::poll;
Expand Down Expand Up @@ -117,6 +117,43 @@ fn is_insert_newline_key(key: crossterm::event::KeyEvent) -> bool {
key.modifiers.contains(KeyModifiers::CONTROL) && key.code == KeyCode::Char('j')
}

fn is_capture_history_previous_key(key: crossterm::event::KeyEvent) -> bool {
key.modifiers.contains(KeyModifiers::CONTROL) && key.code == KeyCode::Up
}

fn is_capture_history_next_key(key: crossterm::event::KeyEvent) -> bool {
key.modifiers.contains(KeyModifiers::CONTROL) && key.code == KeyCode::Down
}

fn is_note_history_entry(input: &str) -> bool {
let trimmed = input.trim();
!trimmed.is_empty() && !trimmed.starts_with('/')
}

fn textarea_value(textarea: &TextArea) -> String {
textarea.lines().join("\n")
}

fn reset_textarea(textarea: &mut TextArea) {
*textarea = TextArea::default();
textarea.set_cursor_line_style(ratatui::style::Style::default());
}

fn replace_textarea_value(textarea: &mut TextArea, value: &str) {
reset_textarea(textarea);
if !value.is_empty() {
textarea.insert_str(value);
}
}

fn textarea_cursor_is_on_first_line(textarea: &TextArea) -> bool {
textarea.cursor().0 == 0
}

fn textarea_cursor_is_on_last_line(textarea: &TextArea) -> bool {
textarea.cursor().0 + 1 >= textarea.lines().len()
}

fn selected_command_target(state: &TuiState) -> Option<input::CommandTarget> {
match &state.selected {
SelectedItem::Thread(i) => state
Expand Down Expand Up @@ -546,9 +583,7 @@ fn run_loop(
KeyCode::Esc => {
state.show_command_palette = false;
// Clear the `/` from the textarea
textarea = TextArea::default();
textarea
.set_cursor_line_style(ratatui::style::Style::default());
reset_textarea(&mut textarea);
}
KeyCode::Up => {
let query = textarea.lines().join("\n");
Expand Down Expand Up @@ -577,11 +612,7 @@ fn run_loop(
{
let completed =
complete_command_palette_selection(&query, cmd);
textarea = TextArea::default();
textarea.set_cursor_line_style(
ratatui::style::Style::default(),
);
textarea.insert_str(completed);
replace_textarea_value(&mut textarea, &completed);
state.show_command_palette = false;
}
}
Expand All @@ -607,34 +638,50 @@ fn run_loop(
match key.code {
KeyCode::Esc => {
state.show_hints = false;
textarea = TextArea::default();
textarea
.set_cursor_line_style(ratatui::style::Style::default());
reset_textarea(&mut textarea);
}
KeyCode::Backspace => {
state.show_hints = false;
textarea = TextArea::default();
textarea
.set_cursor_line_style(ratatui::style::Style::default());
reset_textarea(&mut textarea);
}
_ => {
state.show_hints = false;
// Clear the `?` and forward the new key
textarea = TextArea::default();
textarea
.set_cursor_line_style(ratatui::style::Style::default());
reset_textarea(&mut textarea);
textarea.input(Event::Key(key));
}
}
} else {
// Normal Insert mode handling
if is_insert_newline_key(key) {
if is_capture_history_previous_key(key) {
if textarea_cursor_is_on_first_line(&textarea) {
if let Some(previous) = state
.previous_capture_history(&textarea_value(&textarea))
{
replace_textarea_value(&mut textarea, &previous);
refresh_command_palette_state(&mut state, &previous);
}
} else {
textarea.move_cursor(CursorMove::Up);
}
} else if is_capture_history_next_key(key) {
if textarea_cursor_is_on_last_line(&textarea) {
if let Some(next) = state.next_capture_history() {
replace_textarea_value(&mut textarea, &next);
refresh_command_palette_state(&mut state, &next);
}
} else {
textarea.move_cursor(CursorMove::Down);
}
} else if is_insert_newline_key(key) {
state.clear_capture_history_navigation();
textarea.insert_newline();
let query = textarea.lines().join("\n");
let query = textarea_value(&textarea);
refresh_command_palette_state(&mut state, &query);
} else {
match key.code {
KeyCode::Esc => {
state.clear_capture_history_navigation();
state.mode = Mode::Normal;
state.show_command_palette = false;
state.show_hints = false;
Expand Down Expand Up @@ -667,12 +714,11 @@ fn run_loop(
// Submit the input
let lines: Vec<String> = textarea.lines().to_vec();
let text = lines.join("\n");
let should_record_note = is_note_history_entry(&text);

// Clear the textarea
textarea = TextArea::default();
textarea.set_cursor_line_style(
ratatui::style::Style::default(),
);
reset_textarea(&mut textarea);
state.clear_capture_history_navigation();

// Process the input
let follow_active =
Expand All @@ -683,7 +729,13 @@ fn run_loop(
&text,
command_target.as_ref(),
);
let record_note =
matches!(result, InputResult::Reply(_))
&& should_record_note;
apply_input_result(&mut state, result);
if record_note {
state.push_capture_history(&text);
}

// Refresh state from DB after mutation
state.refresh_from_db(conn);
Expand All @@ -694,17 +746,27 @@ fn run_loop(
state.poll_watermark = poll::current_watermark(conn);
}
KeyCode::Char('?') if is_empty => {
state.clear_capture_history_navigation();
// Show shortcut hints
state.show_hints = true;
textarea.input(Event::Key(key));
}
_ => {
if matches!(
key.code,
KeyCode::Backspace
| KeyCode::Delete
| KeyCode::Char(_)
| KeyCode::Tab
) {
state.clear_capture_history_navigation();
}
// Forward to textarea, then refresh palette
// state — text-modifying keys (Char, Backspace,
// Delete) and cursor-movement keys (Left, Right)
// can both affect whether the palette should open.
textarea.input(Event::Key(key));
let query = textarea.lines().join("\n");
let query = textarea_value(&textarea);
refresh_command_palette_state(&mut state, &query);
}
}
Expand Down Expand Up @@ -802,4 +864,55 @@ mod tests {
state: crossterm::event::KeyEventState::NONE,
}));
}

#[test]
fn ctrl_up_is_treated_as_capture_history_previous() {
assert!(is_capture_history_previous_key(
crossterm::event::KeyEvent {
code: KeyCode::Up,
modifiers: KeyModifiers::CONTROL,
kind: crossterm::event::KeyEventKind::Press,
state: crossterm::event::KeyEventState::NONE,
}
));
assert!(!is_capture_history_previous_key(
crossterm::event::KeyEvent {
code: KeyCode::Up,
modifiers: KeyModifiers::NONE,
kind: crossterm::event::KeyEventKind::Press,
state: crossterm::event::KeyEventState::NONE,
}
));
}

#[test]
fn ctrl_down_is_treated_as_capture_history_next() {
assert!(is_capture_history_next_key(crossterm::event::KeyEvent {
code: KeyCode::Down,
modifiers: KeyModifiers::CONTROL,
kind: crossterm::event::KeyEventKind::Press,
state: crossterm::event::KeyEventState::NONE,
}));
assert!(!is_capture_history_next_key(crossterm::event::KeyEvent {
code: KeyCode::Down,
modifiers: KeyModifiers::NONE,
kind: crossterm::event::KeyEventKind::Press,
state: crossterm::event::KeyEventState::NONE,
}));
}

#[test]
fn textarea_first_and_last_line_detection_tracks_cursor_position() {
let mut textarea = TextArea::from(["first", "second", "third"]);
assert!(textarea_cursor_is_on_first_line(&textarea));
assert!(!textarea_cursor_is_on_last_line(&textarea));

textarea.move_cursor(CursorMove::Down);
assert!(!textarea_cursor_is_on_first_line(&textarea));
assert!(!textarea_cursor_is_on_last_line(&textarea));

textarea.move_cursor(CursorMove::Down);
assert!(!textarea_cursor_is_on_first_line(&textarea));
assert!(textarea_cursor_is_on_last_line(&textarea));
}
}
Loading
Loading