code-review

Code Review

📋 実行前チェック(必須)

このスキルを使うべきか?

• PRをレビューする?
• コード品質をチェックする?
• 他者のコードを読む?
• マージ前の最終確認を行う?

前提条件

• PRの目的を理解しているか?
• 変更の影響範囲を把握しているか?
• 関連する仕様・チケットを確認したか?

禁止事項の確認

• 個人攻撃になるコメントをしようとしていないか?
• 根拠なく否定しようとしていないか?
• スタイルの好みだけで指摘しようとしていないか?
• ブロッキングイシューを見逃していないか?

トリガー

• PRレビュー時
• コード品質チェック時
• 他者のコードを読む時
• マージ前の最終確認時

🚨 鉄則

可読性最優先。書く時間より読まれる時間が長い。

レビュー順序

1. ブロッキング: セキュリティ、クラッシュ
2. シンプル: タイポ、インポート
3. 複雑: ロジック、設計

チェックポイント

設計

• 単一責任か
• 重複がないか(DRY)
• 適切な抽象化レベルか

セキュリティ

• 入力検証があるか
• 認可チェックがあるか
• 機密情報が漏れていないか

可読性

• 命名は明確か
• 複雑すぎないか
• コメントは適切か

🚫 禁止事項まとめ

• 個人攻撃になるコメント
• 根拠なき否定
• スタイル好みだけの指摘
• ブロッキングイシューの見逃し