alkdev/alknet
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs(http): mark http/review-websocket and http/review-http completed

Browse Source
This commit is contained in:
glm-5.2
2026-07-01 23:39:40 +00:00
parent b3ab6ef097
commit 5a51734291
2 changed files with 25 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
tasks/http/review-http.md
View File

@@ -1,7 +1,7 @@
--- ---
id: http/review-http id: http/review-http
name: Review alknet-http server surface + OpenAPI adapters for spec conformance name: Review alknet-http server surface + OpenAPI adapters for spec conformance
status: pending status: completed
depends_on: [http/server/gateway-endpoints, http/adapters/to-openapi, http/adapters/from-openapi, http/server/healthz-decoy] depends_on: [http/server/gateway-endpoints, http/adapters/to-openapi, http/adapters/from-openapi, http/server/healthz-decoy]
scope: broad scope: broad
risk: low risk: low
@@ -163,4 +163,15 @@ core of the crate.
## Summary ## Summary
> To be filled on completion > HTTP server surface + OpenAPI adapters reviewed against all 12 checklist items. All
> conformance criteria pass: HttpAdapter (struct, DecoyConfig, ProtocolHandler, axum
> over QUIC, h3 not registered), gateway endpoints (5 fixed, no direct-call ADR-047,
> flat JSON /call, GatewayDispatch::invoke, Internal → 404, 401/403 split, SSE /subscribe),
> error mapping (all codes, HTTP_<status> prefix, 401/403 split, Retry-After), auth
> (Bearer-only, shared middleware, ResolvedIdentity, no env vars), /healthz raw + decoy
> fallback, to_openapi (5 endpoints, info.version 1.0.0, per-caller via /search, error
> fidelity ADR-023), from_openapi (OperationAdapter, no-env-vars, HTTP_<status>, SSE),
> SharedHttpClient (ClientWithMiddleware, retry, RetryAfter, ArcSwap), GatewayDispatch
> (concrete struct, not trait), security constraints (no secrets in responses, no env
> vars, Internal → 404, AccessControl sole gate). 230 default + 265 mcp tests pass.
> Clippy clean on both feature configs. Fmt clean.

14
tasks/http/review-websocket.md
View File

@@ -1,7 +1,7 @@
--- ---
id: http/review-websocket id: http/review-websocket
name: Review WebSocket path for ADR-044/048 conformance (native session, no length prefix, browsers-not-peers) name: Review WebSocket path for ADR-044/048 conformance (native session, no length prefix, browsers-not-peers)
status: pending status: completed
depends_on: [http/websocket/connection-overlay] depends_on: [http/websocket/connection-overlay]
scope: moderate scope: moderate
risk: low risk: low
@@ -151,4 +151,14 @@ gateway shape, no length prefix).
## Summary ## Summary
> To be filled on completion > WebSocket path reviewed against all 11 checklist items. All conformance criteria
> pass: dispatcher transport abstraction (pub API, non-QUIC CallConnection, no
> regressions — 277+2 alknet-call tests), WS upgrade (/alknet/call, Bearer auth, 401,
> HTTP/1.1+HTTP/2), framing (binary WS = EventEnvelope, no length prefix, text → close
> 1002), dispatch (dispatch_requested, AccessControl gates, FORBIDDEN → call.error,
> correlation by id, handle_abort), bidirectionality (hub calls browser ops via overlay),
> connection-local overlay (no PeerId, no PeerEntry, overlay_env() routing,
> PeerRef::Specific → NOT_FOUND, overlay dies on close), browsers-not-peers rationale,
> streaming (native call.responded, no SSE, abort cascade on disconnect), all ADRs
> (012/016/024/029/034/044/048), security constraints. 230 tests pass, clippy clean,
> fmt clean.