fix(memoryLeak): disable cache for transitjs (#993)

* fix(memoryLeak): disable cache for transitjs

* chore(pr): pr comments addressed

---------

Co-authored-by: Jonny Adshead <JAdshead@users.noreply.github.com>

src/universal/utils/transit.js

@@ -18,7 +18,8 @@
 // the browser or on the server
 /* eslint-disable global-require */
 
-import transit from 'transit-immutable-js';
+import instance, { handlers } from 'transit-immutable-js';
+import transit from 'transit-js';
 import { serializeError } from 'serialize-error';
 
 const concealOrigin = (href) => href && href.replace(/\/\/[^/]+/g, '//***');
@@ -27,15 +28,14 @@ export function writeError(value) {
   const error = serializeError(value);
   delete error.stack;
   error.message = concealOrigin(error.message);
-
   if (error.response) {
     error.response.url = concealOrigin(error.response.url);
   }
 
   return error;
 }
 
-export default transit.withExtraHandlers([
+const extraHandlers = [
   {
     tag: 'error',
     class: Error,
@@ -57,4 +57,33 @@ export default transit.withExtraHandlers([
       : require('url').parse(value)
     ),
   },
-]);
+];
+
+const modifiedHandlers = handlers.withExtraHandlers(extraHandlers);
+
+const reader = transit.reader('json', {
+  handlers: modifiedHandlers.read,
+  cache: false,
+  // Taken from transit-immutable-js. Without this, Error fails to read.
+  // https://github.com/glenjamin/transit-immutable-js/blob/cb0ac0799d730080ea2403dba4061cf9c9d7b9bd/index.js#L6
+  mapBuilder: {
+    init() {
+      return {};
+    },
+    add(m, k, v) {
+      // eslint-disable-next-line no-param-reassign -- Keeping what is done by transit-immutable-js
+      m[k] = v;
+      return m;
+    },
+    finalize(m) {
+      return m;
+    },
+  },
+});
+const writer = transit.writer('json', { handlers: modifiedHandlers.write, cache: false });
+
+export default {
+  ...instance.withExtraHandlers(extraHandlers), // added for test cases
+  toJSON: (data) => writer.write(data),
+  fromJSON: (json) => reader.read(json),
+};