Skip to content

Conversation

OskarStark
Copy link
Contributor

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

Changed #[With(maximum: 500)] to #[With(maxLength: 500)] for the $query parameter since maximum is for numeric values while maxLength is for string length validation.

Changed #[With(maximum: 500)] to #[With(maxLength: 500)] for the $query parameter since maximum is for numeric values while maxLength is for string length validation.
@carsonbot carsonbot added Bug Something isn't working Status: Needs Review labels Sep 12, 2025
@carsonbot carsonbot changed the title Fix validation attribute for string parameter in Brave tool Fix validation attribute for string parameter in Brave tool Sep 12, 2025
@OskarStark OskarStark added the Agent Issues & PRs about the AI Agent component label Sep 12, 2025
@carsonbot carsonbot changed the title Fix validation attribute for string parameter in Brave tool [Agent] Fix validation attribute for string parameter in Brave tool Sep 12, 2025
@OskarStark OskarStark changed the title [Agent] Fix validation attribute for string parameter in Brave tool [Agent]Brave] Fix validation attribute for string parameter in tool Sep 12, 2025
@OskarStark OskarStark merged commit a0450a0 into symfony:main Sep 12, 2025
15 checks passed
@chr-hertel chr-hertel added the Hackathon 2025 This issue or pull request was part of the Symfony AI Hackathon 2025 label Sep 13, 2025
@carsonbot carsonbot changed the title [Agent]Brave] Fix validation attribute for string parameter in tool [Agent] Brave] Fix validation attribute for string parameter in tool Sep 13, 2025
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 Bug Something isn't working Hackathon 2025 This issue or pull request was part of the Symfony AI Hackathon 2025 Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants