SuperAPI AI 工具集

code-review

测试与质量

查看项目

更新于 10-16 05:41

技能介绍

用途:该技能用于按 Sentry 工程实践对代码进行审查,适用于审查拉取请求、检查代码变更或提供代码质量反馈。

主要能力:审查时关注运行时错误(潜在异常、空指针、越界访问)、性能问题(无界 O(n²) 操作、N+1 查询、不必要的内存分配)、副作用(影响其他组件的非预期行为变更)、向后兼容性(无迁移路径的破坏性 API 变更)、ORM 查询(复杂 Django ORM 的意外查询性能)以及安全漏洞(注入、XSS、访问控制缺失、密钥泄露)。同时评估组件交互是否合理、变更是否符合现有架构、是否与当前需求或目标冲突。

测试覆盖:每个 PR 都应有适当测试覆盖,包括业务逻辑的功能测试、组件交互的集成测试、关键用户路径的端到端测试;需验证测试覆盖真实需求与边界情况,避免测试代码中过多分支或循环。

长期影响:涉及数据库模式修改、API 契约变更、引入新框架或库、性能关键路径、安全敏感功能时,应标记给高级工程师审查。

反馈方式:保持礼貌与共情,提供可执行的建议而非模糊批评,不确定时以提问方式表达,如“你是否考虑过……?”。仅剩小问题时即可批准,不要因风格偏好阻塞 PR;目标是降低风险,而非追求完美代码。

常见需标记的模式:Python/Django 中循环内访问关联对象导致 N+1 查询,应使用 prefetch_related;TypeScript/React 中 useEffect 缺少依赖项,应补全依赖;安全方面避免字符串拼接 SQL,应使用参数化查询。

限制:资料未提供安装方式、平台支持、收费与价格等信息。