feat(core)!: Remove memoBuilder export & WeakSet fallback

docs/migration/v8-to-v9.md

@@ -132,6 +132,7 @@ Sentry.init({
 - The `urlEncode` method has been removed. There is no replacement.
 - The `getDomElement` method has been removed. There is no replacement.
 - The `Request` type has been removed. Use `RequestEventData` type instead.
+- The `memoBuilder` method has been removed. There is no replacement.
 
 ### `@sentry/browser`
 

packages/core/src/utils-hoist/index.ts

@@ -38,8 +38,6 @@ export {
 } from './is';
 export { isBrowser } from './isBrowser';
 export { CONSOLE_LEVELS, consoleSandbox, logger, originalConsoleMethods } from './logger';
-// eslint-disable-next-line deprecation/deprecation
-export { memoBuilder } from './memo';
 export {
   addContextToFrame,
   addExceptionMechanism,

packages/core/src/utils-hoist/normalize.ts

@@ -1,8 +1,6 @@
 import type { Primitive } from '../types-hoist';
 
 import { isSyntheticEvent, isVueViewModel } from './is';
-import type { MemoFunc } from './memo';
-import { memoBuilder } from './memo';
 import { convertToPlainObject } from './object';
 import { getFunctionName } from './stacktrace';
 
@@ -13,6 +11,13 @@ type Prototype = { constructor: (...args: unknown[]) => unknown };
 // might be arrays.
 type ObjOrArray<T> = { [key: string]: T };
 
+type MemoFunc = [
+  // memoize
+  (obj: object) => boolean,
+  // unmemoize
+  (obj: object) => void,
+];
+
 /**
  * Recursively normalizes the given object.
  *
@@ -74,8 +79,7 @@ function visit(
   value: unknown,
   depth: number = +Infinity,
   maxProperties: number = +Infinity,
-  // eslint-disable-next-line deprecation/deprecation
-  memo: MemoFunc = memoBuilder(),
+  memo = memoBuilder(),
 ): Primitive | ObjOrArray<unknown> {
   const [memoize, unmemoize] = memo;
 
@@ -304,3 +308,22 @@ export function normalizeUrlToBase(url: string, basePath: string): string {
       .replace(new RegExp(`(file://)?/*${escapedBase}/*`, 'ig'), 'app:///')
   );
 }
+
+/**
+ * Helper to decycle json objects
+ */
+function memoBuilder(): MemoFunc {
+  const inner = new WeakSet<object>();
+  function memoize(obj: object): boolean {
+    if (inner.has(obj)) {
+      return true;
+    }
+    inner.add(obj);
+    return false;
+  }
+
+  function unmemoize(obj: object): void {
+    inner.delete(obj);
+  }
+  return [memoize, unmemoize];
+}