master red: export-cmd.test.ts asserts pre-#377 panel format ("1 block") — cross-PR regression between #272 and #377

[ranxianglei]

现象

合并后的 master 上 tests/export-cmd.test.ts 7/10 失败：

AssertionError: compress created a block

PR #380 的 CI 失败即继承自此(master 红)，不是 #380 自身问题。

根因

• feat: add /acp-export command (session handoff export) #272(/acp-export)的 setupSession 断言压缩面板匹配旧格式 /1 block/(tests/export-cmd.test.ts:89)
• feat: report new block ids and actual ref spans in compress result panel (#376) #377(compress 结果带新块 ID 与 ref 跨度)把面板改为 blocks: b1=m00002 格式(单块、startRef===endRef 时省略区间右端)
• feat: report new block ids and actual ref spans in compress result panel (#376) #377 分支基于 feat: add /acp-export command (session handoff export) #272 合并前的 master,其 CI 从未运行过 export-cmd.test.ts;feat: add /acp-export command (session handoff export) #272 的 CI 也不含 feat: report new block ids and actual ref spans in compress result panel (#376) #377 的新格式 → 两个各自全绿的 PR 合并后交叉回归

实测(master):

▣ ACP | 8.0K → 6.2K tokens (~1.8K reclaimed, blocks: b1=m00002)

修复

tests/export-cmd.test.ts:89 断言改为新格式 /blocks: b1=m00002/。

教训(流程)

顺序合并的 PR 彼此看不到对方新增的测试;面板格式类变更合并后应跑一次 master 全量(或后续考虑 merge-queue)。

[ranxianglei]

🤖 Powered by ework · qwen3.8-27b

[bot] 🏷 已修复,PR: #393

审核三步(bug triage):

1. 复现 ✅ 在 origin/master(2bdc831,已含 feat: add /acp-export command (session handoff export) #272 + feat: report new block ids and actual ref spans in compress result panel (#376) #377)上跑 tests/export-cmd.test.ts:7/10 失败,错误均为 AssertionError: compress created a block,实际面板输出 ▣ ACP | 8.0K → 6.2K tokens (~1.8K reclaimed, blocks: b1=m00002) —— 与你实测完全一致。7 个失败全部出自同一处:setupSession 的断言行(第 89 行)。
2. 层次判断 ✅ 不是功能缺陷——压缩管线本身工作正常(block 成功创建、span 标签正确),只是 feat: add /acp-export command (session handoff export) #272 的测试断言仍钉在旧面板格式上。根因与你分析一致:feat: report new block ids and actual ref spans in compress result panel (#376) #377 把面板改为 blocks: b1=m00002 形式(src/compress-tool.ts:206-231 的 blockSpanLabel,单块且 startRef===endRef 时省略区间右端),而 feat: report new block ids and actual ref spans in compress result panel (#376) #377 分支基于 feat: add /acp-export command (session handoff export) #272 合并前的 master,两个 PR 的 CI 各自全绿、合并后交叉回归。feat: report new block ids and actual ref spans in compress result panel (#376) #377 的 diff 也证实它更新了 compress-tool/integration/view-recount/decompress-cmd 四个测试文件,唯独漏了当时尚不存在的 export-cmd。
3. 方案评估 ✅ 你给的修复最小且打到核心。另做了一处补充检查:测试里其余 "N blocks)" 旧格式字符串(compress-tool/floor-stale/growth-scale-flip/nudge-reinject/state-rebuild/sent-view-arbitration/compress-retry)是有意保留的 fixture,用于验证旧格式解析路径(src/compress-tool.ts:154-156,历史 transcript 回放兼容),不能动;真正 stale 的断言只有 export-cmd.test.ts:89 这一处。

改动:一行——tests/export-cmd.test.ts:89 由 /1 block/ 改为 /blocks: b1=m00002/。

完整验证:修复后 export-cmd 10/10 绿;master 全量套件 722 pass / 0 fail / 3 skip;typecheck + build 通过。

关于流程教训:同意。顺序合并下这类交叉回归只能靠合并后跑一次全量或 merge-queue 兜底,本次已在分支上跑过全量确认绿。

中文摘要:修复 #272/#377 顺序合并后 export-cmd.test.ts 仍断言旧面板格式("1 block")导致 master 7/10 红的问题,断言改为新格式 blocks: b1=m00002,全量测试已验证为绿,可以合并。