From e4733273bb4ad26aa6ec6e0ec69ade475ca026ad Mon Sep 17 00:00:00 2001
From: Rola Abuhasna <rola.abuhasna@sentry.io>
Date: Thu, 31 Jul 2025 11:24:24 +0200
Subject: [PATCH] fix(node): Fix exports for openai instrumentation (#17238)

Fixed an issue where default exports in CommonJS were failing. Returning
a new exports object was overriding some default export functionality. I
removed some redundant code as well.
---
 .../tracing/openai/instrumentation.ts           | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/packages/node/src/integrations/tracing/openai/instrumentation.ts b/packages/node/src/integrations/tracing/openai/instrumentation.ts
index 2cce987db182..714c0f872261 100644
--- a/packages/node/src/integrations/tracing/openai/instrumentation.ts
+++ b/packages/node/src/integrations/tracing/openai/instrumentation.ts
@@ -83,12 +83,19 @@ export class SentryOpenAiInstrumentation extends InstrumentationBase<Instrumenta
       }
     }
 
-    const isESM = Object.prototype.toString.call(exports) === '[object Module]';
-    if (isESM) {
+    // Constructor replacement - handle read-only properties
+    // The OpenAI property might have only a getter, so use defineProperty
+    try {
       exports.OpenAI = WrappedOpenAI;
-      return exports;
+    } catch (error) {
+      // If direct assignment fails, override the property descriptor
+      Object.defineProperty(exports, 'OpenAI', {
+        value: WrappedOpenAI,
+        writable: true,
+        configurable: true,
+        enumerable: true,
+      });
     }
-
-    return { ...exports, OpenAI: WrappedOpenAI };
+    return exports;
   }
 }