[Schema] No support for additonalProperties in tool input schema
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
Research direction
Start by locating the tool input schema definition and compare it with the existing tool output schema support for additionalProperties. Verify the resulting input schema against strict JSON Schema requirements and confirm that integrations accepting strict tool schemas now work as expected.
Written by the indexing model from the issue text.
Description
Is your feature request related to a problem? Please describe.
There is no proper support for "strict" json schema mode in tool input schema due to not having 'additionalProperties' field.
Describe the solution you'd like
Support 'additionalProperties' in tool input schema same like in tool output schema.
Describe alternatives you've considered
Ignoring phpstan sealed array rule - makes code less reliable.
Additional context
This is sometimes needed when integrating mcp server with software that requires tool input/output schema to follow strict json schema (no additional properties allowed for example, all fields required etc). These sometimes refuse to work with tools whose schema isn't 'strict' enough.
- Dominant language
- PHP
- Stars
- 1.6k
- Forks
- 173
- Avg merge
- 2d 49m
- Merged PRs (30d)
- 23
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from modelcontextprotocol/php-sdk
-
[Server] Handler type uses bare Closure, hard to decorate RegistryInterface under strict PHPStan OpenServer
Difficulty 1/5 Under an hour Newbie friendliness 78/100
modelcontextprotocol/php-sdk#468 · 2 comments ·
-
needs confirmation needs maintainer action Server
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
modelcontextprotocol/php-sdk#398 · 1 reaction ·
-
enhancement
Difficulty 4/5 3-5 days Newbie friendliness 55/100
modelcontextprotocol/php-sdk#510 · 1 comment ·
-
bug
Difficulty 4/5 3-5 days Newbie friendliness 45/100
modelcontextprotocol/php-sdk#504 ·
-
bug P2 Server
Difficulty 4/5 3-5 days Newbie friendliness 68/100
modelcontextprotocol/php-sdk#498 · 2 comments ·
All issues in modelcontextprotocol/php-sdk
Similar issues
-
priority: p3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
googleapis/librarian#7636 ·
-
0. Needs triage bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
nextcloud/fulltextsearch#1011 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
phpstan/phpstan-doctrine#794 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Automattic/static-site-importer#1767 ·