Test Coverage Review — PR #
【免费下载链接】ArchonThe first open-source harness builder for AI coding. Make AI coding deterministic and repeatable.项目地址: https://gitcode.com/GitHub_Trending/archon3/Archon
Summary
<1-2 sentences. Coverage: adequate / minor-gaps / significant-gaps.>
Findings
CRITICAL — bug fix without regression test
- file:line:
- Suggested test: <what to test, what assertion>
HIGH — new behavior without coverage
- (same format)
MEDIUM — edge cases / error paths missing
- (same format)
LOW — improvements
- (same format)
Mock isolation concerns
<bullet list of any cross-test mock pollution risks, or "None.">
Notes for synthesizer
<which aspects this overlaps with — e.g., error-handling review may have flagged the same uncovered path.>
模板有几个值得注意的设计细节: 1. **每条发现必须带 `<file:line>` 定位**,并且附上具体的测试建议(测什么、断言什么),而不是笼统地说"缺测试"; 2. **Summary 只有三档结论**:`adequate` / `minor-gaps` / `significant-gaps`,给合成节点一个可以直接消费的口径; 3. **Mock isolation concerns 单独成节**:因为仓库的 `mock.module` 进程级污染问题是一个跨文件风险,需要显式列出可能污染相邻测试文件的用例,没有风险就写 "None."; 4. **Notes for synthesizer 是跨节点协作接口**:测试覆盖缺口常常与错误处理审查撞车(例如一个未被覆盖的错误路径),在这里声明重叠,合成阶段才能去重; 5. **明确的反向要求**:"If coverage is adequate, write `## Findings\n\nAdequate coverage for the changed behavior.` and stop."——如果覆盖足够,就如实说足够并停止,**不得发明缺口**。 这个模板与 `maintainer-review-synthesize` 命令([`.archon/commands/maintainer-review-synthesize.md`](https://link.gitcode.com/i/f1279ef4a0d089817df9d4dc4e1d6711))的读取约定严格对应:合成节点会读取 `review/` 目录下所有 `*-findings.md`,按严重级别跨方面聚合、去重("缺失测试的错误路径"同时出现在 error-handling 与 test-coverage 时只保留最可操作的一条),并按文件路径排序,最终产出 `synthesis.md` 与可直接贴到 PR 上的 `review-comment.md` 草稿。 ## 五、Phase 4:RETURN——单行摘要与检查点 审查结束时返回一行状态摘要:Test-coverage review complete. CRITICAL, HIGH, MEDIUM, LOW findings. Coverage: <adequate|minor-gaps|significant-gaps>.
【免费下载链接】ArchonThe first open-source harness builder for AI coding. Make AI coding deterministic and repeatable.项目地址: https://gitcode.com/GitHub_Trending/archon3/Archon
创作声明:本文部分内容由AI辅助生成(AIGC),仅供参考