Skip to content

fix(cohere): Stop swallowing exceptions during stream iteration - #8017

Merged
alexander-alderman-webb merged 1 commit into
getsentry:masterfrom
intertermux-code:fix/7889-cohere-stream-iteration-errors
Oct 13, 2026
Merged

alexander-alderman-webb merged 1 commit into
getsentry:masterfrom
intertermux-code:fix/7889-cohere-stream-iteration-errors

Conversation

@intertermux-code

Copy link
Copy Markdown
Contributor

Fixes #7889.

Split out of #7946 per review — this one covers the cohere integration only. Mid-stream provider exceptions raised by the chat iterator were swallowed by the outer capture_internal_exceptions(); they now bubble out to the caller while the span is still closed out.

Split out of getsentry#7946 per review. Mid-stream provider exceptions raised
by the chat iterator were swallowed by the outer
capture_internal_exceptions(); they now bubble out to the caller
while the span is still closed out.
@intertermux-code
intertermux-code requested a review from a team as a code owner October 13, 2026 11:26

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a285b7f. Configure here.

Comment thread sentry_sdk/integrations/cohere.py
Comment thread sentry_sdk/integrations/cohere.py
@alexander-alderman-webb alexander-alderman-webb changed the title fix(cohere): stop swallowing stream iteration errors fix(cohere): Stop swallowing exceptions during stream iteration Oct 13, 2026
@alexander-alderman-webb
alexander-alderman-webb merged commit cc90b4d into getsentry:master Oct 13, 2026
161 checks passed
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.

Hugging Face Hub and Cohere integrations swallow errors raised while iterating a stream

2 participants