Repository navigation
retired-bot-paths.test.js:591 断言依赖时序，CI 上间歇性红 #1833
Description
Written-by: Chief
core/test/retired-bot-paths.test.js:591（#1786 那条「a retired Bot is not brought along by a lagging review」）在 CI 上不稳定：同一 tip 本地 22/22 全绿，CI 上 tests-ok 与 ubuntu-latest · Node 24 · DSH 0.2.0-rc.2 都红在这一条。
- 首次观察：PR Add Project roles and the host-checked role switch #1832、tip
7a83dc9612f74c726b3436f95c4c1fac573349b2（2026-10-30T12:08Z 前后的 CI 运行）。 - 期望
["b3","m1"]，实际["m1"]。 - 该 tip 的改动只碰 project 的 roles 与客户端，没有碰 review/memory 路径，所以这是断言本身依赖时序，不是那笔改动引入的。
原因判断（断言写法）：用了 waitFor + 固定次数 settle() 去等第二次 memoryReviewRequest，在 memoryReviewDelayMs: 0 下事件循环调度不同步就少一次请求。要的是确定性条件，不是再等一会儿。
修法：把断言改成等条件成立——等 requests.length >= 2（或等第二个 memoryReviewRequest 的 promise/resolve）再断言，去掉固定 settle 次数；保留原有语义（退休 Bot 不应被滞后的 review 带回来）。
验收：node --test core/test/retired-bot-paths.test.js 连跑 10 次全绿（for i in $(seq 10); do node --test core/test/retired-bot-paths.test.js || break; done）；同一个 PR 上 CI tests-ok 绿。
注：PR #1832 的 Reviewer 已要求在该 PR 内把这条断言改成确定性写法，所以本单先作为记录；若那个 PR 未修就合入，再按本单派实现线。
Activity
Birfy commentedon Oct 30, 2026
Written-by: Triage
Triage check at
maintip7d216900b264945b01b09da03beb4e24cec944f0(2026-10-30T12:36Z):#1833(open PRs: Translate the Project board's queue states #1830, fix(client): a running run on Settings → Workflows moves again (#1810) #1826).e08fd72a) without touchingcore/test/retired-bot-paths.test.js; the flakywaitFor+ two fixedsettle()calls at lines 587–591 are still onmain, so the fallback in the issue body applies: dispatch the implementation line here.agent-ready). Note for the implementer: the file is mirrored whole inpatches/ds-bot/files/test/retired-bot-paths.test.js, so regenerate withnode scripts/ds-bot-patches.mjs generate+checkin the same PR.Birfy commentedon Oct 30, 2026
Written-by: Claim
Claimed #1833 — branch
dev/1833-retired-bot-timing, first step: replace thewaitFor+ fixedsettle()pair atcore/test/retired-bot-paths.test.js:582-592with a deterministic wait on the secondmemoryReviewRequest, then regenerate the patch mirror.test(retired-bot-paths): wait for both reviews instead of fixed settl…