严格代码审查

问题

很多代码审查停留在“看有没有明显错误”，容易漏掉与需求不符、边界条件、事务一致性、安全默认值和测试覆盖。code-review 把审查对象限定为一次分支或工作区差异，目标是输出可定位、可追责、可决定是否合并的工程判断，而不只是代码风格建议。

工作方式

技能会先确认 PRD、issue、验收标准或业务目标；没有时标记审查受限。随后选择 origin/master...HEAD 或 master...HEAD 作为比较基，列出覆盖文件与 hunk。审查重点包括：

• 正确性：业务语义、空值、类型转换、时区、精度、幂等。
• 可靠性：事务、重试、并发、批处理部分失败。
• 安全：认证、授权、注入、路径穿越、不安全反序列化、敏感日志。
• 数据：迁移兼容、重复数据、回滚风险。
• 性能：N+1 查询、无界循环、大对象加载、阻塞调用。
• 测试：正常路径、错误路径、边界和回归覆盖。

最终生成 code-review-report-YYYYMMDD-HHmm.md，按 Critical、Major、Minor、Suggestion 分级，并总结最高风险、验证状态和合并建议。

适用边界

它默认不修改产品代码，除非明确要求修复。若仓库命令不可信、测试不可安全运行，或差异过大导致上下文不足，报告应说明验证受限。涉及隐私时，需按规则脱敏密钥、内部地址、客户标识和绝对路径。