From e258ce052358f783dbc161e6ef06fb18ab111b44 Mon Sep 17 00:00:00 2001 From: "glm-5.2" Date: Thu, 2 Jul 2026 10:12:19 +0000 Subject: [PATCH] =?UTF-8?q?docs(review):=20mark=20review-streaming-impl=20?= =?UTF-8?q?completed=20=E2=80=94=20ADR-049=20streaming=20handler=20review?= =?UTF-8?q?=20passes=20all=2012=20checklist=20points?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- tasks/review-streaming-impl.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tasks/review-streaming-impl.md b/tasks/review-streaming-impl.md index 724471f..9af238b 100644 --- a/tasks/review-streaming-impl.md +++ b/tasks/review-streaming-impl.md @@ -1,7 +1,7 @@ --- id: review-streaming-impl name: Review ADR-049 streaming handler implementation for spec conformance and end-to-end correctness -status: pending +status: completed depends_on: [call/protocol/dispatch-streaming-branch, call/client/from-call-streaming-forwarding, http/gateway/invoke-streaming, http/server/subscribe-sse-streaming, http/adapters/from-openapi-sse-streaming] scope: broad risk: low @@ -207,4 +207,4 @@ review. ## Summary -> To be filled on completion \ No newline at end of file +> Reviewed ADR-049 streaming handler implementation across all 12 checklist points. All type surface, registry, builder, dispatch, from_call, gateway, /subscribe SSE, from_openapi SSE, ADR conformance, end-to-end correctness, pattern consistency, and test coverage items verified. 555 tests pass (306 call + 2 integration + 247 http), clippy clean, fmt clean. Fixed 2 pre-existing websocket subscription tests that expected INVALID_OPERATION_TYPE but now get call.responded (dispatch_requested routes Subscription via invoke_streaming). All 9 ADR-049 decisions implemented. Placeholders removed (subscribe_stream_from_envelope, envelope_to_sse_stream, stream_subscription). from_mcp unchanged (always HandlerKind::Once). \ No newline at end of file