From 9d5156b095af0cda3c41e03497db3e44403c6a67 Mon Sep 17 00:00:00 2001
From: RulaKhaled <rola.abuhasna@sentry.io>
Date: Wed, 30 Jul 2025 20:13:44 +0200
Subject: [PATCH 1/2] fix(node): Fix exports for openai instrumentation

---
 .../tracing/openai/instrumentation.ts         | 19 ++++++++++++++-----
 1 file changed, 14 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..b4ab4368fa70 100644
--- a/packages/node/src/integrations/tracing/openai/instrumentation.ts
+++ b/packages/node/src/integrations/tracing/openai/instrumentation.ts
@@ -83,12 +83,21 @@ 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
+    // Constructor replacement works the same for both ESM and CJS modules
+    // We can directly assign to exports.OpenAI in both module systems
+    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;
   }
 }

From 0adeb67cafd8e56a0c3cca48682894f2efa09d11 Mon Sep 17 00:00:00 2001
From: RulaKhaled <rola.abuhasna@sentry.io>
Date: Thu, 31 Jul 2025 10:20:40 +0200
Subject: [PATCH 2/2] update comment

---
 .../node/src/integrations/tracing/openai/instrumentation.ts     | 2 --
 1 file changed, 2 deletions(-)

diff --git a/packages/node/src/integrations/tracing/openai/instrumentation.ts b/packages/node/src/integrations/tracing/openai/instrumentation.ts
index b4ab4368fa70..714c0f872261 100644
--- a/packages/node/src/integrations/tracing/openai/instrumentation.ts
+++ b/packages/node/src/integrations/tracing/openai/instrumentation.ts
@@ -85,8 +85,6 @@ export class SentryOpenAiInstrumentation extends InstrumentationBase<Instrumenta
 
     // Constructor replacement - handle read-only properties
     // The OpenAI property might have only a getter, so use defineProperty
-    // Constructor replacement works the same for both ESM and CJS modules
-    // We can directly assign to exports.OpenAI in both module systems
     try {
       exports.OpenAI = WrappedOpenAI;
     } catch (error) {