@mintjamsinc/ichigojs 0.1.52 → 0.1.54

package/dist/ichigo.cjs

@@ -7891,7 +7891,7 @@
             }
             // 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) {
@@ -7910,16 +7910,42 @@
                         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;
                 },