feat: mcp bundle tool integration #7595
Open
+2,830
−24
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
soyuka
force-pushed
the
mcp-bundle-integration
branch
3 times, most recently
from
d136520 to
b904a0d
Compare
rvanlaak
reviewed
Jan 12, 2026
Contributor
rvanlaak
left a comment
rvanlaak
left a comment
There was a problem hiding this comment.
Exciting that the mcp repo's schema PR is merged now, and you're able to spend time on API Platform integration.
Tried reviewing this PR without checking it out and testing it, so please feel free to ignore all my comments / suggestions / nitpicks 🙌 Just trying to provide you with another pov at some classes.
src/Mcp/Metadata/Operation/Factory/OperationMetadataFactory.php
Outdated
Show resolved
Hide resolved
Member
Author
Yeah it was the missing part + now I can also test out to use only the json-schema component's of API Platform to produce MCP SDK's schemas! |
OskarStark
added a commit
to symfony/ai
that referenced
this pull request
Jan 13, 2026
This PR was merged into the main branch. Discussion ---------- [MCP Bundle] bump php-sdk to 0.3 | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | Docs? | no | Issues | api-platform/core#7595 | License | MIT Commits ------- ce5758f bump php-sdk to 0.3
symfony-splitter
pushed a commit
to symfony/mcp-bundle
that referenced
this pull request
Jan 13, 2026
This PR was merged into the main branch. Discussion ---------- [MCP Bundle] bump php-sdk to 0.3 | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | Docs? | no | Issues | api-platform/core#7595 | License | MIT Commits ------- ce5758fc bump php-sdk to 0.3
soyuka
force-pushed
the
mcp-bundle-integration
branch
5 times, most recently
from
192498f to
7ff8ba8
Compare
soyuka
force-pushed
the
mcp-bundle-integration
branch
from
7ff8ba8 to
f4151f8
Compare
soyuka
force-pushed
the
mcp-bundle-integration
branch
from
620cc60 to
13f91e9
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rework of #7458
Now with a simpler approach, this gives the ability to add McpTool (upcoming McpResource) to any API Platform environment giving access to:
item_uri_template)CallToolResult(from the php-sdk itself for images, sound etc.)TODO:
#[McpTool]attribute#[McpResource]Because MCP uses JSON-RPC under the hood with some particular response objects (from php-sdk) we can not change returned headers yet (we'll work on an extension point).