Skip to content

feat(opentelemetry)!: Remove SentryPropagator - #22702

Draft
mydea wants to merge 1 commit into
fn/continue-start-trace-testsfrom
fn/propagator
Draft

feat(opentelemetry)!: Remove SentryPropagator#22702
mydea wants to merge 1 commit into
fn/continue-start-trace-testsfrom
fn/propagator

Conversation

@mydea

@mydea mydea commented Jul 27, 2026

Copy link
Copy Markdown
Member

Let's see first....

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.82 kB - -
@sentry/browser - with treeshaking flags 28.02 kB - -
@sentry/browser (incl. Tracing) 47.1 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 47.1 kB - -
@sentry/browser (incl. Tracing, Profiling) 51.81 kB - -
@sentry/browser (incl. Tracing, Replay) 86.4 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 75.84 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 91.12 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 103.77 kB - -
@sentry/browser (incl. Feedback) 47.13 kB - -
@sentry/browser (incl. sendFeedback) 34.66 kB - -
@sentry/browser (incl. FeedbackAsync) 39.76 kB - -
@sentry/browser (incl. Metrics) 30.89 kB - -
@sentry/browser (incl. Logs) 31.12 kB - -
@sentry/browser (incl. Metrics & Logs) 31.8 kB - -
@sentry/react 31.6 kB - -
@sentry/react (incl. Tracing) 49.33 kB - -
@sentry/vue 34.74 kB - -
@sentry/vue (incl. Tracing) 49.05 kB - -
@sentry/svelte 29.85 kB - -
CDN Bundle 31.88 kB +0.03% +8 B 🔺
CDN Bundle (incl. Tracing) 47.46 kB +0.01% +2 B 🔺
CDN Bundle (incl. Logs, Metrics) 33.42 kB +0.01% +1 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) 48.83 kB +0.01% +2 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics) 72.78 kB +0.01% +4 B 🔺
CDN Bundle (incl. Tracing, Replay) 85.09 kB +0.01% +3 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 86.39 kB +0.01% +3 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 90.86 kB +0.01% +7 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.18 kB +0.01% +4 B 🔺
CDN Bundle - uncompressed 95.04 kB +0.01% +5 B 🔺
CDN Bundle (incl. Tracing) - uncompressed 142.25 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 99.75 kB +0.01% +5 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 146.23 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 224.51 kB +0.01% +5 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed 261.51 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 265.48 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 275.22 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 279.17 kB - -
@sentry/nextjs (client) 51.92 kB - -
@sentry/sveltekit (client) 47.51 kB - -
@sentry/core/server 79.61 kB +0.01% +4 B 🔺
@sentry/core/browser 51.6 kB +0.01% +2 B 🔺
@sentry/node 120.18 kB -0.92% -1.11 kB 🔽
@sentry/node/import (ESM hook with diagnostics-channel injection) 166 B - -
@sentry/node - without tracing 83.61 kB -1.38% -1.17 kB 🔽
@sentry/aws-serverless 91.89 kB -1.23% -1.14 kB 🔽
@sentry/cloudflare (withSentry) - minified 197.55 kB - -
@sentry/cloudflare (withSentry) 485.71 kB +0.01% +23 B 🔺

View base workflow run

Comment thread packages/opentelemetry/src/propagator.ts
Comment thread packages/node/src/sdk/initOtel.ts
Comment thread packages/nextjs/src/server/index.ts Outdated
Comment thread packages/opentelemetry/src/propagator.ts
Comment thread packages/opentelemetry/src/propagator.ts
Comment thread packages/opentelemetry/src/propagator.ts
Comment thread packages/opentelemetry/src/propagator.ts
Comment thread packages/opentelemetry/src/propagator.ts
Comment thread packages/opentelemetry/src/asyncContextStrategy.ts
@mydea
mydea force-pushed the fn/propagator branch 2 times, most recently from c4a4b13 to 93d6542 Compare July 29, 2026 11:15
@mydea
mydea changed the base branch from develop to fn/continue-start-trace-tests July 29, 2026 11:15

@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 1 potential issue.

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 6535b4e. Configure here.


export { enhanceDscWithOpenTelemetryRootSpanName } from './utils/enhanceDscWithOpenTelemetryRootSpanName';

export { getTraceContextForScope } from './trace';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Public API export removed

Medium Severity

This violates the Breaking Changes review rule: publicly exported getTraceContextForScope was removed without a deprecation notice, and NodeClient’s related override was dropped in the same change.

Additional Locations (1)
Fix in Cursor Fix in Web

Triggered by project rule: PR Review Guidelines for Cursor Bot

Reviewed by Cursor Bugbot for commit 6535b4e. Configure here.

@mydea
mydea force-pushed the fn/propagator branch 2 times, most recently from afdbe2c to 7aa618c Compare July 29, 2026 14:13
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