SuperAPI AI 工具集

code-review

测试与质量

查看项目

更新于 10-16 09:49

技能介绍

用途:对 OpenVINO 提议的差异提供简洁、基于证据的评审,只审查变更代码及直接受影响的邻近代码,不对无关遗留代码做广泛审计。适用于评审拉取请求或提议的 diff。

评审流程:阅读 diff、提交或 PR 说明及相关上下文,识别受影响组件、公共 API、运行时路径、输入边界与预期测试,加载相关组件参考;先查功能回归,再查输入安全、性能、兼容性、测试覆盖与可维护性。报告问题前须确认问题确实存在于源码中,区分源码内容与 diff 标记、行号等评审标注,依据已配置行为或文档需求验证失败;证据不足则省略该发现。

条件模式:回退型 PR 按回归评审处理,识别原 PR 及所解决的问题,检查回退是否重新引入该问题或产生新兼容性问题,不为恢复代码要求新测试或重构,除非为防具体回归所必需;无法识别原改动及理由时,应说明为验证缺口而非猜测。外部贡献 PR 应基于证据,聚焦阻塞合并或重大风险行为,优先检查必需 CI 结果、可构建性、变更路径风险与提交正确性,确认提交在 PR 描述范围内,不因外部作者推断质量较低。

优先级检查:正确性与兼容性(形状、元素类型、属性、动态行为、状态、错误路径、跨设备行为,公共 API 与序列化模型的向后兼容,生成代码、绑定、文档与注册一致性);安全性(将模型元数据、文件内容与用户输入视为不可信,检查边界、溢出、符号转换、收窄转换、所有权、生命周期与资源清理,要求不安全运算或内存访问前有校验,优先快速失败);性能(避免不必要的张量或容器拷贝、热路径分配、重复不变量计算、同步变化与内存复用改变,声称性能回归需有证据);测试与验证(缺陷修复与行为变更应有回归覆盖,已有等价覆盖时不要求新测试)。组件参考按路径加载,覆盖核心与推理、变换、插件、前端、绑定与公共 API、CI 构建与依赖等。

输出:按实际影响降序列出所有重要、高置信、可操作的发现,每条含确切变更位置、失败模式及对用户/构建/运行时的影响、最小具体修复方向;保持一条发现对应一个问题,重复根因用一处代表位置;省略推测问题、工具强制的风格偏好与无关重构请求;区分已确认问题与疑问、假设及验证缺口;不包含密钥或敏感数据。若无问题,应明确说明并总结已审查区域、已运行测试或检查及仍不可用的验证。