review-tasklisted

# 代码审查 审查最新实现轮次，并产出 `review.md` 或 `review-r{N}.md`。 ## 行为边界 / 关键规则 - 本技能只审查代码并写报告，不修改业务代码 - 执行本技能后，你**必须**立即更新 task.md 版本戳规则：创建或更新 `task.md` frontmatter 时，先读取 `.agents/rules/version-stamp.md`，并写入或刷新 `agent_infra_version`。 ## 常见违规借口与反驳 | 借口 | 反驳 | |------|------| | 「只改了一行，不影响功能」 | 行数不等于影响面；必须读完整 `git diff` 并定位每处改动的下游效果。 | | 「大体没问题，给个 Approved」 | 结论必须由 blocker/major/minor 计数支撑，每个问题引用文件:行号，不能凭印象放行。 | | 「测试改动看着合理，跳过细看」 | 审查测试变更前必须逐条核对 `.agents/rules/testing-discipline.md`（见步骤 4 门禁）。 | ## 第 0 步：状态核对（执行前硬约束） 在加载 workflow / skill / rules 指令之后、做任何任务状态判断或用户可见结论之前，必须先执行状态核对。指令类文件读取不算对外动作或结论。 运行以下命令，并把原文粘贴到回复正文和本轮产物的 `## 状态核对` 段： ```bash git status -s ls -la .agents/workspace/active/{task-id}/ tail .agents/workspace/active/{task-id}/task.md ``` 状态核对完成前，禁止任何关于��部状态的断言（例如“代码没变”“测试已通过”“没有其他引用”），包括思考阶段。本门禁只提供结构下限；逐条证据配对和真实性仍需按报告模板与审查要求核对。 ## 执行步骤 ### 1. 验证前置条件 要求存在： - `.agents/workspace/active/{task-id}/task.md` - 至少一个实现产物：`implementation.md` 或 `implementation-r{N}.md` ### 2. 确定审查轮次 扫描任务目录并记录： - `{review-round}` - 作为本轮产物的 `{review-artifact}`，格式为 `review.md` 或 `review-r{N}.md` ### 3. 阅读实现与修复上下文 读取最高轮次的实现产物；如存在修复产物，也读取最高轮次的修复产物。 ### 4. 执行审查 遵循 `.agents/workflows/feature-development.yaml`，并同时检查 `git diff` 获取完整变更上下文。 > 详细审查标准、严重程度划分和 reviewer 关注点见 `reference/review-criteria.md`。执行此步骤前先读取 `reference/review-criteria.md`。 > 测试审查硬门禁：当 `git diff` 触及测试文件时，必须先读取 `.agents/rules/testing-discipline.md` 并逐条核对（尤其"正向已覆盖时不应再加反向断言"）。 ### 5. 编写审查报告 创建 `.agents/workspace/active/{task-id}/