Fix PR review comments for MCP Bundle migration

- Remove prefer-stable from root composer.json
- Move CHANGELOG content to 0.1 section and remove BC BREAK labels
- Use ServerBuilder::class and Server::class in services configuration
- Fix "client vs server" comment in documentation
- Use ServerBuilder::class in test instead of string

Author: camilleislasse

composer.json

@@ -12,7 +12,6 @@
         "symfony/finder": "^7.3|^8.0"
     },
     "minimum-stability": "dev",
-    "prefer-stable": true,
     "config": {
         "sort-packages": true
     }

src/mcp-bundle/CHANGELOG.md

@@ -1,20 +1,16 @@
 CHANGELOG
 =========
 
-0.2
+0.1
 ---
 
- * [BC BREAK] Migration from `symfony/mcp-sdk` to official `mcp/sdk`
- * [BC BREAK] Tool definition changed from interfaces to PHP attributes `#[McpTool]`
- * [BC BREAK] Automatic tool discovery via attributes instead of manual registration
- * [BC BREAK] Server creation now uses `Server::make()` builder pattern
+ * Migration from `symfony/mcp-sdk` to official `mcp/sdk`
+ * Tool definition changed from interfaces to PHP attributes `#[McpTool]`
+ * Automatic tool discovery via attributes instead of manual registration
+ * Server creation now uses `Server::make()` builder pattern
  * Updated STDIO transport command to `mcp:server`
  * Simplified service configuration using native SDK patterns
  * Enhanced auto-discovery scanning in `src/` directory
-
-0.1
----
-
  * Add Symfony bundle bridging MCP-SDK with Symfony applications
  * Add server mode exposing Symfony tools to MCP clients:
    - STDIO transport via `php bin/console mcp:server` command

src/mcp-bundle/config/services.php

@@ -11,6 +11,8 @@
 
 namespace Symfony\Component\DependencyInjection\Loader\Configurator;
 
+use Mcp\Server;
+use Mcp\Server\ServerBuilder;
 use Mcp\Server\Transport\Sse\Store\CachePoolStore;
 
 return static function (ContainerConfigurator $container): void {
@@ -19,13 +21,13 @@
             ->args([
                 service('cache.app'),
             ])
-        ->set('mcp.server.builder', 'Mcp\\Server\\ServerBuilder')
-            ->factory(['Mcp\\Server', 'make'])
+        ->set('mcp.server.builder', ServerBuilder::class)
+            ->factory([Server::class, 'make'])
             ->call('setServerInfo', [param('mcp.app'), param('mcp.version')])
             ->call('setLogger', [service('logger')])
             ->call('setDiscovery', [param('kernel.project_dir'), ['src']])
 
-        ->set('mcp.server', 'Mcp\\Server')
+        ->set('mcp.server', Server::class)
             ->factory([service('mcp.server.builder'), 'build'])
 
     ;

src/mcp-bundle/doc/index.rst

@@ -75,7 +75,7 @@ Configuration
             name:
                 transport: 'stdio' # Transport method to use, either 'stdio' or 'sse'
                 stdio:
-                    command: 'php /path/bin/console mcp:server' # Command to execute to start the client
+                    command: 'php /path/bin/console mcp:server' # Command to execute to start the server
                     arguments: [] # Arguments to pass to the command
                 sse:
                     url: 'http://localhost:8000/sse' # URL to SSE endpoint of MCP server

src/mcp-bundle/tests/DependencyInjection/McpToolPassTest.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\AI\McpBundle\Tests\DependencyInjection;
 
+use Mcp\Server\ServerBuilder;
 use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use Symfony\AI\McpBundle\DependencyInjection\McpToolPass;
@@ -23,7 +24,7 @@ public function testCreatesServiceLocatorForTaggedServices()
     {
         $container = new ContainerBuilder();
 
-        $container->register('mcp.server.builder', 'Mcp\\Server\\ServerBuilder');
+        $container->register('mcp.server.builder', ServerBuilder::class);
 
         $container->register('tool1', 'MockTool1')
             ->addTag('mcp.tool');
@@ -53,7 +54,7 @@ public function testDoesNothingWhenNoToolsTagged()
     {
         $container = new ContainerBuilder();
 
-        $container->register('mcp.server.builder', 'Mcp\\Server\\ServerBuilder');
+        $container->register('mcp.server.builder', ServerBuilder::class);
         $container->register('service1', 'MockService');
         $container->register('service2', 'MockService');