Skip to content

chore(sync): 同步 smoke 断言修复到 main - #5

Merged
graydovee merged 2 commits into
mainfrom
omdsh-dev/dsh-browser
Oct 19, 2026
Merged

graydovee merged 2 commits into
mainfrom
omdsh-dev/dsh-browser

Conversation

@graydovee

Copy link
Copy Markdown
Owner

概述 / Summary

把 omdsh-dev/dsh-browser 承载分支上的 smoke 断言修复同步到 main(承载分支 → main 的常规合入)。

改动内容 / Changes

  • test:smoke 改为断言 dsh 0.2.1-rc.2 的真实 dispose 语义:未认领 inbox 重启时恰好一条 agent/inbox/spliced、outcome: canceled、removedCount: 1
  • 信标探测在多实例并存时优先选择指向本次 host 的应答(本机同时运行 DSH Desktop 时不再误判)
  • 无产品行为、协议、权限或版本变化

测试 / Testing

关联 / Related

agent-loop 在优雅关闭时调用 machine.cancel({ kind: 'disposed' }),不带
keepInbox(只有用户的停止按钮会保留待处理输入),所以未认领的页面快照会被
记成一条 canceled 的 agent/inbox/spliced。原断言假设 dsh 会用 keepInbox 保留,
与 0.2.1-rc.2 的实际行为相反,导致 test:smoke 在 main 上一直失败。

断言改为校验真实契约(恰好一条 canceled splice、removedCount=1);信标探测在
本机已有其他 dsh 占用更早端口时,优先选择指向本次 host 的应答。
content-selection.spec.ts 每个用例都新建 SelectionWatcher 但从未 disarm。
watcher 残留的 selectionchange 监听与 debounce 定时器可能跨到 jsdom 环境销毁
之后才触发,CI 偶发报 unhandled ReferenceError: document is not defined
(run #7 的 clean-install 即因此失败)。改为在 afterEach 统一 dispose。
@graydovee
graydovee merged commit 8a2f20d into main Oct 19, 2026
1 check passed
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.

1 participant