Fix deepClone so we can overwrite with undefined in when we mock kcContext

src/lib/tools/deepAssign.ts

@@ -1,32 +1,6 @@
 import { assert } from "tsafe/assert";
 import { is } from "tsafe/is";
-
+import { deepClone } from "./deepClone";
-function deepClone<T>(src: T): T {
-    const generateId = (() => {
-        const prefix = "xIfKdLsIIdJdLdOeJqePe";
-
-        let counter = 0;
-
-        return () => `${prefix}${counter++}`;
-    })();
-
-    const map = new Map<string, Function>();
-
-    return JSON.parse(
-        JSON.stringify(src, (...[, value]) => {
-            if (typeof value === "function") {
-                const id = generateId();
-
-                map.set(id, value);
-
-                return id;
-            }
-
-            return value;
-        }),
-        (...[, value]) => (typeof value === "string" && map.has(value) ? map.get(value) : value),
-    );
-}
 
 //Warning: Be mindful that because of array this is not idempotent.
 export function deepAssign(params: { target: Record<string, unknown>; source: Record<string, unknown> }) {

src/lib/tools/deepClone.ts

@@ -1,3 +1,17 @@
-export function deepClone<T>(arg: T): T {
+import "minimal-polyfills/Object.fromEntries";
-    return JSON.parse(JSON.stringify(arg));
+
+export function deepClone<T>(o: T): T {
+    if (!(o instanceof Object)) {
+        return o;
+    }
+
+    if (typeof o === "function") {
+        return o;
+    }
+
+    if (o instanceof Array) {
+        return o.map(deepClone) as any;
+    }
+
+    return Object.fromEntries(Object.entries(o).map(([key, value]) => [key, deepClone(value)])) as any;
 }