Skip to content

test(node): Port basic and nested startSpan tests to streaming - #25214

Merged
nicohrubec merged 3 commits into
developfrom
feat/start-span-basic-tests-span-streaming
Oct 12, 2026
Merged

nicohrubec merged 3 commits into
developfrom
feat/start-span-basic-tests-span-streaming

Conversation

@nicohrubec

@nicohrubec nicohrubec commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

Exercise basic and nested span creation with the default streaming lifecycle. Keep static counterparts for every suite, including basic usage with children, to preserve transaction coverage.

Part of #24140

@nicohrubec
nicohrubec marked this pull request as ready for review October 9, 2026 11:36
@nicohrubec
nicohrubec requested a review from a team as a code owner October 9, 2026 11:36
@nicohrubec
nicohrubec requested review from JPeer264, Lms24, isaacs and mydea and removed request for a team October 9, 2026 11:36

@Lms24 Lms24 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

l: I'd prefer keeping static and streamed versions of these test around.

approving to unblock.

@nicohrubec

Copy link
Copy Markdown
Member Author

@Lms24 sure, will add a static copy before merging. thanks!

nicohrubec and others added 2 commits October 12, 2026 11:24
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.82 kB - -
@sentry/browser - with treeshaking flags 27.95 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 27.86 kB - -
@sentry/browser (incl. Tracing) 51.88 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 51.9 kB - -
@sentry/browser (incl. Tracing, Profiling) 54.83 kB - -
@sentry/browser (incl. Tracing, Replay) 91.65 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 80.5 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 96.37 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 109.33 kB - -
@sentry/browser (incl. Feedback) 47.34 kB - -
@sentry/browser (incl. sendFeedback) 34.87 kB - -
@sentry/browser (incl. FeedbackAsync) 39.98 kB - -
@sentry/browser (incl. Metrics) 30.84 kB - -
@sentry/browser (incl. Logs) 31.13 kB - -
@sentry/browser (incl. Metrics & Logs) 31.77 kB - -
@sentry/react 31.65 kB - -
@sentry/react (incl. Tracing) 54.21 kB - -
@sentry/vue 37.89 kB - -
@sentry/vue (incl. Tracing) 54.81 kB - -
@sentry/svelte 29.84 kB - -
@sentry/remix (Remix 3 client bundle) 56.91 kB - -
CDN Bundle 31.55 kB - -
CDN Bundle (incl. Tracing) 52.41 kB - -
CDN Bundle (incl. Logs, Metrics) 33.75 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 54.34 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74.67 kB - -
CDN Bundle (incl. Tracing, Replay) 90.06 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 92 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 96.23 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 98.18 kB - -
CDN Bundle - uncompressed 92.91 kB - -
CDN Bundle (incl. Tracing) - uncompressed 155.53 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 99.45 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 161.48 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 229.48 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 275.7 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 281.63 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 289.4 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 295.33 kB - -
@sentry/nextjs (client) 56.62 kB - -
@sentry/sveltekit (client) 52.27 kB - -
@sentry/core/server 40.88 kB - -
@sentry/core/browser 13.74 kB - -
@sentry/node 151.72 kB +0.01% +9 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 83.77 kB - -
@sentry/node - without tracing 94.29 kB +0.02% +12 B 🔺
@sentry/node - without channel injection 129.9 kB +0.01% +9 B 🔺
@sentry/aws-serverless 102.44 kB +0.01% +4 B 🔺
@sentry/cloudflare (withSentry) - minified 210.2 kB - -
@sentry/cloudflare (withSentry) 521.15 kB - -
@sentry/nextjs/cloudflare (withSentry) - minified 227.88 kB - -

View base workflow run

@nicohrubec
nicohrubec merged commit 0f33366 into develop Oct 12, 2026
359 checks passed
@nicohrubec
nicohrubec deleted the feat/start-span-basic-tests-span-streaming branch October 12, 2026 10:46
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.

3 participants