@creact-labs/creact 0.2.6 → 0.2.8

package/dist/primitives/instance.js

@@ -4,6 +4,26 @@
 import { createSignal } from '../reactive/signal';
 import { batch } from '../reactive/tracking';
 import { getCurrentFiber, getCurrentResourcePath, pushResourcePath } from '../runtime/render';
+/**
+ * Shallow equality check for output deduplication
+ */
+function shallowEqual(a, b) {
+    if (a === b)
+        return true;
+    if (a == null || b == null)
+        return false;
+    if (typeof a !== 'object' || typeof b !== 'object')
+        return false;
+    const keysA = Object.keys(a);
+    const keysB = Object.keys(b);
+    if (keysA.length !== keysB.length)
+        return false;
+    for (const key of keysA) {
+        if (a[key] !== b[key])
+            return false;
+    }
+    return true;
+}
 // Registry of all instance nodes by ID
 const nodeRegistry = new Map();
 // Track which fiber owns each nodeId (to detect duplicate siblings)
@@ -101,9 +121,23 @@ export function useInstance(construct, props) {
             outputSignals: new Map(),
             children: [],
             setOutputs(outputs) {
-                // Clear ownership before batch triggers re-renders
-                // When signals update, components re-execute and create new fibers
-                // that need to claim the same nodeIds
+                // First check if any values actually changed
+                let hasChanges = false;
+                for (const [key, value] of Object.entries(outputs)) {
+                    if (!this.outputSignals.has(key)) {
+                        hasChanges = true;
+                        break;
+                    }
+                    const [read] = this.outputSignals.get(key);
+                    if (!shallowEqual(read(), value)) {
+                        hasChanges = true;
+                        break;
+                    }
+                }
+                // Early exit if nothing changed - no re-render needed
+                if (!hasChanges)
+                    return;
+                // Only clear ownership and batch if there are actual changes
                 nodeOwnership.clear();
                 batch(() => {
                     for (const [key, value] of Object.entries(outputs)) {
@@ -112,8 +146,7 @@ export function useInstance(construct, props) {
                         }
                         else {
                             const [read, write] = this.outputSignals.get(key);
-                            // Only update if value actually changed
-                            if (read() !== value) {
+                            if (!shallowEqual(read(), value)) {
                                 write(value);
                             }
                         }
@@ -139,7 +172,7 @@ export function useInstance(construct, props) {
             else {
                 const [read, write] = node.outputSignals.get(key);
                 // Only update if value actually changed
-                if (read() !== value) {
+                if (!shallowEqual(read(), value)) {
                     write(value);
                 }
             }
@@ -194,16 +227,29 @@ export function fillInstanceOutputs(nodeId, outputs) {
     const node = nodeRegistry.get(nodeId);
     if (!node)
         return;
-    // Clear ownership before batch triggers re-renders
-    // When signals update, components re-execute and create new fibers
-    // that need to claim the same nodeIds
+    // First check if any values actually changed
+    let hasChanges = false;
+    for (const [key, value] of Object.entries(outputs)) {
+        if (!node.outputSignals.has(key)) {
+            hasChanges = true;
+            break;
+        }
+        const [read] = node.outputSignals.get(key);
+        if (!shallowEqual(read(), value)) {
+            hasChanges = true;
+            break;
+        }
+    }
+    // Early exit if nothing changed - no re-render needed
+    if (!hasChanges)
+        return;
+    // Only clear ownership and batch if there are actual changes
     nodeOwnership.clear();
     batch(() => {
         for (const [key, value] of Object.entries(outputs)) {
             if (node.outputSignals.has(key)) {
                 const [read, write] = node.outputSignals.get(key);
-                // Only update if value actually changed
-                if (read() !== value) {
+                if (!shallowEqual(read(), value)) {
                     write(value);
                 }
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@creact-labs/creact",
-  "version": "0.2.6",
+  "version": "0.2.8",
   "type": "module",
   "description": "Declarative universal reactive runtime",
   "main": "dist/index.js",