用途:在审查拉取请求、检查代码变更或提供代码质量反馈时使用,目标是降低风险而非追求完美代码。
主要能力:提供审查清单。识别问题方面,关注运行时错误(潜在异常、空指针、越界访问)、性能(无界 O(n²) 操作、N+1 查询、不必要的内存分配)、副作用(影响其他组件的非预期行为变更)、向后兼容(无迁移路径的破坏性 API 变更)、ORM 查询(复杂 Django ORM 的意外查询性能)、安全漏洞(注入、XSS、访问控制缺口、密钥泄露)。设计评估方面,检查组件交互是否合理、变更是否符合现有项目架构、是否与当前需求或目标冲突。
测试覆盖:每个 PR 都应有适当测试覆盖,包括业务逻辑的功能测试、组件交互的集成测试、关键用户路径的端到端测试;验证测试覆盖真实需求与边界情况,避免测试代码中过多分支或循环。
长期影响:涉及数据库模式修改、API 契约变更、引入新框架或库、性能关键路径、安全敏感功能时,标记给资深工程师审查。
反馈准则:语气礼貌且富有同理心;提供可操作建议而非模糊批评;不确定时以提问方式表达,如“你是否考虑过……?”。批准准则:仅剩小问题时批准;不要因风格偏好阻塞 PR。
限制:资料未提供安装命令、平台支持、收费方式或具体使用步骤。