コードレビュー

コードの品質、セキュリティ、テスト、パフォーマンス、アーキテクチャの観点から包括的なコードレビューを実施します。

レビュー優先順位

🔴 CRITICAL（マージブロック）

• セキュリティ: 脆弱性、シークレットの露出、認証/認可の問題
• 正確性: ロジックエラー、データ破損リスク、競合状態
• 破壊的変更: バージョニングなしの API 契約変更
• データ損失: データ損失・破損のリスク

🟡 IMPORTANT（要議論）

• コード品質: SOLID 原則の重大な違反、過度な重複
• テストカバレッジ: クリティカルパスや新機能のテスト欠落
• パフォーマンス: 明らかなボトルネック（N+1 クエリ、メモリリーク）
• アーキテクチャ: 確立されたパターンからの大幅な逸脱