fix(overmind): safely serialize arrays with unsafe classes

Author: christianalfoni

packages/node_modules/overmind/src/Devtools.ts

@@ -11,6 +11,16 @@ export type DevtoolsMessage = {
   data: any
 }
 
+function isClass(value: any) {
+  return (
+    typeof value === 'object' &&
+    !Array.isArray(value) && 
+    value !== null &&
+    value.constructor &&
+    value.constructor.name !== 'Object'
+  )
+}
+
 export class Devtools {
   private safeClassNames: Set<string> = new Set()
   private unsafeClassNames: Set<string> = new Set()
@@ -90,13 +100,26 @@ export class Devtools {
           }
         }
 
-        if (
-          typeof value === 'object' &&
-          value !== null &&
-          !Array.isArray(value) &&
-          value.constructor &&
-          value.constructor.name !== 'Object'
-        ) {
+        if (Array.isArray(value) && isClass(value[0])) {
+          if (!safeClassNames.has(value[0].constructor.name) && !unsafeClassNames.has(value[0].constructor.name)) {
+            try {
+              JSON.stringify(value[0])
+              safeClassNames.add(value[0].constructor.name)
+            } catch {
+              unsafeClassNames.add(value[0].constructor.name)
+            }
+          }
+          
+          if (safeClassNames.has(value[0].constructor.name)) {
+            return value.map((item) => ({
+              __CLASS__: true,
+              name: value[0].constructor.name,
+              value: item
+            }))
+          } else {
+            return value.map(() => `[${value.constructor.name || 'NOT SERIALIZABLE'}]`)
+          }
+        } else if (isClass(value)) {
           if (!safeClassNames.has(value.constructor.name) && !unsafeClassNames.has(value.constructor.name)) {
             try {
               JSON.stringify(value)