Skip to content

Conversation

@sunyuhan1998
Copy link
Contributor

Fixes #312
As mentioned in the issue, it is highly necessary to support annotations. This PR includes the following changes:

  1. Added the annotations property to McpSchema.Tool.
  2. Added corresponding unit tests.

@sunyuhan1998
Copy link
Contributor Author

sunyuhan1998 commented Jun 16, 2025

@tzolov @chemicL What do you think of this?

@tzolov tzolov self-assigned this Jun 24, 2025
@tzolov tzolov added this to the 0.11.0 milestone Jun 24, 2025
@tzolov
Copy link
Contributor

tzolov commented Jun 24, 2025

Thanks @sunyuhan1998 !
LGTM

tzolov pushed a commit that referenced this pull request Jun 24, 2025
@tzolov
Copy link
Contributor

tzolov commented Jun 24, 2025

Rebased and merged at ba5bc94

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

McpSchema.Tool requires the annotations attribute.

2 participants