用途:在代码合并前对变更进行结构化审查,确保代码正确、安全、可维护。适用于审查 pull request/merge request、检查待提交的 diff 或补丁、用户请求代码审查,以及完成功能后请求审查、重构前基线检查或修复复杂 bug 之后。
主要能力:按五个阶段推进。一,理解范围:确定 base/head 引用或 diff,阅读 PR 描述与关联工单,扫描改动文件并标记认证、支付、数据处理、新依赖等风险区域。二,正确性与逻辑:追踪关键路径与边界情况,检查错误处理、前置条件、空值、边界校验,以及竞态、重复提交、事务一致性。三,安全:检查 SQL/命令/HTML 注入、硬编码密钥、认证与授权、越权与敏感数据泄漏。四,规范与可维护性:命名与结构、函数与文件大小、重复逻辑、测试覆盖、文档。五,反馈:按 Critical(合并前必须修复)、Suggestion(应修复或讨论)、Nice to have(可选)分级,每条给出文件与行号、问题描述和具体修复建议。
输出结构:Summary、Critical、Suggestions、Nice to have、Checklist 五部分。核心原则是审查代码而非作者,具体指出文件行号并给出可执行建议。
限制:资料未提供安装命令、收费信息或具体平台支持;若项目有 CONTRIBUTING.md 或审查文档应优先遵循;遇到不熟悉的语言或框架时,可只覆盖能判断的阶段并注明未深入检查的区域。