stacklok / toolhive / 22612385118

64%

Build Type

push

github

Committed by web-flow

Commit Message

Handle embedded ResourceContents in vmcp bridge (#3953)

Handle embedded ResourceContents in vmcp bridge content conversion

The vmcp bridge silently discarded embedded ResourceContents from MCP
tool results, replacing them with empty strings. This broke tools like
github_get_file_contents that return file content via NewToolResultResource.

Add bidirectional EmbeddedResource handling:
- ConvertMCPContent (MCP→vmcp): extract text/blob/URI/MIME from
  EmbeddedResource into vmcp.Content fields
- convertToMCPContent (vmcp→MCP): reconstruct EmbeddedResource with
  TextResourceContents or BlobResourceContents from vmcp.Content

The empty-resource fallback now returns an EmbeddedResource (preserving
type: "resource" semantics) instead of converting to empty TextContent.

Closes #3952

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Run Details

26 of 28 new or added lines in 2 files covered. (92.86%)

7 existing lines in 3 files now uncovered.

45450 of 71914 relevant lines covered (63.2%)

76.98 hits per line