Skip to content

Conversation

chargome
Copy link
Member

@chargome chargome commented Sep 3, 2025

Adds tests for tracing and exception handling in turbopack route handlers.

Note: this will fail until vercel/next.js#82612 is resolved

@chargome chargome self-assigned this Sep 3, 2025
@chargome chargome marked this pull request as ready for review September 3, 2025 15:25
@chargome chargome requested a review from mydea September 3, 2025 15:25
@chargome chargome enabled auto-merge (squash) September 3, 2025 15:26
Copy link
Contributor

github-actions bot commented Sep 3, 2025

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.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 8,953 - 8,640 +4%
GET With Sentry 1,326 15% 1,325 +0%
GET With Sentry (error only) 5,970 67% 5,913 +1%
POST Baseline 1,183 - 1,092 +8%
POST With Sentry 487 41% 482 +1%
POST With Sentry (error only) 1,033 87% 976 +6%
MYSQL Baseline 3,292 - 3,271 +1%
MYSQL With Sentry 386 12% 564 -32%
MYSQL With Sentry (error only) 2,678 81% 2,622 +2%

View base workflow run

cursor[bot]

This comment was marked as outdated.

@chargome chargome force-pushed the cg-add-route-handler-tests branch from 99ea8e1 to 3727f46 Compare September 15, 2025 07:42
Copy link
Contributor

github-actions bot commented Sep 15, 2025

size-limit report 📦

Path Size % Change Change
@sentry/browser 24.26 kB - -
@sentry/browser - with treeshaking flags 22.77 kB - -
@sentry/browser (incl. Tracing) 40.43 kB - -
@sentry/browser (incl. Tracing, Replay) 78.82 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 68.48 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 83.49 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 95.69 kB - -
@sentry/browser (incl. Feedback) 40.97 kB - -
@sentry/browser (incl. sendFeedback) 28.91 kB - -
@sentry/browser (incl. FeedbackAsync) 33.84 kB - -
@sentry/react 25.98 kB - -
@sentry/react (incl. Tracing) 42.41 kB - -
@sentry/vue 28.78 kB - -
@sentry/vue (incl. Tracing) 42.24 kB - -
@sentry/svelte 24.29 kB - -
CDN Bundle 25.77 kB - -
CDN Bundle (incl. Tracing) 40.33 kB - -
CDN Bundle (incl. Tracing, Replay) 76.57 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 82.06 kB - -
CDN Bundle - uncompressed 75.37 kB - -
CDN Bundle (incl. Tracing) - uncompressed 119.39 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 234.55 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 247.31 kB - -
@sentry/nextjs (client) 44.42 kB - -
@sentry/sveltekit (client) 40.85 kB - -
@sentry/node-core 50.02 kB +0.01% +1 B 🔺
@sentry/node 153.06 kB -0.01% -1 B 🔽
@sentry/node - without tracing 91.94 kB -0.01% -1 B 🔽
@sentry/aws-serverless 105.39 kB - -

View base workflow run

cursor[bot]

This comment was marked as outdated.

@chargome chargome merged commit 8424fdc into develop Sep 26, 2025
186 of 189 checks passed
@chargome chargome deleted the cg-add-route-handler-tests branch September 26, 2025 09: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