mithril-lynx 2.6.0 → 2.6.2

package/test/remove-event.test.ts

@@ -0,0 +1,26 @@
+import { describe, expect, it } from "@rstest/core";
+import { createPatchApplier } from "../src/apply-patch.js";
+import { Op } from "../src/patch-protocol.js";
+
+// R3: Op.RemoveEvent must actually remove the native listener, not silently
+// no-op — otherwise a remove→re-add cycle on a kept element accumulates
+// duplicate native listeners and the app's handler fires N times per event
+// (see informe-contrato-mithril-lynx.md §R3).
+describe("Op.RemoveEvent removes the native listener (no duplicate-fire leak)", () => {
+	it("AddEvent → RemoveEvent → AddEvent leaves exactly one listener", () => {
+		lynxTestingEnv.switchToMainThread();
+		const pageId = __GetElementUniqueID(__CreatePage());
+		const applier = createPatchApplier(pageId);
+		applier.registerPageRoot(__CreateView(pageId));
+
+		applier.applyPatch([Op.CreateElement, "view", 1, Op.AddEvent, 1, "tap"]);
+		const handle = applier.getHandle(1) as any;
+		expect(handle.__vanillaListeners.tap.size).toBe(1);
+
+		applier.applyPatch([Op.RemoveEvent, 1, "tap"]);
+		expect(handle.__vanillaListeners.tap.size).toBe(0);
+
+		applier.applyPatch([Op.AddEvent, 1, "tap"]);
+		expect(handle.__vanillaListeners.tap.size).toBe(1);
+	});
+});

package/test/route-hot-reload.test.ts

@@ -15,7 +15,8 @@ describe("route.js + stable-host: a same-path re-resolve patches in place, never
 		lynxTestingEnv.switchToMainThread();
 		const capturedOps: unknown[][] = [];
 		lynx.getJSContext().addEventListener("MithrilLynx:Patch", (event: any) => {
-			capturedOps.push(event.data);
+			// event.data is [PROTOCOL_VERSION, ...ops] — drop the version prefix.
+			capturedOps.push(event.data.slice(1));
 		});
 
 		lynxTestingEnv.switchToBackgroundThread();

package/test/route.test.ts

@@ -22,9 +22,12 @@ function setupRealTree() {
 	// tree inspected.
 	lynxTestingEnv.switchToMainThread();
 	lynx.getJSContext().addEventListener("MithrilLynx:Patch", (event: any) => {
-		capturedOps.push(event.data);
+		// event.data is [PROTOCOL_VERSION, ...ops] — mirror main-thread.js's
+		// own onPatch by stripping the version prefix before applying.
+		const ops = event.data.slice(1);
+		capturedOps.push(ops);
 		lynxTestingEnv.switchToMainThread();
-		applier.applyPatch(event.data);
+		applier.applyPatch(ops);
 	});
 
 	return { applier, capturedOps };

package/test/setup.ts

@@ -4,6 +4,16 @@
 // for everyone else (src/testing.js's installTestingPolyfills) — see that
 // file's header for what it covers and why.
 
+import { afterEach } from "@rstest/core";
 import { installTestingPolyfills } from "../src/testing.js";
+import { unregister } from "../src/mount-redraw.js";
 
 globalThis.onInjectMainThreadGlobals = installTestingPolyfills;
+
+// mount-redraw.js is a single-slot singleton and register() now fails fast
+// on a second registration (R2) — clear it between tests so each test's own
+// renderApp() starts from a clean slate instead of colliding with the
+// previous test's still-registered redraw.
+afterEach(() => {
+	unregister();
+});

package/.omo/run-continuation/ses_f48265d07ffenKG0GAz9ZRFvY6.json

@@ -1,10 +0,0 @@
-{
-  "sessionID": "ses_f48265d07ffenKG0GAz9ZRFvY6",
-  "updatedAt": "2026-09-19T04:26:35.138Z",
-  "sources": {
-    "background-task": {
-      "state": "idle",
-      "updatedAt": "2026-09-19T04:26:35.138Z"
-    }
-  }
-}