fresh-eyes

重新审视代码

以全新的视角重新阅读你刚刚编写或修改的所有代码。查找那些在埋头实现时容易忽略的明显漏洞、错误、问题和容易引起困惑之处。

适用时机

• 完成功能或修复后
• 提交更改前
• 觉得可能有哪里不对时
• 长时间编程后

流程

1. 确定更改的代码

找出本次会话中修改的所有文件。如果不确定，请询问用户或检查最近的 git 更改：

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

2. 以全新的视角重新阅读

完整阅读每个修改过的文件。假装自己从未见过这些代码，并留意以下方面：

逻辑错误

• 差一错误
• 条件判断反了
• 缺少 null/undefined 检查
• 竞态条件
• 比较错误（== 与 ===，> 与 >=）

明显漏洞

• 变量名拼写错误
• 复制粘贴错误
• 忘记写 return 语句
• 本应使用却未使用的变量
• 调用了错误的函数

缺失内容

• 错误处理不完整
• 未覆盖边界情况
• 缺少清理代码（关闭连接、清除超时）
• 缺少边界验证

容易引起困惑之处

• 容易误导的变量名
• 复杂逻辑缺少注释
• 文件内的模式不一致
• 未作说明的魔法数字

3. 修复问题

对于发现的每个问题：

1. 用 1 句话说明问题所在
2. 立即修复
3. 继续处理下一个问题

不要征求许可。直接修复明显的问题。

4. 汇报摘要

修复后，简要汇报：

## Fresh Eyes Review

Fixed 3 issues:
- `api/users.ts:47` — Missing null check on user.profile
- `api/users.ts:82` — Off-by-one in pagination (used > instead of >=)
- `utils/format.ts:15` — Typo: `formattedDte` → `formattedDate`

No other issues found.

如果没有发现问题：

## Fresh Eyes Review

Reviewed 4 files. No issues found.

不要做的事

• 不要重构可正常运行的代码
• 不要添加功能
• 不要更改风格偏好
• 不要过早优化
• 不要给显而易见的代码添加注释
• 不要重新组织文件结构

只关注漏洞、错误和问题。如果代码能正常工作且没有问题，就不要改动。

检查清单

针对每个文件，在脑海中逐项检查：

• 所有变量在使用前都已初始化？
• 所有函数都返回了应返回的内容？
• 所有循环都能正确终止？
• 所有条件都处理了两个分支？
• 所有异步操作都已等待？
• 所有错误都已捕获或传递？
• 所有资源都已清理？
• 所有边界情况都已处理（空、null、零、负数）？