Refactor src/core/json-utils.ts to replace any with unknown and improve type safety. (#206)

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

Author: zknpr

src/core/json-utils.ts

@@ -9,7 +9,7 @@
 
 const MAX_DEPTH = 1000;
 
-export function generateMergePatch(original: any, modified: any, depth = 0): any {
+export function generateMergePatch(original: unknown, modified: unknown, depth = 0): unknown {
     if (depth > MAX_DEPTH) {
         throw new Error('JSON merge patch depth limit exceeded');
     }
@@ -27,7 +27,7 @@ export function generateMergePatch(original: any, modified: any, depth = 0): any
         return modified;
     }
 
-    const patch: Record<string, any> = {};
+    const patch: Record<string, unknown> = {};
     let hasChanges = false;
 
     // Check for modifications and additions
@@ -67,7 +67,7 @@ export function generateMergePatch(original: any, modified: any, depth = 0): any
  * @param patch - The patch to apply
  * @returns The modified object (new instance or mutated)
  */
-export function applyMergePatch(target: any, patch: any, depth = 0): any {
+export function applyMergePatch(target: unknown, patch: unknown, depth = 0): unknown {
     if (depth > MAX_DEPTH) {
         throw new Error('JSON apply merge patch depth limit exceeded');
     }
@@ -83,27 +83,29 @@ export function applyMergePatch(target: any, patch: any, depth = 0): any {
         return patch;
     }
 
+    let targetObj: Record<string, unknown>;
     if (typeof target !== 'object' || target === null || Array.isArray(target)) {
         // If target is not an object (or is null/array), it is treated as empty object for patching.
-        target = {};
+        targetObj = {};
     } else {
         // Clone target to avoid mutation if we want immutability,
         // Clone shallowly for safety.
-        target = { ...target };
+        targetObj = { ...(target as Record<string, unknown>) };
     }
 
-    for (const key of Object.keys(patch)) {
-        const val = patch[key];
+    const patchObj = patch as Record<string, unknown>;
+    for (const key of Object.keys(patchObj)) {
+        const val = patchObj[key];
         if (val === null) {
-            delete target[key];
+            delete targetObj[key];
         } else {
-            target[key] = applyMergePatch(target[key], val, depth + 1);
+            targetObj[key] = applyMergePatch(targetObj[key], val, depth + 1);
         }
     }
 
-    return target;
+    return targetObj;
 }
 
-function isObject(val: any): boolean {
+function isObject(val: unknown): val is Record<string, unknown> {
     return val !== null && typeof val === 'object';
 }