@abide/abide 0.105.0 → 0.105.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @abide/abide
 
+## 0.105.1
+
+### Patch Changes
+
+- fdfe14c: an emptied list clears its region in one write ([`b190cd1`](https://github.com/briancray/abide/commit/b190cd19cd5cf0bda5797a6308eddb5ada1af13b))
+
 ## 0.105.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abide/abide",
-    "version": "0.105.0",
+    "version": "0.105.1",
     "type": "module",
     "description": "Isomorphic type-safe framework for reactive async interfaces for humans and machines",
     "license": "MIT",

package/src/ui/leave.ts

@@ -23,6 +23,11 @@ export function holdWhileLeaving(
     settling.then(done, done)
 }
 
+/** Whether any element is leaving, so a removal has to ask each node whether it holds one. */
+export function anyLeaving(): boolean {
+    return LEAVING.size !== 0
+}
+
 /** Removes `node`, or holds it where it is while an element it holds is leaving. */
 export function leave(node: ChildNode): void {
     if (LEAVING.size !== 0)

package/src/ui/reconcile.ts

@@ -3,6 +3,7 @@ import { LAYER_RECONCILE } from './LAYER_NAMES.ts'
 import type { List } from './List.ts'
 import { HYDRATE_LOG } from './Mismatch.ts'
 import { beginLayer, endLayer } from './measureLayer.ts'
+import { emptyRegion } from './regions.ts'
 import { dropRow, makeRow, moveRow, type Row } from './rows.ts'
 
 // 41.8 — a KEYED list, reconciled by moving the nodes that are already there rather than by
@@ -21,6 +22,10 @@ import { dropRow, makeRow, moveRow, type Row } from './rows.ts'
  * survive are MOVED, never remade.
  */
 export function reconcile(list: List, data: unknown[]): void {
+    if (data.length === 0) {
+        clear(list)
+        return
+    }
     const region = list.region
     const current = list.rows
     const wanted = data.length
@@ -182,6 +187,26 @@ export function reconcile(list: List, data: unknown[]): void {
     }
 }
 
+/**
+ * Drops every row, emptying the region in one write rather than one removal a row.
+ *
+ * NOTHING WANTED IS ITS OWN SHAPE: there is no key to match and no order to keep, so the map,
+ * the survivor walk and the placement walk all answer nothing. Every row is still STOPPED first,
+ * as `dropRow` would, so a binding cannot wake over a node that is already gone.
+ */
+function clear(list: List): void {
+    const parent = list.region.close.parentNode
+    if (parent === null) return
+    beginLayer(LAYER_RECONCILE)
+    try {
+        for (const row of list.rows) row.scope.stop()
+        list.rows = []
+        emptyRegion(list.region, parent)
+    } finally {
+        endLayer(LAYER_RECONCILE)
+    }
+}
+
 /**
  * Whether a patch wrote to the row's item since the row was made.
  *

package/src/ui/regions.ts

@@ -1,6 +1,6 @@
 import { REGION_CLOSE, REGION_OPEN } from '#shared/ANCHORS.ts'
 import { LAYER_FILL } from './LAYER_NAMES.ts'
-import { leave } from './leave.ts'
+import { anyLeaving, leave } from './leave.ts'
 import { Mismatch } from './Mismatch.ts'
 import { beginLayer, endLayer } from './measureLayer.ts'
 
@@ -97,7 +97,7 @@ export function fillRegion(
         throw new Mismatch(
             `abide: this region is not in a document, so the client and the server disagree about this region's shape (41.10).`,
         )
-    removeUntil(region.open.nextSibling, region.close)
+    emptyRegion(region, parent)
     if (build === undefined) return
     beginLayer(LAYER_FILL)
     try {
@@ -126,6 +126,31 @@ export function fillEmpty(
     return true
 }
 
+/**
+ * Removes everything between a region's anchors, which stay.
+ *
+ * ONE WRITE WHERE THE REGION IS ALL ITS PARENT HOLDS — a `<tbody>` or a `<ul>` around one
+ * `{#for}`, the common shape of a list emptied — and the anchors put back after it. A removal per
+ * node is a crossing into the engine per node; the engine's own clear is one.
+ *
+ * NODE BY NODE WHILE ANYTHING IS LEAVING (32.38), because a node holding a leaving element has to
+ * stay where it stands, and only `leave` asks.
+ */
+export function emptyRegion(region: Region, parent: ParentNode): void {
+    const { open, close } = region
+    if (
+        anyLeaving() ||
+        parent.firstChild !== open ||
+        parent.lastChild !== close
+    ) {
+        removeUntil(open.nextSibling, close)
+        return
+    }
+    if (open.nextSibling === close) return
+    parent.textContent = ''
+    parent.append(open, close)
+}
+
 /** Removes `from` and every sibling after it, up to and not including `end`. */
 export function removeUntil(from: ChildNode | null, end: Node | null): void {
     let node = from