Add the O2 I1 stage 6 review package
Why: O2 I1 is implemented in Agent-Native #48 and needs user stage 6 review before merge. Evidence: Worker attempt 2 success at 899fd70; host vet and tests (with -race) pass; PR #48 CI success; independent review PASS with no blocking issues. Impact: Task card and review package only; Agent-Native #48 stays draft until stage 6 approval. Refs: Agent-Native#48, Agent-System-Dev#639 Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
@@ -1,6 +1,6 @@
|
||||
# feature/fast-slow-o2-i1-agent-native 任务卡
|
||||
|
||||
任务主记录:<PR 链接,交付前填写>;状态:进行中;owner:weichen
|
||||
任务主记录:Agent-Native 草稿 PR [#48](https://git-home.u1680832.nyat.app:30319/weichen/Agent-Native/pulls/48)(Dev PR 见下方合入与清理节);状态:待用户阶段 6 人审;owner:weichen
|
||||
|
||||
## 基线与授权
|
||||
|
||||
@@ -29,7 +29,8 @@
|
||||
|
||||
| 单元 ID | 预期范围 | 配置摘要 | 角色 | 候选顺序 | attempt | CLI/模型 | 终态 | transport | fallback | 源码外收据路径 |
|
||||
|---|---|---|---|---|---|---|---|---|---|---|
|
||||
| 待定义 | — | NOT_RUN | NOT_RUN | NOT_RUN | NOT_RUN | NOT_RUN | NOT_RUN | NOT_RUN | NOT_RUN | — |
|
||||
| o2-i1-agent-native-route-contract | runtime/llm、runtime/argschema、registry.go(+test) | AGENT-EXECUTION.md | implementation | cursor→antigravity→pi | 1 | cursor / auto | scope-expansion(单元包 `allowed_paths` 误用 `**` 通配,执行器按前缀匹配,7 个范围内文件被判越界;fail-stop) | 未集成 | never | /tmp/o2i1-20261020/o2-i1-receipt.attempt1-scope-expansion.json |
|
||||
| o2-i1-agent-native-route-contract | 同上(改为前缀写法 `runtime/llm/`、`runtime/argschema/`,范围不变) | AGENT-EXECUTION.md | implementation | cursor→antigravity→pi | 2(用户 2026-10-20 授权全新重跑) | cursor / auto | success | `899fd70`(ref refs/agent-exec/e38bcf1c…/445c377b,parent 9c24e22),`agent-exec apply` 快进集成 | never | /tmp/o2i1-20261020/o2-i1-receipt.json |
|
||||
|
||||
## 关联方向与规划
|
||||
|
||||
@@ -92,9 +93,9 @@
|
||||
|
||||
## 结果与验证(完成后填写)
|
||||
|
||||
- 结果:
|
||||
- 验证:<结论摘要 PASS/FAIL/SKIP/NOT_RUN 逐项;逐条证据由 make evidence 写入上方证据表>
|
||||
- 影响与剩余项:
|
||||
- 结果:I1 在 Agent-Native 分支 `feature/fast-slow-o2-i1-route-contract`(`899fd70`,草稿 PR #48)交付;阶段 6 评审包 docs/superpowers/specs/2026-10-20-fast-slow-o2-i1-review.md。
|
||||
- 验证:host(SELF)@899fd70 vet、定向测试(含 -race)、`go test ./...` PASS;CI(GATE)PR #48 三项 success;独立 Review PASS。
|
||||
- 影响与剩余项:仅 Agent-Native `runtime/llm`、`runtime/argschema`、registry 委托;合同默认关闭。剩余:用户阶段 6;I2、I3、P1;评审包第 3 节非阻塞项。
|
||||
|
||||
## 规划进展与下一步
|
||||
|
||||
@@ -108,7 +109,8 @@
|
||||
|
||||
## 合入与清理
|
||||
|
||||
- Review 记录:NOT_RUN(审阅智能体/平台身份、base/head、结论、问题闭合及重放依据)
|
||||
- Review 记录:Cursor Task 子代理 code-reviewer(fresh_session=true,session=531ce711-73e7-44d8-8b9e-ed90d93856f7;只接收需求、方案/计划、base 9c24e22/head 899fd70 与 diff,未转交实施会话);结论 PASS,0 阻塞,3 条非阻塞记入评审包第 3 节;评审者自跑 vet 与 -race 定向测试通过(SELF)。
|
||||
- CI 记录(Agent-Native):PR #48 head `899fd70d27dd25bb88088a677546a71e54a86d9e` commit status success:`ci / vet-and-test`(runs/4350/jobs/7018)、`ci / docker-checkout-hostgateway`(runs/4350/jobs/7019)、`governance / governance`(runs/4351/jobs/7020)(GATE)。
|
||||
- 冲突记录:NOT_RUN(最新目标同步、冲突处置、组合兼容核验及复审/补测)
|
||||
- CI 记录:NOT_RUN(当前 HEAD 完整 SHA、全部必需检查、运行链接、执行身份及 GATE 结果)
|
||||
- 合入记录:NOT_RUN(交付授权范围、PR、被核验 HEAD、合入 SHA、目标分支及内容核验)
|
||||
|
||||
@@ -0,0 +1,36 @@
|
||||
# 快慢通道 O2 I1(Agent-Native `route_contract_eof`):阶段 6 评审包
|
||||
|
||||
- 状态:**待用户阶段 6 人审**。人审通过前不合入 Agent-Native main;I2(Backend)、I3(Dev 改钉)待本增量通过后另行启动,P1 付费探针另提资源方案。
|
||||
- 日期:2026-10-20;作者:host;Dev 任务:`feature/fast-slow-o2-i1-agent-native`。
|
||||
- 依据:[阶段 4 方案](2026-10-19-fast-slow-o2-route-contract-eof-design.md)与[阶段 5 计划 §2](../plans/2026-10-19-fast-slow-o2-route-contract-eof-plan.md)(用户 2026-10-20 通过;C2 同时接受 `http2_stream` 与 `known_length`+`match`)。
|
||||
- 交付对象:Agent-Native 草稿 PR [#48](https://git-home.u1680832.nyat.app:30319/weichen/Agent-Native/pulls/48),base `9c24e22`(origin/main),head `899fd70`。
|
||||
|
||||
## 1. 交付内容
|
||||
|
||||
| 方案条目 | 实现 | 位置 |
|
||||
|---|---|---|
|
||||
| 路由合同与身份绑定(§4.1) | `RouteContractEOF{Model, BaseURL, IdentitySHA256}`、`RouteContractIdentity`、`NormalizeRouteBaseURL`(仅 https,含 userinfo/query/fragment 一律不匹配);`Client.RouteContract` 为 nil/零值即关闭;重定向逐项比较 scheme、host、port、path | `runtime/llm/route_contract.go`、`client.go` |
|
||||
| 解析器延后判定(计划 §2.2) | 公开 `ParseSSE` 签名与零值行为不变;内部 `parseSSE(…, sseHooks)` 只在“有工具且缺 `[DONE]`”时把成功候选与今天的失败事件一起交给传输层;新增 `finishChunks`、`multiChoice`、`fragmentAfterFinish` 统计 | `runtime/llm/sse.go` |
|
||||
| 放行条件 C1–C9(§4.2) | 按顺序判定,记录第一个不满足的编号;放行时 `StreamComplete=false`、`CompletionBasis=route_contract_eof` | `route_contract.go` `check` |
|
||||
| 参数 schema 子集(§4.3) | 新包 `runtime/argschema.Validate`,由 `validateArgs`/`matchesType` 逐字迁移;registry 改为委托 | `runtime/argschema/`、`runtime/tools/registry.go` |
|
||||
| 可观测性(§4.4) | `Event.CompletionBasis`;`TerminalObservation.completionBasis`/`contractCheck`(仅枚举值);`[DONE]` 成功路径记 `done_marker` | `types.go`、`client.go` |
|
||||
|
||||
## 2. 验证
|
||||
|
||||
- 实施:Cursor worker(`agent-exec`,attempt 2 success;attempt 1 因单元包 `allowed_paths` 误用 `**` 被判 scope-expansion,修正写法后经用户授权全新重跑),`agent-exec apply` 快进集成。
|
||||
- host 验证(SELF)@`899fd70`:改动文件 `gofmt` 合格;`git diff --check`;`go vet ./runtime/...`;`go test ./runtime/llm/... ./runtime/argschema/... ./runtime/tools/... -count=1`(另跑 `-race`)与 `go test ./... -count=1` 全部通过。
|
||||
- CI(GATE):PR #48 head `899fd70` 三项检查 success(`ci / vet-and-test`、`ci / docker-checkout-hostgateway`、`governance / governance`)。
|
||||
- 独立 Review(全新会话 code-reviewer,只接收需求、base/head 与 diff;评审者自跑为 SELF):**PASS,无阻塞项**。确认零值合同行为不变;拒绝事件与合同关闭事件除 `contractCheck` 外 `reflect.DeepEqual`;`argschema` 与旧实现逐行一致;无数据竞争;计划 §2 测试矩阵每一行都有带断言的用例,并同时断言 Usage 保留与放行次数。
|
||||
|
||||
## 3. 需人审确认与非阻塞项
|
||||
|
||||
1. **C3 行的编号差异**:计划写的是 HTTP/2 `RST_STREAM`/中途断连期望 C3,实现返回 **C2**。原因是 C2(传输边界)先于 C3 判定,断流时没有见到 EOF,先在 C2 失败。结论同样是拒绝,`contractCheck` 记录的是“第一个不满足的条件”,不一定是根因。请确认接受。
|
||||
2. **无工具的错误终态也会带 `contractCheck`**(非阻塞):合同开启时,纯文本流遇到供应商错误这类终态也会补一次判定(如记为 `C2`),这类流本来就不可能放行。不影响验收条件 3,也不泄漏内容;可在后续把补判限定为含工具调用的终态。
|
||||
3. **重复 `finish_reason` 会被 C4 拒绝**(非阻塞,失败关闭):若供应商在 usage 块里重复 `finish_reason`,一律拒绝。MiniMax 真实流是否这样,列为 P1 付费探针的观察点。
|
||||
4. **测试可加强**(非阻塞):`release/http2` 与 `close-delimited` 用例可直接断言 `BodyBoundary.FramingClass`;补一个“合同开启时纯文本流不变”的回归用例(评审者已用临时探针确认行为不变)。
|
||||
|
||||
## 4. 下一步(通过后)
|
||||
|
||||
1. 合入 Agent-Native #48(组件仓,无需 #518 窗口)。
|
||||
2. 启动 I2(Backend:系统层配置键、`openLLMClient` 按身份挂合同、透传 `completionBasis`;启动前核对 Backend 是否有读 `StreamComplete` 的工具门控)。
|
||||
3. I3 Dev 改钉组合验证;P1 另提资源方案。
|
||||
Reference in New Issue
Block a user