feat(plugin): requires.devbase をインストール時に検証する by takemi-ohama · Pull Request #112 · devbasex/devbase

takemi-ohama · 2026-08-23T03:10:05Z

Summary

plugin.yml の requires.devbase を devbase plugin install の時点で検証し、要件を満たさない Plugin のインストールを中止します。

これまで requires.devbase は値を読むだけで比較しておらず、宣言だけの状態でした。そのため project.yml 形式（devbase 3.0.0 以降）の Plugin を 2.x へインストールでき、devbase up の段階で初めて失敗していました。

Error: プラグイン 'carmo-web' は devbase >=3.0.0 を要求していますが、現在の devbase は 2.2.0 です。
devbase を更新してから再度インストールしてください (検証を飛ばす場合は DEVBASE_IGNORE_PLUGIN_REQUIRES=1)。

設計上の判断

検証は既存インストールに触れる前に行います。--link 経路は入れ替えのため既存ディレクトリを消してから登録するので、後から落とすと消した既存 Plugin が戻らないまま失敗します
解釈できない書式・版数はエラーにしません（警告して続行）。"^3.0.0" のような独自記法を書いた Plugin をインストール不能にするより実害が小さく、依存が本当に足りなければ後段で失敗します
DEVBASE_IGNORE_PLUGIN_REQUIRES=1 で検証を無効化できます。検証側の判断が誤ったときに作業が詰まらないための逃げ道です
対応書式は PEP 440 の部分集合（>= / <= / > / < / == / !=、カンマ区切りは AND、演算子省略は ==）。比較は数値で行うため 10.0.0 >= 3.0.0 が正しく判定されます

変更

新規 lib/devbase/plugin/requirements.py
installer.py の 2 つのインストール経路（repos 経由 / --link）へ組み込み
docs/plugin-dev/plugin-yml-reference.md に書式一覧と検証の挙動を追記
CHANGELOG.md、issues/PLAN32_multi-repo-project.md（リリース後の対応と残作業）を更新

Test plan

uv run pytest tests/plugin/test_requirements.py — 31 件（満たす/満たさない各書式、要件なし、解釈不能な書式・版数の警告続行、環境変数での無効化、既定で動作中の devbase 版を見ること、インストール経路の中止、中止時に既存 Plugin が壊れないこと）
uv run pytest — 全 1408 件 green

plugin.yml の requires.devbase は値を読むだけで比較しておらず、宣言だけの状態だった。project.yml 形式 (devbase 3.0.0 以降) の Plugin を 2.x へインストールできてしまい、devbase up の段階で初めて失敗する。 devbase plugin install の経路 (repos 経由 / --link) で検証し、満たさない場合は中止する。検証は既存インストールに触れる前に行う。後から落とすと、入れ替えのために消した既存プラグインが戻らないまま失敗するため。解釈できない書式や版数はエラーにせず警告に留める。独自記法を書いた Plugin をインストール不能にするより実害が小さく、依存が本当に足りなければ後段で失敗する。検証側の判断が誤ったときのために DEVBASE_IGNORE_PLUGIN_REQUIRES=1 も用意した。あわせて PLAN32 の plan にリリース後の対応と残作業を追記した。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

takemi-ohama

🤖 cross-review | round 1 | codex | REQUEST_CHANGES

requires.devbase の版数比較で、サポート外の要素を無視せず誤判定を防いでください。

takemi-ohama

🤖 cross-review | round 1 | gemini | REQUEST_CHANGES

総評・PR横断の指摘事項:

devbase plugin update 時の考慮漏れ: 本 PR は plugin install 時のチェックを追加していますが、devbase plugin update コマンドで git pull を行った際にもプラグインが要求する devbase の版が上がり、互換性がなくなる可能性があります。lib/devbase/plugin/updater.py の _update_repo_plugins でも要件チェックを呼び出し、要件を満たさなくなったプラグインがある場合は警告やエラーを表示して、ユーザーに devbase 本体の更新を促す処理の追加を検討してください（事後報告であっても問題に気づけることが重要です）。

インストールをクラッシュさせる可能性がある AttributeError の指摘を含んでいるため、修正をお願いします。

cross-review round 1 の指摘対応。 - 版数の 4 要素目以降を [:3] で黙って切り捨てていたため、要件 >=3.0.0.1 を現在版 3.0.0 が満たすと誤判定していた。切り捨てをやめ、比較時に短い方を 0 で埋めて桁を揃える。">=3.0" と 3.0.0 が等しいことは変わらない。 - plugin.yml に `devbase: 3.0` とクォート無しで書くと YAML が float にするため、 .strip() が AttributeError になりインストールがクラッシュしていた。 load_plugin_info で str へ寄せ、検証側でも型を信用せず str() を通す。 - DEVBASE_IGNORE_PLUGIN_REQUIRES=FALSE / No のような書き方で、無効化のつもりが無い値なのに検証が黙って飛んでいた。比較を小文字化して揃える。 - devbase plugin update は git pull で requires.devbase が上がっても素通しで、 devbase up まで気づけなかった。更新はもう済んでいて中止できないため、 warn_unmet_devbase_requirement で警告だけ出す。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XVfeqnhZx7meRPUQAyBTta

takemi-ohama · 2026-08-23T03:21:05Z

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

対応件数: critical=1 / major=1 / minor=1 / review-body 指摘=1 (合計 4 件)
deferred: 0 件 / rejected: 0 件
commit: 1b8b9ef
CI: 前コミット時点は全 pass（本コミット分は再実行中、完了待ちはしていません）
テスト: uv run pytest 全 1427 件 green（1408 → +19 件）

対応内容

#	指摘元	箇所	内容	対応
1	codex `major`	requirements.py:115	4 要素以上の版数を `[:3]` で切り捨て → `>=3.0.0.1` を `3.0.0` が満たすと誤判定	切り捨てを廃止。全要素をタプル化し、比較時に `_align` で短い方を `0` 埋め
2	gemini `critical`	requirements.py:55	`devbase: 3.0` (クォート無し) が `float` になり `.strip()` で `AttributeError`	`load_plugin_info` で `str()` へ正規化（根本）＋検証側でも `str(...)` で防御。2 か所の文字列操作を 1 か所へ集約
3	gemini `minor`	requirements.py:57	`DEVBASE_IGNORE_PLUGIN_REQUIRES=FALSE` で検証が黙って飛ぶ	`.strip().lower()` で比較
4	gemini (総評)	`updater.py`	`devbase plugin update` の考慮漏れ	`_update_repo_plugins` に `warn_unmet_devbase_requirement` を追加。警告のみで中止はしない

#4 の設計判断

devbase plugin update は git pull 済みの作業ツリーを registry へ追認する処理で、
呼ばれた時点で更新はもう完了しています。ここでエラーを投げると registry だけが古いまま
取り残されるため、警告に留めて更新は通す方針にしました
（ご指摘の「事後報告であっても問題に気づけることが重要」と同じ判断です）。

WARNING プラグイン 'carmo-web' は devbase >=4.0.0 を要求していますが、現在の devbase は 3.0.0 です。
devbase 本体を更新してください (この警告を止める場合は DEVBASE_IGNORE_PLUGIN_REQUIRES=1)。

インストール経路は従来どおり 既存に触れる前に検証してエラーで中止 します（非対称なのは意図的です）。

追加テスト (+19 件)

4 要素以上の版数: 満たす / 満たさない両方向（>=3.0.0.1 vs 3.0.0 / 3.0.0.1 / 3.0.1、<3.0.0.1 vs 3.0.0.2）
YAML 型ゆれ: PluginInfo へ 3.0 / 3 を直接渡すケースと、実 plugin.yml から load_plugin_info が "3.0" を返すケース
環境変数: 0 / false / FALSE / No / no / 空白 / 空文字 で検証が走ること
更新経路: _update_repo_plugins が要件違反でも errors == [] で version を更新し、警告を出すこと

ドキュメント (docs/plugin-dev/plugin-yml-reference.md) と CHANGELOG.md にも桁合わせの規則と更新時警告を追記しました。

@takemi-ohama 再レビューをお願いします。

takemi-ohama

🤖 cross-review | round 2 | codex | REQUEST_CHANGES

数値として解釈された requires.devbase が元の版表記を失い、互換性を誤判定するケースを修正してください。

takemi-ohama

🤖 cross-review | round 2 | gemini | APPROVE

Round 1 の指摘事項（型のゆれ、バージョンの切り捨て比較、検証スキップの環境変数の判定など）がすべて解消され、テストで保護されていることを確認しました。新たな修正提案はありません。

`devbase: 3.10` とクォート無しで書くと YAML は float の 3.1 として読む。 str へ寄せると "3.1" になり元の 3.10 へ戻せないため、現在版 3.1 が ">=3.10" を満たすと誤判定していた。非文字列は復元不能として requires_devbase を None にし、クォートを促す警告を出す。誤った版で比較するより、検証しない方が安全。requirements 側にも同じ防御を残し、直接構築された PluginInfo でも落ちないようにした。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

takemi-ohama

🤖 cross-review | round 3 | codex | REQUEST_CHANGES

update 時の plugin 分割移行で、互換性検証より先に既存登録を削除しないよう修正が必要です。

takemi-ohama

🤖 cross-review | round 3 | gemini | APPROVE

実装は十分堅牢であり、エラー系やエッジケース（YAML起因の型ゆれ、不正な版数指定等）への防衛的対処・テストも網羅されています。追加の修正アクションはありません。

プラグイン分割の移行 (_migrate_removed_plugin) は旧登録を削除してから移行先を登録する。移行先の requires.devbase が満たせないときに例外へすると、移行先が登録されないまま旧登録も失われる。 git pull は既に済んでおり、ここで止めても registry と作業ツリーの整合は取れない。_register_repo_plugin に enforce_requirements を足し、この経路だけ警告に切り替える。install 経路は従来どおり中止する。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

takemi-ohama

🤖 cross-review | round 4 | codex | APPROVE

修正必須の指摘はありません。

takemi-ohama

🤖 cross-review | round 4 | gemini | APPROVE

実装を確認しました。PEP 440 の部分集合による比較ロジック、誤った型・未知の記法に遭遇した際のフォールバック（警告に留めて中断しない）、および更新時（update）の警告化など、既存の運用を壊さないための防御的な対応が徹底されており、非常に安定した設計だと思います。

テストケースも境界値や無効値、環境変数による迂回手段まで網羅されており、懸念点はありません。このままマージして問題ありません。