mithril-lynx 0.0.3 → 0.0.5

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mithril-lynx",
-  "version": "0.0.3",
-  "description": "Mithril.js rendered through Lynx's Element PAPI — a contract-complete port of mithril/render/render.js@2.3.8 to the Lynx main thread, packaged as a reusable framework.",
+  "version": "0.0.5",
+  "description": "Mithril.js rendered through Lynx's Element PAPI \u2014 a contract-complete port of mithril/render/render.js@2.3.8 to the Lynx main thread, packaged as a reusable framework.",
   "license": "MIT",
   "type": "module",
   "repository": {

package/plugin.js

@@ -35,6 +35,28 @@ function findSibling(dir, candidates) {
 	return null;
 }
 
+/**
+ * Walks up from a resolved file to the root of the package that owns it,
+ * verified by name rather than assumed from the layout — this package's
+ * exports map deliberately doesn't expose ./package.json, so the usual
+ * require.resolve("<pkg>/package.json") trick isn't available here.
+ */
+function packageRootOf(resolvedFile) {
+	let dir = path.dirname(resolvedFile);
+	for (let i = 0; i < 10; i++) {
+		try {
+			const pkg = JSON.parse(fs.readFileSync(path.join(dir, "package.json"), "utf8"));
+			if (pkg.name === "mithril-lynx") return dir;
+		} catch {
+			// keep walking
+		}
+		const parent = path.dirname(dir);
+		if (parent === dir) break;
+		dir = parent;
+	}
+	return null;
+}
+
 export function pluginMithrilLynx(options = {}) {
 	const targetSdkVersion = options.targetSdkVersion ?? "3.5";
 
@@ -69,6 +91,34 @@ export function pluginMithrilLynx(options = {}) {
 					// leave resolution as-is rather than guessing.
 				}
 
+				// Same class of problem, worse symptom: mithril-lynx itself keeps
+				// per-app state in module-level variables — the shim's rootWrapper/
+				// redraw/runRender, main-thread.js's latestData and its cross-thread
+				// handler maps, background.js's mirror of those. Two physical copies
+				// means two disconnected renderers: the app renders through one, and
+				// any LIBRARY that depends on mithril-lynx (a component library, say,
+				// resolving its own nested copy once linked) calls shim.redraw() on
+				// the other — whose `redraw` is still null. That's a silent no-op:
+				// no error, nothing logged, components simply never update. Confirmed
+				// on real hardware 2026-09-11 while building mithril-lynx-ui, where
+				// it read as "the animation just doesn't run".
+				try {
+					const appRequire = createRequire(path.join(process.cwd(), "package.json"));
+					const selfDir = packageRootOf(appRequire.resolve("mithril-lynx"));
+					if (selfDir != null) {
+						// The bare specifier has to point at the entry FILE: aliasing it
+						// to the directory would bypass this package's own exports map
+						// (which has no "main" to fall back on) and fail to resolve.
+						// The prefix alias then keeps subpaths — "mithril-lynx/main-thread"
+						// and friends, which hold state of their own — on that same copy.
+						chain.resolve.alias.set("mithril-lynx$", path.join(selfDir, "src", "lynx-mithril-shim.js"));
+						chain.resolve.alias.set("mithril-lynx", selfDir);
+					}
+				} catch {
+					// App doesn't resolve mithril-lynx from its own root (it's being
+					// consumed some other way) — leave resolution alone.
+				}
+
 				const rawEntries = Object.entries(chain.entryPoints.entries() ?? {});
 				chain.entryPoints.clear();
 

package/src/lynx-mithril-shim.js

@@ -696,23 +696,41 @@ function factory() {
 
 	function initComponent(vnode, hooks) {
 		var sentinel
-		if (typeof vnode.tag.view === "function") {
+		var classStyle = typeof vnode.tag.view !== "function"
+		if (!classStyle) {
 			vnode.state = Object.create(vnode.tag)
 			sentinel = vnode.state.view
-			if (sentinel.$$reentrantLock$$ != null) return
-			sentinel.$$reentrantLock$$ = true
 		} else {
-			vnode.state = void 0
 			sentinel = vnode.tag
-			if (sentinel.$$reentrantLock$$ != null) return
-			sentinel.$$reentrantLock$$ = true
-			vnode.state = (vnode.tag.prototype != null && typeof vnode.tag.prototype.view === "function") ? new vnode.tag(vnode) : vnode.tag(vnode)
 		}
-		initLifecycle(vnode.state, vnode, hooks)
-		if (vnode.attrs != null) initLifecycle(vnode.attrs, vnode, hooks)
-		vnode.instance = Vnode.normalize(callHook.call(vnode.state.view, vnode))
-		if (vnode.instance === vnode) throw Error("A view cannot return the vnode it received as argument")
-		sentinel.$$reentrantLock$$ = null
+		if (sentinel.$$reentrantLock$$ != null) return
+		sentinel.$$reentrantLock$$ = true
+		// The lock above is stored on `sentinel` — the component's OWN shared
+		// view function (or, for class-style components, the tag itself), NOT
+		// on this vnode instance — so it's shared by EVERY use of this
+		// component anywhere in the app. Without the try/finally below, a
+		// view() (or, for a class-style component, its CONSTRUCTOR) that
+		// throws on a component's first render (e.g. a fail-loudly validation
+		// error) skips the `sentinel.$$reentrantLock$$ = null` reset entirely,
+		// leaving that lock stuck at `true` forever. Every later attempt to
+		// create that SAME component type — anywhere, not just the one
+		// instance that threw — then silently no-ops right here
+		// (`if (sentinel.$$reentrantLock$$ != null) return`): no render, no
+		// error, nothing. Found via mithril-lynx-ui's own test suite: a
+		// component whose view() deliberately throws to validate a prop,
+		// exercised in one test, made every SUBSEQUENT test mounting that
+		// same component silently render nothing, no matter how unrelated.
+		try {
+			if (classStyle) {
+				vnode.state = (vnode.tag.prototype != null && typeof vnode.tag.prototype.view === "function") ? new vnode.tag(vnode) : vnode.tag(vnode)
+			}
+			initLifecycle(vnode.state, vnode, hooks)
+			if (vnode.attrs != null) initLifecycle(vnode.attrs, vnode, hooks)
+			vnode.instance = Vnode.normalize(callHook.call(vnode.state.view, vnode))
+			if (vnode.instance === vnode) throw Error("A view cannot return the vnode it received as argument")
+		} finally {
+			sentinel.$$reentrantLock$$ = null
+		}
 	}
 
 	function createComponent(parent, vnode, hooks, ns, nextSibling) {