From bd0b1010e565e2fea1590a20d25e75652d56f188 Mon Sep 17 00:00:00 2001
From: Nicolas Hrubec <nico.hrubec@sentry.io>
Date: Fri, 3 Apr 2026 13:35:05 +0200
Subject: [PATCH] use replace exports utility

---
 .../tracing/anthropic-ai/instrumentation.ts   | 33 ++-----------------
 .../tracing/google-genai/instrumentation.ts   |  2 +-
 .../tracing/openai/instrumentation.ts         | 33 ++-----------------
 3 files changed, 7 insertions(+), 61 deletions(-)

diff --git a/packages/node/src/integrations/tracing/anthropic-ai/instrumentation.ts b/packages/node/src/integrations/tracing/anthropic-ai/instrumentation.ts
index b01b34f2f8ad..341fa24b3ccf 100644
--- a/packages/node/src/integrations/tracing/anthropic-ai/instrumentation.ts
+++ b/packages/node/src/integrations/tracing/anthropic-ai/instrumentation.ts
@@ -9,6 +9,7 @@ import {
   _INTERNAL_shouldSkipAiProviderWrapping,
   ANTHROPIC_AI_INTEGRATION_NAME,
   instrumentAnthropicAiClient,
+  replaceExports,
   SDK_VERSION,
 } from '@sentry/core';
 
@@ -76,36 +77,8 @@ export class SentryAnthropicAiInstrumentation extends InstrumentationBase<Anthro
       }
     }
 
-    // Constructor replacement - handle read-only properties
-    // The Anthropic property might have only a getter, so use defineProperty
-    try {
-      exports.Anthropic = WrappedAnthropic;
-    } catch {
-      // If direct assignment fails, override the property descriptor
-      Object.defineProperty(exports, 'Anthropic', {
-        value: WrappedAnthropic,
-        writable: true,
-        configurable: true,
-        enumerable: true,
-      });
-    }
-
-    // Wrap the default export if it points to the original constructor
-    // Constructor replacement - handle read-only properties
-    // The Anthropic property might have only a getter, so use defineProperty
-    if (exports.default === Original) {
-      try {
-        exports.default = WrappedAnthropic;
-      } catch {
-        // If direct assignment fails, override the property descriptor
-        Object.defineProperty(exports, 'default', {
-          value: WrappedAnthropic,
-          writable: true,
-          configurable: true,
-          enumerable: true,
-        });
-      }
-    }
+    // Replace exports with the wrapped constructor
+    replaceExports(exports, 'Anthropic', WrappedAnthropic);
     return exports;
   }
 }
diff --git a/packages/node/src/integrations/tracing/google-genai/instrumentation.ts b/packages/node/src/integrations/tracing/google-genai/instrumentation.ts
index f9c542c45f37..cadbe7ec3136 100644
--- a/packages/node/src/integrations/tracing/google-genai/instrumentation.ts
+++ b/packages/node/src/integrations/tracing/google-genai/instrumentation.ts
@@ -93,7 +93,7 @@ export class SentryGoogleGenAiInstrumentation extends InstrumentationBase<Google
       }
     }
 
-    // Replace google genai exports with the wrapped constructor
+    // Replace exports with the wrapped constructor
     replaceExports(exports, 'GoogleGenAI', WrappedGoogleGenAI);
 
     return exports;
diff --git a/packages/node/src/integrations/tracing/openai/instrumentation.ts b/packages/node/src/integrations/tracing/openai/instrumentation.ts
index 0d44a056838a..8fe29def2e54 100644
--- a/packages/node/src/integrations/tracing/openai/instrumentation.ts
+++ b/packages/node/src/integrations/tracing/openai/instrumentation.ts
@@ -9,6 +9,7 @@ import {
   _INTERNAL_shouldSkipAiProviderWrapping,
   instrumentOpenAiClient,
   OPENAI_INTEGRATION_NAME,
+  replaceExports,
   SDK_VERSION,
 } from '@sentry/core';
 
@@ -90,36 +91,8 @@ export class SentryOpenAiInstrumentation extends InstrumentationBase<OpenAiInstr
       }
     }
 
-    // Constructor replacement - handle read-only properties
-    // The OpenAI property might have only a getter, so use defineProperty
-    try {
-      exports[exportKey] = WrappedOpenAI;
-    } catch {
-      // If direct assignment fails, override the property descriptor
-      Object.defineProperty(exports, exportKey, {
-        value: WrappedOpenAI,
-        writable: true,
-        configurable: true,
-        enumerable: true,
-      });
-    }
-
-    // Wrap the default export if it points to the original constructor
-    // Constructor replacement - handle read-only properties
-    // The OpenAI property might have only a getter, so use defineProperty
-    if (exports.default === Original) {
-      try {
-        exports.default = WrappedOpenAI;
-      } catch {
-        // If direct assignment fails, override the property descriptor
-        Object.defineProperty(exports, 'default', {
-          value: WrappedOpenAI,
-          writable: true,
-          configurable: true,
-          enumerable: true,
-        });
-      }
-    }
+    // Replace exports with the wrapped constructor
+    replaceExports(exports, exportKey, WrappedOpenAI);
     return exports;
   }
 }