@kudzujs/core 0.17.1 → 0.17.2

package/README.md

@@ -14,7 +14,7 @@ Kudzu compiles ordinary React-shaped TypeScript and TSX into complete static HTM
 
 > Experimental `0.17.x`: the compiler API and supported TSX surface may change.
 
-**Current release: [0.17.1](https://github.com/kudzujs/kudzu/releases/tag/v0.17.1).** Binding lifetime tracking uses one WeakMap instead of duplicate WeakMap/WeakSet membership, preserving state release and explicit DOM remount behavior. C2 saves 52 raw JavaScript bytes (7 gzip); the project application deploy saves 114 raw bytes. Generator `create-kudzu@0.1.162` targets core `^0.17.1`. This is a size and simplification improvement, not a demonstrated speed gain. Restoration latency and AI-cost/1.0 gates remain unresolved.
+**Current release: [0.17.2](https://github.com/kudzujs/kudzu/releases/tag/v0.17.2).** Delayed evaluators cannot regain authority after DOM or source-state release, including same-node remount and state-ID reuse. Keyed conditions now settle correctly when row data and parent selection change in one turn. Native Chrome regressions cover both fixes; static siblings remain JavaScript-free. Generator `create-kudzu@0.1.163` targets core `^0.17.2`. Output sizes have mixed deltas; no speed improvement is claimed. Restoration latency and AI-cost/1.0 gates remain unresolved.
 
 - [Documentation](https://kudzujs.cloud/docs)
 - [Installation guide](https://kudzujs.cloud/docs#install)

package/framework/binding-runtime.js

@@ -5,7 +5,6 @@ import { serializeStyle } from "./style.js"
 const imports = new Map()
 const bindingTargets = new Map()
 const conditionTargets = globalThis.__KUDZU_CONDITIONS__ && new Map()
-const mountedConditions = globalThis.__KUDZU_CONDITIONS__ && new WeakSet()
 const bindingRegistrations = new WeakMap()
 const conditionRegistrations = globalThis.__KUDZU_CONDITIONS__ && new WeakMap()
 const textDescriptors = globalThis.__KUDZU_TEXT_BINDINGS__ && typeof document !== "undefined" ? JSON.parse(document.body.dataset.kTextBindings ?? "[]") : []
@@ -43,7 +42,7 @@ function commitBindings(id) {
   if (bindings) {
     for (const binding of bindings) {
       if (!binding.node.isConnected) bindings.delete(binding)
-      else patchBinding(binding.node, binding.target, binding.read())
+      else if (binding.read) patchBinding(binding.node, binding.target, binding.read())
     }
   }
   if (!globalThis.__KUDZU_CONDITIONS__) return
@@ -83,15 +82,7 @@ function mountBindings(root) {
         patchBinding(node, target, binding.read())
         continue
       }
-      loadEvaluator(descriptor).then(evaluator => {
-        if (!node.isConnected) return
-        const binding = { node, target, read: evaluator.read }
-        for (const id of evaluator.stateIds) {
-          register(bindingTargets, id, binding)
-          registrations.push([id, binding])
-        }
-        patchBinding(node, target, binding.read())
-      }).catch(error => console.error(error))
+      mountBindingEvaluator(node, target, descriptor, registrations)
     }
   }
   if (globalThis.__KUDZU_TEXT_BINDINGS__) {
@@ -108,47 +99,56 @@ function mountBindings(root) {
         patchBinding(node, "text", binding.read())
         continue
       }
-      loadEvaluator(descriptor).then(evaluator => {
-        if (!node.isConnected) return
-        const binding = { node, target: "text", read: evaluator.read }
-        for (const id of evaluator.stateIds) {
-          register(bindingTargets, id, binding)
-          registrations.push([id, binding])
-        }
-        patchBinding(node, "text", binding.read())
-      }).catch(error => console.error(error))
+      mountBindingEvaluator(node, "text", descriptor, registrations)
     }
   }
 }
 
+function mountBindingEvaluator(node, target, descriptor, registrations) {
+  const binding = { node, target }
+  // Register dependencies before loading so state release also revokes pending work.
+  for (const id of bindingStateIds(descriptor)) {
+    register(bindingTargets, id, binding)
+    registrations.push([id, binding])
+  }
+  loadEvaluator(descriptor).then(evaluator => {
+    if (!node.isConnected || bindingRegistrations.get(node) !== registrations) return
+    binding.read = evaluator.read
+    patchBinding(node, target, binding.read())
+  }).catch(error => console.error(error))
+}
+
 function mountConditions(root) {
   for (const start of matching(root, "template[data-k-if]")) {
-    if (mountedConditions.has(start)) continue
-    mountedConditions.add(start)
+    if (conditionRegistrations.has(start)) continue
+    // Reserve membership while the evaluator loads, before subscriptions exist.
+    conditionRegistrations.set(start, undefined)
     const descriptor = JSON.parse(start.dataset.kIf)
     const end = findEnd(start, descriptor.id)
     const truthy = globalThis.__KUDZU_SVG_CONDITIONS__ && descriptor.svg ? start.dataset.kSvgTrue : start.content.querySelector("template[data-k-true]")
     const falsy = globalThis.__KUDZU_SVG_CONDITIONS__ && descriptor.svg ? start.dataset.kSvgFalse : start.content.querySelector("template[data-k-false]")
     if (!end || !truthy || !falsy) continue
     const condition = { start, end, truthy, falsy, svg: globalThis.__KUDZU_SVG_CONDITIONS__ && descriptor.svg, kind: descriptor.kind, current: conditionKey(descriptor.kind, descriptor.initial), mount: globalThis.__KUDZU_CONDITION_MOUNTS__ !== false && descriptor.mount, owned: globalThis.__KUDZU_CONDITION_STATE__ !== false && descriptor.owned }
+    const registrations = []
+    const registration = { condition, registrations }
+    conditionRegistrations.set(start, registration)
+    for (const id of descriptor.state ? [descriptor.state] : bindingStateIds(descriptor)) {
+      register(conditionTargets, id, condition)
+      registrations.push([id, condition])
+    }
     if (globalThis.__KUDZU_CONDITION_STATE__ !== false) mountConditionStates(condition, Boolean(descriptor.initial), false)
     const mount = evaluator => {
-      if (!start.isConnected) return
+      if (!start.isConnected || conditionRegistrations.get(start) !== registration) return
       condition.read = evaluator.read
-      const registrations = []
-      for (const id of evaluator.stateIds) {
-        register(conditionTargets, id, condition)
-        registrations.push([id, condition])
-      }
-      conditionRegistrations.set(start, { condition, registrations })
       updateCondition(condition)
     }
-    if (descriptor.state) mount({ read: () => browserState.get(descriptor.state), stateIds: [descriptor.state] })
+    if (descriptor.state) mount({ read: () => browserState.get(descriptor.state) })
     else loadEvaluator(descriptor).then(mount).catch(error => console.error(error))
   }
 }
 
 function updateCondition(condition) {
+  if (!condition.read) return
   const value = condition.read()
   const next = conditionKey(condition.kind, value)
   if (next === condition.current) return
@@ -170,7 +170,7 @@ function updateCondition(condition) {
   const select = globalThis.__KUDZU_VALUE_BINDINGS__ !== false && condition.start.closest("select[data-k-bind-value]")
   if (select) {
     for (const binding of new Set((bindingRegistrations.get(select) ?? []).map(([, entry]) => entry))) {
-      patchBinding(binding.node, binding.target, binding.read())
+      if (binding.read) patchBinding(binding.node, binding.target, binding.read())
     }
   }
 }
@@ -194,7 +194,6 @@ function unmountConditions(root) {
     for (const [id, condition] of registration?.registrations ?? []) conditionTargets.get(id)?.delete(condition)
     if (globalThis.__KUDZU_CONDITION_STATE__ !== false && registration?.condition) unmountConditionStates(registration.condition, registration.condition.current === "true")
     conditionRegistrations.delete(start)
-    mountedConditions.delete(start)
   }
 }
 
@@ -210,7 +209,7 @@ function releaseBindings(id) {
     const registration = conditionRegistrations.get(condition.start)
     for (const [conditionId, entry] of registration?.registrations ?? []) conditionTargets.get(conditionId)?.delete(entry)
     if (globalThis.__KUDZU_CONDITION_STATE__ !== false) unmountConditionStates(condition, condition.current === "true")
-    conditionRegistrations.delete(condition.start)
+    conditionRegistrations.set(condition.start, undefined)
   }
   conditionTargets.delete(id)
 }

package/framework/list-runtime.js

@@ -650,13 +650,6 @@ function fillListParts(root, parts, item, revision, index = 0, previous) {
     }
   }
   fillListExpressions(root, parts, item, revision, index)
-  if (__KUDZU_LIST_CONDITIONS__) {
-    for (const [marker, descriptor] of parts.conditions) {
-      evaluate(descriptor, item, index).then(value => {
-        if (revisions.get(root) === revision && marker.isConnected) updateListCondition(marker, descriptor.kind, value, item, index)
-      }).catch(error => console.error(error))
-    }
-  }
 }
 
 function fillListExpressions(root, parts, item, revision, index) {
@@ -676,6 +669,14 @@ function fillListExpressions(root, parts, item, revision, index) {
       }
     }
   }
+  // Parent-state commits invalidate the shared row revision, including pending conditions.
+  if (__KUDZU_LIST_CONDITIONS__) {
+    for (const [marker, descriptor] of parts.conditions) {
+      evaluate(descriptor, item, index).then(value => {
+        if (revisions.get(root) === revision && marker.isConnected) updateListCondition(marker, descriptor.kind, value, item, index)
+      }).catch(error => console.error(error))
+    }
+  }
 }
 
 function updateListExpressions(list) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kudzujs/core",
-  "version": "0.17.1",
+  "version": "0.17.2",
   "description": "HTML-first TSX framework with synchronous state semantics and no virtual DOM",
   "type": "module",
   "license": "MIT",