fix! Litellm handle non streaming response fix for issue #477

[schleidl]

Description

Changed the litellm model provider to handle non-streaming responses fixing issue #477

Related Issues

Documentation PR

Type of Change

Bug fix

Testing

How have you tested the change? Verify that the changes do not break functionality or introduce warnings in consuming repositories: agents-docs, agents-tools, agents-cli

• I ran hatch run prepare

Checklist

• I have read the CONTRIBUTING document
• I have added any necessary tests that prove my fix is effective or my feature works
• I have updated the documentation accordingly
• I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
• My changes generate no new warnings
• Any dependent changes have been merged and published

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[dkmiller]

LGTM, thank you so much for the fix 🎉!

[schleidl]

Hi all! Please consider this pull request for review.
@zastrowm @zastrowm @clareliguori @signoredems

[dbschmigelski]

Hey, thanks for this, we actually just merged #1183 and would love to get this in as the long term fix.

Sorry for the delay on this, I'll be looking at this this week.

Before reviewing though, I see that you marked the commit message with fix!, can you describe why this is breaking first to help me review?

[schleidl]

Hi, nothing is breaking. The fix! was a mistake. Thanks for working on it!

[mkmeral]

Can we merge from main, and then followup on the comments? There are some updates to litellm that we should follow here (e.g. log styling, exception handling, etc)

[codecov]

Codecov Report

❌ Patch coverage is 91.01124% with 8 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/strands/models/litellm.py	91.01%	1 Missing and 7 partials ⚠️

📢 Thoughts on this report? Let us know!

[dbschmigelski]

Updated to reduce duplication