Skip to content

feat: PLAN31_2-env-ops env 全操作の TUI 追加#58

Merged
takemi-ohama merged 4 commits into
release/PLAN31_2from
feature/PLAN31_2-env-ops
Jun 11, 2026
Merged

feat: PLAN31_2-env-ops env 全操作の TUI 追加#58
takemi-ohama merged 4 commits into
release/PLAN31_2from
feature/PLAN31_2-env-ops

Conversation

@takemi-ohama

@takemi-ohama takemi-ohama commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

PLAN31_2 PR3 (env-ops)。実装プラン: issues/PLAN31_2_list-tui-unified.md / release PR: #55

devbase list の TUI に env カテゴリ (init/list/set/get/delete/edit/sync/project/export/import の全 10 サブコマンド) を追加する。

  • lib/devbase/tui/actions_env.py 新設: 操作選択メニュー → 引数収集 (tui.menu ヘルパ) → dispatch_group(cmd_env, ...) で既存ハンドラへ委譲 (plan 2.3 の属性契約。ロジック二重実装なし)
  • tui/app.py の _route に env を 1 行配線 (PR2〜5 申し送りの方式)
  • project スコープ依存 (set --project / project) は事前にプロジェクト選択メニュー (actions_project と同じ一覧取得を流用) → chdir + PWD 切替で実行し、try/finally で必ず元の CWD/PWD へ復帰 (plan 3.3)。cmd_env_* は os.environ['PWD'] で現在地判定するため os.chdir に加えて PWD も切り替える
  • 破壊的操作 env delete は実行前に menu.confirm()。拒否/中止時は実行せずメニューへ戻る (plan 3.4)
  • export/import は主要引数 (dest / source) のみ TUI で収集し、残りは CLI parser 既定値と同一の属性を明示付与 (parser との同期テスト 2 件で既定値乖離を検知)
  • ナビ規約は actions_project と同一 (Esc/← → 戻る = MENU_BACK、Ctrl-C → None 伝搬、引数収集中止 → サブメニュー再表示)

plan 契約表との乖離 (実装を正とした)

  • plan 2.3/3.3 は env edit を CWD スコープとするが、実装 cmd_env_edit は常に $DEVBASE_ROOT/.env を開くグローバル操作のため、TUI ではプロジェクト選択・chdir を行わない (actions_env.py の docstring に明記)

Test plan

  • tests/cli/tui/test_actions_env.py 新設 (48 テスト): menu.* monkeypatch + cmd_env mock で、各サブコマンドが plan 2.3 契約どおりの属性を持つ Namespace で呼ばれることを検証
  • project スコープの chdir → 実行 → CWD/PWD 復帰 (例外時の finally 復帰・PWD 未設定環境含む)
  • env delete の confirm 拒否 (False) / 中止 (None) で未実行
  • Esc/← (MENU_BACK) / Ctrl-C (None) / 引数収集中止 (サブメニュー再表示) の遷移
  • export/import の既定値が cli._create_parser() の parse 結果と一致する同期テスト
  • uv run pytest 全体: 592 passed / 1 skipped (ベースライン 544 から +49、退行ゼロ)

🤖 Generated with Claude Code

@takemi-ohama takemi-ohama mentioned this pull request Jun 10, 2026
Merged
3 tasks
@takemi-ohama takemi-ohama marked this pull request as ready for review June 10, 2026 04:10
takemi-ohama and others added 2 commits June 11, 2026 03:01
@takemi-ohama
chore: PLAN31_2-env-ops Draft PR 作成
e1a7203
@takemi-ohama @claude
feat: PLAN31_2-env-ops env 全操作の TUI 追加
ebdd906 
- tui/actions_env.py 新設: env init/list/set/get/delete/edit/sync/project/
  export/import を選択メニュー + 引数収集で cmd_env へ委譲 (plan 2.3 契約)
- project スコープ (set --project / project) は事前にプロジェクト選択 →
  chdir + PWD 切替で実行し、try/finally で必ず元の CWD/PWD へ復帰 (plan 3.3)
- 破壊的な env delete は実行前に menu.confirm で確認 (plan 3.4)
- export/import は主要引数 (dest/source) のみ収集し、残りは CLI parser 既定値と
  同一の属性を明示付与 (parser との同期テスト付き)
- tui/app.py の _route に env を配線、未実装カテゴリ前提のテストを plugin へ更新
- tests/cli/tui/test_actions_env.py 新設 (契約・chdir 復帰・confirm・ナビ遷移)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@takemi-ohama takemi-ohama force-pushed the feature/PLAN31_2-env-ops branch from fe14808 to ebdd906 Compare June 11, 2026 03:10
takemi-ohama
takemi-ohama commented Jun 11, 2026
View reviewed changes

@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 | REQUEST_CHANGES

TUI の project スコープ操作で、選択したプロジェクトの .env に到達できない経路があります。

Comment thread lib/devbase/tui/actions_env.py
takemi-ohama
takemi-ohama commented Jun 11, 2026
View reviewed changes

@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

PLAN31_2 に定義された仕様（project スコープでの chdir と復帰、破壊的操作時の確認プロンプト、引数収集と CLI parser との既定値同期）がテスト含め過不足なく正確に実装されていることを確認しました。指摘事項はありません。

@takemi-ohama @claude
fix: env list の「プロジェクトのみ」で対象プロジェクト選択 + chdir/PWD 切替を行う
03bbf68 
cmd_env_list は PWD が projects/ 配下のときだけプロジェクト .env を表示する
ため、TUI (通常 DEVBASE_ROOT で稼働) から project スコープを選んでも何も
表示されなかった (codex round1 major 指摘)。set/project と同様に
_select_project で対象を選ばせ、_run_in_project 経由で実行するよう修正。
回帰テスト (chdir + 復帰 / 選択中止) を追加。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@takemi-ohama

takemi-ohama commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ (round 1)

  • 対応件数: 1 件 (critical=0 / major=1 / minor=0 / nit=0)
  • deferred: 0 件 / rejected: 0 件
  • 修正コミット: 03bbf68
  • テスト: uv run pytest -q → 593 passed, 1 skipped
  • CI: チェック未設定 (no checks reported)

対応内容

# severity path:line 指摘 対応
1 major lib/devbase/tui/actions_env.py:239 env list の「プロジェクトのみ」選択時に project 選択・chdir/PWD 切替がなく、トップメニュー起動では project .env が表示されない _select_project で対象プロジェクトを選ばせ、_run_in_project 経由 (chdir + PWD 切替 → 実行 → 復帰) で cmd_env_list を呼ぶよう修正。chdir + 復帰・選択中止の回帰テストを追加

該当 thread には reply 済み・Resolve 済みです。

🤖 Generated with Claude Code

takemi-ohama
takemi-ohama commented Jun 11, 2026
View reviewed changes

@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 2 | codex | REQUEST_CHANGES

TUI の env get が project .env を取得できないため、project スコープを選択して PWD 切替後に実行する経路を追加してください。

Comment thread lib/devbase/tui/actions_env.py Outdated
takemi-ohama
takemi-ohama commented Jun 11, 2026
View reviewed changes

@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 2 | gemini | APPROVE

PLAN 2.3/3.3 の契約に従い、env カテゴリの TUI 配線、CWD / PWD のスコープ切り替え機構、および CLI parser 既定値との同期が適切に実装・テストされていることを確認しました。問題ありません。

@takemi-ohama @claude
fix: env get に取得元選択を追加しプロジェクト .env のキーを取得可能にする
b6ee3ec 
cmd_env_get はグローバル .env に無いキーを CWD (PWD) のプロジェクト .env へ
フォールバックして探すが、TUI は常に DEVBASE_ROOT で実行されるため
プロジェクト固有キーを取得できなかった (codex round2 指摘)。
list/set と同様に取得元 (グローバル / プロジェクト) を選ばせ、プロジェクト
選択時は _run_in_project 経由で chdir + PWD 切替後に dispatch する。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@takemi-ohama

takemi-ohama commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

🔧 /ndf:fix サマリ (round 2)

対応結果

  • 対応件数: 1 件 (major 1 / critical 0 / minor 0 / nit 0)
  • deferred: 0 件 / rejected: 0 件
  • 修正 commit: b6ee3ec

対応内容

# 指摘 (codex) 重要度 対応
1 env get は TUI が常に DEVBASE_ROOT で実行されるため、CWD のプロジェクト .env へのフォールバック (cmd_env_get) が効かずプロジェクト固有キーを取得できない major (再判定: major 妥当) 取得元選択 (グローバル / プロジェクト) を追加し、プロジェクト選択時は list/set と同様に _run_in_project 経由 (chdir + PWD 切替) で dispatch。chdir + 復帰・各種中止経路の回帰テストを追加

テスト

  • uv run pytest -q: 597 passed, 1 skipped (回帰テスト 5 件追加)

なお gemini review は APPROVE (指摘 0 件) でした。

takemi-ohama
takemi-ohama commented Jun 11, 2026
View reviewed changes

@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 3 | gemini | APPROVE

特に修正すべき点はありません。PLAN31_2 に沿って正確に実装・テストされています。

takemi-ohama
takemi-ohama commented Jun 11, 2026
View reviewed changes

@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 3 | codex | APPROVE

新規の修正要求はありません。

@takemi-ohama takemi-ohama merged commit 56dc0f5 into release/PLAN31_2 Jun 11, 2026
takemi-ohama added a commit that referenced this pull request Jun 11, 2026
@takemi-ohama @claude
merge: release/PLAN31_2 (PR #58 env-ops) を取り込み app 配線競合を解消
81337a7 
- _route に env + plugin の両配線を統合
- test_app.py の「未実装カテゴリ」前提テストを snapshot へ差し替え

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
takemi-ohama added a commit that referenced this pull request Jun 11, 2026
@takemi-ohama @claude
merge: release/PLAN31_2 (PR #58/#59) を取り込み app 配線競合を解消
cbf6725 
- _route に env/plugin/snapshot/status の全配線を統合
- 全カテゴリ配線済みに伴い「未実装カテゴリ」前提のテストを
  MENU_BACK mock / 未知カテゴリ fallback 検証へ書き換え

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
takemi-ohama added a commit that referenced this pull request Jun 11, 2026
@takemi-ohama @claude
feat(list): devbase list を全操作対応の階層メニュー TUI へ統合 (PLAN31_2) (#55)
e6814ed 
* chore: PLAN31_2 release ブランチ初期化 (Draft PR 用)

* feat: PLAN31_2-tui-framework TUI 土台 (tui/ + メニューエンジン + 既存 project 操作の非回帰移送) (#56)

* chore: PLAN31_2-tui-framework Draft PR 作成

* feat(tui): PLAN31_2 TUI 土台を tui/ へ分離・トップ階層メニュー化 (PR1 #56)

devbase list の対話 TUI を commands/project.py から lib/devbase/tui/ パッケージへ
分離し、全カテゴリを束ねるトップ階層メニューの入口を新設する (PLAN31_2 PR1)。

- tui/menu.py: questionary ラッパ・MENU_BACK 番兵・Esc/← バインド・引数収集
  ヘルパ (text/confirm/integer/path, input フォールバック)
- tui/dispatch.py: dispatch_lifecycle / dispatch_group (旧 _start_project_action
  を一般化、PR3〜5 の group 委譲も先行整備)
- tui/actions_project.py: 既存 project up/down/rebuild の非回帰移送 (一覧選択→
  running はサブメニュー→他は直接 up、番号入力フォールバック)
- tui/app.py: トップ階層メニュー (project/env/plugin/snapshot/status)。
  「プロジェクト操作」を先頭・既定ハイライトで従来フローへ Enter 連打到達 (3.2)。
  env 以降は後続 PR 用プレースホルダ (_route に1行追加で配線)
- commands/project.py: listing/整形の純粋ロジックへ縮約、cmd_project_list は
  tui.run への薄い委譲に置換

後方互換: --plain/非TTY→table、questionary 不在→従来の番号入力フォールバックへ縮退。

テスト: TUI エンジン系を tests/cli/tui/ へ移設・拡充。全 pytest 510 passed/1 skipped。

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

* fix(tui): 実行 rc をトップメニューまで伝搬 + text/path 再帰除去 + Esc バインド (PR56 round1)

cross-review round1 の 3 件を修正:

1. [major] actions_project.run() が dispatch_lifecycle の戻り値を捨てており
   project up/down/rebuild 失敗時も devbase list の終了コードが常に 0 になる問題を
   修正。run() は操作実行時に rc(int) を返すプロトコルへ拡張し、app._top_menu_loop が
   last_rc を記憶して終了コードへ伝搬する (判定は is 同一性で rc=0 を誤マッチさせない)。

2. [minor] menu.text() / menu.path() の allow_empty=False 空入力時の自己再帰呼び出しを
   while ループへ変更し RecursionError リスクを除去。

3. [minor] questionary 経路の text/confirm/path に Esc→中止 (None) バインドが無く
   docstring/select のナビ規約と不整合だったため _ask_with_escape ヘルパで
   with_escape_cancel を適用。

テスト追従: rc 伝搬・非0 rc 伝搬・Esc バインド登録・while 再入力ループの検証を追加
(510 passed → 520 passed / 1 skipped)。

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

---------

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

* docs(PLAN31_2): PR1 (#56) 完了を進捗に反映 + PR2〜5 申し送り追記

PR1 を cross-review approved → release へ merge (50ab9c2) 済みに更新し、
後続 PR (project/env/plugin/snapshot/status) が tui 土台へ配線する手順
(_route 1 行追加・menu 引数収集・dispatch_group・confirm/chdir) を申し送りとして追記。

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

* feat: PLAN31_2-project-ops project login/ps/logs/scale/build の TUI 追加 (#57)

* chore: PLAN31_2-project-ops Draft PR 作成

* feat(tui): PLAN31_2 project 操作を login/ps/logs/scale/build へ拡張 (PR2 #57)

running 行の操作サブメニューを up/down/login/ps/logs/scale/build/rebuild の全操作へ
拡張し、各操作の引数を tui.menu の収集ヘルパで CLI と同じ属性 (plan 2.3 契約) として
集める。stopped/unknown は従来どおり直接 up (PR1 非回帰)。

- login: index (既定 "1") を text で収集
- ps: --all を confirm で収集
- logs: --follow を confirm、--tail を optional int (空=全件) で収集
- scale: new_scale を integer (min=1) で収集
- build: containers/<image>/Dockerfile を列挙して選択 (compose.yml 全体= image None)
- down: 破壊的操作のため confirm で確認 (plan 3.4)
- 引数収集を Esc/Ctrl-C で中止したら操作サブメニューへ戻る (_ARG_CANCEL)

login/ps/logs/scale は running コンテナ対象のため running 行限定とした。
全 pytest 542 passed / 1 skipped (PR2 で +22)。

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

* fix(tui): project login の空 index バグ修正 + logs --tail 負数バリデーション (PR2 #57 round1)

- login: menu.text(空入力で "" → --index= 失敗) を menu.integer(default=1,
  min_value=1) に変更し、str(index) で文字列契約を満たして dispatch。
- _optional_int: min_value=0 検証を追加し、logs --tail への負数入力を弾いて
  再入力を促す (docker compose エラー防止)。
- テスト: login を menu.integer モックに追従、負数再入力テストを追加。

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

---------

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

* feat: PLAN31_2-env-ops env 全操作の TUI 追加 (#58)

* chore: PLAN31_2-env-ops Draft PR 作成

* feat: PLAN31_2-env-ops env 全操作の TUI 追加

- tui/actions_env.py 新設: env init/list/set/get/delete/edit/sync/project/
  export/import を選択メニュー + 引数収集で cmd_env へ委譲 (plan 2.3 契約)
- project スコープ (set --project / project) は事前にプロジェクト選択 →
  chdir + PWD 切替で実行し、try/finally で必ず元の CWD/PWD へ復帰 (plan 3.3)
- 破壊的な env delete は実行前に menu.confirm で確認 (plan 3.4)
- export/import は主要引数 (dest/source) のみ収集し、残りは CLI parser 既定値と
  同一の属性を明示付与 (parser との同期テスト付き)
- tui/app.py の _route に env を配線、未実装カテゴリ前提のテストを plugin へ更新
- tests/cli/tui/test_actions_env.py 新設 (契約・chdir 復帰・confirm・ナビ遷移)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix: env list の「プロジェクトのみ」で対象プロジェクト選択 + chdir/PWD 切替を行う

cmd_env_list は PWD が projects/ 配下のときだけプロジェクト .env を表示する
ため、TUI (通常 DEVBASE_ROOT で稼働) から project スコープを選んでも何も
表示されなかった (codex round1 major 指摘)。set/project と同様に
_select_project で対象を選ばせ、_run_in_project 経由で実行するよう修正。
回帰テスト (chdir + 復帰 / 選択中止) を追加。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix: env get に取得元選択を追加しプロジェクト .env のキーを取得可能にする

cmd_env_get はグローバル .env に無いキーを CWD (PWD) のプロジェクト .env へ
フォールバックして探すが、TUI は常に DEVBASE_ROOT で実行されるため
プロジェクト固有キーを取得できなかった (codex round2 指摘)。
list/set と同様に取得元 (グローバル / プロジェクト) を選ばせ、プロジェクト
選択時は _run_in_project 経由で chdir + PWD 切替後に dispatch する。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

* feat: PLAN31_2-plugin-ops plugin 全操作 + repo の TUI 追加 (#59)

* chore: PLAN31_2-plugin-ops Draft PR 作成

* feat: PLAN31_2-plugin-ops plugin 全操作 + repo の TUI 追加

- tui/actions_plugin.py 新設: plugin list/install/uninstall/update/info/
  sync/migrate と repo add/remove/list/refresh のサブ階層メニューを実装。
  引数は menu.* で収集し dispatch_group 経由で cmd_plugin へ委譲
  (plan 2.3 の属性契約は cli.py parser と突き合わせて検証済み・乖離なし)
- uninstall/update/info と repo remove/refresh の name は registry
  (plugins.yml) の一覧から選択。破壊的な uninstall / repo remove は
  menu.confirm で実行前確認 (plan 3.4)
- tui/app.py の _route に plugin を配線 (未実装プレースホルダ案内を解消)
- tests/cli/tui/test_actions_plugin.py 新設 + test_app.py の routing 検証更新
  (597 passed / 1 skipped・退行なし)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

* feat: PLAN31_2-snapshot-status snapshot 全操作 + status 閲覧の TUI 追加 (#60)

* chore: PLAN31_2-snapshot-status Draft PR 作成

* feat: PLAN31_2-snapshot-status snapshot 全操作 + status 閲覧の TUI 追加

- tui/actions_snapshot.py 新設: create/list/restore/copy/delete/rotate の
  操作メニューと引数収集 (属性契約は cli.py parser と突き合わせ済み)。
  restore/copy/delete の対象は SnapshotManager.list() の既存一覧から選択
  (取得失敗時は自由入力へ縮退)。破壊的な restore/delete は menu.confirm で
  確認し、拒否時は実行しない (plan 3.4)
- tui/actions_status.py 新設: cmd_status(devbase_root) への薄い委譲
  (引数なし・閲覧のみ。表示後トップへ rc を返す)
- tui/app.py: _route に snapshot/status を配線 (残りは PR3 env / PR4 plugin)
- テスト 39 件追加 (582 passed / 1 skipped、ベースライン 544 から退行ゼロ)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

* docs(PLAN31_2): PR2〜5 (#57-#60) 完了を進捗に反映 + env edit スコープの plan 訂正

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix(tui): dispatch 委譲層で CWD/環境変数を実行後に復元 (PR #55 round1 major)

TUI から project 操作を実行すると _resolve_project_name の
os.chdir / env 反映 / COMPOSE_PROJECT_NAME 上書きが同一プロセスに
残留し、トップメニュー復帰後の env get 等が直前プロジェクトの
CWD・環境変数 (PWD 含む) を参照していた。

委譲チョークポイントの tui.dispatch に _preserve_cwd_env コンテキスト
マネージャを追加し、dispatch_lifecycle / dispatch_group の前後で
CWD と os.environ を保存・復元する (例外時も finally で保証)。
回帰テスト 3 件を追加。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix(tui): 選択メニューの Ctrl-C を Esc と区別し全体中止を伝搬 (PR #55 round2 minor)

menu.select(back=True) ベースの選択ヘルパが Ctrl-C (None) と Esc/←
(MENU_BACK) を両方 _ARG_CANCEL に畳んでいたため、選択中の Ctrl-C が
ナビ規約の「全体中止」にならずサブメニュー再表示になっていた。

- actions_env: _select_project / list・set・get の scope 選択で
  None (Ctrl-C) を呼び出し元まで伝搬し、Esc のみ _ARG_CANCEL とする
- actions_plugin: _select_name で同様に区別。all_label の「全対象」は
  "" のまま返し、呼び出し側 (update / repo refresh) で None へ変換
- actions_snapshot: _select_snapshot_name で同様に区別 (text 縮退時は
  Esc/Ctrl-C を区別できないため従来どおり _ARG_CANCEL)
- actions_project: _select_build_image で同様に区別。「compose.yml
  全体」は "" 番兵に統一し、呼び出し側で image=None へ変換
- 各 _run_operation は選択ヘルパの None を return し、run ループ経由で
  トップループの全体中止 (None) に到達させる

テスト: 既存のキャンセル系を Esc=_ARG_CANCEL / Ctrl-C=None の新契約に
更新し、Ctrl-C 伝搬の回帰テスト 8 件を追加 (693→701 passed)。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix(tui): env list の「グローバル + プロジェクト」もプロジェクト選択後に chdir して実行 (PR #55 round3 major)

TUI は DEVBASE_ROOT で動くため、both スコープを CWD 判定のまま dispatch すると
cmd_env_list の projects/ 配下チェックに掛からずグローバルしか表示されなかった。
project スコープと同様に対象プロジェクトを選ばせ、_run_in_project で
chdir + PWD 切替後に global_only=False / project_only=False で実行する。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix(tui): 引数入力プロンプトの Ctrl-C を Esc と区別し全体中止を伝搬 (PR #55 round4 major)

menu.text/confirm/path が Esc と Ctrl-C を同じ None で返し、各 actions_* が
それを _ARG_CANCEL に畳んでいたため、引数入力中の Ctrl-C がナビ規約の
「全体中止」にならずサブメニュー再表示になっていた (round2 で対応した
選択メニューと同じ問題が入力プロンプトに残存)。

- menu: _ask_with_escape を with_escape_back(bind_left=False) ベースへ変更し、
  text/confirm/path/integer が Esc=MENU_BACK / Ctrl-C=None を区別して返す
  新契約に統一 (← は入力カーソル移動用に空ける)
- actions_env/plugin/project/snapshot: 入力ヘルパの None (Ctrl-C) を
  呼び出し元まで return し、MENU_BACK (Esc) のみ _ARG_CANCEL とする。
  confirm の「not ok」判定は MENU_BACK が truthy のため is 判定を先行
- _optional_int / _optional_point: 「空入力 = None (既定動作)」と Ctrl-C の
  None が衝突するため専用番兵 _ABORT を導入し、呼び出し側で None へ変換

テスト: 入力系キャンセルを Esc=_ARG_CANCEL / Ctrl-C=None の新契約へ
パラメトライズ更新し、Ctrl-C 全体中止と MENU_BACK 伝搬の回帰テストを追加
(701→732 passed)。

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@takemi-ohama