@kudzujs/core 0.17.1 → 0.17.3

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.3](https://github.com/kudzujs/kudzu/releases/tag/v0.17.3).** Calculated lists release row state even while their evaluator module is loading, so explicit remount starts fresh. Initial evaluation also updates visible row text from the latest source data. Native Chrome regressions cover both fixes; static siblings remain JavaScript-free. Generator `create-kudzu@0.1.164` targets core `^0.17.3`. The fixes add a small measured byte cost; 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

@@ -97,6 +97,7 @@ function mountLists(root) {
       ownedLists.set(list.owner, lists)
       listRegistrations.set(start, { list, owner: list.owner })
     } else if (descriptor.source) {
+      listRegistrations.set(start, { states: [], list })
       if (descriptor.children) updateList(list)
       const load = {}
       listLoads.set(start, load)
@@ -180,7 +181,7 @@ function updateList(list) {
     keys.add(token)
     entries.push({ item, index, key, token, value: referenceOnly ? item : seededValue === undefined ? JSON.stringify(item) : seededValue })
   }
-  if (list.items === undefined && list.parts.directFill && entries.length === list.roots.size) {
+  if (list.items === undefined && !list.sourceEvaluator && list.parts.directFill && entries.length === list.roots.size) {
     const roots = __KUDZU_LIST_STABLE_FAST_PATHS__ ? list.orderedRoots : [...list.roots.values()]
     if (entries.every((entry, index) => list.roots.get(entry.token) === roots[index])) {
       for (let index = 0; index < entries.length; index++) {
@@ -650,13 +651,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 +670,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.3",
   "description": "HTML-first TSX framework with synchronous state semantics and no virtual DOM",
   "type": "module",
   "license": "MIT",