feat(tui): devbase list トップをプロジェクト一覧化 + 実 TTY バグ 3 件修正#61
Merged
Conversation
利用頻度に合わせ `devbase list` の起動直後をプロジェクト一覧 (絞り込み付き) とし、env/plugin/snapshot/status は一覧末尾のカテゴリ項目から遷移する構成に 変更。actions_project は選択済み 1 行の処理 (handle_row) に責務を縮小。 あわせて実 TTY でのみ発現する不具合 3 件を修正 (単体テストが menu/questionary を monkeypatch しているためすり抜けていた): - app: トップメニューの choices が (key, label) 順で menu.select の (title, value) 契約と逆になっており、表示が内部キー・routing が全カテゴリ プレースホルダ落ちしていた (一覧トップ化で構造ごと解消) - menu: confirm/text/path の application は _MergedKeyBindings (.add 無し) の ため Esc 後付けが AttributeError でクラッシュ → 新しい KeyBindings を merge_key_bindings で再マージする _add_key_binding に変更 - menu: Esc/← で戻る際に未回答のまま collapse した質問行が残り、次メニューと 重なって 1 行ずれて見える → 戻り時は erase_when_done を立てて描画を消去 回帰テストは実 questionary オブジェクト (monkeypatch なし) で検証。 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
takemi-ohama
commented
Jun 11, 2026
takemi-ohama
left a comment
Contributor
Author
There was a problem hiding this comment.
🤖 cross-review | round 1 | gemini | APPROVE
修正すべき点はありません。
takemi-ohama
commented
Jun 11, 2026
takemi-ohama
left a comment
Contributor
Author
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | APPROVE
指摘なし。
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
何のために
PLAN31_2 で
devbase listを階層メニュー TUI 化したが、実端末での利用で以下が判明した:何を
トップ画面をプロジェクト一覧に変更
環境変数 (env)等、key でも絞り込み可)から遷移actions_projectは選択済み 1 行の処理(handle_row: running → 操作サブメニュー / 他は up)へ責務を縮小実 TTY バグ修正 3 件
TOP_CATEGORIESの(key, label)をmenu.selectの(title, value)契約と逆順で渡していた_select_topは正順で構築)confirm/text/pathの application は_MergedKeyBindings(.add無し)のため Esc 後付けがクラッシュKeyBindingsをmerge_key_bindingsで再マージする_add_key_bindingを新設erase_when_doneを立てて描画を消去Test plan
_MergedKeyBindingsへの Esc 後付け / 戻り時の erase_when_done