feat: PLAN32-migrate-cmd env から project.yml への変換コマンド

[takemi-ohama]

Summary

旧 env 形式のプロジェクト定義を project.yml へ変換するコマンド devbase project migrate-config を追加します。

devbase project migrate-config --dry-run # 変換結果を確認（書き換えない）
devbase project migrate-config # projects/ 配下を全件変換
devbase project migrate-config carmo uttarov2 # プロジェクトを指定
devbase project migrate-config --projects-dir repos/<repo>/<plugin>/projects

変換対象キー: GIT_USER / GIT_REPO / GIT_HOST / WORK_DIR / CONTAINER_SCALE / DEVBASE_OPEN_EDITOR

設計上の判断

• 変換対象キーは allowlist で限定し、それ以外（ENABLE_SSH、AWS_CONFIG_BASE64 など）は env にそのまま残す。配布中の定義は 3 リポジトリ 136 件あり、取りこぼしを機械的に防ぐ必要がある
• 既存の project.yml は上書きしない。手で複数リポジトリ構成へ整えたものを壊さないため。env の旧キー掃除だけは行うので、何度実行しても同じ状態に収束する
• 旧キーを説明していた直前のコメント行も一緒に落とす。キーだけ消すと、何を説明しているのか分からないコメントが残る
• env ファイル自体は消さない。compose.yml が env_file: - env で参照しており、実在しないと compose が起動時に落ちるため。空になった場合はファイルの役割を書いた雛形を残す
• 生成した YAML はローダで検証してから書き出す。検証に通らない場合は書き出さず failed として報告する
• --projects-dir: projects/<name> は plugin リポジトリへの symlink で、実体側が書き換わる（そこが定義の正なので意図どおり）。一方 plugin リポジトリには devbase へリンクしていない projects もあるため、ディレクトリを直接指定できるようにした

実データでの確認

--dry-run を実際の projects/（42 件）に対して実行し、gitlab.com ホストのプロジェクトが host: gitlab.com を持つこと、書き換え先が plugin リポジトリ内の実体パスになることを確認済みです。

• plan: issues/PLAN32_multi-repo-project.md の Task 4
• release PR: release: PLAN32 1 プロジェクト = 複数リポジトリ構成への変更 (v3.0.0) #103

Test plan

• uv run pytest tests/project/test_migrate.py — 15 件（生成内容 / 旧キー削除 / コメント削除 / 空 env の扱い / 非既定 host・work_dir / open_editor: false / 任意キー未指定 / dry-run / 冪等性 / 既存 project.yml の保護 / 変換不能・env 欠落のスキップ / 検証失敗 / ディレクトリ走査 / symlink 解決）
• uv run pytest tests/commands/test_project_migrate_config.py — 7 件（全件変換 / dry-run / プロジェクト指定 / 未知プロジェクト / 変換失敗時の終了コード / --projects-dir / 不在ディレクトリ）
• uv run pytest — 全 1344 件 green
• 実データ 42 プロジェクトに対する --dry-run

[takemi-ohama]

🤖 cross-review | round 1 | codex | REQUEST_CHANGES

移行のデータ保全とプロジェクト単位の失敗隔離を修正してください。

[takemi-ohama]

🤖 cross-review | round 1 | gemini | REQUEST_CHANGES

自動マイグレーションの堅牢性（エラーハンドリング）に 1 点懸念があります。レガシーな env ファイルから予期せぬ文字列が流れてきた場合でも、一括移行のバッチ処理全体がクラッシュしないように例外処理を調整してください。

[takemi-ohama]

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

対応件数: critical=0 / major=4 / minor=0 (合計 4 件)
deferred: 0 件 / rejected: 0 件
commit: 8f2860a
CI: NONE (このリポジトリはこのブランチに対する GitHub Actions チェックなし)

codex 3 件・gemini 1 件すべて独自に再判定した結果、いずれも実害のある major と判断し全件修正しました。

詳細

#	指摘元	箇所	対応
1	codex migrate.py:78	既存 project.yml が壊れていても env の旧キー (復旧元) を消してしまう	load_project_config で先に検証し、失敗時は env を一切変更せず failed を返す
2	codex migrate.py:100	直接 write_text は truncate されうる / 順序も危険	同一 dir の一時ファイル + fsync + os.replace の atomic write に変更。project.yml 永続化後にのみ env を掃除
3	codex migrate.py:73	1 件の I/O・デコード失敗で 136 件の一括移行が中断	migrate_project をガードで包み OSError / UnicodeDecodeError / yaml.YAMLError / ConfigError を failed に畳んで継続
4	gemini migrate.py:95	閉じ引用符欠けの env で yaml.YAMLError が捕捉されずクラッシュ	_load_yaml で ConfigError にラップ (ローダと同じ挙動) + #3 のガードで二重に防御

atomic write では追加で symlink 自体を置き換えない (os.path.realpath 解決) / 既存ファイルの mode を引き継ぐ (mkstemp の 0600 のままにしない) 点も入れています。

検証

• uv run pytest → 1348 passed (回帰テスト 4 本追加)
  • test_broken_existing_project_yml_keeps_env_untouched
  • test_writes_are_atomic
  • test_unreadable_env_is_reported_as_failed
  • test_broken_yaml_from_env_fails_only_that_project
• 実データ dry-run: devbase project migrate-config --dry-run → migrated=42 (exit 0、挙動変化なし)

@codex@gemini 再レビューをお願いします。

[takemi-ohama]

🤖 cross-review | round 2 | codex | REQUEST_CHANGES

YAML 文字列スカラーの暗黙型変換を防ぎ、旧 env で有効なリポジトリ名を欠落なく移行してください。

[takemi-ohama]

🤖 cross-review | round 2 | gemini | APPROVE

• 移行対象のキーを allowlist で限定し、既存の project.yml を保護するフェイルセーフな設計が移行スクリプトとして適切です。
• os.replace によるアトミックな書き込みや、パース失敗時に当該プロジェクトのみを failed に倒して後続を止めないエラーハンドリングが実装されており、バッチ処理として堅牢です。
• env が空になった場合もファイルを残す処理（compose の制約への配慮）や、symlink 先の実体を正しく解決する仕組みがテストと共に正しく実装されていることを確認しました。

[takemi-ohama]

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

対応件数: critical=0 / major=1 / minor=0 (合計 1 件)
deferred: 0 件 / rejected: 0 件
commit: d4ee1b9
CI: NONE (このブランチに対する GitHub Actions チェックなし)

round 2 の未解決指摘は codex の 1 件のみ (gemini は APPROVE)。独自に再判定した結果、GIT_REPO=123 / on / 2026-08-22 などで実際に移行が failed になる major と確認し修正しました。

詳細

#	指摘元	箇所	対応
1	codex migrate.py:293	旧 env では有効な文字列 (GIT_REPO=123 / on) が YAML の暗黙型で int・bool になり移行失敗	host / owner / repo / work_dir を yaml.safe_dump でスカラー出力し、引用が必要な値だけ引用

生成物の形式は変わりません。 最小引用なので carmo-web / volareinc / /work/carmo のような通常の値は素のまま出力され、既に移行済みの 8 プロジェクトの project.yml と同一の見た目になります (test_plain_values_are_written_without_quotes で固定)。

引用化の副作用として GIT_REPO="carmo (閉じ引用符欠け) が repo: '"carmo' として YAML 的に valid になってしまうため、round 1 の gemini 指摘で failed に倒していた挙動を保つガード (移行対象の文字列値に引用符が残っていたら malformed な env として failed) を併せて追加しています。

検証

• uv run pytest → 1350 passed (回帰テスト 2 本追加)
  • test_string_values_that_look_like_yaml_scalars_stay_strings
  • test_plain_values_are_written_without_quotes
• 実データ dry-run: devbase project migrate-config --dry-run → already=39 / skipped=3 (failed なし)

@codex@gemini 再レビューをお願いします。

[takemi-ohama]

🤖 cross-review | round 3 | codex | APPROVE

追加の修正必須事項はありません。

[takemi-ohama]

🤖 cross-review | round 3 | gemini | APPROVE

変更内容を確認しました。スキーマの厳密な検証（型チェックや制御文字の排除）、移行ツールの安全なファイル書き換え（アトミック操作）、コンテナ側でのシェルスクリプトの堅牢なパース処理など、全体的に手堅く実装されており、テストも十分に書かれているため指摘事項はありません。