Fixed missing string representations of client and transports

Author: thekid

src/main/php/io/modelcontextprotocol/McpClient.class.php

@@ -1,5 +1,6 @@
 <?php namespace io\modelcontextprotocol;
 
+use lang\Value;
 use util\log\Traceable;
 
 /**
@@ -8,7 +9,7 @@
  * @see  https://deadprogrammersociety.com/2025/03/calling-mcp-servers-the-hard-way.html
  * @test io.modelcontextprotocol.unittest.McpClientTest
  */
-class McpClient implements Traceable {
+class McpClient implements Traceable, Value {
   private $transport, $version, $capabilities;
   private $server= null;
 
@@ -92,4 +93,24 @@ public function close() {
     $this->transport->close();
     $this->server= null;
   }
+
+  /** @return string */
+  public function hashCode() {
+    return 'M'.$this->version.':'.$this->transport->hashCode();
+  }
+
+  /** @return string */
+  public function toString() {
+    return nameof($this).'(->'.$this->transport->toString().', version= '.$this->version.')';
+  }
+
+  /**
+   * Comparison
+   *
+   * @param  var $value
+   * @return int
+   */
+  public function compareTo($value) {
+    return $value instanceof self ? $this->hashCode() <=> $value->hashCode() : 1;
+  }
 }

src/main/php/io/modelcontextprotocol/StdIo.class.php

@@ -94,4 +94,24 @@ public function close() {
       $this->process->close();
     }
   }
+
+  /** @return string */
+  public function hashCode() {
+    return 'I'.crc32($this->process->getCommandLine());
+  }
+
+  /** @return string */
+  public function toString() {
+    return nameof($this).'<'.$this->process->getCommandLine().'>';
+  }
+
+  /**
+   * Comparison
+   *
+   * @param  var $value
+   * @return int
+   */
+  public function compareTo($value) {
+    return $value instanceof self ? $this->process <=> $value->process : 1;
+  }
 }

src/main/php/io/modelcontextprotocol/StreamableHttp.class.php

@@ -1,8 +1,8 @@
 <?php namespace io\modelcontextprotocol;
 
 use lang\FormatException;
-use webservices\rest\Endpoint;
 use util\URI;
+use webservices\rest\Endpoint;
 
 /**
  * Streamable HTTP MCP transport
@@ -125,4 +125,24 @@ public function close() {
     $this->endpoint->resource($this->path)->delete();
     $this->endpoint->with(self::SESSION, null);
   }
+
+  /** @return string */
+  public function hashCode() {
+    return 'H'.$this->endpoint->base()->hashCode();
+  }
+
+  /** @return string */
+  public function toString() {
+    return nameof($this).'<'.$this->endpoint->base().'>';
+  }
+
+  /**
+   * Comparison
+   *
+   * @param  var $value
+   * @return int
+   */
+  public function compareTo($value) {
+    return $value instanceof self ? $this->endpoint->compareTo($value->endpoint) : 1;
+  }
 }

src/main/php/io/modelcontextprotocol/Transport.class.php

@@ -1,11 +1,11 @@
 <?php namespace io\modelcontextprotocol;
 
-use lang\{Closeable, CommandLine, IllegalArgumentException};
+use lang\{Closeable, CommandLine, IllegalArgumentException, Value};
 use util\URI;
 use util\log\Traceable;
 
 /** @test io.modelcontextprotocol.unittest.TransportTest */
-abstract class Transport implements Closeable, Traceable {
+abstract class Transport implements Closeable, Traceable, Value {
 
   /** @param string $version */ 
   public abstract function version($version);

src/test/php/io/modelcontextprotocol/unittest/McpClientTest.class.php

@@ -64,6 +64,14 @@ public function call($method, $params= null) {
       }
 
       public function close() { }
+
+      public function hashCode() { return spl_object_id($this); }
+
+      public function toString() { return nameof($this); }
+
+      public function compareTo($value) {
+        return $value instanceof self ? $this <=> $value : 1;
+      }
     };
     $fixture= new McpClient($transport);
 
@@ -87,4 +95,12 @@ public function close() { }
       $transport->sent
     );
   }
+
+  #[Test]
+  public function string_representation() {
+    Assert::equals(
+      'io.modelcontextprotocol.McpClient(->io.modelcontextprotocol.StreamableHttp<http://localhost/>, version= 2025-06-18)',
+      (new McpClient(self::ENDPOINT))->toString()
+    );
+  }
 }