stacklok / toolhive / 29569646387

69%

Build Type

push

github

Committed by web-flow

Commit Message

feat: migrate from mark3labs/mcp-go to toolhive-core/mcpcompat (go-sdk) (#5729)

* feat: migrate from mark3labs/mcp-go to toolhive-core/mcpcompat

Swaps all github.com/mark3labs/mcp-go/{mcp,server,client,client/transport}
imports to the go-sdk-backed compatibility shim in toolhive-core
(github.com/stacklok/toolhive-core/mcpcompat/*). No call-site logic changes —
the shim mirrors mcp-go's API. This moves ToolHive's MCP protocol handling
onto the official modelcontextprotocol/go-sdk.

Depends on stacklok/toolhive-core#150; the toolhive-core dependency is pinned
to that branch commit and should be bumped to the released version once #150
merges.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* chore: bump toolhive-core shim + fix gci import grouping

Bump github.com/stacklok/toolhive-core to the mcpcompat shim commit that
closes the server/client compatibility gaps exercised by the vMCP and MCP
test suites (pulls the required cel-go v0.29.1). Also fix gci import
grouping in three files flagged by lint (toolhive-core shares the
github.com/stacklok/toolhive prefix, so its imports group with toolhive's).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* style: fix gci import grouping after mcp-go->mcpcompat swap

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* Add regression tests gating the mcp-go to go-sdk migration (#5772)

* Add Serve-path regression tests and bridge list_changed re-fetch

Pin current mcpcompat shim behavior for session lifecycle,
capability advertisement, tool schema fidelity, and concurrent
context isolation so the go-sdk migration can be validated.

Fix bridge to re-fetch tools/resources/prompts when a list_changed
notification arrives, keeping the local stdio server's advertised set
in sync with the upstream backend.

Two real regressions found and pinned with documented assertions:
- Initialize response advertises only {logging:{}} instead of
  capabilities.tools and capabilities.resources (toolhive-... (continued)

Coverage Stats

25 of 25 new or added lines in 7 files covered. (100.0%)

15 existing lines in 5 files now uncovered.

74046 of 108530 relevant lines covered (68.23%)

65.16 hits per line