@mintjamsinc/ichigojs 0.1.82 → 0.1.83

package/dist/ichigo.esm.js

@@ -10130,9 +10130,9 @@ class VNode {
      */
     #fragmentRange;
     /**
-     * Set by {@link destroy}. A destroyed clone stays in its parent's child
-     * list (v-if / v-for render new clones under the same parent), so walks
-     * over the children must skip it.
+     * Set by {@link destroy}. A destroyed node also leaves its parent's child
+     * list, but a walk that took its copy of the list before the node was
+     * destroyed may still reach it, and must skip it.
      */
     #destroyed = false;
     /**
@@ -10302,7 +10302,8 @@ class VNode {
         // walk runs once the subtree is in the document, which the outlet
         // cannot know when it renders inside a detached v-if / v-for clone.
         this.#directiveManager?.directives?.forEach(d => d.expandComponents?.());
-        this.#childVNodes?.forEach(child => child.expandComponents());
+        // A copy: expanding can render and destroy clones under this node.
+        this.#childVNodes?.slice().forEach(child => child.expandComponents());
     }
     /**
      * The application instance associated with this virtual node.
@@ -10614,8 +10615,9 @@ class VNode {
                     updater.applyToDOM();
                 }
             }
-            // Recursively update child virtual nodes
-            this.#childVNodes?.forEach(childVNode => {
+            // Recursively update child virtual nodes. A copy: an update can
+            // destroy a v-if / v-for clone, which removes it from this list.
+            this.#childVNodes?.slice().forEach(childVNode => {
                 childVNode.forceUpdate();
             });
             // If no directive requires template preservation, call onUpdated for directives that do not templatize
@@ -10627,7 +10629,7 @@ class VNode {
         }
         else if (this.#nodeType === Node.DOCUMENT_FRAGMENT_NODE) {
             // For document fragments, recursively force update child VNodes
-            this.#childVNodes?.forEach(childVNode => {
+            this.#childVNodes?.slice().forEach(childVNode => {
                 childVNode.forceUpdate();
             });
         }
@@ -10709,6 +10711,7 @@ class VNode {
      * 4. Unregister dependencies
      * 5. Clean up directive manager
      * 6. Call onUnmounted lifecycle hooks
+     * 7. Leave the parent's child list
      */
     destroy() {
         this.#destroyed = true;
@@ -10735,9 +10738,9 @@ class VNode {
             }
             this.#userData.clear();
         }
-        // Recursively destroy child nodes
+        // Recursively destroy child nodes (a copy: each one leaves this list)
         if (this.#childVNodes) {
-            for (const childVNode of this.#childVNodes) {
+            for (const childVNode of this.#childVNodes.slice()) {
                 try {
                     childVNode.destroy();
                 }
@@ -10765,6 +10768,15 @@ class VNode {
                 d.onUnmounted?.();
             });
         }
+        // Leave the parent's child list. v-if and v-for render every new clone
+        // under the same parent VNode, so a destroyed clone left in the list
+        // would keep its whole subtree — VNodes, DOM nodes, bindings — alive
+        // for as long as the parent, growing with each toggle or row change.
+        const siblings = this.#parentVNode ? this.#parentVNode.#childVNodes : undefined;
+        const index = siblings ? siblings.indexOf(this) : -1;
+        if (index !== -1) {
+            siblings.splice(index, 1);
+        }
     }
 }