code-review

コードレビュー

構造化された、実行可能なコードレビューを行います。

使用するタイミング

• プルリクエストや変更一式をレビューするとき
• マージ前にコード品質を評価するとき
• モジュールの技術的負債を監査するとき
• レビュー基準やチェックリストを策定するとき

レビューの観点

以下の観点を優先度順に確認します。

1. 正確性

• コードは意図したとおりに動作しますか？
• エッジケース（null、空、オーバーフロー、並行処理）に対応していますか？
• エラー処理の経路はテストされていますか？
• 変更した動作をテストでカバーしていますか？

2. セキュリティ

• システム境界で入力を検証しているか
• コードにシークレット（API キー、パスワード、トークン）が含まれていないか
• SQL インジェクションやコマンドインジェクションを防止しているか
• 認証と認可を適切に確認しているか
• 詳細な分析については security-audit スキルを参照してください

3. 可読性

• 名前（変数、関数、クラス）が明確か
• 関数が単一の処理を行っているか
• 深いネストがないか（最大 3 レベル）
• コメントで「何を」ではなく「なぜ」を説明しているか
• 周囲のコードベースとスタイルが一貫しているか

4. 保守性

• 不要な抽象化がないか
• 過度な抽象化を避けつつ DRY を実践しているか（3 回ルール）
• 依存関係に妥当性があるか
• 破壊的変更に注意が促されているか

5. パフォーマンス

• 実際に懸念がある場合（ホットパス、大量のデータ、N+1 クエリ）のみ指摘してください
• 文脈上必要でない限り、マイクロ最適化は行わないでください

レビュー結果の形式

次の形式でフィードバックを構成してください。

## Review: <PR title or file>

### Must Fix
- [ ] **file.py:42** — [Correctness] Description of the issue and suggested fix

### Should Fix
- [ ] **file.py:78** — [Readability] Description and suggestion

### Consider
- [ ] **file.py:100** — [Performance] Optional improvement

### Positive
- file.py:15 — Good use of context manager for resource cleanup

重要度レベル：

レベル	意味	マージ可否
必ず修正	バグ、セキュリティ上の問題、または契約の破損	ブロック
修正を推奨	可読性または保守性に関する重大な懸念	変更を依頼
検討事項	任意の改善、スタイル上の好み	コメント付きで承認
肯定的な点	特筆すべき優れたパターン	-

ガイドライン

1. 具体的にする - 該当する行を正確に示し、具体的な代替案を提案します
2. 理由を説明する - 単に「ここを変更してください」ではなく、「Y のため、X が発生する可能性があります」と説明します
3. スタイルと本質を分ける - スタイルはリンターで自動化し、ロジックは手動でレビューします
4. 範囲を限定する - ファイル全体を求められない限り、変更箇所をレビューします
5. 良い仕事を認める - 肯定的な所見を少なくとも 1 つ含めます
6. 押し付けず提案する - 「X の使用を検討してください」と伝えます。「必ず X を使用してください」とはしません（必ず修正が必要な場合を除く）