Fresh Eyes レビュー

先ほど書いたまたは変更したコードをフレッシュな視点で読み直します。実装に没頭しているときに見落としやすい明らかなバグ、エラー、問題を探します。

使用するタイミング

• 機能の実装が完了した後
• 変更をコミットする前
• 何か違和感を感じたとき
• 長時間のコーディングセッションの後

プロセス

1. 変更されたコードの特定

このセッションで変更したすべてのファイルを見つけます。不明な場合はユーザーに確認するか、最近のgit変更を確認してください：

git diff --name-only HEAD~1
git diff --name-only --cached

2. フレッシュな視点で読み直す

変更されたファイルを完全に読み直します。このコードを初めて見たつもりで読みます。以下の点に注目してください：

ロジックエラー

• オフバイワンエラー
• 条件式が反転している
• null/undefined チェックの漏れ
• レースコンディション
• 誤った比較演算子（== vs ===、> vs >=）

明らかなバグ

• 変数名のタイプミス
• コピペエラー
• 忘れた return 文
• 使用すべき未使用変数
• 間違った関数の呼び出し

不足している部分

• エラーハンドリングの漏れ
• 未対応のエッジケース
• クリーンアップコードの欠落（接続を閉じる、タイムアウトをクリアする）
• 境界での検証の不足

混乱のリスク

• 誤解を招く変数名
• コメントのない複雑なロジック
• ファイル内で一貫性のないパターン
• 説明のないマジックナンバー

3. 問題を修正

見つかった問題ごとに：

1. 何が問題なのかを1文で説明する
2. すぐに修正する
3. 次の問題に移る

許可を求めずに、明らかな問題を修正してください。

4. サマリーをレポート

修正後、簡潔なサマリーを提供してください：

## Fresh Eyes レビュー

3つの問題を修正：
- `api/users.ts:47` — user.profile の null チェックが不足
- `api/users.ts:82` — ページネーション内のオフバイワン（> を使用すべき場所で >= を使用）
- `utils/format.ts:15` — タイプミス：`formattedDte` → `formattedDate`

その他の問題はなし。

問題が見つからない場合：

## Fresh Eyes レビュー

4つのファイルをレビューしました。問題はありません。

してはいけないこと

• 動作するコードをリファクタリングしない
• 機能を追加しない
• スタイルの好みを変えない
• 時期尚早な最適化をしない
• 明らかなコードにコメントを追加しない
• ファイル構造を再編成しない

バグ、エラー、問題のみに焦点を当てます。動作していて壊れていなければ、触らないでください。

チェックリスト

ファイルごとにメンタルに走り抜けてください：

• すべての変数は使用前に初期化されている？
• すべての関数は適切な値を返している？
• すべてのループは正しく終了する？
• すべての条件式は両方のブランチを処理している？
• すべての非同期操作は await されている？
• すべてのエラーは catch またはスローされている？
• すべてのリソースはクリーンアップされている？
• すべてのエッジケースは処理されている（空、null、ゼロ、負の値）？