Skip to content

mcp: read _meta from content item instead of response level#287107

Merged
connor4312 merged 1 commit intomicrosoft:mainfrom
qchuchu:mcp/fix-meta-content-level
Jan 12, 2026
Merged

mcp: read _meta from content item instead of response level#287107
connor4312 merged 1 commit intomicrosoft:mainfrom
qchuchu:mcp/fix-meta-content-level

Conversation

@qchuchu
Copy link
Copy Markdown
Contributor

@qchuchu qchuchu commented Jan 12, 2026

Summary

Fixes #287106

The _meta.ui field containing CSP configuration was being read from the response level instead of from inside the contents array item.

Related discussion: modelcontextprotocol/ext-apps#242

Changes

Test Plan

  • Create an MCP server returning _meta inside contents[0] (as per spec)
  • Verify CSP domains are now correctly applied

Fixes microsoft#287106

The ext-apps spec defines _meta inside the contents array item,
not at the response level. This fix aligns the implementation
with the spec.
Copy link
Copy Markdown
Member

@connor4312 connor4312 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@connor4312 connor4312 enabled auto-merge (squash) January 12, 2026 16:05
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 12, 2026
@connor4312 connor4312 merged commit 67d0e5b into microsoft:main Jan 12, 2026
17 checks passed
@qchuchu qchuchu deleted the mcp/fix-meta-content-level branch January 12, 2026 17:14
eli-w-king pushed a commit that referenced this pull request Jan 14, 2026
Fixes #287106

The ext-apps spec defines _meta inside the contents array item,
not at the response level. This fix aligns the implementation
with the spec.
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Feb 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MCP Apps: _meta should be read from content item, not response level

3 participants