Release tool calls on a verified route contract without [DONE] #48

Merged
weichen merged 1 commits from feature/fast-slow-o2-i1-route-contract into main 2026-10-20 05:54:30 +00:00
Owner

Why: Fast/slow O2 increment I1. One exactly bound OpenAI-chat route may release tool calls from a stream that ended without the literal [DONE], but only when the HTTP body boundary and every frame prove the calls complete (conditions C1-C9). The design and plan passed user stage 4/5 review on 2026-10-20; C2 accepts both http2_stream+EOF and known_length+match.

Evidence (SELF, host @899fd70): gofmt clean for changed files; git diff --check; go vet ./runtime/...; go test ./runtime/llm/... ./runtime/argschema/... ./runtime/tools/... (also -race); go test ./... all ok. Independent review: see Dev task card. CI (GATE): this PR.

Impact: New optional Client.RouteContract (nil/zero = off, behaviour unchanged); Event/TerminalObservation gain completionBasis and contractCheck enums; new runtime/argschema package shared by the registry. StreamComplete keeps its literal meaning. Backend wiring (I2), Dev repin (I3) and the paid probe (P1) are separate.

Refs: Dev task feature/fast-slow-o2-i1-agent-native; design docs/superpowers/specs/2026-10-19-fast-slow-o2-route-contract-eof-design.md; plan docs/superpowers/plans/2026-10-19-fast-slow-o2-route-contract-eof-plan.md (Agent-System-Dev #639). Draft pending user stage 6 review.

Why: Fast/slow O2 increment I1. One exactly bound OpenAI-chat route may release tool calls from a stream that ended without the literal [DONE], but only when the HTTP body boundary and every frame prove the calls complete (conditions C1-C9). The design and plan passed user stage 4/5 review on 2026-10-20; C2 accepts both http2_stream+EOF and known_length+match. Evidence (SELF, host @899fd70): gofmt clean for changed files; git diff --check; go vet ./runtime/...; go test ./runtime/llm/... ./runtime/argschema/... ./runtime/tools/... (also -race); go test ./... all ok. Independent review: see Dev task card. CI (GATE): this PR. Impact: New optional Client.RouteContract (nil/zero = off, behaviour unchanged); Event/TerminalObservation gain completionBasis and contractCheck enums; new runtime/argschema package shared by the registry. StreamComplete keeps its literal meaning. Backend wiring (I2), Dev repin (I3) and the paid probe (P1) are separate. Refs: Dev task feature/fast-slow-o2-i1-agent-native; design docs/superpowers/specs/2026-10-19-fast-slow-o2-route-contract-eof-design.md; plan docs/superpowers/plans/2026-10-19-fast-slow-o2-route-contract-eof-plan.md (Agent-System-Dev #639). Draft pending user stage 6 review.
weichen added 1 commit 2026-10-20 04:04:22 +00:00
agent-exec transport e38bcf1cbe415c5d6a3e91773d811b0a/445c377b4484
ci / docker-checkout-hostgateway (pull_request) Successful in 12s
governance / governance (pull_request) Successful in 35s
ci / vet-and-test (pull_request) Successful in 1m24s
899fd70d27
weichen marked the pull request as ready for review 2026-10-20 05:54:26 +00:00
weichen merged commit 2fc66be652 into main 2026-10-20 05:54:30 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: weichen/Agent-Native#48