@blueshed/railroad 0.3.2 → 0.3.4

package/.claude/skills/railroad/SKILL.md

@@ -89,3 +89,6 @@ loggedRequest(tag, handler): Handler // wrap route handler with access logging
 3. **No `text()` for attributes.** `text()` creates a DOM node. Use `computed()` for reactive attributes.
 4. **No JSX in effects without dispose scopes.** Any effect that rebuilds DOM must use `pushDisposeScope`/`popDisposeScope` and return a cleanup function. See `jsx.ts` source for the pattern.
 5. **No `transition-all` in CSS** near layout boundaries. Use specific properties.
+6. **No bare nested `when()`.** `when()` returns a fragment — nesting fragments inside another `when()` breaks dispose scope tracking. Always wrap an inner `when()` in a real element: `<div>{when(...)}</div>`.
+7. **No shared DOM nodes across `when()` branches.** Nodes must be created fresh inside each branch function. A node created outside and reused across branches will be torn out of the DOM when the other branch activates.
+8. **Guard against null inside `when()` branches.** Signal cascade order is not guaranteed — an inner `when()` can fire before the outer `when()` swaps it away. Always null-check even inside a branch that "shouldn't" be reached (e.g. `text(() => item.get()?.name ?? "")`).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blueshed/railroad",
-  "version": "0.3.2",
+  "version": "0.3.4",
   "description": "Signals, JSX, and routes — a micro UI framework for Bun",
   "type": "module",
   "main": "index.ts",

package/signals.ts

@@ -165,7 +165,19 @@ export function trackDispose(d: Dispose): void {
 // === computed() ===
 
 export function computed<T>(fn: () => T): Signal<T> {
-  const s = new Signal<T>(fn());
+  // Suspend outer tracking during initial evaluation so computed()
+  // inside list() render functions doesn't leak subscriptions to the
+  // list effect's listener — which causes infinite sync re-entry.
+  const prevListener = currentListener;
+  const prevDeps = currentDeps;
+  currentListener = null;
+  currentDeps = null;
+  let initial: T;
+  try { initial = fn(); } finally {
+    currentListener = prevListener;
+    currentDeps = prevDeps;
+  }
+  const s = new Signal<T>(initial);
   const dispose = effect(() => s.set(fn()));
   trackDispose(dispose);
   return s;