#7076 functional.component.Base: onEffectRunStateChange() => depth check fix

Author: tobiu

src/functional/component/Base.mjs

@@ -313,6 +313,25 @@ class FunctionalBase extends Base {
                     }
                 });
 
+                // If this component created other classic or functional components,
+                // include their full vdom into the next update cycle.
+                const oldKeys = me.childComponents ? new Set(me.childComponents.keys()) : new Set();
+                let hasNewChildren = false;
+
+                for (const newKey of me.#nextChildComponents.keys()) {
+                    if (!oldKeys.has(newKey)) {
+                        hasNewChildren = true;
+                        break
+                    }
+                }
+
+                if (hasNewChildren) {
+                    // When new child components are created, we need to send their full VDOM
+                    // to the vdom-worker, so they can get rendered.
+                    // Subsequent updates will be granular via diffAndSet() => set() on the child.
+                    me.updateDepth = -1;
+                }
+
                 // Update the main map of instantiated components
                 me.childComponents = me.#nextChildComponents;
 
@@ -336,25 +355,6 @@ class FunctionalBase extends Base {
                     root.id = me.id
                 }
 
-                // If this component created other classic or functional components,
-                // include their full vdom into the next update cycle.
-                const oldKeys = me.childComponents ? new Set(me.childComponents.keys()) : new Set();
-                let hasNewChildren = false;
-
-                for (const newKey of me.#nextChildComponents.keys()) {
-                    if (!oldKeys.has(newKey)) {
-                        hasNewChildren = true;
-                        break
-                    }
-                }
-
-                if (hasNewChildren) {
-                    // When new child components are created, we need to send their full VDOM
-                    // to the vdom-worker, so they can get rendered.
-                    // Subsequent updates will be granular via diffAndSet() => set() on the child.
-                    me.updateDepth = -1;
-                }
-
                 me.updateVdom();
 
                 // Update DOM event listeners based on the new render

src/mixin/VdomLifecycle.mjs

@@ -680,4 +680,4 @@ class VdomLifecycle extends Base {
 
 Neo.setupClass(VdomLifecycle);
 
-export default VdomLifecycle;
+export default VdomLifecycle;