stacklok / toolhive / 21436272532

60%

Build Type

push

github

Committed by web-flow

Commit Message

Infrastructure improvements and bugfixes for vMCP (#3439)

* Infrastructure improvements and bugfixes for vMCP

- Add OpenTelemetry tracing to capability aggregation
- Add singleflight deduplication for discovery requests
- Add health checker self-check prevention
- Add HTTP client timeout fixes
- Improve E2E test reliability
- Various build and infrastructure improvements

* fix: Update CallTool and GetPrompt signatures to match BackendClient interface

- Add conversion import for meta field handling
- Update CallTool to accept meta parameter and return *vmcp.ToolCallResult
- Update GetPrompt to return *vmcp.PromptGetResult
- Add convertContent helper function

* fix: Update ReadResource signature to match BackendClient interface

- Update ReadResource to return *vmcp.ResourceReadResult instead of []byte
- Extract and include meta field from backend response
- Include MIME type in result

* fix: Pass selfURL parameter to health.NewMonitor

- Construct selfURL from Host, Port, and EndpointPath
- Prevents health checker from checking itself

* Fix NewHealthChecker calls in checker_test.go to include selfURL parameter

* Fix NewMonitor calls in monitor_test.go to include selfURL parameter

All 10 calls to NewMonitor in monitor_test.go were missing the new selfURL parameter that was added to the function signature. This was causing compilation failures in CI.

* Fix Go import formatting issues (gci linter)

Fixed import ordering in:
- pkg/vmcp/client/client.go
- pkg/vmcp/health/checker_test.go
- pkg/vmcp/health/monitor_test.go

* Fix Chart.yaml version - restore to 0.0.103

The version was incorrectly downgraded to 0.0.102. Restore it to 0.0.103 to match main branch.

* Bump Chart.yaml version to 0.0.104

The chart-testing tool requires version bumps to be higher than the base branch version (0.0.103).

* Update README.md version badge to 0.0.104

Match the Chart.yaml version update to satisfy helm-docs pre-commit hook.

* Refactor vMCP tracing and remov... (continued)

Run Details

105 of 279 new or added lines in 6 files covered. (37.63%)

2 existing lines in 1 file now uncovered.

38842 of 64359 relevant lines covered (60.35%)

75.0 hits per line