feat: add test of MCP

Author: alexislefebvre

src/Mcp/Capability/Registry/Loader.php

@@ -62,8 +62,7 @@ public function load(RegistryInterface $registry): void
                                 annotations: $mcp->getAnnotations() ? ToolAnnotations::fromArray($mcp->getAnnotations()) : null,
                                 icons: $mcp->getIcons(),
                                 meta: $mcp->getMeta(),
-                                outputSchema: $outputSchema->getDefinitions()[$outputSchema->getRootDefinitionKey()]->getArrayCopy(),
-                                // outputSchema: $outputSchema->getArrayCopy(),
+                                outputSchema: $outputSchema->getArrayCopy(),
                             ),
                             self::HANDLER,
                             true,

src/Mcp/composer.json

@@ -30,7 +30,8 @@
         "php": ">=8.2",
         "api-platform/metadata": "^4.2",
         "api-platform/json-schema": "^4.2",
-        "mcp/sdk": "^0.3.0"
+        "mcp/sdk": "^0.3.0",
+        "symfony/polyfill-php85": "^1.32"
     },
     "autoload": {
         "psr-4": {

tests/Fixtures/TestBundle/Dto/SearchDto.php

@@ -0,0 +1,19 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <dunglas@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\Tests\Fixtures\TestBundle\Dto;
+
+final class SearchDto
+{
+    public string $search;
+}

tests/Fixtures/TestBundle/Entity/McpBook.php

@@ -15,6 +15,9 @@
 
 use ApiPlatform\Metadata\ApiResource;
 use ApiPlatform\Metadata\McpTool;
+use ApiPlatform\Metadata\McpToolCollection;
+use ApiPlatform\Tests\Fixtures\TestBundle\Dto\SearchDto;
+use ApiPlatform\Tests\Fixtures\TestBundle\State\McpBookListProcessor;
 use Doctrine\ORM\Mapping as ORM;
 
 #[ApiResource(
@@ -27,9 +30,16 @@
         'update_book_status' => new McpTool(
             processor: [self::class, 'process']
         ),
+        'list_books' => new McpToolCollection(
+            description: 'List Books',
+            input: SearchDto::class,
+            processor: McpBookListProcessor::class,
+            structuredContent: true,
+        ),
     ]
 )]
 #[ORM\Entity]
+#[ORM\Table(name: 'mcp_book')]
 class McpBook
 {
     #[ORM\Column(type: 'integer')]

tests/Fixtures/TestBundle/State/McpBookListProcessor.php

@@ -0,0 +1,31 @@
+<?php
+
+/*
+* This file is part of the API Platform project.
+*
+* (c) Kévin Dunglas <dunglas@gmail.com>
+*
+* For the full copyright and license information, please view the LICENSE
+* file that was distributed with this source code.
+*/
+
+declare(strict_types=1);
+
+namespace ApiPlatform\Tests\Fixtures\TestBundle\State;
+
+use ApiPlatform\Metadata\Operation;
+use ApiPlatform\State\ProcessorInterface;
+use ApiPlatform\Tests\Fixtures\TestBundle\Entity\McpBook;
+use Doctrine\Persistence\ManagerRegistry;
+
+class McpBookListProcessor implements ProcessorInterface
+{
+    public function __construct(private readonly ManagerRegistry $managerRegistry)
+    {
+    }
+
+    public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): ?iterable
+    {
+        return $this->managerRegistry->getRepository(McpBook::class)->findAll();
+    }
+}

tests/Fixtures/app/config/config_common.yml

@@ -142,6 +142,11 @@ services:
         tags:
             - name: 'api_platform.state_processor'
 
+    ApiPlatform\Tests\Fixtures\TestBundle\State\McpBookListProcessor:
+        class: 'ApiPlatform\Tests\Fixtures\TestBundle\State\McpBookListProcessor'
+        arguments: [ '@doctrine' ]
+        tags:
+            - name: 'api_platform.state_processor'
 
     ApiPlatform\Tests\Fixtures\TestBundle\State\ContainNonResourceProvider:
         class: 'ApiPlatform\Tests\Fixtures\TestBundle\State\ContainNonResourceProvider'

tests/Functional/McpTest.php

@@ -418,6 +418,7 @@ public function testToolsList(): void
         self::assertContains('validate_input', $toolNames);
         self::assertContains('generate_markdown', $toolNames);
         self::assertContains('process_message', $toolNames);
+        self::assertContains('list_books', $toolNames);
 
         foreach ($tools as $tool) {
             self::assertArrayHasKey('name', $tool);
@@ -651,4 +652,88 @@ public function testMcpMarkdownContent(): void
         self::assertStringContainsString("echo 'Hello, World!';", $content);
         self::assertStringContainsString('```', $content);
     }
+
+    public function testMcpListBooks(): void
+    {
+        if (!class_exists(McpBundle::class)) {
+            $this->markTestSkipped('MCP bundle is not installed');
+        }
+
+        if ($this->isMongoDB()) {
+            $this->markTestSkipped('MCP is not supported with MongoDB');
+        }
+
+        if (!$this->isPsr17FactoryAvailable()) {
+            $this->markTestSkipped('PSR-17 HTTP factory implementation not available (required for MCP)');
+        }
+
+        $this->recreateSchema([
+            McpBook::class,
+        ]);
+
+        $book = new McpBook();
+        $book->setTitle('API Platform Guide for MCP');
+        $book->setIsbn('1-528491');
+        $book->setStatus('available');
+        $manager = $this->getContainer()->get('doctrine.orm.entity_manager');
+        $manager->persist($book);
+        $manager->flush();
+
+        $client = self::createClient();
+        $sessionId = $this->initializeMcpSession($client);
+
+        $res = $client->request('POST', '/mcp', [
+            'headers' => [
+                'Accept' => 'application/json, text/event-stream',
+                'Content-Type' => 'application/json',
+                'mcp-session-id' => $sessionId,
+            ],
+            'json' => [
+                'jsonrpc' => '2.0',
+                'id' => 2,
+                'method' => 'tools/call',
+                'params' => [
+                    'name' => 'list_books',
+                    'arguments' => [
+                        'search' => '',
+                    ],
+                ],
+            ],
+        ]);
+
+        self::assertResponseIsSuccessful();
+        $result = $res->toArray()['result'] ?? null;
+        self::assertIsArray($result);
+        self::assertArrayHasKey('content', $result);
+        $content = $result['content'][0]['text'] ?? null;
+        self::assertNotNull($content, 'No text content in result');
+        self::assertStringContainsString('API Platform Guide for MCP', $content);
+        self::assertStringContainsString('1-528491', $content);
+
+        $structuredContent = $result['structuredContent'] ?? null;
+        $this->assertIsArray($structuredContent);
+
+        // when api_platform.use_symfony_listeners is true, the result has JSON-LD format
+        if (true === $this->getContainer()->getParameter('api_platform.use_symfony_listeners')) {
+            self::assertArrayHasKeyAndValue('@context', '/contexts/McpBook', $structuredContent);
+            self::assertArrayHasKeyAndValue('hydra:totalItems', 1, $structuredContent);
+            $members = $structuredContent['hydra:member'];
+        } else {
+            $members = $structuredContent;
+        }
+
+        $this->assertCount(1, $members, json_encode($members, \JSON_PRETTY_PRINT));
+        $actualBook = array_first($members);
+
+        self::assertArrayHasKeyAndValue('id', 1, $actualBook);
+        self::assertArrayHasKeyAndValue('title', 'API Platform Guide for MCP', $actualBook);
+        self::assertArrayHasKeyAndValue('isbn', '1-528491', $actualBook);
+        self::assertArrayHasKeyAndValue('status', 'available', $actualBook);
+    }
+
+    private static function assertArrayHasKeyAndValue(string $key, mixed $value, array $data): void
+    {
+        self::assertArrayHasKey($key, $data);
+        self::assertSame($value, $data[$key]);
+    }
 }