iter92 cluster-645: soft-deprecate StreamingProxy with Sunset header#1002
Merged
loning merged 3 commits intoMay 24, 2026
Merged
Conversation
…+ docs + guard (#645) Phase 9 r2 consensus soft-deprecate-streamingproxy-with-sunset-no-hard-delete: - StreamingProxy Host route + SDK surface preserved (public contract, no in-repo client, doc claim) - Added Deprecation + Sunset HTTP header (RFC 8594) pointing to /v1/responses - README / docs/canon / ADR-0015 / streaming-proxy-flow doc marked deprecated with sunset date and migration path - Architecture guard prevents new StreamingProxy consumers without explicit exception - No code/proto/test deleted; backward compat preserved Closes #645 ⟦AI:AUTO-LOOP⟧
loning
added
auto-loop
Contributor
Author
🤖 Phase 8 Reviewer —
|
Contributor
Author
🤖 Phase 8 Reviewer —
|
Contributor
Author
🤖 Phase 8 Reviewer —
|
Contributor
Author
📊 codex 进展 review-pr1002-architect-r1 (⏳ 进行中; 已跑 4 min)
|
Contributor
Author
📊 codex 进展 review-pr1002-quality-r1 (⏳ 进行中; 已跑 4 min)
|
Contributor
Author
📊 codex 进展 review-pr1002-tests-r1 (⏳ 进行中; 已跑 4 min)
|
Contributor
Author
🔧 Phase 8 r1 评审 — quality reject(self-doc 缺)→ fix r2
🔧 fix r2 codex 派出:加 cluster self-doc 到 StreamingProxyEndpoints + guard 上方。 ⟦AI:AUTO-LOOP⟧ |
… StreamingProxy endpoints + guard - agents/Aevatar.GAgents.StreamingProxy/StreamingProxyEndpoints.cs: Old/New self-doc above the deprecation header block; old = public streaming proxy entry, new = soft-deprecated compatibility-only with Sunset header - tools/ci/architecture_guards.sh: Old/New self-doc above new guard; old = no block on adding StreamingProxy consumer, new = guard rejects new consumers, direct model streaming goes via /v1/responses ⟦AI:AUTO-LOOP⟧
Contributor
Author
🤖 Phase 8 Reviewer —
|
Contributor
Author
🤖 Phase 8 Reviewer —
|
| 命令 | 结果 |
|---|---|
bash tools/ci/test_stability_guards.sh |
通过 |
bash tools/ci/architecture_guards.sh |
通过 |
dotnet test test/Aevatar.AI.Tests/Aevatar.AI.Tests.csproj --nologo --filter StreamingProxyCoverageTests |
通过，50 passed / 0 failed / 0 skipped |
📎 完整 codex 原始输出(存档备查)
---
pr: 1002
role: tests
verdict: approve
---
## Verdict
Test coverage and guard verification are adequate for the StreamingProxy soft-deprecation diff after the round-1 quality self-doc fix.
## Evidence
- `agents/Aevatar.GAgents.StreamingProxy/StreamingProxyEndpoints.cs:29` now has the requested `Refactor (iter92/cluster-645)` Old/New self-documentation above `MapStreamingProxyEndpoints`, matching the r1 quality reject fix request.
- `tools/ci/architecture_guards.sh:183` now has the requested `Refactor (iter92/cluster-645)` Old/New self-documentation above the StreamingProxy consumer guard.
- `test/Aevatar.AI.Tests/StreamingProxyCoverageTests.cs:125` asserts the endpoint source keeps `AddDeprecationHeadersAsync` attached to the mapped route group, covering the regression risk that deprecated StreamingProxy routes stop advertising deprecation.
- `test/Aevatar.AI.Tests/StreamingProxyCoverageTests.cs:136` asserts the emitted `Deprecation`, `Sunset`, and `Link` header values, plus the `/v1/responses` successor route.
- `tools/ci/architecture_guards.sh:187` adds the static no-new-production-consumers guard for StreamingProxy; `bash tools/ci/architecture_guards.sh` passed locally.
- `bash tools/ci/test_stability_guards.sh` passed locally; the PR did not add `Task.Delay`, `Thread.Sleep`, `[Skip]`, manual traits, polling allowlist entries, or loosened assertions.
- `dotnet test test/Aevatar.AI.Tests/Aevatar.AI.Tests.csproj --nologo --filter StreamingProxyCoverageTests` passed locally: 50 passed, 0 failed, 0 skipped.
REVIEW_DONE:1002:tests-r2:approve
⟦AI:AUTO-LOOP⟧REVIEW_DONE:1002:tests-r2:approve
⟦AI:AUTO-LOOP⟧
Contributor
Author
🤖 Phase 8 Reviewer —
|
Contributor
Author
✅ Phase 8 r2 集中评审
3/3 → auto-merge。 ⟦AI:AUTO-LOOP⟧ |
Contributor
Author
🤖 Phase 8 Reviewer —
|
…ster-645-streamingproxy-deprecate # Conflicts: # tools/ci/architecture_guards.sh
Contributor
Author
📊 codex 进展 fix-pr1002-r2 (⏳ 进行中; 已跑 11 min)
|
Contributor
Author
📊 codex 进展 resolve-pr1002-conflict (⏳ 进行中; 已跑 7 min)
|
Contributor
Author
📊 codex 进展 review-pr1002-architect-r2 (⏳ 进行中; 已跑 11 min)
|
Contributor
Author
📊 codex 进展 review-pr1002-quality-r2 (⏳ 进行中; 已跑 11 min)
|
Contributor
Author
📊 codex 进展 review-pr1002-tests-r2 (⏳ 进行中; 已跑 11 min)
|
Closed
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
1 participant
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.
Summary
/v1/responses9 files: +122 / -4。
Phase 9 r2 consensus
META_JUDGE_DONE:consensus:soft-deprecate-streamingproxy-with-sunset-no-hard-delete详 #645 评论。
Closes #645
⟦AI:AUTO-LOOP⟧