@mintjamsinc/ichigojs 0.1.53 → 0.1.54

package/dist/ichigo.esm.js

@@ -7904,22 +7904,42 @@ class ReactiveProxy {
                     const nestedPath = path ? (Array.isArray(obj) ? `${path}[${keyStr}]` : `${path}.${keyStr}`) : keyStr;
                     return ReactiveProxy.create(value, onChange, nestedPath);
                 }
-                // For arrays and Maps, intercept mutation methods
+                // 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') {
-                    let mutationMethods = [];
+                    // For arrays, we only want to wrap mutation methods, not read methods like 'slice', 'concat', etc.
                     if (Array.isArray(obj)) {
-                        mutationMethods.push('push', 'pop', 'shift', 'unshift', 'splice', 'sort', 'reverse');
+                        const arrayMutationMethods = ['push', 'pop', 'shift', 'unshift', 'splice', 'sort', 'reverse'];
+                        if (!arrayMutationMethods.includes(key)) {
+                            return value;
+                        }
+                        return function (...args) {
+                            const result = value.apply(this === receiver ? obj : this, args);
+                            onChange(path || undefined);
+                            return result;
+                        };
                     }
-                    else if (obj.constructor.name === 'Map') {
-                        mutationMethods.push('set', 'delete', 'clear');
+                    // 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 function (...args) {
-                        const result = value.apply(this === receiver ? obj : this, args);
-                        if (mutationMethods.includes(key)) {
-                            onChange(path || undefined);
-                        }
-                        return result;
-                    };
                 }
                 return value;
             },