--- name: comet-verify description: '验证 Classic change 并记录结果。在用户调用 /comet-verify,或 Classic Runtime 进入 Verify 时使用。' --- # Comet 阶段 4:验证(Verify) 收到入口返回的 layout 后,按 `comet-classic/reference/classic-layout.md` 确定各逻辑根对应的目录。当前上下文已有这份协议时,无需重复加载。本文件中的 OpenSpec CLI 调用均通过适配器执行,文件路径均基于已绑定的 `` 根目录,无需先额外运行 root show。 ## 前置条件 - 代码已提交(阶段 3 完成) - tasks.md 全部任务已完成 ## 步骤 ### 0a. 设置输出语言 验证报告使用本轮入口返回的 configuration.language,不再单独查询语言字段。 ### 0b. 入口状态验证(Entry Check) 按 `comet-classic/reference/scripts.md` 使用正式支持的 `comet` CLI,执行以下入口验证。从任意入口恢复任务时,先按 `comet-classic/reference/context-recovery.md` 检查恢复状态: ```bash comet state select comet state check verify --json ``` 上一阶段 guard 已成功返回本阶段状态信息时,直接使用其中的状态与 `agent.continuation` 继续,不重复 select/check;恢复任务、工作区变化或外部状态变化时,才执行上述入口验证。根据入口返回的 layout、configuration、nextAction、任务信息和协作进度摘要继续。已有检查结果和集成审查仍然有效时,只补未完成的工作,不重新执行整个阶段。丢失上下文后恢复任务,需要完整记录时,使用 --recover --details --json。验证失败时,处理返回的具体原因。 若上述 `select` / `check` 输出 `BLOCKED` 或分支绑定 `ERROR`,且原因是 `bound_branch` 与当前分支不一致,立即按 `comet-classic/reference/decision-point.md` 暂停,让用户单选:切回绑定分支后重新运行入口验证,或在用户明确确认当前分支应接管该 change 后运行 `comet state rebind ` 并重新入口验证。不得自行切换分支,不得自行换绑。 **按已记录结果继续**:`verify_result` 已为 `pass` 时,进入 archive;在归档提交和最终分支处理完成前,`branch_status` 保持 `pending`。`verify_result` 为 `pending` 时,先核对已有报告和检查结果,再从未完成的检查继续。 丢失上下文后恢复任务时,按 `evidence.scopes` 处理:标记为 `revalidated` 的本地结果可以复用;标记为 `rerun-required` 的部分需要重新检查。已经完成且仍然有效的需求分析与审查,不重复执行。外部检查不能假定具有幂等性,也不能假定其环境一直不变。 ### 1. 改动规模评估 执行规模评估: ```bash comet state scale ``` 脚本统计任务数、增量规格数和变更文件数,只返回 light/full 建议,不修改 `verify_mode`。使用 `--json` 读取 `data.recommendation`、`data.selected` 和 `data.metrics`。已有验证模式时保留;尚未选择时,根据风险确定模式,再用 `comet state set verify_mode ` 明确记录。满足以下任一条件时,规模评估建议 full:任务数 > 3、delta spec 能力数 > 1、变更文件数 > 8。 `comet state scale` 会自行从 plan 的 `base-ref` 解析提交基线,并在 plan 不可用时回退到状态中的 `base_ref`;Verify 不再重复读取 plan frontmatter 或手工拼接第二套规模评估。 验证开始前,按 `comet-classic/reference/dirty-worktree.md` 协议检查并处理未提交改动。verify 阶段的特殊处理: 1. 未提交改动明确属于当前 change 时,将这些改动纳入本次验证;继续验证,但不在 verify 阶段修改或提交实现、测试、tasks、delta spec 或 Design Doc。纯文档编辑(comet-build 定义的中性文档集:仓库根目录的 Markdown 与 `LICENSE`/`NOTICE`/`AUTHORS`,以及 `docs/`、`doc/`、`documentation/`、`.github/` 下的 Markdown/`.txt`/`.rst`,OpenSpec 与 Superpowers 产物除外)可以在 verify 阶段正常完成——它们不是实现写入,默认不作废检查证据,也不需要 `verify-fail` 2. 未提交改动只是 verify 本阶段的产物时,例如验证报告草稿,可以继续在 verify 阶段完成并记录状态 3. 未提交改动表明代码已经实现、但 tasks.md 尚未勾选时,说明 build 的任务记录落后于实现;直接运行 `verify-fail` 返回 build,核对已有结果并更新任务状态,不得询问是否接受未完成任务 4. 无法确认未提交改动的归属,或改动属于其他 change 时,按 dirty-worktree 协议说明为何需要停止;不要在尚未确定归属时让用户选择“继续/忽略” 需要回到 build 修复或补齐状态时运行: ```bash comet state transition verify-fail ``` **调整验证模式**:如 Agent 或用户认为自动评估结果不合适,可随时通过 `comet state set verify_mode ` 修改验证模式。 不能因为改动小就省略风险检查。涉及认证授权、数据迁移、并发、公共 API 或跨模块接口约定时,必须验证对应的风险场景;轻量验证无法覆盖时,改用 full。 ### 1b. 验证失败自动修复与例外决策 验证失败时,读取最新入口返回的连续失败次数和 nextAction;只有字段缺失时,才查询 `comet state get verify_failures`,不能把缺失当作零。前 3 次可修复的失败自动回到 build:先报告失败项,再运行 `comet state transition verify-fail`,然后调用 `/comet-build`,只补未完成的实现、检查、审查或任务勾选,不重复已经完成的实施。 报告必须列出: - 失败项 - 是否属于 CRITICAL 或 IMPORTANT(构建失败、测试失败、安全问题、核心验收场景失败、简化代码审查发现的正确性/安全/边界问题) - 推荐处理方式 **无法确定严重程度时**,使用较低级别。仅对构建失败、测试失败、安全问题使用 CRITICAL;明确影响核心验收或正确性的项使用 IMPORTANT;模糊或不确定的问题标为 WARNING 或 SUGGESTION。 按以下方式处理: - **CRITICAL/IMPORTANT 或范围内可明确修复的问题**:未达到上限时,自动回到 build 修复;不额外询问“是否修复”,也不允许接受偏差 - **WARNING/SUGGESTION 且修复会引入行为、范围或风险取舍**:按 `comet-classic/reference/decision-point.md` 让用户选择修复或接受偏差;接受时必须在验证报告中记录原因和影响范围 - **WARNING/SUGGESTION 且修复安全、范围局部、不涉及取舍**:未达到上限时自动修复,不因级别较低而强制停顿 只有接受 WARNING/SUGGESTION 偏差或第 4 次失败后的策略选择才是用户决策点。当前 `verify_failures >= 3` 时不得自动执行下一次 `verify-fail`;按协议只提供「继续修复」或「停止当前 workflow 并寻求外部决策」两个选项。用户选择继续后才记录下一次失败并回到 build。CRITICAL/IMPORTANT 始终不可豁免。 ### 2. 读取验证所需的产物 验证需要读取 OpenSpec 产物时,使用入口已提供的 handoff 状态;入口未提供当前 hash 核对结果时才执行: ```bash comet handoff --hash-only ``` - 读取 stderr 中的 `[HANDOFF] status:` 结论并照此执行(stdout 的裸 hash 供脚本调用方使用,不要自行比对): - `FRESH`:复用上下文已有的产物内容,只补读当前验收点仍缺的章节,tasks 仍须核对勾选。 - `STALE (changed: )`:先完整读取列出的变更产物,再按其内容验收;runtime 的 NEXT 若给出 `comet handoff design --write`,先执行它再读取。 - 不带清单的 `STALE`:记录的 handoff 缺失或早于当前产物,正常读取所有所需文件全文。 FRESH 不代表当前上下文仍保留该内容。丢失上下文后恢复任务、摘要被截断,或无法确认此前已读取内容时,应重新读取对应源文件;不能用 handoff 摘要代替尚未读取的验收条款。 autonomous 直接按本 Skill 执行实际检查并记录结果,不强制加载外部验证 Skill;其他策略使用 Skill 工具加载 Superpowers `verification-before-completion`。任何策略都不能仅凭自评宣布验证通过。 Verify 负责整个 change 的唯一最终集成代码审查。Build 只保留任务级或分段审查;在按 `verify_mode` 分支执行前,先对包含 Build 审查修复在内的最终 diff 执行一次集成审查: - `review_mode: off`:跳过自动代码审查,并在验证报告中记录原因 - `review_mode: standard|thorough`:安排独立审查者(reviewer)审查整个 change,核对需求、实际代码差异、检查结果和修复情况,重点检查正确性、安全和边界条件。autonomous 无需外部审查 Skill,其他策略加载一次 requesting-code-review。已有审查覆盖当前最终 diff 且仍然有效时,直接复用;输入变化后只补查受影响的部分,不无条件重做整轮。无法进行独立审查时停止,不能用实现代理(implementer)的自评代替 集成审查发现 CRITICAL/IMPORTANT 问题时按 Step 1b 返回 Build;非 CRITICAL 偏差按 Step 1b 的取舍规则处理。然后按 `verify_mode` 分支执行: ### 2a. 轻量验证(小改动) 按以下 7 项进行检查: 1. tasks.md 全部任务已完成 `[x]` 2. 改动文件与 tasks.md 描述一致(`git diff --stat` / `git diff --cached --stat` / `git diff --stat ...HEAD` 对照 tasks 内容);中性文档集内的纯文档编辑在对照结果旁说明即可,不作为实现不一致处理 3. 编译通过(复用 Runtime 判定仍有效的 Build 证据;失效时重跑) 4. 相关测试通过 5. 无明显安全问题(无硬编码密钥、无新增 unsafe 操作) 6. 最终集成代码审查已通过,或非 full autonomous 的 `review_mode: off` 跳过原因已记录;full autonomous 不允许跳过独立审查 7. 核心成功场景、关键失败和边界场景,以及本次涉及的高风险功能要求均已验证通过;小改动也不可省略 复用构建时,用 Build 相同的 cwd、程序和参数再次调用 `comet check run build --local -- [args...]`;Runtime 返回 `reused=true` 才算复用,输入或环境已变化时会真正重跑。证据的 cwd 必须等于之后调用 guard 的目录(通常是项目根);子目录构建用 `npm --prefix run build` 这类从根可执行的形式记录,或在 `.comet/check-policy.json`(v2)中声明该命令的 cwd 后用 `--cwd ` 记录,否则守卫会拒绝并说明原因。不得仅凭旧对话中的“构建通过”跳过检查。证据按“输入面”判定:默认输入是工作区文件内容,commit、暂存、勾选任务和环境变量变化默认不作废证据;guard 报告证据失效时会给出原因和变化文件清单,按清单处理即可。Build 阶段留下的 `--incremental` 增量证据可用于预览确认,`--apply` 前必须用完整命令重新取得 full 证据。 light/full 均必须通过 Runtime 执行真实验证命令。先记录将要写入的报告路径,避免将报告的修改也算作验证输入的变化;测试和验收检查完成后再填写结果: ```bash comet state set verification_report docs/superpowers/reports/YYYY-MM-DD--verify.md comet check run verify --local -- [args...] ``` 只有确定性的本地检查才使用 `--local`。外部服务检查省略该参数,其结果只能用于一次成功的阶段转换:Guard 预览不会用掉该结果,`--apply` 会重新核对,并在转换成功后将其标记为不可再次使用。丢失上下文后恢复任务,或输入、环境发生变化时,仍由 Runtime 判断哪些检查需要重跑。 Windows 普通 npm/pnpm shim 由平台适配器处理,包含 shell 元字符的 batch 参数会被拒绝。多条必要命令应通过项目已有验证入口统一执行;任何一条失败,入口都必须返回失败,不能用最后一条命令成功掩盖之前的失败。手工 `record-check` 只保存声明,不能据此自动推进阶段,还会遮蔽之前有效的 Runtime 证据并要求重新 `comet check run`。不同检查要求的证据彼此独立;verify 与 build 的检查结果彼此独立,不能互相替代。只有 argv、cwd、输入、环境和语义完全相同的本地 full 命令才能由 Runtime 在 Build 与 Verify 之间复用。`COMET_SKIP_BUILD=1` 不能作为可核实的检查记录。需要查看日志时,按 `logRef` 读取。首次实际执行就通过 `comet check run` 完成;失败时修复后使用 `comet check rerun` 精确重试,成功后立即运行 guard,中间不插入任何 `comet state` 写入;提交放在 guard 通过之后。 集成代码审查的输入限定为本次改动 diff、tasks.md 和必要测试结果;它不替代 spec 覆盖率、Design Doc 一致性或漂移检查。`review_mode: off` 只跳过自动 code review,不跳过构建、测试、安全检查或异常调试协议。 **通过标准**:7 项全部 OK,无 CRITICAL 或 IMPORTANT 问题。 **不通过时**:报告失败项并按 Step 1b 分类。未达到自动修复上限且问题必须或适合修复时,直接执行以下命令回到 build 阶段,然后调用 `/comet-build`: ```bash comet state transition verify-fail ``` **报告格式**:简表列出 7 项检查结果、证据引用及 PASS/FAIL。 **跳过项**(不在轻量验证中检查): - 逐条统计全部规格场景的覆盖情况(核心和高风险场景仍必查) - 深入比对实现与 Design Doc 的一致性 - 不影响正确性、安全或边界条件的代码写法一致性建议 - 检查 delta spec 与 Design Doc 是否已不一致 ### 2b. 完整验证(大改动) 当规模评估结果为"大"时: **立即执行:** 使用 Skill 工具加载 `openspec-verify-change` 技能。禁止跳过此步骤。 **外部 OpenSpec Skill 适配规则:** 加载后只采用其验证方法。直接运行官方 CLI、采用固定 cwd 或读写固定 OpenSpec 目录的指令,都必须改为通过 `comet classic openspec -- ` 执行,并使用路径解析器返回的 `` 逻辑根目录。 技能加载后,按其指引验证。检查项: 1. tasks.md 全部任务已完成(`[x]`) 2. 实现符合 `/design.md` 高层设计决策 3. 实现符合 Design Doc(`docs/superpowers/specs/` 下的技术设计文档) 4. 能力规格场景全部通过 5. proposal.md 目标已满足 6. delta spec 与 design doc 无矛盾(若 Build 阶段有增量修改 spec,检查 design doc 是否有对应记录) 7. `docs/superpowers/specs/` 关联的设计文档可定位(文件存在且与当前 change 相关) 验证不通过时:报告缺失项并按 Step 1b 分类。未达到自动修复上限且缺失项可在当前 change 内补齐时,直接执行以下命令回到 build 阶段,然后调用 `/comet-build`: ```bash comet state transition verify-fail ``` **Spec 漂移处理**(用户决策点): - 若检查项 6 发现矛盾(delta spec 有内容但 design doc 未体现),**必须以单选题形式暂停、展示处理方式并等待用户选择**,不得自动选择。选项: - 选项 A:在 design doc 追加 "Implementation Divergence" 节记录偏差原因。选项 A 属于 verify 阶段允许产物;写入后不得因该 design doc 变更再次触发 Step 1b dirty-worktree 决策。design doc 属于检查输入(非中性文档),写入后必须重跑 `comet check run verify --local -- <命令>`,再运行 `comet guard verify --apply` - 选项 B:用户选择 B 后,运行 `comet state transition verify-fail`,然后调用 `/comet-build`;由 `/comet-build` 的 Spec 增量更新规则加载 Superpowers `brainstorming` 更新 Design Doc + delta spec - 选项 C:确认偏差可接受,继续验证(归档时 design doc 将标记为 `superseded-by-main-spec`) ### 3. 记录验证证据 验证报告必须保存为文件,并在 `.comet.yaml` 中记录路径。verify 阶段不处理、合并或丢弃分支,也不写入 `branch_status: handled`。归档还会修改 spec 和元数据,这些修改必须包含在最终提交中,因此分支收尾统一由 `/comet-archive` 在归档提交后执行。不要手动设置 `verify_result: pass`,由阶段守卫 `--apply` 更新状态并推进阶段。 ```bash comet state set verification_report docs/superpowers/reports/YYYY-MM-DD--verify.md ``` 使用文件工具创建 `docs/superpowers/reports/` 和报告文件,不依赖 POSIX 专用目录命令。 ## 退出条件 - 验证报告通过 - `.comet.yaml` 中 `verification_report` 指向已存在的验证报告文件 - `.comet.yaml` 中 `branch_status` 仍为 `pending` - **阶段守卫**:运行 `comet guard verify --apply`,全部 PASS 后由守卫通过 `comet state transition verify-pass` 推进到 `phase: archive`(此步骤更新 `phase` 字段,与 `auto_transition` 无关) 验证证据完成后,运行阶段守卫推进 phase(此步骤与 `auto_transition` 无关): ```bash comet guard verify --apply ``` 状态文件自动更新为 `phase: archive`、`verify_result: pass`、`verified_at: YYYY-MM-DD`。 ## 上下文压缩恢复 按 `comet-classic/reference/context-recovery.md` 执行,phase 参数为 `verify`。 ## 自动衔接下一阶段 按 `comet-classic/reference/auto-transition.md` 和成功结果中的 `agent.continuation` 继续。已有仍然有效的状态信息时,不重复 next、select 或 check。只有丢失上下文后恢复任务、外部状态变化,或旧结果未提供这些信息时,才运行: ```bash comet state next ``` - `NEXT: auto` → 调用 `SKILL` 指向的 skill 进入下一阶段 - `NEXT: manual` → 不调用下一 skill,按 `HINT` 交还控制权并结束当前调用;不再创建确认点 - `NEXT: done` → 流程已完成,无需继续 注意:无论 NEXT 为 auto 还是 manual,归档都必须有用户的明确授权。首次归档时,按 comet-archive 请用户确认;恢复时,核对已保存的 delivery 记录,已有选择仍然有效就不重复询问。验证通过本身不代表用户已授权归档。