Skip to content

fix(streaming): do not raise when a collected stream carries raw SSE frames - #37943

Draft
vineethsaivs wants to merge 1 commit into
BerriAI:mainfrom
vineethsaivs:fix-stream-chunk-builder-raw-sse-frames
Draft

vineethsaivs wants to merge 1 commit into
BerriAI:mainfrom
vineethsaivs:fix-stream-chunk-builder-raw-sse-frames

Conversation

@vineethsaivs

@vineethsaivs vineethsaivs commented Aug 22, 2026 •

Copy link
Copy Markdown
Contributor

@-

@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@codspeed

codspeed Bot commented Aug 22, 2026 •

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 31 untouched benchmarks


Comparing vineethsaivs:fix-stream-chunk-builder-raw-sse-frames (fc73e3c) with main (9413b82)

Open in CodSpeed

@yuneng-berri
yuneng-berri deleted the branch BerriAI:main September 13, 2026 04:44
@yuneng-berri yuneng-berri reopened this Sep 13, 2026
@vineethsaivs
vineethsaivs force-pushed the fix-stream-chunk-builder-raw-sse-frames branch from 462199d to d5f180d Compare September 18, 2026 19:28
@devin-ai-integration
devin-ai-integration Bot changed the base branch from litellm_internal_staging to main September 23, 2026 14:58
@vineethsaivs
vineethsaivs force-pushed the fix-stream-chunk-builder-raw-sse-frames branch from d5f180d to 139d8ea Compare September 26, 2026 22:25
…frames

Skip frames that are not chunk objects instead of raising on them.
@vineethsaivs
vineethsaivs force-pushed the fix-stream-chunk-builder-raw-sse-frames branch from 139d8ea to fc73e3c Compare September 26, 2026 22:26
@vineethsaivs

Copy link
Copy Markdown
Contributor Author

Rebased onto main as a single commit. The diff had picked up unrelated staging commits after the retarget; it now contains only this fix.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants