fix: sentry node instrumentation version support + sdk double patching issue (#158)

* fix: update supported versions

* fix: only specify instrumentations if programmaticImports is disabled

* feat: export handlers + disable before patch

* docs: add changesets

* style: remove unused imports

Author: wrn14897

.changeset/dirty-houses-tease.md

@@ -0,0 +1,5 @@
+---
+'@hyperdx/node-opentelemetry': patch
+---
+
+fix: sdk double patching issue

.changeset/hot-bananas-hear.md

@@ -0,0 +1,5 @@
+---
+'@hyperdx/instrumentation-sentry-node': patch
+---
+
+fix: sentry node instrumentation version support

packages/instrumentation-sentry-node/README.md

@@ -15,7 +15,7 @@ npm install --save @hyperdx/instrumentation-sentry-node
 
 ### Supported Versions
 
-- `>=7 <9`
+- `>=7.30.0 <9`
 
 ## Usage
 

packages/instrumentation-sentry-node/src/instrumentation.ts

@@ -27,7 +27,7 @@ export class SentryNodeInstrumentation extends InstrumentationBase {
     return [
       new InstrumentationNodeModuleDefinition(
         '@sentry/node',
-        ['^7.0.0', '^8.0.0'],
+        ['>=7.30.0 <9'],
         (moduleExports: typeof Sentry) => {
           diag.debug(
             `Detected Sentry installed with SDK version: ${moduleExports.SDK_VERSION}`,

packages/node-opentelemetry/src/index.ts

@@ -1,4 +1,8 @@
 export * from './gcp';
 export * from './logger';
 export * from './otel';
-export { recordException } from '@hyperdx/instrumentation-exception';
+export {
+  recordException,
+  setupExpressErrorHandler,
+  setupKoaErrorHandler,
+} from '@hyperdx/instrumentation-exception';

packages/node-opentelemetry/src/otel.ts

@@ -11,7 +11,6 @@ import { RuntimeNodeInstrumentation } from '@opentelemetry/instrumentation-runti
 import { SentryNodeInstrumentation } from '@hyperdx/instrumentation-sentry-node';
 import {
   InstrumentationBase,
-  Instrumentation,
   InstrumentationModuleDefinition,
 } from '@opentelemetry/instrumentation';
 import { NodeSDK } from '@opentelemetry/sdk-node';
@@ -350,7 +349,7 @@ export const initSDK = (config: SDKConfig) => {
             }),
           ]),
     ],
-    instrumentations: allInstrumentations,
+    instrumentations: config.programmaticImports ? [] : allInstrumentations,
     contextManager: contextManager,
   });