feat: v2.4.4 Phase 1 — add --apply-plan for merge-plan.json dry-run

[aliksir]

Summary

v2.4.4 Phase 1 として --apply-plan <path> フラグを追加。--merge-twice --execute が生成した merge-plan.json を読み取り、各 candidate の action ('merge' / 'skip' / 'ignore') を集計して dry-run サマリを出力する。ファイル変更は一切しない。

なぜ段階導入か

• Phase 1 (本 PR): スキーマバリデーション + dry-run サマリのみ
• Phase 2 (v2.4.5+): 実マージロジック（file append / delete / git commit）

Phase 1 で JSON スキーマとバリデーションを確定させることで、Phase 2 で実マージロジックを安全に積み上げる。破壊操作 CLI の段階導入パターン（第210回 dev-lessons 教訓 `[cli-design][destructive-ops][lesson-skill-loop]`）に従う。

変更内容

• `skill-loop.js`: +doApplyPlan / +validatePlanSchema / +--apply-plan parser
• `README.md` / `README.ja.md`: +v2.4.4 Phase 1 セクション（使用例 + バリデーション規則）
• `test/skill-loop.test.mjs`:
  • +9 件の新規テスト (T16-T24)
  • +Nit-1 対応: T14 内に `stdoutJson.candidates.length === plan.totalCandidates` assert 追加
  • +Nit-2 対応: `--json --execute --jaccard-min 0.99` 3 フラグ組合せテスト追加
• `test/fixtures/`: 新規 3 ファイル (`sample-plan-{all-merge,mixed,tbd-residual}.json`)
• `package.json`: version 2.4.3 → 2.4.4

バリデーション規則

ケース	期待挙動
`action: 'TBD'` 残存	exit 1（手動レビュー強制）
必須フィールド欠落	exit 1
不正 JSON	exit 1
ファイル不在	exit 1
不正 action 値 (e.g. 'delete')	exit 1

Test Plan

• 自動テスト 107/107 PASS（既存 95 + 新規 12）
• 手動確認: `--apply-plan sample-plan-mixed.json` → merge:1 / skip:1 / ignore:1 サマリ正常
• 手動確認: `--apply-plan --json sample-plan-mixed.json` → 構造化 JSON 出力（mode='apply-plan' / dryRun=true）
• 手動確認: TBD 残存 fixture → exit 1
• C3 [critical]: `--apply-plan` 実行で fixture mtime 不変 → 実マージ未実施保証

Refs

• 計画書: `plans/20260524_lesson-skill-loop-v2.4.4-phase1.md`
• 設計書: `designs/20260524_lesson-skill-loop-v2.4.4-phase1.md`
• v2.4.3 Findings 一括消化の継続（feat: v2.4.3 — resolve kurouto Findings (F1/F2/F3) for --execute spec #13 のフォロー）

🤖 Generated with Claude Code