feature #634 [Agent][Brave] Add symfony/ai-brave-tool (OskarStark)

OskarStark · OskarStark · commit d82878843019 · 2025-11-22T22:11:41.000+01:00
This PR was squashed before being merged into the main branch. Discussion ---------- [Agent][Brave] Add `symfony/ai-brave-tool` | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | Docs? | yes | Issues | Closes #625 | License | MIT ### Needs - [ ] A new repo `symfony/ai-brave-tool` - [ ] Add Packagist configuration - [ ] Create a new label `Brave` - [x] Create a recipe in `symfony/recipes` - [ ] Merge the recipe symfony/recipes#1451 Commits ------- c3acf10 [Agent][Brave] Add `symfony/ai-brave-tool`
diff --git a/.github/build-packages.php b/.github/build-packages.php
@@ -9,7 +9,7 @@
 use Symfony\Component\Finder\Finder;
 
 $finder = (new Finder())
-    ->in([__DIR__.'/../src/*/', __DIR__.'/../examples/', __DIR__.'/../demo/'])
+    ->in([__DIR__.'/../src/*/', __DIR__.'/../src/*/src/Bridge/*/', __DIR__.'/../examples/', __DIR__.'/../demo/'])
     ->depth(0)
     ->name('composer.json')
 ;
diff --git a/.github/workflows/code-quality.yaml b/.github/workflows/code-quality.yaml
@@ -32,7 +32,7 @@ jobs:
                   echo COMPOSER_UP='composer update --no-progress --no-interaction --no-scripts --ansi --ignore-platform-req=ext-mongodb' >> $GITHUB_ENV
                   echo PHPSTAN='vendor/bin/phpstan' >> $GITHUB_ENV
 
-                  PACKAGES=$(find src/ -mindepth 2 -type f -name composer.json -not -path "*/vendor/*" -printf '%h\n' | sed 's/^src\///' | grep -Ev "examples" | sort |  tr '\n' ' ')
+                  PACKAGES=$(find src/ -mindepth 2 -type f -name composer.json -not -path "*/vendor/*" -not -path "*/Bridge/*" -printf '%h\n' | sed 's/^src\///' | grep -Ev "examples" | sort |  tr '\n' ' ')
                   echo "Packages: $PACKAGES"
                   echo "PACKAGES=$PACKAGES" >> $GITHUB_ENV
 
diff --git a/.github/workflows/integration-tests.yaml b/.github/workflows/integration-tests.yaml
@@ -65,19 +65,19 @@ jobs:
             - name: Install root dependencies
               uses: ramsey/composer-install@v3
 
-            - name: Install examples dependencies
+            - name: Build root packages
+              run: php .github/build-packages.php
+
+            - name: Install root dependencies
               uses: ramsey/composer-install@v3
               with:
                   working-directory: examples
 
-            - name: Link examples
-              working-directory: examples
-              run: ../link
-
             - name: Run commands examples
+              working-directory: examples
               run: |
-                php examples/commands/stores.php
-                php examples/commands/message-stores.php
+                php commands/stores.php
+                php commands/message-stores.php
 
     demo:
         runs-on: ubuntu-latest
diff --git a/examples/composer.json b/examples/composer.json
@@ -21,6 +21,7 @@
         "symfony/ai-chat": "@dev",
         "symfony/ai-platform": "@dev",
         "symfony/ai-store": "@dev",
+        "symfony/ai-brave-tool": "@dev",
         "symfony/cache": "^7.3|^8.0",
         "symfony/clock": "^7.3|^8.0",
         "symfony/console": "^7.3|^8.0",
diff --git a/examples/toolbox/brave.php b/examples/toolbox/brave.php
@@ -10,8 +10,8 @@
  */
 
 use Symfony\AI\Agent\Agent;
+use Symfony\AI\Agent\Bridge\Brave\Brave;
 use Symfony\AI\Agent\Toolbox\AgentProcessor;
-use Symfony\AI\Agent\Toolbox\Tool\Brave;
 use Symfony\AI\Agent\Toolbox\Tool\Clock;
 use Symfony\AI\Agent\Toolbox\Tool\Scraper;
 use Symfony\AI\Agent\Toolbox\Toolbox;
diff --git a/link b/link
@@ -38,7 +38,10 @@ if (!is_dir("$pathToProject/vendor/symfony")) {
 $sfPackages = array();
 
 $filesystem = new Filesystem();
-$directories = glob(__DIR__.'/src/*', GLOB_ONLYDIR | GLOB_NOSORT);
+$braces = array('*', '*/src/Bridge/*');
+$directories = array_merge(...array_values(array_map(function ($part) {
+    return glob(__DIR__.'/src/'.$part, GLOB_ONLYDIR | GLOB_NOSORT);
+}, $braces)));
 
 foreach ($directories as $dir) {
     if ($filesystem->exists($composer = "$dir/composer.json")) {
diff --git a/src/agent/phpstan.dist.neon b/src/agent/phpstan.dist.neon
@@ -6,6 +6,8 @@ parameters:
     paths:
         - src/
         - tests/
+    excludePaths:
+        - src/Bridge/*/vendor/
     ignoreErrors:
         -
             message: "#^Method .*::test.*\\(\\) has no return type specified\\.$#"
diff --git a/src/agent/src/Bridge/Brave/.gitattributes b/src/agent/src/Bridge/Brave/.gitattributes
@@ -0,0 +1,3 @@
+/Tests export-ignore
+/phpunit.xml.dist export-ignore
+/.git* export-ignore
diff --git a/src/agent/src/Bridge/Brave/.github/PULL_REQUEST_TEMPLATE.md b/src/agent/src/Bridge/Brave/.github/PULL_REQUEST_TEMPLATE.md
@@ -0,0 +1,8 @@
+Please do not submit any Pull Requests here. They will be closed.
+---
+
+Please submit your PR here instead:
+https://github.com/symfony/ai
+
+This repository is what we call a "subtree split": a read-only subset of that main repository.
+We're looking forward to your PR there!
diff --git a/src/agent/src/Bridge/Brave/.github/close-pull-request.yml b/src/agent/src/Bridge/Brave/.github/close-pull-request.yml
@@ -0,0 +1,20 @@
+name: Close Pull Request
+
+on:
+  pull_request_target:
+    types: [opened]
+
+jobs:
+  run:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: superbrothers/close-pull-request@v3
+      with:
+        comment: |
+          Thanks for your Pull Request! We love contributions.
+
+          However, you should instead open your PR on the main repository:
+          https://github.com/symfony/ai
+
+          This repository is what we call a "subtree split": a read-only subset of that main repository.
+          We're looking forward to your PR there!
diff --git a/src/agent/src/Bridge/Brave/.gitignore b/src/agent/src/Bridge/Brave/.gitignore
@@ -0,0 +1,4 @@
+vendor/
+composer.lock
+phpunit.xml
+.phpunit.result.cache
diff --git a/src/agent/src/Bridge/Brave/Brave.php b/src/agent/src/Bridge/Brave/Brave.php
@@ -9,7 +9,7 @@
  * file that was distributed with this source code.
  */
 
-namespace Symfony\AI\Agent\Toolbox\Tool;
+namespace Symfony\AI\Agent\Bridge\Brave;
 
 use Symfony\AI\Agent\Toolbox\Attribute\AsTool;
 use Symfony\AI\Agent\Toolbox\Source\HasSourcesInterface;
diff --git a/src/agent/src/Bridge/Brave/CHANGELOG.md b/src/agent/src/Bridge/Brave/CHANGELOG.md
@@ -0,0 +1,7 @@
+CHANGELOG
+=========
+
+0.1
+---
+
+ * Add the bridge
diff --git a/src/agent/src/Bridge/Brave/LICENSE b/src/agent/src/Bridge/Brave/LICENSE
@@ -0,0 +1,19 @@
+Copyright (c) 2025-present Fabien Potencier
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is furnished
+to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.
diff --git a/src/agent/src/Bridge/Brave/Tests/BraveTest.php b/src/agent/src/Bridge/Brave/Tests/BraveTest.php
@@ -9,19 +9,18 @@
  * file that was distributed with this source code.
  */
 
-namespace Symfony\AI\Agent\Tests\Toolbox\Tool;
+namespace Symfony\AI\Agent\Bridge\Brave\Tests;
 
 use PHPUnit\Framework\TestCase;
-use Symfony\AI\Agent\Toolbox\Tool\Brave;
+use Symfony\AI\Agent\Bridge\Brave\Brave;
 use Symfony\Component\HttpClient\MockHttpClient;
 use Symfony\Component\HttpClient\Response\JsonMockResponse;
-use Symfony\Component\HttpClient\Response\MockResponse;
 
 final class BraveTest extends TestCase
 {
     public function testReturnsSearchResults()
     {
-        $result = JsonMockResponse::fromFile(__DIR__.'/../../fixtures/Tool/brave.json');
+        $result = JsonMockResponse::fromFile(__DIR__.'/fixtures/search-results.json');
         $httpClient = new MockHttpClient($result);
         $brave = new Brave($httpClient, 'test-api-key');
 
@@ -38,7 +37,7 @@ public function testReturnsSearchResults()
 
     public function testPassesCorrectParametersToApi()
     {
-        $result = JsonMockResponse::fromFile(__DIR__.'/../../fixtures/Tool/brave.json');
+        $result = JsonMockResponse::fromFile(__DIR__.'/fixtures/search-results.json');
         $httpClient = new MockHttpClient($result);
         $brave = new Brave($httpClient, 'test-api-key', ['extra' => 'option']);
 
@@ -52,13 +51,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()
     {
-        $result = new MockResponse(json_encode(['web' => ['results' => []]]));
-        $httpClient = new MockHttpClient($result);
+        $httpClient = new MockHttpClient(new JsonMockResponse(['web' => ['results' => []]]));
         $brave = new Brave($httpClient, 'test-api-key');
 
         $results = $brave('this should return nothing');
diff --git a/src/agent/src/Bridge/Brave/Tests/fixtures/search-results.json b/src/agent/src/Bridge/Brave/Tests/fixtures/search-results.json
diff --git a/src/agent/src/Bridge/Brave/composer.json b/src/agent/src/Bridge/Brave/composer.json
@@ -0,0 +1,45 @@
+{
+    "name": "symfony/ai-brave-tool",
+    "description": "Brave Search AI tool bridge for Symfony applications.",
+    "license": "MIT",
+    "type": "library",
+    "keywords": ["ai", "bridge", "brave", "agent", "tool"],
+    "authors": [
+        {
+            "name": "Christopher Hertel",
+            "email": "mail@christopher-hertel.de"
+        },
+        {
+            "name": "Oskar Stark",
+            "email": "oskarstark@googlemail.com"
+        }
+    ],
+    "require": {
+        "php": ">=8.2",
+        "symfony/ai-agent": "@dev",
+        "symfony/http-client": "^7.3|^8.0"
+    },
+    "require-dev": {
+        "phpunit/phpunit": "^11.5.13"
+    },
+    "autoload": {
+        "psr-4": {
+            "Symfony\\AI\\Agent\\Bridge\\Brave\\": ""
+        }
+    },
+    "autoload-dev": {
+        "psr-4": {
+            "Symfony\\AI\\Agent\\Bridge\\Brave\\Tests\\": "Tests/"
+        }
+    },
+    "config": {
+        "sort-packages": true
+    },
+    "extra": {
+        "thanks": {
+            "name": "symfony/ai",
+            "url": "https://github.com/symfony/ai"
+        }
+    },
+    "minimum-stability": "dev"
+}
diff --git a/src/agent/src/Bridge/Brave/phpunit.xml.dist b/src/agent/src/Bridge/Brave/phpunit.xml.dist
@@ -0,0 +1,32 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/11.3/phpunit.xsd"
+         backupGlobals="false"
+         colors="true"
+         bootstrap="vendor/autoload.php"
+         failOnDeprecation="true"
+         failOnRisky="true"
+         failOnWarning="true"
+>
+    <php>
+        <ini name="error_reporting" value="-1" />
+    </php>
+
+    <testsuites>
+        <testsuite name="Symfony Brave AI Tool Test Suite">
+            <directory>./Tests/</directory>
+        </testsuite>
+    </testsuites>
+
+    <source ignoreSuppressionOfDeprecations="true">
+        <include>
+            <directory>./</directory>
+        </include>
+        <exclude>
+            <directory>./Resources</directory>
+            <directory>./Tests</directory>
+            <directory>./vendor</directory>
+        </exclude>
+    </source>
+</phpunit>

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`use Symfony\Component\Finder\Finder;`
`10`	`10`
`11`	`11`	`$finder = (new Finder())`
`12`		`- ->in([__DIR__.'/../src/*/', __DIR__.'/../examples/', __DIR__.'/../demo/'])`
	`12`	`+ ->in([__DIR__.'/../src//', __DIR__.'/../src//src/Bridge/*/', __DIR__.'/../examples/', __DIR__.'/../demo/'])`
`13`	`13`	`->depth(0)`
`14`	`14`	`->name('composer.json')`
`15`	`15`	`;`
Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,8 @@ parameters:`
`6`	`6`	`paths:`
`7`	`7`	`- src/`
`8`	`8`	`- tests/`
	`9`	`+ excludePaths:`
	`10`	`+ - src/Bridge/*/vendor/`
`9`	`11`	`ignoreErrors:`
`10`	`12`	`-`
`11`	`13`	`message: "#^Method .::test.\\(\\) has no return type specified\\.$#"`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+/Tests export-ignore`
	`2`	`+/phpunit.xml.dist export-ignore`
	`3`	`+/.git* export-ignore`