review

name	review
description	変更差分を品質軸でレビューする。実装が一段落した時、PR を出す前、コミットする前に使う。自分の差分にも他者の差分にも使える。

Review

差分に対して、重要度を付けてレビューを行い、指摘する

Use when

• 実装が一段落し、PR/コミットの前
• 他者の差分をレビューするとき

Procedure

1. 変更点の全体像を把握してから読み始める。
2. 4つの軸で見る。
   • 正しさ: 仕様どおりか、エッジケースとエラー処理は妥当か
   • 設計: 命名・責務・重複、既存パターンとの整合
   • 安全性: 入力検証、機密の漏れ、権限の扱い
   • テスト: 新しい挙動に対応するテストがあるか
3. 重要度を付ける。 blocker（直さないと出せない）／ 改善提案 ／ nit に分ける。
4. 直すべき点は具体的に。 場所と修正方針をセットで示す。良い点も1つ挙げる。

Hand off

• blocker があれば implement へ戻す。

Done when

• 4つの軸すべてに目を通した
• 各指摘に重要度が付いている
• blocker には修正方針が添えてある