@visibe.ai/node 0.1.22 → 0.1.24

package/dist/cjs/client.js

@@ -224,19 +224,28 @@ function applyIntegration(client, name, visibe) {
     // Integrations are loaded lazily to avoid import errors when the peer
     // dependency is not installed.  Each integration module exports a
     // `patchClient(client, name, visibe)` that returns a restore function.
-    const constructorName = client?.constructor?.name ?? '';
+    // Walk the prototype chain to collect all ancestor class names.
+    // This handles subclasses (including our own PatchedOpenAI wrapper) without
+    // requiring the direct constructor to have a specific name.
+    const ancestorNames = new Set();
+    let ctor = client?.constructor;
+    while (ctor && ctor.name) {
+        ancestorNames.add(ctor.name);
+        ctor = Object.getPrototypeOf(ctor);
+    }
+    const has = (name) => ancestorNames.has(name);
     try {
-        if (constructorName === 'OpenAI') {
+        if (has('OpenAI')) {
             // eslint-disable-next-line @typescript-eslint/no-require-imports
             const { patchOpenAIClient } = require('./integrations/openai');
             return patchOpenAIClient(client, name, visibe);
         }
-        if (constructorName === 'Anthropic') {
+        if (has('Anthropic')) {
             // eslint-disable-next-line @typescript-eslint/no-require-imports
             const { patchAnthropicClient } = require('./integrations/anthropic');
             return patchAnthropicClient(client, name, visibe);
         }
-        if (constructorName === 'BedrockRuntimeClient') {
+        if (has('BedrockRuntimeClient')) {
             // eslint-disable-next-line @typescript-eslint/no-require-imports
             const { patchBedrockClient } = require('./integrations/bedrock');
             return patchBedrockClient(client, name, visibe);

package/dist/cjs/index.js

@@ -100,6 +100,8 @@ async function patchFramework(framework, client) {
     try {
         switch (framework) {
             case 'openai': {
+                if (_originalOpenAI)
+                    break; // already patched by _syncPatch()
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 const openaiModule = await Promise.resolve().then(() => __importStar(require('openai')));
                 _originalOpenAI = openaiModule.OpenAI ?? openaiModule.default;
@@ -128,6 +130,8 @@ async function patchFramework(framework, client) {
                 break;
             }
             case 'anthropic': {
+                if (_originalAnthropic)
+                    break; // already patched by _syncPatch()
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 const anthropicModule = await Promise.resolve().then(() => __importStar(require('@anthropic-ai/sdk')));
                 _originalAnthropic = anthropicModule.Anthropic ?? anthropicModule.default;
@@ -154,6 +158,8 @@ async function patchFramework(framework, client) {
                 break;
             }
             case 'bedrock': {
+                if (_originalBedrockClient)
+                    break; // already patched by _syncPatch()
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 const bedrockModule = await Promise.resolve().then(() => __importStar(require('@aws-sdk/client-bedrock-runtime')));
                 _originalBedrockClient = bedrockModule.BedrockRuntimeClient;
@@ -238,6 +244,98 @@ async function _autoPatch(client, frameworks) {
     }
 }
 // ---------------------------------------------------------------------------
+// _syncPatch() — synchronous CJS patching.
+//
+// In CJS builds, require() is synchronous so we can patch framework modules
+// inline inside init() BEFORE it returns.  This means instances created right
+// after init() (e.g. `const client = new OpenAI()`) are already using the
+// patched class — no async race condition.
+//
+// In ESM builds, require is undefined — the entire function silently no-ops
+// and _autoPatch() handles instrumentation via dynamic import() instead.
+// ---------------------------------------------------------------------------
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+function _makePatchedClass(Orig, client) {
+    return class Patched extends Orig {
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+        constructor(...args) {
+            super(...args);
+            try {
+                client.instrument(this);
+            }
+            catch { /* never crash constructor */ }
+        }
+    };
+}
+function _syncPatch(client, frameworksToTry) {
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
+    if (typeof require === 'undefined')
+        return;
+    for (const fw of frameworksToTry) {
+        try {
+            switch (fw) {
+                case 'openai': {
+                    if (_originalOpenAI)
+                        break;
+                    // eslint-disable-next-line @typescript-eslint/no-require-imports, @typescript-eslint/no-explicit-any
+                    const m = require('openai');
+                    const Orig = m.OpenAI ?? m.default;
+                    if (!Orig)
+                        break;
+                    _originalOpenAI = Orig;
+                    const Patched = _makePatchedClass(Orig, client);
+                    try {
+                        _setProp(m, 'OpenAI', Patched);
+                        _setProp(m, 'default', Patched);
+                    }
+                    catch {
+                        _originalOpenAI = null;
+                    }
+                    break;
+                }
+                case 'anthropic': {
+                    if (_originalAnthropic)
+                        break;
+                    // eslint-disable-next-line @typescript-eslint/no-require-imports, @typescript-eslint/no-explicit-any
+                    const m = require('@anthropic-ai/sdk');
+                    const Orig = m.Anthropic ?? m.default;
+                    if (!Orig)
+                        break;
+                    _originalAnthropic = Orig;
+                    const Patched = _makePatchedClass(Orig, client);
+                    try {
+                        _setProp(m, 'Anthropic', Patched);
+                        _setProp(m, 'default', Patched);
+                    }
+                    catch {
+                        _originalAnthropic = null;
+                    }
+                    break;
+                }
+                case 'bedrock': {
+                    if (_originalBedrockClient)
+                        break;
+                    // eslint-disable-next-line @typescript-eslint/no-require-imports, @typescript-eslint/no-explicit-any
+                    const m = require('@aws-sdk/client-bedrock-runtime');
+                    const Orig = m.BedrockRuntimeClient;
+                    if (!Orig)
+                        break;
+                    _originalBedrockClient = Orig;
+                    const Patched = _makePatchedClass(Orig, client);
+                    try {
+                        m.BedrockRuntimeClient = Patched;
+                    }
+                    catch {
+                        _originalBedrockClient = null;
+                    }
+                    break;
+                }
+            }
+        }
+        catch { /* package not installed — skip */ }
+    }
+}
+// ---------------------------------------------------------------------------
 // init()
 // ---------------------------------------------------------------------------
 function init(options) {
@@ -246,10 +344,13 @@ function init(options) {
         return _globalClient;
     }
     _globalClient = new client_1.Visibe(options ?? {});
-    // Fire async patching — works in both CJS and ESM via dynamic import().
-    // Patching typically completes within microseconds (cached modules) so there
-    // is no practical race condition for normal usage patterns.
     const frameworksToTry = options?.frameworks ?? ALL_FRAMEWORKS;
+    // Synchronous CJS path — patches module exports inline so instances created
+    // right after init() (e.g. `const client = new OpenAI()`) are instrumented.
+    // In ESM, require is undefined and this silently no-ops.
+    _syncPatch(_globalClient, frameworksToTry);
+    // Async path — handles ESM and framework detection for LangChain/LangGraph
+    // (prototype patches don't need to be synchronous).
     _autoPatch(_globalClient, frameworksToTry).catch(() => { });
     if (!_shutdownRegistered) {
         const graceful = async () => { await shutdown(); process.exit(0); };

package/dist/cjs/integrations/openai.js

@@ -152,8 +152,9 @@ class OpenAIIntegration extends base_1.BaseIntegration {
             throw err;
         }
         const model = response.model ?? params.model ?? 'unknown';
-        const inputTokens = response.usage?.prompt_tokens ?? 0;
-        const outputTokens = response.usage?.completion_tokens ?? 0;
+        // Support both openai v4 (prompt_tokens/completion_tokens) and v5/v6 (input_tokens/output_tokens)
+        const inputTokens = response.usage?.prompt_tokens ?? response.usage?.input_tokens ?? 0;
+        const outputTokens = response.usage?.completion_tokens ?? response.usage?.output_tokens ?? 0;
         const cost = (0, utils_1.calculateCost)(model, inputTokens, outputTokens);
         const choice = response.choices?.[0];
         const rawContent = choice?.message?.content;
@@ -244,9 +245,10 @@ class OpenAIIntegration extends base_1.BaseIntegration {
                             if (delta)
                                 outputText += delta;
                             // Last chunk carries usage when stream_options.include_usage is set
+                            // Support both openai v4 (prompt_tokens/completion_tokens) and v5/v6 (input_tokens/output_tokens)
                             if (chunk.usage) {
-                                inputTokens = chunk.usage.prompt_tokens ?? 0;
-                                outputTokens = chunk.usage.completion_tokens ?? 0;
+                                inputTokens = chunk.usage.prompt_tokens ?? chunk.usage.input_tokens ?? 0;
+                                outputTokens = chunk.usage.completion_tokens ?? chunk.usage.output_tokens ?? 0;
                             }
                         }
                         return result;

package/dist/esm/client.js

@@ -220,19 +220,28 @@ function applyIntegration(client, name, visibe) {
     // Integrations are loaded lazily to avoid import errors when the peer
     // dependency is not installed.  Each integration module exports a
     // `patchClient(client, name, visibe)` that returns a restore function.
-    const constructorName = client?.constructor?.name ?? '';
+    // Walk the prototype chain to collect all ancestor class names.
+    // This handles subclasses (including our own PatchedOpenAI wrapper) without
+    // requiring the direct constructor to have a specific name.
+    const ancestorNames = new Set();
+    let ctor = client?.constructor;
+    while (ctor && ctor.name) {
+        ancestorNames.add(ctor.name);
+        ctor = Object.getPrototypeOf(ctor);
+    }
+    const has = (name) => ancestorNames.has(name);
     try {
-        if (constructorName === 'OpenAI') {
+        if (has('OpenAI')) {
             // eslint-disable-next-line @typescript-eslint/no-require-imports
             const { patchOpenAIClient } = require('./integrations/openai');
             return patchOpenAIClient(client, name, visibe);
         }
-        if (constructorName === 'Anthropic') {
+        if (has('Anthropic')) {
             // eslint-disable-next-line @typescript-eslint/no-require-imports
             const { patchAnthropicClient } = require('./integrations/anthropic');
             return patchAnthropicClient(client, name, visibe);
         }
-        if (constructorName === 'BedrockRuntimeClient') {
+        if (has('BedrockRuntimeClient')) {
             // eslint-disable-next-line @typescript-eslint/no-require-imports
             const { patchBedrockClient } = require('./integrations/bedrock');
             return patchBedrockClient(client, name, visibe);

package/dist/esm/index.js

@@ -61,6 +61,8 @@ async function patchFramework(framework, client) {
     try {
         switch (framework) {
             case 'openai': {
+                if (_originalOpenAI)
+                    break; // already patched by _syncPatch()
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 const openaiModule = await import('openai');
                 _originalOpenAI = openaiModule.OpenAI ?? openaiModule.default;
@@ -89,6 +91,8 @@ async function patchFramework(framework, client) {
                 break;
             }
             case 'anthropic': {
+                if (_originalAnthropic)
+                    break; // already patched by _syncPatch()
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 const anthropicModule = await import('@anthropic-ai/sdk');
                 _originalAnthropic = anthropicModule.Anthropic ?? anthropicModule.default;
@@ -115,6 +119,8 @@ async function patchFramework(framework, client) {
                 break;
             }
             case 'bedrock': {
+                if (_originalBedrockClient)
+                    break; // already patched by _syncPatch()
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 const bedrockModule = await import('@aws-sdk/client-bedrock-runtime');
                 _originalBedrockClient = bedrockModule.BedrockRuntimeClient;
@@ -199,6 +205,98 @@ async function _autoPatch(client, frameworks) {
     }
 }
 // ---------------------------------------------------------------------------
+// _syncPatch() — synchronous CJS patching.
+//
+// In CJS builds, require() is synchronous so we can patch framework modules
+// inline inside init() BEFORE it returns.  This means instances created right
+// after init() (e.g. `const client = new OpenAI()`) are already using the
+// patched class — no async race condition.
+//
+// In ESM builds, require is undefined — the entire function silently no-ops
+// and _autoPatch() handles instrumentation via dynamic import() instead.
+// ---------------------------------------------------------------------------
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+function _makePatchedClass(Orig, client) {
+    return class Patched extends Orig {
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+        constructor(...args) {
+            super(...args);
+            try {
+                client.instrument(this);
+            }
+            catch { /* never crash constructor */ }
+        }
+    };
+}
+function _syncPatch(client, frameworksToTry) {
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
+    if (typeof require === 'undefined')
+        return;
+    for (const fw of frameworksToTry) {
+        try {
+            switch (fw) {
+                case 'openai': {
+                    if (_originalOpenAI)
+                        break;
+                    // eslint-disable-next-line @typescript-eslint/no-require-imports, @typescript-eslint/no-explicit-any
+                    const m = require('openai');
+                    const Orig = m.OpenAI ?? m.default;
+                    if (!Orig)
+                        break;
+                    _originalOpenAI = Orig;
+                    const Patched = _makePatchedClass(Orig, client);
+                    try {
+                        _setProp(m, 'OpenAI', Patched);
+                        _setProp(m, 'default', Patched);
+                    }
+                    catch {
+                        _originalOpenAI = null;
+                    }
+                    break;
+                }
+                case 'anthropic': {
+                    if (_originalAnthropic)
+                        break;
+                    // eslint-disable-next-line @typescript-eslint/no-require-imports, @typescript-eslint/no-explicit-any
+                    const m = require('@anthropic-ai/sdk');
+                    const Orig = m.Anthropic ?? m.default;
+                    if (!Orig)
+                        break;
+                    _originalAnthropic = Orig;
+                    const Patched = _makePatchedClass(Orig, client);
+                    try {
+                        _setProp(m, 'Anthropic', Patched);
+                        _setProp(m, 'default', Patched);
+                    }
+                    catch {
+                        _originalAnthropic = null;
+                    }
+                    break;
+                }
+                case 'bedrock': {
+                    if (_originalBedrockClient)
+                        break;
+                    // eslint-disable-next-line @typescript-eslint/no-require-imports, @typescript-eslint/no-explicit-any
+                    const m = require('@aws-sdk/client-bedrock-runtime');
+                    const Orig = m.BedrockRuntimeClient;
+                    if (!Orig)
+                        break;
+                    _originalBedrockClient = Orig;
+                    const Patched = _makePatchedClass(Orig, client);
+                    try {
+                        m.BedrockRuntimeClient = Patched;
+                    }
+                    catch {
+                        _originalBedrockClient = null;
+                    }
+                    break;
+                }
+            }
+        }
+        catch { /* package not installed — skip */ }
+    }
+}
+// ---------------------------------------------------------------------------
 // init()
 // ---------------------------------------------------------------------------
 export function init(options) {
@@ -207,10 +305,13 @@ export function init(options) {
         return _globalClient;
     }
     _globalClient = new Visibe(options ?? {});
-    // Fire async patching — works in both CJS and ESM via dynamic import().
-    // Patching typically completes within microseconds (cached modules) so there
-    // is no practical race condition for normal usage patterns.
     const frameworksToTry = options?.frameworks ?? ALL_FRAMEWORKS;
+    // Synchronous CJS path — patches module exports inline so instances created
+    // right after init() (e.g. `const client = new OpenAI()`) are instrumented.
+    // In ESM, require is undefined and this silently no-ops.
+    _syncPatch(_globalClient, frameworksToTry);
+    // Async path — handles ESM and framework detection for LangChain/LangGraph
+    // (prototype patches don't need to be synchronous).
     _autoPatch(_globalClient, frameworksToTry).catch(() => { });
     if (!_shutdownRegistered) {
         const graceful = async () => { await shutdown(); process.exit(0); };

package/dist/esm/integrations/openai.js

@@ -148,8 +148,9 @@ export class OpenAIIntegration extends BaseIntegration {
             throw err;
         }
         const model = response.model ?? params.model ?? 'unknown';
-        const inputTokens = response.usage?.prompt_tokens ?? 0;
-        const outputTokens = response.usage?.completion_tokens ?? 0;
+        // Support both openai v4 (prompt_tokens/completion_tokens) and v5/v6 (input_tokens/output_tokens)
+        const inputTokens = response.usage?.prompt_tokens ?? response.usage?.input_tokens ?? 0;
+        const outputTokens = response.usage?.completion_tokens ?? response.usage?.output_tokens ?? 0;
         const cost = calculateCost(model, inputTokens, outputTokens);
         const choice = response.choices?.[0];
         const rawContent = choice?.message?.content;
@@ -240,9 +241,10 @@ export class OpenAIIntegration extends BaseIntegration {
                             if (delta)
                                 outputText += delta;
                             // Last chunk carries usage when stream_options.include_usage is set
+                            // Support both openai v4 (prompt_tokens/completion_tokens) and v5/v6 (input_tokens/output_tokens)
                             if (chunk.usage) {
-                                inputTokens = chunk.usage.prompt_tokens ?? 0;
-                                outputTokens = chunk.usage.completion_tokens ?? 0;
+                                inputTokens = chunk.usage.prompt_tokens ?? chunk.usage.input_tokens ?? 0;
+                                outputTokens = chunk.usage.completion_tokens ?? chunk.usage.output_tokens ?? 0;
                             }
                         }
                         return result;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@visibe.ai/node",
-  "version": "0.1.22",
+  "version": "0.1.24",
   "description": "AI Agent Observability — Track OpenAI, LangChain, LangGraph, Bedrock, Vercel AI, Anthropic",
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.js",