Code Review
📋 実行前チェック(必須)
このスキルを使うべきか?
- • PRをレビューする?
- • コード品質をチェックする?
- • 他者のコードを読む?
- • マージ前の最終確認を行う?
前提条件
- • PRの目的を理解しているか?
- • 変更の影響範囲を把握しているか?
- • 関連する仕様・チケットを確認したか?
禁止事項の確認
- • 個人攻撃になるコメントをしようとしていないか?
- • 根拠なく否定しようとしていないか?
- • スタイルの好みだけで指摘しようとしていないか?
- • ブロッキングイシューを見逃していないか?
トリガー
- •PRレビュー時
- •コード品質チェック時
- •他者のコードを読む時
- •マージ前の最終確認時
🚨 鉄則
可読性最優先。書く時間より読まれる時間が長い。
レビュー順序
- •ブロッキング: セキュリティ、クラッシュ
- •シンプル: タイポ、インポート
- •複雑: ロジック、設計
チェックポイント
設計
- •単一責任か
- •重複がないか(DRY)
- •適切な抽象化レベルか
セキュリティ
- •入力検証があるか
- •認可チェックがあるか
- •機密情報が漏れていないか
可読性
- •命名は明確か
- •複雑すぎないか
- •コメントは適切か
🚫 禁止事項まとめ
- •個人攻撃になるコメント
- •根拠なき否定
- •スタイル好みだけの指摘
- •ブロッキングイシューの見逃し