review-code

# 代码审查 审查最新代码轮次，并产出 `review-code.md` 或 `review-code-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 门禁）。 | | 「记得就是这一行，不用查」 | 行号会漂移；下结论前必须用 rg/nl 复核 `file:line`，不能复现的判断不要写成 blocker。 | ## 第 0 步：状态核对（执行前硬约束） 在加载 workflow / skill / rules 指令之后、做任何任务状态判断或用户可见结论之前，必须先执行状态核对。指令类文件读取不算对外动作或结论。 运行以下命令，并把原文粘贴到回复正文和本轮产物的 `## 状态核对` 段： ```bash agent-infra-internal task-snapshot {task-id} --format text ``` 状态核对完成前，禁止任何关于外部状态的断言（例如“代码没变”“测试已通过”“没有其他引用”），包括思考阶段。本门禁只提供结构下限；逐条证据配对和真实性仍需按报告模板与审查要求核对。 ## 任务上下文解析 > 入口允许省略 task ref，也接受旧位置 task ref 或 `--task <ref>` / `-t <ref>`。先从完整参数中分离 task scope 并原样保留其他业务操作数，再调用 `agent-infra-internal task-context resolve {task-scope}`；`{task-scope}` 为空、位置 ref 或 task flag 之一。只读取结构化结果的 `taskId`，后续把 `{task-id}` 绑定为该完整 `TASK-YYYYMMDD-HHMMSS`。解析失败时透传非零退出码，不自行扫描任务。 > 解析任务引用，并确认任务位于本技能支持的状态或目录且存在 `task.md`；无法定位时按未找到任务处理并停止。 ## 步骤开始：声明 started 事件 确认前置条件和产物上下文后、本轮第一个产出动作之前执行 `agent-infra-internal task-event {task-id} review-code.started --agent {agent}`。 ## 执行步骤 ### 1. 验证前置条件 要求存在： - `.agents/workspace/active/{task-id}/task.md` - 至少一个实现产物：`code.md` 或 `code-r{N}.md` ### 2....