langsmith 0.8.8 → 0.8.10

package/dist/utils/profiles.cjs

@@ -35,6 +35,7 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ProfileAuth = exports.DEFAULT_API_URL = void 0;
 exports.hasValue = hasValue;
+exports.resolveTokenEndpoint = resolveTokenEndpoint;
 exports.loadProfileClientConfig = loadProfileClientConfig;
 const env_js_1 = require("./env.cjs");
 const fsUtils = __importStar(require("./fs.cjs"));
@@ -43,6 +44,8 @@ exports.DEFAULT_API_URL = "https://api.smith.langchain.com";
 const OAUTH_CLIENT_ID = "langsmith-cli";
 const TOKEN_REFRESH_LEEWAY_MS = 60_000;
 const TOKEN_REFRESH_TIMEOUT_MS = 10_000;
+const OAUTH_DISCOVERY_TIMEOUT_MS = 5_000;
+const WELL_KNOWN_OAUTH_PATH = "/.well-known/oauth-authorization-server";
 function isBrowserLikeRuntime() {
     const env = (0, env_js_1.getEnv)();
     return env === "browser" || env === "webworker";
@@ -125,6 +128,126 @@ function normalizeConfigUrl(apiUrl) {
         ? normalized.slice(0, -apiV1Suffix.length)
         : normalized;
 }
+/**
+ * Metadata base URLs to probe, most specific first: the configured mount point,
+ * then the self-hosted and SaaS locations, so both a bare origin and an
+ * explicit `/api` resolve.
+ */
+function oauthDiscoveryCandidates(apiUrl) {
+    const given = normalizeConfigUrl(apiUrl);
+    const origin = given.endsWith("/api")
+        ? given.slice(0, -"/api".length)
+        : given;
+    const candidates = [];
+    for (const candidate of [given, `${origin}/api`, origin]) {
+        if (candidate && candidate !== "/api" && !candidates.includes(candidate)) {
+            candidates.push(candidate);
+        }
+    }
+    return candidates;
+}
+/**
+ * Check the document describes the deployment we probed. RFC 8414 requires the
+ * issuer to match the URL the well-known path was built from, and every
+ * endpoint must share the issuer's origin. Refresh tokens are posted to these
+ * endpoints, so an unvalidated document could redirect credentials elsewhere.
+ */
+function isTrustedOAuthMetadata(doc, base) {
+    const { issuer } = doc;
+    if (typeof issuer !== "string" ||
+        issuer.replace(/\/+$/, "") !== base.replace(/\/+$/, "")) {
+        return false;
+    }
+    let issuerUrl;
+    try {
+        issuerUrl = new URL(issuer);
+    }
+    catch {
+        return false;
+    }
+    for (const endpoint of [
+        doc.device_authorization_endpoint,
+        doc.token_endpoint,
+    ]) {
+        if (typeof endpoint !== "string" || !endpoint) {
+            return false;
+        }
+        try {
+            const url = new URL(endpoint);
+            if (url.protocol !== issuerUrl.protocol || url.host !== issuerUrl.host) {
+                return false;
+            }
+        }
+        catch {
+            return false;
+        }
+    }
+    return true;
+}
+/**
+ * Metadata URLs for an issuer, RFC 8414 form first. RFC 8414 inserts the
+ * well-known segment between the origin and the issuer path, so
+ * `https://host/api` is described at
+ * `https://host/.well-known/oauth-authorization-server/api`. Deployments
+ * commonly also serve the appended form, and for a path-less issuer the two are
+ * identical.
+ */
+function oauthMetadataUrls(base) {
+    const appended = `${base}${WELL_KNOWN_OAUTH_PATH}`;
+    let inserted;
+    try {
+        const url = new URL(base);
+        inserted = `${url.origin}${WELL_KNOWN_OAUTH_PATH}${url.pathname === "/" ? "" : url.pathname}`;
+    }
+    catch {
+        return [appended];
+    }
+    return inserted === appended ? [inserted] : [inserted, appended];
+}
+async function fetchOAuthMetadata(url, base, fetchImplementation) {
+    let response;
+    try {
+        response = await fetchImplementation(url, {
+            method: "GET",
+            headers: { Accept: "application/json" },
+            signal: AbortSignal.timeout(OAUTH_DISCOVERY_TIMEOUT_MS),
+        });
+    }
+    catch {
+        return undefined;
+    }
+    if (!response.ok) {
+        return undefined;
+    }
+    let doc;
+    try {
+        // Self-hosted answers unknown paths with an HTML 200 from the SPA.
+        doc = (await response.json());
+    }
+    catch {
+        return undefined;
+    }
+    if (!doc || typeof doc !== "object" || !isTrustedOAuthMetadata(doc, base)) {
+        return undefined;
+    }
+    return doc;
+}
+/**
+ * Resolve the token endpoint, preferring the deployment's metadata document.
+ * Falls back to `<mount>/oauth/token`, keeping a trailing `/api` because
+ * self-hosted mounts the authorization server under it.
+ */
+async function resolveTokenEndpoint(apiUrl, fetchImplementation) {
+    for (const base of oauthDiscoveryCandidates(apiUrl)) {
+        for (const url of oauthMetadataUrls(base)) {
+            const doc = await fetchOAuthMetadata(url, base, fetchImplementation);
+            if (doc) {
+                return doc.token_endpoint;
+            }
+        }
+    }
+    return `${normalizeConfigUrl(apiUrl)}/oauth/token`;
+}
 function applyTokenResponse(profile, token) {
     profile.oauth ??= {};
     if (token.access_token) {
@@ -262,7 +385,8 @@ class ProfileAuth {
                 client_id: OAUTH_CLIENT_ID,
                 refresh_token: this.state.profile.oauth?.refresh_token ?? refreshToken,
             });
-            const response = await fetchImplementation(`${normalizeConfigUrl(refreshApiUrl)}/oauth/token`, {
+            const tokenEndpoint = await resolveTokenEndpoint(refreshApiUrl, fetchImplementation);
+            const response = await fetchImplementation(tokenEndpoint, {
                 method: "POST",
                 headers: {
                     "Content-Type": "application/x-www-form-urlencoded",

package/dist/utils/profiles.d.ts

@@ -33,6 +33,12 @@ export type ProfileClientConfig = {
     profileAuth?: ProfileAuth;
 };
 export declare function hasValue(value?: string | null): boolean;
+/**
+ * Resolve the token endpoint, preferring the deployment's metadata document.
+ * Falls back to `<mount>/oauth/token`, keeping a trailing `/api` because
+ * self-hosted mounts the authorization server under it.
+ */
+export declare function resolveTokenEndpoint(apiUrl: string, fetchImplementation: typeof fetch): Promise<string>;
 export declare function loadProfileClientConfig(): ProfileClientConfig;
 export declare class ProfileAuth {
     private state;

package/dist/utils/profiles.js

@@ -5,6 +5,8 @@ export const DEFAULT_API_URL = "https://api.smith.langchain.com";
 const OAUTH_CLIENT_ID = "langsmith-cli";
 const TOKEN_REFRESH_LEEWAY_MS = 60_000;
 const TOKEN_REFRESH_TIMEOUT_MS = 10_000;
+const OAUTH_DISCOVERY_TIMEOUT_MS = 5_000;
+const WELL_KNOWN_OAUTH_PATH = "/.well-known/oauth-authorization-server";
 function isBrowserLikeRuntime() {
     const env = getEnv();
     return env === "browser" || env === "webworker";
@@ -87,6 +89,126 @@ function normalizeConfigUrl(apiUrl) {
         ? normalized.slice(0, -apiV1Suffix.length)
         : normalized;
 }
+/**
+ * Metadata base URLs to probe, most specific first: the configured mount point,
+ * then the self-hosted and SaaS locations, so both a bare origin and an
+ * explicit `/api` resolve.
+ */
+function oauthDiscoveryCandidates(apiUrl) {
+    const given = normalizeConfigUrl(apiUrl);
+    const origin = given.endsWith("/api")
+        ? given.slice(0, -"/api".length)
+        : given;
+    const candidates = [];
+    for (const candidate of [given, `${origin}/api`, origin]) {
+        if (candidate && candidate !== "/api" && !candidates.includes(candidate)) {
+            candidates.push(candidate);
+        }
+    }
+    return candidates;
+}
+/**
+ * Check the document describes the deployment we probed. RFC 8414 requires the
+ * issuer to match the URL the well-known path was built from, and every
+ * endpoint must share the issuer's origin. Refresh tokens are posted to these
+ * endpoints, so an unvalidated document could redirect credentials elsewhere.
+ */
+function isTrustedOAuthMetadata(doc, base) {
+    const { issuer } = doc;
+    if (typeof issuer !== "string" ||
+        issuer.replace(/\/+$/, "") !== base.replace(/\/+$/, "")) {
+        return false;
+    }
+    let issuerUrl;
+    try {
+        issuerUrl = new URL(issuer);
+    }
+    catch {
+        return false;
+    }
+    for (const endpoint of [
+        doc.device_authorization_endpoint,
+        doc.token_endpoint,
+    ]) {
+        if (typeof endpoint !== "string" || !endpoint) {
+            return false;
+        }
+        try {
+            const url = new URL(endpoint);
+            if (url.protocol !== issuerUrl.protocol || url.host !== issuerUrl.host) {
+                return false;
+            }
+        }
+        catch {
+            return false;
+        }
+    }
+    return true;
+}
+/**
+ * Metadata URLs for an issuer, RFC 8414 form first. RFC 8414 inserts the
+ * well-known segment between the origin and the issuer path, so
+ * `https://host/api` is described at
+ * `https://host/.well-known/oauth-authorization-server/api`. Deployments
+ * commonly also serve the appended form, and for a path-less issuer the two are
+ * identical.
+ */
+function oauthMetadataUrls(base) {
+    const appended = `${base}${WELL_KNOWN_OAUTH_PATH}`;
+    let inserted;
+    try {
+        const url = new URL(base);
+        inserted = `${url.origin}${WELL_KNOWN_OAUTH_PATH}${url.pathname === "/" ? "" : url.pathname}`;
+    }
+    catch {
+        return [appended];
+    }
+    return inserted === appended ? [inserted] : [inserted, appended];
+}
+async function fetchOAuthMetadata(url, base, fetchImplementation) {
+    let response;
+    try {
+        response = await fetchImplementation(url, {
+            method: "GET",
+            headers: { Accept: "application/json" },
+            signal: AbortSignal.timeout(OAUTH_DISCOVERY_TIMEOUT_MS),
+        });
+    }
+    catch {
+        return undefined;
+    }
+    if (!response.ok) {
+        return undefined;
+    }
+    let doc;
+    try {
+        // Self-hosted answers unknown paths with an HTML 200 from the SPA.
+        doc = (await response.json());
+    }
+    catch {
+        return undefined;
+    }
+    if (!doc || typeof doc !== "object" || !isTrustedOAuthMetadata(doc, base)) {
+        return undefined;
+    }
+    return doc;
+}
+/**
+ * Resolve the token endpoint, preferring the deployment's metadata document.
+ * Falls back to `<mount>/oauth/token`, keeping a trailing `/api` because
+ * self-hosted mounts the authorization server under it.
+ */
+export async function resolveTokenEndpoint(apiUrl, fetchImplementation) {
+    for (const base of oauthDiscoveryCandidates(apiUrl)) {
+        for (const url of oauthMetadataUrls(base)) {
+            const doc = await fetchOAuthMetadata(url, base, fetchImplementation);
+            if (doc) {
+                return doc.token_endpoint;
+            }
+        }
+    }
+    return `${normalizeConfigUrl(apiUrl)}/oauth/token`;
+}
 function applyTokenResponse(profile, token) {
     profile.oauth ??= {};
     if (token.access_token) {
@@ -224,7 +346,8 @@ export class ProfileAuth {
                 client_id: OAUTH_CLIENT_ID,
                 refresh_token: this.state.profile.oauth?.refresh_token ?? refreshToken,
             });
-            const response = await fetchImplementation(`${normalizeConfigUrl(refreshApiUrl)}/oauth/token`, {
+            const tokenEndpoint = await resolveTokenEndpoint(refreshApiUrl, fetchImplementation);
+            const response = await fetchImplementation(tokenEndpoint, {
                 method: "POST",
                 headers: {
                     "Content-Type": "application/x-www-form-urlencoded",

package/dist/utils/v2_migration.cjs

@@ -1,7 +1,25 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.QueryBackend = void 0;
+exports.getQueryBackend = getQueryBackend;
 exports.v2RunToRun = v2RunToRun;
 exports.loadTracesV2 = loadTracesV2;
+exports.QueryBackend = {
+    CLICKHOUSE_ONLY: "clickhouse_only",
+    SMITHDB_ONLY: "smithdb_only",
+    DUAL: "dual",
+};
+/** Which backend(s) `/info`'s `instance_flags` indicate for run/trace queries. */
+function getQueryBackend(instanceFlags) {
+    const flags = instanceFlags ?? {};
+    const chEnabled = Boolean(flags.ch_query_enabled ?? true);
+    const sdbEnabled = Boolean(flags.sdb_query_enabled ?? false);
+    if (!chEnabled && sdbEnabled)
+        return exports.QueryBackend.SMITHDB_ONLY;
+    if (chEnabled && sdbEnabled)
+        return exports.QueryBackend.DUAL;
+    return exports.QueryBackend.CLICKHOUSE_ONLY;
+}
 // Omitting selects from `/v2/runs/query` returns only the run ID.
 const V2_RUN_SELECTS = [
     "ID",

package/dist/utils/v2_migration.d.ts

@@ -1,6 +1,14 @@
 import type { Run as V2Run } from "../_openapi_client/resources/runs/runs.js";
 import type { Client } from "../client.js";
 import type { Run, TracerSession } from "../schemas.js";
+export declare const QueryBackend: {
+    readonly CLICKHOUSE_ONLY: "clickhouse_only";
+    readonly SMITHDB_ONLY: "smithdb_only";
+    readonly DUAL: "dual";
+};
+export type QueryBackend = (typeof QueryBackend)[keyof typeof QueryBackend];
+/** Which backend(s) `/info`'s `instance_flags` indicate for run/trace queries. */
+export declare function getQueryBackend(instanceFlags?: Record<string, unknown> | null): QueryBackend;
 export declare function v2RunToRun(run: V2Run): Run;
 export declare function loadTracesV2(client: Client, project: TracerSession, options: {
     isRoot?: boolean;

package/dist/utils/v2_migration.js

@@ -1,3 +1,19 @@
+export const QueryBackend = {
+    CLICKHOUSE_ONLY: "clickhouse_only",
+    SMITHDB_ONLY: "smithdb_only",
+    DUAL: "dual",
+};
+/** Which backend(s) `/info`'s `instance_flags` indicate for run/trace queries. */
+export function getQueryBackend(instanceFlags) {
+    const flags = instanceFlags ?? {};
+    const chEnabled = Boolean(flags.ch_query_enabled ?? true);
+    const sdbEnabled = Boolean(flags.sdb_query_enabled ?? false);
+    if (!chEnabled && sdbEnabled)
+        return QueryBackend.SMITHDB_ONLY;
+    if (chEnabled && sdbEnabled)
+        return QueryBackend.DUAL;
+    return QueryBackend.CLICKHOUSE_ONLY;
+}
 // Omitting selects from `/v2/runs/query` returns only the run ID.
 const V2_RUN_SELECTS = [
     "ID",

package/dist/utils/warn.cjs

@@ -1,10 +1,36 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports._resetWarnedMessages = _resetWarnedMessages;
 exports.warnOnce = warnOnce;
 const warnedMessages = {};
-function warnOnce(message) {
-    if (!warnedMessages[message]) {
-        console.warn(message);
-        warnedMessages[message] = true;
+/**
+ * Clear the warn-once record so a later call warns again.
+ *
+ * Test-only. `warnedMessages` is module-global and never otherwise reset, so
+ * without this a test asserting "warning X was (not) emitted" depends on whether
+ * an earlier test in the same file already consumed X.
+ *
+ * @internal
+ */
+function _resetWarnedMessages() {
+    for (const key of Object.keys(warnedMessages)) {
+        delete warnedMessages[key];
+    }
+}
+function warnOnce(message, options) {
+    const key = options?.code ?? message;
+    if (!warnedMessages[key]) {
+        warnedMessages[key] = true;
+        if (options?.type &&
+            typeof process !== "undefined" &&
+            typeof process.emitWarning === "function") {
+            process.emitWarning(message, { type: options.type, code: options.code });
+        }
+        else if (options?.type && options?.code) {
+            console.warn(`${options.type} [${options.code}]: ${message}`);
+        }
+        else {
+            console.warn(message);
+        }
     }
 }

package/dist/utils/warn.d.ts

@@ -1 +1,4 @@
-export declare function warnOnce(message: string): void;
+export declare function warnOnce(message: string, options?: {
+    type?: string;
+    code?: string;
+}): void;

package/dist/utils/warn.js

@@ -1,7 +1,32 @@
 const warnedMessages = {};
-export function warnOnce(message) {
-    if (!warnedMessages[message]) {
-        console.warn(message);
-        warnedMessages[message] = true;
+/**
+ * Clear the warn-once record so a later call warns again.
+ *
+ * Test-only. `warnedMessages` is module-global and never otherwise reset, so
+ * without this a test asserting "warning X was (not) emitted" depends on whether
+ * an earlier test in the same file already consumed X.
+ *
+ * @internal
+ */
+export function _resetWarnedMessages() {
+    for (const key of Object.keys(warnedMessages)) {
+        delete warnedMessages[key];
+    }
+}
+export function warnOnce(message, options) {
+    const key = options?.code ?? message;
+    if (!warnedMessages[key]) {
+        warnedMessages[key] = true;
+        if (options?.type &&
+            typeof process !== "undefined" &&
+            typeof process.emitWarning === "function") {
+            process.emitWarning(message, { type: options.type, code: options.code });
+        }
+        else if (options?.type && options?.code) {
+            console.warn(`${options.type} [${options.code}]: ${message}`);
+        }
+        else {
+            console.warn(message);
+        }
     }
 }

package/dist/wrappers/openai.cjs

@@ -2,6 +2,8 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.wrapOpenAI = void 0;
 const traceable_js_1 = require("../traceable.cjs");
+const traceable_js_2 = require("../singletons/traceable.cjs");
+const ls_agent_type_js_1 = require("../utils/ls_agent_type.cjs");
 const TRACED_INVOCATION_KEYS = [
     "frequency_penalty",
     "n",
@@ -261,6 +263,7 @@ const getChatModelInvocationParamsFn = (provider, prepopulatedInvocationParams,
                 ...prepopulatedInvocationParams,
                 ...ls_invocation_params,
             },
+            ...(0, ls_agent_type_js_1.defaultLsAgentTypeMetadata)(requestMetadata, (0, traceable_js_2.getCurrentRunTree)(true)),
         };
     };
 };
@@ -416,6 +419,7 @@ const wrapOpenAI = (openai, options) => {
                         ...prepopulatedInvocationParams,
                         ...ls_invocation_params,
                     },
+                    ...(0, ls_agent_type_js_1.defaultLsAgentTypeMetadata)({}, (0, traceable_js_2.getCurrentRunTree)(true)),
                 };
             },
             ...cleanedOptions,

package/dist/wrappers/openai.js

@@ -1,4 +1,6 @@
 import { isTraceableFunction, traceable, } from "../traceable.js";
+import { getCurrentRunTree } from "../singletons/traceable.js";
+import { defaultLsAgentTypeMetadata } from "../utils/ls_agent_type.js";
 const TRACED_INVOCATION_KEYS = [
     "frequency_penalty",
     "n",
@@ -258,6 +260,7 @@ const getChatModelInvocationParamsFn = (provider, prepopulatedInvocationParams,
                 ...prepopulatedInvocationParams,
                 ...ls_invocation_params,
             },
+            ...defaultLsAgentTypeMetadata(requestMetadata, getCurrentRunTree(true)),
         };
     };
 };
@@ -413,6 +416,7 @@ export const wrapOpenAI = (openai, options) => {
                         ...prepopulatedInvocationParams,
                         ...ls_invocation_params,
                     },
+                    ...defaultLsAgentTypeMetadata({}, getCurrentRunTree(true)),
                 };
             },
             ...cleanedOptions,

package/dist/wrappers/openai_agents.cjs

@@ -10,6 +10,7 @@ const node_async_hooks_1 = require("node:async_hooks");
 const run_trees_js_1 = require("../run_trees.cjs");
 const client_js_1 = require("../client.cjs");
 const traceable_js_1 = require("../singletons/traceable.cjs");
+const ls_agent_type_js_1 = require("../utils/ls_agent_type.cjs");
 traceable_js_1.AsyncLocalStorageProviderSingleton.initializeGlobalInstance(new node_async_hooks_1.AsyncLocalStorage());
 /**
  * Set the current AsyncLocalStorage store to the given RunTree without a
@@ -521,14 +522,15 @@ class OpenAIAgentsTracingProcessor {
         else {
             runName = "Agent workflow";
         }
-        // Build metadata
-        const runExtra = {
-            metadata: {
-                ...this._metadata,
-                ls_integration: "openai-agents-sdk",
-                ls_agent_type: "root",
-            },
+        // Merge three sources: processor defaults, trace.metadata (user's per-run
+        // ls_agent_type), and force-set ls_integration.
+        const mergedMetadata = {
+            ...this._metadata,
+            ...(trace.metadata ?? {}),
+            ls_integration: "openai-agents-sdk",
         };
+        Object.assign(mergedMetadata, (0, ls_agent_type_js_1.defaultLsAgentTypeMetadata)(mergedMetadata, currentRunTree));
+        const runExtra = { metadata: mergedMetadata };
         const traceDict = trace.toJSON() ?? {};
         const groupId = trace.groupId ??
             traceDict.groupId ??
@@ -589,9 +591,10 @@ class OpenAIAgentsTracingProcessor {
         }
         this._runs.delete(trace.traceId);
         const traceDict = trace.toJSON() ?? {};
+        // trace.metadata may have new keys since onTraceStart; pull them in.
         const metadata = {
-            ...traceDict.metadata,
             ...this._metadata,
+            ...traceDict.metadata,
         };
         try {
             // Update run with final inputs/outputs
@@ -695,10 +698,10 @@ class OpenAIAgentsTracingProcessor {
                 if (!childRun.extra.metadata) {
                     childRun.extra.metadata = {};
                 }
-                childRun.extra.metadata = {
-                    ...childRun.extra.metadata,
-                    ls_agent_type: "subagent",
-                };
+                const meta = childRun.extra.metadata;
+                if (!ls_agent_type_js_1.NON_ROOT_LS_AGENT_TYPES.has(meta.ls_agent_type)) {
+                    meta.ls_agent_type = "subagent";
+                }
             }
         }
         // Track span data type for parent lookups

package/dist/wrappers/openai_agents.js

@@ -7,6 +7,7 @@ import { AsyncLocalStorage } from "node:async_hooks";
 import { RunTree } from "../run_trees.js";
 import { Client } from "../client.js";
 import { AsyncLocalStorageProviderSingleton, getCurrentRunTree, } from "../singletons/traceable.js";
+import { NON_ROOT_LS_AGENT_TYPES, defaultLsAgentTypeMetadata, } from "../utils/ls_agent_type.js";
 AsyncLocalStorageProviderSingleton.initializeGlobalInstance(new AsyncLocalStorage());
 /**
  * Set the current AsyncLocalStorage store to the given RunTree without a
@@ -518,14 +519,15 @@ export class OpenAIAgentsTracingProcessor {
         else {
             runName = "Agent workflow";
         }
-        // Build metadata
-        const runExtra = {
-            metadata: {
-                ...this._metadata,
-                ls_integration: "openai-agents-sdk",
-                ls_agent_type: "root",
-            },
+        // Merge three sources: processor defaults, trace.metadata (user's per-run
+        // ls_agent_type), and force-set ls_integration.
+        const mergedMetadata = {
+            ...this._metadata,
+            ...(trace.metadata ?? {}),
+            ls_integration: "openai-agents-sdk",
         };
+        Object.assign(mergedMetadata, defaultLsAgentTypeMetadata(mergedMetadata, currentRunTree));
+        const runExtra = { metadata: mergedMetadata };
         const traceDict = trace.toJSON() ?? {};
         const groupId = trace.groupId ??
             traceDict.groupId ??
@@ -586,9 +588,10 @@ export class OpenAIAgentsTracingProcessor {
         }
         this._runs.delete(trace.traceId);
         const traceDict = trace.toJSON() ?? {};
+        // trace.metadata may have new keys since onTraceStart; pull them in.
         const metadata = {
-            ...traceDict.metadata,
             ...this._metadata,
+            ...traceDict.metadata,
         };
         try {
             // Update run with final inputs/outputs
@@ -692,10 +695,10 @@ export class OpenAIAgentsTracingProcessor {
                 if (!childRun.extra.metadata) {
                     childRun.extra.metadata = {};
                 }
-                childRun.extra.metadata = {
-                    ...childRun.extra.metadata,
-                    ls_agent_type: "subagent",
-                };
+                const meta = childRun.extra.metadata;
+                if (!NON_ROOT_LS_AGENT_TYPES.has(meta.ls_agent_type)) {
+                    meta.ls_agent_type = "subagent";
+                }
             }
         }
         // Track span data type for parent lookups

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "langsmith",
-  "version": "0.8.8",
+  "version": "0.8.10",
   "description": "Client library to connect to the LangSmith Observability and Evaluation Platform.",
   "packageManager": "pnpm@10.33.0",
   "files": [