feat: mcp bundle tool integration #7595

soyuka · 2025-12-09T10:10:23Z

Q	A
Branch?	main
Tickets	na
License	MIT
Doc PR	needed

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:

automatic input JSON Schema's (waiting for feat: Add output schema support to MCP tools modelcontextprotocol/php-sdk#153 for output schema compatiblity)
automatic structured content
reuses the Symfony serializer under the hood (with all the capabilities of API Platform including optional iris using item_uri_template)
allows to return a custom CallToolResult (from the php-sdk itself for images, sound etc.)
ability to reuse Symfony validation constraints as usual
works with doctrine entities if needed (calls the provider and the processor on demand)
integrates with the symfony security features

TODO:

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).

rvanlaak

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/Capability/Registry/Loader.php

src/Mcp/Metadata/Operation/Factory/OperationMetadataFactory.php

src/Symfony/Bundle/DependencyInjection/ApiPlatformExtension.php

src/Metadata/McpTool.php

soyuka · 2026-01-12T13:21:43Z

Exciting that the mcp repo's schema PR is merged now, and you're able to spend time on API Platform integration.

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!

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

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

Copilot

Pull request overview

This PR integrates MCP (Model Context Protocol) tool and resource capabilities into API Platform, enabling developers to expose their API resources as MCP tools and resources with automatic JSON Schema generation, structured content support, and seamless integration with Symfony's serializer and validation systems.

Changes:

Added McpTool and McpResource metadata classes that extend HttpOperation for defining MCP capabilities
Implemented MCP-specific state providers and processors for handling tool calls and resource reads
Added comprehensive test coverage for MCP functionality including validation, custom results, and markdown content

Reviewed changes

Copilot reviewed 28 out of 28 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/Symfony/Bundle/DependencyInjection/ConfigurationTest.php	Added test assertion for new `mcp.enabled` configuration option
tests/Functional/McpTest.php	Added comprehensive functional tests covering MCP tools, resources, validation, and content handling
tests/Fixtures/app/config/routing_test.php	Added MCP route imports and removed outdated comments
tests/Fixtures/app/config/config_common.yml	Added MCP bundle configuration with HTTP transport and session settings
tests/Fixtures/app/AppKernel.php	Registered McpBundle and MonologBundle when MCP is available
tests/Fixtures/TestBundle/Entity/McpBook.php	Created test entity demonstrating MCP tool integration with Doctrine entities
tests/Fixtures/TestBundle/ApiResource/McpWithMarkdown.php	Created test resource for markdown content generation
tests/Fixtures/TestBundle/ApiResource/McpTools.php	Created test resource demonstrating custom results and validation
tests/Fixtures/TestBundle/ApiResource/McpToolAttribute.php	Created test resource using `#[McpTool]` attribute
tests/Fixtures/TestBundle/ApiResource/McpResourceExample.php	Created test resource using `#[McpResource]` attribute
src/Symfony/Bundle/Resources/config/mcp/state.php	Configured MCP state processors and handler service
src/Symfony/Bundle/Resources/config/mcp/mcp.php	Configured core MCP services including loader and IRI converter
src/Symfony/Bundle/Resources/config/mcp/events.php	Configured MCP event-based state processing
src/Symfony/Bundle/DependencyInjection/Configuration.php	Added `mcp` configuration section
src/Symfony/Bundle/DependencyInjection/ApiPlatformExtension.php	Registered MCP metadata classes and loaded MCP service configuration
src/Metadata/Tests/Extractor/Adapter/resources.yaml	Added `mcp` field to test resources
src/Metadata/Tests/Extractor/Adapter/XmlResourceAdapter.php	Added support for building MCP operations in XML
src/Metadata/Resource/Factory/MetadataCollectionFactoryTrait.php	Added logic to process MCP operations during resource metadata building
src/Metadata/McpTool.php	Created metadata class for defining MCP tools
src/Metadata/McpResource.php	Created metadata class for defining MCP resources
src/Metadata/ApiResource.php	Added `mcp` property to store MCP operations
src/Mcp/State/ToolProvider.php	Implemented state provider for MCP tool requests
src/Mcp/State/StructuredContentProcessor.php	Implemented processor for generating structured MCP responses
src/Mcp/Server/Handler.php	Implemented MCP request handler
src/Mcp/Routing/IriConverter.php	Added IRI converter decorator for MCP operations
src/Mcp/Metadata/Operation/Factory/OperationMetadataFactory.php	Implemented factory for retrieving MCP operation metadata
src/Mcp/Capability/Registry/Loader.php	Implemented loader for registering MCP tools and resources
composer.json	Added MCP SDK and bundle dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

tests/Fixtures/app/config/routing_test.php

src/Symfony/Bundle/Resources/config/mcp/events.php

src/Mcp/Capability/Registry/Loader.php

src/Symfony/Bundle/DependencyInjection/ApiPlatformExtension.php

VincentLanglet · 2026-01-16T16:10:28Z

composer.json

        "symfony/intl": "^6.4 || ^7.0 || ^8.0",
        "symfony/json-streamer": "^7.4 || ^8.0",
        "symfony/maker-bundle": "^1.24",
+        "symfony/mcp-bundle": "dev-main",


There is available tags now
https://github.com/symfony/mcp-bundle/tags

VincentLanglet · 2026-01-16T16:11:49Z

src/Mcp/Metadata/Operation/Factory/OperationMetadataFactory.php

+    /**
+     * @throws RuntimeException
+     *
+     * @return HttpOperation


Then you could use it as native typehint no ?

VincentLanglet · 2026-01-16T16:14:43Z

src/Metadata/ApiResource.php

-     * @param mixed|null                                                                      $messenger
-     * @param mixed|null                                                                      $input
-     * @param mixed|null                                                                      $output
+     * @param array<int, HttpOperation>|array<string, HttpOperation>|Operations|null $operations   Operations is a list of HttpOperation


array<int, HttpOperation>|array<string, HttpOperation>

could basically be resume to HttpOperation[]

VincentLanglet · 2026-01-16T16:15:25Z

src/Metadata/ApiResource.php

        }
    }

+    public function getMcp(): ?array


@return @param array<string, McpResource|McpTool>|null

VincentLanglet · 2026-01-16T16:15:38Z

src/Metadata/ApiResource.php

+        return $this->mcp;
+    }
+
+    public function withMcp(array $mcp): static


@param array<string, McpResource|McpTool> $mcp

VincentLanglet · 2026-01-16T16:16:30Z

src/Metadata/McpResource.php

+        return $this->meta;
+    }
+
+    public function withMeta(?array $meta): static


array<string, mixed>|null

VincentLanglet · 2026-01-16T16:16:59Z

src/Metadata/McpTool.php

+        return $self;
+    }
+
+    public function getMeta(): ?array


array<string, mixed>|null

VincentLanglet · 2026-01-16T16:17:06Z

src/Metadata/McpTool.php

+        return $this->meta;
+    }
+
+    public function withMeta(?array $meta): static


array<string, mixed>|null

VincentLanglet · 2026-01-16T16:18:50Z

src/Metadata/Tests/Extractor/Adapter/XmlResourceAdapter.php

+        foreach ($values as $value) {
+            $child = $node->addChild('mcpOperation');
+            foreach ($value as $index => $data) {
+                if (method_exists($this, 'build'.ucfirst($index))) {


What if $index is numeric ? Is it possible ?
Since it's an array key, it could be casted to int and then ucfirst($int) will crash.

VincentLanglet · 2026-01-16T16:19:20Z

src/Metadata/Tests/Extractor/Adapter/XmlResourceAdapter.php

+            $child = $node->addChild('mcpOperation');
+            foreach ($value as $index => $data) {
+                if (method_exists($this, 'build'.ucfirst($index))) {
+                    $this->{'build'.ucfirst($index)}($child, $data);


you could put 'build'.ucfirst($index) into a variable eventually

dunglas

Partial review

dunglas · 2026-01-16T00:12:49Z

src/Mcp/Capability/Registry/Loader.php

+                                meta: $mcp->getMeta()
+                            ),
+                            self::HANDLER,
+                            true


Suggested change

true

true,

dunglas · 2026-01-16T00:13:56Z

src/Mcp/Metadata/Operation/Factory/OperationMetadataFactory.php

+                        return $operation;
+                    }
+
+                    if ($operation instanceof McpResource && $operation->getUri() === $operationName) {


Nit: can be merged with the previous condition.

dunglas · 2026-01-16T00:15:31Z

composer.json

+        "symfony/mcp-bundle": "dev-main",
        "symfony/mercure-bundle": "*",
        "symfony/messenger": "^6.4 || ^7.0 || ^8.0",
+        "symfony/monolog-bundle": "^4.0",


dunglas · 2026-01-16T00:15:50Z

src/Mcp/Capability/Registry/Loader.php

+                                outputSchema: $outputSchema->getDefinitions()[$outputSchema->getRootDefinitionKey()]->getArrayCopy(),
+                            ),
+                            self::HANDLER,
+                            true


Suggested change

true

true,

dunglas · 2026-01-16T00:18:09Z

src/Mcp/Server/Handler.php

+
+declare(strict_types=1);
+
+/*


To adapt or remove?

dunglas · 2026-01-16T00:18:09Z

src/Mcp/Server/Handler.php

+
+declare(strict_types=1);
+
+/*


To adapt or remove?

soyuka force-pushed the mcp-bundle-integration branch 3 times, most recently from d136520 to b904a0d Compare January 12, 2026 08:37

rvanlaak reviewed Jan 12, 2026

View reviewed changes

soyuka mentioned this pull request Jan 12, 2026

[MCP Bundle] bump php-sdk to 0.3 symfony/ai#1377

Merged

soyuka force-pushed the mcp-bundle-integration branch 5 times, most recently from 192498f to 7ff8ba8 Compare January 14, 2026 10:55

soyuka mentioned this pull request Jan 14, 2026

fix(metadata): enforce HttpOperation when needed #7671

Closed

soyuka force-pushed the mcp-bundle-integration branch 3 times, most recently from 13f91e9 to cc5f55c Compare January 15, 2026 12:18

soyuka requested a review from Copilot January 15, 2026 20:38

Copilot AI reviewed Jan 15, 2026

View reviewed changes

feat: mcp bundle tool integration

f750e9c

soyuka force-pushed the mcp-bundle-integration branch from 320f687 to f750e9c Compare January 15, 2026 20:57

VincentLanglet reviewed Jan 16, 2026

View reviewed changes

dunglas reviewed Jan 16, 2026

View reviewed changes


		declare(strict_types=1);

		/*


		declare(strict_types=1);

		/*

Uh oh!

feat: mcp bundle tool integration #7595

Are you sure you want to change the base?

feat: mcp bundle tool integration #7595

Conversation

soyuka commented Dec 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rvanlaak left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

soyuka commented Jan 12, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

dunglas left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

soyuka commented Dec 9, 2025 •

edited

Loading