langsmith 0.1.5 → 0.1.6

package/dist/client.cjs

@@ -98,6 +98,18 @@ function assertUuid(str) {
         throw new Error(`Invalid UUID: ${str}`);
     }
 }
+const handle429 = async (response) => {
+    if (response?.status === 429) {
+        const retryAfter = parseInt(response.headers.get("retry-after") ?? "30", 10) * 1000;
+        if (retryAfter > 0) {
+            await new Promise((resolve) => setTimeout(resolve, retryAfter));
+            // Return directly after calling this check
+            return true;
+        }
+    }
+    // Fall back to existing status checks
+    return false;
+};
 class Queue {
     constructor() {
         Object.defineProperty(this, "items", {
@@ -161,6 +173,12 @@ class Client {
             writable: true,
             value: void 0
         });
+        Object.defineProperty(this, "batchIngestCaller", {
+            enumerable: true,
+            configurable: true,
+            writable: true,
+            value: void 0
+        });
         Object.defineProperty(this, "timeout_ms", {
             enumerable: true,
             configurable: true,
@@ -246,6 +264,10 @@ class Client {
         this.webUrl = trimQuotes(config.webUrl ?? defaultConfig.webUrl);
         this.timeout_ms = config.timeout_ms ?? 12000;
         this.caller = new async_caller_js_1.AsyncCaller(config.callerOptions ?? {});
+        this.batchIngestCaller = new async_caller_js_1.AsyncCaller({
+            ...(config.callerOptions ?? {}),
+            onFailedResponseHook: handle429,
+        });
         this.hideInputs = config.hideInputs ?? defaultConfig.hideInputs;
         this.hideOutputs = config.hideOutputs ?? defaultConfig.hideOutputs;
         this.autoBatchTracing = config.autoBatchTracing ?? this.autoBatchTracing;
@@ -448,7 +470,9 @@ class Client {
         if (this.autoBatchQueue.size > 0) {
             this.autoBatchTimeout = setTimeout(() => {
                 this.autoBatchTimeout = undefined;
-                void this.drainAutoBatchQueue();
+                // This error would happen in the background and is uncatchable
+                // from the outside. So just log instead.
+                void this.drainAutoBatchQueue().catch(console.error);
             }, oldTimeout
                 ? this.autoBatchAggregationDelayMs
                 : this.autoBatchInitialDelayMs);
@@ -487,7 +511,7 @@ class Client {
             void this.processRunOperation({
                 action: "create",
                 item: runCreate,
-            });
+            }).catch(console.error);
             return;
         }
         const mergedRunCreateParams = await mergeRuntimeEnvIntoRunCreates([
@@ -562,18 +586,13 @@ class Client {
             "Content-Type": "application/json",
             Accept: "application/json",
         };
-        try {
-            const response = await this.caller.call(fetch, `${this.apiUrl}/runs/batch`, {
-                method: "POST",
-                headers,
-                body: JSON.stringify(body),
-                signal: AbortSignal.timeout(this.timeout_ms),
-            });
-            await raiseForStatus(response, "batch create run");
-        }
-        catch (e) {
-            console.error(`Failed to batch create runs: ${e}`);
-        }
+        const response = await this.batchIngestCaller.call(fetch, `${this.apiUrl}/runs/batch`, {
+            method: "POST",
+            headers,
+            body: JSON.stringify(body),
+            signal: AbortSignal.timeout(this.timeout_ms),
+        });
+        await raiseForStatus(response, "batch create run");
     }
     async updateRun(runId, run) {
         assertUuid(runId);
@@ -598,7 +617,7 @@ class Client {
                 return;
             }
             else {
-                void this.processRunOperation({ action: "update", item: data });
+                void this.processRunOperation({ action: "update", item: data }).catch(console.error);
             }
             return;
         }

package/dist/client.d.ts

@@ -77,6 +77,7 @@ export declare class Client {
     private apiUrl;
     private webUrl?;
     private caller;
+    private batchIngestCaller;
     private timeout_ms;
     private _tenantId;
     private hideInputs?;

package/dist/client.js

@@ -72,6 +72,18 @@ function assertUuid(str) {
         throw new Error(`Invalid UUID: ${str}`);
     }
 }
+const handle429 = async (response) => {
+    if (response?.status === 429) {
+        const retryAfter = parseInt(response.headers.get("retry-after") ?? "30", 10) * 1000;
+        if (retryAfter > 0) {
+            await new Promise((resolve) => setTimeout(resolve, retryAfter));
+            // Return directly after calling this check
+            return true;
+        }
+    }
+    // Fall back to existing status checks
+    return false;
+};
 export class Queue {
     constructor() {
         Object.defineProperty(this, "items", {
@@ -134,6 +146,12 @@ export class Client {
             writable: true,
             value: void 0
         });
+        Object.defineProperty(this, "batchIngestCaller", {
+            enumerable: true,
+            configurable: true,
+            writable: true,
+            value: void 0
+        });
         Object.defineProperty(this, "timeout_ms", {
             enumerable: true,
             configurable: true,
@@ -219,6 +237,10 @@ export class Client {
         this.webUrl = trimQuotes(config.webUrl ?? defaultConfig.webUrl);
         this.timeout_ms = config.timeout_ms ?? 12000;
         this.caller = new AsyncCaller(config.callerOptions ?? {});
+        this.batchIngestCaller = new AsyncCaller({
+            ...(config.callerOptions ?? {}),
+            onFailedResponseHook: handle429,
+        });
         this.hideInputs = config.hideInputs ?? defaultConfig.hideInputs;
         this.hideOutputs = config.hideOutputs ?? defaultConfig.hideOutputs;
         this.autoBatchTracing = config.autoBatchTracing ?? this.autoBatchTracing;
@@ -421,7 +443,9 @@ export class Client {
         if (this.autoBatchQueue.size > 0) {
             this.autoBatchTimeout = setTimeout(() => {
                 this.autoBatchTimeout = undefined;
-                void this.drainAutoBatchQueue();
+                // This error would happen in the background and is uncatchable
+                // from the outside. So just log instead.
+                void this.drainAutoBatchQueue().catch(console.error);
             }, oldTimeout
                 ? this.autoBatchAggregationDelayMs
                 : this.autoBatchInitialDelayMs);
@@ -460,7 +484,7 @@ export class Client {
             void this.processRunOperation({
                 action: "create",
                 item: runCreate,
-            });
+            }).catch(console.error);
             return;
         }
         const mergedRunCreateParams = await mergeRuntimeEnvIntoRunCreates([
@@ -535,18 +559,13 @@ export class Client {
             "Content-Type": "application/json",
             Accept: "application/json",
         };
-        try {
-            const response = await this.caller.call(fetch, `${this.apiUrl}/runs/batch`, {
-                method: "POST",
-                headers,
-                body: JSON.stringify(body),
-                signal: AbortSignal.timeout(this.timeout_ms),
-            });
-            await raiseForStatus(response, "batch create run");
-        }
-        catch (e) {
-            console.error(`Failed to batch create runs: ${e}`);
-        }
+        const response = await this.batchIngestCaller.call(fetch, `${this.apiUrl}/runs/batch`, {
+            method: "POST",
+            headers,
+            body: JSON.stringify(body),
+            signal: AbortSignal.timeout(this.timeout_ms),
+        });
+        await raiseForStatus(response, "batch create run");
     }
     async updateRun(runId, run) {
         assertUuid(runId);
@@ -571,7 +590,7 @@ export class Client {
                 return;
             }
             else {
-                void this.processRunOperation({ action: "update", item: data });
+                void this.processRunOperation({ action: "update", item: data }).catch(console.error);
             }
             return;
         }

package/dist/index.cjs

@@ -6,4 +6,4 @@ Object.defineProperty(exports, "Client", { enumerable: true, get: function () {
 var run_trees_js_1 = require("./run_trees.cjs");
 Object.defineProperty(exports, "RunTree", { enumerable: true, get: function () { return run_trees_js_1.RunTree; } });
 // Update using yarn bump-version
-exports.__version__ = "0.1.5";
+exports.__version__ = "0.1.6";

package/dist/index.d.ts

@@ -1,4 +1,4 @@
 export { Client } from "./client.js";
 export type { Dataset, Example, TracerSession, Run, Feedback, } from "./schemas.js";
 export { RunTree, type RunTreeConfig } from "./run_trees.js";
-export declare const __version__ = "0.1.5";
+export declare const __version__ = "0.1.6";

package/dist/index.js

@@ -1,4 +1,4 @@
 export { Client } from "./client.js";
 export { RunTree } from "./run_trees.js";
 // Update using yarn bump-version
-export const __version__ = "0.1.5";
+export const __version__ = "0.1.6";

package/dist/utils/async_caller.cjs

@@ -52,13 +52,21 @@ class AsyncCaller {
             writable: true,
             value: void 0
         });
+        Object.defineProperty(this, "onFailedResponseHook", {
+            enumerable: true,
+            configurable: true,
+            writable: true,
+            value: void 0
+        });
         this.maxConcurrency = params.maxConcurrency ?? Infinity;
         this.maxRetries = params.maxRetries ?? 6;
         const PQueue = "default" in p_queue_1.default ? p_queue_1.default.default : p_queue_1.default;
         this.queue = new PQueue({ concurrency: this.maxConcurrency });
+        this.onFailedResponseHook = params?.onFailedResponseHook;
     }
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     call(callable, ...args) {
+        const onFailedResponseHook = this.onFailedResponseHook;
         return this.queue.add(() => (0, p_retry_1.default)(() => callable(...args).catch((error) => {
             // eslint-disable-next-line no-instanceof/no-instanceof
             if (error instanceof Error) {
@@ -68,7 +76,7 @@ class AsyncCaller {
                 throw new Error(error);
             }
         }), {
-            onFailedAttempt(error) {
+            async onFailedAttempt(error) {
                 if (error.message.startsWith("Cancel") ||
                     error.message.startsWith("TimeoutError") ||
                     error.message.startsWith("AbortError")) {
@@ -79,7 +87,8 @@ class AsyncCaller {
                     throw error;
                 }
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-                const status = error?.response?.status;
+                const response = error?.response;
+                const status = response?.status;
                 if (status) {
                     if (STATUS_NO_RETRY.includes(+status)) {
                         throw error;
@@ -87,6 +96,9 @@ class AsyncCaller {
                     else if (STATUS_IGNORE.includes(+status)) {
                         return;
                     }
+                    if (onFailedResponseHook) {
+                        await onFailedResponseHook(response);
+                    }
                 }
             },
             // If needed we can change some of the defaults here,

package/dist/utils/async_caller.d.ts

@@ -1,3 +1,4 @@
+type ResponseCallback = (response?: Response) => Promise<boolean>;
 export interface AsyncCallerParams {
     /**
      * The maximum number of concurrent calls that can be made.
@@ -9,6 +10,7 @@ export interface AsyncCallerParams {
      * with an exponential backoff between each attempt. Defaults to 6.
      */
     maxRetries?: number;
+    onFailedResponseHook?: ResponseCallback;
 }
 export interface AsyncCallerCallOptions {
     signal?: AbortSignal;
@@ -30,8 +32,10 @@ export declare class AsyncCaller {
     protected maxConcurrency: AsyncCallerParams["maxConcurrency"];
     protected maxRetries: AsyncCallerParams["maxRetries"];
     private queue;
+    private onFailedResponseHook?;
     constructor(params: AsyncCallerParams);
     call<A extends any[], T extends (...args: A) => Promise<any>>(callable: T, ...args: Parameters<T>): Promise<Awaited<ReturnType<T>>>;
     callWithOptions<A extends any[], T extends (...args: A) => Promise<any>>(options: AsyncCallerCallOptions, callable: T, ...args: Parameters<T>): Promise<Awaited<ReturnType<T>>>;
     fetch(...args: Parameters<typeof fetch>): ReturnType<typeof fetch>;
 }
+export {};

package/dist/utils/async_caller.js

@@ -46,13 +46,21 @@ export class AsyncCaller {
             writable: true,
             value: void 0
         });
+        Object.defineProperty(this, "onFailedResponseHook", {
+            enumerable: true,
+            configurable: true,
+            writable: true,
+            value: void 0
+        });
         this.maxConcurrency = params.maxConcurrency ?? Infinity;
         this.maxRetries = params.maxRetries ?? 6;
         const PQueue = "default" in PQueueMod ? PQueueMod.default : PQueueMod;
         this.queue = new PQueue({ concurrency: this.maxConcurrency });
+        this.onFailedResponseHook = params?.onFailedResponseHook;
     }
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     call(callable, ...args) {
+        const onFailedResponseHook = this.onFailedResponseHook;
         return this.queue.add(() => pRetry(() => callable(...args).catch((error) => {
             // eslint-disable-next-line no-instanceof/no-instanceof
             if (error instanceof Error) {
@@ -62,7 +70,7 @@ export class AsyncCaller {
                 throw new Error(error);
             }
         }), {
-            onFailedAttempt(error) {
+            async onFailedAttempt(error) {
                 if (error.message.startsWith("Cancel") ||
                     error.message.startsWith("TimeoutError") ||
                     error.message.startsWith("AbortError")) {
@@ -73,7 +81,8 @@ export class AsyncCaller {
                     throw error;
                 }
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-                const status = error?.response?.status;
+                const response = error?.response;
+                const status = response?.status;
                 if (status) {
                     if (STATUS_NO_RETRY.includes(+status)) {
                         throw error;
@@ -81,6 +90,9 @@ export class AsyncCaller {
                     else if (STATUS_IGNORE.includes(+status)) {
                         return;
                     }
+                    if (onFailedResponseHook) {
+                        await onFailedResponseHook(response);
+                    }
                 }
             },
             // If needed we can change some of the defaults here,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "langsmith",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "description": "Client library to connect to the LangSmith LLM Tracing and Evaluation Platform.",
   "packageManager": "yarn@1.22.19",
   "files": [