stacklok / toolhive / 27228850861
73%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 751
Run time 3min
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.281% (+0.02%) from 66.259%
27228850861

push

github

web-flow 
Add vMCP Serve skeleton and ServerConfig (#5467)

* Add vMCP Serve skeleton and ServerConfig

Introduce the transport-side entry point of the vMCP New/Serve split.
Serve wraps an already-constructed core VMCP and returns the existing
*Server, building the mcp-go server and applying the transport defaults
that server.New applies today. The route mux and HTTP lifecycle are
provided by the carried-forward Handler/Start/Stop, so observable
behavior is unchanged and server.New is not yet routed through Serve.

Implements changes for issue #5439:
- Add ServerConfig (transport-only fields plus the cross-cutting
  TelemetryProvider/AuditConfig) and Serve in pkg/vmcp/server/serve.go
- Validate nil cfg / nil core / nil SessionFactory via ErrInvalidConfig
- Wire the core's idempotent Close into the server shutdown sequence
- Guard the /status handler against a nil backend registry, which the
  transport does not own yet in this phase

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

* Address Serve skeleton review feedback

- Clarify the Serve doc comment: it builds only the mcp-go server and
  shutdown wiring; the route mux and HTTP lifecycle stay in the
  carried-forward Handler/Start until later phases relocate them
- Document the Config fields buildServeConfig deliberately leaves
  unmapped (AuthzMiddleware, HealthMonitorConfig) and why
- Add a reflection drift guard asserting every other Config field is
  populated by buildServeConfig, so a future field can't be dropped
  silently

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

* Tighten Serve skeleton per PR review

Addresses stacklok/toolhive#5467 review comments:
- MEDIUM serve.go (3381738359): extract shared default consts
  (defaultServerName/Version/Host/EndpointPath) referenced by New and
  buildServeConfig, and assert against them in tests, killing the
  triplicated literals
- MEDIUM serve.go (3381738361): drop the dead Config.StatusReporter
  mapping (set... (continued)

67232 of 101435 relevant lines covered (66.28%)

64.16 hits per line

Coverage Regressions

Lines Coverage ∆ File
3
97.36
-0.53% pkg/authz/authorizers/cedar/core.go
2
94.77
-1.31% pkg/vmcp/composer/dag_executor.go
Jobs
ID Job ID Ran Files Coverage
1 27228850861.1 751
66.28
GitHub Action Run
Source Files on build 27228850861