reviewing-code

Code Review Skill

Overview

このスキルは、高品質なコードレビューを実施するための専門知識を提供します。 建設的なフィードバックと具体的な改善提案を心がけます。

When to Use

• PR作成前のセルフレビュー
• コード品質のチェック
• ベストプラクティスの確認
• リファクタリング前の現状分析

Review Focus Areas

1. Architecture & Design

• レイヤー間の依存関係が適切か
• 単一責務の原則に従っているか
• 適切な抽象化レベルか
• パターンの適用が妥当か

2. Code Quality

• 可読性と保守性
• 命名規則の遵守
• コードの重複
• 複雑度（関数の長さ、ネスト深さ）

3. Performance

• 不要な計算やループ
• メモリリーク の可能性
• 非効率なデータ構造の使用
• N+1 問題

4. Security

• 入力バリデーション
• SQLインジェクション対策
• XSS対策
• 認証・認可の実装

5. Testing

• テストカバレッジ
• エッジケースの考慮
• テストの可読性
• テストの独立性

Output Format

レビュー結果は以下の形式で報告します：

## 重大度: 高 🔴
- 問題点の説明
- 影響範囲
- 推奨される対応

## 重大度: 中 🟡
- 改善提案
- より良い代替案

## 重大度: 低 🟢
- 細かい指摘
- スタイルの統一

Golden Pattern

1. 全体像の把握 - 変更の目的と影響範囲を理解
2. アーキテクチャチェック - 設計原則への適合を確認
3. 詳細レビュー - コード行単位での品質確認
4. テスト確認 - テストの妥当性と十分性を検証
5. フィードバック - 建設的で具体的な改善提案

Reference Files

• checklist.md - レビューチェックリスト
• patterns.md - よくあるパターンと対処法