Skip to content

Conversation

@vhorvath2010
Copy link
Contributor

@vhorvath2010 vhorvath2010 commented Sep 20, 2025

Solves #946 by adding type definitions for the optional annotations attribute for resources.

Also adds tests to validate the schema -> SDK typings and creation of a resource with annotations

@vhorvath2010 vhorvath2010 requested review from a team and felixweinberger September 20, 2025 05:58
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 2, 2025

Open in StackBlitz

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/sdk@954

commit: 96e9f95

@KKonstantinov
Copy link
Contributor

Hello. Thank you for your work on this.

Got my eyes on this PR and have updated it to the latest, as well as added annotations everywhere as specified by the spec.

Need to add some tests to lock it down for all the other types which now include annotations.

Will follow this through.

@KKonstantinov KKonstantinov added the improves spec compliance When a change improves ability of SDK users to comply with spec definition label Dec 3, 2025
@KKonstantinov
Copy link
Contributor

Thank you for your work! Apologies about the wait time here.

@KKonstantinov KKonstantinov merged commit 4b5c25d into modelcontextprotocol:main Dec 3, 2025
6 checks passed
@KKonstantinov KKonstantinov linked an issue Dec 3, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improves spec compliance When a change improves ability of SDK users to comply with spec definition

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Add Resource Annotations to SDK Types

2 participants