stacklok / toolhive / 23527114078

65%

Build Type

push

github

Committed by web-flow

Commit Message

Preserve content-level annotations through vMCP relay (#4336)

* Preserve content-level annotations through vMCP relay

MCP content items (text, image, audio, embedded resource, resource link)
can carry per-content annotations (audience, priority, lastModified) that
inform how clients display results. The vMCP relay layer was silently
dropping these annotations during the mcp->vmcp->mcp conversion round
trip.

Add ContentAnnotations domain type following the existing Anti-Corruption
Layer pattern, and update ConvertMCPContent/ToMCPContent to preserve
annotations for all five content types.

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

* Address review comments on content annotations

- Deep-copy Priority pointer in ConvertMCPAnnotations and ToMCPAnnotations
  to prevent pointer aliasing across the ACL boundary
- Add zero-value guard in ToMCPAnnotations for symmetry with
  ConvertMCPAnnotations (empty non-nil input now returns nil)
- Document Priority range constraint [0.0, 1.0] and valid Audience
  values per MCP spec
- Add test case for empty non-nil ContentAnnotations
- Add blob embedded resource test case for round-trip verification

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

* Fix lint issues in content annotations conversion

Break long lines exceeding 130-char limit in ConvertMCPContent and
fix gci formatting alignment in content_test.go.

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

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Run Details

96 of 108 new or added lines in 1 file covered. (88.89%)

29 existing lines in 3 files now uncovered.

51092 of 79573 relevant lines covered (64.21%)

147.65 hits per line