Skip to content

feat(react-router): Drop low-quality transactions via ignoreSpans#20514

Draft
nicohrubec wants to merge 6 commits intodevelopfrom
nh/span-streaming-reactrouter-lowqualitytransaction
Draft

feat(react-router): Drop low-quality transactions via ignoreSpans#20514
nicohrubec wants to merge 6 commits intodevelopfrom
nh/span-streaming-reactrouter-lowqualitytransaction

Conversation

@nicohrubec
Copy link
Copy Markdown
Member

@nicohrubec nicohrubec commented Apr 25, 2026

Migrates the React Router low-quality transactions filter from a dedicated event-processor integration (lowQualityTransactionsFilterIntegration) to ignoreSpans so it also works in the streaming path. Adds some unit tests plus a new e2e test in react-router-7-framework that asserts no server transaction is sent for filtered /__manifest? requests during client-side navigation (no e2e covered this filter before).

Closes #20362

Stacked on #20513

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2026

size-limit report 📦

Path Size % Change Change
@sentry/browser 26.07 kB +0.44% +112 B 🔺
@sentry/browser - with treeshaking flags 24.55 kB +0.46% +110 B 🔺
@sentry/browser (incl. Tracing) 44.02 kB +0.31% +135 B 🔺
@sentry/browser (incl. Tracing + Span Streaming) 45.69 kB +0.34% +153 B 🔺
@sentry/browser (incl. Tracing, Profiling) 48.97 kB +0.27% +129 B 🔺
@sentry/browser (incl. Tracing, Replay) 83.23 kB +0.17% +141 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 72.71 kB +0.17% +121 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas) 87.9 kB +0.16% +137 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback) 100.52 kB +0.15% +144 B 🔺
@sentry/browser (incl. Feedback) 43.31 kB +0.3% +127 B 🔺
@sentry/browser (incl. sendFeedback) 30.88 kB +0.37% +112 B 🔺
@sentry/browser (incl. FeedbackAsync) 36.05 kB +0.35% +123 B 🔺
@sentry/browser (incl. Metrics) 27.36 kB +0.42% +113 B 🔺
@sentry/browser (incl. Logs) 27.5 kB +0.46% +125 B 🔺
@sentry/browser (incl. Metrics & Logs) 28.19 kB +0.41% +113 B 🔺
@sentry/react 27.82 kB +0.37% +102 B 🔺
@sentry/react (incl. Tracing) 46.26 kB +0.28% +126 B 🔺
@sentry/vue 30.94 kB +0.42% +127 B 🔺
@sentry/vue (incl. Tracing) 45.85 kB +0.32% +142 B 🔺
@sentry/svelte 26.09 kB +0.43% +110 B 🔺
CDN Bundle 28.75 kB +0.32% +91 B 🔺
CDN Bundle (incl. Tracing) 46.25 kB +0.3% +135 B 🔺
CDN Bundle (incl. Logs, Metrics) 30.14 kB +0.37% +110 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) 47.31 kB +0.31% +143 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics) 69.1 kB +0.17% +112 B 🔺
CDN Bundle (incl. Tracing, Replay) 83.3 kB +0.14% +113 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 84.33 kB +0.14% +113 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 89.12 kB +0.14% +122 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 90.17 kB +0.13% +115 B 🔺
CDN Bundle - uncompressed 84.32 kB +0.48% +402 B 🔺
CDN Bundle (incl. Tracing) - uncompressed 138.37 kB +0.4% +548 B 🔺
CDN Bundle (incl. Logs, Metrics) - uncompressed 88.46 kB +0.46% +402 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 141.78 kB +0.39% +548 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 212.04 kB +0.19% +402 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed 255.81 kB +0.22% +548 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 259.21 kB +0.22% +548 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 269.51 kB +0.21% +548 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 272.89 kB +0.21% +548 B 🔺
@sentry/nextjs (client) 48.76 kB +0.28% +134 B 🔺
@sentry/sveltekit (client) 44.47 kB +0.33% +144 B 🔺
@sentry/node-core 58.65 kB +0.23% +130 B 🔺
@sentry/node 175.98 kB +0.09% +150 B 🔺
@sentry/node - without tracing 98.43 kB +0.13% +121 B 🔺
@sentry/aws-serverless 115.64 kB +0.11% +122 B 🔺

View base workflow run

@nicohrubec nicohrubec changed the title Nh/span streaming reactrouter lowqualitytransaction feat(react-router): Drop low-quality transactions via ignoreSpans Apr 25, 2026
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.

Event processor migration: React Router lowQualityTransactionsFilterIntegration

1 participant