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

README.md

@@ -8,10 +8,10 @@ Claude Code / Codex / Kiro CLI向けのスキル・MCP設定を共有するた
 
 **NDFプラグイン v4.20.1** は、同じ `ndf@ai-plugins` という名前で Claude Code / Codex / Kiro CLI へ配布されるランタイム別プラグインです。共通ソースは `plugins/ndf-shared/` に集約し、利用者が install する配布物は `plugins/ndf-claude/` / `plugins/ndf-codex/` / `plugins/ndf-kiro/` に分かれています。
 
-- **公開Skills**: Claude Code向け core 24個、Kiro向け core 23個、Codex向け core 24個に分離。
-- **元Skills（33個）**:
- - PR/レビューワークフロー (9): pr, pr-tests, fix, review, cherry-pick-pr, deploy, sync-main, merged, clean
- - 原則・ガイドライン (10): ndf-policies, branch-fix-strategy, implementation-plan, plan-to-spec, investigation-rules, problem-solving, logging-guidelines, markdown-writing, issue-plan-strategy, ml-model-structure
+- **公開Skills**: Claude Code向け core 21個、Kiro向け core 20個、Codex向け core 21個に分離。
+- **元Skills（30個）**:
+ - PR/レビューワークフロー (7): pr, pr-tests, fix, review, cherry-pick-pr, deploy, merged
+ - 原則・ガイドライン (9): ndf-policies, implementation-plan, plan-to-spec, investigation-rules, problem-solving, logging-guidelines, markdown-writing, issue-plan-strategy, ml-model-structure
  - データ分析・品質・環境 (5): qa-security-scan, docker-container-access, google-auth, codex, official-skills-autoloader
  - E2Eテスト/Playwright (4): playwright-planning, playwright-authoring, playwright-evidence, playwright-kit-ops
  - 外部サービス連携 (1): google-drive
@@ -100,7 +100,7 @@ kiro-cli chat
 
 | プラグイン名 | バージョン | 説明 | 詳細 |
 |------------|----------|------|------|
-| **ndf** | 4.20.1 | Claude Code / Codex / Kiro CLI 向けに runtime 別配布物を提供する NDF プラグイン。8個の専門エージェント（Claude版）、公開Skills（Claude Code向け core 26個、Kiro向け core 25個、Codex向け core 27個）、Claude SessionStart/Stopフック、Codex/Kiro向け通知・実行補助を提供。v4.0.0 で Codex MCP サーバを廃止し、`/ndf:codex` skill + `corder` エージェント経由の CLI 直接実行に一本化。 | [Claude](./plugins/ndf-claude/README.md) / [Codex](./plugins/ndf-codex/README.md) / [Kiro](./plugins/ndf-kiro/README.md) |
+| **ndf** | 4.20.1 | Claude Code / Codex / Kiro CLI 向けに runtime 別配布物を提供する NDF プラグイン。8個の専門エージェント（Claude版）、公開Skills（Claude Code向け core 21個、Kiro向け core 20個、Codex向け core 21個）、Claude SessionStart/Stopフック、Codex/Kiro向け通知・実行補助を提供。v4.0.0 で Codex MCP サーバを廃止し、`/ndf:codex` skill + `corder` エージェント経由の CLI 直接実行に一本化。 | [Claude](./plugins/ndf-claude/README.md) / [Codex](./plugins/ndf-codex/README.md) / [Kiro](./plugins/ndf-kiro/README.md) |
 
 ### NDF v4.20.1 の主な変更
 

docs/ndf-plugin-reference.md

@@ -65,7 +65,7 @@ NDF の Skill 実装は `plugins/ndf-shared/skills/` が編集元です。公開
 主な Skill 領域:
 
 - PR / review workflow: `pr`, `pr-tests`, `fix`, `review`, `cross-review`
-- branch / release workflow: `deploy`, `cherry-pick-pr`, `sync-main`, `merged`, `clean`
+- branch / release workflow: `deploy`, `cherry-pick-pr`, `merged`
 - planning / documentation: `implementation-plan`, `issue-plan-strategy`, `plan-to-spec`, `markdown-writing`
 - quality / execution: `playwright-*`, `docker-container-access`
 - external services: `google-drive`

issues/ndf-development-skills/07-tasks.md

@@ -35,7 +35,7 @@
 | --- | --- |
 | 0-2 | `cross-review` が `fix` をループ内で呼ぶ。呼び出し規約を壊さない。`review` は `--branch` 引数でローカル差分レビューに切り替える |
 | 0-3 | 外部 AI 呼び出しの差分を `references/cli-codex.md` / `references/cli-gemini.md` に分離する。`cross-review` は両方を呼ぶため呼び出し箇所を更新する |
-| 0-4 | 起動 247 回の `merged` を残し、`clean` を吸収する。改名しない。`cherry-pick-pr`(16 回) に `branch-fix-strategy`(4 回) を吸収し、実行コマンド側の名前を残す |
+| 0-4 | 起動 247 回の `merged` を残し、`clean` を吸収する。改名しない。`cherry-pick-pr`(16 回) に `branch-fix-strategy`(4 回) を吸収し、実行コマンド側の名前を残す。`cherry-pick-pr` は明示指示専用のため、`branch-fix-strategy` 由来の核心ルール（環境ブランチへの適用原則、ブランチ汚染の回避）は常時読み込まれる `ndf-policies` へ移し、自然文の質問から参照できなくなる退行を防ぐ |
 | 0-5 | ブラウザ自動テストは [02-skill-inventory.md](02-skill-inventory.md) の対応表どおり 4 個へまとめる。`playwright-kit-ops` は実行環境ディレクトリとスクリプトを持つため単独で残し、`build-runtime-plugins.sh` の除外パターンが効く配置を保つ |
 | 0-6 | 削除対象は台帳で削除判定した 9 個に限り、判定は [02-skill-inventory.md](02-skill-inventory.md) の判断基準表に従う。うち `sync-main` は 0-4 で処理するため、この PR の対象は 8 個 |
 

plugins/ndf-claude/.claude-plugin/plugin.json

@@ -34,16 +34,13 @@
  "./skills/review",
  "./skills/cross-review",
  "./skills/merged",
- "./skills/clean",
  "./skills/ndf-policies",
  "./skills/markdown-writing",
  "./skills/docker-container-access",
- "./skills/branch-fix-strategy",
  "./skills/implementation-plan",
  "./skills/investigation-rules",
  "./skills/problem-solving",
  "./skills/logging-guidelines",
- "./skills/sync-main",
  "./skills/cherry-pick-pr",
  "./skills/deploy",
  "./skills/playwright-authoring",

plugins/ndf-claude/skills/cherry-pick-pr/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: cherry-pick-pr
-description: "Create cherry-pick PRs for environment branches."
+description: "Create cherry-pick PRs for environment branches and apply the same fix across multiple branches."
 argument-hint: "<base-branch> (例: qa/staging, release/v2)"
 disable-model-invocation: true
 allowed-tools:
@@ -11,7 +11,7 @@ allowed-tools:
 
 # cherry-pick PR 作成コマンド
 
-featureブランチから指定ベースブランチへ、短命ブランチ経由で cherry-pick PR を作成する。`feature → main` の PR にベースブランチ固有コードが混入するのを防ぐ。
+featureブランチから指定ベースブランチ（`qa/*`, `staging/*`, `release/*` 等の環境ブランチ）へ、短命ブランチ経由で cherry-pick PR を作成する。同じ修正を複数ブランチへ並行適用する場面全般で、この原則と手順に従う。
 
 ## 使用方法
 
@@ -24,7 +24,22 @@ featureブランチから指定ベースブランチへ、短命ブランチ経
 
 featureブランチに環境ブランチ(`qa/staging`等)を merge して conflict を解消すると、`feature → main` の PR に環境ブランチ固有のコードが混入する（main汚染）。短命ブランチ + cherry-pick で、必要なコミットだけを対象ブランチに届ける。
 
-詳細な原則は `/ndf:branch-fix-strategy` スキル参照。
+| 観点 | 正しい順序 | 誤った順序 |
+|------|-----------|-----------|
+| 単一ソース | feature ブランチが唯一の正 | 二箇所で実装 |
+| 一貫性 | cherry-pick で完全一致 | 手書き差分でズレる |
+| 追跡性 | `-x` で元 commit が明記 | 関連 commit 不明確 |
+
+## 核心ルール
+
+原則は `ndf-policies`「ブランチ運用の原則」に定義されている。本 Skill の処理フローはその原則を手順へ落としたもので、対応は次のとおり。
+
+| 原則 | 対応する処理フロー |
+|------|------------------|
+| feature に先に commit し cherry-pick で届ける | 3・6 |
+| 環境ブランチを feature に merge しない | 「なぜ必要か」 |
+| push 前に `origin/main` を取り込む | 5 |
+| マージ済みブランチには push しない | 2 |
 
 ## 処理フロー
 
@@ -110,11 +125,11 @@ git checkout <original-branch>
 
 - 短命ブランチは PR マージ後に削除してよい
 - `feature → main` の PR には影響しない
-- ベースブランチを feature ブランチに merge するのは **禁止**（main汚染の原因）
-- `-x` オプションで元commit参照を残す（追跡性）
+- revert の扱いは `ndf-policies`「ブランチ運用の原則」5 に従う
 
 ## 関連
 
-- `/ndf:branch-fix-strategy` — なぜこの手順が必要かの原則
-- `/ndf:pr` — 通常のPR作成（base=main）
+- `ndf-policies` — 環境ブランチへの適用原則とブランチ汚染の回避（本 Skill の前提）
+- `/ndf:pr` — 通常のPR作成（base=main）。非 main ベースは本 Skill に誘導される
+- `/ndf:merged` — マージ後のブランチ整理と、現ブランチへの main 取り込み
 - `/ndf:deploy` — ブランチ全体を環境へデプロイ（cherry-pickとは別用途）

plugins/ndf-claude/skills/deploy/SKILL.md

@@ -109,6 +109,5 @@ git checkout "$FEATURE_BRANCH"
 
 ## 関連
 
-- `/ndf:cherry-pick-pr` — 一部コミットだけを環境に届ける場合
-- `/ndf:branch-fix-strategy` — ブランチ運用戦略の原則
-- `/ndf:sync-main` — featureブランチに main を取り込む
+- `/ndf:cherry-pick-pr` — 一部コミットだけを環境に届ける場合とブランチ運用戦略の原則
+- `/ndf:merged` — featureブランチに main を取り込む / マージ後のブランチ整理

plugins/ndf-claude/skills/issue-plan-strategy/SKILL.md

@@ -332,7 +332,7 @@ git checkout release/<PLAN-ID>
 /ndf:cherry-pick-pr qa/staging
 ```
 
-詳細は `/ndf:cherry-pick-pr` と `/ndf:branch-fix-strategy` を参照。`feature → main` 系 PR を汚染しないため、検証ブランチ向けは必ず短命ブランチ経由で扱う。
+詳細は `/ndf:cherry-pick-pr` を参照。`feature → main` 系 PR を汚染しないため、検証ブランチ向けは必ず短命ブランチ経由で扱う。
 
 ## アンチパターン
 
@@ -345,14 +345,13 @@ git checkout release/<PLAN-ID>
 | release PR で個別 PR 範囲の指摘を解決しようとする | 該当 PR が既に閉じている場合、コミット意図がずれる |
 | release PR の body を個別 PR リンクの列挙だけにする | レビュアーは release PR 単体で変更を把握できず、個別 PR や plan を辿ることになる。body は self-contained 必須 (Step 3 / Step 8) |
 | body 最終化せずに Ready for review にする | Draft 作成時の plan ベースの暫定 body のままだと実装の最終形と乖離する |
-| 検証ブランチを feature/release に merge する | `feature → main` PR への汚染 (詳細: `/ndf:branch-fix-strategy`) |
+| 検証ブランチを feature/release に merge する | `feature → main` PR への汚染 (詳細: `/ndf:cherry-pick-pr`) |
 
 ## 関連 skill
 
 - `/ndf:implementation-plan` — plan ファイルのフォーマット (本 skill が依存)
-- `/ndf:branch-fix-strategy` — ブランチ汚染を避ける原則
 - `/ndf:pr` — 通常の PR 作成 / 更新
-- `/ndf:cherry-pick-pr` — 検証ブランチへの cherry-pick PR
+- `/ndf:cherry-pick-pr` — 検証ブランチへの cherry-pick PR とブランチ汚染を避ける原則
 - `/ndf:review` / `/ndf:cross-review` — レビュー（`--branch` で PR 前のセルフレビュー）
 - `/ndf:fix` — コメントの分類・修正・返信・Resolve
 - `/ndf:playwright-planning` — release ブランチでの E2E 結合テスト