Fix PHPStan type errors in Bridge Brave package

- Add proper type checking and casting in Brave.php
- Add return type annotations to test methods
- Fix MockResponse parameter type in tests
- Add proper array type checking for HTTP response data
- Update .gitignore to exclude PHPUnit cache files

Author: OskarStark

src/agent/src/Bridge/Brave/.gitignore

@@ -1,3 +1,3 @@
 vendor/
 composer.lock
-phpunit.xml
+phpunit.xml/.phpunit.result.cache

src/agent/src/Bridge/Brave/Brave.php

@@ -62,8 +62,21 @@ public function __invoke(
 
         $data = $result->toArray();
 
-        return array_map(static function (array $result) {
-            return ['title' => $result['title'], 'description' => $result['description'], 'url' => $result['url']];
-        }, $data['web']['results'] ?? []);
+        $results = [];
+        if (isset($data['web']) && is_array($data['web']) && isset($data['web']['results']) && is_array($data['web']['results'])) {
+            $results = $data['web']['results'];
+        }
+
+        return array_values(array_map(static function (mixed $result): array {
+            if (!is_array($result)) {
+                return ['title' => '', 'description' => '', 'url' => ''];
+            }
+
+            return [
+                'title' => is_string($result['title'] ?? null) ? $result['title'] : '',
+                'description' => is_string($result['description'] ?? null) ? $result['description'] : '',
+                'url' => is_string($result['url'] ?? null) ? $result['url'] : '',
+            ];
+        }, $results));
     }
 }

src/agent/src/Bridge/Brave/Tests/BraveTest.php

@@ -19,7 +19,7 @@
 
 final class BraveTest extends TestCase
 {
-    public function testReturnsSearchResults()
+    public function testReturnsSearchResults(): void
     {
         $result = JsonMockResponse::fromFile(__DIR__.'/fixtures/search-results.json');
         $httpClient = new MockHttpClient($result);
@@ -36,7 +36,7 @@ public function testReturnsSearchResults()
         $this->assertSame('https://www.espn.com/nfl/team/_/name/dal/dallas-cowboys', $results[0]['url']);
     }
 
-    public function testPassesCorrectParametersToApi()
+    public function testPassesCorrectParametersToApi(): void
     {
         $result = JsonMockResponse::fromFile(__DIR__.'/fixtures/search-results.json');
         $httpClient = new MockHttpClient($result);
@@ -52,12 +52,13 @@ public function testPassesCorrectParametersToApi()
 
         $requestOptions = $result->getRequestOptions();
         $this->assertArrayHasKey('headers', $requestOptions);
+        $this->assertIsArray($requestOptions['headers']);
         $this->assertContains('X-Subscription-Token: test-api-key', $requestOptions['headers']);
     }
 
-    public function testHandlesEmptyResults()
+    public function testHandlesEmptyResults(): void
     {
-        $result = new MockResponse(json_encode(['web' => ['results' => []]]));
+        $result = new MockResponse((string) json_encode(['web' => ['results' => []]]));
         $httpClient = new MockHttpClient($result);
         $brave = new Brave($httpClient, 'test-api-key');