@mintjamsinc/ichigojs 0.1.53 → 0.1.54

package/dist/ichigo.cjs

@@ -7910,22 +7910,42 @@
                         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;
                 },