fix a bug where method calls of class instances wouldn't synchronise

Author: drinking-code

src/__test__/runCreateFieldsTest.ts

@@ -30,9 +30,13 @@ export default function createFieldsTest(
                     }
                 },
                 (ipos_for_probing) => {
+                    // get originals of if is proxy
+                    const fieldByProperty = (ipos_for_probing.myField as any).__original ?? ipos_for_probing.myField
+                    const fieldByFunction = (ipos_for_probing.get('myField') as any).__original ?? ipos_for_probing.get('myField')
+
                     // lodash equal to compare maps and sets
-                    expect(_.isEqualWith(ipos_for_probing.myField, value, customizer)).toEqual(true)
-                    expect(_.isEqualWith(ipos_for_probing.get('myField'), value, customizer)).toEqual(true)
+                    expect(_.isEqualWith(fieldByProperty, value, customizer)).toEqual(true)
+                    expect(_.isEqualWith(fieldByFunction, value, customizer)).toEqual(true)
                 }
             )
         })

src/intercept.ts

@@ -1,3 +1,5 @@
+import {classes as registeredClassesMap} from './serialize.js'
+
 export default function intercept<V>(value: V, key: string, interceptCallback: (key: string, method: string, ...args: any) => void): V {
     const arrayMutatingMethods = ['copyWithin', 'fill', 'pop', 'push', 'reverse', 'shift', 'sort', 'splice', 'unshift']
     const objectMutatingMethods: string[] = []
@@ -11,12 +13,18 @@ export default function intercept<V>(value: V, key: string, interceptCallback: (
     mutatingMethods.set(Set, setMutatingMethods)
     mutatingMethods.set(Function, functionMutatingMethods)
 
-    if (!value || typeof value !== 'object' || !mutatingMethods.has(value.constructor))
+    const registeredClasses = Array.from(registeredClassesMap.values())
+
+    if (!value || typeof value !== 'object' || !(mutatingMethods.has(value.constructor) || registeredClasses.includes(value.constructor)))
         return value
 
     return new Proxy(value, {
         get(target, name: string) {
-            if (Reflect.has(target, name) && mutatingMethods.get(value.constructor).includes(name)) {
+            if (name === '__proxy')
+                return true
+            if (name === '__original')
+                return value
+            if (Reflect.has(target, name) && (mutatingMethods.get(value.constructor)?.includes(name) || registeredClasses.includes(value.constructor))) {
                 const method = Reflect.get(target, name)
                 return (...args: any) => {
                     interceptCallback(key, name, ...args)

src/serialize.ts

@@ -32,7 +32,7 @@ export function serialize(value: any): any | void {
         return value
     } else if (Array.isArray(value)) {
         return value.map(v => serialize(v))
-    } else if (value.constructor === {}.constructor || value.valueOf().constructor === {}.constructor) {
+    } else if (value.constructor === {}.constructor || value.valueOf()?.constructor === {}.constructor) {
         return Object.fromEntries(
             Array.from(Object.entries(value))
                 .map(([key, value]) => [key, serialize(value)])
@@ -59,6 +59,7 @@ export function serialize(value: any): any | void {
     } else if (isNativeObject(value)) {
         throw new Error(`Could not serialise: \`${value.constructor.name}\`.`)
     } else {
+        value = value.__original ?? value
         const serializeMethod = deSerialize.get(value.constructor)?.serialize ?? value.stringify ?? value.serialize
         if (!serializeMethod)
             throw new Error(