Uh oh!
There was an error while loading. Please reload this page.
PLAN32: プロジェクト設定を env から project.yml へ移行する - #4
Conversation
devbase project migrate-config による変換 (6 プロジェクト)。 GIT_USER / GIT_REPO / WORK_DIR / CONTAINER_SCALE / DEVBASE_OPEN_EDITOR を project.yml へ移し、env はコンテナへ渡す環境変数だけを持つファイルにする。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | gemini | APPROVE
環境変数ベースから project.yml への設定移行が、すべてのサンプルプロジェクトで一貫して行われており、設定値の欠落もなく妥当です。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | REQUEST_CHANGES
*/plugin.yml:5:project.ymlを必須にする破壊的変更にもかかわらず、全プラグインが現行リリース v2.2.0 を引き続き許可しています。v2.2.0 はこの設定を読めず、インストール時の互換性チェックを通過した後にdevbase upが失敗するため、PLAN32 を含むリリースで devbase のバージョンを更新し、6 件のrequires.devbaseの下限もそのバージョンへ揃えてください。README.md:7およびREADME.md:48:GIT_USER/GIT_REPOと env のみを示す説明が、今回の唯一の正であるproject.ymlと矛盾しています。表の設定名とリポジトリ構成例を新形式へ更新してください。
cross-review round 1 (codex REQUEST_CHANGES) の指摘対応。 - `*/plugin.yml` 6 件の `requires.devbase` を `">=2.2.0"` → `">=3.0.0"` へ。 `project.yml` 形式は devbase 2.2.0 では読めないため、旧版を許可したままだと install 時の互換チェックを通過した後に `devbase up` が失敗する。 - README を `project.yml` を正とする新形式へ更新。 - 収録プラグイン表の見出しを `GIT_USER/GIT_REPO` → `repos[].owner/repo` へ変更し、 registry.yml に載る 6 件すべてを掲載(従来は 3 件のみ) - 「プロジェクト設定 (project.yml)」節を追加。実ファイル `adminer/projects/adminer/project.yml` と同一の例、複数リポジトリの例、 主要キーの表、devbase 本体 `docs/user/project-yml.md` への誘導を記載 - リポジトリ構成のツリーに `project.yml` を追加し、`env` の役割を明記 - 必要な devbase バージョン (3.0.0 以上) を冒頭に明記 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
takemi-ohama
commented
Aug 22, 2026
🔧 /ndf:fix サマリ (round 1)対応件数: critical=0 / major=2 / minor=0 (合計 2 件) codex のレビュー body の 2 点はいずれもインラインコメントではなく PR レベルの指摘だったため、この PR レベルコメントで対応内容を報告します(Resolve 対象の review thread はありません)。 1. |
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | gemini | APPROVE
devbase 3.0.0 の仕様(project.yml の導入と env の責務変更)に合わせた移行が全プラグインで正確に行われています。設計意図とのズレや矛盾はなく、対応アクションが必要な点はありません。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | codex | APPROVE
追加の修正提案はありません。
Summary
devbase 本体の変更(1 プロジェクト = 1 コンテナ = 複数リポジトリ)に追随し、サンプルプラグイン 6 件のプロジェクト定義を
project.yml方式へ移行します。devbase project migrate-configによる機械的な変換で、設定値は変えていません。envのキーproject.yml)GIT_USERrepos[].ownerGIT_REPOrepos[].repoWORK_DIRwork_dir(既定値と同じ場合は書きません)CONTAINER_SCALEscaleDEVBASE_OPEN_EDITORopen_editorenvは「コンテナへ渡す環境変数」だけを持つ役割になります。中身が無くてもファイル自体は残します(compose.ymlがenv_file: - envで参照するため)。サンプルとしての意味もあるため、1 プロジェクトに複数リポジトリを持たせたい場合は
reposに要素を足すだけで済むことがそのまま示せる形になっています。⚠ マージ順序
devbase 本体の PLAN32 リリース(devbasex/devbase#103)が merge され、
devbase build --no-cacheでベースイメージを再ビルドした後に merge してください。順序が逆だと、旧 devbase がproject.ymlを読めず起動できません。Test plan
project.ymlがローダの検証を通ることenvに旧キーが残っていないことdevbase build --no-cache後、adminerがdevbase upで起動すること