Drop reverse route tool because complex schema issues

Author: josbeir

README.md

@@ -225,7 +225,6 @@ Inspect and analyze your application routes:
 - `list_routes` - List all routes with filtering and sorting
 - `get_route` - Get detailed information about a specific route
 - `match_url` - Find which route matches a given URL
-- `reverse_route` - Generate URLs from route names or parameters
 - `detect_route_collisions` - Find potential route conflicts
 
 ### Documentation Search

src/Mcp/RouteTools.php

@@ -8,7 +8,6 @@
 use Cake\Routing\Router;
 use Exception;
 use Mcp\Capability\Attribute\McpTool;
-use Mcp\Capability\Attribute\Schema;
 use Mcp\Exception\ToolCallException;
 
 /**
@@ -182,60 +181,6 @@ public function matchUrl(string $url, string $method = 'GET'): array
         }
     }
 
-    /**
-     * Generate URL from route parameters (reverse routing).
-     *
-     * Creates a URL from either a named route or route parameters.
-     * Supports generating full URLs with domain.
-     *
-     * @param string|null $name Named route (e.g., 'projects:view')
-     * @param array<string, mixed> $params Route parameters
-     * @param bool $full Generate full URL with domain
-     * @return array<string, mixed> Generated URL or error
-     */
-    #[McpTool(
-        name: 'reverse_route',
-        description: 'Generate URL from route name or parameters (reverse routing)',
-    )]
-    public function reverseRoute(
-        ?string $name = null,
-        #[Schema(
-            type: 'object',
-            description: 'Route parameters like controller, action, plugin, prefix, and pass parameters. ' .
-                'Examples: {"controller": "Articles", "action": "view", "id": "123"} or ' .
-                '{"plugin": "MyPlugin", "controller": "Users", "action": "index"}',
-            additionalProperties: true,
-        )]
-        array $params = [],
-        bool $full = false,
-    ): array {
-        try {
-            $url = null;
-
-            if ($name !== null) {
-                // Named route - merge params properly
-                $urlArray = ['_name' => $name];
-                foreach ($params as $key => $value) {
-                    $urlArray[$key] = $value;
-                }
-
-                $url = Router::url($urlArray, $full);
-            } elseif ($params !== []) {
-                // Parameter-based routing
-                $url = Router::url($params, $full);
-            } else {
-                throw new ToolCallException('Either name or params must be provided');
-            }
-
-            return [
-                'url' => $url,
-                'full' => $full,
-            ];
-        } catch (Exception $exception) {
-            throw new ToolCallException('Error generating URL: ' . $exception->getMessage());
-        }
-    }
-
     /**
      * Detect potential route collisions and conflicts.
      *

tests/TestCase/Mcp/RouteToolsTest.php

@@ -280,61 +280,6 @@ public function testMatchUrlNotFound(): void
         }
     }
 
-    /**
-     * Test reverseRoute with named route
-     */
-    public function testReverseRouteWithName(): void
-    {
-        $result = $this->RouteTools->reverseRoute(name: 'projects:view', params: ['id' => 123]);
-
-        $this->assertArrayHasKey('url', $result);
-        $this->assertIsString($result['url']);
-        $this->assertStringContainsString('projects', $result['url']);
-        $this->assertArrayHasKey('full', $result);
-        $this->assertFalse($result['full']);
-    }
-
-    /**
-     * Test reverseRoute with parameters
-     */
-    public function testReverseRouteWithParams(): void
-    {
-        $result = $this->RouteTools->reverseRoute(
-            params: ['controller' => 'Pages', 'action' => 'display', 'pass' => ['home']],
-        );
-
-        $this->assertArrayHasKey('url', $result);
-        $this->assertIsString($result['url']);
-    }
-
-    /**
-     * Test reverseRoute with full URL
-     */
-    public function testReverseRouteWithFullUrl(): void
-    {
-        $result = $this->RouteTools->reverseRoute(
-            name: 'home',
-            params: [],
-            full: true,
-        );
-
-        $this->assertArrayHasKey('url', $result);
-        $this->assertArrayHasKey('full', $result);
-        $this->assertTrue($result['full']);
-        $this->assertStringContainsString('http', $result['url']);
-    }
-
-    /**
-     * Test reverseRoute without name or params
-     */
-    public function testReverseRouteWithoutNameOrParams(): void
-    {
-        $this->expectException(ToolCallException::class);
-        $this->expectExceptionMessage('Either name or params must be provided');
-
-        $this->RouteTools->reverseRoute();
-    }
-
     /**
      * Test detectRouteCollisions
      */
@@ -519,31 +464,6 @@ public function testMatchUrlWithMixedCaseMethod(): void
         $this->assertEquals('GET', $result['method']);
     }
 
-    /**
-     * Test reverseRoute error handling with invalid name
-     */
-    public function testReverseRouteWithInvalidName(): void
-    {
-        $this->expectException(ToolCallException::class);
-        $this->expectExceptionMessage('Error generating URL');
-
-        $this->RouteTools->reverseRoute(name: 'totally:nonexistent:route:name:xyz');
-    }
-
-    /**
-     * Test reverseRoute with name and multiple parameters
-     */
-    public function testReverseRouteWithNameAndMultipleParams(): void
-    {
-        $result = $this->RouteTools->reverseRoute(
-            name: 'projects:edit',
-            params: ['id' => 456, 'extra' => 'value'],
-        );
-
-        $this->assertArrayHasKey('url', $result);
-        $this->assertIsString($result['url']);
-    }
-
     /**
      * Test detectRouteCollisions with routes without methods
      */