Fixed incorrect changed path when mutating unrelated area of proxy during iteration operations on cloned areas of the proxy.

index.js

@@ -48,7 +48,7 @@ const onChange = (object, onChange, options = {}) => {
 
 	// eslint-disable-next-line max-params
 	const handleChange = (changePath, property, value, previous, applyData) => {
-		if (smartClone.isCloning) {
+		if (smartClone.isCloning && smartClone.isPartOfClone(changePath)) {
 			smartClone.update(changePath, property, previous);
 		} else {
 			onChange(path.concat(changePath, property), value, previous, applyData);

lib/path.js

@@ -107,6 +107,43 @@ const path = {
 
 		return object;
 	},
+	isSubPath(path, subPath) {
+		if (isArray(path)) {
+			if (path.length < subPath.length) {
+				return false;
+			}
+
+			// eslint-disable-next-line unicorn/no-for-loop
+			for (let i = 0; i < subPath.length; i++) {
+				if (path[i] !== subPath[i]) {
+					return false;
+				}
+			}
+
+			return true;
+		}
+
+		if (path.length < subPath.length) {
+			return false;
+		}
+
+		if (path === subPath) {
+			return true;
+		}
+
+		if (path.startsWith(subPath)) {
+			return path[subPath.length] === PATH_SEPARATOR;
+		}
+
+		return false;
+	},
+	isRootPath(path) {
+		if (isArray(path)) {
+			return path.length === 0;
+		}
+
+		return path === '';
+	},
 };
 
 export default path;

lib/smart-clone/clone/clone-object.js

@@ -108,4 +108,8 @@ export default class CloneObject {
 			? this._isChanged
 			: this._onIsChanged(this.clone, value);
 	}
+
+	isPathApplicable(changePath) {
+		return path.isRootPath(this._path) || path.isSubPath(changePath, this._path);
+	}
 }

lib/smart-clone/smart-clone.js

@@ -81,6 +81,10 @@ export default class SmartClone {
 		return this._stack.at(-1).isChanged(isMutable, value, equals);
 	}
 
+	isPartOfClone(changePath) {
+		return this._stack.at(-1).isPathApplicable(changePath);
+	}
+
 	undo(object) {
 		if (this._previousClone !== undefined) {
 			this._previousClone.undo(object);

tests/on-change.test.js

@@ -828,3 +828,55 @@ test('array path should be the shorter one in the same object for circular refer
 	t.is(resultPath[1], '2');
 	t.is(resultPath[2], 'value');
 });
+
+test('should trigger when methods are called that mutate unrelated area of proxy when pathAsArray is false', t => {
+	const object = {
+		a: [
+			{
+				quantity: 1,
+			},
+		],
+		b: {
+			c: {
+				quantity: 8,
+			},
+		},
+	};
+
+	testRunner(t, object, {pathAsArray: false}, (proxy, verify) => {
+		// eslint-disable-next-line unicorn/no-array-for-each
+		proxy.a.forEach(() => {
+			proxy.b.c = {
+				quantity: 3,
+			};
+		});
+
+		verify(1, proxy, 'b.c', {quantity: 3}, {quantity: 8});
+	});
+});
+
+test('should trigger when methods are called that mutate unrelated area of proxy when pathAsArray is true', t => {
+	const object = {
+		a: [
+			{
+				quantity: 1,
+			},
+		],
+		b: {
+			c: {
+				quantity: 8,
+			},
+		},
+	};
+
+	testRunner(t, object, {pathAsArray: true}, (proxy, verify) => {
+		// eslint-disable-next-line unicorn/no-array-for-each
+		proxy.a.forEach(() => {
+			proxy.b.c = {
+				quantity: 3,
+			};
+		});
+
+		verify(1, proxy, ['b', 'c'], {quantity: 3}, {quantity: 8});
+	});
+});