feat(overmind): optimize stringifying classes

Author: christianalfoni

packages/node_modules/overmind/src/Devtools.ts

@@ -10,6 +10,8 @@ export type DevtoolsMessage = {
 }
 
 export class Devtools {
+  private safeClassNames: Set<string> = new Set()
+  private unsafeClassNames: Set<string> = new Set()
   private buffer: string[] = []
   private ws: WebSocket
   private isConnected: boolean = false
@@ -65,24 +67,39 @@ export class Devtools {
     )
   }
   send(message: Message) {
+    const safeClassNames = this.safeClassNames
+    const unsafeClassNames = this.unsafeClassNames
     const stringifiedMessage = JSON.stringify(message, function (_, value) {
       if (typeof value === 'function') {
         return '[Function]'
       }
       if (this.__CLASS__) {
         return value
       }
-      if (typeof value === 'object' && value !== null && !Array.isArray(value) && value.constructor && value.constructor.name !== 'Object') {
-        try {
-          JSON.stringify(value)
-        } catch {
-          return `[${value.constructor.name || 'NOT SERIALIZABLE'}]`
+      if (
+        typeof value === 'object' &&
+        value !== null &&
+        !Array.isArray(value) &&
+        value.constructor &&
+        value.constructor.name !== 'Object'
+      ) {
+        if (!safeClassNames.has(value.constructor.name) && !unsafeClassNames.has(value.constructor.name)) {
+          try {
+            JSON.stringify(value)
+            safeClassNames.add(value.constructor.name)
+          } catch {
+            unsafeClassNames.add(value.constructor.name)
+          }
         }
-
-        return {
-          __CLASS__: true,
-          name: value.constructor.name,
-          value
+        
+        if (safeClassNames.has(value.constructor.name)) {
+          return {
+            __CLASS__: true,
+            name: value.constructor.name,
+            value
+          }
+        } else {
+          return `[${value.constructor.name || 'NOT SERIALIZABLE'}]`
         }
       }