Skip to content

Fix tool input schemas in README examples - #294

Open
EhsanAzish80 wants to merge 1 commit into
modelcontextprotocol:mainfrom
EhsanAzish80:docs/fix-readme-tool-schemas
Open

EhsanAzish80 wants to merge 1 commit into
modelcontextprotocol:mainfrom
EhsanAzish80:docs/fix-readme-tool-schemas

Conversation

@EhsanAzish80

Copy link
Copy Markdown

The weather and calculator examples used description strings where the tool input schema needs property objects. This gives each example an object schema with typed string properties. Related to #128.

Checked both examples and the diff. Documentation only.

@varun06

varun06 commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Hi @EhsanAzish80, thanks for this PR! I've reviewed it and it looks good to merge.

Could you please rebase it onto the latest main? CI was broken on macOS runners for every PR (a toolchain/SDK mismatch, fixed in #306), so rebasing will pick up the fix and give us a green run. I'll review and merge once it's updated.

If you're no longer able to work on this, no problem: just let us know. If we don't hear back in about two weeks, we may carry the change forward in a new PR, with credit to you.

@EhsanAzish80
EhsanAzish80 force-pushed the docs/fix-readme-tool-schemas branch from 6d5c968 to 42f5c2e Compare October 22, 2026 15:27
@EhsanAzish80
EhsanAzish80 force-pushed the docs/fix-readme-tool-schemas branch from 42f5c2e to 70d6da5 Compare October 22, 2026 16:49
This was referenced Oct 22, 2026
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