Skip to content

Conversation

@mikekistler
Copy link
Contributor

Motivation and Context

This PR modifies the tools-call-simple-text to omit the "arguments" property in the tool call, which is not required according to the schema.

https://github.com/modelcontextprotocol/modelcontextprotocol/blob/main/schema/2025-06-18/schema.json#L133

image

This will flag server implementations that incorrectly fail tool call requests that do not include "arguments".

How Has This Been Tested?

All tests, including the modified tools-call-simple-text test, pass when run on a server using the C# MCP SDK.

Breaking Changes

No breaking change, just a slightly more strict version of the test.

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

This might expose a latent conformance bug in the TypeScript SDK.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 24, 2025

Open in StackBlitz

npx https://pkg.pr.new/modelcontextprotocol/conformance/@modelcontextprotocol/conformance@53

commit: 66fd3e7

@pcarleton pcarleton merged commit 314f0f6 into modelcontextprotocol:main Nov 24, 2025
4 checks passed
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