fix: improve wildcard sanitization logic in ListRoutes

Author: pushpak1300

src/Mcp/Tools/ListRoutes.php

@@ -59,7 +59,7 @@ public function handle(array $arguments): ToolResult
 
         foreach ($optionMap as $argKey => $cliOption) {
             if (! empty($arguments[$argKey])) {
-                $sanitizedValue = $this->sanitizeWildcards($arguments[$argKey], $argKey);
+                $sanitizedValue = str_replace(['*', '?'], '', $arguments[$argKey]);
                 if (filled($sanitizedValue)) {
                     $options['--'.$cliOption] = $sanitizedValue;
                 }
@@ -81,15 +81,6 @@ public function handle(array $arguments): ToolResult
         return ToolResult::text($routesOutput);
     }
 
-    private function sanitizeWildcards(string $value, string $parameter): string
-    {
-        if (in_array($parameter, ['path', 'except_path'])) {
-            return $value;
-        }
-
-        return str_replace(['*', '?'], '', $value);
-    }
-
     /**
      * @param array<string|bool> $options
      */

tests/Feature/Mcp/Tools/ListRoutesTest.php

@@ -98,25 +98,6 @@
         ->and($output)->not->toContain('admin.dashboard');
 });
 
-test('it preserves wildcards in path parameters', function () {
-    $tool = new ListRoutes;
-
-    $result = $tool->handle(['path' => '/admin/*']);
-    expect($result)->toBeInstanceOf(ToolResult::class)
-        ->and($result->toArray()['isError'])->toBeFalse();
-
-    $output = $result->toArray()['content'][0]['text'];
-    expect($output)->not->toContain('Failed to list routes');
-
-    $result = $tool->handle(['except_path' => '/nonexistent/*']);
-    expect($result)->toBeInstanceOf(ToolResult::class)
-        ->and($result->toArray()['isError'])->toBeFalse();
-
-    $output = $result->toArray()['content'][0]['text'];
-    expect($output)->toContain('admin.dashboard')
-        ->and($output)->toContain('user.profile');
-});
-
 test('it handles edge cases and empty results correctly', function () {
     $tool = new ListRoutes;