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

stacklok / toolhive / 27228850861
67%

Build:
DEFAULT BRANCH: main
Ran 09 Jun 2026 07:04PM UTC
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

09 Jun 2026 06:58PM UTC 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)

78 of 78 new or added lines in 3 files covered. (100.0%)

5 existing lines in 2 files now uncovered.

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 09 Jun 2026 07:04PM UTC 751
66.28
GitHub Action Run
Source Files on build 27228850861
  • Tree
  • List 751
  • Changed 5
  • Source Changed 2
  • Coverage Changed 5
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #27228850861
  • c311dc0c on github
  • Prev Build on main (#27221404552)
  • Next Build on main (#27236379146)
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