langsmith 0.8.7 → 0.8.8

package/dist/_openapi_client/resources/traces.d.ts

@@ -108,7 +108,7 @@ export interface TraceListRunsParams {
      * (repeatable query parameter). Accepts any value of the `RunSelectField` enum. If
      * omitted, only `id` is returned.
      */
-    selects?: Array<'ID' | 'NAME' | 'RUN_TYPE' | 'STATUS' | 'START_TIME' | 'END_TIME' | 'LATENCY_SECONDS' | 'FIRST_TOKEN_TIME' | 'ERROR' | 'ERROR_PREVIEW' | 'EXTRA' | 'METADATA' | 'EVENTS' | 'INPUTS' | 'INPUTS_PREVIEW' | 'OUTPUTS' | 'OUTPUTS_PREVIEW' | 'MANIFEST' | 'PARENT_RUN_IDS' | 'PROJECT_ID' | 'TRACE_ID' | 'THREAD_ID' | 'DOTTED_ORDER' | 'IS_ROOT' | 'REFERENCE_EXAMPLE_ID' | 'REFERENCE_DATASET_ID' | 'TOTAL_TOKENS' | 'PROMPT_TOKENS' | 'COMPLETION_TOKENS' | 'TOTAL_COST' | 'PROMPT_COST' | 'COMPLETION_COST' | 'PROMPT_TOKEN_DETAILS' | 'COMPLETION_TOKEN_DETAILS' | 'PROMPT_COST_DETAILS' | 'COMPLETION_COST_DETAILS' | 'PRICE_MODEL_ID' | 'TAGS' | 'APP_PATH' | 'ATTACHMENTS' | 'THREAD_EVALUATION_TIME' | 'IS_IN_DATASET' | 'SHARE_URL' | 'FEEDBACK_STATS'>;
+    selects?: Array<'ID' | 'NAME' | 'RUN_TYPE' | 'STATUS' | 'START_TIME' | 'END_TIME' | 'LATENCY_SECONDS' | 'FIRST_TOKEN_TIME' | 'ERROR' | 'ERROR_PREVIEW' | 'EXTRA' | 'METADATA' | 'EVENTS' | 'INPUTS' | 'INPUTS_PREVIEW' | 'OUTPUTS' | 'OUTPUTS_PREVIEW' | 'MANIFEST' | 'PARENT_RUN_IDS' | 'PROJECT_ID' | 'TRACE_ID' | 'THREAD_ID' | 'DOTTED_ORDER' | 'IS_ROOT' | 'REFERENCE_EXAMPLE_ID' | 'REFERENCE_DATASET_ID' | 'TOTAL_TOKENS' | 'PROMPT_TOKENS' | 'COMPLETION_TOKENS' | 'TOTAL_COST' | 'PROMPT_COST' | 'COMPLETION_COST' | 'PROMPT_TOKEN_DETAILS' | 'COMPLETION_TOKEN_DETAILS' | 'PROMPT_COST_DETAILS' | 'COMPLETION_COST_DETAILS' | 'PRICE_MODEL_ID' | 'TAGS' | 'APP_PATH' | 'ATTACHMENTS' | 'THREAD_EVALUATION_TIME' | 'IS_IN_DATASET' | 'LAST_QUEUED_AT' | 'SHARE_URL' | 'FEEDBACK_STATS'>;
     /**
      * Header param: application/json
      */

package/dist/_openapi_client/resources/traces.js

@@ -20,7 +20,7 @@ export class Traces extends APIResource {
      */
     listRuns(traceID, params, options) {
         const { Accept, ...query } = params;
-        return this._client.get(path `/v2/traces/${traceID}/runs`, {
+        return this._client.get(path `/api/v2/traces/${traceID}/runs`, {
             query,
             ...options,
             headers: buildHeaders([{ ...(Accept != null ? { Accept: Accept } : undefined) }, options?.headers]),
@@ -48,7 +48,7 @@ export class Traces extends APIResource {
      * ```
      */
     query(body, options) {
-        return this._client.getAPIList('/v2/traces/query', (ItemsCursorPostPagination), {
+        return this._client.getAPIList('/api/v2/traces/query', (ItemsCursorPostPagination), {
             body,
             method: 'post',
             ...options,

package/dist/client.cjs

@@ -46,7 +46,6 @@ const index_js_1 = require("./index.cjs");
 const index_js_2 = require("./_openapi_client/index.cjs");
 const _uuid_js_1 = require("./utils/_uuid.cjs");
 const warn_js_1 = require("./utils/warn.cjs");
-const constants_js_1 = require("./utils/constants.cjs");
 const prompts_js_1 = require("./utils/prompts.cjs");
 const error_js_1 = require("./utils/error.cjs");
 const index_js_3 = require("./utils/prompt_cache/index.cjs");
@@ -161,9 +160,13 @@ function _formatFeedbackScore(score) {
     }
     return score;
 }
-function _checkBackendVersion(version, minVersion = constants_js_1._MIN_BACKEND_VERSION) {
+function _checkBackendVersion(backendVersion, minVersion) {
+    if (!backendVersion) {
+        // /info was unreachable (or skipped): nothing to compare against.
+        return;
+    }
     const parse = (v) => v.split(".").map((s) => parseInt(s, 10));
-    const [maj, min, pat] = parse(version);
+    const [maj, min, pat] = parse(backendVersion);
     const [rMaj, rMin, rPat] = parse(minVersion);
     if (isNaN(maj) ||
         isNaN(min) ||
@@ -171,13 +174,16 @@ function _checkBackendVersion(version, minVersion = constants_js_1._MIN_BACKEND_
         isNaN(rMaj) ||
         isNaN(rMin) ||
         isNaN(rPat)) {
-        console.warn(`[LANGSMITH]: Could not parse backend version ${JSON.stringify(version)} for compatibility check.`);
+        console.warn(`[LANGSMITH]: Could not parse backend version ${JSON.stringify(backendVersion)} for compatibility check.`);
         return;
     }
     if (maj < rMaj ||
         (maj === rMaj && min < rMin) ||
         (maj === rMaj && min === rMin && pat < rPat)) {
-        console.warn(`[LANGSMITH]: Backend version ${JSON.stringify(version)} is older than the minimum version required by this SDK (${JSON.stringify(minVersion)}). Some features may not work as expected.`);
+        console.warn(`[LANGSMITH]: Backend version ${JSON.stringify(backendVersion)} is older than ` +
+            `the minimum version required by this SDK (${JSON.stringify(minVersion)}). ` +
+            "Some features may not work as expected. See " +
+            "https://docs.langchain.com/langsmith/smithdb-sdk-migration");
     }
 }
 exports.DEFAULT_UNCOMPRESSED_BATCH_SIZE_LIMIT_BYTES = 24 * 1024 * 1024;
@@ -648,11 +654,11 @@ class Client {
             writable: true,
             value: void 0
         });
-        Object.defineProperty(this, "_stainlessVersionChecked", {
+        Object.defineProperty(this, "_stainlessVersionsChecked", {
             enumerable: true,
             configurable: true,
             writable: true,
-            value: false
+            value: new Set()
         });
         Object.defineProperty(this, "manualFlushMode", {
             enumerable: true,
@@ -958,7 +964,12 @@ class Client {
         this._customHeaders = value ?? {};
     }
     _getOpenAPIBaseUrl() {
-        return this.apiUrl.endsWith("/v1") ? this.apiUrl.slice(0, -3) : this.apiUrl;
+        const url = this.apiUrl.replace(/\/$/, "");
+        for (const suffix of ["/api/v1", "/api"]) {
+            if (url.endsWith(suffix))
+                return url.slice(0, -suffix.length);
+        }
+        return url;
     }
     _newOpenAPIClient() {
         const defaultHeaders = this.apiKey === undefined && this.workspaceId === undefined
@@ -981,35 +992,42 @@ class Client {
         return needsV1Prefix ? `/v1/platform/${path}` : `/platform/${path}`;
     }
     get evaluators() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.onlineEvaluators;
     }
     get runs() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.runs;
     }
     /** Access the v2 sandboxes resource (registries, snapshots, boxes). */
     get sandboxes() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.sandboxes;
     }
     /** Access the v2 datasets resource (experimentRuns, etc.). */
     get datasets() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.datasets;
     }
+    /** Access the annotation queues resource (runs, items). */
+    get annotationQueues() {
+        // The items endpoints landed in backend 0.16.14; the rest are older.
+        this._checkStainlessVersion("0.16.14");
+        return this.openAPIClient.annotationQueues;
+    }
     /** Access the threads resource (query, stats, listTraces). */
     get threads() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.threads;
     }
     /** Access the traces resource (query, listRuns). */
     get traces() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.traces;
     }
     /** Access the public shared-run resource. */
     get public() {
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.public;
     }
     async processInputs(inputs) {
@@ -1498,15 +1516,13 @@ class Client {
         }
         return json;
     }
-    _checkStainlessVersion() {
-        if (this._stainlessVersionChecked)
+    _checkStainlessVersion(minVersion) {
+        if (this._stainlessVersionsChecked.has(minVersion))
             return;
-        this._stainlessVersionChecked = true;
+        this._stainlessVersionsChecked.add(minVersion);
         this._ensureServerInfo()
             .then((serverInfo) => {
-            if (serverInfo?.version) {
-                _checkBackendVersion(serverInfo.version);
-            }
+            _checkBackendVersion(serverInfo?.version, minVersion);
         })
             .catch(() => {
             // _ensureServerInfo handles and logs its own errors
@@ -1537,6 +1553,20 @@ class Client {
         const serverInfo = await this._ensureServerInfo();
         return serverInfo.instance_flags?.sdb_query_enabled === true;
     }
+    /**
+     * Throw on SmithDB-only deployments, warn elsewhere. Call only when run-level
+     * feedback has no sessionId.
+     */
+    async _checkFeedbackSessionId() {
+        const docs = "https://docs.langchain.com/langsmith/smithdb-sdk-migration#feedback-create";
+        const serverInfo = await this._ensureServerInfo();
+        if (serverInfo.instance_flags?.ch_query_enabled === false) {
+            throw new Error(`sessionId must be provided when creating feedback for a run: this ` +
+                `deployment cannot locate the run without it. See ${docs}`);
+        }
+        (0, warn_js_1.warnOnce)(`Creating feedback for a run without sessionId is deprecated and will ` +
+            `stop working in a future release. See ${docs}`);
+    }
     async _getSettings() {
         if (!this.settings) {
             this.settings = this._get("/settings");
@@ -3631,13 +3661,19 @@ class Client {
             return res;
         });
     }
-    async createFeedback(runId, key, { score, value, correction, comment, sourceInfo, feedbackSourceType = "api", sourceRunId, feedbackId, feedbackConfig, projectId, comparativeExperimentId, sessionId, startTime, extendTraceRetention, }) {
+    async createFeedback(runIdOrParams, keyArg, optionsArg) {
+        const { runId = null, key, score, value, correction, comment, sourceInfo, feedbackSourceType = "api", sourceRunId, feedbackId, feedbackConfig, projectId, comparativeExperimentId, sessionId, startTime, extendTraceRetention, } = typeof runIdOrParams === "object" && runIdOrParams !== null
+            ? runIdOrParams
+            : { runId: runIdOrParams, key: keyArg, ...optionsArg };
         if (!runId && !projectId) {
             throw new Error("One of runId or projectId must be provided");
         }
         if (runId && projectId) {
             throw new Error("Only one of runId or projectId can be provided");
         }
+        if (runId && sessionId === undefined) {
+            await this._checkFeedbackSessionId();
+        }
         const feedback_source = {
             type: feedbackSourceType ?? "api",
             metadata: sourceInfo ?? {},

package/dist/client.d.ts

@@ -7,6 +7,7 @@ import { OnlineEvaluators as Evaluators } from "./_openapi_client/resources/onli
 import { Runs as OpenAPIRuns } from "./_openapi_client/resources/runs.js";
 import { Sandboxes } from "./_openapi_client/resources/sandboxes/sandboxes.js";
 import { Datasets } from "./_openapi_client/resources/datasets/datasets.js";
+import { AnnotationQueues } from "./_openapi_client/resources/annotation-queues/annotation-queues.js";
 import { Threads } from "./_openapi_client/resources/threads.js";
 import { Traces } from "./_openapi_client/resources/traces.js";
 import { Public } from "./_openapi_client/resources/public/public.js";
@@ -335,6 +336,51 @@ interface ProjectOptions {
 }
 type RecordStringAny = Record<string, any>;
 export type FeedbackSourceType = "model" | "api" | "app";
+export type CreateFeedbackOptions = {
+    /** The metric name, tag, or aspect to provide feedback on. */
+    key: string;
+    score?: ScoreType;
+    value?: ValueType;
+    correction?: object;
+    comment?: string;
+    sourceInfo?: object;
+    feedbackSourceType?: FeedbackSourceType;
+    feedbackConfig?: FeedbackConfig;
+    sourceRunId?: string;
+    feedbackId?: string;
+    comparativeExperimentId?: string;
+    /**
+     * The run's start time, ISO string or epoch ms. Better performance if provided.
+     */
+    startTime?: number | string;
+    /** If false, create feedback without extending the trace's retention tier. */
+    extendTraceRetention?: boolean;
+};
+/** @deprecated Pass all params within an object and populate sessionId. */
+export type CreateFeedbackLegacyOptions = Omit<CreateFeedbackOptions, "key"> & {
+    /** @deprecated This option is no longer used. */
+    eager?: boolean;
+    /**
+     * The session (project) ID of the run. Required for run-level feedback;
+     * omitting it is deprecated. See
+     * https://docs.langchain.com/langsmith/smithdb-sdk-migration#feedback-create
+     */
+    sessionId?: string;
+    /** The project (or experiment) to provide feedback on, for session-level feedback. */
+    projectId?: string;
+};
+export type CreateFeedbackParams = CreateFeedbackOptions & ({
+    /** The run to provide feedback on. */
+    runId: string;
+    /** The session (project) ID of the run. */
+    sessionId: string;
+    projectId?: never;
+} | {
+    runId?: null;
+    /** The project (or experiment) to provide feedback on. */
+    projectId: string;
+    sessionId?: never;
+});
 export type CreateExampleOptions = {
     /** The ID of the dataset to create the example in. */
     datasetId?: string;
@@ -397,7 +443,7 @@ export interface ListThreadsItem extends Thread {
     runs: Run[];
 }
 export declare function mergeRuntimeEnvIntoRun<T extends RunCreate | RunUpdate>(run: T, cachedEnvVars?: Record<string, string>, omitTracedRuntimeInfo?: boolean): T;
-export declare function _checkBackendVersion(version: string, minVersion?: string): void;
+export declare function _checkBackendVersion(backendVersion: string, minVersion: string): void;
 export declare const DEFAULT_UNCOMPRESSED_BATCH_SIZE_LIMIT_BYTES: number;
 /** Default maximum memory (1GB) for queue size limits. */
 export declare const DEFAULT_MAX_SIZE_BYTES: number;
@@ -462,7 +508,7 @@ export declare class Client implements LangSmithTracingClientInterface {
     private traceBatchConcurrency;
     private _serverInfo;
     private _getServerInfoPromise?;
-    private _stainlessVersionChecked;
+    private _stainlessVersionsChecked;
     private manualFlushMode;
     private _serializeWorker?;
     /**
@@ -526,6 +572,8 @@ export declare class Client implements LangSmithTracingClientInterface {
     get sandboxes(): Sandboxes;
     /** Access the v2 datasets resource (experimentRuns, etc.). */
     get datasets(): Datasets;
+    /** Access the annotation queues resource (runs, items). */
+    get annotationQueues(): AnnotationQueues;
     /** Access the threads resource (query, stats, listTraces). */
     get threads(): Threads;
     /** Access the traces resource (query, listRuns). */
@@ -587,6 +635,11 @@ export declare class Client implements LangSmithTracingClientInterface {
     private _checkStainlessVersion;
     protected _ensureServerInfo(): Promise<Record<string, any>>;
     _supportsSDBQuery(): Promise<boolean>;
+    /**
+     * Throw on SmithDB-only deployments, warn elsewhere. Call only when run-level
+     * feedback has no sessionId.
+     */
+    private _checkFeedbackSessionId;
     protected _getSettings(): Promise<LangSmithSettings>;
     /**
      * Flushes current queued traces.
@@ -963,26 +1016,9 @@ export declare class Client implements LangSmithTracingClientInterface {
         exampleIds: string[];
         remove?: boolean;
     }): Promise<void>;
-    createFeedback(runId: string | null, key: string, { score, value, correction, comment, sourceInfo, feedbackSourceType, sourceRunId, feedbackId, feedbackConfig, projectId, comparativeExperimentId, sessionId, startTime, extendTraceRetention, }: {
-        score?: ScoreType;
-        value?: ValueType;
-        correction?: object;
-        comment?: string;
-        sourceInfo?: object;
-        feedbackSourceType?: FeedbackSourceType;
-        feedbackConfig?: FeedbackConfig;
-        sourceRunId?: string;
-        feedbackId?: string;
-        eager?: boolean;
-        projectId?: string;
-        comparativeExperimentId?: string;
-        /** The session (project) ID of the run this feedback is for. */
-        sessionId?: string;
-        /** The start time of the run this feedback is for. Accepts ISO string or epoch ms. */
-        startTime?: number | string;
-        /** If false, create feedback without extending the trace's retention tier. */
-        extendTraceRetention?: boolean;
-    }): Promise<Feedback>;
+    createFeedback(params: CreateFeedbackParams): Promise<Feedback>;
+    /** @deprecated Pass all params within an object and populate sessionId. */
+    createFeedback(runId: string | null, key: string, options: CreateFeedbackLegacyOptions): Promise<Feedback>;
     updateFeedback(feedbackId: string, { score, value, correction, comment, }: {
         score?: number | boolean | null;
         value?: number | boolean | string | object | null;

package/dist/client.js

@@ -8,7 +8,6 @@ import { __version__ } from "./index.js";
 import { Langsmith as OpenAPILangsmith } from "./_openapi_client/index.js";
 import { assertUuid } from "./utils/_uuid.js";
 import { warnOnce } from "./utils/warn.js";
-import { _MIN_BACKEND_VERSION } from "./utils/constants.js";
 import { parseHubIdentifier } from "./utils/prompts.js";
 import { raiseForStatus, isLangSmithNotFoundError, isLangSmithConflictError, } from "./utils/error.js";
 import { promptCacheSingleton, } from "./utils/prompt_cache/index.js";
@@ -123,9 +122,13 @@ function _formatFeedbackScore(score) {
     }
     return score;
 }
-export function _checkBackendVersion(version, minVersion = _MIN_BACKEND_VERSION) {
+export function _checkBackendVersion(backendVersion, minVersion) {
+    if (!backendVersion) {
+        // /info was unreachable (or skipped): nothing to compare against.
+        return;
+    }
     const parse = (v) => v.split(".").map((s) => parseInt(s, 10));
-    const [maj, min, pat] = parse(version);
+    const [maj, min, pat] = parse(backendVersion);
     const [rMaj, rMin, rPat] = parse(minVersion);
     if (isNaN(maj) ||
         isNaN(min) ||
@@ -133,13 +136,16 @@ export function _checkBackendVersion(version, minVersion = _MIN_BACKEND_VERSION)
         isNaN(rMaj) ||
         isNaN(rMin) ||
         isNaN(rPat)) {
-        console.warn(`[LANGSMITH]: Could not parse backend version ${JSON.stringify(version)} for compatibility check.`);
+        console.warn(`[LANGSMITH]: Could not parse backend version ${JSON.stringify(backendVersion)} for compatibility check.`);
         return;
     }
     if (maj < rMaj ||
         (maj === rMaj && min < rMin) ||
         (maj === rMaj && min === rMin && pat < rPat)) {
-        console.warn(`[LANGSMITH]: Backend version ${JSON.stringify(version)} is older than the minimum version required by this SDK (${JSON.stringify(minVersion)}). Some features may not work as expected.`);
+        console.warn(`[LANGSMITH]: Backend version ${JSON.stringify(backendVersion)} is older than ` +
+            `the minimum version required by this SDK (${JSON.stringify(minVersion)}). ` +
+            "Some features may not work as expected. See " +
+            "https://docs.langchain.com/langsmith/smithdb-sdk-migration");
     }
 }
 export const DEFAULT_UNCOMPRESSED_BATCH_SIZE_LIMIT_BYTES = 24 * 1024 * 1024;
@@ -609,11 +615,11 @@ export class Client {
             writable: true,
             value: void 0
         });
-        Object.defineProperty(this, "_stainlessVersionChecked", {
+        Object.defineProperty(this, "_stainlessVersionsChecked", {
             enumerable: true,
             configurable: true,
             writable: true,
-            value: false
+            value: new Set()
         });
         Object.defineProperty(this, "manualFlushMode", {
             enumerable: true,
@@ -919,7 +925,12 @@ export class Client {
         this._customHeaders = value ?? {};
     }
     _getOpenAPIBaseUrl() {
-        return this.apiUrl.endsWith("/v1") ? this.apiUrl.slice(0, -3) : this.apiUrl;
+        const url = this.apiUrl.replace(/\/$/, "");
+        for (const suffix of ["/api/v1", "/api"]) {
+            if (url.endsWith(suffix))
+                return url.slice(0, -suffix.length);
+        }
+        return url;
     }
     _newOpenAPIClient() {
         const defaultHeaders = this.apiKey === undefined && this.workspaceId === undefined
@@ -942,35 +953,42 @@ export class Client {
         return needsV1Prefix ? `/v1/platform/${path}` : `/platform/${path}`;
     }
     get evaluators() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.onlineEvaluators;
     }
     get runs() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.runs;
     }
     /** Access the v2 sandboxes resource (registries, snapshots, boxes). */
     get sandboxes() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.sandboxes;
     }
     /** Access the v2 datasets resource (experimentRuns, etc.). */
     get datasets() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.datasets;
     }
+    /** Access the annotation queues resource (runs, items). */
+    get annotationQueues() {
+        // The items endpoints landed in backend 0.16.14; the rest are older.
+        this._checkStainlessVersion("0.16.14");
+        return this.openAPIClient.annotationQueues;
+    }
     /** Access the threads resource (query, stats, listTraces). */
     get threads() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.threads;
     }
     /** Access the traces resource (query, listRuns). */
     get traces() {
-        this._checkStainlessVersion();
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.traces;
     }
     /** Access the public shared-run resource. */
     get public() {
+        this._checkStainlessVersion("0.16.0");
         return this.openAPIClient.public;
     }
     async processInputs(inputs) {
@@ -1459,15 +1477,13 @@ export class Client {
         }
         return json;
     }
-    _checkStainlessVersion() {
-        if (this._stainlessVersionChecked)
+    _checkStainlessVersion(minVersion) {
+        if (this._stainlessVersionsChecked.has(minVersion))
             return;
-        this._stainlessVersionChecked = true;
+        this._stainlessVersionsChecked.add(minVersion);
         this._ensureServerInfo()
             .then((serverInfo) => {
-            if (serverInfo?.version) {
-                _checkBackendVersion(serverInfo.version);
-            }
+            _checkBackendVersion(serverInfo?.version, minVersion);
         })
             .catch(() => {
             // _ensureServerInfo handles and logs its own errors
@@ -1498,6 +1514,20 @@ export class Client {
         const serverInfo = await this._ensureServerInfo();
         return serverInfo.instance_flags?.sdb_query_enabled === true;
     }
+    /**
+     * Throw on SmithDB-only deployments, warn elsewhere. Call only when run-level
+     * feedback has no sessionId.
+     */
+    async _checkFeedbackSessionId() {
+        const docs = "https://docs.langchain.com/langsmith/smithdb-sdk-migration#feedback-create";
+        const serverInfo = await this._ensureServerInfo();
+        if (serverInfo.instance_flags?.ch_query_enabled === false) {
+            throw new Error(`sessionId must be provided when creating feedback for a run: this ` +
+                `deployment cannot locate the run without it. See ${docs}`);
+        }
+        warnOnce(`Creating feedback for a run without sessionId is deprecated and will ` +
+            `stop working in a future release. See ${docs}`);
+    }
     async _getSettings() {
         if (!this.settings) {
             this.settings = this._get("/settings");
@@ -3592,13 +3622,19 @@ export class Client {
             return res;
         });
     }
-    async createFeedback(runId, key, { score, value, correction, comment, sourceInfo, feedbackSourceType = "api", sourceRunId, feedbackId, feedbackConfig, projectId, comparativeExperimentId, sessionId, startTime, extendTraceRetention, }) {
+    async createFeedback(runIdOrParams, keyArg, optionsArg) {
+        const { runId = null, key, score, value, correction, comment, sourceInfo, feedbackSourceType = "api", sourceRunId, feedbackId, feedbackConfig, projectId, comparativeExperimentId, sessionId, startTime, extendTraceRetention, } = typeof runIdOrParams === "object" && runIdOrParams !== null
+            ? runIdOrParams
+            : { runId: runIdOrParams, key: keyArg, ...optionsArg };
         if (!runId && !projectId) {
             throw new Error("One of runId or projectId must be provided");
         }
         if (runId && projectId) {
             throw new Error("Only one of runId or projectId can be provided");
         }
+        if (runId && sessionId === undefined) {
+            await this._checkFeedbackSessionId();
+        }
         const feedback_source = {
             type: feedbackSourceType ?? "api",
             metadata: sourceInfo ?? {},

package/dist/index.cjs

@@ -35,6 +35,6 @@ Object.defineProperty(exports, "UnprocessableEntityError", { enumerable: true, g
 Object.defineProperty(exports, "RateLimitError", { enumerable: true, get: function () { return error_js_1.RateLimitError; } });
 Object.defineProperty(exports, "InternalServerError", { enumerable: true, get: function () { return error_js_1.InternalServerError; } });
 // Update using pnpm bump-version
-exports.__version__ = "0.8.7";
+exports.__version__ = "0.8.8";
 // Metadata key to hide a traced run from LangSmith's Messages View.
 exports.LS_MESSAGE_VIEW_EXCLUDE = "ls_message_view_exclude";

package/dist/index.d.ts

@@ -7,5 +7,5 @@ export { computeRunIdForSecondaryReplica, uuid7, uuid7FromTime, } from "./uuid.j
 export { isTracingEnabled } from "./utils/guard.js";
 export { Cache, PromptCache, type CacheConfig, type CacheMetrics, configureGlobalPromptCache, promptCacheSingleton, } from "./utils/prompt_cache/index.js";
 export { LangsmithError, APIError, APIUserAbortError, APIConnectionError, APIConnectionTimeoutError, BadRequestError, AuthenticationError, PermissionDeniedError, NotFoundError, ConflictError, UnprocessableEntityError, RateLimitError, InternalServerError, } from "./_openapi_client/core/error.js";
-export declare const __version__ = "0.8.7";
+export declare const __version__ = "0.8.8";
 export declare const LS_MESSAGE_VIEW_EXCLUDE: "ls_message_view_exclude";

package/dist/index.js

@@ -7,6 +7,6 @@ export { isTracingEnabled } from "./utils/guard.js";
 export { Cache, PromptCache, configureGlobalPromptCache, promptCacheSingleton, } from "./utils/prompt_cache/index.js";
 export { LangsmithError, APIError, APIUserAbortError, APIConnectionError, APIConnectionTimeoutError, BadRequestError, AuthenticationError, PermissionDeniedError, NotFoundError, ConflictError, UnprocessableEntityError, RateLimitError, InternalServerError, } from "./_openapi_client/core/error.js";
 // Update using pnpm bump-version
-export const __version__ = "0.8.7";
+export const __version__ = "0.8.8";
 // Metadata key to hide a traced run from LangSmith's Messages View.
 export const LS_MESSAGE_VIEW_EXCLUDE = "ls_message_view_exclude";

package/dist/sandbox/command_handle.cjs

@@ -247,9 +247,9 @@ class CommandHandle {
                 return; // Stream ended normally (exit message received)
             }
             catch (e) {
-                const eName = e != null && typeof e === "object" ? e.name : "";
-                if (eName !== "LangSmithSandboxConnectionError" &&
-                    eName !== "LangSmithSandboxServerReloadError") {
+                // instanceof, not a name check: subclasses (connect timeout, server
+                // reload) must stay retryable here.
+                if (!(e instanceof errors_js_1.LangSmithSandboxConnectionError)) {
                     throw e;
                 }
                 if (this._control && this._control.killed) {
@@ -259,7 +259,7 @@ class CommandHandle {
                 if (reconnectAttempts > CommandHandle.MAX_AUTO_RECONNECTS) {
                     throw new errors_js_1.LangSmithSandboxConnectionError(`Lost connection ${reconnectAttempts} times in succession, giving up`);
                 }
-                const isHotReload = eName === "LangSmithSandboxServerReloadError";
+                const isHotReload = e instanceof errors_js_1.LangSmithSandboxServerReloadError;
                 if (!isHotReload) {
                     const delay = Math.min(CommandHandle.BACKOFF_BASE * 2 ** (reconnectAttempts - 1), CommandHandle.BACKOFF_MAX);
                     await new Promise((r) => setTimeout(r, delay * 1000));

package/dist/sandbox/command_handle.js

@@ -4,7 +4,7 @@
  *
  * Port of Python's AsyncCommandHandle to TypeScript.
  */
-import { LangSmithSandboxConnectionError, LangSmithSandboxOperationError, LangSmithStreamEndedBeforeStartedError, } from "./errors.js";
+import { LangSmithSandboxConnectionError, LangSmithSandboxServerReloadError, LangSmithSandboxOperationError, LangSmithStreamEndedBeforeStartedError, } from "./errors.js";
 /**
  * Async handle to a running command with streaming output and auto-reconnect.
  *
@@ -244,9 +244,9 @@ export class CommandHandle {
                 return; // Stream ended normally (exit message received)
             }
             catch (e) {
-                const eName = e != null && typeof e === "object" ? e.name : "";
-                if (eName !== "LangSmithSandboxConnectionError" &&
-                    eName !== "LangSmithSandboxServerReloadError") {
+                // instanceof, not a name check: subclasses (connect timeout, server
+                // reload) must stay retryable here.
+                if (!(e instanceof LangSmithSandboxConnectionError)) {
                     throw e;
                 }
                 if (this._control && this._control.killed) {
@@ -256,7 +256,7 @@ export class CommandHandle {
                 if (reconnectAttempts > CommandHandle.MAX_AUTO_RECONNECTS) {
                     throw new LangSmithSandboxConnectionError(`Lost connection ${reconnectAttempts} times in succession, giving up`);
                 }
-                const isHotReload = eName === "LangSmithSandboxServerReloadError";
+                const isHotReload = e instanceof LangSmithSandboxServerReloadError;
                 if (!isHotReload) {
                     const delay = Math.min(CommandHandle.BACKOFF_BASE * 2 ** (reconnectAttempts - 1), CommandHandle.BACKOFF_MAX);
                     await new Promise((r) => setTimeout(r, delay * 1000));

package/dist/sandbox/errors.cjs

@@ -7,7 +7,7 @@
  * properties for specific handling when needed.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.LangSmithSandboxServerReloadError = exports.LangSmithStreamEndedBeforeStartedError = exports.LangSmithCommandTimeoutError = exports.LangSmithSandboxOperationError = exports.LangSmithSandboxNotReadyError = exports.LangSmithDataplaneNotConfiguredError = exports.LangSmithSandboxCreationError = exports.LangSmithResourceCreationError = exports.LangSmithQuotaExceededError = exports.LangSmithValidationError = exports.LangSmithResourceNameConflictError = exports.LangSmithResourceAlreadyExistsError = exports.LangSmithResourceInUseError = exports.LangSmithResourceTimeoutError = exports.LangSmithResourceNotFoundError = exports.LangSmithSandboxConnectionError = exports.LangSmithSandboxAuthenticationError = exports.LangSmithSandboxAPIError = exports.LangSmithSandboxError = void 0;
+exports.LangSmithSandboxServerReloadError = exports.LangSmithSandboxConnectTimeoutError = exports.LangSmithStreamEndedBeforeStartedError = exports.LangSmithCommandTimeoutError = exports.LangSmithSandboxOperationError = exports.LangSmithSandboxNotReadyError = exports.LangSmithDataplaneNotConfiguredError = exports.LangSmithSandboxCreationError = exports.LangSmithResourceCreationError = exports.LangSmithQuotaExceededError = exports.LangSmithValidationError = exports.LangSmithResourceNameConflictError = exports.LangSmithResourceAlreadyExistsError = exports.LangSmithResourceInUseError = exports.LangSmithResourceTimeoutError = exports.LangSmithResourceNotFoundError = exports.LangSmithSandboxConnectionError = exports.LangSmithSandboxAuthenticationError = exports.LangSmithSandboxAPIError = exports.LangSmithSandboxError = void 0;
 /**
  * Base exception for sandbox client errors.
  */
@@ -372,6 +372,21 @@ class LangSmithStreamEndedBeforeStartedError extends LangSmithSandboxOperationEr
     }
 }
 exports.LangSmithStreamEndedBeforeStartedError = LangSmithStreamEndedBeforeStartedError;
+/**
+ * Thrown when the socket fails or times out before the WebSocket handshake.
+ *
+ * Distinct from its parent because it is safely retryable: the execute frame
+ * was never sent, so re-issuing the same command_id cannot double-run a
+ * command. run() retries this with backoff; a plain connection error (a
+ * rejected handshake) is permanent and propagates immediately.
+ */
+class LangSmithSandboxConnectTimeoutError extends LangSmithSandboxConnectionError {
+    constructor(message) {
+        super(message);
+        this.name = "LangSmithSandboxConnectTimeoutError";
+    }
+}
+exports.LangSmithSandboxConnectTimeoutError = LangSmithSandboxConnectTimeoutError;
 /**
  * Raised when the sandbox server is reloading (close code 1001).
  *

package/dist/sandbox/errors.d.ts

@@ -165,6 +165,17 @@ export declare class LangSmithCommandTimeoutError extends LangSmithSandboxOperat
 export declare class LangSmithStreamEndedBeforeStartedError extends LangSmithSandboxOperationError {
     constructor(message: string);
 }
+/**
+ * Thrown when the socket fails or times out before the WebSocket handshake.
+ *
+ * Distinct from its parent because it is safely retryable: the execute frame
+ * was never sent, so re-issuing the same command_id cannot double-run a
+ * command. run() retries this with backoff; a plain connection error (a
+ * rejected handshake) is permanent and propagates immediately.
+ */
+export declare class LangSmithSandboxConnectTimeoutError extends LangSmithSandboxConnectionError {
+    constructor(message: string);
+}
 /**
  * Raised when the sandbox server is reloading (close code 1001).
  *