-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Closed
Labels
BugExternal DependencyTo close this issue, an external dependency needs to be adjusted/fixedTo close this issue, an external dependency needs to be adjusted/fixedNode.jsOpenTelemetry
Description
Is there an existing issue for this?
- I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- I have reviewed the documentation https://docs.sentry.io/
- I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/node
SDK Version
9.15.0
Framework Version
Node 20.19.1, TypeScript 5.8.3
Link to Sentry event
No response
Reproduction Example/SDK Setup
The @sentry/node@9.15.0
package pulls in @opentelemetry/instrumentation-pg@0.51.1
, which in turn pulls in @types/pg@8.6.1
. When compiling, I get this:
│ /home/runner/work/lib-api-framework/lib-api-framework/node_modules/@types/pg/index.d.ts
│ 12:31 Cannot find module 'pg-protocol/dist/messages' or its corresponding type declarations. [TS2307]
│ | import { NoticeMessage } from 'pg-protocol/dist/messages';
The problem arises when building against @sentry/node
with module resolution set to bundler
, and it's rather annoying as we (for example) don't use any instrumentation.
Reported upstream here.
Steps to Reproduce
Set module resolution in tsconfig.json
to bundler
and run tsc
Expected Result
Types should compile
Actual Result
Got this error:
Cannot find module 'pg-protocol/dist/messages' or its corresponding type declarations. [TS2307]
achwo and andersk
Metadata
Metadata
Assignees
Labels
BugExternal DependencyTo close this issue, an external dependency needs to be adjusted/fixedTo close this issue, an external dependency needs to be adjusted/fixedNode.jsOpenTelemetry
Projects
Status
No status