@sigil-dev/runtime 0.7.6 → 0.8.0

package/index.test.ts

@@ -28,11 +28,11 @@ import {
 	easeOutSine,
 	hydrateKeyedList,
 	linear,
-	reconcile,
-	tick,
 	ReactiveMap,
 	ReactiveSet,
+	reconcile,
 	snapshot,
+	tick,
 	tracking,
 	withEffectScope,
 } from "./index.ts";
@@ -360,9 +360,13 @@ describe("Runtime", () => {
 			count();
 			runs++;
 		});
-		batch(() => { count.set(1); });
+		batch(() => {
+			count.set(1);
+		});
 		stop();
-		batch(() => { count.set(2); });
+		batch(() => {
+			count.set(2);
+		});
 		expect(runs).toBe(2); // initial + one update, not three
 	});
 
@@ -558,9 +562,9 @@ describe("createSignal with arrays", () => {
 	test("index access is reactive", () => {
 		const items = createSignal([1, 2, 3]);
 		let first = 0;
-		//@ts-expect-error the assertion for that is below
 		createEffect(() => {
-			first = items()[0];
+		//@ts-expect-error the assertion for that is below
+            first = items()[0];
 		});
 		expect(first).toBe(1);
 		batch(() => {
@@ -1239,11 +1243,14 @@ describe("A2 snapshot", () => {
 		const s = snapshot(obj());
 		expect(s.a.b.c).toBe(1);
 		// s.a.b should NOT be a proxy — it's a plain object from the deep copy
+		//biome-ignore lint: shut up
 		const proxyTargetCheck = (val: object) =>
-			(typeof val === "object" && val !== null) &&
+			typeof val === "object" &&
+			val !== null &&
 			!Object.getPrototypeOf(val)?.constructor?.name?.includes("Map");
 		// Verify no nested proxy: accessing s.a.b.c should not trigger tracking
-		let tracked = false;
+        let tracked = false;
+		//biome-ignore lint: shut up
 		const prev = (globalThis as any).__sigil_tracking_test;
 		(globalThis as any).__sigil_tracking_test = () => (tracked = true);
 		const raw = s.a.b;
@@ -1288,7 +1295,10 @@ describe("A4 createInspectEffect", () => {
 	test("accepts custom callback", () => {
 		const count = createSignal(0);
 		const logged: unknown[][] = [];
-		createInspectEffect(() => [count()], (vals) => logged.push(vals));
+		createInspectEffect(
+			() => [count()],
+			(vals) => logged.push(vals),
+		);
 		expect(logged.length).toBe(1);
 		expect(logged[0]).toEqual([0]);
 	});
@@ -1369,7 +1379,8 @@ describe("A7 ReactiveMap", () => {
 		map.set("a", 1);
 		let count = 0;
 		createEffect(() => {
-			count = 0;
+            count = 0;
+            //biome-ignore lint: shut up
 			map.forEach(() => count++);
 		});
 		expect(count).toBe(1);
@@ -1485,7 +1496,8 @@ describe("A7 ReactiveSet", () => {
 		set.add("a");
 		let count = 0;
 		createEffect(() => {
-			count = 0;
+            count = 0;
+			//biome-ignore lint: shut up
 			set.forEach(() => count++);
 		});
 		expect(count).toBe(1);
@@ -1516,11 +1528,21 @@ describe("A8 easing functions", () => {
 
 	test("all easing functions return 0 at t=0 and 1 at t=1", () => {
 		const fns = [
-			easeInQuad, easeOutQuad, easeInOutQuad,
-			easeInCubic, easeOutCubic, easeInOutCubic,
-			easeInQuart, easeOutQuart, easeInOutQuart,
-			easeInQuint, easeOutQuint, easeInOutQuint,
-			easeInSine, easeOutSine, easeInOutSine,
+			easeInQuad,
+			easeOutQuad,
+			easeInOutQuad,
+			easeInCubic,
+			easeOutCubic,
+			easeInOutCubic,
+			easeInQuart,
+			easeOutQuart,
+			easeInOutQuart,
+			easeInQuint,
+			easeOutQuint,
+			easeInOutQuint,
+			easeInSine,
+			easeOutSine,
+			easeInOutSine,
 		];
 		for (const fn of fns) {
 			expect(fn(0)).toBeCloseTo(0, 10);
@@ -1551,3 +1573,44 @@ describe("A9 createProfilerEffect", () => {
 		expect(seen).toBe(5);
 	});
 });
+
+describe("Array reactivity", () => {
+    test("push on $state array triggers reactivity", async () => {
+        const items = createSignal([] as string[]);
+        let count = 0;
+        createEffect(() => {
+            items(); // now auto-tracks length
+            count++;
+        });
+        await tick();
+        expect(count).toBe(1); // initial run
+        items().push("a");
+        await tick();
+        expect(count).toBe(2); // re-ran after push
+        expect(items().length).toBe(1);
+    });
+
+    test("pop on $state array triggers reactivity", async () => {
+        const items = createSignal(["a", "b"] as string[]);
+        let len = 0;
+        createEffect(() => {
+            len = items().length; // reads length, subscribes
+        });
+        await tick();
+        expect(len).toBe(2);
+        items().pop();
+        await tick();
+        expect(len).toBe(1);
+    });
+
+    test("splice on $state array triggers reactivity", async () => {
+        const items = createSignal(["a", "b", "c"] as string[]);
+        let len = 0;
+        createEffect(() => { len = items().length; });
+        await tick();
+        expect(len).toBe(3);
+        items().splice(1, 1);
+        await tick();
+        expect(len).toBe(2);
+    });
+})