feat: runtime-split-kiro Kiro plugin#49
Conversation
e923453 to
df30747
Compare
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | REQUEST_CHANGES
scripts/install-kiro.sh を削除して Kiro の入口を plugins/ndf-kiro/install.sh に移した一方で、plugins/ndf/README.md には旧コマンドが複数残っています。マージ後に存在しない installer を案内するため、同 README の Kiro セクションと hooks 表の参照も新パスへ更新してください。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | gemini | APPROVE
- Kiro プラグインの分離、および関連する構成変更が意図通りに行われていることを確認しました。
- 新規追加された
build-runtime-plugins.shや各種 shell script についても、安全な作法 (set -euo pipefail, 適切なクオート等) で記述されていることを確認しました。
[minor / 堅牢性]
現在 build-runtime-plugins.sh の rewrite_codex_skill_paths にて Codex 環境向けの ${PLUGIN_ROOT} フォールバックを置換していますが、Kiro 環境でもユーザーが明示的に PLUGIN_ROOT を export していない場合、${CLAUDE_PLUGIN_ROOT} も空となりパスが /skills/... に解決されてエラーになる懸念があります(fix, cross-review, statusline 等のスキル)。
将来的な不具合を防ぐため、Kiro 向けにも KIRO_PLUGIN_ROOT 等を挟む置換処理 (rewrite_kiro_skill_paths) を追加するか、KIRO.md 内で PLUGIN_ROOT の設定手順を明記することを検討してください。
cross-review fix summary対応コミット: 対応内容:
検証:
codex review body の README 指摘、および gemini review body の Kiro path 懸念も同じ修正で対応済みです。deferred / rejected はありません。 |
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | gemini | APPROVE
設計と変更範囲に問題はありません。仕様通りに Kiro 用プラグインが分離・構成されています。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | codex | REQUEST_CHANGES
.kiro/ 生成物が install 後に未追跡で残るため、生成先の扱いを .gitignore か installer 側で明示してください。
cross-review fix summary / round 2対応コミット: 対応内容:
検証:
今回の deferred / rejected はありません。 |
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 3 | gemini | APPROVE
変更内容を確認しました。とくに指摘事項はありません。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 3 | codex | COMMENT
インラインに1件、installer の再実行時に生成物がオプション状態とずれる修正提案があります。
Summary
plugins/ndf-kiro/を追加し、Kiro 向け README、agent template、workflow prompts、生成済み skills / scripts を同梱しました。scripts/install-kiro.shと root.kiro/生成物を削除し、Kiro installer の入口をplugins/ndf-kiro/install.shに移動しました。.claude-plugin/plugin.jsonに依存しないようにし、plugins/ndf-kiro/skills/から.kiro/skills/symlink と.kiro/agents/default.jsonを生成する方式へ変更しました。README.md/KIRO.md/ Kiro 仕様書 / 親 plan の進捗を新構成に合わせて更新しました。Test plan
bash -n plugins/ndf-kiro/install.shbash scripts/build-runtime-plugins.sh --checkbash plugins/ndf-kiro/install.sh --dry-run --with-slack --with-codex.kiro/agents/default.jsonの JSON validity、prsymlink、codex.mdprompt 生成を確認後、生成.kiroを削除git diff --check