@mulanjs/mulanjs 1.0.1-dev.20260227181914 → 1.0.1-dev.20260228083459

package/dist/core/reactive.js

@@ -146,12 +146,23 @@ function trigger(target, key) {
         });
     }
 }
+// Array mutating methods that must trigger reactive updates
+const ARRAY_MUTATION_METHODS = ['push', 'pop', 'shift', 'unshift', 'splice', 'sort', 'reverse', 'fill'];
+// Proxy cache: ensures the same object always returns the same Proxy
+// This is CRITICAL for array reactivity — mu-for and array mutations must
+// share the same Proxy instance, otherwise triggers reach different subscribers.
+const proxyCache = new WeakMap();
 /**
  * Creates a reactive proxy object (Vue-compatible).
- * Now optimized to respect Mulan Cycle.
+ * Intercepts array mutation methods to trigger reactive updates,
+ * since methods like push() bypass the Proxy 'set' trap.
  */
 function reactive(target) {
-    return new Proxy(target, {
+    // Return cached proxy if it already exists for this target
+    if (proxyCache.has(target)) {
+        return proxyCache.get(target);
+    }
+    const proxy = new Proxy(target, {
         get(obj, prop, receiver) {
             // IRON FORTRESS: Prototype Pollution Protection (Read)
             if (prop === '__proto__' || prop === 'constructor' || prop === 'prototype') {
@@ -159,6 +170,17 @@ function reactive(target) {
             }
             track(obj, prop);
             const val = Reflect.get(obj, prop, receiver);
+            // Intercept array mutation methods to trigger length + index updates
+            if (Array.isArray(obj) && typeof prop === 'string' && ARRAY_MUTATION_METHODS.includes(prop)) {
+                return function (...args) {
+                    const result = val.apply(obj, args);
+                    // Trigger on 'length' — this is what mu-for subscribes to
+                    trigger(obj, 'length');
+                    // Also trigger on the array itself for any parent effects
+                    trigger(obj, prop);
+                    return result;
+                };
+            }
             if (val !== null && typeof val === 'object') {
                 return reactive(val);
             }
@@ -172,9 +194,15 @@ function reactive(target) {
             }
             const result = Reflect.set(obj, prop, value, receiver);
             trigger(obj, prop);
+            // If array length changed (e.g. index assignment), also trigger length
+            if (Array.isArray(obj) && prop !== 'length') {
+                trigger(obj, 'length');
+            }
             return result;
         },
     });
+    proxyCache.set(target, proxy);
+    return proxy;
 }
 exports.reactive = reactive;
 /**

package/dist/mulan.esm.js

@@ -242,12 +242,23 @@ function trigger(target, key) {
         });
     }
 }
+// Array mutating methods that must trigger reactive updates
+const ARRAY_MUTATION_METHODS = ['push', 'pop', 'shift', 'unshift', 'splice', 'sort', 'reverse', 'fill'];
+// Proxy cache: ensures the same object always returns the same Proxy
+// This is CRITICAL for array reactivity — mu-for and array mutations must
+// share the same Proxy instance, otherwise triggers reach different subscribers.
+const proxyCache = new WeakMap();
 /**
  * Creates a reactive proxy object (Vue-compatible).
- * Now optimized to respect Mulan Cycle.
+ * Intercepts array mutation methods to trigger reactive updates,
+ * since methods like push() bypass the Proxy 'set' trap.
  */
 function reactive(target) {
-    return new Proxy(target, {
+    // Return cached proxy if it already exists for this target
+    if (proxyCache.has(target)) {
+        return proxyCache.get(target);
+    }
+    const proxy = new Proxy(target, {
         get(obj, prop, receiver) {
             // IRON FORTRESS: Prototype Pollution Protection (Read)
             if (prop === '__proto__' || prop === 'constructor' || prop === 'prototype') {
@@ -255,6 +266,17 @@ function reactive(target) {
             }
             track(obj, prop);
             const val = Reflect.get(obj, prop, receiver);
+            // Intercept array mutation methods to trigger length + index updates
+            if (Array.isArray(obj) && typeof prop === 'string' && ARRAY_MUTATION_METHODS.includes(prop)) {
+                return function (...args) {
+                    const result = val.apply(obj, args);
+                    // Trigger on 'length' — this is what mu-for subscribes to
+                    trigger(obj, 'length');
+                    // Also trigger on the array itself for any parent effects
+                    trigger(obj, prop);
+                    return result;
+                };
+            }
             if (val !== null && typeof val === 'object') {
                 return reactive(val);
             }
@@ -268,9 +290,15 @@ function reactive(target) {
             }
             const result = Reflect.set(obj, prop, value, receiver);
             trigger(obj, prop);
+            // If array length changed (e.g. index assignment), also trigger length
+            if (Array.isArray(obj) && prop !== 'length') {
+                trigger(obj, 'length');
+            }
             return result;
         },
     });
+    proxyCache.set(target, proxy);
+    return proxy;
 }
 /**
  * Creates a standalone reactive reference.