@hunter-harness/workflow-harness 0.2.16 → 0.2.17
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/harness/bundles/general/claude-code/.harness-build.json +1 -1
- package/harness/bundles/general/claude-code/CONTEXT.md +14 -1
- package/harness/bundles/general/claude-code/README.md +1 -1
- package/harness/bundles/general/claude-code/harness-archive/SKILL.md +7 -4
- package/harness/bundles/general/claude-code/harness-archive/reference.md +1 -1
- package/harness/bundles/general/claude-code/harness-codebase-map/SKILL.md +4 -2
- package/harness/bundles/general/claude-code/harness-knowledge-ingest/SKILL.md +4 -2
- package/harness/bundles/general/claude-code/harness-knowledge-query/SKILL.md +4 -2
- package/harness/bundles/general/claude-code/harness-plan/SKILL.md +8 -4
- package/harness/bundles/general/claude-code/harness-plan/checklist.md +5 -5
- package/harness/bundles/general/claude-code/harness-review/SKILL.md +5 -2
- package/harness/bundles/general/claude-code/harness-run/SKILL.md +7 -3
- package/harness/bundles/general/claude-code/harness-run/checklist.md +4 -4
- package/harness/bundles/general/claude-code/harness-submit/SKILL.md +6 -3
- package/harness/bundles/general/claude-code/harness-submit/checklist.md +5 -5
- package/harness/bundles/general/claude-code/harness-submit/reference.md +3 -3
- package/harness/bundles/general/claude-code/harness-sync/SKILL.md +4 -2
- package/harness/bundles/general/claude-code/harness-sync/reference.md +1 -1
- package/harness/bundles/general/claude-code/harness-test/SKILL.md +7 -4
- package/harness/bundles/general/claude-code/harness-test/checklist.md +8 -5
- package/harness/bundles/general/claude-code/harness-test/pitfalls.md +1 -1
- package/harness/bundles/general/claude-code/harness-test/reference.md +28 -13
- package/harness/bundles/general/claude-code/protocols/submit-protocol.md +1 -1
- package/harness/bundles/general/claude-code/scripts/harness_deploy.py +7 -0
- package/harness/bundles/general/claude-code/scripts/harness_integration.py +26 -2
- package/harness/bundles/general/claude-code/scripts/harness_plan_finalize.py +159 -7
- package/harness/bundles/general/claude-code/scripts/harness_test_cleanup.py +175 -0
- package/harness/bundles/general/codebuddy/.harness-build.json +1 -1
- package/harness/bundles/general/codebuddy/CONTEXT.md +14 -1
- package/harness/bundles/general/codebuddy/README.md +1 -1
- package/harness/bundles/general/codebuddy/harness-archive/SKILL.md +7 -4
- package/harness/bundles/general/codebuddy/harness-archive/reference.md +1 -1
- package/harness/bundles/general/codebuddy/harness-codebase-map/SKILL.md +4 -2
- package/harness/bundles/general/codebuddy/harness-knowledge-ingest/SKILL.md +4 -2
- package/harness/bundles/general/codebuddy/harness-knowledge-query/SKILL.md +4 -2
- package/harness/bundles/general/codebuddy/harness-plan/SKILL.md +8 -4
- package/harness/bundles/general/codebuddy/harness-plan/checklist.md +5 -5
- package/harness/bundles/general/codebuddy/harness-review/SKILL.md +5 -2
- package/harness/bundles/general/codebuddy/harness-run/SKILL.md +7 -3
- package/harness/bundles/general/codebuddy/harness-run/checklist.md +4 -4
- package/harness/bundles/general/codebuddy/harness-submit/SKILL.md +6 -3
- package/harness/bundles/general/codebuddy/harness-submit/checklist.md +5 -5
- package/harness/bundles/general/codebuddy/harness-submit/reference.md +3 -3
- package/harness/bundles/general/codebuddy/harness-sync/SKILL.md +4 -2
- package/harness/bundles/general/codebuddy/harness-sync/reference.md +1 -1
- package/harness/bundles/general/codebuddy/harness-test/SKILL.md +7 -4
- package/harness/bundles/general/codebuddy/harness-test/checklist.md +8 -5
- package/harness/bundles/general/codebuddy/harness-test/pitfalls.md +1 -1
- package/harness/bundles/general/codebuddy/harness-test/reference.md +28 -13
- package/harness/bundles/general/codebuddy/protocols/submit-protocol.md +1 -1
- package/harness/bundles/general/codebuddy/scripts/harness_deploy.py +7 -0
- package/harness/bundles/general/codebuddy/scripts/harness_integration.py +26 -2
- package/harness/bundles/general/codebuddy/scripts/harness_plan_finalize.py +159 -7
- package/harness/bundles/general/codebuddy/scripts/harness_test_cleanup.py +175 -0
- package/harness/bundles/general/codex/.harness-build.json +1 -1
- package/harness/bundles/general/codex/CONTEXT.md +14 -1
- package/harness/bundles/general/codex/README.md +1 -1
- package/harness/bundles/general/codex/harness-archive/SKILL.md +7 -4
- package/harness/bundles/general/codex/harness-archive/reference.md +1 -1
- package/harness/bundles/general/codex/harness-codebase-map/SKILL.md +4 -2
- package/harness/bundles/general/codex/harness-knowledge-ingest/SKILL.md +4 -2
- package/harness/bundles/general/codex/harness-knowledge-query/SKILL.md +4 -2
- package/harness/bundles/general/codex/harness-plan/SKILL.md +8 -4
- package/harness/bundles/general/codex/harness-plan/checklist.md +5 -5
- package/harness/bundles/general/codex/harness-review/SKILL.md +5 -2
- package/harness/bundles/general/codex/harness-run/SKILL.md +7 -3
- package/harness/bundles/general/codex/harness-run/checklist.md +4 -4
- package/harness/bundles/general/codex/harness-submit/SKILL.md +6 -3
- package/harness/bundles/general/codex/harness-submit/checklist.md +5 -5
- package/harness/bundles/general/codex/harness-submit/reference.md +3 -3
- package/harness/bundles/general/codex/harness-sync/SKILL.md +4 -2
- package/harness/bundles/general/codex/harness-sync/reference.md +1 -1
- package/harness/bundles/general/codex/harness-test/SKILL.md +7 -4
- package/harness/bundles/general/codex/harness-test/checklist.md +8 -5
- package/harness/bundles/general/codex/harness-test/pitfalls.md +1 -1
- package/harness/bundles/general/codex/harness-test/reference.md +28 -13
- package/harness/bundles/general/codex/protocols/submit-protocol.md +1 -1
- package/harness/bundles/general/codex/scripts/harness_deploy.py +7 -0
- package/harness/bundles/general/codex/scripts/harness_integration.py +26 -2
- package/harness/bundles/general/codex/scripts/harness_plan_finalize.py +159 -7
- package/harness/bundles/general/codex/scripts/harness_test_cleanup.py +175 -0
- package/harness/bundles/general/cursor/.harness-build.json +1 -1
- package/harness/bundles/general/cursor/CONTEXT.md +14 -1
- package/harness/bundles/general/cursor/README.md +1 -1
- package/harness/bundles/general/cursor/harness-archive/SKILL.md +7 -4
- package/harness/bundles/general/cursor/harness-archive/reference.md +1 -1
- package/harness/bundles/general/cursor/harness-codebase-map/SKILL.md +4 -2
- package/harness/bundles/general/cursor/harness-knowledge-ingest/SKILL.md +4 -2
- package/harness/bundles/general/cursor/harness-knowledge-query/SKILL.md +4 -2
- package/harness/bundles/general/cursor/harness-plan/SKILL.md +8 -4
- package/harness/bundles/general/cursor/harness-plan/checklist.md +5 -5
- package/harness/bundles/general/cursor/harness-review/SKILL.md +5 -2
- package/harness/bundles/general/cursor/harness-run/SKILL.md +7 -3
- package/harness/bundles/general/cursor/harness-run/checklist.md +4 -4
- package/harness/bundles/general/cursor/harness-submit/SKILL.md +6 -3
- package/harness/bundles/general/cursor/harness-submit/checklist.md +5 -5
- package/harness/bundles/general/cursor/harness-submit/reference.md +3 -3
- package/harness/bundles/general/cursor/harness-sync/SKILL.md +4 -2
- package/harness/bundles/general/cursor/harness-sync/reference.md +1 -1
- package/harness/bundles/general/cursor/harness-test/SKILL.md +7 -4
- package/harness/bundles/general/cursor/harness-test/checklist.md +8 -5
- package/harness/bundles/general/cursor/harness-test/pitfalls.md +1 -1
- package/harness/bundles/general/cursor/harness-test/reference.md +28 -13
- package/harness/bundles/general/cursor/protocols/submit-protocol.md +1 -1
- package/harness/bundles/general/cursor/scripts/harness_deploy.py +7 -0
- package/harness/bundles/general/cursor/scripts/harness_integration.py +26 -2
- package/harness/bundles/general/cursor/scripts/harness_plan_finalize.py +159 -7
- package/harness/bundles/general/cursor/scripts/harness_test_cleanup.py +175 -0
- package/harness/bundles/java/claude-code/.harness-build.json +1 -1
- package/harness/bundles/java/claude-code/CONTEXT.md +14 -1
- package/harness/bundles/java/claude-code/README.md +1 -1
- package/harness/bundles/java/claude-code/harness-apidoc/SKILL.md +5 -2
- package/harness/bundles/java/claude-code/harness-archive/SKILL.md +7 -4
- package/harness/bundles/java/claude-code/harness-archive/reference.md +1 -1
- package/harness/bundles/java/claude-code/harness-codebase-map/SKILL.md +4 -2
- package/harness/bundles/java/claude-code/harness-knowledge-ingest/SKILL.md +4 -2
- package/harness/bundles/java/claude-code/harness-knowledge-query/SKILL.md +4 -2
- package/harness/bundles/java/claude-code/harness-package/SKILL.md +5 -2
- package/harness/bundles/java/claude-code/harness-package/checklist.md +4 -4
- package/harness/bundles/java/claude-code/harness-plan/SKILL.md +8 -4
- package/harness/bundles/java/claude-code/harness-plan/checklist.md +5 -5
- package/harness/bundles/java/claude-code/harness-review/SKILL.md +5 -2
- package/harness/bundles/java/claude-code/harness-run/SKILL.md +7 -3
- package/harness/bundles/java/claude-code/harness-run/checklist.md +4 -4
- package/harness/bundles/java/claude-code/harness-submit/SKILL.md +6 -3
- package/harness/bundles/java/claude-code/harness-submit/checklist.md +5 -5
- package/harness/bundles/java/claude-code/harness-submit/reference.md +3 -3
- package/harness/bundles/java/claude-code/harness-sync/SKILL.md +4 -2
- package/harness/bundles/java/claude-code/harness-sync/reference.md +1 -1
- package/harness/bundles/java/claude-code/harness-test/SKILL.md +8 -5
- package/harness/bundles/java/claude-code/harness-test/pitfalls.md +1 -1
- package/harness/bundles/java/claude-code/harness-test/reference.md +1 -1
- package/harness/bundles/java/claude-code/protocols/submit-protocol.md +1 -1
- package/harness/bundles/java/claude-code/scripts/harness_deploy.py +7 -0
- package/harness/bundles/java/claude-code/scripts/harness_integration.py +26 -2
- package/harness/bundles/java/claude-code/scripts/harness_plan_finalize.py +159 -7
- package/harness/bundles/java/claude-code/scripts/harness_test_cleanup.py +175 -0
- package/harness/bundles/java/codebuddy/.harness-build.json +1 -1
- package/harness/bundles/java/codebuddy/CONTEXT.md +14 -1
- package/harness/bundles/java/codebuddy/README.md +1 -1
- package/harness/bundles/java/codebuddy/harness-apidoc/SKILL.md +5 -2
- package/harness/bundles/java/codebuddy/harness-archive/SKILL.md +7 -4
- package/harness/bundles/java/codebuddy/harness-archive/reference.md +1 -1
- package/harness/bundles/java/codebuddy/harness-codebase-map/SKILL.md +4 -2
- package/harness/bundles/java/codebuddy/harness-knowledge-ingest/SKILL.md +4 -2
- package/harness/bundles/java/codebuddy/harness-knowledge-query/SKILL.md +4 -2
- package/harness/bundles/java/codebuddy/harness-package/SKILL.md +5 -2
- package/harness/bundles/java/codebuddy/harness-package/checklist.md +4 -4
- package/harness/bundles/java/codebuddy/harness-plan/SKILL.md +8 -4
- package/harness/bundles/java/codebuddy/harness-plan/checklist.md +5 -5
- package/harness/bundles/java/codebuddy/harness-review/SKILL.md +5 -2
- package/harness/bundles/java/codebuddy/harness-run/SKILL.md +7 -3
- package/harness/bundles/java/codebuddy/harness-run/checklist.md +4 -4
- package/harness/bundles/java/codebuddy/harness-submit/SKILL.md +6 -3
- package/harness/bundles/java/codebuddy/harness-submit/checklist.md +5 -5
- package/harness/bundles/java/codebuddy/harness-submit/reference.md +3 -3
- package/harness/bundles/java/codebuddy/harness-sync/SKILL.md +4 -2
- package/harness/bundles/java/codebuddy/harness-sync/reference.md +1 -1
- package/harness/bundles/java/codebuddy/harness-test/SKILL.md +8 -5
- package/harness/bundles/java/codebuddy/harness-test/pitfalls.md +1 -1
- package/harness/bundles/java/codebuddy/harness-test/reference.md +1 -1
- package/harness/bundles/java/codebuddy/protocols/submit-protocol.md +1 -1
- package/harness/bundles/java/codebuddy/scripts/harness_deploy.py +7 -0
- package/harness/bundles/java/codebuddy/scripts/harness_integration.py +26 -2
- package/harness/bundles/java/codebuddy/scripts/harness_plan_finalize.py +159 -7
- package/harness/bundles/java/codebuddy/scripts/harness_test_cleanup.py +175 -0
- package/harness/bundles/java/codex/.harness-build.json +1 -1
- package/harness/bundles/java/codex/CONTEXT.md +14 -1
- package/harness/bundles/java/codex/README.md +1 -1
- package/harness/bundles/java/codex/harness-apidoc/SKILL.md +5 -2
- package/harness/bundles/java/codex/harness-archive/SKILL.md +7 -4
- package/harness/bundles/java/codex/harness-archive/reference.md +1 -1
- package/harness/bundles/java/codex/harness-codebase-map/SKILL.md +4 -2
- package/harness/bundles/java/codex/harness-knowledge-ingest/SKILL.md +4 -2
- package/harness/bundles/java/codex/harness-knowledge-query/SKILL.md +4 -2
- package/harness/bundles/java/codex/harness-package/SKILL.md +5 -2
- package/harness/bundles/java/codex/harness-package/checklist.md +4 -4
- package/harness/bundles/java/codex/harness-plan/SKILL.md +8 -4
- package/harness/bundles/java/codex/harness-plan/checklist.md +5 -5
- package/harness/bundles/java/codex/harness-review/SKILL.md +5 -2
- package/harness/bundles/java/codex/harness-run/SKILL.md +7 -3
- package/harness/bundles/java/codex/harness-run/checklist.md +4 -4
- package/harness/bundles/java/codex/harness-submit/SKILL.md +6 -3
- package/harness/bundles/java/codex/harness-submit/checklist.md +5 -5
- package/harness/bundles/java/codex/harness-submit/reference.md +3 -3
- package/harness/bundles/java/codex/harness-sync/SKILL.md +4 -2
- package/harness/bundles/java/codex/harness-sync/reference.md +1 -1
- package/harness/bundles/java/codex/harness-test/SKILL.md +8 -5
- package/harness/bundles/java/codex/harness-test/pitfalls.md +1 -1
- package/harness/bundles/java/codex/harness-test/reference.md +1 -1
- package/harness/bundles/java/codex/protocols/submit-protocol.md +1 -1
- package/harness/bundles/java/codex/scripts/harness_deploy.py +7 -0
- package/harness/bundles/java/codex/scripts/harness_integration.py +26 -2
- package/harness/bundles/java/codex/scripts/harness_plan_finalize.py +159 -7
- package/harness/bundles/java/codex/scripts/harness_test_cleanup.py +175 -0
- package/harness/bundles/java/cursor/.harness-build.json +1 -1
- package/harness/bundles/java/cursor/CONTEXT.md +14 -1
- package/harness/bundles/java/cursor/README.md +1 -1
- package/harness/bundles/java/cursor/harness-apidoc/SKILL.md +5 -2
- package/harness/bundles/java/cursor/harness-archive/SKILL.md +7 -4
- package/harness/bundles/java/cursor/harness-archive/reference.md +1 -1
- package/harness/bundles/java/cursor/harness-codebase-map/SKILL.md +4 -2
- package/harness/bundles/java/cursor/harness-knowledge-ingest/SKILL.md +4 -2
- package/harness/bundles/java/cursor/harness-knowledge-query/SKILL.md +4 -2
- package/harness/bundles/java/cursor/harness-package/SKILL.md +5 -2
- package/harness/bundles/java/cursor/harness-package/checklist.md +4 -4
- package/harness/bundles/java/cursor/harness-plan/SKILL.md +8 -4
- package/harness/bundles/java/cursor/harness-plan/checklist.md +5 -5
- package/harness/bundles/java/cursor/harness-review/SKILL.md +5 -2
- package/harness/bundles/java/cursor/harness-run/SKILL.md +7 -3
- package/harness/bundles/java/cursor/harness-run/checklist.md +4 -4
- package/harness/bundles/java/cursor/harness-submit/SKILL.md +6 -3
- package/harness/bundles/java/cursor/harness-submit/checklist.md +5 -5
- package/harness/bundles/java/cursor/harness-submit/reference.md +3 -3
- package/harness/bundles/java/cursor/harness-sync/SKILL.md +4 -2
- package/harness/bundles/java/cursor/harness-sync/reference.md +1 -1
- package/harness/bundles/java/cursor/harness-test/SKILL.md +8 -5
- package/harness/bundles/java/cursor/harness-test/pitfalls.md +1 -1
- package/harness/bundles/java/cursor/harness-test/reference.md +1 -1
- package/harness/bundles/java/cursor/protocols/submit-protocol.md +1 -1
- package/harness/bundles/java/cursor/scripts/harness_deploy.py +7 -0
- package/harness/bundles/java/cursor/scripts/harness_integration.py +26 -2
- package/harness/bundles/java/cursor/scripts/harness_plan_finalize.py +159 -7
- package/harness/bundles/java/cursor/scripts/harness_test_cleanup.py +175 -0
- package/harness/manifests/general/claude-code.json +31 -27
- package/harness/manifests/general/codebuddy.json +31 -27
- package/harness/manifests/general/codex.json +31 -27
- package/harness/manifests/general/cursor.json +31 -27
- package/harness/manifests/java/claude-code.json +33 -29
- package/harness/manifests/java/codebuddy.json +33 -29
- package/harness/manifests/java/codex.json +33 -29
- package/harness/manifests/java/cursor.json +33 -29
- package/hunter-workflow-family.json +2 -2
- package/package.json +1 -1
|
@@ -12,7 +12,7 @@ description: harness-submit 的8 步工作流详细步骤和执行日志记录
|
|
|
12
12
|
- commit message 必须写入 `.harness/changes/<change>/runtime/commit-message.txt`。
|
|
13
13
|
- 使用 `git commit -F`,禁止先英文 commit 再 amend。
|
|
14
14
|
- commit message 禁止 `Co-Authored-By`、`Generated by`、`AI generated`。
|
|
15
|
-
-
|
|
15
|
+
- blocking user confirmation 参数错误时不得自行绕过,必须改用固定模板重问。
|
|
16
16
|
- staged diff 是唯一提交依据。
|
|
17
17
|
|
|
18
18
|
|
|
@@ -130,7 +130,7 @@ powershell.exe -Command "Select-String -Path '<项目路径>\.gitignore' -Patter
|
|
|
130
130
|
|
|
131
131
|
如果未包含:
|
|
132
132
|
- **必须提醒用户**:`⚠️ .gitignore 未包含 .harness/,会导致 .harness/ 下文件被提交(含日志、报告、可能含敏感信息)`
|
|
133
|
-
- **默认不提交 .harness/ 下文件**:使用
|
|
133
|
+
- **默认不提交 .harness/ 下文件**:使用 blocking user confirmation 询问用户是否:
|
|
134
134
|
1. 添加 .harness/ 到 .gitignore(推荐)
|
|
135
135
|
2. 仅 stage 业务代码文件,不提交 .harness/
|
|
136
136
|
3. 取消提交
|
|
@@ -146,7 +146,7 @@ powershell.exe -Command "git -C '<项目路径>' status --porcelain | Select-Str
|
|
|
146
146
|
|
|
147
147
|
**worktree 模式(requested=true)**:固定为「仅本地 commit」,不展示「commit+push」选项;commit 完成后继续下方「worktree 合并流程」,跳过下方三选项询问。
|
|
148
148
|
|
|
149
|
-
`harness-submit` 不调用 Superpowers 呈现合并选项。只允许使用固定
|
|
149
|
+
`harness-submit` 不调用 Superpowers 呈现合并选项。只允许使用固定 blocking user confirmation 模板:
|
|
150
150
|
|
|
151
151
|
```text
|
|
152
152
|
本次变更已完成验证,请选择提交方式:
|
|
@@ -156,7 +156,7 @@ powershell.exe -Command "git -C '<项目路径>' status --porcelain | Select-Str
|
|
|
156
156
|
3. 取消提交
|
|
157
157
|
```
|
|
158
158
|
|
|
159
|
-
|
|
159
|
+
禁止在此步骤引入"创建 PR""丢弃所有变更""直接合并到主分支"等旧选项,避免 blocking user confirmation 参数错误和流程歧义。
|
|
160
160
|
|
|
161
161
|
## 步骤 5:展示提交摘要 + 确认 ⚠️ 强制阻断
|
|
162
162
|
|
|
@@ -256,7 +256,7 @@ powershell.exe -Command "git -C '<项目路径>' log HEAD..@{u} --oneline"
|
|
|
256
256
|
- **不得直接 pull 后 push**
|
|
257
257
|
- 必须 pull/rebase 后**重新构建/测试**(ledger 复用条件被破坏,须重跑并写回 ledger)
|
|
258
258
|
- 验证通过后再 push
|
|
259
|
-
- 使用
|
|
259
|
+
- 使用 blocking user confirmation 询问用户:
|
|
260
260
|
```
|
|
261
261
|
远程有新提交(N 个),需要 pull/rebase 后重新验证:
|
|
262
262
|
1. 重新验证(pull/rebase + 构建 + 测试)
|
|
@@ -4,7 +4,7 @@ description: harness-submit 的提交格式、固定交互模板和中文 commit
|
|
|
4
4
|
|
|
5
5
|
# harness-submit 参考
|
|
6
6
|
|
|
7
|
-
## 提交方式
|
|
7
|
+
## 提交方式 blocking user confirmation 固定模板
|
|
8
8
|
|
|
9
9
|
问题标题:`请选择提交方式`
|
|
10
10
|
|
|
@@ -14,7 +14,7 @@ description: harness-submit 的提交格式、固定交互模板和中文 commit
|
|
|
14
14
|
2. `仅本地 commit,不 push`
|
|
15
15
|
3. `取消提交`
|
|
16
16
|
|
|
17
|
-
不得临场增加复杂对象或嵌套参数,避免
|
|
17
|
+
不得临场增加复杂对象或嵌套参数,避免 blocking user confirmation 参数错误。
|
|
18
18
|
|
|
19
19
|
> **worktree 模式(meta/worktree.json requested=true)**:固定选项 2「仅本地 commit」,不展示「commit+push」;commit 成功后 skill **自动接续** worktree 合并流程(`/harness-merge` 为别名)。push 只在主分支完成,产出 `mergeFinalHash`。
|
|
20
20
|
|
|
@@ -148,5 +148,5 @@ powershell.exe -NoProfile -Command "git -C '<项目路径>' log HEAD..@{u} --one
|
|
|
148
148
|
|
|
149
149
|
- 步骤 1(合并最新代码):按无 upstream / 远端无新提交判定 N/A,在执行日志记录理由。
|
|
150
150
|
- 步骤 5(commit-message.txt + `git commit -F`)N/A:无可暂存 diff,无新 commit 需生成。执行日志标注 `commit-message.txt / git commit -F: N/A(工作树 clean,无新 commit;N 个 wip 提交已存在,原样推送)`。
|
|
151
|
-
- 步骤 4(提交方式选择):固定三选项仍适用。但当 review 标了 RED 兼容性破坏等需人工确认项时,可在"commit + push"选项的描述中注明"并确认 <RED 项>",**不得新增第四选项**、不得临场拼复杂
|
|
151
|
+
- 步骤 4(提交方式选择):固定三选项仍适用。但当 review 标了 RED 兼容性破坏等需人工确认项时,可在"commit + push"选项的描述中注明"并确认 <RED 项>",**不得新增第四选项**、不得临场拼复杂 blocking user confirmation 参数。
|
|
152
152
|
- push 仍按步骤 6:fetch 检查远端 → `git push -u`(无 upstream 则新建远端分支)→ 记录 pre-pull hash + final pushed hash。
|
|
@@ -19,7 +19,7 @@ disallowed-tools:
|
|
|
19
19
|
- Bash(awk *)
|
|
20
20
|
- Bash(curl *)
|
|
21
21
|
---
|
|
22
|
-
<!-- generated by harness_deploy.py; core=
|
|
22
|
+
<!-- generated by harness_deploy.py; core=6e87bc67bd51cbe9; overlay=java; agent=claude-code; do not edit -->
|
|
23
23
|
# harness-sync — 元数据同步
|
|
24
24
|
|
|
25
25
|
## Purpose
|
|
@@ -55,7 +55,7 @@ python <skills-root>/scripts/harness_sync.py finalize --project . --run-id <run-
|
|
|
55
55
|
| 3 | `.harness/codebase/map/` 是否过期 → 报告建议 `/harness-codebase-map` |
|
|
56
56
|
| 3.5 | `harness_knowledge.py sync`;可 `sync --update`;失败不得假装可用。**知识闭环主入口**是 `/harness-knowledge-ingest auto`(含 Agent judge),sync 不重复列人工知识待办 |
|
|
57
57
|
| 3.6 | 单次运行 `harness_knowledge.py maintain --project . --drain --json`,有界推进全部 `.harness/knowledge/maintenance-outbox/{pending,failed}`;不得为每个条目重复启动 Python/重建索引(§8.2:archive close 只 enqueue,sync 异步推进 outbox 到 completed/pending-judge) |
|
|
58
|
-
| 4 | CLAUDE.md 完整性/行数 → 超限
|
|
58
|
+
| 4 | CLAUDE.md 完整性/行数 → 超限 blocking user confirmation 瘦身 |
|
|
59
59
|
| 5 | AGENTS.md 与各 Agent 指令入口一致;Claude Code 启用时验证 CLAUDE.md 引用 AGENTS.md,禁止反向循环引用 |
|
|
60
60
|
| 6 | `.harness/` 结构(init 规程 → `reference.md` 第 6 步);已装 skill 新鲜度只经 `hunter-harness refresh --dry-run --json` 的 post-adaptation freshness 判断,禁止 raw build 字节比较 |
|
|
61
61
|
| 7–9 | `项目规则(见 .harness/context-index.json)/`、构建配置、测试目录 — **只提示不自动修复** |
|
|
@@ -67,6 +67,7 @@ python <skills-root>/scripts/harness_sync.py finalize --project . --run-id <run-
|
|
|
67
67
|
- 命令结果不得靠猜测;普通 Bash 被拒 → 立即改用等价 PowerShell 重试一次
|
|
68
68
|
- 仅 PowerShell 成功且有明确证据(构建/git/测试输出、文件存在、exit 0)时可标 ✅OK;否则 ❌FAIL 或 🟡WARN
|
|
69
69
|
- 禁止把 hook 拒绝、静态验证、无输出、用户跳过说成成功 → 详见 [[../protocols/powershell-protocol.md|powershell-protocol]]、[[../protocols/evidence-based-reporting-protocol.md|evidence-based-reporting-protocol]]
|
|
70
|
+
> 片段:p0-trust
|
|
70
71
|
|
|
71
72
|
## 关键规则
|
|
72
73
|
|
|
@@ -97,3 +98,4 @@ python <skills-root>/scripts/harness_events.py append --change-dir ".harness/cha
|
|
|
97
98
|
> **Task 4 §6.1 写入契约**:普通 `append` = 加锁 -> 追加一行 -> fsync -> 解锁,**不 load 历史、不渲染**(O(1),跨进程锁 `events.ndjson.lock`,UUID 用完整 `uuid4().hex` 无需去重扫描)。仅 `--type phase.end` append 在追加成功后渲染一次 `execution-log.md`;显式 `harness_events.py render` 随时从完整 events 重建;`harness_archive.py finalize` 在 collect 前强制 render 一次。高频 command append 期间 log 可能滞后,phase 边界保持最新。
|
|
98
99
|
|
|
99
100
|
每次阶段结束必须写 `--status OK|WARN|FAIL|BLOCKED`;重试同一阶段时增加 `--attempt <n>`,不得覆盖或伪装成一次执行。跨工具继续执行时写 `--executor-tool <codex|claude-code|codebuddy|cursor>`,并在接棒事件写 `--handoff-from-tool` / `--handoff-reason`;也可由 `HUNTER_HARNESS_TOOL/AGENT/MODEL/RUN_ID` 环境变量统一注入。
|
|
101
|
+
> 片段:logging · phase=`sync`;有未归档变更时写入其 change-dir events
|
|
@@ -58,7 +58,7 @@ powershell.exe -Command "git -C '<项目路径>' diff --stat HEAD~5 2>$null"
|
|
|
58
58
|
|
|
59
59
|
**瘦身策略**:如果超过 200 行,不是简单截断——而是把可独立成篇的细节(如完整编码规范、详细架构图)迁移到 `项目规则(见 .harness/context-index.json)/` 下,CLAUDE.md 保留简洁的索引和关键命令。
|
|
60
60
|
|
|
61
|
-
> ⚠️ CLAUDE.md 需要瘦身时,必须用
|
|
61
|
+
> ⚠️ CLAUDE.md 需要瘦身时,必须用 blocking user confirmation 与用户确认拆分方案后再执行。
|
|
62
62
|
|
|
63
63
|
### 5. AGENTS.md 一致性
|
|
64
64
|
|
|
@@ -22,7 +22,7 @@ disallowed-tools:
|
|
|
22
22
|
- Bash(node *)
|
|
23
23
|
- Bash(codegraph *)
|
|
24
24
|
---
|
|
25
|
-
<!-- generated by harness_deploy.py; core=
|
|
25
|
+
<!-- generated by harness_deploy.py; core=6e87bc67bd51cbe9; overlay=java; agent=claude-code; do not edit -->
|
|
26
26
|
# harness-test — 测试执行
|
|
27
27
|
|
|
28
28
|
## Purpose
|
|
@@ -59,6 +59,7 @@ disallowed-tools:
|
|
|
59
59
|
9. **每个 skill 边界只刷新一次状态快照** — 先运行 `python <skills-root>/scripts/harness_state.py capture --project . --change-dir ".harness/changes/<change-name>" --base <baseCommit> --json`;`changedSegments=[]` 时复用已有 profile/rules/map/knowledge/change/code 指纹,不再重复跑全量 `harness-sync`、知识重建或代码库扫描。只有对应 segment 变化或缺失时才定向刷新;快照不能替代源码读取和验证门禁。
|
|
60
60
|
|
|
61
61
|
状态目录分层:新路径优先,旧路径兼容 → [[../protocols/state-layout-protocol.md|state-layout-protocol]]
|
|
62
|
+
> 片段:read-protocol
|
|
62
63
|
|
|
63
64
|
## Workflow
|
|
64
65
|
|
|
@@ -115,12 +116,13 @@ disallowed-tools:
|
|
|
115
116
|
- **runtime overlay**:`-Dspring.config.additional-location=file:<ascii-abs-path>/application-harness-test.yml`;禁止默认 Edit tracked `application*.yml`
|
|
116
117
|
- **known-good-test-profile**:profile 名、baseURL、healthUrl 写在 build-profile,非 skill 硬编码
|
|
117
118
|
- **多租户请求头**:header 名/值来自 build-profile 的 `httpHeaders` 配置
|
|
118
|
-
- **Service Gate**:`harness_service.py ensure` 返回 `needs-user-decision`(用户进程占端口)时**才**
|
|
119
|
+
- **Service Gate**:`harness_service.py ensure` 返回 `needs-user-decision`(用户进程占端口)时**才** blocking user confirmation
|
|
119
120
|
## P0 执行可信度规则
|
|
120
121
|
|
|
121
122
|
- 命令结果不得靠猜测;普通 Bash 被拒 → 立即改用等价 PowerShell 重试一次
|
|
122
123
|
- 仅 PowerShell 成功且有明确证据(构建/git/测试输出、文件存在、exit 0)时可标 ✅OK;否则 ❌FAIL 或 🟡WARN
|
|
123
124
|
- 禁止把 hook 拒绝、静态验证、无输出、用户跳过说成成功 → 详见 [[../protocols/powershell-protocol.md|powershell-protocol]]、[[../protocols/evidence-based-reporting-protocol.md|evidence-based-reporting-protocol]]
|
|
125
|
+
> 片段:p0-trust
|
|
124
126
|
|
|
125
127
|
## 关键规则(硬门禁速查)
|
|
126
128
|
|
|
@@ -160,11 +162,11 @@ python <skills-root>/scripts/harness_test_guard.py record --project . --change-d
|
|
|
160
162
|
|
|
161
163
|
### 六、服务启动 + 生命周期管理
|
|
162
164
|
|
|
163
|
-
启动等待状态机:0–30s 每 2s 探测、30–120s 每 5s 探测、>120s 读日志判定;遇启动失败特征立即停。**Service Gate**:`harness_service.py ensure` 返回 `action=needs-user-decision`(用户自启服务占端口)时 **才**
|
|
165
|
+
启动等待状态机:0–30s 每 2s 探测、30–120s 每 5s 探测、>120s 读日志判定;遇启动失败特征立即停。**Service Gate**:`harness_service.py ensure` 返回 `action=needs-user-decision`(用户自启服务占端口)时 **才** blocking user confirmation;AI 托管服务或端口空闲则自动继续,不询问。服务指纹(`moduleInputsHash`,来自 CLI `--files` ∪ `serviceStart.inputFiles`)+ `startCommandHash` + `profile` + `overlayPath` + 进程身份任一变化即 restart;**空输入被拒绝**,不生成可复用空指纹。测试结束默认清理 AI 启动的服务。详见 `reference.md`「服务决策门」。
|
|
164
166
|
|
|
165
167
|
### 七、运行时配置叠加(不动 tracked 配置)
|
|
166
168
|
|
|
167
|
-
禁止默认 Edit tracked 应用配置文件。默认运行时配置叠加(ASCII 绝对路径);改 tracked 配置 → 默认拒绝,记 `decision` 事件(不
|
|
169
|
+
禁止默认 Edit tracked 应用配置文件。默认运行时配置叠加(ASCII 绝对路径);改 tracked 配置 → 默认拒绝,记 `decision` 事件(不 blocking user confirmation,报告 🟡 WARN)。详见 `reference.md`。
|
|
168
170
|
|
|
169
171
|
### 八、Token 缓存与复用
|
|
170
172
|
|
|
@@ -196,7 +198,7 @@ python <skills-root>/scripts/harness_test_guard.py record --project . --change-d
|
|
|
196
198
|
|
|
197
199
|
## 交互白名单
|
|
198
200
|
|
|
199
|
-
本 skill **仅允许**以下
|
|
201
|
+
本 skill **仅允许**以下 blocking user confirmation;其余默认值 + `decision` 事件:
|
|
200
202
|
|
|
201
203
|
1. **Service Gate**:仅当 `harness_service.py ensure` 返回 `needs-user-decision`(用户进程占端口)时询问处理方式
|
|
202
204
|
|
|
@@ -213,3 +215,4 @@ python <skills-root>/scripts/harness_events.py append --change-dir ".harness/cha
|
|
|
213
215
|
> **Task 4 §6.1 写入契约**:普通 `append` = 加锁 -> 追加一行 -> fsync -> 解锁,**不 load 历史、不渲染**(O(1),跨进程锁 `events.ndjson.lock`,UUID 用完整 `uuid4().hex` 无需去重扫描)。仅 `--type phase.end` append 在追加成功后渲染一次 `execution-log.md`;显式 `harness_events.py render` 随时从完整 events 重建;`harness_archive.py finalize` 在 collect 前强制 render 一次。高频 command append 期间 log 可能滞后,phase 边界保持最新。
|
|
214
216
|
|
|
215
217
|
每次阶段结束必须写 `--status OK|WARN|FAIL|BLOCKED`;重试同一阶段时增加 `--attempt <n>`,不得覆盖或伪装成一次执行。跨工具继续执行时写 `--executor-tool <codex|claude-code|codebuddy|cursor>`,并在接棒事件写 `--handoff-from-tool` / `--handoff-reason`;也可由 `HUNTER_HARNESS_TOOL/AGENT/MODEL/RUN_ID` 环境变量统一注入。
|
|
218
|
+
> 片段:logging · phase=`test` · 事件:phase/command/verification/decision/issue/artifact
|
|
@@ -191,7 +191,7 @@ $cred = $resp.data.accessToken
|
|
|
191
191
|
**严重度**:🟡WARN
|
|
192
192
|
**场景**:测试期间为了切换外部服务配置直接 Edit tracked 配置文件(如 `application-local-dev.yml`),测试后再还原,留下 diff 噪声
|
|
193
193
|
**后果**:git diff --stat 出现意外文件、提交风险、误以为有业务变更
|
|
194
|
-
**正确做法**:生成运行时配置叠加 `.harness/changes/<change>/runtime/application-harness-test.yml`(不提交),启动用 `-Dspring.config.additional-location=file:...` 叠加(按技术栈);如必须改 tracked 配置,先
|
|
194
|
+
**正确做法**:生成运行时配置叠加 `.harness/changes/<change>/runtime/application-harness-test.yml`(不提交),启动用 `-Dspring.config.additional-location=file:...` 叠加(按技术栈);如必须改 tracked 配置,先 blocking user confirmation,最终报告至少 🟡 WARN
|
|
195
195
|
|
|
196
196
|
### 规则 22:唯一约束字段与本地预存数据冲突
|
|
197
197
|
**严重度**:❌FAIL
|
|
@@ -151,7 +151,7 @@ spring:
|
|
|
151
151
|
powershell.exe -NoProfile -Command "mvn spring-boot:run -pl <module-from-build-profile> -Dspring-boot.run.profiles=local-dev -Dspring-boot.run.jvmArguments='-Dspring.config.additional-location=file:C:/temp/harness-test-overlay/<change-name>/application-harness-test.yml'"
|
|
152
152
|
```
|
|
153
153
|
|
|
154
|
-
不得默认使用 `.harness/changes/<change>/runtime/application-harness-test.yml` 相对路径作为 JVM `additional-location`。如必须修改 tracked yml,先
|
|
154
|
+
不得默认使用 `.harness/changes/<change>/runtime/application-harness-test.yml` 相对路径作为 JVM `additional-location`。如必须修改 tracked yml,先 blocking user confirmation,最终报告至少 🟡 WARN。
|
|
155
155
|
|
|
156
156
|
## Service Decision Gate 与服务生命周期管理
|
|
157
157
|
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
---
|
|
2
|
-
description: harness-submit 的确定性提交协议。用于避免
|
|
2
|
+
description: harness-submit 的确定性提交协议。用于避免 blocking user confirmation 参数错误、中文 commit message 失败、Co-Authored-By 违规和先 commit 后 amend。
|
|
3
3
|
---
|
|
4
4
|
|
|
5
5
|
# Submit Protocol
|
|
@@ -26,6 +26,10 @@ if hasattr(sys.stderr, "reconfigure"):
|
|
|
26
26
|
sys.stderr.reconfigure(encoding="utf-8")
|
|
27
27
|
|
|
28
28
|
INCLUDE_RE = re.compile(r"<!--\s*@include\s+shared/([^\s]+)\s*-->")
|
|
29
|
+
# retro §5.17: wiki links [[shared/xxx.md|alias]] reference shared fragments
|
|
30
|
+
# that are inlined by expand_includes. After expansion, replace the wiki link
|
|
31
|
+
# with the alias text (the shared content is already in the document).
|
|
32
|
+
WIKI_LINK_RE = re.compile(r"\[\[shared/([^\]|]+)\|([^\]]*)\]\]")
|
|
29
33
|
# §7.5: overlay anchors support both new section-id and legacy section:"title".
|
|
30
34
|
OVERRIDE_RE = re.compile(r'<!--\s*@override\s+(?:section-id:"([^"]+)"|section:"([^"]+)")\s*-->')
|
|
31
35
|
APPEND_RE = re.compile(r'<!--\s*@append-after\s+(?:section-id:"([^"]+)"|section:"([^"]+)")\s*-->')
|
|
@@ -148,6 +152,9 @@ def expand_includes(text: str, shared_dir: Path) -> str:
|
|
|
148
152
|
return body
|
|
149
153
|
|
|
150
154
|
expanded = INCLUDE_RE.sub(repl, text)
|
|
155
|
+
# §5.17: replace [[shared/xxx.md|alias]] wiki links with the alias text,
|
|
156
|
+
# since the shared content has been inlined by @include expansion above.
|
|
157
|
+
expanded = WIKI_LINK_RE.sub(lambda m: m.group(2), expanded)
|
|
151
158
|
lines = [ln for ln in expanded.splitlines() if not FRAGMENT_HINT_RE.match(ln)]
|
|
152
159
|
return "\n".join(lines) + ("\n" if text.endswith("\n") else "")
|
|
153
160
|
|
|
@@ -1049,10 +1049,11 @@ class IntegrationTransaction:
|
|
|
1049
1049
|
heavy_removed.append(name)
|
|
1050
1050
|
except OSError as exc:
|
|
1051
1051
|
heavy_failures.append({"path": name, "error": str(exc)})
|
|
1052
|
-
self.runner.run(
|
|
1052
|
+
remove_result = self.runner.run(
|
|
1053
1053
|
self.project_root, "worktree", "remove", "--force", str(resolved),
|
|
1054
1054
|
check=False,
|
|
1055
1055
|
)
|
|
1056
|
+
remove_returncode = getattr(remove_result, "returncode", 0)
|
|
1056
1057
|
listed = self.runner.text(self.project_root, "worktree", "list", "--porcelain")
|
|
1057
1058
|
registered_after = {
|
|
1058
1059
|
str(Path(line.removeprefix("worktree ")).resolve())
|
|
@@ -1061,7 +1062,8 @@ class IntegrationTransaction:
|
|
|
1061
1062
|
}
|
|
1062
1063
|
still_registered = str(resolved) in registered_after
|
|
1063
1064
|
residual_paths: list[str] = []
|
|
1064
|
-
|
|
1065
|
+
disk_path_present = resolved.exists()
|
|
1066
|
+
if disk_path_present:
|
|
1065
1067
|
if resolved.is_file():
|
|
1066
1068
|
residual_paths.append(resolved.name)
|
|
1067
1069
|
else:
|
|
@@ -1075,6 +1077,27 @@ class IntegrationTransaction:
|
|
|
1075
1077
|
break
|
|
1076
1078
|
if len(residual_paths) >= 200:
|
|
1077
1079
|
break
|
|
1080
|
+
# §5.30: Windows half-success — git worktree remove returned non-zero
|
|
1081
|
+
# but registration is already deleted and disk path remains. Return a
|
|
1082
|
+
# structured status distinct from generic CLEANUP_RESIDUAL so callers
|
|
1083
|
+
# can apply the allowlisted residual cleaner.
|
|
1084
|
+
if (
|
|
1085
|
+
remove_returncode != 0
|
|
1086
|
+
and not still_registered
|
|
1087
|
+
and disk_path_present
|
|
1088
|
+
and residual_paths
|
|
1089
|
+
and not heavy_failures
|
|
1090
|
+
):
|
|
1091
|
+
return {
|
|
1092
|
+
"ok": False,
|
|
1093
|
+
"code": "REGISTRATION_REMOVED_RESIDUAL_PRESENT",
|
|
1094
|
+
"target": str(resolved),
|
|
1095
|
+
"heavyRootsRemoved": heavy_removed,
|
|
1096
|
+
"heavyRootFailures": heavy_failures,
|
|
1097
|
+
"worktreeRegistered": False,
|
|
1098
|
+
"diskPathPresent": True,
|
|
1099
|
+
"residualPaths": residual_paths,
|
|
1100
|
+
}
|
|
1078
1101
|
ok = not heavy_failures and not still_registered and not residual_paths
|
|
1079
1102
|
return {
|
|
1080
1103
|
"ok": ok,
|
|
@@ -1083,6 +1106,7 @@ class IntegrationTransaction:
|
|
|
1083
1106
|
"heavyRootsRemoved": heavy_removed,
|
|
1084
1107
|
"heavyRootFailures": heavy_failures,
|
|
1085
1108
|
"worktreeRegistered": still_registered,
|
|
1109
|
+
"diskPathPresent": disk_path_present,
|
|
1086
1110
|
"residualPaths": residual_paths,
|
|
1087
1111
|
}
|
|
1088
1112
|
|
|
@@ -300,6 +300,152 @@ def _terminal_exists(change_dir: Path, run_id: str, attempt: int) -> bool:
|
|
|
300
300
|
)
|
|
301
301
|
|
|
302
302
|
|
|
303
|
+
def verify_plan(change_dir: Path) -> dict[str, Any]:
|
|
304
|
+
"""Read-only verification of a finalized plan (retro §5.8).
|
|
305
|
+
|
|
306
|
+
Validates the published artifacts, receipt and events stream without
|
|
307
|
+
requiring the original staging directory. Returns the artifact hash,
|
|
308
|
+
phase.end count/status, frontmatter, and consistency checks. Any parse
|
|
309
|
+
error or inconsistency results in ok=false with a non-zero exit code.
|
|
310
|
+
"""
|
|
311
|
+
change_dir = change_dir.resolve()
|
|
312
|
+
receipt_path = change_dir / "meta" / "plan-finalization.json"
|
|
313
|
+
if not receipt_path.is_file():
|
|
314
|
+
return _result_error("RECEIPT_MISSING", f"receipt not found: {receipt_path}")
|
|
315
|
+
|
|
316
|
+
try:
|
|
317
|
+
receipt = json.loads(receipt_path.read_text(encoding="utf-8-sig"))
|
|
318
|
+
except (OSError, json.JSONDecodeError) as exc:
|
|
319
|
+
return _result_error("RECEIPT_INVALID", str(exc))
|
|
320
|
+
if not isinstance(receipt, dict):
|
|
321
|
+
return _result_error("RECEIPT_INVALID", "receipt top-level must be an object")
|
|
322
|
+
|
|
323
|
+
change_name = str(receipt.get("changeName") or "").strip()
|
|
324
|
+
if not change_name:
|
|
325
|
+
return _result_error("RECEIPT_INVALID", "receipt missing changeName")
|
|
326
|
+
|
|
327
|
+
expected_hash = str(receipt.get("artifactsHash") or "").strip()
|
|
328
|
+
if not expected_hash.startswith("sha256:"):
|
|
329
|
+
return _result_error("RECEIPT_INVALID", "receipt artifactsHash missing or malformed")
|
|
330
|
+
|
|
331
|
+
files_list = receipt.get("files")
|
|
332
|
+
if not isinstance(files_list, list) or not files_list:
|
|
333
|
+
return _result_error("RECEIPT_INVALID", "receipt files list missing or empty")
|
|
334
|
+
|
|
335
|
+
# Recompute artifacts hash from published files.
|
|
336
|
+
digest = hashlib.sha256()
|
|
337
|
+
artifact_names: list[str] = []
|
|
338
|
+
for rel_text in files_list:
|
|
339
|
+
rel = rel_text.as_posix() if hasattr(rel_text, "as_posix") else str(rel_text)
|
|
340
|
+
target = change_dir / rel
|
|
341
|
+
if not target.is_file():
|
|
342
|
+
return _result_error(
|
|
343
|
+
"ARTIFACT_MISSING", f"published artifact missing: {rel}"
|
|
344
|
+
)
|
|
345
|
+
artifact_names.append(rel)
|
|
346
|
+
digest.update(rel.encode("utf-8"))
|
|
347
|
+
digest.update(b"\0")
|
|
348
|
+
digest.update(target.read_bytes())
|
|
349
|
+
digest.update(b"\0")
|
|
350
|
+
actual_hash = "sha256:" + digest.hexdigest()
|
|
351
|
+
if actual_hash != expected_hash:
|
|
352
|
+
return _result_error(
|
|
353
|
+
"ARTIFACT_HASH_DRIFT",
|
|
354
|
+
f"published artifacts hash {actual_hash} != receipt {expected_hash}",
|
|
355
|
+
)
|
|
356
|
+
|
|
357
|
+
# Validate frontmatter on markdown artifacts.
|
|
358
|
+
frontmatter_results: dict[str, Any] = {}
|
|
359
|
+
for rel in artifact_names:
|
|
360
|
+
if not rel.endswith(".md"):
|
|
361
|
+
continue
|
|
362
|
+
target = change_dir / rel
|
|
363
|
+
try:
|
|
364
|
+
text = target.read_text(encoding="utf-8-sig")
|
|
365
|
+
except OSError as exc:
|
|
366
|
+
return _result_error("ARTIFACT_UNREADABLE", f"{rel}: {exc}")
|
|
367
|
+
fm = _frontmatter(text)
|
|
368
|
+
if fm is None:
|
|
369
|
+
return _result_error(
|
|
370
|
+
"ARTIFACT_FRONTMATTER_MISSING", f"{rel}: no frontmatter"
|
|
371
|
+
)
|
|
372
|
+
if fm.get("change-name") != change_name:
|
|
373
|
+
return _result_error(
|
|
374
|
+
"ARTIFACT_FRONTMATTER_INVALID",
|
|
375
|
+
f"{rel}: change-name '{fm.get('change-name')}' != '{change_name}'",
|
|
376
|
+
)
|
|
377
|
+
frontmatter_results[rel] = {
|
|
378
|
+
"change-name": fm.get("change-name"),
|
|
379
|
+
"status": fm.get("status"),
|
|
380
|
+
}
|
|
381
|
+
|
|
382
|
+
# Validate gate-policy.json is parseable JSON.
|
|
383
|
+
gate_policy_path = change_dir / "meta" / "gate-policy.json"
|
|
384
|
+
gate_policy_consistent = True
|
|
385
|
+
if gate_policy_path.is_file():
|
|
386
|
+
try:
|
|
387
|
+
json.loads(gate_policy_path.read_text(encoding="utf-8-sig"))
|
|
388
|
+
except (OSError, json.JSONDecodeError):
|
|
389
|
+
gate_policy_consistent = False
|
|
390
|
+
else:
|
|
391
|
+
gate_policy_consistent = False
|
|
392
|
+
|
|
393
|
+
# Validate events.ndjson: count phase.end, check for parse errors.
|
|
394
|
+
events_path = change_dir / "events.ndjson"
|
|
395
|
+
phase_end_count = 0
|
|
396
|
+
phase_end_status: str | None = None
|
|
397
|
+
events_parse_errors: list[str] = []
|
|
398
|
+
if events_path.is_file():
|
|
399
|
+
for line_no, line in enumerate(events_path.read_text(encoding="utf-8").splitlines(), 1):
|
|
400
|
+
if not line.strip():
|
|
401
|
+
continue
|
|
402
|
+
try:
|
|
403
|
+
event = json.loads(line)
|
|
404
|
+
except json.JSONDecodeError as exc:
|
|
405
|
+
events_parse_errors.append(f"line {line_no}: {exc}")
|
|
406
|
+
continue
|
|
407
|
+
if not isinstance(event, dict):
|
|
408
|
+
events_parse_errors.append(f"line {line_no}: not an object")
|
|
409
|
+
continue
|
|
410
|
+
if (
|
|
411
|
+
event.get("phase") == "plan"
|
|
412
|
+
and event.get("type") == "phase.end"
|
|
413
|
+
):
|
|
414
|
+
phase_end_count += 1
|
|
415
|
+
phase_end_status = str(event.get("status") or "").upper()
|
|
416
|
+
else:
|
|
417
|
+
events_parse_errors.append("events.ndjson missing")
|
|
418
|
+
|
|
419
|
+
if events_parse_errors:
|
|
420
|
+
return _result_error(
|
|
421
|
+
"EVENTS_PARSE_ERROR", "; ".join(events_parse_errors[:5])
|
|
422
|
+
)
|
|
423
|
+
|
|
424
|
+
if phase_end_count == 0:
|
|
425
|
+
return _result_error("PHASE_END_MISSING", "no plan phase.end event found")
|
|
426
|
+
if phase_end_count > 1:
|
|
427
|
+
return _result_error(
|
|
428
|
+
"PHASE_END_DUPLICATE", f"found {phase_end_count} phase.end events"
|
|
429
|
+
)
|
|
430
|
+
|
|
431
|
+
# Receipt consistency: status should be finalized.
|
|
432
|
+
receipt_consistent = receipt.get("status") == "finalized"
|
|
433
|
+
|
|
434
|
+
return {
|
|
435
|
+
"ok": True,
|
|
436
|
+
"action": "verify",
|
|
437
|
+
"changeDir": str(change_dir),
|
|
438
|
+
"changeName": change_name,
|
|
439
|
+
"artifactsHash": actual_hash,
|
|
440
|
+
"phaseEndCount": phase_end_count,
|
|
441
|
+
"phaseEndStatus": phase_end_status,
|
|
442
|
+
"frontmatter": frontmatter_results,
|
|
443
|
+
"gatePolicyConsistent": gate_policy_consistent,
|
|
444
|
+
"receiptConsistent": receipt_consistent,
|
|
445
|
+
"files": artifact_names,
|
|
446
|
+
}
|
|
447
|
+
|
|
448
|
+
|
|
303
449
|
def _read_design_capabilities(staging: Path, change_name: str) -> list[str]:
|
|
304
450
|
"""Read capabilities from design frontmatter (retro §5.4)."""
|
|
305
451
|
design_path = staging / "spec" / f"{change_name}-design.md"
|
|
@@ -569,18 +715,24 @@ def build_parser() -> argparse.ArgumentParser:
|
|
|
569
715
|
finalize.add_argument("--run-id", required=True)
|
|
570
716
|
finalize.add_argument("--attempt", required=True, type=int)
|
|
571
717
|
finalize.add_argument("--json", action="store_true")
|
|
718
|
+
verify = sub.add_parser("verify")
|
|
719
|
+
verify.add_argument("--change-dir", required=True)
|
|
720
|
+
verify.add_argument("--json", action="store_true")
|
|
572
721
|
return parser
|
|
573
722
|
|
|
574
723
|
|
|
575
724
|
def main(argv: list[str] | None = None) -> int:
|
|
576
725
|
args = build_parser().parse_args(argv)
|
|
577
|
-
|
|
578
|
-
Path(args.change_dir)
|
|
579
|
-
|
|
580
|
-
|
|
581
|
-
|
|
582
|
-
|
|
583
|
-
|
|
726
|
+
if args.command == "verify":
|
|
727
|
+
result = verify_plan(Path(args.change_dir))
|
|
728
|
+
else:
|
|
729
|
+
result = finalize_plan(
|
|
730
|
+
Path(args.change_dir),
|
|
731
|
+
Path(args.staging_dir),
|
|
732
|
+
change_name=args.change,
|
|
733
|
+
run_id=args.run_id,
|
|
734
|
+
attempt=args.attempt,
|
|
735
|
+
)
|
|
584
736
|
stream = sys.stdout if result["ok"] else sys.stderr
|
|
585
737
|
if args.json:
|
|
586
738
|
stream.write(json.dumps(result, ensure_ascii=False, indent=2) + "\n")
|
|
@@ -0,0 +1,175 @@
|
|
|
1
|
+
#!/usr/bin/env python3
|
|
2
|
+
"""Controlled cleanup helper for test fixtures (retro §5.23).
|
|
3
|
+
|
|
4
|
+
Provides a host-policy-friendly entry point for removing gitignored test
|
|
5
|
+
fixtures (e.g. .pytest_data) within an execution root. Performs realpath
|
|
6
|
+
containment, rejects symlink/reparse escapes, lists exact counts before
|
|
7
|
+
deletion, and outputs a structured receipt.
|
|
8
|
+
"""
|
|
9
|
+
from __future__ import annotations
|
|
10
|
+
|
|
11
|
+
import argparse
|
|
12
|
+
import json
|
|
13
|
+
import os
|
|
14
|
+
import shutil
|
|
15
|
+
import sys
|
|
16
|
+
from pathlib import Path
|
|
17
|
+
from typing import Any
|
|
18
|
+
|
|
19
|
+
if hasattr(sys.stdout, "reconfigure"):
|
|
20
|
+
sys.stdout.reconfigure(encoding="utf-8")
|
|
21
|
+
if hasattr(sys.stderr, "reconfigure"):
|
|
22
|
+
sys.stderr.reconfigure(encoding="utf-8")
|
|
23
|
+
|
|
24
|
+
|
|
25
|
+
def _resolve_within(execution_root: Path, rel: str) -> Path | None:
|
|
26
|
+
"""Resolve rel under execution_root, rejecting escapes and symlinks."""
|
|
27
|
+
if not rel or rel.startswith("/"):
|
|
28
|
+
return None
|
|
29
|
+
# Reject absolute paths (Windows drive letter or POSIX).
|
|
30
|
+
if len(rel) >= 2 and rel[1] == ":":
|
|
31
|
+
return None
|
|
32
|
+
parts = rel.replace("\\", "/").split("/")
|
|
33
|
+
if ".." in parts:
|
|
34
|
+
return None
|
|
35
|
+
candidate = (execution_root / rel).resolve(strict=False)
|
|
36
|
+
# Containment check: candidate must be under execution_root.
|
|
37
|
+
try:
|
|
38
|
+
candidate.relative_to(execution_root.resolve(strict=False))
|
|
39
|
+
except ValueError:
|
|
40
|
+
return None
|
|
41
|
+
# Reject symlink/reparse escape: if the path is a symlink, check its target.
|
|
42
|
+
if candidate.is_symlink():
|
|
43
|
+
target = Path(os.readlink(candidate)).resolve(strict=False)
|
|
44
|
+
try:
|
|
45
|
+
target.relative_to(execution_root.resolve(strict=False))
|
|
46
|
+
except ValueError:
|
|
47
|
+
return None
|
|
48
|
+
# Even if target is inside, a symlinked cleanup root is suspicious;
|
|
49
|
+
# reject to avoid following links into unexpected locations.
|
|
50
|
+
return None
|
|
51
|
+
return candidate
|
|
52
|
+
|
|
53
|
+
|
|
54
|
+
def _count_files(root: Path) -> tuple[int, int]:
|
|
55
|
+
"""Count files and total bytes under root."""
|
|
56
|
+
count = 0
|
|
57
|
+
total = 0
|
|
58
|
+
for dirpath, _dirs, files in os.walk(root):
|
|
59
|
+
for name in files:
|
|
60
|
+
fp = Path(dirpath) / name
|
|
61
|
+
try:
|
|
62
|
+
total += fp.stat().st_size
|
|
63
|
+
count += 1
|
|
64
|
+
except OSError:
|
|
65
|
+
continue
|
|
66
|
+
return count, total
|
|
67
|
+
|
|
68
|
+
|
|
69
|
+
def cleanup(execution_root: Path, cleanup_roots: list[str]) -> dict[str, Any]:
|
|
70
|
+
"""Remove allowlisted cleanup roots within execution_root.
|
|
71
|
+
|
|
72
|
+
Returns a structured receipt. Any path escape, symlink escape, or
|
|
73
|
+
unallowed root results in ok=false with zero files removed.
|
|
74
|
+
"""
|
|
75
|
+
execution_root = Path(execution_root).resolve(strict=False)
|
|
76
|
+
if not execution_root.is_dir():
|
|
77
|
+
return {
|
|
78
|
+
"ok": False,
|
|
79
|
+
"code": "EXECUTION_ROOT_MISSING",
|
|
80
|
+
"executionRoot": str(execution_root),
|
|
81
|
+
"removedFiles": 0,
|
|
82
|
+
"removedBytes": 0,
|
|
83
|
+
"rejectedEscapes": [],
|
|
84
|
+
}
|
|
85
|
+
|
|
86
|
+
rejected: list[dict[str, str]] = []
|
|
87
|
+
removed_files = 0
|
|
88
|
+
removed_bytes = 0
|
|
89
|
+
cleaned: list[str] = []
|
|
90
|
+
|
|
91
|
+
for rel in cleanup_roots:
|
|
92
|
+
target = _resolve_within(execution_root, rel)
|
|
93
|
+
if target is None:
|
|
94
|
+
rejected.append({"path": rel, "reason": "PATH_ESCAPE_REJECTED"})
|
|
95
|
+
continue
|
|
96
|
+
if not target.exists():
|
|
97
|
+
# Already absent — idempotent reentry.
|
|
98
|
+
continue
|
|
99
|
+
# Re-check symlink after resolution (the parent might be a symlink).
|
|
100
|
+
if target.is_symlink():
|
|
101
|
+
rejected.append({"path": rel, "reason": "SYMLINK_ESCAPE_REJECTED"})
|
|
102
|
+
continue
|
|
103
|
+
# Count before deletion for the receipt.
|
|
104
|
+
try:
|
|
105
|
+
count, total = _count_files(target)
|
|
106
|
+
except OSError:
|
|
107
|
+
count, total = 0, 0
|
|
108
|
+
# Delete.
|
|
109
|
+
try:
|
|
110
|
+
shutil.rmtree(target)
|
|
111
|
+
except OSError as exc:
|
|
112
|
+
rejected.append({"path": rel, "reason": f"DELETE_FAILED: {exc}"})
|
|
113
|
+
continue
|
|
114
|
+
removed_files += count
|
|
115
|
+
removed_bytes += total
|
|
116
|
+
cleaned.append(rel)
|
|
117
|
+
|
|
118
|
+
if rejected:
|
|
119
|
+
return {
|
|
120
|
+
"ok": False,
|
|
121
|
+
"code": rejected[0]["reason"],
|
|
122
|
+
"executionRoot": str(execution_root),
|
|
123
|
+
"removedFiles": removed_files,
|
|
124
|
+
"removedBytes": removed_bytes,
|
|
125
|
+
"rejectedEscapes": rejected,
|
|
126
|
+
"cleaned": cleaned,
|
|
127
|
+
}
|
|
128
|
+
|
|
129
|
+
if removed_files == 0 and not cleaned:
|
|
130
|
+
return {
|
|
131
|
+
"ok": True,
|
|
132
|
+
"code": "ALREADY_ABSENT",
|
|
133
|
+
"executionRoot": str(execution_root),
|
|
134
|
+
"removedFiles": 0,
|
|
135
|
+
"removedBytes": 0,
|
|
136
|
+
"rejectedEscapes": [],
|
|
137
|
+
"cleaned": [],
|
|
138
|
+
}
|
|
139
|
+
|
|
140
|
+
return {
|
|
141
|
+
"ok": True,
|
|
142
|
+
"code": "CLEANUP_COMPLETE",
|
|
143
|
+
"executionRoot": str(execution_root),
|
|
144
|
+
"removedFiles": removed_files,
|
|
145
|
+
"removedBytes": removed_bytes,
|
|
146
|
+
"rejectedEscapes": [],
|
|
147
|
+
"cleaned": cleaned,
|
|
148
|
+
}
|
|
149
|
+
|
|
150
|
+
|
|
151
|
+
def main(argv: list[str] | None = None) -> int:
|
|
152
|
+
parser = argparse.ArgumentParser(description=__doc__)
|
|
153
|
+
sub = parser.add_subparsers(dest="action", required=True)
|
|
154
|
+
clean_parser = sub.add_parser("cleanup")
|
|
155
|
+
clean_parser.add_argument("--execution-root", required=True)
|
|
156
|
+
clean_parser.add_argument("--cleanup-roots", required=True, help="JSON array of relative paths")
|
|
157
|
+
clean_parser.add_argument("--json", action="store_true")
|
|
158
|
+
args = parser.parse_args(argv)
|
|
159
|
+
|
|
160
|
+
try:
|
|
161
|
+
roots = json.loads(args.cleanup_roots)
|
|
162
|
+
if not isinstance(roots, list):
|
|
163
|
+
raise ValueError("cleanup-roots must be a JSON array")
|
|
164
|
+
except (json.JSONDecodeError, ValueError) as exc:
|
|
165
|
+
result = {"ok": False, "code": "INVALID_CLEANUP_ROOTS", "error": str(exc)}
|
|
166
|
+
print(json.dumps(result, ensure_ascii=False, indent=2))
|
|
167
|
+
return 1
|
|
168
|
+
|
|
169
|
+
result = cleanup(Path(args.execution_root), roots)
|
|
170
|
+
print(json.dumps(result, ensure_ascii=False, indent=2 if args.json else None))
|
|
171
|
+
return 0 if result["ok"] else 1
|
|
172
|
+
|
|
173
|
+
|
|
174
|
+
if __name__ == "__main__":
|
|
175
|
+
raise SystemExit(main())
|