Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## trunk #290 +/- ##
============================================
+ Coverage 86.58% 86.59% +0.01%
- Complexity 1383 1385 +2
============================================
Files 69 69
Lines 4449 4454 +5
============================================
+ Hits 3852 3857 +5
Misses 597 597
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Summary
toolmessagesWhen a model requests multiple tools in one response, the resulting function responses can be grouped in one user message. The OpenAI-compatible adapter previously handled only a single function response and attempted to process grouped responses as regular content. This change emits one API tool message per response, retaining each
tool_call_id.Fixes #286
Validation
composer test -- --filter AbstractOpenAiCompatibleTextGenerationModelTest— 66 tests, 165 assertionscomposer lint— PHPCS and PHPStan passgit diff --checkAI assistance disclosure
AI assistance was used for drafting and test scaffolding. The implementation and validation were reviewed before submission.