piv-fix-review-findings について
コードレビューの指摘を修正、延期、手動確認、対応不要に分類してから処理するスキルです。すべての指摘をそのまま作業指示にせず、PR の範囲を管理したい開発者向けです。
主な機能
- 01レビュー指摘の事前分類
- 02範囲に応じた修正・延期の判断
- 03個別修正とテストの追加
- 04piv-validate による検証
- 05PR の変更をコミットしてプッシュ
活用シーン
- 01現在の PR で扱う指摘の選別
- 02延期する指摘の課題登録
- 03指摘を修正してテストで確認
インストール
コマンドをコピーしてターミナルで実行します。
npx skills add https:/インストール後、Claude Code で/piv-fix-review-findingsを入力して読み込まれたか確認します。
詳細
レビュー指摘を修正する
レビューで指摘が挙がりました。ただし、レビューは入力であり、作業指示ではありません。 それぞれにどう対応するかは、あなたが決めます。
コードレビュー（問題のファイルまたは説明）：$review
方針／スコープ（今修正するものと後回しにするもの）：$scope
コードレビューがファイルの場合は、トリアージする前にファイル全体を読んで、すべての指摘を把握してください。
1. まずトリアージする（人間が判断）
コードに手を入れる前に、指摘を分類します。スコープ引数で示された方針に従ってください。方針が不明確な場合は、指摘をグループ分けして示し、すべてを既定で修正するのではなく、確認してください。
- 今修正する（この PR） — 実際の問題で、スコープ内にあり、この変更に含めるべきものです。
- 後回し／課題として記録する — 実際の問題ですが、対応は後日です。この PR を肥大化させないでください。ここで修正する代わりに、トラッカーに課題を作成する（または記録する）ようにしてください。
- 人による確認／手動テストが必要 — 信頼する前に、手作業で確認またはテストする必要があるものです。自動で修正して黙って済ませず、指摘してください。
- ノイズ／修正しない — 理由を説明してから、対応対象から外してください。
レビュー担当者にスコープを決めさせないでください。「実際の問題だが、対応は後日」は妥当でよくある判断です。規模の小さい PR のほうが、範囲が広がった PR より優れています。
2. 「今修正する」指摘を1つずつ修正する
それぞれについて、次の手順で進めます。
- 何が問題だったかを説明します。
- 修正します。
- 修正を証明するテストを作成して実行します。
3. 検証する
修正を仕上げるために piv-validate スキルを実行します。
4. PR で作業している場合 — コミットしてプッシュする
これらの修正が PR ブランチ上にある場合は、PR に修正を反映し、更新後の PR でレビューを再実行できるよう、コミットして（piv-commit を使用）プッシュしてください。 何も修正していない（すべて後回しにした）場合は、プッシュするものはありません。後回しにした項目が課題として記録されていることだけ確認してください。
出力
簡潔なレポートとして、修正した内容（テスト内容を含む）、後回しにして記録した内容（課題参照を含む）、手動確認／テストが必要な内容を示してください。PR で作業している場合は、プッシュしたコミットと、PR が更新されたことの確認も記載してください。