Skip to content
This repository was archived by the owner on Jul 16, 2025. It is now read-only.

Conversation

@valtzu
Copy link
Contributor

@valtzu valtzu commented Jul 7, 2025

Register ToolResultConverter and ToolCallArgumentResolver as autowired services to make the usual @serializer get wired with all the app-specific normalizers and denormalizers.

@valtzu valtzu changed the title Make tool input and output to use autowired serializer chore: Make tool input and output to use autowired serializer Jul 7, 2025
@valtzu valtzu changed the title chore: Make tool input and output to use autowired serializer chore: Make tool input and output use autowired serializer Jul 7, 2025
Copy link
Member

@chr-hertel chr-hertel left a comment

Choose a reason for hiding this comment

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

Thanks @valtzu!

@chr-hertel chr-hertel merged commit dbc374e into php-llm:main Jul 8, 2025
7 checks passed
chr-hertel added a commit to symfony/ai that referenced this pull request Jul 13, 2025
… (valtzu)

This PR was merged into the main branch.

Discussion
----------

chore: Make tool input and output use autowired serializer

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

Cherry picking php-llm/llm-chain-bundle#104

Commits
-------

0d01636 chore: Make tool input and output use autowired serializer (#104)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants