Merge remote-tracking branch 'origin/1.x' into 1.x

Author: ashleyhindle

src/Mcp/Tools/SearchDocs.php

@@ -5,7 +5,6 @@
 namespace Laravel\Boost\Mcp\Tools;
 
 use Generator;
-use Illuminate\Support\Facades\Log;
 use Laravel\Boost\Concerns\MakesHttpRequests;
 use Laravel\Mcp\Server\Tool;
 use Laravel\Mcp\Server\Tools\ToolInputSchema;
@@ -17,13 +16,11 @@ class SearchDocs extends Tool
 {
     use MakesHttpRequests;
 
-    public function __construct(protected Roster $roster)
-    {
-    }
+    public function __construct(protected Roster $roster) {}
 
     public function description(): string
     {
-        return 'Search for up-to-date version-specific documentation related to this project and its packages. This tool will search Laravel hosted documentation based on the packages installed and is perfect for all Laravel related packages. Laravel, inertia, pest, livewire, nova, nightwatch, and more.' . PHP_EOL . 'You must use this tool to search for Laravel-ecosystem docs before using other approaches.';
+        return 'Search for up-to-date version-specific documentation related to this project and its packages. This tool will search Laravel hosted documentation based on the packages installed and is perfect for all Laravel related packages. Laravel, inertia, pest, livewire, nova, nightwatch, and more.'.PHP_EOL.'You must use this tool to search for Laravel-ecosystem docs before using other approaches.';
     }
 
     public function schema(ToolInputSchema $schema): ToolInputSchema
@@ -37,27 +34,27 @@ public function schema(ToolInputSchema $schema): ToolInputSchema
 
     public function handle(array $arguments): ToolResult|Generator
     {
-        $apiUrl = config('boost.hosted.api_url', 'https://boost.laravel.com') . '/api/docs';
+        $apiUrl = config('boost.hosted.api_url', 'https://boost.laravel.com').'/api/docs';
 
         $queries = array_filter(
             array_map('trim', explode('###', $arguments['queries'])),
-            fn($query) => $query !== '' && $query !== '*'
+            fn ($query) => $query !== '' && $query !== '*'
         );
 
         try {
             $packagesCollection = $this->roster->packages();
 
             $packages = $packagesCollection->map(function (Package $package) {
                 $name = $package->rawName();
-                $version = $package->majorVersion() . '.x';
+                $version = $package->majorVersion().'.x';
 
                 return [
                     'name' => $name,
-                    'version' => $version
+                    'version' => $version,
                 ];
             })->values()->toArray();
         } catch (\Throwable $e) {
-            return ToolResult::error('Failed to get packages: ' . $e->getMessage());
+            return ToolResult::error('Failed to get packages: '.$e->getMessage());
         }
 
         $tokenLimit = $arguments['token_limit'] ?? 10000;
@@ -66,29 +63,29 @@ public function handle(array $arguments): ToolResult|Generator
         $payload = [
             'queries' => $queries,
             'packages' => $packages,
-            'token_limit' => $tokenLimit
+            'token_limit' => $tokenLimit,
         ];
         try {
             $response = $this->client()->asJson()->post($apiUrl, $payload);
 
-            if (!$response->successful()) {
-                return ToolResult::error('Failed to search documentation: ' . $response->body());
+            if (! $response->successful()) {
+                return ToolResult::error('Failed to search documentation: '.$response->body());
             }
         } catch (\Throwable $e) {
-            return ToolResult::error('HTTP request failed: ' . $e->getMessage());
+            return ToolResult::error('HTTP request failed: '.$e->getMessage());
         }
 
         $data = $response->json();
         $results = $data['results'] ?? [];
 
         $concatenatedKnowledge = collect($results)
-            ->map(fn($result) => $result['content'] ?? '')
+            ->map(fn ($result) => $result['content'] ?? '')
             ->filter()
             ->join("\n\n---\n\n");
 
         return ToolResult::json([
             'knowledge_count' => count($results),
-            'knowledge' => $concatenatedKnowledge
+            'knowledge' => $concatenatedKnowledge,
         ]);
     }
 }

tests/Feature/Mcp/Tools/SearchDocsTest.php

@@ -26,7 +26,7 @@
         'results' => [
             ['content' => 'Laravel documentation content'],
             ['content' => 'Pest documentation content'],
-        ]
+        ],
     ]);
 
     $mockClient = Mockery::mock(PendingRequest::class);
@@ -39,10 +39,10 @@
     $result = $tool->handle(['queries' => 'authentication, testing']);
 
     expect($result)->toBeInstanceOf(ToolResult::class);
-    
+
     $data = $result->toArray();
     expect($data['isError'])->toBeFalse();
-    
+
     $content = json_decode($data['content'][0]['text'], true);
     expect($content['knowledge_count'])->toBe(2);
     expect($content['knowledge'])->toContain('Laravel documentation content');
@@ -73,7 +73,7 @@
     $result = $tool->handle(['queries' => 'authentication']);
 
     expect($result)->toBeInstanceOf(ToolResult::class);
-    
+
     $data = $result->toArray();
     expect($data['isError'])->toBeTrue();
     expect($data['content'][0]['text'])->toBe('Failed to search documentation: API Error');
@@ -91,10 +91,10 @@
 
     $mockClient = Mockery::mock(PendingRequest::class);
     $mockClient->shouldReceive('asJson')->andReturnSelf();
-    $mockClient->shouldReceive('post')->withArgs(function($url, $payload) {
+    $mockClient->shouldReceive('post')->withArgs(function ($url, $payload) {
         return $url === 'https://boost.laravel.com/api/docs' &&
-               $payload['queries'] === ['test'] && 
-               empty($payload['packages']) && 
+               $payload['queries'] === ['test'] &&
+               empty($payload['packages']) &&
                $payload['token_limit'] === 10000;
     })->andReturn($mockResponse);
 
@@ -104,7 +104,7 @@
     $result = $tool->handle(['queries' => 'test###  ###*### ']);
 
     expect($result)->toBeInstanceOf(ToolResult::class);
-    
+
     $data = $result->toArray();
     expect($data['isError'])->toBeFalse();
 });
@@ -129,7 +129,7 @@
         Mockery::on(function ($payload) {
             return $payload['packages'] === [
                 ['name' => 'laravel/framework', 'version' => '11.x'],
-                ['name' => 'livewire/livewire', 'version' => '3.x']
+                ['name' => 'livewire/livewire', 'version' => '3.x'],
             ] && $payload['token_limit'] === 10000;
         })
     )->andReturn($mockResponse);
@@ -162,10 +162,10 @@
     $result = $tool->handle(['queries' => 'nonexistent']);
 
     expect($result)->toBeInstanceOf(ToolResult::class);
-    
+
     $data = $result->toArray();
     expect($data['isError'])->toBeFalse();
-    
+
     $content = json_decode($data['content'][0]['text'], true);
     expect($content['knowledge_count'])->toBe(0);
     expect($content['knowledge'])->toBe('');
@@ -223,4 +223,4 @@
     $result = $tool->handle(['queries' => 'test', 'token_limit' => 2000000]); // Request 2M but get capped at 1M
 
     expect($result)->toBeInstanceOf(ToolResult::class);
-});
+});