Skip to content

feat(bedrock): Set gen_ai.output.messages for streamed responses - #8251

Merged
alexander-alderman-webb merged 6 commits into
masterfrom
webb/bedrock/streaming-output-messages
Nov 12, 2026
Merged

alexander-alderman-webb merged 6 commits into
masterfrom
webb/bedrock/streaming-output-messages

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Nov 11, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Keep track of blocks as they arrive in parts indexed by event["contentBlockDelta"]["contentBlockIndex"].
The parts are directly stored in the final gen_ai.output.messages format.

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb changed the title feat(bedrock): Set gen_ai.output.messages for streamed responses feat(bedrock): Set gen_ai.output.messages for streamed responses Nov 11, 2026
@github-actions

github-actions Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Coverage Results 📊

✅ 139222 passed | ⏭️ 7861 skipped | Total: 147083 | Pass Rate: 94.66% | Execution Time: 452m 47s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +104
Passed Tests 📈 +56
Failed Tests —
Skipped Tests 📈 +48

All tests are passing successfully.

✅ Patch coverage is 90.53% (86 of 95 changed executable lines covered; target 80%).
Project statement coverage is 90.22% (down 0.02 percentage points from base (42f925e) to head (7013669)).

Changed files with executable lines (1)
File Patch coverage Changed executable lines
sentry_sdk/integrations/boto3/_services/bedrock.py 90.53% 86/95 covered; missed: 211, 216, 217, 236, 245, 246, 292, 296, 297; partial branches: 210, 235, 271, 291
Coverage diff
@@            Coverage Diff             @@
##        master     #8251       +/-##
==========================================
- Coverage    90.24%    90.22%    -0.02%
==========================================
  Files          203       203         —
  Tracked lines     27235     27316       +81
  Branches     10200     10240       +40
==========================================
+ Hits         24577     24645       +68
- Misses        2658      2671       +13
- Partials      1555      1556        +1

Generated by Coverage Action

@alexander-alderman-webb
alexander-alderman-webb force-pushed the webb/bedrock/streaming-output-messages branch from db1a5a4 to 6a90c1d Compare November 11, 2026 19:51
@alexander-alderman-webb
alexander-alderman-webb force-pushed the webb/bedrock/streaming-output-messages branch from 6a90c1d to e72921f Compare November 11, 2026 19:59
@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review November 12, 2026 08:10
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner November 12, 2026 08:10

@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 2 potential issues.

Fix All in Cursor

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

Reviewed by Cursor Bugbot for commit e72921f. Configure here.

Base automatically changed from webb/bedrock/nonstreaming-finish-reasons to master November 12, 2026 09:31
@alexander-alderman-webb
alexander-alderman-webb force-pushed the webb/bedrock/streaming-output-messages branch from 0fbbfe9 to 7013669 Compare November 12, 2026 09:51
@alexander-alderman-webb
alexander-alderman-webb merged commit 7e42926 into master Nov 12, 2026
326 of 328 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/bedrock/streaming-output-messages branch November 12, 2026 10:16
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