fix: simplify exception signatures to accept simple messages

Author: CodeWithKyrian

src/Exception/PromptGetException.php

@@ -11,17 +11,9 @@
 
 namespace Mcp\Exception;
 
-use Mcp\Schema\Request\GetPromptRequest;
-
 /**
  * @author Tobias Nyholm <[email protected]>
  */
 final class PromptGetException extends \RuntimeException implements ExceptionInterface
 {
-    public function __construct(
-        public readonly GetPromptRequest $request,
-        ?\Throwable $previous = null,
-    ) {
-        parent::__construct(\sprintf('Handling prompt "%s" failed with error: "%s".', $request->name, $previous->getMessage()), previous: $previous);
-    }
 }

src/Exception/ResourceReadException.php

@@ -11,17 +11,9 @@
 
 namespace Mcp\Exception;
 
-use Mcp\Schema\Request\ReadResourceRequest;
-
 /**
  * @author Tobias Nyholm <[email protected]>
  */
 final class ResourceReadException extends \RuntimeException implements ExceptionInterface
 {
-    public function __construct(
-        public readonly ReadResourceRequest $request,
-        ?\Throwable $previous = null,
-    ) {
-        parent::__construct(\sprintf('Reading resource "%s" failed with error: "%s".', $request->uri, $previous?->getMessage() ?? ''), previous: $previous);
-    }
 }

src/Exception/ToolCallException.php

@@ -11,17 +11,9 @@
 
 namespace Mcp\Exception;
 
-use Mcp\Schema\Request\CallToolRequest;
-
 /**
  * @author Tobias Nyholm <[email protected]>
  */
 final class ToolCallException extends \RuntimeException implements ExceptionInterface
 {
-    public function __construct(
-        public readonly CallToolRequest $request,
-        ?\Throwable $previous = null,
-    ) {
-        parent::__construct(\sprintf('Tool call "%s" failed with error: "%s".', $request->name, $previous?->getMessage() ?? ''), previous: $previous);
-    }
 }

tests/Unit/Server/Handler/Request/CallToolHandlerTest.php

@@ -180,7 +180,7 @@ public function testHandleToolNotFoundExceptionReturnsError(): void
     public function testHandleToolCallExceptionReturnsResponseWithErrorResult(): void
     {
         $request = $this->createCallToolRequest('failing_tool', ['param' => 'value']);
-        $exception = new ToolCallException($request, new \RuntimeException('Tool execution failed'));
+        $exception = new ToolCallException('Tool execution failed');
 
         $toolReference = $this->createMock(ToolReference::class);
         $this->referenceProvider
@@ -209,7 +209,7 @@ public function testHandleToolCallExceptionReturnsResponseWithErrorResult(): voi
         $this->assertTrue($result->isError);
         $this->assertCount(1, $result->content);
         $this->assertInstanceOf(TextContent::class, $result->content[0]);
-        $this->assertEquals('Tool call "failing_tool" failed with error: "Tool execution failed".', $result->content[0]->text);
+        $this->assertEquals('Tool execution failed', $result->content[0]->text);
     }
 
     public function testHandleWithNullResult(): void
@@ -252,7 +252,7 @@ public function testConstructorWithDefaultLogger(): void
     public function testHandleLogsErrorWithCorrectParameters(): void
     {
         $request = $this->createCallToolRequest('test_tool', ['key1' => 'value1', 'key2' => 42]);
-        $exception = new ToolCallException($request, new \RuntimeException('Custom error message'));
+        $exception = new ToolCallException('Custom error message');
 
         $toolReference = $this->createMock(ToolReference::class);
         $this->referenceProvider
@@ -271,7 +271,7 @@ public function testHandleLogsErrorWithCorrectParameters(): void
             ->expects($this->once())
             ->method('error')
             ->with(
-                'Error while executing tool "test_tool": "Tool call "test_tool" failed with error: "Custom error message".".',
+                'Error while executing tool "test_tool": "Custom error message".',
                 [
                     'tool' => 'test_tool',
                     'arguments' => ['key1' => 'value1', 'key2' => 42, '_session' => $this->session],
@@ -289,7 +289,7 @@ public function testHandleLogsErrorWithCorrectParameters(): void
         $this->assertTrue($result->isError);
         $this->assertCount(1, $result->content);
         $this->assertInstanceOf(TextContent::class, $result->content[0]);
-        $this->assertEquals('Tool call "test_tool" failed with error: "Custom error message".', $result->content[0]->text);
+        $this->assertEquals('Custom error message', $result->content[0]->text);
     }
 
     public function testHandleGenericExceptionReturnsError(): void

tests/Unit/Server/Handler/Request/GetPromptHandlerTest.php

@@ -250,7 +250,7 @@ public function testHandlePromptNotFoundExceptionReturnsError(): void
     public function testHandlePromptGetExceptionReturnsError(): void
     {
         $request = $this->createGetPromptRequest('failing_prompt');
-        $exception = new PromptGetException($request, new \RuntimeException('Failed to get prompt'));
+        $exception = new PromptGetException('Failed to get prompt');
 
         $this->referenceProvider
             ->expects($this->once())
@@ -263,7 +263,7 @@ public function testHandlePromptGetExceptionReturnsError(): void
         $this->assertInstanceOf(Error::class, $response);
         $this->assertEquals($request->getId(), $response->id);
         $this->assertEquals(Error::INTERNAL_ERROR, $response->code);
-        $this->assertEquals('Handling prompt "failing_prompt" failed with error: "Failed to get prompt".', $response->message);
+        $this->assertEquals('Failed to get prompt', $response->message);
     }
 
     public function testHandlePromptGetWithComplexArguments(): void

tests/Unit/Server/Handler/Request/ReadResourceHandlerTest.php

@@ -198,10 +198,7 @@ public function testHandleResourceReadExceptionReturnsActualErrorMessage(): void
     {
         $uri = 'file://corrupted/file.txt';
         $request = $this->createReadResourceRequest($uri);
-        $exception = new ResourceReadException(
-            $request,
-            new \RuntimeException('Failed to read resource: corrupted data'),
-        );
+        $exception = new ResourceReadException('Failed to read resource: corrupted data');
 
         $this->referenceProvider
             ->expects($this->once())
@@ -214,7 +211,7 @@ public function testHandleResourceReadExceptionReturnsActualErrorMessage(): void
         $this->assertInstanceOf(Error::class, $response);
         $this->assertEquals($request->getId(), $response->id);
         $this->assertEquals(Error::INTERNAL_ERROR, $response->code);
-        $this->assertEquals('Reading resource "file://corrupted/file.txt" failed with error: "Failed to read resource: corrupted data".', $response->message);
+        $this->assertEquals('Failed to read resource: corrupted data', $response->message);
     }
 
     public function testHandleGenericExceptionReturnsGenericError(): void