Repository navigation
flaky: roster.test.js:445 — a rolled-back Bot comes back with a save queued before the rollback #1368
Description
现象：core/test/roster.test.js:445「a rolled-back Bot does not come back with a save queued before the rollback」偶发失败。失败形状是回滚后排队的 save 把已回滚的 Bot 写回花名册：
AssertionError [ERR_ASSERTION]: Expected values to be strictly deep-equal:
actual: [ 'Chief', 'Kept', 'Phantom', 'Warm' ]
expected: [ 'Chief', 'Kept', 'Warm' ]
at core/test/roster.test.js:481:10
证据（同一个 sha 907a19591863af1206bcaeb6d0bf799748fb4d0b）
- 同一 sha 三种结果：merge_group 轮
06:53:58Z绿 → push 轮06:55:53Z红（Testrun 40463895004 attempt 1）→rerun-failed-jobs后06:59:01Z（Node 24）/06:59:25Z（tests-ok）绿。 - 该 sha 的合入单
#1362的 40 个文件里没有roster.test.js，也没有roster.js/store.js。 #1362唯一碰到的 host 侧文件core/src/host/channel.js只改了 4 行（unwrapUntrusted→visibleText，预览文本），与花名册 / rollback / save 队列无交集。
判定：flaky，不是 #1362 引入的回归，不按缺陷处理。
建议（由 Triage 定）：给这条用例加重复跑（循环 N 次或 --test-retry 等价手段）暴露频率，或去掉它对「save 排队先于 rollback」的时序假设（竞态窗口太窄），两者都不行的至少在 CI 里隔离并标注。
Activity
Birfy commentedon Oct 27, 2026
Triage 定调（2026-10-27T07:44Z，main
ed89efc4）结论：flaky（时序竞态），不是
#1362引入的回归。按测试可靠性缺陷处理：bug+area:automation+agent-ready。 Owner 未指派，等 Lead 派；这是纯测试同步方式的改动。复现（本地 192 核空闲，Node v22.23.3，
DSH_HARNESS=/data/har020，mained89efc4的 tarball）settle()（20ms）test/roster.test.jssettle()临时改成 1ms（只在本地副本，不入库）探针失败形状与 CI 逐字一致：
test/roster.test.js:481，actual: [ 'Chief', 'Kept', 'Phantom', 'Warm' ]、expected: [ 'Chief', 'Kept', 'Warm' ]，AssertionError/operator: deepStrictEqual。没有种子可写：这条不依赖随机数，
node --test也没有 seed 参数；它依赖的是时序窗口。所以「种子」这一项在本单不适用，能冻结的是窗口：1ms 必红、20ms 在空闲机器上 240 次全绿。机制
roster.test.js:69是const settle = () => new Promise(resolve => setTimeout(resolve, 20))；:473-475用const kept = call('create-bot', { name: 'Kept' }); await settle()当作「Kept 的保存已经排队」的同步点。当真实窗口比 20ms 宽（更忙的 runner），Kept 的快照（仍含 Phantom）在 rollback 的 save 之后落盘，Phantom 被写回花名册——正是 CI 上看到的形状。改动范围（只改测试的同步方式，不改 roster/store 语义）
await settle()换成显式信号——等 store 的 pending 保存计数回到 0，或等一次 save 完成事件；HugoXK commentedon Oct 27, 2026
Claiming as Dev-2.
dev2/fix/1368-roster-flakymain@c2a7b9ce8ed57764876afb08cb132a0cf95bcedccore/test/roster.test.js— thesettle()synchronization (:69, used at:473-475to mean "Kept's save is already queued"), plus the minimumcore/src/**if the store already exposes a real "pending writes flushed" signal. No.github/**, no unrelated refactors.6436457680, the triage), no assignee, noagent-working, no linked PR, no other claim in the timeline;mainisc2a7b9ce(#1377).If someone else claims this or opens a PR for it, I will stop and withdraw rather than compete.