Skip to content

feat(core): Add StreamedSpanEnvelope creation function - #19153

Merged
Lms24 merged 5 commits into
lms/feat-span-firstfrom
lms/feat-core-envelope-creation
Feb 4, 2026
Merged

feat(core): Add StreamedSpanEnvelope creation function#19153
Lms24 merged 5 commits into
lms/feat-span-firstfrom
lms/feat-core-envelope-creation

Conversation

@Lms24

@Lms24Lms24 commented Feb 4, 2026

Copy link
Copy Markdown
Member

Adds a utility to create a span v2 envelope from a SerializedSpan array + tests.

Note: I think here, the "v2" naming makes more sense than the StreamSpan patter we use for user-facing functionality. This function should never be called by users, and the envelope is type span with content type span.v2+json

ref #17836

@Lms24
Lms24 marked this pull request as ready for review February 4, 2026 10:05
@Lms24Lms24 self-assigned this Feb 4, 2026

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

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment threadpackages/core/src/index.ts
Comment threadpackages/core/test/lib/tracing/spans/envelope.test.ts
Comment threadpackages/core/src/tracing/spans/envelope.ts Outdated
@Lms24Lms24 mentioned this pull request Feb 4, 2026
20 tasks
@github-actions

github-actionsBot commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊


Generated by Codecov Action

@github-actions

github-actionsBot commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser25.43 kBaddedadded
@sentry/browser - with treeshaking flags23.89 kBaddedadded
@sentry/browser (incl. Tracing)42.25 kBaddedadded
@sentry/browser (incl. Tracing, Profiling)46.88 kBaddedadded
@sentry/browser (incl. Tracing, Replay)80.88 kBaddedadded
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags70.48 kBaddedadded
@sentry/browser (incl. Tracing, Replay with Canvas)85.58 kBaddedadded
@sentry/browser (incl. Tracing, Replay, Feedback)97.76 kBaddedadded
@sentry/browser (incl. Feedback)42.15 kBaddedadded
@sentry/browser (incl. sendFeedback)30.11 kBaddedadded
@sentry/browser (incl. FeedbackAsync)35.12 kBaddedadded
@sentry/browser (incl. Metrics)26.55 kBaddedadded
@sentry/browser (incl. Logs)26.7 kBaddedadded
@sentry/browser (incl. Metrics & Logs)27.36 kBaddedadded
@sentry/react27.16 kBaddedadded
@sentry/react (incl. Tracing)44.48 kBaddedadded
@sentry/vue29.88 kBaddedadded
@sentry/vue (incl. Tracing)44.05 kBaddedadded
@sentry/svelte25.45 kBaddedadded
CDN Bundle27.96 kBaddedadded
CDN Bundle (incl. Tracing)43.02 kBaddedadded
CDN Bundle (incl. Logs, Metrics)28.8 kBaddedadded
CDN Bundle (incl. Tracing, Logs, Metrics)43.85 kBaddedadded
CDN Bundle (incl. Replay, Logs, Metrics)67.75 kBaddedadded
CDN Bundle (incl. Tracing, Replay)79.76 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)80.62 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Feedback)85.2 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)86.11 kBaddedadded
CDN Bundle - uncompressed81.78 kBaddedadded
CDN Bundle (incl. Tracing) - uncompressed127.35 kBaddedadded
CDN Bundle (incl. Logs, Metrics) - uncompressed84.61 kBaddedadded
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed130.18 kBaddedadded
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed207.99 kBaddedadded
CDN Bundle (incl. Tracing, Replay) - uncompressed243.95 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed246.77 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed256.75 kBaddedadded
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed259.56 kBaddedadded
@sentry/nextjs (client)46.85 kBaddedadded
@sentry/sveltekit (client)42.63 kBaddedadded
@sentry/node-core52.17 kBaddedadded
@sentry/node166.22 kBaddedadded
@sentry/node - without tracing93.95 kBaddedadded
@sentry/aws-serverless109.46 kBaddedadded

@github-actions

github-actionsBot commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,795--added
GET With Sentry1,68119%-added
GET With Sentry (error only)6,15970%-added
POST Baseline1,167--added
POST With Sentry59651%-added
POST With Sentry (error only)1,06091%-added
MYSQL Baseline3,230--added
MYSQL With Sentry41713%-added
MYSQL With Sentry (error only)2,67283%-added

/**
* Creates a span v2 span streaming envelope
*/
export function createSpanV2Envelope(

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: Just that we don't forget: #19140 (comment)

@Lms24Lms24 changed the title feat(core): Add Spanv2 envelope creation functionfeat(core): Add StreamedSpanEnvelope creation functionFeb 4, 2026
@Lms24
Lms24 merged commit a3e74d9 into lms/feat-span-firstFeb 4, 2026
214 checks passed
@Lms24
Lms24 deleted the lms/feat-core-envelope-creation branch February 4, 2026 16:00
@Lms24Lms24 mentioned this pull request Feb 5, 2026
Lms24 added a commit that referenced this pull request Feb 13, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Feb 16, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 2, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 4, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 6, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 9, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 10, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 13, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 18, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
logaretm pushed a commit that referenced this pull request Mar 18, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 23, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Mar 30, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Apr 2, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Apr 8, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Lms24 added a commit that referenced this pull request Apr 9, 2026
Adds a utility to create a span v2 envelope from a `SerializedSpan`
array + tests.
Note: I think here, the "v2" naming makes more sense than the
`StreamSpan` patter we use for user-facing functionality. This function
should never be called by users, and the envelope is type `span` with
content type `span.v2+json`
ref #17836
Sign up for freeto 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

@Lms24@JPeer264@s1gr1d