From d8988ad13add88a790fbcbaa093d7a7bb68eb4c6 Mon Sep 17 00:00:00 2001 From: weichen Date: Wed, 21 Oct 2026 12:27:23 +0000 Subject: [PATCH] Record diagnostics scope, review PASS and PG-backed verification Why: the test-hygiene change needs its scope, the review verdict and the real-PostgreSQL evidence recorded before delivery. Evidence: /tmp/mi-diag-impl-r1-20261021.json: cursor, success (single file) /tmp/mi-diag-review-r1-20261021.json: pi-glm, success (PASS, 6 non-blocking) Real-PG runs: idle 15.1s and 96-load 17.4s, both ok (fsync=on cluster on 127.0.0.1:54331); the Backend fix is already in main via PR #237/#674. Impact: card metadata only; no behaviour change. Refs: PR #679; fix/multi-instance-test-failure-diagnostics --- .task/fix-multi-instance-test-failure-diagnostics.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/.task/fix-multi-instance-test-failure-diagnostics.md b/.task/fix-multi-instance-test-failure-diagnostics.md index b92ac554..250dc378 100644 --- a/.task/fix-multi-instance-test-failure-diagnostics.md +++ b/.task/fix-multi-instance-test-failure-diagnostics.md @@ -7,7 +7,7 @@ - 分支类型：standalone；root_target：main；immediate_target：main；accepting_requirements：false - 需求分流：verdict=NEW；selected_initiative=无；requirement_sha256=e2453ebdabc4cc9c61bb25a3535fbe7cf0d0fe687f2ea7bc565b799dcf0fcb6d；triage_receipt=/tmp/triage-mi-test-hygiene-20261021.json；receipt_sha256=4f0a610213a5f6363e303e7129cd09a630ae1a04acbaa5116635c804b2f7f56c -- 授权与交付终点：<本次允许的动作与终点；未授权事项写明> +- 授权与交付终点：修复 `tests/integration/multi-instance` 两处测试卫生缺陷（轮询静默超时、断言早于恢复视图），让失败直指真正原因。不得改 Backend 或任何生产代码（其根因已由 Backend PR #237 修复）、不得弱化断言、不得放大超时预算。 - 交付模式：goal-first（目标、范围和交付终点一次确认；范围内连续推进） - 实施执行：agent-per-unit（配置：AGENT-EXECUTION.md） - 合入执行：agent（已授权交付范围内，Review 通过、冲突解决且当前 HEAD 的 CI 通过后合入） @@ -39,14 +39,14 @@ ## 结果与验证（完成后填写） -- 结果： -- 验证：<结论摘要 PASS/FAIL/SKIP/NOT_RUN 逐项；逐条证据由 make evidence 写入上方证据表> -- 影响与剩余项： +- 结果：owner_crash_takeover 的 interrupted 轮询改为有界等待 + 显式失败（点名未观察到的 run 状态序列、轮询实例、实际耗时与预算、实例日志），未收口时不再落到后续 POST 的误导性 409；restart_invariants 在断言消息数之前新增「恢复视图」有界等待（存在 interrupted run 且无 pending/running/waiting_hitl），未恢复时给出点名判据的失败信息；其余 `t.Fatal`→`t.Fatalf` 仅补失败信息，断言条件未变。新增 `runStatuses` 与 `e.instanceOf` 两个小helper 供等待条件与失败信息复用。 +- 验证：PASS——真实 PostgreSQL（127.0.0.1:54331，fsync=on）下空闲机器 15.1s 通过、96 忙循环压满 32 核 17.4s 通过（新增失败路径未误触发）；`gofmt`/`go vet` 干净；独立 Review（pi-glm，PASS 无阻塞项）报告 `.task/reviews/2026-10-21-mi-diagnostics-review.md`，6 项非阻塞发现中第 1 项如实指出恢复视图判据的 interrupted 半边由历史 run 平凡满足（不检出 B 重启对账回归，但改动前同样不检出，覆盖无回退）。 +- 影响与剩余项：不改生产代码、不弱化任何断言、未放大任何超时预算、包级 `go test -timeout` 未动。Review 其余非阻塞项（busy 白名单对新增非终态 status 不设防、轮询只附 logA、`instanceOf` 守卫不可达、顺序耦合、瞬态 5xx 即失败）均为既有共享弱点或死代码，不阻塞。 ## 合入与清理 -- Review 记录：NOT_RUN（审阅智能体/平台身份、base/head、结论、问题闭合及重放依据） -- 冲突记录：NOT_RUN（最新目标同步、冲突处置、组合兼容核验及复审/补测） +- Review 记录：PASS（独立会话 pi-glm / zai-coding-cn/glm-5.3-flash；base 5360884e9283d962caf945d944df05b5e5059177，head 232477b22f997bc4ab87b5983ae0b350f6ae4c43；报告 `.task/reviews/2026-10-21-mi-diagnostics-review.md`；无阻塞项，6 项非阻塞发现；重放依据：host 用真实 PG 在空闲与 96 负载两种条件下各跑一次多实例验收，均通过） +- 冲突记录：NOT_RUN（一次交付前同步，0 提交 behind） - CI 记录：NOT_RUN（当前 HEAD 完整 SHA、全部必需检查、运行链接、执行身份及 GATE 结果） - 合入记录：NOT_RUN（交付授权范围、PR、被核验 HEAD、合入 SHA、目标分支及内容核验） - 清理记录：NOT_RUN（授权依据、写者退出确认、worktree/分支/任务卡处置）