@hunter-harness/workflow-harness 0.4.8 → 0.4.9

package/harness/bundles/general/pi/harness-execute/coding-checklist.md

@@ -0,0 +1,279 @@
+---
+description: harness-execute 的执行检查清单。仅在编码执行时读取。
+---
+
+# harness-execute 执行检查清单
+
+> 编码执行时逐项勾选，确保不遗漏关键步骤。
+
+## 步骤 0：加载上下文
+
+### 步骤 0.0：Worktree 决策执行 ⚠️
+
+- [ ] 运行 `harness_runtime.py doctor` 并读取 `meta/runtime.json`；后续使用绝对 executable 与 argv 数组
+- [ ] 读取 `.harness/changes/<change-name>/meta/worktree.json`（旧路径 `worktree.json` 兼容）
+- [ ] 如果文件缺失：🟡WARN，按 legacy 逻辑询问用户是否主目录执行或创建 worktree
+- [ ] `requested=false` → 主目录执行，记录原因
+- [ ] `requested=true` 且 worktree 存在 → 切换到 worktree 执行
+- [ ] `requested=true` 且 worktree 不存在 → 必须创建 worktree
+- [ ] worktree 创建命令必须使用 PowerShell：`git worktree add ...`
+- [ ] 创建后验证 `.worktrees/<change-name>/.git` 存在
+- [ ] 创建成功后更新 `meta/worktree.json` 的 `created=true/createdAt/createdBy`
+- [ ] 创建失败 → 停止，或 blocking user confirmation 询问是否改为主目录执行
+- [ ] 禁止 `requested=true && worktree 不存在` 时直接主目录执行
+- [ ] 记录 `projectRoot / worktreeRoot / stateDir`
+
+
+
+- [ ] 确定变更名：Glob 搜索 `.harness/changes/*/plans/*-plan.md`（排除 `.harness/archive/*/`），提取 `change-name`
+- [ ] 读取并执行 `meta/worktree.json`：如果 `requested=true` 必须创建/切换 worktree，创建失败则停止或询问用户改为主目录；禁止静默降级
+- [ ] **读取计划文件（主任务源）**：`.harness/changes/<change>/plans/<change>-plan.md` — 获取任务列表和依赖关系
+- [ ] **读取详细计划（补充参考）**：`.harness/changes/<change>/plans/<change>-implementation-detail.md`（新版必需；legacy 缺失时 🟡WARN）
+- [ ] **读取设计文档**：`.harness/changes/<change>/plans/<change>-design.md`（v2）；不存在时回退 `spec/<change>-design.md`（legacy）— 获取核心设计决策和不变项
+- [ ] **读取测试场景表**：`.harness/changes/<change>/plans/<change>-test-scenarios.md` — 获取与当前任务相关的测试场景
+- [ ] **读取验证账本**：通过 context 返回的 `executionRoot` 读取 `evidence/verification-ledger.json`（如存在）— 复用已有 compile/unitTest 结果
+- [ ] **读取任务状态**：`.harness/changes/<change>/evidence/run-task-status.md`（如存在）— 恢复上次运行状态
+- [ ] **读取 review fixback**：用户传入 `--fixback` 或要求修复 review 问题时，读取最新 `.harness/changes/<change>/reports/review/fixback-*.md`
+- [ ] 确认未将 `docs/superpowers/` 作为执行输入（旧草稿最多作为人工线索）
+- [ ] 确认 `项目规则（见 .harness/context-index.json）/` 规则已加载
+- [ ] 检查构建配置完整性（worktree 中确认构建配置文件存在，如 Java 的 `.mvn/maven.config`、`settings.xml`，前端的 `package.json`/lockfile 等）
+- [ ] 依赖模块预安装（worktree 中检查上游依赖是否已安装，如 Java 的 `mvn install`、前端的 `npm install`/lockfile 等）
+- [ ] 代码探索优先用 `codegraph_explore`，仅在返回不完整时补充 Read
+- [ ] `harness_context.py prepare/begin` 均带 `--project .` 与 `--change <id>`（缺 `--project` 会被 argparse 直接拒）
+- [ ] 交接凭证缺失时**不自行拼造**：v2 计划由 `prepare` 依据 committed 发布 journal 自动补录；仍报 `HANDOFF_REQUIRED`/`LEGACY_BOOTSTRAP_REQUIRED` 即代表发布证据不存在，回 plan 阶段查，不得用 `classify + configure-plan + close` 现编凭证
+- [ ] `harness_gate.py begin --phase run` 已返回 Plan handoff 校验通过并自动 append `phase.start`；不得手工写事件绕过
+- [ ] 门禁报 `SCENARIO_MANIFEST_V2_UNSUPPORTED` 时按 `missingFields` 回规划阶段补齐场景字段后重新发布；**不得**手改 `meta/scenario-manifest.json`（派生产物，手改必致哈希漂移）
+
+### 步骤 0.1：执行模式（无询问）
+
+- [ ] 统计任务数量和涉及模块数
+- [ ] 如果处于 `--fixback` 模式，将 RED/YELLOW fixback 条目映射为本轮变更簇，并记录 `fixback: applied`
+- [ ] 默认 Inline Execution；`--subagent` / `--inline` 参数覆盖
+- [ ] **禁止**因任务数/模块数 blocking user confirmation 选择执行模式
+
+## 步骤 0.5：测试基础设施探测（⚠️ 必须先于任何 TDD 降级结论）
+
+> 探测完成前，执行日志中只能写 `**测试基础设施**: CHECKING`，不得写任何降级结论。
+
+- [ ] 探测 1：测试目录是否存在（如 Java 的 `src/test/java`）
+- [ ] 探测 2：构建配置/依赖清单是否包含测试依赖（如 Java 的 `pom.xml` 含 `spring-boot-starter-test`/`junit`/`mockito`）
+- [ ] 探测 3：是否存在已有测试文件（按技术栈测试命名约定，如 Java 的 `*Test.java`/`*Tests.java`）
+- [ ] 探测 4：目标模块测试基础设施是否可用（按技术栈，如 Java 的 `mvn test-compile -pl <module> -o -q` 验证测试可编译，或测试枚举/秒级 smoke；**禁止完整模块测试**，spec §3.3）
+- [ ] 四项证据收集完毕 → 写结论：✅ 测试基础设施可用 / 🟡 测试基础设施部分可用 / ❌ 测试基础设施不可用
+- [ ] 如果 ❌ 不可用 → 记录 TDD 降级原因（必须引用具体证据，如"模块 X 无测试目录（如 Java 的 src/test/java）"）
+- [ ] 若探测失败来自具体陈旧测试，先执行「陈旧测试安全修复」判定：当前代码/批准计划/可验证历史唯一确定契约 + 仅改测试 + 立即重跑；满足才修复，否则记录 `BLOCKED_PREEXISTING`
+- [ ] 修复/新增/更新的精确测试路径已用 `harness_test_guard.py record` 记录为 `stale-test-repair` / `tdd-created` / `test-updated`
+- [ ] **禁止临时排除测试**：未使用 `.bak`/改名/移目录/删除/禁用注解/build exclude/skip-tests 绕过失败
+
+## 步骤 0.2：预存变更检测与隔离
+
+> 如果 run 开始前检测到已有未提交变更，必须执行此步骤。
+
+- [ ] 检查 git status 是否有 run 前的预存变更
+- [ ] 如果有预存变更 → 使用 blocking user confirmation 询问用户处理方式（保留/暂存/终止）
+- [ ] 用户选择保留 → 创建 `.harness/changes/<change>/pre-existing-files.json`（文件列表 + diff hash + 用户选择）
+- [ ] 用户选择保留 → 创建 `.harness/changes/<change>/pre-existing-diff.patch`（完整 diff 备份）
+- [ ] 在执行日志中记录预存变更检测结果和用户选择
+
+## 变更簇 TDD 循环
+
+> 默认按变更簇执行 TDD，不按每个小任务单独 RED/GREEN。
+
+### 步骤 1a：划分变更簇
+
+- [ ] 读取 plan.md 任务列表，按业务行为分组为变更簇
+- [ ] 检查是否有低价值 TDD 豁免项（错误码常量、数据契约字段 VO/DTO、注释、代码整理如 import 清理、格式化、数据库迁移脚本、配置模板、文档）→ 标记为豁免，不单独建测试类
+- [ ] 确定每个变更簇的测试范围（哪些测试类合并执行）
+- [ ] 确认变更簇内数据访问层查询条件是否需要用真实 DB 验证（非纯 Mock）
+
+### 进入变更簇 RED 前：执行 run-tdd-protocol
+
+- [ ] 已读取 `protocols.md`
+- [ ] 为当前变更簇选择 RED 类型：真实 RED / 静态 RED / 复用 RED
+- [ ] 真实 RED → 测试编译通过且失败断言指向目标行为
+- [ ] 静态 RED → 记录降级原因、静态验证场景、待 harness-execute 验证场景
+- [ ] 复用 RED → 引用 test-scenarios 编号与前序证据，并在执行日志登记
+- [ ] 任何 RED 类型都已写入 events.ndjson（`decision` / `verification` / `issue`）
+
+### RED（写测试 — 按变更簇批量执行）
+
+- [ ] 从场景表选取对应当前变更簇的测试用例
+- [ ] 写单元测试（按技术栈测试框架，如 Java 的 JUnit 5 + Mockito + AssertJ），**优先通过 public API 验证**（非 private 方法）
+- [ ] **多个测试类合并到一次构建/测试命令执行**（按技术栈模块/用例定位参数，如 Java 的 `mvn test -pl <module> -Dtest=TestA,TestB,TestC -o`）
+- [ ] **RED 有效性判定**（必须逐项确认）：
+  - [ ] 测试编译通过？（编译失败 → ❌无效 RED，先修测试）
+  - [ ] 未直接调用 private 方法？（private 访问限制 → ❌无效 RED）
+  - [ ] 失败断言指向目标业务行为？（mock/stubbing 错误 → ❌无效 RED）
+  - [ ] 失败原因与本次 bug/需求直接相关？（无关 → ❌无效 RED）
+  - [ ] 无依赖注入上下文加载失败（如 Java 的 NoSuchBeanDefinitionException）/ NPE 来自测试搭建错误 / 不必要 stubbing？
+- [ ] 判定结果写入执行日志：`RED: ✅有效 / 🟡部分有效 / ❌无效`
+- [ ] 如果 ❌无效 RED → 必须先修测试，重新运行 RED，**禁止进入 GREEN**
+- [ ] 如果无测试基础设施 → 降级为静态逻辑验证（在执行日志和覆盖报告中标注覆盖关系，**不得在业务代码注释中标注**）
+
+### 低价值 TDD 豁免检查
+
+- [ ] 错误码常量 → 不单独建测试类，构建验证 + 被高层测试间接覆盖
+- [ ] 数据契约字段（VO/DTO）→ 不单独建测试类，被业务层/API 测试间接覆盖
+- [ ] 注释 / 代码整理（import 清理）/ 格式化 → 构建验证即可
+- [ ] 数据库迁移脚本 → 不做 TDD，生成审查清单，标记 NEEDS_DB_VALIDATION
+- [ ] 配置模板 / 文档 → 静态审查
+
+### 数据访问层查询条件验证检查
+
+- [ ] 如果变更涉及数据访问层查询逻辑（Java 的 Mapper/LambdaQueryWrapper/SQL/XML）→ 不得用纯 Mock 宣称"自动化测试通过"
+- [ ] 纯 Mock 数据访问层测试 → 标记为 🟡静态验证，交给 harness-execute 真实 DB 验证
+- [ ] 如果必须自动化 → 使用真实数据访问层（非 mock）或可检查查询条件的测试方式
+
+### 编码前检查（远程客户端 / stub 同源错误）
+
+- [ ] 变更涉及 HTTP/RPC 客户端（Feign/RestTemplate/SDK 封装）时：取客户端注解路径（类级 + 方法级拼接），与服务提供方 controller 的 `@RequestMapping` + 方法级注解**完整拼接路径**逐一比对，在计划/执行记录中列出比对结果。只看方法级注解不算完成。
+- [ ] 修 stub/null 返回时，顺带核对同文件相邻声明（注解、常量、同类方法路径）是否同源错误，一并列出核对结果
+
+### GREEN（最简实现 — 按变更簇批量实现）
+
+- [ ] 写最少代码让变更簇测试全部通过
+- [ ] 遵循约束：接口层只做参数校验和路由；业务层是唯一业务逻辑层；统一返回结构（如 Java 的 `Result<T>`）；集合返回空集合不返回 null；日志用日志框架（如 Java 的 Slf4j）；新增字段允许为空
+
+### REFACTOR（重构）
+
+- [ ] 在测试保护下重构代码结构
+- [ ] 清理过程性注释（如 `// 修复分页查询缺项目类型 Bug`），改写为稳定业务规则描述或删除
+- [ ] 重构后重新运行变更簇测试确认全部通过
+
+### 批量构建验证（变更簇间）
+
+- [ ] 每个变更簇最多执行一次 RED 构建/测试、一次 GREEN 构建/测试 — **不得每新增一个测试类就单独跑一次构建/测试**
+- [ ] 多个测试类合并执行（按技术栈模块/用例定位参数，如 Java 的 `mvn test -pl <module> -Dtest=TestA,TestB,TestC -o`）
+- [ ] 最终构建只执行一次（如前面已有构建成功证据，可复用 verification-ledger）
+- [ ] 使用静默模式（如 Maven `-q`）时：报告中写 `exitCode=0`，不得写"构建成功"字样（如 Java 的 BUILD SUCCESS，除非输出中真实出现）
+- [ ] 按 `change-cluster-review-protocol` 判断是否触发变更簇审查
+- [ ] 高风险触发条件（数据迁移/权限/安全/并发/核心契约/缺真实测试/用户要求）已检查
+- [ ] 触发审查时，已记录方式：只读 reviewer / 主会话自审
+- [ ] 未触发审查时，已记录跳过理由（低风险/已有真实测试证据/后续 harness-review 覆盖）
+
+## 步骤 2：构建验证（默认轻量，批量执行）
+
+- [ ] 构建命令（按技术栈，如 Java 的 `mvn compile -pl <module> -o`）（始终执行，优先不用静默模式以获取构建成功证据）
+- [ ] 如果使用静默模式（如 Maven `-q`）→ 报告中写 `exitCode=0`，**不得写"构建成功"字样（如 Java 的 BUILD SUCCESS）**
+- [ ] 最终构建只执行一次，如果前面已有构建成功证据，可复用 verification-ledger
+- [ ] 判断是否需要全量测试：改了公共模块/数据访问层/数据库迁移/权限认证/接口层/数据契约，或用户要求 full-run-validation → 执行测试命令（按技术栈，如 Java 的 `mvn test -pl <module> -o`）；否则跳过全量测试
+- [ ] 构建失败 → 先分析错误类型（见 coding-reference.md 构建失败策略表）
+- [ ] **经 `harness_ledger.py record` 写入 ledger**（禁止 Write/Edit `verification-ledger.json`）：`compile` 项必写；若执行了测试则 `unitTest` 项必写；未执行时标记 `NOT_RUN_BY_RUN`
+- [ ] 顶层写入 `diffHash` / `currentHead` / `module` / `profile`；`diffHash` 必须执行 `harness_ledger.py diff-hash --repo . --base <baseCommit> --change-dir ".harness/changes/<change-name>" --json`，纳入 test-tracking manifest 中的 ignored tests；`currentHead`=`git rev-parse HEAD`
+- [ ] test/submit/package 阶段如果 diffHash 一致，可复用 run 的 compile/unitTest 结果
+
+## 步骤 3：场景覆盖检查
+
+- [ ] 单元测试场景逐条确认覆盖
+- [ ] 接口测试场景逐条确认代码逻辑覆盖
+- [ ] 数据兼容场景逐条确认（新字段 nullable、旧格式兼容）
+- [ ] 将覆盖结果展示给用户（✅/🟡/❌/⏳ 标注）
+- [ ] **覆盖状态必须正确分类**：✅ 自动化测试通过 / 🟡 静态检查未真实测试 / ❌ 未验证 / ⏳ `ownerPhase=test` 按计划移交
+- [ ] 🟡 静态检查 **不得计入"已测试通过"**
+- [ ] 如果任一 `ownerPhase=run` 的 P0 场景仅静态验证 → 最终结果必须是 🟡WARN
+- [ ] `ownerPhase=test` 的任务或场景只记录为“待测试阶段执行”，不得将编码阶段降级为 WARN
+- [ ] 最终摘要禁止写 `5✅ + 17🟡 = 22/22`，必须写 `自动化测试通过: 5 / 静态检查未真实验证: 17 / 未验证: 0`
+- [ ] **禁止用测试用例数（如 178 tests）冒充场景数**——计数对象是 test-scenarios.md 的场景编号
+- [ ] 四类计数自洽：run-owned 🟡/❌ 计入未验证；test-owned ⏳ 单列移交数量
+- [ ] 输出为场景表映射（UT/API/COM/INT 逐条或范围 ✅/🟡/❌），不得只给聚合测试数
+
+### 权限/组织过滤类变更：安全矩阵
+
+> 凡是修改了管理员/非管理员、orgCode、数据权限、越权异常等逻辑，必须强制生成。
+
+- [ ] 生成安全矩阵（模板见 coding-reference.md）
+- [ ] 覆盖：超级管理员 token 空/非空 × 请求 orgCode 空/指定 × projectType 有/无
+- [ ] 覆盖：非管理员 token 本组织/空 × 请求 orgCode 本组织/其他组织/空 × projectType 有/无
+- [ ] 如果任一权限边界预期不明确 → 不允许标记 ✅OK
+
+## 步骤 4：关门检查（⚠️ 结束前强制执行）
+
+- [ ] `git status --porcelain`
+- [ ] `git diff --stat`
+- [ ] `git diff --check`（如果失败 → 最终结果必须是 ❌FAIL）
+- [ ] 变更文件是否全部在计划范围内
+- [ ] 是否新增/修改了测试文件
+- [ ] 是否误改 .harness/ 以外的非计划文件（如有 → 至少 🟡WARN，要求用户确认）
+- [ ] 是否有 conflict marker：`<<<<<<<` / `=======` / `>>>>>>>`
+- [ ] 是否有临时 debug：`System.out.println` / `console.log` / `debugger` / 临时 TODO/FIXME
+- [ ] 是否有敏感信息：`password` / `token` / `secret` / `accessKey` / 私有 IP/URL
+- [ ] 代码注释污染检查：生产代码中是否有 `// 修复 xxx Bug` / `// 本次变更` 等过程性注释（如有 → 必须在 REFACTOR 阶段清理）
+
+### 变更来源区分（预存变更隔离）
+
+- [ ] 区分本次 run 修改的文件 vs run 前预存变更的文件
+- [ ] 输出变更来源表（文件 / 来源 / 是否计划内 / 是否允许提交）
+- [ ] 如果存在预存变更 → 最终结果至少为 🟡WARN
+
+### 数据库迁移任务审查清单
+
+> 如果变更涉及数据库迁移脚本，必须生成此清单。
+
+- [ ] DROP INDEX 是否存在 IF EXISTS 或等效检查
+- [ ] 新索引名称
+- [ ] 新索引字段
+- [ ] 是否包含 deleted_time
+- [ ] 是否兼容已有数据
+- [ ] 是否包含历史数据修正
+- [ ] 是否需要备份
+- [ ] 是否需要回滚 SQL
+- [ ] 数据库迁移脚本头部已标注"人工审查后手动执行，禁止自动运行"
+- [ ] 数据库迁移任务状态标记为 🟡 NEEDS_DB_VALIDATION（不得标记为自动化测试通过）
+
+## 步骤 5：Worktree checkpoint commit（⚠️ 强制阻断）
+
+> 仅在 worktree 中执行时触发此步骤。在主目录执行时跳过（标记 ⏭️主目录跳过）。
+
+- [ ] 确认当前 cwd 在 `<adapter-worktree-root>/<change-name>/` 下，且 branch 使用 adapter 的 prefix
+- [ ] 生成变更摘要：`git diff --stat` + `git diff --stat --cached`
+- [ ] 构建 commit message：`wip(<scope>): <change-name> 编码完成 — N任务/M文件变更`
+- [ ] **⚠️ 强制阻断**：用 blocking user confirmation 展示变更列表 + commit message，等待用户确认
+- [ ] 用户确认 → 精确暂存业务文件；若存在 test-tracking manifest，先执行 `harness_test_guard.py stage --project . --change-dir ".harness/changes/<change-name>" --json`；禁止 `git add -A` 与目录级 force-add
+- [ ] `git diff --cached --name-only` 包含 manifest 的全部精确测试路径后再 `git commit`（不用 --no-verify、--no-gpg-sign）
+- [ ] 用户拒绝 → 记录 `❌用户拒绝`，继续后续流程
+- [ ] 记录 checkpoint commit hash 到执行日志
+
+## 步骤 6：计划状态持久化
+
+- [ ] 每个任务状态只更新到 `evidence/run-task-status.md`；不得修改 finalized plan 文件
+- [ ] 状态区分：✅ DONE_AUTOMATED_TESTED / 🟡 DONE_STATIC_ONLY / 🟡 DONE_NEEDS_INTERFACE_TEST / 🟡 NEEDS_DB_VALIDATION / ❌ FAILED
+- [ ] 确保后续 harness-execute / harness-review 可读取待验证场景
+
+## 事件收尾记录
+
+- [ ] append `phase.end` **仅**通过 `harness_gate.py close`（禁止手工 Edit `events.ndjson` / 直接 append phase.end 绕过 close 校验）
+- [ ] 如果存在编码侧 🟡静态验证 P0 场景 → 下一步必须写 `必须先完成本阶段的验证执行（见 testing-checklist.md）`，不得并列"提交代码"
+- [ ] 如果 run 阶段负责的数据库迁移/DB 验证未完成 → 最终不得输出纯 ✅OK
+
+### 最终状态分级
+
+- [ ] **✅OK**：所有计划内代码变更完成 + run-owned 关键 P0 场景已自动化测试通过 + 构建成功 + 无预存变更或已隔离 + 无非计划文件混入 + 无 run-owned P0 静态-only 场景
+- [ ] **🟡WARN**：存在 run-owned P0/P1 场景仅静态验证 / 存在预存变更 / run-owned 数据库验证未完成 / 低价值 Mock 替代真实验证
+- [ ] **❌FAIL**：构建失败 / 有效测试失败 / RED 无法建立 / 非计划文件被修改且无法解释 / git diff --check 失败
+- [ ] `ownerPhase=test` 的待办属于正常阶段移交；不得因为后续阶段尚未执行而把已完成的 run 标成 WARN
+## PowerShell-only 命令检查
+
+- [ ] 所有 git 命令使用 `powershell.exe -NoProfile -Command "git ..."`
+- [ ] 所有构建/测试命令使用 `powershell.exe -NoProfile -Command "<构建/测试命令> ..."`
+- [ ] 所有文件系统命令使用 PowerShell
+- [ ] 如果出现 Bash hook error，记录违规命令并将当前阶段至少标为 🟡WARN
+
+## Ledger 必填字段检查
+
+- [ ] verification-ledger.json 包含 changeName/projectRoot/worktreeRoot/stateDir
+- [ ] 包含 currentHead/baseCommit/diffHash/module/profile
+- [ ] 包含 validations.compile 与 validations.unitTest
+- [ ] 缺字段时标记 `ledgerReusable=false`，后续阶段不得复用
+- [ ] diffHash 必须由 `harness_ledger.py diff-hash --change-dir` 生成；manifest hash 漂移或路径越界时硬停止，禁止绕过后复用 ledger
+
+## 新筛选参数非法值行为检查
+
+- [ ] 如果新增筛选参数，设计中已明确非法值行为
+- [ ] 如果采用“忽略并不过滤”，必须有用户确认记录
+- [ ] 测试场景覆盖非法值
+
+
+## Wave-2 — Migration head（H-7）
+
+- [ ] 若变更触及 alembic/versions 或项目约定 migration 路径：列出 config / CI / env / manifest 联动文件
+- [ ] 消费项目存在 .harness/config/migration-head.json 时，跑 harness_migration_head.py check --project . --json