Revert "Add outputSchema to tool schemas"

This reverts commit 69ff8d5.

Author: trandbert37

src/Services/ToolService/BaseToolInterface.php

@@ -46,24 +46,6 @@ public function getDescription(): string;
      */
     public function getInputSchema(): array;
 
-    /**
-     * Gets the JSON Schema definition for the tool's output parameters.
-     *
-     * The schema defines the expected structure, types, and constraints for the
-     * expected output structure. This schema is used
-     * to provide type hints to LLM clients.
-     *
-     * @return array The JSON Schema as an associative array. Common structure:
-     *               [
-     *               'type' => 'object',
-     *               'properties' => [
-     *               'param' => ['type' => 'string', 'description' => '...'],
-     *               ],
-     *               'required' => ['param'],
-     *               ]
-     */
-    public function getOutputSchema(): array;
-
     /**
      * Gets the behavioral annotations for the tool.
      *

src/Services/ToolService/Examples/CodeAnalyzerTool.php

@@ -51,11 +51,6 @@ public function getInputSchema(): array
         ];
     }
 
-    public function getOutputSchema(): array
-    {
-        return [];
-    }
-
     public function execute(array $arguments): ToolResultInterface
     {
         $code = $arguments['code'];

src/Services/ToolService/Examples/HelloWorldTool.php

@@ -34,11 +34,6 @@ public function getInputSchema(): array
         ];
     }
 
-    public function getOutputSchema(): array
-    {
-        return [];
-    }
-
     public function getAnnotations(): ToolAnnotation
     {
         return new ToolAnnotation;

src/Services/ToolService/Examples/ProfileGeneratorTool.php

@@ -59,11 +59,6 @@ public function getInputSchema(): array
         ];
     }
 
-    public function getOutputSchema(): array
-    {
-        return [];
-    }
-
     public function getAnnotations(): ToolAnnotation
     {
         return new ToolAnnotation;

src/Services/ToolService/Examples/StreamingDataTool.php

@@ -52,11 +52,6 @@ public function getInputSchema(): array
         ];
     }
 
-    public function getOutputSchema(): array
-    {
-        return [];
-    }
-
     public function getAnnotations(): ToolAnnotation
     {
         return new ToolAnnotation(

src/Services/ToolService/Examples/VersionCheckTool.php

@@ -31,11 +31,6 @@ public function getInputSchema(): array
         ];
     }
 
-    public function getOutputSchema(): array
-    {
-        return [];
-    }
-
     public function getAnnotations(): ToolAnnotation
     {
         return new ToolAnnotation;

src/Services/ToolService/ToolRepository.php

@@ -109,7 +109,7 @@ public function getTool(string $name): ?StreamableToolInterface
      * Generates an array of schemas for all registered tools, suitable for the MCP capabilities response.
      * Includes name, description, inputSchema, and optional annotations for each tool.
      *
-     * @return array<int, array{name: string, description: string, inputSchema: array<string, mixed>, outputSchema: array<string, mixed>, annotations?: array<string, mixed>}> An array of tool schemas.
+     * @return array<int, array{name: string, description: string, inputSchema: array<string, mixed>, annotations?: array<string, mixed>}> An array of tool schemas.
      */
     public function getToolSchemas(): array
     {
@@ -129,7 +129,6 @@ public function getToolSchemas(): array
                 'name' => $tool->getName(),
                 'description' => $tool->getDescription(),
                 'inputSchema' => $tool->getInputSchema(),
-                'outputSchema' => $tool->getOutputSchema(),
                 'annotations' => $tool->getAnnotations()->toArray(),
                 ...$injectArray,
             ];

tests/Server/Request/ToolsCallHandlerTest.php

@@ -515,11 +515,6 @@ public function getInputSchema(): array
                 return ['type' => 'object', 'properties' => new \stdClass, 'required' => []];
             }
 
-            public function getOutputSchema(): array
-            {
-                return [];
-            }
-
             public function getAnnotations(): ToolAnnotation
             {
                 return new ToolAnnotation;
@@ -791,11 +786,6 @@ public function getInputSchema(): array
                 return ['type' => 'object', 'properties' => new \stdClass, 'required' => []];
             }
 
-            public function getOutputSchema(): array
-            {
-                return [];
-            }
-
             public function getAnnotations(): ToolAnnotation
             {
                 return new ToolAnnotation;

tests/Services/ToolService/ToolRepositoryTest.php

@@ -254,30 +254,4 @@ public function test_get_tool_schemas_includes_annotations_if_present(): void
             'openWorldHint' => true,
         ], $schemas[0]['annotations']);
     }
-
-    /**
-     * Tests that getToolSchemas() includes outputSchema if it is present in a tool.
-     *
-     * Verifies that the method appends the outputSchema to the tool schema, ensuring
-     * that metadata is preserved and available for the MCP capabilities response.
-     */
-    public function test_get_tool_schemas_includes_output_schema_if_present(): void
-    {
-        $tool = $this->createMock(StreamableToolInterface::class);
-
-        $tool->method('getName')->willReturn('toolWithOutputSchema');
-        $tool->method('getDescription')->willReturn('Tool with outputSchema');
-        $tool->method('getInputSchema')->willReturn(['type' => 'object']);
-        $tool->method('getOutputSchema')->willReturn(['type' => 'object']);
-
-        $this->toolRepository->register($tool);
-
-        $schemas = $this->toolRepository->getToolSchemas();
-
-        $this->assertCount(1, $schemas);
-        $this->assertSame('toolWithOutputSchema', $schemas[0]['name']);
-        $this->assertSame('Tool with outputSchema', $schemas[0]['description']);
-        $this->assertSame(['type' => 'object'], $schemas[0]['inputSchema']);
-        $this->assertSame(['type' => 'object'], $schemas[0]['outputSchema']);
-    }
 }