fix useimmutable

src/components/UseImmutableTest.tsx

@@ -2,17 +2,18 @@ import useImmutable from './useImmutable';
 
 export default function UseImmutableTest() {
 	const [imm] = useImmutable({
-		x: 0,
+		index: 0,
-		y: 0,
+		array: [{ count: 0 }, { count: 1 }, { count: 2 }],
-		z: { a: 1, b: 2, c: [0, 1, 2] },
 	});
 
 	return (
 		<div>
 			{JSON.stringify(imm)}
 			<br />
-			<button onClick={() => imm.z.a++}>Increment</button>
+			<button onClick={() => imm.index--}>Previous</button>
-			<button onClick={() => imm.z.c.push(imm.z.c.length)}>Push</button>
+			<button onClick={() => imm.index++}>Next</button>
+			<button onClick={() => imm.array[imm.index].count++}>Increment</button>
+			<button onClick={() => imm.array[imm.index].count--}>Decrement</button>
 		</div>
 	);
 }

src/components/useImmutable.ts

@@ -66,27 +66,41 @@ function createEdgeForArray<T extends PlainJS>(
 	value: PlainJSArray<T>,
 	setValue: Dispatch<SetStateAction<PlainJSArray<T>>>
 ) {
+	const edges = {} as Record<number, T>;
+
+	const set = (property: number, next: SetStateAction<T>) => {
+		const current = value[property];
+		const nextValue = typeof next === 'function' ? next(current) : next;
+		setValue((value) => [
+			...value.slice(0, property),
+			nextValue,
+			...value.slice(property + 1),
+		]);
+	};
+
 	return new Proxy(value, {
 		set: (target, property, value) => {
 			if (typeof property === 'number') {
-				const set = (next: SetStateAction<T>) => {
+				set(property, value);
-					const v = typeof next === 'function' ? next(value) : next;
-					const edge = createEdge(v, set);
-					setValue((v) => [
-						...v.slice(0, property),
-						edge,
-						...v.slice(property + 1),
-					]);
-				};
-
-				set(value);
 			}
 			return true;
 		},
 		// @ts-expect-error
 		get: (target, property: keyof PlainJSArray<T>[]) => {
-			if (typeof property === 'number') {
+			if (
-				return target[property];
+				typeof property === 'number' ||
+				(typeof property === 'string' && /\d+/.test(property))
+			) {
+				property = +property;
+				if (property in edges) {
+					return edges[property];
+				}
+
+				const item = target[property];
+				const setThis = set.bind(null, property);
+				const edge = createEdge(item, setThis);
+				edges[property] = edge;
+				return edge;
 			} else {
 				// @ts-ignore
 				if (inPlaceArrayOperations.includes(property)) {
@@ -133,7 +147,5 @@ export default function useImmutable<T extends PlainJS>(
 
 	const edge = useMemo(() => createEdge(value, setValue), [value]);
 
-	console.log('rerendered useImmutable');
-
 	return [edge, setValue];
 }