fix(anonymizer): make deep cloning the default (#813)

langchain-ai · Jun 26, 2024 · 9cae98a · 9cae98a
2 parents 48eef4e + d37b3f6
commit 9cae98a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 16 deletions.
diff --git a/js/package.json b/js/package.json
@@ -1,6 +1,6 @@
 {
   "name": "langsmith",
-  "version": "0.1.33",
+  "version": "0.1.34",
   "description": "Client library to connect to the LangSmith LLM Tracing and Evaluation Platform.",
   "packageManager": "yarn@1.22.19",
   "files": [
@@ -263,4 +263,4 @@
     },
     "./package.json": "./package.json"
   }
-}
+}
diff --git a/js/src/anonymizer/index.ts b/js/src/anonymizer/index.ts
@@ -36,10 +36,6 @@ function extractStringNodes(data: unknown, options: { maxDepth?: number }) {
 }
 
 function deepClone<T>(data: T): T {
-  if ("structuredClone" in globalThis) {
-    return globalThis.structuredClone(data);
-  }
-
   return JSON.parse(JSON.stringify(data));
 }
 
@@ -60,20 +56,14 @@ export type ReplacerType =
 
 export function createAnonymizer(
   replacer: ReplacerType,
-  options?: {
-    maxDepth?: number;
-    deepClone?: boolean;
-  }
+  options?: { maxDepth?: number }
 ) {
   return <T>(data: T): T => {
-    const nodes = extractStringNodes(data, {
+    let mutateValue = deepClone(data);
+    const nodes = extractStringNodes(mutateValue, {
       maxDepth: options?.maxDepth,
     });
 
-    // by default we opt-in to mutate the value directly
-    // to improve performance
-    let mutateValue = options?.deepClone ? deepClone(data) : data;
-
     const processor: StringNodeProcessor = Array.isArray(replacer)
       ? (() => {
           const replacers: [regex: RegExp, replace: string][] = replacer.map(

diff --git a/js/src/index.ts b/js/src/index.ts
@@ -12,4 +12,4 @@ export type {
 export { RunTree, type RunTreeConfig } from "./run_trees.js";
 
 // Update using yarn bump-version
-export const __version__ = "0.1.33";
+export const __version__ = "0.1.34";