Skip to content

feat: add tool annotations#102

Merged
dickdavis merged 2 commits into
dickdavis:mainfrom
eelkevdbos:feature/tool-annotations
Feb 23, 2026
Merged

feat: add tool annotations#102
dickdavis merged 2 commits into
dickdavis:mainfrom
eelkevdbos:feature/tool-annotations

Conversation

@eelkevdbos

Copy link
Copy Markdown
Contributor

Adds tool annotations described in the tool specification.

dickdavis
dickdavis previously approved these changes Feb 23, 2026

@dickdavis dickdavis left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for the contribution! I'll be cutting a new release soon that'll include this change.

@eelkevdbos

Copy link
Copy Markdown
Contributor Author

@dickdavis, are you open to additions not strictly related to the spec, but needed for proper integration with ChatGPT's authorization hints (e.g. securitySchemes)?

@dickdavis

Copy link
Copy Markdown
Owner

@dickdavis, are you open to additions not strictly related to the spec, but needed for proper integration with ChatGPT's authorization hints (e.g. securitySchemes)?

I'm open to including fields from draft specs, especially given the pace of change. Ideally, we'd have a mechanism for excluding certain fields if the negotiated protocol doesn't support the field. However, I don't think this change would break anything, so that shouldn't be a blocker for what you are proposing. We'll need to consider that mechanism more deeply when we encounter breaking changes between protocol versions.

@dickdavis

Copy link
Copy Markdown
Owner

@eelkevdbos Would you mind running bundle exec standardrb --fix real quick to clean up the linter errors? Happy to approve again and merge once the checks pass.

@dickdavis dickdavis merged commit 89157f7 into dickdavis:main Feb 23, 2026
4 checks passed
@dickdavis

Copy link
Copy Markdown
Owner

@eelkevdbos Thanks again! You should be able to pin to ref 89157f743de68137ea3434007d3a3a3217d8739e in your Gemfile (or just main) to pick up your change. The next release will be in a few days; I'm wrapping up some housekeeping stuff, and then it'll be ready.

Feel free to submit any PRs in the meantime.

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