Skip to content

test(e2e): Add node-express-streaming E2E test app#20684

Merged
chargome merged 3 commits intodevelopfrom
cg/node-express-streaming-e2e
May 5, 2026
Merged

test(e2e): Add node-express-streaming E2E test app#20684
chargome merged 3 commits intodevelopfrom
cg/node-express-streaming-e2e

Conversation

@chargome
Copy link
Copy Markdown
Member

@chargome chargome commented May 5, 2026

Streaming-only clone of node-express E2E app with traceLifecycle: 'stream' + spanStreamingIntegration(), using waitForStreamedSpan/waitForStreamedSpans helpers.

Closes #20669

Clone of node-express with span streaming enabled (traceLifecycle: 'stream'
+ spanStreamingIntegration()). Tests use waitForStreamedSpan/waitForStreamedSpans
helpers instead of waitForTransaction.

Covers HTTP server spans, error handling, outgoing fetch, request header
attributes, tRPC instrumentation, and MCP via StreamableHTTP transport.

SSE MCP test is skipped — MCP handler spans are not emitted as streamed
spans with SSE transport (only the POST /messages HTTP span arrives).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@chargome chargome self-assigned this May 5, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

size-limit report 📦

Path Size % Change Change
@sentry/browser 26.3 kB - -
@sentry/browser - with treeshaking flags 24.78 kB - -
@sentry/browser (incl. Tracing) 44.17 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 46.39 kB - -
@sentry/browser (incl. Tracing, Profiling) 49.14 kB - -
@sentry/browser (incl. Tracing, Replay) 83.55 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 73.01 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 88.23 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 100.84 kB - -
@sentry/browser (incl. Feedback) 43.44 kB - -
@sentry/browser (incl. sendFeedback) 31.11 kB - -
@sentry/browser (incl. FeedbackAsync) 36.19 kB - -
@sentry/browser (incl. Metrics) 27.6 kB - -
@sentry/browser (incl. Logs) 27.73 kB - -
@sentry/browser (incl. Metrics & Logs) 28.43 kB - -
@sentry/react 28.04 kB - -
@sentry/react (incl. Tracing) 46.4 kB - -
@sentry/vue 31.18 kB - -
@sentry/vue (incl. Tracing) 46.02 kB - -
@sentry/svelte 26.32 kB - -
CDN Bundle 28.91 kB - -
CDN Bundle (incl. Tracing) 46.94 kB - -
CDN Bundle (incl. Logs, Metrics) 30.34 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 48.04 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 69.4 kB - -
CDN Bundle (incl. Tracing, Replay) 84.07 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 85.15 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 89.89 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 90.97 kB - -
CDN Bundle - uncompressed 84.88 kB - -
CDN Bundle (incl. Tracing) - uncompressed 140.44 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 89.08 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 143.9 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 212.99 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 258.24 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 261.69 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 271.94 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 275.38 kB - -
@sentry/nextjs (client) 48.9 kB - -
@sentry/sveltekit (client) 44.64 kB - -
@sentry/node-core 59.81 kB +0.02% +9 B 🔺
@sentry/node 163.43 kB +0.01% +8 B 🔺
@sentry/node - without tracing 72.28 kB +0.01% +7 B 🔺
@sentry/aws-serverless 106.95 kB +0.01% +6 B 🔺
@sentry/cloudflare (withSentry) - minified 166.9 kB - -
@sentry/cloudflare (withSentry) 421.79 kB - -

View base workflow run

CI builds fresh tarballs where the type is correct, so the directive is unused.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@chargome chargome requested a review from nicohrubec May 5, 2026 14:00
Copy link
Copy Markdown
Member

@nicohrubec nicohrubec left a comment

Choose a reason for hiding this comment

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

lgtm

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@chargome chargome enabled auto-merge (squash) May 5, 2026 14:47
@chargome chargome merged commit bb5b66c into develop May 5, 2026
686 of 756 checks passed
@chargome chargome deleted the cg/node-express-streaming-e2e branch May 5, 2026 15:28
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.

Add Express span streaming app

2 participants