stacklok / toolhive / 25010200036 / 1
66%
main: 66%

DEFAULT BRANCH: main
Ran
Files 670
Run time 59s
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: 66.215% (+0.001%) from 66.214%
25010200036.1

push

github

web-flow 
Strip Content-Length when wrapping tool filter response (#5077)

* Strip Content-Length when wrapping tool filter response

The toolFilterWriter buffers and may rewrite the response body to apply
tool filters or overrides. When the upstream sets an explicit
Content-Length (e.g. playwright-mcp on streamable-http), the rewritten
body usually has a different size and net/http rejects the write with
"http: wrote more than the declared Content-Length", sending zero body
bytes to the client. The MCP handshake then stalls at tools/list.

Drop Content-Length in WriteHeader so Go falls back to chunked transfer
encoding, which the MCP Streamable HTTP transport spec permits.

Fixes #5075

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Add regression test for Content-Length stripping

Without this test, a future refactor could silently revert the
toolFilterWriter.WriteHeader fix and reintroduce #5075 — TestToolFilterWriter_Flush
exercises WriteHeader but never asserts that an inbound Content-Length is
stripped. Adds a focused table test covering JSON+Content-Length,
SSE+Content-Length, and the no-Content-Length pass-through case.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Dan Barr <6922515+danbarr@users.noreply.github.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

59432 of 89756 relevant lines covered (66.22%)

61.86 hits per line

Source Files on job 25010200036.1