Skip to content

feat(node): Add Prisma integration by default #16073

New issue

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

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

Already on GitHub? # to your account

Merged
merged 5 commits into from
May 5, 2025

Conversation

lforst
Copy link
Contributor

@lforst lforst commented Apr 16, 2025

@lforst lforst changed the title feat(node): Add Prisma ontegration by default feat(node): Add Prisma integration by default Apr 16, 2025
Copy link
Contributor

github-actions bot commented Apr 16, 2025

size-limit report 📦

Path Size % Change Change
@sentry/browser 23.32 KB - -
@sentry/browser - with treeshaking flags 23.13 KB - -
@sentry/browser (incl. Tracing) 36.98 KB - -
@sentry/browser (incl. Tracing, Replay) 74.15 KB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 67.49 KB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 78.8 KB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 90.59 KB - -
@sentry/browser (incl. Feedback) 39.71 KB - -
@sentry/browser (incl. sendFeedback) 27.94 KB - -
@sentry/browser (incl. FeedbackAsync) 32.7 KB - -
@sentry/react 25.12 KB - -
@sentry/react (incl. Tracing) 38.95 KB - -
@sentry/vue 27.55 KB - -
@sentry/vue (incl. Tracing) 38.73 KB - -
@sentry/svelte 23.34 KB - -
CDN Bundle 24.52 KB - -
CDN Bundle (incl. Tracing) 36.99 KB - -
CDN Bundle (incl. Tracing, Replay) 72.03 KB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 77.34 KB - -
CDN Bundle - uncompressed 71.57 KB - -
CDN Bundle (incl. Tracing) - uncompressed 109.41 KB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 220.69 KB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 233.22 KB - -
@sentry/nextjs (client) 40.55 KB - -
@sentry/sveltekit (client) 37.45 KB - -
@sentry/node 144.63 KB +0.83% +1.18 KB 🔺
@sentry/node - without tracing 96.54 KB - -
@sentry/aws-serverless 120.86 KB - -

View base workflow run

@lforst
Copy link
Contributor Author

lforst commented May 5, 2025

The fix for prisma/prisma#23410 (via prisma/prisma#26918) got released in prisma 6.7.0 the other day so we can finally go through with this!

@lforst lforst marked this pull request as ready for review May 5, 2025 08:26
@lforst lforst requested review from mydea, AbhiPrasad and chargome May 5, 2025 08:27
@lforst lforst merged commit 0c983b3 into develop May 5, 2025
153 checks passed
@lforst lforst deleted the lforst-enable-prisma-by-default branch May 5, 2025 14:12
@fern-sp
Copy link

fern-sp commented May 7, 2025

Will you also update the docs to mark this as default integration? I'm making sure I can remove the entry from out Sentry init. integrations: [Sentry.prismaIntegration()]

@AbhiPrasad
Copy link
Member

@fern-sp - good catch, opened a PR to document it here: getsentry/sentry-docs#13628

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Check if prismaIntegration can be enabled by default
4 participants