Skip to content

fix(acp): preserve cancellation and scope transport failures - #186

Open
YoungSx wants to merge 16 commits into
mainfrom
fix/acp-cancellation-error-ownership
Open

fix(acp): preserve cancellation and scope transport failures#186
YoungSx wants to merge 16 commits into
mainfrom
fix/acp-cancellation-error-ownership

Conversation

@YoungSx

@YoungSx YoungSx commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

取消请求时,网络适配器不再把调用方取消误报为发送故障;取消通知的临时发送失败只记诊断,真实断连、读流故障和进程退出仍清理请求并上报。

请求、取消通知和取消过程中发出的回复都绑定原连接。排队写入、延迟返回、重连及 Dispose 不会把旧请求或回复送到新连接。权限与表单响应先核对原请求和选项再取得发送权;发送失败可重试,subscriber 抛错不会二次回应或误删重用 ID 的新请求。

权限界面使用原 SDK callback,并按会话保存提示;旧命令、排队事件、迟到 dismiss 只作用于原请求。保留已发布 public 构造、Respond 及旧 transport 的二进制兼容。WASM 门禁等待原生首帧后一次启用语义树,保留真实鼠标验收。

验证(当前提交 207725cc):

  • SDK 603/603;格式、分析器、Release 构建和以 1.0.0 为基线的包兼容验证通过,零警告、零错误。
  • 真实 nupkg 外部消费、1.0.0 预编译 transport、37 个草案类型的编译限制与两种抑制方式通过。
  • 真实 stdio、WebSocket、HTTP/2、SSE 与错误归属门禁 6/6,零跳过。
  • 共享 UI 提交的完整 Presentation 套件 3360/3360;连接和权限探针均有修复前红灯。各平台 CI 按本次提交重新核验。

Refs #148。本 PR 不关闭总单:生产 stdio-to-WebSocket bridge 缺实际部署验收,Windows GUI 条件跳过不代表真机通过。

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
salmonegg Ready Ready Preview Sep 9, 2026 3:34pm UTC

Wait for the native canvas and splash removal within the existing boot deadline before enabling accessibility once. Uno 6.7.103 (cd452e3ee242c9a2a8dee5e65ad8088b6f5dbf83) can retain initial sibling bounds when ancestor layout moves after early semantic activation, causing pointer gates to click an adjacent card.

Validated against the same Debug product with CPU throttle 4: early activation fails, late activation passes, restoring early fails, restoring late passes. The unchanged two-round pointer gate passes with the final helper at CPU rates 4 and 1. Product focus code is unchanged.
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