stacklok / toolhive / 40334339708
73%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 977
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 71.815% (+0.04%) from 71.778%
40334339708

push

github

web-flow 
Merge commit from fork

* Forward passthrough headers only from the request

A request that omitted an allowlisted passthrough header could reach
the backend with the value sent by the request that created the
session. CaptureMiddleware stored nothing when no allowlisted header
was present, so the handler context fell back to the values captured
at session creation.

Always attach the captured map when an allowlist is configured, even
when it is empty, so an absent header is not forwarded. This matches
the documented contract that a header absent on a request is not
forwarded.

Update the PassthroughHeaders field docs, which still said the headers
are consumed once at session creation, and regenerate the CRDs and CRD
API docs. Background capability resyncs have no triggering request and
keep using the values captured when the session was created.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Correct passthrough header docs and widen test

The PassthroughHeaders docs claimed every backend call carries the
triggering request's headers. That holds for tool calls, resource reads
and prompt gets, but session-scoped work (the session's backend
connections, including teardown, its advertised capability list, and
background resyncs) still uses the session creator's values. Say so,
and fix two server.go comments that still described capture as
feeding only the per-session client at session creation.

Extend the regression test to cover resources/read and an allowlist of
two headers where a request sends only one, bound each request with a
client timeout, and stop calling require from the backend handler
goroutine.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

89532 of 124671 relevant lines covered (71.81%)

128.9 hits per line

Coverage Regressions

Lines Coverage ∆ File
4
63.89
4.27% pkg/workloads/manager.go
3
76.19
-2.38% pkg/state/local.go
2
67.14
-2.86% pkg/state/runconfig.go
1
81.58
-0.12% pkg/vmcp/composer/workflow_engine.go
Jobs
ID Job ID Ran Files Coverage
1 40334339708.1 977
71.81
GitHub Action Run
Source Files on build 40334339708