Skip to content

Conversation

@paulbalandan
Copy link

Motivation and Context

I was comparing the specification (https://modelcontextprotocol.io/specification/draft/server/resources#resource-contents) against the schema reference (https://modelcontextprotocol.io/specification/draft/schema#resourcecontents) when I noticed that the spec shows the name and title properties to both TextResourceContents and BlobResourceContents. This hints that the base ResourceContents extends BaseMetadata but the schema reference is not showing this.

Specification:
image

Schema Reference:
image
image

How Has This Been Tested?

npm run check:schema:ts
npm run generate:schema
npm run serve:docs
npm run check:docs
npm run format

Breaking Changes

This does not break the spec but SDKs might need to update their types if they depend on the schema reference here.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@paulbalandan paulbalandan requested a review from a team September 8, 2025 18:05
@paulbalandan paulbalandan force-pushed the resource-contents-extends-base-metadata branch from cef731f to 2d3731d Compare September 9, 2025 16:04
@dsp-ant
Copy link
Member

dsp-ant commented Sep 19, 2025

I appreciate this PR. I think the correct way to fix this for now is not to introduce the schema change but fix the documentation. If we want a schema change it would require a longer discussion as it's more invasive. I fixed this in #1500, so I will close out your PR.

@dsp-ant dsp-ant closed this Sep 19, 2025
@paulbalandan paulbalandan deleted the resource-contents-extends-base-metadata branch September 19, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants