docs(acp): clarify SDK support boundaries - #183
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This was referenced Sep 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 和正式验收分阶段推进。
验证:
git diff --check通过,文档逐项对照当前源码与官方 schema。未为文字修改另跑本地构建。ac1257a7bcdd59075136f8a9fce601ef215fba74的 SDK 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。