用途:审查用户指定的代码变更,找出可操作的缺陷。适用于未提交改动、基线分支差异、提交、PR 或 MR。仅审查请求保持只读;若用户同时要求修复、开 PR 或发布结论,先完成审查再执行获授权的后续操作,该技能不授权额外外部动作。
目标选择:使用用户指定的目标;未指定时审查未提交改动;工作区干净时报告不存在未提交目标,而非默认审查最后一次提交。只审查本次变更引入或实质影响的行为,仅当变更加剧既有问题或依赖其错误假设时才提及既有问题。
流程:读取适用的 AGENTS.md 与仓库审查说明;确定审查目标并检查完整 diff;基线分支审查需解析指定基线并计算其与审查 head 的合并基点,基线本地引用落后时优先用其远程跟踪引用,特性分支自身的 upstream 不是基线;阅读足够的周边实现、测试、调用方与配置;识别预期行为、保持的不变量、外部可达路径与高风险变更;用代码、相关历史或最小诊断验证每个候选发现;发现一个问题后继续走完整个 diff;重读引用代码,只报告通过最终验证的发现。
验证:优先使用仓库说明与项目配置命令;仅在能确认或否定候选发现或实质回归时运行针对性检查;不要安装依赖、调用可能下载工具的通用包运行器,或仅因存在清单就运行大范围测试套件;先确认失败由被审查变更导致,否则作为验证限制报告。
审查重点:正确性与状态转换;授权、校验、密钥或数据暴露;并发、持久化、重试或部分失败;公共契约与兼容性;可达路径与真实输入规模下的性能;仅当变更造成具体缺陷风险或高成本耦合时才关注可维护性。把最多精力放在高影响或外部可达路径。不报告风格偏好、推测性未来担忧、有意行为变更、既有问题或已由配置工具可靠强制执行的机械问题。
发现标准:须同时满足影响正确性、安全、性能或可维护性;离散且可操作;由被审查变更引入;受影响输入、状态、序列或调用路径可演示;作者知情后大概率会修复。引用导致问题的最小变更行范围,且须与被审查 diff 重叠,在发现段落中说明具体场景。丢弃无法关联到被审查变更的推测。
审查轮次:可按需多轮聚焦,但除非用户或宿主工作流明确要求独立审查者,否则不要委派。对每个汇总发现重新对照代码验证;分歧应促使进一步检查,而非选择更高严重级。
输出:发现优先,按优先级排序,每条格式为 [P1] 祈使式标题 — path/to/file.ts:42,后接一段说明受影响场景及行为为何错误,修复方向不明显时补充。P0 为通用发布阻断或严重失败;P1 为应尽快修复的紧急缺陷;P2 为应修复的普通缺陷;P3 为仍值得修复的低影响问题。若无合格发现,写 No findings,并补充简要总体评估及重要测试缺口或残余风险。