Skip to content

feat(core): Deprecate forceTransaction option in StartSpanOptions - #23821

Merged
Lms24 merged 1 commit into
developfrom
feat/deprecate-force-transaction
Sep 1, 2026
Merged

Lms24 merged 1 commit into
developfrom
feat/deprecate-force-transaction

Conversation

@Lms24

@Lms24 Lms24 commented Sep 1, 2026

Copy link
Copy Markdown
Member

This PR deprecates the forceTransaction StartSpanOptions property.

  • the replacement is to
    • no longer use it if the span was promoted to a transaction for indexing/searchability reasons
    • or wrap it with Sentry.continueTrace(Sentry.withActiveSpan(null, Sentry.startSpan())). Not a pretty workaround but based on feedback and needs, we can evaluate adding a helper or a new forceSegment (naming TBD) option
  • added lint disable comments for our remaining use cases

closes #20278

`forceTransaction` no longer has a concrete use case: all spans are indexed and searchable in Sentry,
so a span does not need to be a transaction to be queried, filtered or aggregated on. The option will
be removed in the next major version.

For the remaining cases where a span genuinely has to be a segment (root) span, the JSDoc points to
starting it without a parent span (`withActiveSpan(null, ...)`, optionally inside `continueTrace`)
instead of forcing it into a transaction.

Internal SDK usages are suppressed with an oxlint directive for now and will be evaluated separately.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 28.56 kB - -
@sentry/browser - with treeshaking flags 26.92 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 26.82 kB - -
@sentry/browser (incl. Tracing) 48.86 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 48.87 kB - -
@sentry/browser (incl. Tracing, Profiling) 51.79 kB - -
@sentry/browser (incl. Tracing, Replay) 88.35 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 77.75 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 93.04 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 106 kB - -
@sentry/browser (incl. Feedback) 46.05 kB - -
@sentry/browser (incl. sendFeedback) 33.62 kB - -
@sentry/browser (incl. FeedbackAsync) 38.73 kB - -
@sentry/browser (incl. Metrics) 29.51 kB - -
@sentry/browser (incl. Logs) 29.8 kB - -
@sentry/browser (incl. Metrics & Logs) 30.43 kB - -
@sentry/react 30.3 kB - -
@sentry/react (incl. Tracing) 51.06 kB - -
@sentry/vue 35.73 kB - -
@sentry/vue (incl. Tracing) 51.13 kB - -
@sentry/svelte 28.59 kB - -
CDN Bundle 30.35 kB - -
CDN Bundle (incl. Tracing) 49.5 kB - -
CDN Bundle (incl. Logs, Metrics) 32.58 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 51.41 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 73.17 kB - -
CDN Bundle (incl. Tracing, Replay) 87 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 88.86 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 92.93 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 94.83 kB - -
CDN Bundle - uncompressed 89.95 kB - -
CDN Bundle (incl. Tracing) - uncompressed 147.56 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 96.24 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 153.25 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 225.41 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 267.05 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 272.73 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 280.75 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 286.42 kB - -
@sentry/nextjs (client) 53.68 kB - -
@sentry/sveltekit (client) 49.29 kB - -
@sentry/core/server 65.82 kB - -
@sentry/core/browser 51.94 kB - -
@sentry/node 123.84 kB +0.02% +20 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 81.6 kB - -
@sentry/node - without tracing 88.14 kB +0.03% +23 B 🔺
@sentry/node - without channel injection 103.28 kB +0.02% +20 B 🔺
@sentry/aws-serverless 96.49 kB +0.03% +22 B 🔺
@sentry/cloudflare (withSentry) - minified 201.21 kB - -
@sentry/cloudflare (withSentry) 500.7 kB - -

View base workflow run

@Lms24 Lms24 self-assigned this Sep 1, 2026
@Lms24
Lms24 marked this pull request as ready for review September 1, 2026 12:52
@Lms24
Lms24 requested a review from a team as a code owner September 1, 2026 12:52
@Lms24
Lms24 requested review from andreiborza, nicohrubec and s1gr1d and removed request for a team and nicohrubec September 1, 2026 12:52
Comment thread packages/core/src/integrations/mcp-server/spans.ts
@Lms24
Lms24 merged commit 8b7b9be into develop Sep 1, 2026
554 of 558 checks passed
@Lms24
Lms24 deleted the feat/deprecate-force-transaction branch September 1, 2026 13:54
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.

Deprecate span-v2-incompatible properties on StartSpanOptions

2 participants