Skip to content

test: repair root smoke harness - #16

Merged
ranxianglei merged 1 commit into
ranxianglei:masterfrom
5258MF:2026-08-13_fix-smoke-harness
Aug 13, 2026
Merged

test: repair root smoke harness#16
ranxianglei merged 1 commit into
ranxianglei:masterfrom
5258MF:2026-08-13_fix-smoke-harness

Conversation

@5258MF

@5258MF 5258MF commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

问题

仓库文档要求在 build 后运行 node smoke.mjs,但 root smoke 仍导入重构前的 ./dist/index.js;当前 monorepo 实际产物位于 packages/billion-context-opencode/dist/index.js,因此 smoke 在加载插件前就失败。

路径修复后还暴露出两个 harness 问题:

  • 启动时递归删除整个 ~/.cache/opencode-bili-acp,会误删用户其他会话的压缩状态。
  • 压缩边界强依赖 assistant 可见 ref;PR fix(v1): normalize ref tag placement #12 有意让 V1 assistant 不携带 ACP 标签,因此旧 smoke 与该行为不兼容。

修改

  • 从实际 monorepo bundle 路径加载插件。
  • 使用唯一 smoke session 对应的状态文件,并只清理该文件,不再删除共享 ACP cache。
  • 压缩边界优先使用 assistant ref;若 assistant ref 不可见,则使用下一个 user ref。这样当前 master 与 PR fix(v1): normalize ref tag placement #12 合并后的行为都可运行。
  • 新增匹配分支的 REQ/WORKLOG;不修改任何 packages/ 产品源码、公共 API 或持久化 schema。

验证

  • npm run typecheck:PASS
  • npm test:26/26 PASS
  • npm run build:PASS
  • node smoke.mjs:PASS(当前 master,assistant ref 路径)
  • 临时叠加 PR fix(v1): normalize ref tag placement #12 后 build + smoke:PASS(user ref fallback 路径)
  • PR validation:PASS
  • git diff --check:PASS

范围

该 PR 只修复 root smoke harness。assistant 必须不带标签的严格行为断言仍由 PR #12 的 converter 单测负责,本 PR 不重复定义产品行为。

@ranxianglei
ranxianglei merged commit 290632f into ranxianglei:master Aug 13, 2026
3 checks passed
@ranxianglei

Copy link
Copy Markdown
Owner

前面的我尽可能的都合并了,新进来再修可能的问题

@ranxianglei ranxianglei mentioned this pull request Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants