tests/export-cmd.test.ts: 7 subtests fail on master (compress-result assertion "/1 block/" stale after #376/#377 panel change)

[ranxianglei]

来源: 分析 PR #380(feat(host) turn-boundary)rebase 到 master 时发现。

现象

tests/export-cmd.test.ts 有 7 个子测试失败,全部经由共享 helper setupSession()(该文件第 89 行)抛出同一断言错误:

error: 'compress created a block'
code: 'ERR_ASSERTION'
operator: 'match'
actual: '▣ ACP | 8.0K → 6.2K tokens (~1.8K reclaimed, blocks: b1=m00002)'

断言是 assert.match(text, /1 block/, "compress created a block"),期望 compress 工具返回文本含 "1 block",实际拿到的是新的 ACP 状态面板串(blocks: b1=m00002)。

失败的 7 个子测试:

• listSessions returns ACP-managed sessions with id, title, and block count
• exportSession with no selector lists persisted sessions
• exportSession renders the folded view (summary in place of the compressed range)
• exportSession --full renders the original messages
• exportSession --output writes the markdown file
• exportSession throws when no session matches the selector
• /acp-export command is registered and lists, folds, and expands

已在 master 上复现(非 #380 引入)

在干净的 worktree 检出 origin/master(2bdc831) 单独跑 node --import tsx --test tests/export-cmd.test.ts:同样 3 pass / 7 fail,报错逐字一致。即这些失败在 master 本身就已存在,与 PR #380 的改动无关(#380 未触碰 export 相关文件)。

环境:acp-kernel 固定 0.0.63(与已安装一致)、node v22;本地全量套件 767 tests / 757 pass / 7 fail / 3 skip。

根因(推断)

compress 结果面板的文案被改成了「报告新 block id + 真实 ref span」的形式(blocks: bN=<ref>),来自 #376/#377("report new block ids and actual ref spans in compress result panel");而 #272 的 acp-export 测试仍按旧措辞断言 /1 block/。两处特性对 compress 返回格式的假设漂移了,面板格式变更时没有同步更新这个测试。

影响

• master CI 在这个文件上是红的。
• 任何 rebase 到 master 的 PR 都会把这 7 个显示为失败(与各自改动无关),干扰 PR review 与 CI 门禁判断。

建议修复

把 tests/export-cmd.test.ts:89 的断言改成匹配当前 compress 结果格式(例如断言新建的 block id b1=,或一个稳定的子串),或让 setupSession() 对面板措辞保持稳健。归属 acp-export(#272)/ compress-result-panel(#376/#377)的维护者。

注:此修复有意未并入 PR #380(与 turn-boundary/host 无关,保持该 PR diff 收敛)。

[ranxianglei]

🤖 Powered by ework · qwen3.8-27b

[bot] 🏷 已修复并提 PR:#390

审核结论(动手前的验证)

1. 复现:在干净的 origin/master(2bdc831)上单独跑 node --import tsx --test tests/export-cmd.test.ts,结果 3 pass / 7 fail,报错与 issue 逐字一致(actual: '▣ ACP | 8.0K → 6.2K tokens (~1.8K reclaimed, blocks: b1=m00002)')。确认失败在 master 本身存在,非 feat(host): unified turn boundary + child-session state derivation (#364, replaces #366) #380 引入。
2. 层次判断:这不是 acp-export(feat: add /acp-export command (session handoff export) #272)的功能 bug,而是测试断言与生产面板格式的词面耦合漂移。setupSession()(tests/export-cmd.test.ts:89)用正则 /1 block/ 去匹配 compress 的返回文本,而 compress 工具结果缺少新块 ID 与实际 ref 跨度,模型块账本随会话漂移 #376/feat: report new block ids and actual ref spans in compress result panel (#376) #377 把成功面板从旧计数形式(… 1 block))改成了 span 形式(blocks: bN=<ref>)。断言的本意是「compress 确实创建了 ≥1 个 block」——这是语义,不是词面。
3. 方案评估:issue 建议改成断言 b1= 或某个稳定子串,可行但仍是词面耦合,下次面板再变还会漂。我采用了更稳健的方案:改用生产代码里现成的语义谓词 isCompressSuccessText()(src/compress-tool.ts:174,compress 重试循环判定成功用的就是它,compress-retry / compress-loop-breaker / compress-tool 三个测试文件也已在用)。它同时兼容新旧两种面板格式,把 fixture 从面板措辞中彻底解耦。

附带排查:全仓 grep 过其他 "N blocks" 字面量(compress-retry、sent-view-arbitration、nudge-reinject、floor-stale、growth-scale-flip 等),那些都是喂给 compressPanelBlocks 解析器的输入 fixture(该函数有意同时支持两种格式,用于回放历史 transcript),不是对实时输出的断言,无其他同类陈旧断言。

验证

• 修复后:tests/export-cmd.test.ts 10/10 pass;全量 npm test 722 pass / 0 fail / 3 skip;npm run typecheck 干净。
• diff 收敛:单文件 2 行改动(import + 断言替换)。

一句话:修复了 export 测试因 #376/#377 面板改版导致的陈旧断言(改用生产语义谓词 isCompressSuccessText,不再耦合面板措辞),master CI 转绿,可以合并(请人工合并)。

[ranxianglei]

本 issue 与 #391 完全重复:同一缺陷——tests/export-cmd.test.ts 仍按旧面板格式断言 /1 block/,而 #377 已把 compress 结果面板改成 blocks: bN=<ref>;同一根因(#272 × #377 交叉回归,master 红)。

修复已由 #392(Fixes #391)跟踪。故关闭本 issue,以 #391 / #392 为准,避免同一问题双开。