Skip to content

Conversation

OskarStark
Copy link
Contributor

@OskarStark OskarStark commented Sep 18, 2025

Q A
Bug fix? no
New feature? yes
Docs? yes
Issues Closes #625
License MIT

Needs

@OskarStark OskarStark self-assigned this Sep 18, 2025
@OskarStark OskarStark force-pushed the refactor/brave-tool-bridge-namespace branch 7 times, most recently from a34cb44 to 7c2b776 Compare September 19, 2025 10:31
@OskarStark OskarStark changed the title [Agent][Brave] Add symfony/brave-ai-tool [Agent][Brave] Add symfony/ai-brave-tool Sep 19, 2025
@OskarStark OskarStark force-pushed the refactor/brave-tool-bridge-namespace branch 2 times, most recently from 0478dd1 to c7c3673 Compare September 19, 2025 10:41
@OskarStark OskarStark marked this pull request as ready for review September 19, 2025 10:42
@carsonbot carsonbot added Agent Issues & PRs about the AI Agent component Feature New feature Status: Needs Review labels Sep 19, 2025
@OskarStark OskarStark force-pushed the refactor/brave-tool-bridge-namespace branch from b181a2c to 19f6b7a Compare September 19, 2025 10:48
@OskarStark OskarStark requested a review from fabpot September 19, 2025 10:50
@OskarStark OskarStark force-pushed the refactor/brave-tool-bridge-namespace branch 4 times, most recently from 14f5aaa to 39bd3e8 Compare September 26, 2025 09:46
@OskarStark
Copy link
Contributor Author

@symfony/mergers I would love to get help on this one with the PHPStan and Integration tests using link binary etc. This would unlock to start the migration of tools, stores and platform into their own packages. Thanks

@OskarStark OskarStark force-pushed the refactor/brave-tool-bridge-namespace branch from eb8a312 to e62d909 Compare October 2, 2025 11:50
@OskarStark OskarStark force-pushed the refactor/brave-tool-bridge-namespace branch from fc9d18f to 7c240dc Compare October 12, 2025 15:51
OskarStark added a commit that referenced this pull request Oct 12, 2025
This PR was squashed before being merged into the main branch.

Discussion
----------

[CI] Fix `build-packages.php`

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Docs?         | no
| Issues        | --
| License       | MIT

Needed to keep the diff small in #634

Commits
-------

89c3e74 [CI] Fix `build-packages.php`
Move Brave search tool from Toolbox/Tool to new Bridge/Brave namespace:
- Create Bridge/Brave directory structure with Tests and fixtures
- Update namespaces and import statements
- Rename fixture file from brave.json to search-results.json
- Add GitHub PR template for subtree split repository
- Add .gitattributes with export-ignore rules
- Add .gitignore for vendor/, composer.lock, and phpunit.xml
- Add GitHub Actions workflow to auto-close PRs with redirect message
- Add CHANGELOG.md and LICENSE files
- Add composer.json with package name symfony/brave-ai-tool
- Add phpunit.xml.dist configuration
- Fix code style with PHP CS Fixer
- Remove old files from Toolbox/Tool location
- Update examples to use symfony/brave-ai-tool package
- Update link script to scan Bridge packages using braces approach
@OskarStark OskarStark force-pushed the refactor/brave-tool-bridge-namespace branch from c299da0 to 30a7a67 Compare October 12, 2025 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Agent Issues & PRs about the AI Agent component Feature New feature Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants