refactor: Update client configuration to use withClientInfo method for name and version

Author: CodeWithKyrian

README.md

@@ -87,8 +87,7 @@ $fsServerConfig = new ServerConfig(
 );
 
 $fsClient = Client::make()
-    ->withName('MyFileSystemApp')
-    ->withVersion('1.0')
+    ->withClientInfo('MyFileSystemApp', '1.0')
     ->withCapabilities($clientCapabilities)
     // ->withLogger(new MyPsrLogger()) // Optional
     ->withServerConfig($fsServerConfig)
@@ -227,15 +226,14 @@ use PhpMcp\Client\Client;
 // ... other use statements for Config, Logger etc...
 
 $client = Client::make()
-    ->withName($clientName)                 // Required
-    ->withVersion($clientVersion)           // Required
-    ->withCapabilities($clientCapabilities) // Optional (defaults provided)
-    ->withServerConfig($stdioConfig)        // Required: Config for THE server
-    ->withLogger($myLogger)                 // Optional
-    ->withCache($myCache, 3600)             // Optional (cache + TTL)
-    ->withEventDispatcher($myDispatcher)    // Optional
-    ->withIdGenerator($myIdGenerator)       // Optional
-    ->withLoop($myEventLoop)                // Optional (defaults to Loop::get())
+    ->withClientInfo($clientName, $clientVersion) // Required
+    ->withCapabilities($clientCapabilities)       // Optional (defaults provided)
+    ->withServerConfig($stdioConfig)              // Required: Config for THE server
+    ->withLogger($myLogger)                       // Optional
+    ->withCache($myCache, 3600)                   // Optional (cache + TTL)
+    ->withEventDispatcher($myDispatcher)          // Optional
+    ->withIdGenerator($myIdGenerator)             // Optional
+    ->withLoop($myEventLoop)                      // Optional (defaults to Loop::get())
     ->build();
 ```
 

examples/01-simple-stdio-sync.php

@@ -34,8 +34,7 @@
 );
 
 $stdioClient = Client::make()
-    ->withName($clientName)
-    ->withVersion($clientVersion)
+    ->withClientInfo($clientName, $clientVersion)
     ->withCapabilities($clientCapabilities)
     ->withLogger($logger)
     ->withServerConfig($stdioServerConfig)

examples/02-simple-http-sync.php

@@ -31,8 +31,7 @@
 );
 
 $httpClient = Client::make()
-    ->withName($clientName)
-    ->withVersion($clientVersion)
+    ->withClientInfo($clientName, $clientVersion)
     ->withCapabilities($clientCapabilities)
     ->withLogger($logger)
     ->withServerConfig($httpServerConfig)

examples/03-multiple-servers-sync.php

@@ -39,16 +39,14 @@
 
 // --- Instantiate Clients ---
 $stdioClient = Client::make()
-    ->withName($clientName)
-    ->withVersion($clientVersion)
+    ->withClientInfo($clientName, $clientVersion)
     ->withCapabilities($clientCapabilities)
     ->withLogger($logger)
     ->withServerConfig($stdioServerConfig)
     ->build();
 
 $httpClient = Client::make()
-    ->withName($clientName)
-    ->withVersion($clientVersion)
+    ->withClientInfo($clientName, $clientVersion)
     ->withCapabilities($clientCapabilities)
     ->withLogger($logger)
     ->withServerConfig($httpServerConfig)

examples/04-multiple-servers-async.php

@@ -44,17 +44,15 @@
 
 // --- Build Clients ---
 $stdioClient = Client::make()
-    ->withName($clientName)
-    ->withVersion($clientVersion)
+    ->withClientInfo($clientName, $clientVersion)
     ->withCapabilities($clientCapabilities)
     ->withLogger($logger)
     ->withLoop($loop)
     ->withServerConfig($stdioServerConfig)
     ->build();
 
 $httpClient = Client::make()
-    ->withName($clientName)
-    ->withVersion($clientVersion)
+    ->withClientInfo($clientName, $clientVersion)
     ->withCapabilities($clientCapabilities)
     ->withLogger($logger)
     ->withLoop($loop)

examples/05-openai-php-integration-sync/client.php

@@ -76,8 +76,7 @@
 foreach ($serversToConfigure as $serverName => $serverConfig) {
     try {
         $mcpClients[$serverName] = Client::make()
-            ->withName($clientName)
-            ->withVersion($clientVersion)
+            ->withClientInfo($clientName, $clientVersion)
             ->withCapabilities($clientCapabilities)
             ->withLogger($logger)
             ->withServerConfig($serverConfig)

src/ClientBuilder.php

@@ -4,6 +4,7 @@
 
 namespace PhpMcp\Client;
 
+use Deprecated;
 use PhpMcp\Client\Exception\ConfigurationException;
 use PhpMcp\Client\Factory\MessageIdGenerator;
 use PhpMcp\Client\Factory\TransportFactory; // Added use
@@ -50,20 +51,30 @@ public static function make(): self
         return new self;
     }
 
+    /** @deprecated 1.0.1 Use withClientInfo() instead. */
     public function withName(string $name): self
     {
         $this->name = $name;
 
         return $this;
     }
 
+    /** @deprecated 1.0.1 Use withClientInfo() instead. */
     public function withVersion(string $version): self
     {
         $this->version = $version;
 
         return $this;
     }
 
+    public function withClientInfo(string $name, string $version): self
+    {
+        $this->name = $name;
+        $this->version = $version;
+
+        return $this;
+    }
+
     public function withCapabilities(ClientCapabilities $capabilities): self
     {
         $this->capabilities = $capabilities;

tests/Unit/ClientBuilderTest.php

@@ -48,8 +48,7 @@
     $dispatcher = Mockery::mock(EventDispatcherInterface::class);
     $loop = Mockery::mock(LoopInterface::class);
     $idGen = new MessageIdGenerator('test-');
-    $server1 = new ServerConfig(name: 's1', transport: TransportType::Stdio, command: 'c');
-    $server2 = new ServerConfig(name: 's2', transport: TransportType::Http, url: 'http://s');
+    $server = new ServerConfig(name: 's1', transport: TransportType::Stdio, command: 'c');
 
     // Act
     $client = Client::make()
@@ -61,7 +60,7 @@
         ->withEventDispatcher($dispatcher)
         ->withLoop($loop)
         ->withIdGenerator($idGen)
-        ->withServerConfig($server1)
+        ->withServerConfig($server)
         ->build();
 
     // Assert
@@ -81,6 +80,26 @@
     expect($internalConfig->loop)->toBe($loop);
 });
 
+it('builds client with client info', function () {
+    $server = new ServerConfig(name: 's1', transport: TransportType::Stdio, command: 'c');
+
+    // Arrange
+    $client = Client::make()
+        ->withClientInfo('TestClient', '1.0')
+        ->withServerConfig($server)
+        ->build();
+
+    // Assert
+    expect($client)->toBeInstanceOf(Client::class);
+    $reflector = new ReflectionClass($client);
+    $configProp = $reflector->getProperty('clientConfig');
+    $configProp->setAccessible(true);
+    $internalConfig = $configProp->getValue($client);
+
+    expect($internalConfig->name)->toBe('TestClient');
+    expect($internalConfig->version)->toBe('1.0');
+});
+
 it('throws exception if client name not provided', function () {
     Client::make()->build();
 })->throws(ConfigurationException::class, 'Name must be provided using withName().');