@mintjamsinc/ichigojs 0.1.82 → 0.1.83

package/dist/ichigo.cjs

@@ -10136,9 +10136,9 @@
          */
         #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;
         /**
@@ -10308,7 +10308,8 @@
             // 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.
@@ -10620,8 +10621,9 @@
                         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
@@ -10633,7 +10635,7 @@
             }
             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();
                 });
             }
@@ -10715,6 +10717,7 @@
          * 4. Unregister dependencies
          * 5. Clean up directive manager
          * 6. Call onUnmounted lifecycle hooks
+         * 7. Leave the parent's child list
          */
         destroy() {
             this.#destroyed = true;
@@ -10741,9 +10744,9 @@
                 }
                 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();
                     }
@@ -10771,6 +10774,15 @@
                     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);
+            }
         }
     }