SuperAPI AI 工具集

review

测试与质量

查看项目

更新于 10-26 16:20

技能介绍

用途:该技能用于对 Docling 仓库的拉取请求进行审查或复审,要求每条发现可复现,并给出明确的验证结果。适用于代码、测试、依赖、文档等变更;若只是进行文档转换而不修改仓库,应使用包使用类技能。

主要流程与能力:

  1. 固定审查范围。阅读实时的 PR 描述、差异、审查意见和 CI 结果,记录 base 与 head 提交 ID;使用隔离的工作树,保留其他改动与暂存内容。复审时逐条阅读未解决发现和作者回应。退出条件为:已知变更文件与受影响契约,被审查 head 精确,且旧发现已列出待验证。
  2. 检查行为与测试。沿受影响代码追踪输入到用户可见结果;对转换变更检查源内容、DoclingDocument 归属与顺序、受影响的导出,相关时检查嵌套与非文本内容;文本存在或条目计数本身不能证明结构正确。对缺陷修复,尽量在 base 与 head 上运行小用例,回归测试须在原缺陷上失败、在修复后通过。检查公共类型、Python 3.10 支持、默认值、错误路径和可选依赖;优先显式契约而非属性探测。检查测试断言与变更的参考数据,确认其能检测缺陷而非无理由接受新输出。对遍历文档内容的循环,检查工作量随输入规模的增长;不要用噪声性计时阈值作为正确性门槛。
  3. 运行适用检查。使用 AGENTS.md 中的命令;对自己所做的修改运行 make validate,检查钩子编辑并重复直到通过;只读审查使用 make check 与针对性测试,只读审查期间不要重新生成参考数据。在精确被审查的 head 上查看 CI 并解释影响结论的失败。check_skill_routes.py 检查贡献者技能路由与 Codex/Claude 链接;现有 Ruff、ty、Tach 与锁检查检查代码与依赖规则,但这些检查不能证明转换质量、测试充分性或完整 ASD-STE100 合规,这些需人工审查决定。记录命令、结果、被审查提交和未运行的检查;不得将跳过、待定或失败的检查描述为通过。
  4. 给出精确结论。每缺陷一条发现,包含位置、触发条件、实际与预期行为、用户影响、复现或测试证据,并链接受影响代码。严重性术语:Blocker(安全缺陷、数据丢失、错误支持行为或必需检查失败,请求修改);Suggestion(无明显契约失败的有用改进,不因偏好阻塞);Question(缺少证据,要求具体证据,不断言缺陷)。仅当无 Blocker 且相关验证完成时批准;若必需证据不可得,说明限制并暂不批准。仅在用户授权时发布审查或评论;发布前确认 PR head 仍与审查提交一致,若变化则检查新差异并重复受影响验证,避免重复发现。

适用场景:Docling 仓库 PR 的代码、测试、依赖、文档变更审查与复审。

限制:资料未提供自动化审查功能、具体工具封装、收费或价格信息;审查结论依赖人工判断与所列检查命令。