From 9e67fb11564ec45d1e9d5599a403b4babc1ed9b4 Mon Sep 17 00:00:00 2001
From: Andrei Borza <andrei.borza@sentry.io>
Date: Fri, 8 Aug 2025 10:15:21 +0200
Subject: [PATCH] fix(v9/node): Assign default export of `openai` to the
 instrumented fn

Both of:
- import OpenAI from `openai`
- import { OpenAI } from`openai`

Should point to the instrumented wrapper function.

Backport of #17320.
---
 .../tracing/openai/instrumentation.ts           | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/packages/node/src/integrations/tracing/openai/instrumentation.ts b/packages/node/src/integrations/tracing/openai/instrumentation.ts
index 714c0f872261..76385009f5ba 100644
--- a/packages/node/src/integrations/tracing/openai/instrumentation.ts
+++ b/packages/node/src/integrations/tracing/openai/instrumentation.ts
@@ -96,6 +96,23 @@ export class SentryOpenAiInstrumentation extends InstrumentationBase<Instrumenta
         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 (error) {
+        // If direct assignment fails, override the property descriptor
+        Object.defineProperty(exports, 'default', {
+          value: WrappedOpenAI,
+          writable: true,
+          configurable: true,
+          enumerable: true,
+        });
+      }
+    }
     return exports;
   }
 }