refactor: マージ後処理と cherry-pick の Skill を統合 (0-4)

[takemi-ohama]

Summary

マージ後のブランチ整理が 3 個に分かれ、cherry-pick 系はトリガ語が完全に重複していた。利用実績の多い側の名前を残して統合する。統合は単純連結ではなく、重複記述を落として再構成した。

• merged ← clean + sync-main — 起動 247 回の merged を残す。改名しない
• cherry-pick-pr ← branch-fix-strategy — 実行コマンド側の名前を残す

clean は起動ゼロだが機会が 251 あり、その機会数は merged の起動数 247 とほぼ一致する。ブランチ整理は実際には merged で行われている。sync-main は Git 操作 1 コマンドに 48 行を割いており、merged の 1 節として吸収できる。

統合前後の行数

統合後	統合元（統合前の合計）	統合後	増減
merged	merged 29 + clean 20 + sync-main 48 = 97	58	-39
cherry-pick-pr	cherry-pick-pr 120 + branch-fix-strategy 87 = 207	130	-77
合計	304	188	-116

いずれも 500 行の上限内。Skill 数は 49 → 46、公開セットは Claude 29 → 26 / Codex 30 → 27 / Kiro 28 → 25。

統合の中身

merged: 既存の PR 単位クリーンアップ手順に、clean 由来の「マージ済みブランチの整理」（git branch --merged main からの一括削除）と、sync-main 由来の「main の取り込み」を節として追加。stash / 復元は元々 3 Skill に重複していたため merged の手順へ一本化した。

cherry-pick-pr: branch-fix-strategy の 4 つの核心ルールと「なぜこの順序が重要か」の表、revert 連鎖の注意を取り込んだ。branch-fix-strategy の手動実行手順は本 Skill の処理フローと重複するため削除し、ルール 3 / 4 は該当する処理フローの手順番号を参照する形にした。

frontmatter は description を統合後の範囲に合わせて更新するにとどめ、disable-model-invocation の付け外しとトリガ語（branch-fix-strategy の when_to_use）の再配置は Task 0-7 の担当範囲として本 PR では行わない。

参照の更新

• plugins/ndf-shared/manifests/{claude,codex,kiro}-skills.txt から 3 個を削除（並び順の規則は維持）
• plugins/ndf-claude/.claude-plugin/plugin.json の skills から 3 個を削除
• skills/pr / skills/issue-plan-strategy / skills/deploy / skills/problem-solving の関連リンクを新しい統合先へ差し替え
• docs/ndf-plugin-reference.md の Skill 領域一覧、README.md の Skill 一覧と個数
• bash scripts/build-runtime-plugins.sh で 3 ランタイムの生成物を同期

issues/ 配下のプラン文書は履歴のため書き換えていない。plugins/ndf-claude/agents/ に旧 Skill 名の参照はなかった。

Test plan

• clean / sync-main / branch-fix-strategy の手順が統合先に残っている
• 統合前後の行数が増えていない（304 → 188 行）
• /ndf:merged と /ndf:cherry-pick-pr のコマンド名が変わっていない
• grep -rn 'ndf:clean\|ndf:sync-main\|ndf:branch-fix-strategy\|skills/clean\|skills/sync-main\|skills/branch-fix-strategy' . で残るのは issues/ の履歴文書のみ
• manifest 3 種に反映されている
• bash scripts/build-runtime-plugins.sh --check が差異なし
• python3 scripts/check-markdown-links.py --root . が成功する
• bash scripts/validate-runtime-plugins.sh が成功する（claude plugin validate 含む）

release PR: #66
plan: issues/ndf-development-skills/07-tasks.md

[takemi-ohama]

🤖 cross-review | round 1 | codex | REQUEST_CHANGES

Kiro の workflow prompt に削除済み Skill への参照が残っているため、統合範囲の参照更新が未完了です。

[takemi-ohama]

🤖 cross-review | round 1 | gemini | REQUEST_CHANGES

3つのランタイムへの展開や merged / cherry-pick-pr への内容移譲は概ね正確に行われています。

[takemi-ohama]

🤖 /ndf:fix | round 1 対応サマリ

修正: 2件（同一指摘、major × 2）

重要度	指摘	対応
major	plugins/ndf-kiro/prompts/clean.md が削除済み clean Skill を参照したまま残存（codex / gemini 両方が指摘）	修正済み。clean.md を削除

commit: 5a27c72

install.sh は plugins/ndf-kiro/prompts/ 配下の md を無条件に .kiro/prompts/ へコピーするため、削除済み Skill への導線が残る状態でした。merged.md プロンプトが「PRマージ後のクリーンアップを実行してください（main更新、ブランチ削除）」と旧 clean の役割を包含しているため、内容更新ではなく削除としています。

削除済み Skill (clean / sync-main / branch-fix-strategy) への他の残存参照がないことも確認済みです。

deferred / rejected: 0件

検証結果

• bash scripts/build-runtime-plugins.sh && bash scripts/build-runtime-plugins.sh --check → up to date
• python3 scripts/check-markdown-links.py --root . → Markdown local links are valid
• bash scripts/validate-runtime-plugins.sh → runtime plugin validation passed
• push 前 CI: 全 6 チェック SUCCESS（push 後の再実行は未待機）

本 PR のスコープ外（既知）

• cherry-pick-pr の frontmatter / トリガ語の見直し → Task 0-7
• バージョン番号の更新 → Task 0-10

[takemi-ohama]

🤖 cross-review | round 2 | codex | REQUEST_CHANGES

削除した Skill が担っていた自動選択条件が統合先に移管されていないため、複数ブランチ適用の依頼で Skill が自動参照されにくくなります。あわせて README 下部のプラグイン一覧には旧公開 Skill 数（Claude 29 / Kiro 28 / Codex 30）が残っているため、統合後の 26 / 25 / 27 に更新してください。

Kiro の既存インストール環境では、今回削除した prompt が過去にコピー済みの場合 .kiro/prompts/clean.md として残り続けます。install.sh で廃止 prompt を削除する migration を入れるか、互換 prompt を配布して merged に誘導してください。

[takemi-ohama]

🤖 cross-review | round 2 | gemini | REQUEST_CHANGES

[major / 網羅性] Kiro ランタイム向けのプロンプト plugins/ndf-kiro/prompts/merged.md が「PRマージ後のクリーンアップを実行してください」のままになっています。sync-main 相当の機能（作業中ブランチへの main 取り込み）を呼び出した際にも LLM に意図が正しく伝わるよう、このプロンプトも更新してください。

[takemi-ohama]

🤖 /ndf:fix | round 2 対応サマリ

修正: 4件

重要度(再判定)	出所	指摘	対応
major	gemini (inline merged/SKILL.md:17)	sync-main 相当の呼び出しでも手順 1 の「merge されていなければ終了」で打ち切られる恐れ	修正済み。「用途の切り分け」節を追加し、見出しを ## クリーンアップの手順 に変更
major	gemini (review body)	plugins/ndf-kiro/prompts/merged.md が「PRマージ後のクリーンアップ」のままで main 取り込み意図が伝わらない	修正済み。両目的を含む内容へ更新
minor	codex (review body)	既存 Kiro 環境に .kiro/prompts/clean.md が残り続ける	修正済み。install.sh に廃止 prompt の除去を追加
minor	codex (review body)	README のプラグイン一覧に旧 Skill 数（29 / 28 / 30）が残存	修正済み。26 / 25 / 27 に更新（README 冒頭の記載と不一致だった）

commit: 1e49e01

補足:

• install.sh は Step 1 で .kiro/skills/ の失効シンボリックリンクを既に除去していましたが、Step 2 の prompt は上書きコピーのみで失効ファイルが残る非対称がありました。DEPRECATED_PROMPTS 一覧を追加して同じ扱いに揃えています。実機で .kiro/prompts/clean.md を置いた状態から install.sh --project を実行し、removed (deprecated): clean の出力と削除を確認済みです。

rejected: 1件

出所	指摘	却下理由
codex (inline cherry-pick-pr/SKILL.md:3)	branch-fix-strategy の when_to_use（qaに反映 / stagingに反映 / multi-branch fix 等）を cherry-pick-pr へ移すべき	cherry-pick-pr は disable-model-invocation: true の実行コマンド型 Skill で暗黙起動されないため、when_to_use を足しても自動選択には作用せず frontmatter を肥大させるだけです。disable-model-invocation を外すか否かの判断は、トリガ語の一意性ルールと frontmatter 総量予算と併せて Task 0-7（frontmatter 一括見直し）で全 Skill 横断に扱う設計のため、本 PR では先行変更しません。詳細は該当スレッドに返信済み（Resolve していません）

deferred: 0件

検証結果

• bash scripts/build-runtime-plugins.sh && bash scripts/build-runtime-plugins.sh --check → up to date
• python3 scripts/check-markdown-links.py --root . → Markdown local links are valid
• bash scripts/validate-runtime-plugins.sh → runtime plugin validation passed
• bash -n plugins/ndf-kiro/install.sh → syntax OK
• push 前 CI: 全 6 チェック SUCCESS（push 後の再実行は未待機）

本 PR のスコープ外（既知）

• cherry-pick-pr の frontmatter / トリガ語の見直し → Task 0-7
• バージョン番号の更新、旧名 → 新名の移行案内（ndf-policies の対応表）→ Task 0-10

#70

[takemi-ohama]

🤖 cross-review | round 3 | codex | APPROVE

追加で修正が必要な新規指摘はありません。既存指摘への対応後の削除・統合範囲、3ランタイム配布物、manifest、Kiro prompt、検証手順に追加の問題は見つかりませんでした。

[takemi-ohama]

🤖 cross-review | round 3 | gemini | REQUEST_CHANGES

• PRの目的に沿った3つのSkillの削除と統合先への手順移行、および関連ドキュメント・manifestの更新が網羅的に行われていることを確認しました。
• 唯一、既存利用者が旧コマンドを実行してエラーになった際の移行手順として、README.md のリリースノートへの記載漏れがあるため、修正をお願いします。

[takemi-ohama]

🤖 /ndf:fix | round 3 対応サマリ

修正: 0件（コード変更なし）

round 3 の codex レビューは APPROVE（新規指摘なし）、gemini レビューは README のリリースノートに関する指摘 1 件のみで、いずれも本 PR でのコード修正を要しませんでした。

rejected: 1件

重要度(再判定)	出所	指摘	却下理由
major → 本 PR 対象外	gemini (inline README.md:105)	削除・統合した 3 Skill について、README に削除の旨と merged / cherry-pick-pr への移行手順を追記すべき	指摘の必要性には同意しますが、記載先が本 PR ではありません。詳細は該当スレッドに返信済み（Resolve していません）

却下理由の要点:

1. 本 PR はバージョンを上げないため、指摘箇所の「NDF v4.20.1 の主な変更」へ書くと出荷版数とずれます。今回の統合は棚卸全体の成果として v5.0.0 で出荷する計画です。
2. リリースノートと旧名 → 新名の対応表は Task 0-10（棚卸の仕上げ）の担当範囲で、ndf-policies への対応表記載と README の版数更新が明記されています。対応 PR chore: 棚卸結果を manifest と文書へ反映し v5.0.0 へ (0-10) #76 は既に開いています。
3. ベースが main ではなく release/skill-inventory で、同じ release ブランチへ Task 0-1 〜 0-9 の PR が並行しています。各 PR が同じ節へ個別に追記すると衝突し、利用者から見た統合の全体像も断片化します。

Resolve

• plugins/ndf-shared/skills/cherry-pick-pr/SKILL.md:3（codex / round 2 に rejected として未 Resolve のまま残していたスレッド）を Resolve しました。codex が round 3 で APPROVE し、本指摘の再提起がなかったためです。

deferred: 0件

検証結果

• bash scripts/build-runtime-plugins.sh && bash scripts/build-runtime-plugins.sh --check → up to date
• python3 scripts/check-markdown-links.py --root . → Markdown local links are valid
• bash scripts/validate-runtime-plugins.sh → runtime plugin validation passed

コード変更がないため新規コミットと push はありません。ブランチ先端は 1e49e01 のままです。

本 PR のスコープ外（既知）

• cherry-pick-pr の frontmatter / トリガ語の見直し → Task 0-7（PR feat: 全 Skill の frontmatter 見直しと機械検査の追加 (0-7) #73）
• バージョン番号の更新、旧名 → 新名の移行案内、README / docs/ndf-plugin-reference.md の Skill 数の最終確定 → Task 0-10（PR chore: 棚卸結果を manifest と文書へ反映し v5.0.0 へ (0-10) #76）

[takemi-ohama]

🤖 cross-review | round 4 | codex | APPROVE

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

[takemi-ohama]

🤖 cross-review | round 4 | gemini | REQUEST_CHANGES

Skill 棚卸（0-4）による統合処理および 3 ランタイムへの展開網羅性を確認しました。スクリプトやマニフェスト、ドキュメントの参照更新等は適切に行われています。

1 点、設計上の懸念があります。知識 Skill であった branch-fix-strategy を cherry-pick-pr へ統合しましたが、後者は disable-model-invocation: true によりモデルから隠蔽されるため、利用者が自然文で質問した際にエージェントが戦略ルールを参照できなくなる退行が発生しています。この点への対応（ポリシー側への移譲や導線確保など）を検討してください。

[takemi-ohama]

🤖 /ndf:fix | round 4 対応サマリ

round 4 は codex が APPROVE（新規指摘なし）、gemini が REQUEST_CHANGES（未 Resolve スレッド 2 件の再提起）でした。2 件とも判断を確定させ、Resolve しました。

修正: 1件（プラン文書のみ、配布物の変更なし）

重要度(再判定)	出所	指摘	対応
major	gemini (inline plugins/ndf-shared/skills/cherry-pick-pr/SKILL.md:5)	知識 Skill branch-fix-strategy を disable-model-invocation: true の cherry-pick-pr へ統合したため、自然文の質問から核心ルールを参照できなくなる退行がある	指摘は妥当と判断。対応方針を確定し、Task 0-7 の変更内容へ追記（issues/ndf-development-skills/07-tasks.md）。本 PR では frontmatter / ndf-policies 自体は変更しない

commit: d6286b4

確定した方針は「cherry-pick-pr は破壊的操作を含むため明示指示専用（disable-model-invocation: true）を維持したうえで、branch-fix-strategy 由来の核心ルール（環境ブランチへの適用原則、ブランチ汚染の回避）を常時読み込まれる ndf-policies へ移す」です。frontmatter と ndf-policies の改訂は全 Skill 横断で判断する Task 0-7 の担当範囲のため、本 PR では先行せず、方針をプラン文書へ記録する形で追跡先を確保しました。

追記内容:

0-4 で cherry-pick-pr に統合した branch-fix-strategy 由来の核心ルール（環境ブランチへの適用原則、ブランチ汚染の回避）を、常時読み込まれる ndf-policies へ移す。明示指示専用の Skill に閉じ込めた知識が自然文の質問から参照できなくなる退行を防ぐ

本 PR では対応しない: 1件（Resolve 済み）

重要度(再判定)	出所	指摘	判断
major → 本 PR 対象外	gemini (inline README.md:105)	削除・統合した 3 Skill の移行手順を README のリリースノートへ追記すべき	必要性には同意するが、記載先が本 PR ではない。round 3 に続く同一論点のため判断を確定して Resolve

理由の要点:

1. 本 PR のベースは main ではなく release/skill-inventory で、本 PR はバージョンを上げません。指摘箇所の「NDF v4.20.1 の主な変更」へ書くと実際の出荷版数（v5.0.0 予定）とずれます。
2. リリースノートと旧名 → 新名の対応表は Task 0-10（issues/ndf-development-skills/07-tasks.md、対応 PR chore: 棚卸結果を manifest と文書へ反映し v5.0.0 へ (0-10) #76）の担当範囲です。
3. 同じ release へ Task 0-1 〜 0-9 の PR が並行しており、各 PR が同じ節へ追記すると衝突し、利用者から見た全体像も断片化します。
4. release PR release: Skill 棚卸 (v5.0.0) #66 の本文「非互換な変更」に、消えるコマンド一覧と移行方針を既に記載済みです。

Resolve: 2件

• README.md:105
• plugins/ndf-shared/skills/cherry-pick-pr/SKILL.md:5

deferred: 0件

検証結果

• bash scripts/build-runtime-plugins.sh && bash scripts/build-runtime-plugins.sh --check → up to date
• python3 scripts/check-markdown-links.py --root . → Markdown local links are valid
• bash scripts/validate-runtime-plugins.sh → runtime plugin validation passed
• CI: push 時点で FAILURE のチェックなし（push 後の再実行は未待機）

本 PR のスコープ外（既知・追跡先あり）

• cherry-pick-pr の frontmatter 見直しと核心ルールの ndf-policies 移譲 → Task 0-7（PR feat: 全 Skill の frontmatter 見直しと機械検査の追加 (0-7) #73）
• バージョン番号の更新、旧名 → 新名の移行案内、リリースノート → Task 0-10（PR chore: 棚卸結果を manifest と文書へ反映し v5.0.0 へ (0-10) #76）

#70

[takemi-ohama]

🤖 cross-review | round 5 | codex | APPROVE

新規に修正必須の指摘はありません。既存コメントと重複する論点は再投稿していません。

[takemi-ohama]

🤖 cross-review | round 5 | gemini | REQUEST_CHANGES

PR の目的（Skillの統合と削除）に沿って、重複コードの排除や関連リンクの更新が網羅的に行われていることを確認しました。Task 0-7 への ndf-policies 移譲の追記も意図通りであることを確認しました。

1点、merged Skill に統合された旧 clean の挙動（マージ済みブランチの整理）に関して、用途の切り分け表の導線に論理的な隙間があり、一部のユースケースで機能しなくなる問題が見つかりました。インラインの修正提案を確認してください。

[takemi-ohama]

cross-review round 5 — 修正対応サマリ

対応: 1 件（major 1）/ deferred 0 件 / rejected 0 件

レビュアー	判定	対応
codex	APPROVE（指摘なし）	—
gemini	REQUEST_CHANGES 1 件	修正済み・Resolve 済み

修正内容

[major / 論理エラー] plugins/ndf-shared/skills/merged/SKILL.md — 「マージ済みブランチの整理のみ」の導線が欠落

clean を merged へ統合した結果、PR のマージ状態に依存せず単独実行できていた「マージ済みブランチの整理」への導線が失われ、OPEN な PR がある状態ではブランチ整理だけを実行できない機能の退行が発生していました。統合前の clean Skill が git branch --merged main 起点の単独完結型だったことを確認のうえ、指摘は妥当と判断して修正しています。

• 用途の切り分け表を 3 分岐（PR マージ後のクリーンアップ / マージ済みブランチの整理のみ / main の取り込み）へ拡張
• 表直後の注記を「クリーンアップの手順」以外の 2 節に共通するルールへ一般化し、いずれも単独実行可能で PR のマージ状態に依存しないことを明記
• 「マージ済みブランチの整理」節の冒頭に、手順 1（PR のマージ確認）を前提にしない旨を追記
• bash scripts/build-runtime-plugins.sh により ndf-claude / ndf-codex / ndf-kiro の生成物へ反映

検証

チェック	結果
build-runtime-plugins.sh --check	pass（生成物同期済み）
check-markdown-links.py --root .	pass
validate-runtime-plugins.sh	pass（plugin.json / marketplace.json 含む）

修正コミット: bc46bbc

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

[takemi-ohama]

🤖 cross-review | round 6 | codex | APPROVE

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

[takemi-ohama]

🤖 cross-review | round 6 | gemini | REQUEST_CHANGES

Skill統合と削除の網羅性を確認しました。統合先の手順更新や不要ファイルの削除は概ね正しく行われていますが、branch-fix-strategy 削除に伴うコンテキスト退行の防止措置（ndf-policies へのルール移行）が本 PR から漏れており、Task 0-7 まで一時的な退行期間が生じてしまいます。

また、既存課題に起因するものですが merged のクリーンアップ手順における stash pop 先の不整合が見つかりました。

以下の指摘事項を確認し、修正をお願いします。

[takemi-ohama]

cross-review round 6 対応サマリー

• codex: APPROVE（指摘なし）
• gemini: REQUEST_CHANGES 2 件 → 2 件とも受理・修正

commit: 006ecff

1. merged/SKILL.md — stash 復元先の誤り（minor / 正確性）[受理]

手順 3 で main へ checkout した後に手順 7 で git stash pop すると、元の作業ブランチの未コミット変更が main の作業ツリーへ展開される実バグでした。

• 手順 2 で git branch --show-current により退避元のブランチ名を記録
• 手順 7 は退避元のブランチへ戻してから復元する形に変更
  • ブランチが残っている場合: git checkout <退避元> → git stash pop
  • 手順 5 / 6 で削除済みの場合（マージ後にブランチを消す通常ケース）: pop しない。stash を残したまま git stash list の該当エントリを報告し、ユーザーが復元先の作成か破棄かを判断する
• 「作業完了報告（必須）」に未復元 stash の報告項目を追加

2. ndf-policies への核心ルール移行（major / 網羅性）[受理・方針変更]

前ラウンドでは Task 0-7 の担当としていましたが、branch-fix-strategy を削除しているのは本 PR であり退行を作っているのも本 PR であるため、方針を変更して本 PR 内で移行を完了しました。

• plugins/ndf-shared/skills/ndf-policies/SKILL.md に「ブランチ運用の原則」節（5 項目・実質 10 行）を追加。user-invocable: false で常時コンテキストに載るため要点のみに絞り、description にも環境ブランチ運用の語を含めて自然文の質問から本文へ到達できるようにした
  1. 修正は feature ブランチに先に commit し cherry-pick で環境ブランチへ届ける
  2. 環境ブランチを feature ブランチに merge しない（ブランチ汚染の回避）
  3. 短命ブランチを push する前に origin/main を取り込む
  4. マージ済みブランチには push しない
  5. revert を連鎖させず最終的なあるべき状態を直接コミットする
• cherry-pick-pr は実行手順（処理フロー 1〜8）を残し、「核心ルール」節を ndf-policies 参照 + 原則↔処理フロー番号の対応表へ置き換え。関連 に ndf-policies を追加
• issues/ndf-development-skills/07-tasks.md は Task 0-7 の該当項目を削除し、Task 0-2〜0-6 の 0-4 行へ移動
• bash scripts/build-runtime-plugins.sh で 3 ランタイムの生成物へ反映

検証

• bash scripts/build-runtime-plugins.sh --check → up to date
• python3 scripts/check-markdown-links.py --root . → valid
• bash scripts/validate-runtime-plugins.sh → passed

[takemi-ohama]

🤖 cross-review | round 7 | codex | APPROVE

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

[takemi-ohama]

🤖 cross-review | round 8 | codex | REQUEST_CHANGES

README の公開 Skill 数に不整合が残っています。

[takemi-ohama]

🤖 cross-review | round 8 | gemini | APPROVE

Skill の削除・統合が漏れなく反映されており、各ランタイムの設定（plugin.json、manifests/*-skills.txt）への追従も正確です。
Kiro の install.sh においても、過去に prompts/ として配布されていた clean.md のみが適切に DEPRECATED_PROMPTS として処理されており、既存環境への配慮も含めて移行の完全性が保たれています。

修正を要する箇所はありません。

[takemi-ohama]

🤖 cross-review | round 9 | codex | APPROVE

修正必須の新規指摘はありません。生成物同期、Markdown リンク、runtime validation は成功しています。

[takemi-ohama]

🤖 cross-review | round 9 | gemini | REQUEST_CHANGES

branch-fix-strategy 由来の原則の ndf-policies への移譲や、Kiro installer における clean.md プロンプトの削除処理など、前回の指摘への対応と全体的な網羅性を確認しました。
1 点のみ、移行手順の記載漏れが残っています。