• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

stacklok / toolhive / 23041861605
64%

Build:
DEFAULT BRANCH: main
Ran 13 Mar 2026 08:11AM UTC
Jobs 1
Files 554
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

13 Mar 2026 08:05AM UTC coverage: 64.469% (+0.1%) from 64.367%
23041861605

push

github

web-flow
Enable session management v2 by default (#4091)

* Enable session management v2 flag by default

* solve bugs and fix tests

Regenerate aggregator mock to restore alphabetical method order

MergeCapabilities was placed after ProcessPreQueriedCapabilities in the
mock file. Running go generate restores the mockgen-expected alphabetical
order to pass the codegen verification CI check.

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

Fix composite tool name collision in SMv2 session registration

The v2 session registration path (handleSessionRegistrationV2) was adding
composite tools without checking for name conflicts with backend tools,
unlike the v1 injectCapabilities path which uses validateNoToolConflicts.

- Add GetMultiSession to the SessionManager interface so server.go can
  access the session's resolved backend tool list for conflict validation
- Call validateNoToolConflicts before registering composite SDK tools,
  skipping them (with an error log) if a collision is detected
- Track actual composite tools added for accurate log output

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

Fix composite tool routing failure in SMv2 path

Composite tool workflow steps call the shared router which reads
DiscoveredCapabilities from the request context, but the discovery
middleware was short-circuiting for MultiSession sessions and never
injecting them, causing "capabilities not found in context" errors.

- Add GetRoutingTable() to the MultiSession interface and implement it
  on defaultMultiSession (routing table is immutable post-creation)
- Update discovery middleware: instead of skipping context injection for
  MultiSession, build AggregatedCapabilities from the session's routing
  table and inject it — enabling composite tool workflow steps to route
  backend tool calls correctly
- Guard against nil routing table (session initialisation not yet complete)
- Add GetRoutingTable() stub to test fakes in session manager and
  integr... (continued)

171 of 255 new or added lines in 12 files covered. (67.06%)

28 existing lines in 5 files now uncovered.

48670 of 75494 relevant lines covered (64.47%)

73.39 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
3
90.61
0.96% pkg/vmcp/aggregator/default_aggregator.go
7
0.0
0.0% cmd/vmcp/app/commands.go
12
12.58
-1.09% pkg/vmcp/aggregator/mocks/mock_interfaces.go
21
76.08
0.11% pkg/vmcp/server/server.go
41
82.8
-16.19% pkg/vmcp/session/factory.go

Uncovered Existing Lines

Lines Coverage ∆ File
1
0.0
0.0% cmd/vmcp/app/commands.go
2
66.99
2.09% cmd/thv-operator/controllers/virtualmcpserver_controller.go
2
79.79
-0.52% pkg/transport/proxy/httpsse/http_proxy.go
2
76.08
0.11% pkg/vmcp/server/server.go
21
50.09
-1.61% pkg/workloads/manager.go
Jobs
ID Job ID Ran Files Coverage
1 23041861605.1 13 Mar 2026 08:11AM UTC 554
64.47
GitHub Action Run
Source Files on build 23041861605
  • Tree
  • List 554
  • Changed 23
  • Source Changed 14
  • Coverage Changed 22
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #23041861605
  • fb49c151 on github
  • Prev Build on main (#23041807322)
  • Next Build on main (#23045791028)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc