Skip to content

#310 を実装する(スプリント m10a) - #440

Merged
takemi-ohama merged 3 commits into
sprint/m10afrom
feat/issue-310-m10a
Oct 7, 2026
Merged

takemi-ohama merged 3 commits into
sprint/m10afrom
feat/issue-310-m10a

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

env コマンドと TUI の env 画面を内部で整理しました。grouped の判定・サブコマンドの振り分け・Enter 待ちを 1 か所へまとめ、env の画面とメニューの循環参照を解消しています(#310、スプリント m10a のブランチへ集める変更)。

利用者向けの変化

  • 無し

移行の手順

  • 無し

変更の要点

  • env コマンドの共通処理を集約(lib/devbase/commands/env.py、subcommands.py)
    • grouped の判定、サブコマンドの振り分け、Enter 待ちを 1 か所にまとめました。
  • sync の同期関数を整理(env.py、env_backend.py、env/secret_store.py)
    • 同期の関数へ文脈を渡す形にしました。
    • GCP のプロファイルの求め方を 1 つに揃えました。
  • TUI の循環参照を解消(tui/actions_env.py、tui/app.py、tui/flow.py)
    • env の画面とメニューの循環をなくしました。
    • 受け渡しの部分は tui/env_delegate.py へ切り出しました。
  • 置き場へ書かない検査の判定方法を変更(tui/actions_env_keys.py、tui/actions_env_openbao.py)
    • 呼び出し元の名前に頼らない形にしました。
  • テストを追随(tests/cli/tui/test_actions_env.py、test_actions_env_keys.py)
    • 上記の構成変更に合わせて更新しました。

テスト

ステップ exit 結果
test-limited 0 範囲テスト 1 本(122.9 秒 / 戦略 local-full)、status: ok

モード: standard / 通した工程: 実装 → 完了判定 → Pull Request

🤖 Generated with Claude Code

takemi-ohama and others added 3 commits October 7, 2026 14:20
- env_rows.is_grouped と _MigrationPlan._grouped をやめ、SecretStore.grouped を使う
- cmd_env と cmd_env_backend の振り分けを run_subcommand へまとめる
- app._pause_for_review は文言を渡して flow.pause_for_review を呼ぶ
- _format_value の伏せる条件と切り詰めの数を名前付きの定数にする

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- sources・targets・store・counts を _SyncContext に束ね、更新件数は write を通して counts へ足す
- _open_sync_targets は (targets, store, rc) の組をやめ、文脈を返すか _SyncAborted を送る
- GCP のキーからプロファイル名と認証ファイルを求める段を _gcp_credentials にまとめ、
  同期 (_sync_gcp) と控えの登録 (_record_gcp_source) が共に使う
- _sync_gcp を控えに無いプロファイルと控えにあるプロファイルの 2 つの関数に分け、入れ子を浅くする

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- cmd_env への委譲と chdir + PWD の差し替えを tui/env_delegate.py へ移す。env メニュー
  (actions_env) と画面はそこを使い、画面からメニューを読まない。_screen の遅延 import をやめる
- グループの選択 (group_attrs) を _select_group と同じ actions_env_keys へ移す
- delegate_attrs などの 'global' を secret_store の KIND_GLOBAL にする
- test_the_tui_never_writes_the_store_itself: inspect.stack で呼び出し元のモジュール名を見る
  検査をやめ、委譲した cmd_env の実行中にだけ置き場への書き込みが起きることを確かめる
  (内部への依存を外した。モジュールを分けても検査が効く)
- test_actions_env の run_in_project のテストは差し替え先が env_delegate に変わった

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@takemi-ohama
takemi-ohama marked this pull request as ready for review October 7, 2026 05:32
@takemi-ohama
takemi-ohama merged commit db39ede into sprint/m10a Oct 7, 2026
10 checks passed
This was referenced Oct 7, 2026
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