Skip to content

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

Draft
alexander-alderman-webb wants to merge 1 commit into
webb/bedrock/nonstreaming-finish-reasonsfrom
webb/bedrock/streaming-output-messages
Draft

alexander-alderman-webb wants to merge 1 commit into
webb/bedrock/nonstreaming-finish-reasonsfrom
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 📊

✅ 139221 passed | ❌ 1 failed | ⏭️ 7861 skipped | Total: 147083 | Pass Rate: 94.65% | Execution Time: 463m 58s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +208
Passed Tests 📈 +111
Failed Tests 📈 +1
Skipped Tests 📈 +96

➕ New Tests (1)

View new tests
  • test_segment_span_has_profiler_id
    • File: tests.tracing.test_span_streaming
    • Status: ❌ Failing

❌ Failed Tests

test_segment_span_has_profiler_id

File: tests.tracing.test_span_streaming
Suite: py3.15t-common
Error: assert 0 > 0 + where 0 = len([])

Stack Trace
tests/tracing/test_span_streaming.py:1767: in test_segment_span_has_profiler_id
    assert len(profile_chunks) > 0
E   assert 0 > 0
E    +  where 0 = len([])

✅ Patch coverage is 90.63% (87 of 96 changed executable lines covered; target 80%).
Project statement coverage is 90.24% (unchanged from base (444f537) to head (e72921f)).

Changed files with executable lines (1)
File Patch coverage Changed executable lines
sentry_sdk/integrations/boto3/_services/bedrock.py 90.63% 87/96 covered; missed: 217, 222, 223, 242, 251, 252, 298, 302, 303; partial branches: 216, 241, 277, 297
Coverage diff
@@            Coverage Diff             @@
##        master     #8251       +/-##
==========================================
  Coverage    90.24%    90.24%        —%
==========================================
  Files          203       203         —
  Tracked lines     27125     27318      +193
  Branches     10142     10240       +98
==========================================
+ Hits         24478     24651      +173
- Misses        2647      2667       +20
- Partials      1541      1559       +18

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

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.

1 participant