Skip to content

docs(acp): clarify SDK support boundaries - #183

Merged
YoungSx merged 1 commit into
mainfrom
docs/acp-sdk-support-boundaries
Sep 8, 2026
Merged

docs(acp): clarify SDK support boundaries#183
YoungSx merged 1 commit into
mainfrom
docs/acp-sdk-support-boundaries

Conversation

@YoungSx

@YoungSx YoungSx commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

ACP SDK 的公开 DTO 容易被误认为已经完成产品能力。README 新增能力边界表,说明鉴权、请求取消、form/URL elicitation 与 v2 的实际支持范围,并链接已更新的开发计划。

只修改 src/SalmonEgg.Acp/README.md。稳定 V1 默认、现有 SDK 和应用行为保持现状;此 PR 不关闭尚有缺口的 issues。

关联:#144#146#147#148#149#154。开发顺序为 #147/#148 的 P1 正确性修复,然后 URL/凭据产品接入;v2 按 wire、状态生命周期、投影/权限、batch 和正式验收分阶段推进。

验证:

  • 文档-only 变更;git diff --check 通过,文档逐项对照当前源码与官方 schema。未为文字修改另跑本地构建。
  • PR 提交 ac1257a7bcdd59075136f8a9fce601ef215fba74SDK CI 544/544 tests 通过,SDK/测试构建均为 0 warning / 0 error;包 SalmonEgg.Acp.1.0.1-alpha.0.163.nupkg 的外部 consumer 和 37 个草案类型的编译保护均通过。WASM GUI gate、Core tests 及平台构建也通过。最终检查为 16 成功、1 条件跳过,无待运行/失败项,PR 为 MERGEABLE / CLEAN
  • 隔离副本中现有 auth/cancellation 定向用例 13/13 通过;临时移除门控后 4 个相应用例失败,恢复源码并重编后再次通过。
  • 新增的临时原始报文探针复现 4 种 auth discriminator 错误放行;另 2 个探针复现网络适配器吞 caller cancellation、cancel-send 错误事件外泄。失败证据已写入 fix(acp): 选择认证方法时不判别 AuthMethod.type,可能把 terminal 方法传给 authenticate(规范禁止) #147/feat(acp): 实现协议级请求取消 $/cancel_request 与 -32800 Cancelled 错误码 #148,探针及临时源码不进入本 PR。这些是现有问题,本 PR 未修复它们。
  • 未执行新的真实第三方 Agent 互操作、五平台 URL 安全打开或 live v2 验收;对应要求保留在 issues。

@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 8, 2026 1:53pm UTC

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