From 25b016f3dafdd712fec11e5cf126abd94a4a5d98 Mon Sep 17 00:00:00 2001 From: Hare Date: Fri, 1 May 2026 23:14:16 +0900 Subject: [PATCH 1/2] =?UTF-8?q?feat(tui):=20=E3=83=9E=E3=82=A6=E3=82=B9?= =?UTF-8?q?=E3=83=9B=E3=82=A4=E3=83=BC=E3=83=AB=E3=81=A7=E3=82=B9=E3=82=AF?= =?UTF-8?q?=E3=83=AD=E3=83=BC=E3=83=AB=E3=81=99=E3=82=8B=E5=AE=9F=E8=A3=85?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- TODO.md | 1 + crates/tui/src/main.rs | 36 +++++++++++++++++++++++++++++++----- tickets/tui-mouse-scroll.md | 28 ++++++++++++++++++++++++++++ 3 files changed, 60 insertions(+), 5 deletions(-) create mode 100644 tickets/tui-mouse-scroll.md diff --git a/TODO.md b/TODO.md index fc9e5f48..33d923f1 100644 --- a/TODO.md +++ b/TODO.md @@ -14,6 +14,7 @@ - [ ] フルスクリーン化によるオーバーホール → [tickets/tui-fullscreen-overhaul.md](tickets/tui-fullscreen-overhaul.md) - [ ] Run 中の入力キューイング → [tickets/tui-input-queue.md](tickets/tui-input-queue.md) - [ ] ユーザーマニフェストのモデル設定 wizard → [tickets/tui-user-model-setup.md](tickets/tui-user-model-setup.md) + - [ ] マウスホイールでのヒストリースクロール → [tickets/tui-mouse-scroll.md](tickets/tui-mouse-scroll.md) - [ ] サブミット入力 - [ ] TUI 補完 + 型付き atom 化 → [tickets/submit-tui-completion.md](tickets/submit-tui-completion.md) - [ ] FileRef リゾルバ → [tickets/submit-file-ref-resolver.md](tickets/submit-file-ref-resolver.md) diff --git a/crates/tui/src/main.rs b/crates/tui/src/main.rs index 7b33f7cb..411f6cfd 100644 --- a/crates/tui/src/main.rs +++ b/crates/tui/src/main.rs @@ -15,8 +15,8 @@ use std::process::ExitCode; use std::time::Duration; use crossterm::event::{ - self, DisableBracketedPaste, EnableBracketedPaste, Event as TermEvent, KeyCode, KeyEvent, - KeyModifiers, + self, DisableBracketedPaste, DisableMouseCapture, EnableBracketedPaste, EnableMouseCapture, + Event as TermEvent, KeyCode, KeyEvent, KeyModifiers, MouseEvent, MouseEventKind, }; use crossterm::execute; use crossterm::terminal::{ @@ -170,7 +170,12 @@ async fn main() -> ExitCode { // shows up cleanly in scrollback rather than inside an active // alternate-screen buffer. let mut stdout = io::stdout(); - let _ = execute!(stdout, LeaveAlternateScreen, DisableBracketedPaste); + let _ = execute!( + stdout, + DisableMouseCapture, + LeaveAlternateScreen, + DisableBracketedPaste + ); let _ = disable_raw_mode(); let _ = execute!(stdout, crossterm::cursor::Show); @@ -229,7 +234,11 @@ async fn run_spawn(resume_from: Option) -> Result<(), Box {} @@ -246,7 +255,7 @@ async fn run_spawn(resume_from: Option) -> Result<(), Box Result>, Box> { let mut stdout = io::stdout(); - execute!(stdout, EnterAlternateScreen)?; + execute!(stdout, EnterAlternateScreen, EnableMouseCapture)?; let backend = CrosstermBackend::new(stdout); Ok(Terminal::new(backend)?) } @@ -302,6 +311,9 @@ async fn run_loop( client.send(&method).await?; } } + TermEvent::Mouse(mouse) => { + handle_mouse(app, mouse); + } TermEvent::Paste(s) => { app.insert_paste(s); } @@ -345,6 +357,20 @@ fn run_disconnected(_app: &mut App) -> Result<(), Box> { Ok(()) } +/// Lines per wheel notch. Faster than Shift+↑/↓ (which is 1 line) so +/// hand-rolling through long histories isn't tedious, but slow enough +/// that a single notch doesn't blow past the section the user is +/// looking for. +const WHEEL_LINES: usize = 3; + +fn handle_mouse(app: &mut App, mouse: MouseEvent) { + match mouse.kind { + MouseEventKind::ScrollUp => app.scroll.scroll_up(WHEEL_LINES), + MouseEventKind::ScrollDown => app.scroll.scroll_down(WHEEL_LINES), + _ => {} + } +} + fn handle_key(app: &mut App, key: KeyEvent) -> Option { let ctrl = key.modifiers.contains(KeyModifiers::CONTROL); let shift = key.modifiers.contains(KeyModifiers::SHIFT); diff --git a/tickets/tui-mouse-scroll.md b/tickets/tui-mouse-scroll.md new file mode 100644 index 00000000..b84a8be8 --- /dev/null +++ b/tickets/tui-mouse-scroll.md @@ -0,0 +1,28 @@ +# TUI: マウスホイールでヒストリーをスクロール + +## 背景 + +TUIのヒストリービューはキー操作（Shift+↑/↓、PageUp/PageDown、Ctrl+Home/End、Ctrl+[/]）でスクロールできる。スクロール状態（`Scroll` 構造体・follow_tail・top_offset）も既に揃っており、ロジック的には外部から `scroll_up`/`scroll_down` を叩けば任意の手段でスクロール可能になっている。 + +一方で、現状のイベントループは `TermEvent::Key` / `Paste` / `Resize` のみを処理しており、マウスイベントは破棄される。crossterm のマウスキャプチャも有効化されていないため、ターミナル側にホイール入力すら届かない。 + +エディタ的に常駐するTUIとして、マウスホイールで直感的にヒストリーを遡れる体験を提供したい。 + +## 要件 + +- フルスクリーンモード中にマウスホイールでヒストリービューをスクロールできる。 +- ホイール上方向で過去方向（`scroll_up`）、下方向で末尾方向（`scroll_down`）に動く。1 notch あたりの行数は既存のキー操作（Shift+↑/↓）と同じ感覚で良い。 +- マウスキャプチャの有効化はalt-screenに入っているあいだに限定し、終了時に確実に解除する（picker / spawn のインラインフェーズはネイティブのマウス挙動を保つ）。 +- クリック・ドラッグ等のホイール以外のマウスイベントは現状無視で構わないが、最低限イベントループでマッチ漏れによるエラーが出ない形にする。 + +## 完了条件 + +- フルスクリーンTUIでマウスホイールを回すと、ヒストリービューが上下に動く。 +- TUIを正常終了・異常終了どちらでも、ターミナルのマウスキャプチャ状態が残らない。 +- 既存のキー操作によるスクロール挙動に変化がない。 + +## 範囲外 + +- ホイール以外のマウス操作（クリックでフォーカス移動、ドラッグ選択、スクロールバー表示など） +- picker / spawn のインラインフェーズでのマウス対応 +- ホイール感度の設定可能化 From 2159711cd0f89bacc4c36c99c1437ebcd1194e7a Mon Sep 17 00:00:00 2001 From: Hare Date: Fri, 1 May 2026 23:16:02 +0900 Subject: [PATCH 2/2] =?UTF-8?q?feat(tui):=20=E3=83=9E=E3=82=A6=E3=82=B9?= =?UTF-8?q?=E3=83=9B=E3=82=A4=E3=83=BC=E3=83=AB=E3=82=B9=E3=82=AF=E3=83=AD?= =?UTF-8?q?=E3=83=BC=E3=83=AB=E5=AE=8C=E4=BA=86?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- tickets/tui-mouse-scroll.md | 5 ++++ tickets/tui-mouse-scroll.review.md | 40 ++++++++++++++++++++++++++++++ 2 files changed, 45 insertions(+) create mode 100644 tickets/tui-mouse-scroll.review.md diff --git a/tickets/tui-mouse-scroll.md b/tickets/tui-mouse-scroll.md index b84a8be8..486d83e9 100644 --- a/tickets/tui-mouse-scroll.md +++ b/tickets/tui-mouse-scroll.md @@ -26,3 +26,8 @@ TUIのヒストリービューはキー操作（Shift+↑/↓、PageUp/PageDown - ホイール以外のマウス操作（クリックでフォーカス移動、ドラッグ選択、スクロールバー表示など） - picker / spawn のインラインフェーズでのマウス対応 - ホイール感度の設定可能化 + +## Review +- 状態: Approve +- レビュー詳細: [./tui-mouse-scroll.review.md](./tui-mouse-scroll.review.md) +- 日付: 2026-05-01 diff --git a/tickets/tui-mouse-scroll.review.md b/tickets/tui-mouse-scroll.review.md new file mode 100644 index 00000000..3e28efba --- /dev/null +++ b/tickets/tui-mouse-scroll.review.md @@ -0,0 +1,40 @@ +# Review: TUI マウスホイールでヒストリーをスクロール + +## 前提・要件の確認 + +- フルスクリーンモード中にマウスホイールでスクロール可能 + - `enter_fullscreen` で `EnableMouseCapture` を `EnterAlternateScreen` と一緒に発行（`crates/tui/src/main.rs:258`）。`run_loop` の `event::read` 分岐に `TermEvent::Mouse(mouse) => handle_mouse(app, mouse)` を追加（`crates/tui/src/main.rs:314-316`）。要件達成。 +- ホイール上 = `scroll_up` / 下 = `scroll_down`、感覚は Shift+↑/↓ と同等 + - `handle_mouse` が `MouseEventKind::ScrollUp/Down` をそのまま `app.scroll.scroll_up/down(WHEEL_LINES)` にマップ（`crates/tui/src/main.rs:366-372`）。`Scroll::scroll_up` は `top_offset` を減らす実装（`scroll.rs:47-50`）で「過去方向」と整合。`WHEEL_LINES = 3` は Shift+↑/↓ の 1 行より速いがページ未満で、ticket の「同じ感覚で良い」と矛盾しない。妥当。 +- マウスキャプチャは alt-screen 中に限定し、終了時に確実に解除 + - 有効化は `enter_fullscreen` 1 箇所のみ（`main.rs:258`）。解除は `run_spawn` の child reap 前（`main.rs:237-241`）と `main` 末尾の最終 cleanup（`main.rs:173-178`）の 2 箇所で `DisableMouseCapture` を `LeaveAlternateScreen` と対で発行。picker / spawn のインラインフェーズでは `enter_fullscreen` を経由しないので有効化されない。要件達成。 +- ホイール以外のマウスイベントでマッチ漏れエラーが出ないこと + - `handle_mouse` の `match` は `_ => {}` で吸収（`main.rs:370`）。`run_loop` 側は `TermEvent::Mouse(_)` を新たに拾うので未処理エラーは発生しない。要件達成。 + +## 完了条件の検証 + +- フルスクリーンTUIでホイールが効く: コードパスは成立。実機確認はユーザー側で実施予定（ticket補足通り）。 +- 正常終了でターミナル状態が残らない: `main` 末尾の cleanup は `result` の match より前に走るので、`Ok` でも `Err` でも必ず `DisableMouseCapture` を発行する。`run_spawn` 経由で `run` がエラー return しても、戻った直後の `execute!` で disable した後に `?` 相当の伝播が起きるため、二重 disable になっても idempotent で安全。 +- 既存キー操作のスクロール挙動に変化なし: `Scroll` API・`handle_key` のスクロール経路は無変更。 + +## アーキテクチャ・スコープ + +- 変更は `crates/tui/src/main.rs` の terminal lifecycle と event loop に限定。新規モジュール・新規 trait・新規抽象は導入していない。既存の `Scroll` API をそのまま再利用しており、過剰な抽象化はない。 +- `WHEEL_LINES` を `main.rs` 内の `const` として置いた選択は、現状のキー側のマジックナンバー（page_up/down 等は `area_height` から導出、Shift+↑/↓ は `1` 直書き）と粒度が揃っている。`scroll.rs` 側に押し込む価値は今のところない。 +- マウス捕捉の対称性（alt-screen と同じスコープでEnable/Disable）は ticket の「範囲外」記述と一致しており、picker / spawn のインラインフェーズでホイール挙動が変わらないことが保証されている。 +- 依存追加なし。crossterm の既存featureで `MouseEvent` 系が利用可能（既に import 済みの `event` モジュールから引いている）。 + +## 指摘事項 + +### Non-blocking / Follow-up + +- パニック時のクリーンアップは未対応。tui には `std::panic::set_hook` 等のフックが存在せず、これは本変更前から同じ（bracketed paste も alt-screen も同じく panic 時には残る）。本ticket の「異常終了でもキャプチャ状態が残らない」を厳密に取ると未達だが、既存の他リソース（alt-screen、bracketed paste、raw mode）と同レベルで揃っており、本ticketで先んじて対応するのは過剰。別ticket（パニック時のターミナル復帰一括対応）として扱うのが適切。 +- `enter_fullscreen` 内で `EnterAlternateScreen` の後に `EnableMouseCapture` が失敗した場合、その関数は `?` で早期 return するが、`main` 末尾の cleanup が `DisableMouseCapture, LeaveAlternateScreen` を流すので結果的に状態は復帰する。明示性を上げるなら `enter_fullscreen` 内でロールバックを書く手もあるが、final cleanup と二重になるだけで価値は薄い。現状維持で良い。 + +### Nits + +- `WHEEL_LINES` のドキュメントコメントは根拠が読みやすく良い。指摘なし。 + +## 判断 + +Approve — ticket の前提・要件・完了条件はすべて実装で達成されており、コードベースを歪めず最小差分で収まっている。実機確認はユーザー側で予定通り実施すれば良い。