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 4, 2025

image

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.

Good catch, thanks @valtzu!

@chr-hertel chr-hertel merged commit 0adf120 into php-llm:main Jul 4, 2025
7 checks passed
chr-hertel added a commit to symfony/ai that referenced this pull request Jul 4, 2025
This PR was merged into the main branch.

Discussion
----------

fix: profiler when tool params are missing description

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

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

![image](https://github.com/user-attachments/assets/01a25199-716c-4d17-a790-c36b886d125d)

Commits
-------

06c6d2b fix: profiler when tool params are missing description (#100)
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