Skip to content

feat(list): list TUI サブメニューの「戻る」に ← を追加し即応化#46

Merged
takemi-ohama merged 1 commit into
mainfrom
feat/list-tui-left-arrow-back
Jun 9, 2026
Merged

feat(list): list TUI サブメニューの「戻る」に ← を追加し即応化#46
takemi-ohama merged 1 commit into
mainfrom
feat/list-tui-left-arrow-back

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

概要

devbase list の running 操作サブメニューで、Esc による「戻る」の反応が体感的に重い問題を解消する。左矢印 () を「戻る」キーとして追加し即時反応させる。

Esc (\x1b) は矢印キーのエスケープシーケンス (\x1b[A 等) の先頭バイトと同一のため、prompt_toolkit は単独 Esc か矢印キーの途中かを判別する目的でフラッシュ待ち (約0.5秒) を挟む。これが「重さ」の正体。左矢印 (\x1b[D) は完結した曖昧さの無いシーケンスなので、タイムアウト待ちゼロで即応する。

サブメニューは検索絞り込み (use_search_filter) を使わないため、 をカーソル移動と衝突させずに割り当てられる。Esc は互換のため残す。

関連 Issue

  • Closes #

変更点

  • _with_escape_back に左矢印 (Keys.Left) のバインドを追加し、 を主たる「戻る」キーとして即応化(Esc は互換のため据え置き)
  • サブメニューのプロンプト文言を Esc 戻る← ・Esc 戻る に更新
  • テストに Keys.Left バインド存在・_MENU_BACK 返却のアサーションを追加

やらないこと

  • トップメニュー側の「中止」は use_search_filter=True が検索カーソル移動に使われうるため変更しない(Esc / Ctrl-C のまま据え置き。Ctrl-C は元々即時)

動作確認

  • tests/cli/test_project_list.py 51 件すべて green (uv run pytest)
  • 実機 (Mac ホスト) で devbase list → running プロジェクト選択 → の即応を確認
  • CI が green である

Esc は矢印キーのエスケープシーケンス先頭バイトと衝突するため、
prompt_toolkit のフラッシュ待ち (約0.5秒) 分の遅延が体感される。
左矢印 (\x1b[D) は完結した曖昧さの無いシーケンスなので、これを
running 操作サブメニューの主たる「戻る」キーとして即時反応させる。
Esc は互換のため残す。プロンプト文言も「← ・Esc 戻る」に更新。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | codex | APPROVE

修正必須の指摘はありません。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 cross-review | round 1 | gemini | APPROVE

PR の変更内容を確認しました。修正すべき点はありません。

@takemi-ohama takemi-ohama merged commit 54b71a5 into main Jun 9, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant