refer to memoization methods by name rather than number

Author: lobsterkatie

packages/utils/src/memo.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/no-unsafe-member-access */
 /* eslint-disable @typescript-eslint/no-explicit-any */
 
-export type MemoFunc = [(obj: any) => boolean, (obj: any) => void];
+export type MemoFunc = { memoize: (obj: any) => boolean; unmemoize: (obj: any) => void };
 
 /**
  * Helper to decycle json objects
@@ -40,5 +40,5 @@ export function memoBuilder(): MemoFunc {
       }
     }
   }
-  return [memoize, unmemoize];
+  return { memoize, unmemoize };
 }

packages/utils/src/object.ts

@@ -338,7 +338,7 @@ export function walk(key: string, value: any, depth: number = +Infinity, memo: M
   const acc: { [key: string]: any } = Array.isArray(value) ? [] : {};
 
   // If we already walked that branch, bail out, as it's circular reference
-  if (memo[0](value)) {
+  if (memo.memoize(value)) {
     return '[Circular ~]';
   }
 
@@ -354,7 +354,7 @@ export function walk(key: string, value: any, depth: number = +Infinity, memo: M
   }
 
   // Once walked through all the branches, remove the parent from memo storage
-  memo[1](value);
+  memo.unmemoize(value);
 
   // Return accumulated values
   return acc;