@estjs/signals 0.0.15 → 0.0.16-beta.2

package/dist/signals.dev.cjs.js

@@ -3,8 +3,6 @@
 var shared = require('@estjs/shared');
 
 var __defProp = Object.defineProperty;
-var __defProps = Object.defineProperties;
-var __getOwnPropDescs = Object.getOwnPropertyDescriptors;
 var __getOwnPropSymbols = Object.getOwnPropertySymbols;
 var __hasOwnProp = Object.prototype.hasOwnProperty;
 var __propIsEnum = Object.prototype.propertyIsEnumerable;
@@ -20,7 +18,6 @@ var __spreadValues = (a, b) => {
     }
   return a;
 };
-var __spreadProps = (a, b) => __defProps(a, __getOwnPropDescs(b));
 
 // src/constants.ts
 var TriggerOpTypes = {
@@ -110,6 +107,23 @@ function shallowPropagate(link) {
 var currentLinkVersion = 0;
 var activeSub;
 var isUntracking = false;
+var Dep = class {
+  constructor(map, key) {
+    this.map = map;
+    this.key = key;
+    this.isDep = true;
+    this.flag = 0 /* NONE */;
+  }
+  get subLink() {
+    return this._subLink;
+  }
+  set subLink(value) {
+    this._subLink = value;
+    if (value === void 0) {
+      this.map.delete(this.key);
+    }
+  }
+};
 function linkReactiveNode(depNode, subNode) {
   if (isUntracking) {
     return void 0;
@@ -198,7 +212,9 @@ function unlinkReactiveNode(linkNode, subNode = linkNode.subNode) {
         toRemove = unlinkReactiveNode(toRemove, depNode);
       }
       depNode.depLinkTail = void 0;
-      depNode.flag |= 16 /* DIRTY */;
+      if (!depNode.isDep) {
+        depNode.flag |= 16 /* DIRTY */;
+      }
       {
         if (depNode.depLink) {
           shared.error(
@@ -329,17 +345,14 @@ function collectTriggeredEffects(dep, effects, version) {
   if (!dep) {
     return;
   }
-  dep.forEach((effect2) => {
-    if (effect2.flag & 2 /* WATCHING */ && !effect2._active) {
-      dep.delete(effect2);
-      return;
-    }
+  for (let link = dep.subLink; link; link = link.nextSubLink) {
+    const effect2 = link.subNode;
     if (effect2._triggerVersion === version) {
-      return;
+      continue;
     }
     effect2._triggerVersion = version;
     effects.push(effect2);
-  });
+  }
 }
 function track(target, key) {
   if (!activeSub || isUntracking) {
@@ -352,19 +365,10 @@ function track(target, key) {
   }
   let dep = depsMap.get(key);
   if (!dep) {
-    dep = /* @__PURE__ */ new Set();
+    dep = new Dep(depsMap, key);
     depsMap.set(key, dep);
   }
-  const sizeBefore = dep.size ;
-  dep.add(activeSub);
-  if (dep.size !== sizeBefore && shared.isFunction(activeSub.onTrack)) {
-    activeSub.onTrack({
-      effect: activeSub,
-      target,
-      type: "get",
-      key
-    });
-  }
+  linkReactiveNode(dep, activeSub);
 }
 function trigger(target, type, key, newValue) {
   var _a5;
@@ -479,8 +483,7 @@ function createArrayInstrumentations() {
     instrumentations[key] = function(...args) {
       const arr = toRaw(this);
       track(arr, ARRAY_ITERATE_KEY);
-      const res = Array.prototype[key].apply(arr, args);
-      return res;
+      return Array.prototype[key].apply(this, args);
     };
   });
   ["join", "toString", "toLocaleString"].forEach((key) => {
@@ -497,6 +500,9 @@ function createArrayInstrumentations() {
       track(arr, ARRAY_KEY);
       const rawIterator = key === Symbol.iterator ? arr[Symbol.iterator]() : arr[key]();
       return {
+        /**
+         * Returns the next iterator result.
+         */
         next() {
           const { value, done } = rawIterator.next();
           if (done) {
@@ -513,6 +519,9 @@ function createArrayInstrumentations() {
             done
           };
         },
+        /**
+         * Returns an iterator for the current collection.
+         */
         [Symbol.iterator]() {
           return this;
         }
@@ -561,6 +570,9 @@ var arrayHandlers = (shallow) => ({
 var shallowArrayHandlers = arrayHandlers(true);
 var deepArrayHandlers = arrayHandlers(false);
 var collectionHandlers = {
+  /**
+   * Exposes collection proxy flags and instrumented methods.
+   */
   get(target, key) {
     if (key === "_IS_REACTIVE" /* IS_REACTIVE */) {
       return true;
@@ -576,6 +588,9 @@ var collectionHandlers = {
   }
 };
 var weakCollectionHandlers = {
+  /**
+   * Exposes weak-collection proxy flags and instrumented methods.
+   */
   get(target, key) {
     if (key === "_IS_REACTIVE" /* IS_REACTIVE */) {
       return true;
@@ -591,6 +606,9 @@ var weakCollectionHandlers = {
   }
 };
 var collectionInstrumentations = {
+  /**
+   * Reads a Map entry with dependency tracking.
+   */
   get(key) {
     const target = toRaw(this);
     track(target, COLLECTION_KEY);
@@ -600,6 +618,9 @@ var collectionInstrumentations = {
     }
     return value;
   },
+  /**
+   * Sets the requested value.
+   */
   set(key, value) {
     const target = toRaw(this);
     const hadKey = target.has(key);
@@ -611,6 +632,9 @@ var collectionInstrumentations = {
     }
     return this;
   },
+  /**
+   * Adds the requested value.
+   */
   add(value) {
     const target = toRaw(this);
     const rawValue = toRaw(value);
@@ -618,11 +642,12 @@ var collectionInstrumentations = {
     target.add(rawValue);
     if (!hadValue) {
       trigger(target, TriggerOpTypes.ADD, COLLECTION_KEY);
-    } else {
-      trigger(target, TriggerOpTypes.SET, COLLECTION_KEY);
     }
     return this;
   },
+  /**
+   * Returns whether the requested value exists.
+   */
   has(key) {
     const target = toRaw(this);
     track(target, COLLECTION_KEY);
@@ -632,6 +657,9 @@ var collectionInstrumentations = {
     }
     return hasKey;
   },
+  /**
+   * Deletes the requested value.
+   */
   delete(key) {
     const target = toRaw(this);
     const hadKey = target.has(key);
@@ -644,6 +672,9 @@ var collectionInstrumentations = {
     }
     return result;
   },
+  /**
+   * Clears the current collection.
+   */
   clear() {
     const target = toRaw(this);
     const hadItems = target.size > 0;
@@ -653,6 +684,9 @@ var collectionInstrumentations = {
     }
     return result;
   },
+  /**
+   * Iterates over each collection entry.
+   */
   forEach(callback, thisArg) {
     const target = toRaw(this);
     const isShallowMode = isShallow(this);
@@ -663,12 +697,18 @@ var collectionInstrumentations = {
       callback.call(thisArg, wrappedValue, wrappedKey, this);
     });
   },
+  /**
+   * Returns an iterator for the current collection.
+   */
   [Symbol.iterator]() {
     const target = toRaw(this);
     const isShallowMode = isShallow(this);
     track(target, COLLECTION_KEY);
     const rawIterator = target[Symbol.iterator]();
     return {
+      /**
+       * Returns the next iterator result.
+       */
       next() {
         const { value, done } = rawIterator.next();
         if (done) {
@@ -688,22 +728,34 @@ var collectionInstrumentations = {
           done
         };
       },
+      /**
+       * Returns an iterator for the current collection.
+       */
       [Symbol.iterator]() {
         return this;
       }
     };
   },
+  /**
+   * Returns the current collection size.
+   */
   get size() {
     const target = toRaw(this);
     track(target, COLLECTION_KEY);
     return target.size;
   },
+  /**
+   * Returns an iterator over the current keys.
+   */
   keys() {
     const target = toRaw(this);
     const isShallowMode = isShallow(this);
     track(target, COLLECTION_KEY);
     const rawIterator = target.keys();
     return {
+      /**
+       * Returns the next iterator result.
+       */
       next() {
         const { value, done } = rawIterator.next();
         if (done) {
@@ -714,17 +766,26 @@ var collectionInstrumentations = {
           done
         };
       },
+      /**
+       * Returns an iterator for the current collection.
+       */
       [Symbol.iterator]() {
         return this;
       }
     };
   },
+  /**
+   * Returns an iterator over the current values.
+   */
   values() {
     const target = toRaw(this);
     const isShallowMode = isShallow(this);
     track(target, COLLECTION_KEY);
     const rawIterator = target.values();
     return {
+      /**
+       * Returns the next iterator result.
+       */
       next() {
         const { value, done } = rawIterator.next();
         if (done) {
@@ -735,17 +796,26 @@ var collectionInstrumentations = {
           done
         };
       },
+      /**
+       * Returns an iterator for the current collection.
+       */
       [Symbol.iterator]() {
         return this;
       }
     };
   },
+  /**
+   * Returns an iterator over the current entries.
+   */
   entries() {
     const target = toRaw(this);
     const isShallowMode = isShallow(this);
     track(target, COLLECTION_KEY);
     const rawIterator = target.entries();
     return {
+      /**
+       * Returns the next iterator result.
+       */
       next() {
         const { value, done } = rawIterator.next();
         if (done) {
@@ -759,6 +829,9 @@ var collectionInstrumentations = {
           done
         };
       },
+      /**
+       * Returns an iterator for the current collection.
+       */
       [Symbol.iterator]() {
         return this;
       }
@@ -766,6 +839,9 @@ var collectionInstrumentations = {
   }
 };
 var weakInstrumentations = {
+  /**
+   * Reads a WeakMap entry with dependency tracking.
+   */
   get(key) {
     const target = toRaw(this);
     track(target, WEAK_COLLECTION_KEY);
@@ -778,6 +854,9 @@ var weakInstrumentations = {
     }
     return value;
   },
+  /**
+   * Sets the requested value.
+   */
   set(key, value) {
     const target = toRaw(this);
     const rawKey = toRaw(key);
@@ -790,6 +869,9 @@ var weakInstrumentations = {
     }
     return this;
   },
+  /**
+   * Adds the requested value.
+   */
   add(value) {
     const target = toRaw(this);
     const rawValue = toRaw(value);
@@ -800,6 +882,9 @@ var weakInstrumentations = {
     }
     return this;
   },
+  /**
+   * Returns whether the requested value exists.
+   */
   has(key) {
     const target = toRaw(this);
     track(target, WEAK_COLLECTION_KEY);
@@ -809,6 +894,9 @@ var weakInstrumentations = {
     }
     return hasKey;
   },
+  /**
+   * Deletes the requested value.
+   */
   delete(key) {
     const target = toRaw(this);
     const rawKey = toRaw(key);
@@ -821,6 +909,9 @@ var weakInstrumentations = {
   }
 };
 var objectHandlers = (shallow) => ({
+  /**
+   * Reads an object property, unwraps signals, and tracks the access.
+   */
   get(target, key, receiver) {
     if (key === "_RAW" /* RAW */) {
       return target;
@@ -913,6 +1004,7 @@ var toReactive = (value) => shared.isObject(value) ? reactive(value) : value;
 var _a;
 _a = "_IS_SIGNAL" /* IS_SIGNAL */;
 var SignalImpl = class {
+  // Mark as Signal
   /**
    * Create a new Signal with the given initial value.
    *
@@ -922,12 +1014,12 @@ var SignalImpl = class {
   constructor(value, shallow = false) {
     this.flag = 1 /* MUTABLE */;
     // Mark whether it's shallow reactive
-    // @ts-ignore: used internally by isSignal typeguard
+    // @ts-ignore
     this[_a] = true;
     const unwrapped = toRaw(value);
     this._rawValue = unwrapped;
     this["_IS_SHALLOW" /* IS_SHALLOW */] = shallow;
-    if (!shared.isObject(unwrapped)) {
+    if (!shouldWrapReactiveValue(unwrapped)) {
       this._value = unwrapped;
     } else {
       if (isReactive(value)) {
@@ -937,10 +1029,19 @@ var SignalImpl = class {
       }
     }
   }
-  // dep getter, returns itself for dependency collection
+  /**
+   * Returns the dependency node used for tracking.
+   *
+   * @returns {this} The dependency node.
+   */
   get dep() {
     return this;
   }
+  /**
+   * Returns the current value.
+   *
+   * @returns {T} The current value.
+   */
   get value() {
     const sub = activeSub;
     if (sub) {
@@ -955,7 +1056,11 @@ var SignalImpl = class {
     }
     return this._value;
   }
-  // value setter, triggers update when value changes
+  /**
+   * Updates the current value.
+   *
+   * @param newValue - The new value to set.
+   */
   set value(newValue) {
     if (isSignal(newValue)) {
       {
@@ -973,7 +1078,7 @@ var SignalImpl = class {
     this._oldValue = this._rawValue;
     this._rawValue = rawValue;
     this.flag |= 16 /* DIRTY */;
-    if (!shared.isObject(rawValue)) {
+    if (!shouldWrapReactiveValue(rawValue)) {
       this._value = rawValue;
     } else if (isReactive(originalValue)) {
       this._value = originalValue;
@@ -986,7 +1091,11 @@ var SignalImpl = class {
       propagate(subs);
     }
   }
-  // Check if the value should be updated
+  /**
+   * Check if the value should be updated.
+   *
+   * @returns {boolean} True if the value should be updated.
+   */
   shouldUpdate() {
     this.flag &= -17 /* DIRTY */;
     if (!("_oldValue" in this)) {
@@ -996,15 +1105,29 @@ var SignalImpl = class {
     this._oldValue = this._rawValue;
     return changed;
   }
-  // Get current value without triggering dependency tracking
+  /**
+   * Get current value without triggering dependency tracking.
+   *
+   * @returns {T} The current value.
+   */
   peek() {
     return this._value;
   }
-  // set method is an alias for the value setter
+  /**
+   * Sets the requested value.
+   *
+   * @param value - The new value to set.
+   * @returns {void}
+   */
   set(value) {
     this.value = value;
   }
-  // Update value using an updater function
+  /**
+   * Update value using an updater function.
+   *
+   * @param updater - A function that receives the current value and returns the new value.
+   * @returns {void}
+   */
   update(updater) {
     const nextValue = updater(this.peek());
     if (isSignal(nextValue)) {
@@ -1019,6 +1142,10 @@ var SignalImpl = class {
     }
   }
 };
+function shouldWrapReactiveValue(value) {
+  if (!shared.isObject(value)) return false;
+  return shared.isArray(value) || shared.isMap(value) || shared.isSet(value) || shared.isWeakMap(value) || shared.isWeakSet(value) || shared.isPlainObject(value);
+}
 function signal(value) {
   if (isSignal(value)) {
     {
@@ -1142,25 +1269,24 @@ var _a2;
 _a2 = "_IS_EFFECT" /* IS_EFFECT */;
 var EffectImpl = class {
   /**
-   * Create an Effect instance
+   * Create an Effect instance.
    *
-   * @param fn - The effect function
-   * @param options - Configuration options
+   * @param fn - The effect function.
+   * @param options - Configuration options.
    */
   constructor(fn, options) {
     this.flag = 2 /* WATCHING */ | 16 /* DIRTY */;
-    // @ts-ignore: used internally by isEffect typeguard
+    // @ts-ignore
     this[_a2] = true;
-    // State management
+    //  State management
     this._active = true;
-    var _a5;
     this.fn = fn;
     if (options) {
-      const scheduler = (_a5 = options.scheduler) != null ? _a5 : options.flush;
-      if (scheduler) {
-        this._flushScheduler = shared.isFunction(scheduler) ? () => scheduler(this) : createScheduler(() => this.run(), scheduler);
+      this.options = options;
+      const scheduler = options.flush || options.scheduler;
+      if (scheduler && !shared.isFunction(scheduler)) {
+        this._flushScheduler = createScheduler(() => this.run(), scheduler);
       }
-      if (options.onStop) this.onStop = options.onStop;
       {
         if (options.onTrack) this.onTrack = options.onTrack;
         if (options.onTrigger) this.onTrigger = options.onTrigger;
@@ -1168,13 +1294,17 @@ var EffectImpl = class {
     }
   }
   /**
-   * Check if the Effect is active
+   * Check if the Effect is active.
+   *
+   * @returns {boolean} True if the effect is active.
    */
   get active() {
     return this._active;
   }
   /**
-   * Check if the Effect is dirty (needs re-execution)
+   * Check if the Effect is dirty (needs re-execution).
+   *
+   * @returns {boolean} True if the effect is dirty.
    */
   get dirty() {
     const flags = this.flag;
@@ -1191,57 +1321,39 @@ var EffectImpl = class {
     return false;
   }
   /**
-   * Pause Effect execution
+   * Pause Effect execution.
    *
    * When an effect is paused:
-   * - It stops responding to dependency changes
-   * - Notifications are ignored (see notify method)
-   * - DIRTY and PENDING flags are still set when dependencies change
-   * - The effect remains active and maintains its dependency links
+   * - It stops responding to dependency changes.
+   * - Notifications are ignored (see notify method).
+   * - DIRTY and PENDING flags are still set when dependencies change.
+   * - The effect remains active and maintains its dependency links.
    *
    * Use cases:
-   * - Temporarily disable effects during bulk updates
-   * - Prevent effects from running during initialization
-   * - Control when side effects should execute
+   * - Temporarily disable effects during bulk updates.
+   * - Prevent effects from running during initialization.
+   * - Control when side effects should execute.
    *
-   * @example
-   * ```typescript
-   * const count = signal(0);
-   * const runner = effect(() => console.log(count.value));
-   *
-   * runner.effect.pause();
-   * count.value = 1; // Effect won't run
-   * count.value = 2; // Effect won't run
-   * runner.effect.resume(); // Effect runs once with latest value
-   * ```
+   * @returns {void}
    */
   pause() {
     this.flag |= 256 /* PAUSED */;
   }
   /**
-   * Resume Effect execution
+   * Resume Effect execution.
    *
    * When an effect is resumed:
-   * - The PAUSED flag is cleared
+   * - The PAUSED flag is cleared.
    * - If dependencies changed during pause (DIRTY or PENDING flags set),
-   *   the effect executes immediately via notify()
-   * - If no changes occurred, the effect simply becomes active again
+   *   the effect executes immediately via notify().
+   * - If no changes occurred, the effect simply becomes active again.
    *
    * State management:
-   * - Clears PAUSED flag atomically
-   * - Checks for accumulated DIRTY/PENDING flags
-   * - Triggers execution if needed
-   *
-   * @example
-   * ```typescript
-   * const count = signal(0);
-   * const runner = effect(() => console.log(count.value));
+   * - Clears PAUSED flag atomically.
+   * - Checks for accumulated DIRTY/PENDING flags.
+   * - Triggers execution if needed.
    *
-   * runner.effect.pause();
-   * count.value = 1; // Queued
-   * count.value = 2; // Queued
-   * runner.effect.resume(); // Executes once with count.value = 2
-   * ```
+   * @returns {void}
    */
   resume() {
     const flags = this.flag;
@@ -1254,17 +1366,17 @@ var EffectImpl = class {
     }
   }
   /**
-     * Execute the Effect function
-     *
-     * Core execution flow:
-     * 1. Check if active
-     * 2. Clear dirty flag
-     * 3. Start tracking dependencies
-     * 4. Execute user function
-     * 5. End tracking, clean up stale dependencies
-  
-     * @returns The return value of the effect function
-     */
+   * Execute the Effect function.
+   *
+   * Core execution flow:
+   * 1. Check if active
+   * 2. Clear dirty flag
+   * 3. Start tracking dependencies
+   * 4. Execute user function
+   * 5. End tracking, clean up stale dependencies
+   *
+   * @returns {T} The return value of the effect function.
+   */
   run() {
     if (!this._active) {
       return this.fn();
@@ -1283,7 +1395,9 @@ var EffectImpl = class {
     }
   }
   /**
-   * Get or create the job function for this effect
+   * Get or create the job function for this effect.
+   *
+   * @returns {() => void} The job function.
    */
   getJob() {
     if (!this._job) {
@@ -1292,27 +1406,34 @@ var EffectImpl = class {
     return this._job;
   }
   /**
-   * Notify that the Effect needs to execute
+   * Notify that the Effect needs to execute.
    *
    * Called by dependent reactive values.
    * Decides whether to execute immediately or defer based on scheduling strategy.
+   *
+   * @returns {void}
    */
   notify() {
-    var _a5;
+    var _a5, _b2, _c, _d;
     const flags = this.flag;
     if (!this._active || flags & (256 /* PAUSED */ | 512 /* RUNNING */ | 16 /* DIRTY */)) {
       return;
     }
     this.flag = flags | 16 /* DIRTY */;
-    if (this == null ? void 0 : this.onTrigger) {
-      this.onTrigger({
+    if ((_a5 = this.options) == null ? void 0 : _a5.onTrigger) {
+      this.options.onTrigger({
         effect: this,
         target: {},
         type: "set"
       });
     }
-    if (this._flushScheduler) {
-      (_a5 = this._flushScheduler) == null ? void 0 : _a5.call(this);
+    const scheduler = ((_b2 = this.options) == null ? void 0 : _b2.flush) || ((_c = this.options) == null ? void 0 : _c.scheduler);
+    if (scheduler) {
+      if (shared.isFunction(scheduler)) {
+        scheduler(this);
+      } else {
+        (_d = this._flushScheduler) == null ? void 0 : _d.call(this);
+      }
     } else if (isBatching()) {
       queueJob(this.getJob());
     } else {
@@ -1320,16 +1441,19 @@ var EffectImpl = class {
     }
   }
   /**
-   * Stop the Effect
+   * Stop the Effect.
    *
    * After stopping:
-   * - No longer responds to dependency changes
-   * - Disconnects all dependency links
-   * - Clears cached job function
-   * - Calls onStop callback
-   * - Verifies complete cleanup in development mode
+   * - No longer responds to dependency changes.
+   * - Disconnects all dependency links.
+   * - Clears cached job function.
+   * - Calls onStop callback.
+   * - Verifies complete cleanup in development mode.
+   *
+   * @returns {void}
    */
   stop() {
+    var _a5;
     if (!this._active) {
       {
         shared.warn("[Effect] Attempting to stop an already stopped effect.");
@@ -1361,8 +1485,8 @@ var EffectImpl = class {
         );
       }
     }
-    if (this == null ? void 0 : this.onStop) {
-      this.onStop();
+    if ((_a5 = this.options) == null ? void 0 : _a5.onStop) {
+      this.options.onStop();
     }
   }
 };
@@ -1404,16 +1528,16 @@ var _a3;
 _a3 = "_IS_COMPUTED" /* IS_COMPUTED */;
 var ComputedImpl = class {
   /**
-   * Create a Computed instance
+   * Create a Computed instance.
    *
-   * @param getter - The computation function
-   * @param setter - Optional setter function
-   * @param onTrack - Optional debug callback for dependency tracking
-   * @param onTrigger - Optional debug callback for triggers
+   * @param getter - The computation function.
+   * @param setter - Optional setter function.
+   * @param onTrack - Optional debug callback for dependency tracking.
+   * @param onTrigger - Optional debug callback for triggers.
    */
   constructor(getter, setter, onTrack, onTrigger) {
     this.flag = 1 /* MUTABLE */ | 16 /* DIRTY */;
-    // @ts-ignore: used internally by isComputed typeguard
+    //@ts-ignore
     this[_a3] = true;
     //  Cache
     // Use symbol sentinel to distinguish "no value" from undefined/null values
@@ -1424,6 +1548,11 @@ var ComputedImpl = class {
     this.onTrigger = onTrigger;
     this.flag |= 16 /* DIRTY */;
   }
+  /**
+   * Returns the current value.
+   *
+   * @returns {T} The current value.
+   */
   get value() {
     if (activeSub) {
       linkReactiveNode(this, activeSub);
@@ -1447,9 +1576,9 @@ var ComputedImpl = class {
     return this._value;
   }
   /**
-   * Set value (only effective when setter is provided)
+   * Set value (only effective when setter is provided).
    *
-   * @param newValue - The new value
+   * @param newValue - The new value.
    */
   set value(newValue) {
     if (this.setter) {
@@ -1461,9 +1590,9 @@ var ComputedImpl = class {
     }
   }
   /**
-   * Read value without tracking dependencies
+   * Read value without tracking dependencies.
    *
-   * @returns Current value
+   * @returns {T} The current value.
    */
   peek() {
     if (this._value === NO_VALUE) {
@@ -1474,7 +1603,7 @@ var ComputedImpl = class {
   /**
    * Recompute the value
    *
-   * computation logic:
+   *  computation logic:
    * 1. Start tracking dependencies
    * 2. Execute getter function
    * 3. Check if value changed using optimized comparison
@@ -1526,11 +1655,11 @@ var ComputedImpl = class {
     }
   }
   /**
-   * Check if update is needed
+   * Check if update is needed.
    *
    * Internal use, called by reactive system.
    *
-   * @returns true if value changed
+   * @returns {boolean} True if value changed.
    */
   shouldUpdate() {
     const hadValue = this._value !== NO_VALUE;
@@ -1597,6 +1726,9 @@ function createOptionsStore(options) {
     actionCallbacks.forEach((callback) => callback(state2));
   };
   const defaultActions = {
+    /**
+     * Applies a partial patch to the reactive store state.
+     */
     patch$(payload) {
       if (!payload) {
         shared.warn("Patch payload is required");
@@ -1607,6 +1739,9 @@ function createOptionsStore(options) {
       });
       notifySubscribers(reactiveState);
     },
+    /**
+     * Registers a store subscriber callback.
+     */
     subscribe$(callback) {
       if (!callback) {
         shared.warn("Subscribe callback is required");
@@ -1614,9 +1749,15 @@ function createOptionsStore(options) {
       }
       subscriptions.add(callback);
     },
+    /**
+     * Removes a previously registered store subscriber.
+     */
     unsubscribe$(callback) {
       subscriptions.delete(callback);
     },
+    /**
+     * Registers a callback for store action notifications.
+     */
     onAction$(callback) {
       if (!callback) {
         shared.warn("Action callback is required");
@@ -1624,6 +1765,15 @@ function createOptionsStore(options) {
       }
       actionCallbacks.add(callback);
     },
+    /**
+     * Removes a previously registered action callback.
+     */
+    offAction$(callback) {
+      actionCallbacks.delete(callback);
+    },
+    /**
+     * Resets the reactive state back to its initial snapshot.
+     */
     reset$() {
       batch(() => {
         Object.assign(reactiveState, initState);
@@ -1631,15 +1781,31 @@ function createOptionsStore(options) {
       notifySubscribers(reactiveState);
     }
   };
-  const store = __spreadValues(__spreadProps(__spreadValues({}, reactiveState), {
-    state: reactiveState
-  }), defaultActions);
+  const store = {};
+  for (const key of Object.keys(initState)) {
+    Object.defineProperty(store, key, {
+      get: () => reactiveState[key],
+      set: (value) => {
+        reactiveState[key] = value;
+      },
+      enumerable: true,
+      configurable: true
+    });
+  }
+  Object.defineProperty(store, "state", {
+    value: reactiveState,
+    enumerable: true,
+    configurable: true,
+    writable: true
+  });
+  Object.assign(store, defaultActions);
   if (getters) {
     for (const key in getters) {
       const getter = getters[key];
       if (!getter) continue;
+      const getterValue = computed(() => getter.call(store, reactiveState));
       Object.defineProperty(store, key, {
-        get: () => computed(() => getter.call(store, reactiveState)).value,
+        get: () => getterValue.value,
         enumerable: true,
         configurable: true
       });
@@ -1649,11 +1815,11 @@ function createOptionsStore(options) {
     for (const key in actions) {
       const action = actions[key];
       if (action) {
-        Reflect.set(store, key, (...args) => {
-          const result = action.apply(reactiveState, args);
+        store[key] = (...args) => {
+          const result = action.apply(store, args);
           actionCallbacks.forEach((callback) => callback(reactiveState));
           return result;
-        });
+        };
       }
     }
   }
@@ -1670,11 +1836,11 @@ function createClassStore(StoreClass) {
   Object.getOwnPropertyNames(StoreClass.prototype).forEach((key) => {
     const descriptor = Object.getOwnPropertyDescriptor(StoreClass.prototype, key);
     if (descriptor) {
-      if (typeof descriptor.get === "function") {
+      if (shared.isFunction(descriptor.get)) {
         getters[key] = function() {
           return descriptor.get.call(this);
         };
-      } else if (typeof descriptor.value === "function" && key !== "constructor") {
+      } else if (shared.isFunction(descriptor.value) && key !== "constructor") {
         actions[key] = function(...args) {
           return descriptor.value.apply(this, args);
         };
@@ -1699,13 +1865,7 @@ function createStore(storeDefinition) {
     } else {
       options = storeDefinition;
     }
-    const store = createOptionsStore(options);
-    if (shared.isFunction(storeDefinition) && options.actions) {
-      Object.keys(options.actions).forEach((key) => {
-        Reflect.set(store, key, options.actions[key].bind(store));
-      });
-    }
-    return store;
+    return createOptionsStore(options);
   };
 }
 var _a4, _b;
@@ -1717,9 +1877,14 @@ var RefImpl = class extends (_b = SignalImpl, _a4 = "_IS_REF" /* IS_REF */, _b)
    */
   constructor(value) {
     super(value, true);
-    // @ts-ignore: used internally by isRef typeguard
+    // @ts-ignore
     this[_a4] = true;
   }
+  /**
+   * Returns the current value.
+   *
+   * @returns {T} The current value.
+   */
   get value() {
     const sub = activeSub;
     if (sub) {
@@ -1727,6 +1892,11 @@ var RefImpl = class extends (_b = SignalImpl, _a4 = "_IS_REF" /* IS_REF */, _b)
     }
     return this._value;
   }
+  /**
+   * Updates the current value.
+   *
+   * @param newValue - The new value.
+   */
   set value(newValue) {
     if (isSignal(newValue)) {
       newValue = newValue.peek();
@@ -1757,8 +1927,12 @@ function isRef(value) {
   return !!value && !!value["_IS_REF" /* IS_REF */];
 }
 var INITIAL_WATCHER_VALUE = {};
-var cleanupMap = /* @__PURE__ */ new WeakMap();
-function traverse(value, seen = /* @__PURE__ */ new Set()) {
+var _traverseSeen = /* @__PURE__ */ new Set();
+function traverse(value, seen) {
+  if (!seen) {
+    _traverseSeen.clear();
+    seen = _traverseSeen;
+  }
   if (!shared.isObject(value) || seen.has(value)) {
     return value;
   }
@@ -1789,54 +1963,13 @@ function traverse(value, seen = /* @__PURE__ */ new Set()) {
   return value;
 }
 function cloneValue(value) {
-  if (!shared.isObject(value)) {
-    return value;
-  }
-  if (Array.isArray(value)) {
-    return value.map((item) => cloneValue(item));
-  }
-  if (shared.isMap(value)) {
-    const cloned2 = /* @__PURE__ */ new Map();
-    value.forEach((v, k) => {
-      cloned2.set(k, cloneValue(v));
-    });
-    return cloned2;
-  }
-  if (shared.isSet(value)) {
-    const cloned2 = /* @__PURE__ */ new Set();
-    value.forEach((v) => {
-      cloned2.add(cloneValue(v));
-    });
-    return cloned2;
-  }
-  const cloned = {};
-  for (const key of Object.keys(value)) {
-    cloned[key] = cloneValue(value[key]);
-  }
-  return cloned;
+  return value;
 }
-function resolveSource(source) {
-  if (Array.isArray(source)) {
-    return () => source.map((s) => {
-      if (isSignal(s) || isComputed(s)) {
-        return s.value;
-      }
-      if (isReactive(s)) {
-        return traverse(s);
-      }
-      if (shared.isFunction(s)) {
-        return s();
-      }
-      return s;
-    });
-  }
+function resolveSingleSource(source) {
   if (shared.isFunction(source)) {
     return source;
   }
-  if (isSignal(source)) {
-    return () => source.value;
-  }
-  if (shared.isObject(source) && "value" in source) {
+  if (isSignal(source) || isComputed(source)) {
     return () => source.value;
   }
   if (isReactive(source)) {
@@ -1844,6 +1977,13 @@ function resolveSource(source) {
   }
   return () => source;
 }
+function resolveSource(source) {
+  if (Array.isArray(source)) {
+    const getters = source.map((s) => resolveSingleSource(s));
+    return () => getters.map((g) => g());
+  }
+  return resolveSingleSource(source);
+}
 function watch(source, callback, options = {}) {
   const { immediate = false, deep = false } = options;
   let oldValue = INITIAL_WATCHER_VALUE;
@@ -1854,7 +1994,7 @@ function watch(source, callback, options = {}) {
       return;
     }
     const newValue = currentEffect.run();
-    if (shared.hasChanged(newValue, oldValue)) {
+    if (deep || shared.isObject(newValue) || shared.hasChanged(newValue, oldValue)) {
       callback(newValue, oldValue === INITIAL_WATCHER_VALUE ? void 0 : oldValue);
       oldValue = cloneValue(newValue);
     }
@@ -1879,11 +2019,6 @@ function watch(source, callback, options = {}) {
   }
   return () => {
     runner.stop();
-    const cleanups = cleanupMap.get(runner.effect);
-    if (cleanups) {
-      cleanups.forEach((fn) => fn());
-      cleanupMap.delete(runner.effect);
-    }
   };
 }