Skip to content

Conversation

@ajevans99
Copy link

@ajevans99 ajevans99 commented Oct 22, 2025

Added/Changed

Not Handled

Motivation and Context

Support many changes in recent MCP spec update.
https://modelcontextprotocol.io/specification/2025-06-18/changelog

How Has This Been Tested?

  • New unit tests
  • MCP Inspector npx @modelcontextprotocol/inspector@latest

Breaking Changes

  • New case on public enum Tool.Content.resourceLink
  • Updated associated values on enum case Tool.Content.resource to include optional title and annoations properties

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

_meta is not a Swifty name but does map nicely to the spec. Open to suggestions

Copy link

@alainakafkes alainakafkes left a comment

Choose a reason for hiding this comment

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

hi there and thank you for taking a stab at supporting the latest MCP spec! i saw this PR because i am hoping to use Result's structuredOutput for a project. i have no special role in this repo but wanted to offer some feedback to help build interest in this work

@ajevans99 ajevans99 marked this pull request as ready for review October 28, 2025 01:33
@ajevans99
Copy link
Author

hi there and thank you for taking a stab at supporting the latest MCP spec! i saw this PR because i am hoping to use Result's structuredOutput for a project. i have no special role in this repo but wanted to offer some feedback to help build interest in this work

Thanks for the early review. Implemented a bunch of your suggestions and re-thought the meta and general fields.

@aehlke
Copy link

aehlke commented Nov 12, 2025

I see you had some changes last week. Is this ready to use as a fork until it gets reviewed? I hope this will be merged if it is ready. Thanks!

@ajevans99
Copy link
Author

I see you had some changes last week. Is this ready to use as a fork until it gets reviewed? I hope this will be merged if it is ready. Thanks!

Ready on my side. @mattt could we get a review when you get a chance

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.

3 participants