Skip to content

Conversation

@chr-hertel
Copy link
Member

@chr-hertel chr-hertel commented Oct 9, 2025

Q A
Bug fix? no
New feature? yes
Docs? no
Issues
License MIT
image image

@chr-hertel chr-hertel added AI Bundle Issues & PRs about the AI integration bundle Demo Issues & PRs about the demo application labels Oct 9, 2025
@carsonbot carsonbot changed the title [AiBundle][Demo] Add config options for keeping tool call messages and sources + expose metadata in profiler [AI Bundle][Demo][AiBundle] Add config options for keeping tool call messages and sources + expose metadata in profiler Oct 9, 2025
@chr-hertel chr-hertel changed the title [AI Bundle][Demo][AiBundle] Add config options for keeping tool call messages and sources + expose metadata in profiler [AI Bundle][Demo] Add config options for keeping tool call messages and sources + expose metadata in profiler Oct 9, 2025
include_tools: true
tools:
- 'Symfony\AI\Agent\Toolbox\Tool\Wikipedia'
include_sources: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only thing is, that this should be under the tools key, or not?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, thought the same, but wasn't able to find think of a good target structure. Should also include the fault_toletant_toolbox.
Buuuuut, i like the direct tools array. Maybe an additional tool_settings or just settings node?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets keep it for now, can be optimized later. Ready to merge from my side

@chr-hertel chr-hertel merged commit c3ac16c into symfony:main Oct 11, 2025
15 checks passed
@chr-hertel chr-hertel deleted the feat-tool-config branch October 11, 2025 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Bundle Issues & PRs about the AI integration bundle Demo Issues & PRs about the demo application Feature New feature Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants