stacklok / toolhive / 30335461841
70%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 842
Run time 2min
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: 69.748% (+0.06%) from 69.691%
30335461841

push

github

web-flow 
Make forwarding tests' Legacy dependency explicit (#6051)

The seven real-backend forwarding tests exercise server-initiated elicitation
and sampling, which MCP 2026-07-28 removes outright: go-sdk's
assertServerInitiatedRequestAllowed refuses them on negotiated protocol version
alone, never consulting client capabilities. They passed only because
newRealTestServer leaves Modern dispatch disabled, so the downstream client
landed on Legacy incidentally.

Pin the downstream client to Legacy explicitly by answering its server/discover
probe with a successful Legacy-only DiscoverResult -- what a Legacy-only server
actually sends -- and assert the pin fired, so deleting it fails loudly instead
of silently relying on a server-side default.

Two of the seven passed vacuously without the pin: a sessionless connect error
satisfied their lenient assertions regardless of the capability advertisement
they exist to exercise. Verified by falsifying their premise -- with the
capability granted they pass unpinned and fail pinned.

Add Modern-path tests recording today's actual behaviour for progress and
logging, so all five dispositions are executable rather than prose, and
document why each is what it is: progress is pinned because dispatchModern
cannot stream, not because Modern lacks the channel.

Refs #5959, #6033, #5743, #6058, #6062.

80850 of 115917 relevant lines covered (69.75%)

87.2 hits per line

Coverage Regressions

Lines Coverage ∆ File
6
66.64
 4.56% pkg/workloads/manager.go
3
67.14
 -4.29% pkg/state/runconfig.go
Jobs
ID Job ID Ran Files Coverage
1 30335461841.1 842
69.75
 GitHub Action Run
Source Files on build 30335461841