> ⚠️ 本技能单独使用效果有限，建议配合完整技能集（12 步工作流）使用。安装：npx skills add Kokxi/qa-test-skills
> ⚠️ 安全警告：本技能的示例可能涉及代码变更分析（新增/修改/删除/重构）。
> 实际使用时请勿直接基于评审结论修改源码，先与开发确认变更范围和回归风险。
> 本技能仅在 workspace/ 输出评估文件，不持久化、不外传、不跨会话复用。
测试视角的代码评审
核心原则
看代码不是为了Review代码质量，而是判断变更影响范围、识别需要重点回归的区域。
> 代码读取许可：本技能需要读取代码变更（git diff）进行分析，仅关注测试影响，不评估代码质量。
深度要求（参考值）
关键指标：根据代码变更复杂度调整CR深度
| 复杂度 | CR检查点要求 | 说明 |
|---|---|---|
| 简单变更 | 5-8个检查点 | 单文件/配置/文案变更 |
| 中等变更 | 12-20个检查点 | 多文件/逻辑修改 |
| 复杂变更 | 25-40个检查点 | 重构/重写/核心模块变更 |
加载时机
| 什么时候读 | 读哪个 |
|---|---|
| 从测试角度评审代码变更时 | references/cr-lens.md |
> 测试视角CR四看的完整内容已下沉至 references/cr-lens.md，避免每次触发都占用上下文。
代码评审检查清单
功能变更检查
- [ ] 业务逻辑是否正确？
- [ ] 边界条件是否处理？
- [ ] 异常处理是否完善？
- [ ] 数据校验是否完整？
数据变更检查
- [ ] 数据库变更是否兼容？
- [ ] 数据迁移是否安全？
- [ ] 索引变更是否影响性能？
- [ ] 数据一致性是否保证？
接口变更检查
- [ ] 接口入参是否兼容？
- [ ] 接口出参是否变化？
- [ ] 错误码是否变化？
- [ ] 文档是否更新？
配置变更检查
- [ ] 配置项是否新增？
- [ ] 默认值是否合理？
- [ ] 环境差异是否考虑？
- [ ] 回滚方案是否准备？
性能变更检查
- [ ] 是否有性能影响？
- [ ] 是否有内存泄漏风险？
- [ ] 是否有并发问题？
- [ ] 是否有资源竞争？
应用场景
开发提交了登录模块的代码变更
→ CR四看分析：
- 看diff：修改了LoginController.java，新增密码加密逻辑
- 看影响：影响登录流程、密码验证、Session创建
- 看风险模式：加密算法变更可能导致兼容性问题
- 看测试建议：回归登录成功/失败流程，新增加密算法兼容性测试
配置项变更（数据库连接池大小调整）
→ 影响范围评估：确定需要回归的接口和场景
自检清单
代码评审完成后检查：
- [ ] 是否分析了diff内容？
- [ ] 是否评估了影响范围？
- [ ] 是否识别了风险模式？
- [ ] 是否给出了测试建议？
- [ ] 建议是否可执行？
检查清单
- [ ] 变更类型是否识别？
- [ ] 影响范围是否分析？
- [ ] 高风险模式是否标注？
- [ ] 回归范围是否建议？
- [ ] 可测试性问题是否提出？
