---
name: pr-review-checklist
description: GitHub PR をレビューする / レビュー観点を整理する時に使う
---

# PR レビュー チェックリスト

## 必ず 4 観点で分けて出力する

### 🔒 セキュリティ
- [ ] ユーザー入力が SQL/シェル/eval に直接入っていないか
- [ ] 認証チェックが新エンドポイントに入っているか
- [ ] 秘密情報が hard-coded されていないか
- [ ] CSRF / XSS 対策

### ⚡ パフォーマンス
- [ ] N+1 クエリ
- [ ] 大きいループ内の同期 I/O
- [ ] キャッシュの活用余地
- [ ] 大規模データのメモリ消費

### 🧪 テスト
- [ ] 新規ロジックに単体テストあり
- [ ] エッジケース(空・null・境界)を網羅
- [ ] 既存テストが壊れていない

### 📖 可読性 / 設計
- [ ] 命名が意図を表しているか
- [ ] 単一責任原則
- [ ] コメントが「何を」より「なぜ」を説明しているか
- [ ] 変更が必要最小限か(scope creep していないか)

## 出力フォーマット

```markdown
# PR #<番号> Review

## ✅ 良い点
- ...

## 🔴 必須修正 (マージブロック)
- [ファイル名:行番号] ... → 提案
  ```diff
  - old_code
  + new_code
  ```

## 🟡 改善提案 (任意)
- ...

## 💬 質問
- ...

### CLAUDE.md 追記項目

```markdown
## PR レビューの社内規約
- 全 PR にユニットテスト必須(coverage > 80%)
- DB スキーマ変更は migration 付き必須
- 外部 API 呼び出しはタイムアウト/リトライ設定必須
- ログ出力は logger 経由(print 禁止)

# 一回だけセットアップ
claude /mcp install github  # GitHub MCP サーバー設定

# 各 PR で
claude
> PR #123 をレビューして

> 出力をそのまま PR にコメントとして投稿して

name: AI PR Review
on:
  pull_request:
    types: [opened, synchronize]
jobs:
  review:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - run: |
          curl -fsSL https://claude.com/install.sh | bash
          claude --print "差分をセキュリティ/パフォーマンス/テスト/可読性 の 4 観点でレビューして" > review.md
        env:
          ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
      - run: gh pr comment ${{ github.event.pull_request.number }} --body-file review.md
        env:
          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}