fix(vercel): add more debug logs, fix tracing detection for vercel, b…

…ump to 0.2.4 (#1168) langchain-ai/langsmith-docs#506
langchain-ai · Nov 5, 2024 · 21e4486 · 21e4486
2 parents 9e17e05 + 62c7702
commit 21e4486
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 3 deletions.
diff --git a/js/package.json b/js/package.json
@@ -1,6 +1,6 @@
 {
   "name": "langsmith",
-  "version": "0.2.4-dev.0",
+  "version": "0.2.4",
   "description": "Client library to connect to the LangSmith LLM Tracing and Evaluation Platform.",
   "packageManager": "[email protected]",
   "files": [

diff --git a/js/src/index.ts b/js/src/index.ts
@@ -14,4 +14,4 @@ export { RunTree, type RunTreeConfig } from "./run_trees.js";
 export { overrideFetchImplementation } from "./singletons/fetch.js";
 
 // Update using yarn bump-version
-export const __version__ = "0.2.4-dev.0";
+export const __version__ = "0.2.4";
diff --git a/js/src/tests/vercel.int.test.ts b/js/src/tests/vercel.int.test.ts
@@ -52,6 +52,7 @@ test("generateText", async () => {
       }),
     },
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       runId,
       functionId: "functionId",
       metadata: { userId: "123", language: "english" },
@@ -86,6 +87,7 @@ test("generateText with image", async () => {
       },
     ],
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       runId,
       runName: "vercelImageTest",
       functionId: "functionId",
@@ -125,6 +127,7 @@ test("streamText", async () => {
       }),
     },
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       runId,
       functionId: "functionId",
       metadata: { userId: "123", language: "english" },
@@ -152,6 +155,7 @@ test("generateObject", async () => {
     }),
     prompt: "What's the weather in Prague?",
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       runId,
       functionId: "functionId",
       metadata: { userId: "123", language: "english" },
@@ -177,6 +181,7 @@ test("streamObject", async () => {
     }),
     prompt: "What's the weather in Prague?",
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       runId,
       functionId: "functionId",
       metadata: {
@@ -217,6 +222,7 @@ test("traceable", async () => {
           }),
         },
         experimental_telemetry: AISDKExporter.getSettings({
+          isEnabled: true,
           functionId: "functionId",
           runName: "nestedVercelTrace",
           metadata: { userId: "123", language: "english" },

diff --git a/js/src/tests/vercel.test.ts b/js/src/tests/vercel.test.ts
@@ -134,6 +134,7 @@ test("generateText", async () => {
       }),
     },
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       runName: "generateText",
       functionId: "functionId",
       metadata: { userId: "123", language: "english" },
@@ -354,6 +355,7 @@ test("streamText", async () => {
       }),
     },
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       functionId: "functionId",
       metadata: { userId: "123", language: "english" },
     }),
@@ -547,6 +549,7 @@ test("generateObject", async () => {
     }),
     prompt: "What's the weather in Prague?",
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       functionId: "functionId",
       metadata: { userId: "123", language: "english" },
     }),
@@ -656,6 +659,7 @@ test("streamObject", async () => {
     }),
     prompt: "What's the weather in Prague?",
     experimental_telemetry: AISDKExporter.getSettings({
+      isEnabled: true,
       functionId: "functionId",
       metadata: { userId: "123", language: "english" },
     }),
@@ -755,6 +759,7 @@ test("traceable", async () => {
           }),
         },
         experimental_telemetry: AISDKExporter.getSettings({
+          isEnabled: true,
           runName: "generateText",
           functionId: "functionId",
           metadata: { userId: "123", language: "english" },

diff --git a/js/src/vercel.ts b/js/src/vercel.ts
@@ -13,6 +13,7 @@ import {
   getLangSmithEnvironmentVariable,
   getEnvironmentVariable,
 } from "./utils/env.js";
+import { isTracingEnabled } from "./env.js";
 
 // eslint-disable-next-line @typescript-eslint/ban-types
 type AnyString = string & {};
@@ -319,6 +320,8 @@ export class AISDKExporter {
     this.client = args?.client ?? new Client();
     this.debug =
       args?.debug ?? getEnvironmentVariable("OTEL_LOG_LEVEL") === "DEBUG";
+
+    this.logDebug("creating exporter", { tracingEnabled: isTracingEnabled() });
   }
 
   static getSettings(settings?: TelemetrySettings) {
@@ -328,7 +331,7 @@ export class AISDKExporter {
     if (runName != null) metadata[RUN_NAME_METADATA_KEY.input] = runName;
 
     // attempt to obtain the run tree if used within a traceable function
-    let defaultEnabled = true;
+    let defaultEnabled = settings?.isEnabled ?? isTracingEnabled();
     try {
       const runTree = getCurrentRunTree();
       const headers = runTree.toHeaders();