neo.mjs 10.0.0-beta.6 → 10.0.0

package/test/siesta/tests/state/ProviderNestedDataConfigs.mjs

@@ -69,6 +69,65 @@ StartTest(t => {
         component.destroy();
     });
 
+    t.it('setData with a nested object should deep-merge and bubble reactivity', t => {
+        const component = Neo.create(MockComponent, {
+            stateProvider: {
+                data: {
+                    user: {
+                        firstname: 'John',
+                        lastname : 'Doe'
+                    }
+                }
+            }
+        });
+
+        const provider = component.getStateProvider();
+        let effectRunCount = 0;
+
+        provider.createBinding(component.id, 'user', data => {
+            effectRunCount++;
+            return data.user;
+        });
+
+        t.is(effectRunCount, 1, 'Effect ran initially');
+        t.isDeeply(proxyToObject(component.user), { firstname: 'John', lastname: 'Doe' }, 'Initial user object is correct');
+
+        // ACTION: Set data with a nested object. This should MERGE, not replace.
+        provider.setData({
+            user: { firstname: 'Jane' }
+        });
+
+        // ASSERT: The object was merged, and the old 'lastname' property is preserved.
+        t.is(effectRunCount, 2, 'Effect re-ran after setting the branch node');
+
+        const updatedUser = proxyToObject(component.user);
+        t.isDeeply(updatedUser, { firstname: 'Jane', lastname: 'Doe' }, 'User object should be deep-merged');
+        t.is(updatedUser.lastname, 'Doe', 'The "lastname" property should be preserved after merge');
+
+        // For contrast, let's show the path-based "bubbling" behavior which has the same outcome.
+        // First, reset the state.
+        provider.setData({
+            user: {
+                firstname: 'John',
+                lastname: 'Doe'
+            }
+        });
+        t.is(effectRunCount, 3, 'Effect ran after resetting state');
+
+        // ACTION: Set a leaf node using a path string.
+        provider.setData({
+            'user.firstname': 'Robert'
+        });
+
+        // ASSERT: The object was updated via bubbling, preserving the 'lastname' property.
+        t.is(effectRunCount, 4, 'Effect re-ran after setting a leaf node via path');
+        const mergedUser = proxyToObject(component.user);
+        t.isDeeply(mergedUser, { firstname: 'Robert', lastname: 'Doe' }, 'Path-based set should merge/preserve other properties');
+        t.is(mergedUser.lastname, 'Doe', 'The "lastname" property should be preserved');
+
+        component.destroy();
+    });
+
     t.it('Formulas should react to leaf node changes via bubbling', t => {
         let effectRunCount = 0;
 

package/test/siesta/tests/vdom/Advanced.mjs

@@ -1,6 +1,10 @@
-import Neo        from '../../../../src/Neo.mjs';
-import * as core  from '../../../../src/core/_export.mjs';
-import VdomHelper from '../../../../src/vdom/Helper.mjs';
+import Neo             from '../../../../src/Neo.mjs';
+import * as core       from '../../../../src/core/_export.mjs';
+import StringFromVnode from '../../../../src/vdom/util/StringFromVnode.mjs';
+import VdomHelper      from '../../../../src/vdom/Helper.mjs';
+
+// tests are designed for this rendering mode
+Neo.config.useDomApiRenderer = false;
 
 let deltas, output, vdom, vnode;
 
@@ -30,7 +34,7 @@ StartTest(t => {
             ]}
         ]};
 
-        vnode = VdomHelper.create(vdom);
+        vnode = VdomHelper.create({vdom}).vnode;
 
         vdom =
         {id: 'neo-calendar-week', cn: [
@@ -136,7 +140,7 @@ StartTest(t => {
             {id: 'neo-column-2', cls: ['foo4'], cn: []}
         ]};
 
-        vnode = VdomHelper.create(vdom);
+        vnode = VdomHelper.create({vdom}).vnode;
 
         vdom =
         {id: 'neo-calendar-week', cn: [
@@ -246,7 +250,7 @@ StartTest(t => {
             {id: 'neo-component-6'}
         ]};
 
-        vnode = VdomHelper.create(vdom);
+        vnode = VdomHelper.create({vdom}).vnode;
 
         vdom =
         {id: 'neo-container-1', cn: [
@@ -342,9 +346,11 @@ StartTest(t => {
             ]}
         ]};
 
-        vnode = VdomHelper.create(vdom);
+        const result = vnode = VdomHelper.create({vdom});
+        const outerHTML = result.outerHTML;
+        vnode = result.vnode;
 
-        t.is(vnode.outerHTML.includes('static'), false, 'The generated DOM does not include "static"');
+        t.is(outerHTML.includes('static'), false, 'The generated DOM does not include "static"');
         t.isDeeplyStrict(vnode.static, undefined, 'Top-level VNode did not get the static attribute');
         t.isDeeplyStrict(vnode.childNodes[0].static, true, 'First VNode childNode got the static attribute');
 

package/test/siesta/tests/vdom/VdomAsymmetricUpdates.mjs

@@ -1,15 +1,15 @@
-import Neo              from '../../../../src/Neo.mjs';
-import * as core        from '../../../../src/core/_export.mjs';
-import ComponentManager from '../../../../src/manager/Component.mjs';
-import TreeBuilder      from '../../../../src/util/vdom/TreeBuilder.mjs';
-import VDomUpdate       from '../../../../src/manager/VDomUpdate.mjs';
-import VdomLifecycle    from '../../../../src/mixin/VdomLifecycle.mjs';
-import VdomHelper       from '../../../../src/vdom/Helper.mjs';
-import VDomUtil         from '../../../../src/util/VDom.mjs';
+import Neo                from '../../../../src/Neo.mjs';
+import * as core          from '../../../../src/core/_export.mjs';
+import ComponentManager   from '../../../../src/manager/Component.mjs';
+import DomApiVnodeCreator from '../../../../src/vdom/util/DomApiVnodeCreator.mjs';
+import TreeBuilder        from '../../../../src/util/vdom/TreeBuilder.mjs';
+import VDomUpdate         from '../../../../src/manager/VDomUpdate.mjs';
+import VdomLifecycle      from '../../../../src/mixin/VdomLifecycle.mjs';
+import VdomHelper         from '../../../../src/vdom/Helper.mjs';
+import VDomUtil           from '../../../../src/util/VDom.mjs';
 
 // IMPORTANT: Test with the new standard renderer
 Neo.config.useDomApiRenderer = true;
-VdomHelper.onNeoConfigChange({useDomApiRenderer: true});
 
 /**
  * Creates a mock component object for testing.

package/test/siesta/tests/vdom/VdomRealWorldUpdates.mjs

@@ -1,8 +1,9 @@
-import Neo        from '../../../../src/Neo.mjs';
-import * as core  from '../../../../src/core/_export.mjs';
-import Component  from '../../../../src/component/Base.mjs';
-import Container  from '../../../../src/container/Base.mjs';
-import VdomHelper from '../../../../src/vdom/Helper.mjs';
+import Neo                from '../../../../src/Neo.mjs';
+import * as core          from '../../../../src/core/_export.mjs';
+import Component          from '../../../../src/component/Base.mjs';
+import Container          from '../../../../src/container/Base.mjs';
+import DomApiVnodeCreator from '../../../../src/vdom/util/DomApiVnodeCreator.mjs';
+import VdomHelper         from '../../../../src/vdom/Helper.mjs';
 
 // IMPORTANT: This test file uses real components and expects them to render.
 // We need to enable unitTestMode for isolation, but also allow VDOM updates.
@@ -10,7 +11,6 @@ Neo.config.unitTestMode = true;
 Neo.config.allowVdomUpdatesInTests = true;
 // This ensures that the VdomHelper uses the correct renderer for the assertions.
 Neo.config.useDomApiRenderer = true;
-VdomHelper.onNeoConfigChange({useDomApiRenderer: true});
 
 // Create a mock application context, as the component lifecycle requires it for updates.
 const appName = 'VdomRealWorldTestApp';

package/test/siesta/tests/vdom/layout/Cube.mjs

@@ -1,6 +1,10 @@
-import Neo        from '../../../../../src/Neo.mjs';
-import * as core  from '../../../../../src/core/_export.mjs';
-import VdomHelper from '../../../../../src/vdom/Helper.mjs';
+import Neo             from '../../../../../src/Neo.mjs';
+import * as core       from '../../../../../src/core/_export.mjs';
+import StringFromVnode from '../../../../../src/vdom/util/StringFromVnode.mjs';
+import VdomHelper      from '../../../../../src/vdom/Helper.mjs';
+
+// tests are designed for this rendering mode
+Neo.config.useDomApiRenderer = false;
 
 let oldVdom, oldVnode, vdom;
 
@@ -16,7 +20,7 @@ StartTest(t => {
             {id: 'neo-component-6'}
         ]};
 
-        let oldVnode = VdomHelper.create(oldVdom);
+        let oldVnode = VdomHelper.create({vdom: oldVdom}).vnode;
 
         vdom =
         {id: 'neo-container-1', cn: [
@@ -67,7 +71,7 @@ StartTest(t => {
             ]}
         ]};
 
-        oldVnode = VdomHelper.create(oldVdom);
+        oldVnode = VdomHelper.create({vdom: oldVdom}).vnode;
 
         vdom =
         {id: 'neo-container-1', cn: [
@@ -105,7 +109,7 @@ StartTest(t => {
             {id: 'neo-component-6'}
         ]};
 
-        oldVnode = VdomHelper.create(oldVdom);
+        oldVnode = VdomHelper.create({vdom: oldVdom}).vnode;
 
         vdom =
         {id: 'neo-container-1', cn: [
@@ -158,7 +162,7 @@ StartTest(t => {
             ]}
         ]};
 
-        oldVnode = VdomHelper.create(oldVdom);
+        oldVnode = VdomHelper.create({vdom: oldVdom}).vnode;
 
         vdom =
         {id: 'neo-container-1', cn: [

package/test/siesta/tests/vdom/table/Container.mjs

@@ -1,6 +1,10 @@
-import Neo        from '../../../../../src/Neo.mjs';
-import * as core  from '../../../../../src/core/_export.mjs';
-import VdomHelper from '../../../../../src/vdom/Helper.mjs';
+import Neo             from '../../../../../src/Neo.mjs';
+import * as core       from '../../../../../src/core/_export.mjs';
+import StringFromVnode from '../../../../../src/vdom/util/StringFromVnode.mjs';
+import VdomHelper      from '../../../../../src/vdom/Helper.mjs';
+
+// tests are designed for this rendering mode
+Neo.config.useDomApiRenderer = false;
 
 let deltas, output, vdom, vnode;
 
@@ -17,7 +21,7 @@ StartTest(t => {
             ]}
         ]};
 
-        vnode = VdomHelper.create(vdom);
+        vnode = VdomHelper.create({vdom}).vnode;
 
         vdom =
         {id: 'neo-table-container-1', cn: [
@@ -75,7 +79,7 @@ StartTest(t => {
             ]}
         ]};
 
-        vnode = VdomHelper.create(vdom);
+        vnode = VdomHelper.create({vdom}).vnode;
 
         vdom =
         {id: 'neo-wrapper-1', cn: [

package/src/core/EffectBatchManager.mjs

@@ -1,67 +0,0 @@
-/**
- * A singleton manager responsible for batching `Neo.core.Effect` executions.
- * This ensures that effects triggered by multiple config changes within a single
- * synchronous operation (e.g., `Neo.core.Base#set()`) are executed only once
- * per batch, after all changes have been applied.
- * @class Neo.core.EffectBatchManager
- * @singleton
- */
-const EffectBatchManager = {
-    /**
-     * The current count of active batch operations.
-     * Incremented by `startBatch()`, decremented by `endBatch()`.
-     * @member {Number} batchCount=0
-     */
-    batchCount: 0,
-    /**
-     * A Set of `Neo.core.Effect` instances that are pending execution within the current batch.
-     * @member {Set<Neo.core.Effect>} pendingEffects=new Set()
-     */
-    pendingEffects: new Set(),
-
-    /**
-     * Decrements the batch counter. If `batchCount` reaches 0, all queued effects
-     * are executed and the `pendingEffects` Set is cleared.
-     */
-    endBatch() {
-        const me = this;
-
-        me.batchCount--;
-
-        if (me.batchCount === 0) {
-            const effectsToRun = [...me.pendingEffects]; // Create a snapshot
-            me.pendingEffects.clear();
-
-            effectsToRun.forEach(effect => {
-                effect.run()
-            })
-        }
-    },
-
-    /**
-     * Checks if there is an active batch operation.
-     * @returns {Boolean}
-     */
-    isBatchActive() {
-        return this.batchCount > 0
-    },
-
-    /**
-     * Queues an effect for execution at the end of the current batch.
-     * If the effect is already queued, it will not be added again.
-     * @param {Neo.core.Effect} effect The effect to queue.
-     */
-    queueEffect(effect) {
-        this.pendingEffects.add(effect)
-    },
-
-    /**
-     * Increments the batch counter. When `batchCount` is greater than 0,
-     * effects will be queued instead of running immediately.
-     */
-    startBatch() {
-        this.batchCount++
-    }
-};
-
-export default Neo.gatekeep(EffectBatchManager, 'Neo.core.EffectBatchManager');