stacklok / toolhive / 23584042049

65%

Build Type

push

github

Committed by web-flow

Commit Message

Wire session-aware backend routing in proxy transports (RC-12) (#4318)

* fix(ci): clear full build cache before coverage to prevent NumStmt mismatch

The CI Go build cache is keyed on go.sum, so source-only changes do not
bust it. With -coverpkg=./..., every test binary instruments all packages;
if any binary was compiled from a stale cached artifact whose coverage
statement count differs from the current source, go tool cover -func errors
with "inconsistent NumStmt: changed from N to M".

Replace go clean -testcache with go clean -cache -testcache so that both
the test-result cache and the compiled artifact cache are wiped before each
coverage run, guaranteeing every package is instrumented from fresh source.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat(session): add WithSessionStorage option to streamable proxy (RC-12)

Adds Option/WithSessionStorage functional option to the streamable HTTP
proxy, enabling injection of a custom session storage backend for
multi-replica deployments. Updates NewHTTPProxy signature from variadic
middlewares to explicit slice + variadic opts, and fixes all call sites.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat(session): add WithSessionStorage option to HTTP-SSE proxy (RC-12)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat(session): add WithSessionStorage option to transparent proxy (RC-12)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat(session): store backend_url on initialize and route via metadata in Rewrite (RC-12)

On initialize responses that return Mcp-Session-Id, RoundTrip now creates
the session via AddSession and stores backend_url=targetURI in metadata.
The Rewrite closure in Start looks up that metadata to override the outbound
URL, enabling session-aware routing to originating backend pods.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat(session): return HTTP 400 for non-initialize requests with unknown s... (continued)

Run Details

137 of 188 new or added lines in 6 files covered. (72.87%)

2 existing lines in 1 file now uncovered.

51751 of 79200 relevant lines covered (65.34%)

65.26 hits per line