qa-code-review-for-test

从测试视角评审代码变更（git diff），识别变更类型与影响范围、高危风险模式和测试遗漏点，给出最小回归测试范围建议，并按约定结构输出评估文件。仅关注测试影响，不评估代码质量，不直接修改源码。

kokxi

热度 59.48

最近更新 2026-09-29 12:27:49

分类：开发依赖：代码执行 / 读写文件 / 内部调用大模型应用场景：代码审查 / 报告撰写

安装技能

复制下面的话术发给你的 Agent，它会通过 secure-skills 检索并安装该技能。

执行命令安装此 skill： npx secure-skills-cli add qa-code-review-for-test

Metadata

displayName: 测试视角代码评审
name: qa-code-review-for-test
description: 当开发提了 PR、代码变更需要确定测试范围、或者想通过分析代码来预测可能出 Bug 的区域时使用此技能。从测试视角分析代码变更的影响范围、识别高危模式和典型风险区域。不要看完整代码逻辑——你只需要关注变更类型（新增/修改/删除/重构）、影响范围（接口定义/数据库字段/业务逻辑）和相关依赖，据此确定最小回归测试范围。输出代码变更影响分析报告。触发场景：代码评审、CR、测试视角、看代码、代码变更、Diff、代码变更后需要确定测试范围时。 Use when the user asks about: reviewing a PR or code diff from a testing perspective to determine regression scope and high-risk areas.
license: MIT
allowed-tools: Read Grep Glob Bash
metadata: {"slug":"qa-code-review-for-test","display-name":"测试视角代码评审","version":"1.8.0","when-to-use":"用户说\"代码评审\"、\"CR\"、\"测试视角\"、\"看代码\"、\"代码变更\"、\"Diff\"、需要从测试角度分析代码变更、代码变更后需要确定测试范围时","related-skills":"{\"upstream\":[\"qa-boundary-deep-dive\",\"qa-risk-intuition\"],\"downstream\":[\"qa-execution-observation\",\"qa-test-strategy-design\",\"qa-regression-testing\"]}","references":"[\"references/cr-lens.md\"]","input-format":"{\"required\":[{\"name\":\"代码变更\",\"type\":\"string\",\"description\":\"代码变更的diff或描述\"},{\"name\":\"测试用例\",\"type\":\"array\",\"description\":\"相关测试用例列表\"}],\"optional\":[{\"name\":\"需求文档\",\"type\":\"string\",\"description\":\"相关功能需求\"}]}","output-format":"{\"traceability\":[\"每次代码评审带唯一ID（CR-XXXX）\",\"关联变更ID或需求ID\"],\"structure\":[\"覆盖率：标注口径（基于现有需求/输入文档），禁止\\\"全覆盖/100%\\\"绝对化表述；缺失模块标注\\\"未覆盖+原因\\\"\",{\"review_findings\":\"代码评审发现\"},{\"test_gaps\":\"测试遗漏点\"},{\"impact_analysis\":\"变更影响分析\"},{\"high_risk_patterns\":\"高危模式识别\"},{\"regression_scope\":\"回归测试范围建议\"}]}","error-recovery-guidance":"{\"on_failure\":\"代码变更分析遗漏依赖时回退到边界分析补充\",\"retry_behavior\":\"补全依赖分析后重新确定回归范围\"}","categories":"[\"Development\",\"Team\"]","depth-requirement":"{\"reference_value\":\"根据代码变更量调整评审深度：简单×1/中等×2/复杂×3\",\"minimum\":\"至少分析变更类型、影响范围、回归范围3个维度\"}"}

⚠️ 本技能单独使用效果有限，建议配合完整技能集（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内容？
是否评估了影响范围？
是否识别了风险模式？
是否给出了测试建议？
建议是否可执行？

检查清单

变更类型是否识别？
影响范围是否分析？
高风险模式是否标注？
回归范围是否建议？
可测试性问题是否提出？