Skip to content

Comments

Disable Prisma instrumentation if event-based logging is used#894

Merged
hansott merged 4 commits intomainfrom
prisma-on-extends
Jan 26, 2026
Merged

Disable Prisma instrumentation if event-based logging is used#894
hansott merged 4 commits intomainfrom
prisma-on-extends

Conversation

@hansott
Copy link
Member

@hansott hansott commented Jan 26, 2026

Summary by Aikido

Security Issues: 0 🔍 Quality Issues: 1 Resolved Issues: 0

⚡ Enhancements

  • Disabled Prisma instrumentation when client used event-based logging

🔧 Refactors

  • Changed wrapNewInstance to pass constructor arguments to interceptor

More info

@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

❌ Patch coverage is 92.30769% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
library/sinks/Prisma.ts 91.89% 2 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@hansott hansott merged commit 84a8a10 into main Jan 26, 2026
25 checks passed
@hansott hansott deleted the prisma-on-extends branch January 26, 2026 17:34
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.

2 participants