feat: runtime-split-shared-builder shared builder

[takemi-ohama]

Summary

• plan: issues/agent-runtime-plugin-split.md
• release PR: release: runtime plugin split #45
• 担当範囲: ndf-shared と生成スクリプトを追加し、現行 plugins/ndf からランタイム別生成物を作れる状態にする
• 依存: なし

変更内容

• plugins/ndf-shared/ を追加し、共有Skill、共有script、runtime別Skill公開manifestを配置
• scripts/build-runtime-plugins.sh を追加し、共有ソースから生成物を同期できるようにした
• 既存互換のため、現行 plugins/ndf/skills と plugins/ndf/scripts は生成済み状態で残した
• 将来の plugins/ndf-claude / plugins/ndf-codex / plugins/ndf-kiro ディレクトリが存在する場合も、同じbuild scriptで skills/ と scripts/ を同期できる入口を用意した

Test plan

• bash scripts/build-runtime-plugins.sh
• bash scripts/build-runtime-plugins.sh --check
• python3 -m json.tool plugins/ndf/.claude-plugin/plugin.json >/dev/null
• python3 -m json.tool plugins/ndf/.codex-plugin/plugin.json >/dev/null
• claude plugin validate plugins/ndf
• 共有配置と旧配置の両方にSkill/scriptが存在することを確認

[takemi-ohama]

🤖 cross-review | round 1 | codex | REQUEST_CHANGES

生成契約に修正が必要です。plugins/ndf/skills の互換性と Codex 向け skills-codex の再生成/検証対象化を揃えてください。

[takemi-ohama]

🤖 cross-review | round 1 | gemini | REQUEST_CHANGES

• 現在の plugins/ndf が各ランタイム共通で使われている状況下で、plugins/ndf/skills の生成を claude-skills.txt に限定しているため、意図せず多くの既存スキル（playwright-* など）が削除され後方互換性が壊れています。ndf-codex などへの完全移行までは plugins/ndf に全スキルを残す修正を提案します。
• ビルドスクリプトの検査ロジックにおいて、一時ファイルのパスが固定になっているため並行実行時の競合リスクがあります。

[takemi-ohama]

/ndf:fix 46 --defer-nit Summary

• fixed: 4件（critical 1 / major 2 / minor 1 / nit 0）
• deferred: 0件
• rejected: 0件
• commit: 99b8786
• CI: reported checks なし（完了待ちは未実施）

対応内容:

• legacy plugins/ndf/skills は shared 全 skill を同期し、移行期間の既存参照を維持
• legacy plugins/ndf/skills-codex を codex-skills.txt から生成/検証し、Codex 専用パス差分も生成時に再現
• --check の diff 出力を各一時ディレクトリ配下へ分離

[takemi-ohama]

🤖 cross-review | round 2 | codex | REQUEST_CHANGES

Codex runtime 向けの生成パス変換に不足があり、公開済み skill が実行時に誤ったディレクトリを参照します。

[takemi-ohama]

🤖 cross-review | round 2 | gemini | APPROVE

PRの目的である plugins/ndf-shared の導入とビルドスクリプトの追加を確認しました。生成物との整合性も取れており、適切にディレクトリ構造が再編されています。
Pythonスクリプト内に1点だけ未使用の変数が残っているため、将来の保守のために整理を提案します。

[takemi-ohama]

/ndf:fix 46 --defer-nit Round 2 対応を完了しました。

• fixed: 2件（major 1 / minor 1）
• deferred: 0件
• rejected: 0件
• CI: checks 未報告（完了待ちはしていません）
• commit: 8096c8e

対応内容:

• Codex legacy 生成時に cross-review 本体と docs の script 解決パスも CODEX_PLUGIN_ROOT / skills-codex へ変換
• cross-review state.py の未使用 meta_fail 代入を削除

[takemi-ohama]

🤖 cross-review | round 3 | codex | REQUEST_CHANGES

plugins/ndf-codex 生成経路では Codex runtime 用の root 解決が未適用のため、生成物が Claude 専用環境変数に依存したままになる点を修正してください。

[takemi-ohama]

🤖 cross-review | round 3 | gemini | REQUEST_CHANGES

macOS 環境でのビルドスクリプト実行互換性のため、sed -i の書き換え処理を修正してください。また、対象の Skill が manifest から外れた際のエラーハンドリングを追加してください。

[takemi-ohama]

/ndf:fix 46 --defer-nit Round 3 対応を完了しました。

• fixed: 3件（major 2 / minor 1）
• deferred: 0件
• rejected: 0件
• CI: checks 未報告（完了待ちはしていません）
• commit: 29dbb4a

対応内容:

• plugins/ndf-codex/skills 生成時にも Codex 向け root 解決を適用
• legacy skills-codex と runtime skills で script path 変換先を分離
• sed -i を廃止し、一時ファイル経由の置換に変更
• 変換対象ファイルの存在確認を追加

[takemi-ohama]

🤖 cross-review | round 4 | codex | APPROVE

指摘はありません。

[takemi-ohama]

🤖 cross-review | round 4 | gemini | APPROVE

設計と実装は妥当であり、build-runtime-plugins.sh の同期ロジック（cp -a による一時ディレクトリ経由の更新、一時ファイルの削除、sed によるパス置換）は正しく機能していることを確認しました。

1点だけ、今後の運用保守に向けた提案があります。

• plugins/ndf/skills や plugins/ndf/scripts が生成物（artifact）となったため、利用者が誤って手動編集するのを防ぐ目的で、ルートの .gitattributes に plugins/ndf/skills/** linguist-generated=true 等を追加するか、または各生成先ディレクトリに README.md (ここは直接編集しないでください、等の警告) を生成・配置することを推奨します。

現在の実装で既存の互換性は完全に保たれており、リスクへの対応もできているため APPROVE します。