59 lines
2.3 KiB
Markdown
59 lines
2.3 KiB
Markdown
|
# 30 | コードを確認する
|
|||
|
|
|
|||
|
|
---
|
|||
|
|
|
|||
|
|
## AIへの指示文（ここから下をそのままAIに貼り付けてください）
|
|||
|
|
|
|||
|
|
あなたは実運用前の最終レビュー担当です。
|
|||
|
|
以下のコードを**厳しく**確認してください。曖昧な褒め言葉は不要です。
|
|||
|
|
|
|||
|
|
### レビュー対象
|
|||
|
|
|
|||
|
|
- **目的・要件**: <!-- 00_clarify.md で整理した仕様の該当部分 -->
|
|||
|
|
- **変更したファイル**: <!-- 例: src/pages/login.tsx -->
|
|||
|
|
- **変更したコード**: <!-- 以下のいずれかを貼り付けてください
|
|||
|
|
- Claude Code / Cursor を使っている場合 → 変更したファイルの中身をそのまま貼る
|
|||
|
|
- git が使える場合 → ターミナルで「git diff」を実行して出力を貼る
|
|||
|
|
- どちらも難しい場合 → AIに書いてもらったコードをそのまま貼る -->
|
|||
|
|
- **既知の制約**: <!-- 例: Node.js 18以上、外部APIは使えない、など -->
|
|||
|
|
|
|||
|
|
---
|
|||
|
|
|
|||
|
|
### 確認してほしい観点
|
|||
|
|
|
|||
|
|
- **仕様通りか**: 要件・完了条件を満たしているか
|
|||
|
|
- **バグがないか**: 想定外の入力・エラー処理・抜け漏れ
|
|||
|
|
- **セキュリティ**: APIキーの直書き・認証の抜け・危険な入力を受け付けていないか
|
|||
|
|
- **既存機能への影響**: 他の画面や機能が壊れていないか
|
|||
|
|
- **読みやすさ**: 意図が伝わるコードになっているか
|
|||
|
|
- **テスト不足**: 確認が必要なケースが抜けていないか
|
|||
|
|
|
|||
|
|
### 出力してほしい形式
|
|||
|
|
|
|||
|
|
#### 指摘事項
|
|||
|
|
|
|||
|
|
1. [重大度] タイトル
|
|||
|
|
- 問題: （何が問題か）
|
|||
|
|
- 根拠: （どのコードが問題か）
|
|||
|
|
- 影響: （放置するとどうなるか）
|
|||
|
|
- 修正案: （どう直すか）
|
|||
|
|
|
|||
|
|
重大度は `Critical`（必ず直す） / `High`（直した方がよい） / `Medium` / `Low` の4段階。
|
|||
|
|
|
|||
|
|
#### 未確認リスク
|
|||
|
|
|
|||
|
|
- 動作確認できていない部分を列挙
|
|||
|
|
|
|||
|
|
#### 判定
|
|||
|
|
|
|||
|
|
`Approve`（マージしてよい） または `Request Changes`（修正が必要）
|
|||
|
|
を明記して、理由を1〜2文で書いてください。
|
|||
|
|
|
|||
|
|
---
|
|||
|
|
<!-- ここまでをAIに貼り付けてください。以下はあなた自身へのメモです -->
|
|||
|
|
|
|||
|
|
## 次にすること
|
|||
|
|
|
|||
|
|
- `Request Changes` の場合 → 指摘を修正してから再度レビューを依頼する
|
|||
|
|
- `Approve` の場合 → マージしてタスク完了、次のタスクへ
|