From 6bc836c0c0bd5a0b40a3fc8c4a3ff41a4e87c7df Mon Sep 17 00:00:00 2001 From: Tianyi Cui <53024+tianyicui@users.noreply.github.com> Date: Wed, 9 Sep 2026 13:18:08 +0800 Subject: [PATCH] docs(ci): distinguish fixture controls from completion policy --- .../2026-09-08-ci-completion-observations.i18n.yaml | 4 ++-- .../testing/2026-09-08-ci-completion-observations.md | 10 ++++++---- .../2026-09-08-ci-completion-observations.zh.md | 12 +++++++----- 3 files changed, 15 insertions(+), 11 deletions(-) diff --git a/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.i18n.yaml b/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.i18n.yaml index c72d961baf..c5a83678cc 100644 --- a/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.i18n.yaml +++ b/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.i18n.yaml @@ -2,5 +2,5 @@ # side as of the last confirmed-consistent state. Both languages carry equal authority; # after editing either side, bring the other along and re-record with: # pnpm run verify-translation-pairing --write .agents/notes/implemented/testing/2026-09-08-ci-completion-observations.md -2026-09-08-ci-completion-observations.md: 6ecf7eb477ac5446df61bb68f6d49870464d67c4 -2026-09-08-ci-completion-observations.zh.md: 9cd1f2b741aa828219559e499cc46eb93b1349df +2026-09-08-ci-completion-observations.md: 8af4685a5e2c51c1edf4a41b47088916223e7a6c +2026-09-08-ci-completion-observations.zh.md: e3147bb7ac39877b46820862e9af4645803a37a2 diff --git a/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.md b/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.md index 6ecf7eb477..8af4685a5e 100644 --- a/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.md +++ b/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.md @@ -6,7 +6,7 @@ English | [中文](2026-09-08-ci-completion-observations.zh.md) ## Problem -The [reference CI run](https://github.com/deepseek-harness/deepseek-harness/actions/runs/34206953049) reports a webhook-created Session absent after a one-second poll and empty PowerShell output before a five-second read deadline. HTTP acceptance, projected UI state, process startup, and durable completion are separate observations. Tests need an explicit completion condition and controls that prevent an intermediate state from satisfying it. +The [reference CI run](https://github.com/deepseek-harness/deepseek-harness/actions/runs/34206953049) reports a webhook-created Session absent after a one-second poll and empty PowerShell output before a five-second read deadline. HTTP acceptance, projected UI state, process startup, and durable completion are separate observations. Tests need an explicit completion condition and controls that prevent an intermediate state from satisfying it. The [completion-wait decision](2026-09-08-ci-readiness-and-completion.md) owns those conditions and lane budgets; these fixtures make their ordering and cleanup observable under controlled delays. ## Decision @@ -34,9 +34,11 @@ The [Node import sweep](../../../../packages/experimental/webworker-runtime/test ## Alternatives considered -- Increase production timeouts, add retries, or serialize the suite: none establishes the missing completion observation. -- Infer completion from HTTP 202, an optimistic image, or a projected label: each can precede the operation being asserted. -- Replace real worker timing coverage with controlled samples: that would omit verification of Node's actual ELU and transport behavior. +**Production timeouts, retries, or suite serialization.** Rejected because none establishes the missing completion observation. + +**Completion inferred from acceptance or a preview.** HTTP 202 and an optimistic image can precede the operation being asserted. + +**Controlled samples replacing measured worker coverage.** Rejected because they omit verification of Node's actual ELU and transport behavior. ## Consequences diff --git a/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.zh.md b/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.zh.md index 9cd1f2b741..e3147bb7ac 100644 --- a/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.zh.md +++ b/.agents/notes/implemented/testing/2026-09-08-ci-completion-observations.zh.md @@ -1,4 +1,4 @@ -# Agent Note：CI fixture 的完成与隔离 +# Agent Note: CI fixture 的完成与隔离 Status: implemented @@ -6,7 +6,7 @@ Status: implemented ## 问题 -[参考 CI 运行](https://github.com/deepseek-harness/deepseek-harness/actions/runs/34206953049)报告：轮询一秒后 webhook 创建的 Session 仍不存在，五秒读取期限内 PowerShell 输出为空。HTTP 接受、UI 投影状态、进程启动和持久化完成是不同的观察。测试需要明确的完成条件，并用对照阻止中间状态满足该条件。 +[参考 CI 运行](https://github.com/deepseek-harness/deepseek-harness/actions/runs/34206953049)报告：轮询一秒后 webhook 创建的 Session 仍不存在，五秒读取期限内 PowerShell 输出为空。HTTP 接受、UI 投影状态、进程启动和持久化完成是不同的观察。测试需要明确的完成条件，并用对照阻止中间状态满足该条件。[完成等待决策](2026-09-08-ci-readiness-and-completion.zh.md)拥有这些条件与 lane 预算；这些 fixture 通过受控延迟使顺序与清理可观察。 ## 决策 @@ -34,9 +34,11 @@ Status: implemented ## 考虑过的替代方案 -- 增加生产超时、增加重试或串行化整个套件：均不能建立缺少的完成观察。 -- 从 HTTP 202、乐观图片或投影标签推断完成：它们均可能早于被断言的操作。 -- 用受控样本替换真实 worker 计时覆盖：会遗漏对 Node 实际 ELU 与传输行为的验证。 +**生产超时、重试或套件串行化。** 拒绝，因为均不能建立缺少的完成观察。 + +**从接受或预览推断完成。** HTTP 202 和乐观图片可能早于被断言的操作。 + +**用受控样本替换实测 worker 覆盖。** 拒绝，因为会遗漏对 Node 实际 ELU 与传输行为的验证。 ## 影响