Skip to content

Conversation

lforst
Copy link
Contributor

@lforst lforst commented Jan 20, 2025

Pulls the default logic to create a session for each Node process into an integration. This is done so you can replicate the autoSessionTracking: false behavior for these "process sessions".

Ref: #14299

Copy link
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 22.97 KB - -
@sentry/browser - with treeshaking flags 21.63 KB - -
@sentry/browser (incl. Tracing) 35.67 KB - -
@sentry/browser (incl. Tracing, Replay) 72.47 KB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 62.98 KB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 76.72 KB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 88.74 KB - -
@sentry/browser (incl. Feedback) 39.19 KB - -
@sentry/browser (incl. sendFeedback) 27.6 KB - -
@sentry/browser (incl. FeedbackAsync) 32.36 KB - -
@sentry/react 25.65 KB - -
@sentry/react (incl. Tracing) 38.45 KB - -
@sentry/vue 27.03 KB - -
@sentry/vue (incl. Tracing) 37.43 KB - -
@sentry/svelte 23.1 KB - -
CDN Bundle 24.36 KB - -
CDN Bundle (incl. Tracing) 35.99 KB - -
CDN Bundle (incl. Tracing, Replay) 70.64 KB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 75.78 KB - -
CDN Bundle - uncompressed 71.15 KB - -
CDN Bundle (incl. Tracing) - uncompressed 106.81 KB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 217.66 KB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 230.19 KB - -
@sentry/nextjs (client) 38.57 KB - -
@sentry/sveltekit (client) 36.21 KB - -
@sentry/node 161.33 KB -0.02% -29 B 🔽
@sentry/node - without tracing 97.14 KB -0.02% -10 B 🔽
@sentry/aws-serverless 126.99 KB -0.04% -44 B 🔽

View base workflow run

@lforst lforst requested review from AbhiPrasad and Lms24 January 20, 2025 15:24
Copy link
Member

@mydea mydea left a comment

Choose a reason for hiding this comment

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

makes sense to me!

@lforst lforst merged commit 8433d45 into develop Jan 20, 2025
105 checks passed
@lforst lforst deleted the lforst-process-session-integration branch January 20, 2025 15:40
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