Skip to content

feat(nextjs): Prefix webpack plugin log messages with runtime #11173

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 19, 2024

Conversation

lforst
Copy link
Contributor

@lforst lforst commented Mar 18, 2024

Small QoL win. This PR prefixes all webpack plugin log messages in the Next.js SDK with [@sentry/nextjs - Node.js] , [@sentry/nextjs - Edge] or [@sentry/nextjs - Client] depending which one's turn it is.

@lforst lforst requested review from Lms24, AbhiPrasad and s1gr1d March 18, 2024 16:59
Copy link
Contributor

github-actions bot commented Mar 18, 2024

size-limit report 📦

Path Size
@sentry/browser (incl. Tracing, Replay, Feedback) 80.75 KB (added)
@sentry/browser (incl. Tracing, Replay) 72.1 KB (added)
@sentry/browser (incl. Tracing, Replay with Canvas) 75.9 KB (added)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 65.66 KB (added)
@sentry/browser (incl. Tracing) 36.7 KB (added)
@sentry/browser (incl. browserTracingIntegration) 36.7 KB (added)
@sentry/browser (incl. feedbackIntegration) 31.38 KB (added)
@sentry/browser (incl. feedbackModalIntegration) 31.49 KB (added)
@sentry/browser (incl. feedbackScreenshotIntegration) 31.5 KB (added)
@sentry/browser (incl. sendFeedback) 27.45 KB (added)
@sentry/browser 22.62 KB (added)
CDN Bundle (incl. Tracing, Replay, Feedback) 75.09 KB (added)
CDN Bundle (incl. Tracing, Replay) 69.92 KB (added)
CDN Bundle (incl. Tracing) 36.28 KB (added)
CDN Bundle 23.99 KB (added)
CDN Bundle (incl. Tracing, Replay) - uncompressed 219.62 KB (added)
CDN Bundle (incl. Tracing) - uncompressed 109.56 KB (added)
CDN Bundle - uncompressed 71.09 KB (added)
@sentry/react (incl. Tracing, Replay) 72.09 KB (added)
@sentry/react 22.64 KB (added)

@lforst lforst merged commit fea2480 into develop Mar 19, 2024
@lforst lforst deleted the lforst-nextjs-webpack-plugin-log-message-prefix branch March 19, 2024 08:54
cadesalaberry pushed a commit to cadesalaberry/sentry-javascript that referenced this pull request Apr 19, 2024
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