@mintjamsinc/ichigojs 0.1.52 → 0.1.54

package/dist/ichigo.esm.js

@@ -7885,7 +7885,7 @@ class ReactiveProxy {
         }
         // Create the proxy with path captured in closure
         const proxy = new Proxy(target, {
-            get(obj, key) {
+            get(obj, key, receiver) {
                 const value = Reflect.get(obj, key);
                 // If the value is an object or array, make it reactive too
                 if (typeof value === 'object' && value !== null) {
@@ -7904,16 +7904,42 @@ class ReactiveProxy {
                     const nestedPath = path ? (Array.isArray(obj) ? `${path}[${keyStr}]` : `${path}.${keyStr}`) : keyStr;
                     return ReactiveProxy.create(value, onChange, nestedPath);
                 }
-                // For arrays, intercept mutation methods
-                if (Array.isArray(obj) && typeof value === 'function') {
-                    const arrayMutationMethods = ['push', 'pop', 'shift', 'unshift', 'splice', 'sort', 'reverse'];
-                    if (arrayMutationMethods.includes(key)) {
+                // If the value is a function, we need to wrap it to ensure that any mutations it performs also trigger onChange
+                if (typeof value === 'function') {
+                    // For arrays, we only want to wrap mutation methods, not read methods like 'slice', 'concat', etc.
+                    if (Array.isArray(obj)) {
+                        const arrayMutationMethods = ['push', 'pop', 'shift', 'unshift', 'splice', 'sort', 'reverse'];
+                        if (!arrayMutationMethods.includes(key)) {
+                            return value;
+                        }
                         return function (...args) {
-                            const result = value.apply(obj, args);
+                            const result = value.apply(this === receiver ? obj : this, args);
                             onChange(path || undefined);
                             return result;
                         };
                     }
+                    // For Map, we only want to wrap mutation methods, not read methods like 'get' or 'has'
+                    if (obj.constructor.name === 'Map') {
+                        const mapMutationMethods = ['set', 'delete', 'clear'];
+                        return function (...args) {
+                            const result = value.apply(this === receiver ? obj : this, args);
+                            if (mapMutationMethods.includes(key)) {
+                                onChange(path || undefined);
+                            }
+                            return result;
+                        };
+                    }
+                    // For Set, we only want to wrap mutation methods, not read methods like 'has'
+                    if (obj.constructor.name === 'Set') {
+                        const setMutationMethods = ['add', 'delete', 'clear'];
+                        return function (...args) {
+                            const result = value.apply(this === receiver ? obj : this, args);
+                            if (setMutationMethods.includes(key)) {
+                                onChange(path || undefined);
+                            }
+                            return result;
+                        };
+                    }
                 }
                 return value;
             },