@estjs/signals 0.0.15-beta.1 → 0.0.15-beta.11

package/dist/signals.d.cts

@@ -348,6 +348,7 @@ type SignalType<T> = T extends Signal<infer V> ? V : never;
  * const empty = signal(); // undefined
  * ```
  */
+declare function signal<T>(value: Signal<T>): Signal<T>;
 declare function signal<T>(value?: T): Signal<T>;
 /**
  * Create a new shallow signal with the given initial value.
@@ -363,6 +364,7 @@ declare function signal<T>(value?: T): Signal<T>;
  * // Only state.nested is reactive, not state.nested.value
  * ```
  */
+declare function shallowSignal<T>(value: Signal<T>): Signal<T>;
 declare function shallowSignal<T>(value?: T): Signal<T>;
 /**
  * Type guard to check if a value is a Signal instance.
@@ -992,7 +994,7 @@ declare function startBatch(): void;
  * End batch update
  *
  * Decreases batch depth.
- * When depth reaches zero, flush all queued Effects and clean up.
+ * When depth reaches zero, flush all queued Effects.
  *
  * ## Cleanup Process
  *
@@ -1001,6 +1003,12 @@ declare function startBatch(): void;
  * 2. Job queue is automatically cleared by flushJobs()
  * 3. Temporary flags (QUEUED, DIRTY) are cleared by effect execution
  *
+ * ## Development Mode Checks
+ *
+ * In development mode, this function performs additional validation:
+ * - Detects unbalanced batch calls (endBatch without startBatch)
+ * - Prevents batchDepth from becoming negative
+ * - Provides clear error messages to help debug batch management issues
  */
 declare function endBatch(): void;
 /**

package/dist/signals.d.ts

@@ -348,6 +348,7 @@ type SignalType<T> = T extends Signal<infer V> ? V : never;
  * const empty = signal(); // undefined
  * ```
  */
+declare function signal<T>(value: Signal<T>): Signal<T>;
 declare function signal<T>(value?: T): Signal<T>;
 /**
  * Create a new shallow signal with the given initial value.
@@ -363,6 +364,7 @@ declare function signal<T>(value?: T): Signal<T>;
  * // Only state.nested is reactive, not state.nested.value
  * ```
  */
+declare function shallowSignal<T>(value: Signal<T>): Signal<T>;
 declare function shallowSignal<T>(value?: T): Signal<T>;
 /**
  * Type guard to check if a value is a Signal instance.
@@ -992,7 +994,7 @@ declare function startBatch(): void;
  * End batch update
  *
  * Decreases batch depth.
- * When depth reaches zero, flush all queued Effects and clean up.
+ * When depth reaches zero, flush all queued Effects.
  *
  * ## Cleanup Process
  *
@@ -1001,6 +1003,12 @@ declare function startBatch(): void;
  * 2. Job queue is automatically cleared by flushJobs()
  * 3. Temporary flags (QUEUED, DIRTY) are cleared by effect execution
  *
+ * ## Development Mode Checks
+ *
+ * In development mode, this function performs additional validation:
+ * - Detects unbalanced batch calls (endBatch without startBatch)
+ * - Prevents batchDepth from becoming negative
+ * - Provides clear error messages to help debug batch management issues
  */
 declare function endBatch(): void;
 /**

package/dist/signals.dev.cjs.js

@@ -2,11 +2,6 @@
 
 var shared = require('@estjs/shared');
 
-/**
-* @estjs/signals v0.0.15-beta.1
-* (c) 2023-Present jiangxd <jiangxd2016@gmail.com>
-* @license MIT
-**/
 var __defProp = Object.defineProperty;
 var __defProps = Object.defineProperties;
 var __getOwnPropDescs = Object.getOwnPropertyDescriptors;
@@ -34,11 +29,11 @@ var TriggerOpTypes = {
   DELETE: "DELETE",
   CLEAR: "CLEAR"
 };
-var SIGNAL_KEY = Symbol("Signal_Key" );
-var ARRAY_KEY = Symbol("Array_Key" );
-var COLLECTION_KEY = Symbol("Collection_Key" );
-var WEAK_COLLECTION_KEY = Symbol("WeakCollection_Key" );
-var ARRAY_ITERATE_KEY = Symbol("Array_Iterate_Key" );
+var SIGNAL_KEY = /* @__PURE__ */ Symbol("Signal_Key" );
+var ARRAY_KEY = /* @__PURE__ */ Symbol("Array_Key" );
+var COLLECTION_KEY = /* @__PURE__ */ Symbol("Collection_Key" );
+var WEAK_COLLECTION_KEY = /* @__PURE__ */ Symbol("WeakCollection_Key" );
+var ARRAY_ITERATE_KEY = /* @__PURE__ */ Symbol("Array_Iterate_Key" );
 
 // src/propagation.ts
 function propagate(link) {
@@ -375,9 +370,9 @@ function trigger(target, type, key, newValue) {
     }
   }
   if (type === "ADD" || type === "DELETE" || type === "CLEAR") {
-    const ITERATE_KEY2 = Symbol("iterate");
-    const ARRAY_ITERATE_KEY2 = Symbol("arrayIterate");
-    const iterationKey = Array.isArray(target) ? ARRAY_ITERATE_KEY2 : ITERATE_KEY2;
+    const ITERATE_KEY = /* @__PURE__ */ Symbol("iterate");
+    const ARRAY_ITERATE_KEY2 = /* @__PURE__ */ Symbol("arrayIterate");
+    const iterationKey = Array.isArray(target) ? ARRAY_ITERATE_KEY2 : ITERATE_KEY;
     const iterationDep = depsMap.get(iterationKey);
     if (iterationDep) {
       iterationDep.forEach((effect2) => effects.add(effect2));
@@ -884,44 +879,18 @@ function isReactive(target) {
 }
 function reactive(target) {
   if (isReactive(target)) {
-    {
-      shared.warn(
-        "[Reactive] Target is already reactive. Returning existing reactive proxy to avoid double wrapping."
-      );
-    }
     return target;
   }
   if (isSignal(target)) {
-    {
-      shared.warn(
-        "[Reactive] Creating a reactive proxy from a signal is not recommended. Use the signal directly or access its value property."
-      );
-    }
     return target;
   }
   return reactiveImpl(target);
 }
 function shallowReactive(target) {
   if (isReactive(target)) {
-    {
-      if (isShallow(target)) {
-        shared.warn(
-          "[ShallowReactive] Target is already a shallow reactive proxy. Returning existing proxy to avoid double wrapping."
-        );
-      } else {
-        shared.warn(
-          "[ShallowReactive] Target is already a deep reactive proxy. Cannot convert deep reactive to shallow reactive. Returning existing proxy."
-        );
-      }
-    }
     return target;
   }
   if (isSignal(target)) {
-    {
-      shared.warn(
-        "[ShallowReactive] Creating a reactive proxy from a signal is not recommended. Use the signal directly or access its value property."
-      );
-    }
     return target;
   }
   return reactiveImpl(target, true);
@@ -947,7 +916,7 @@ var SignalImpl = class {
     // Mark whether it's shallow reactive
     // @ts-ignore
     this[_a] = true;
-    this._oldValue = this._rawValue = value;
+    this._rawValue = value;
     if (shallow) {
       this._value = shared.isObject(value) ? shallowReactive(value) : value;
     } else {
@@ -964,7 +933,8 @@ var SignalImpl = class {
     if (sub) {
       linkReactiveNode(this, sub);
     }
-    if (this.flag & 16 /* DIRTY */ && this.shouldUpdate()) {
+    const flags = this.flag;
+    if (flags & 16 /* DIRTY */ && this.shouldUpdate()) {
       const subs = this.subLink;
       if (subs) {
         shallowPropagate2(subs);
@@ -986,7 +956,11 @@ var SignalImpl = class {
     if (!shared.hasChanged(this._rawValue, value)) {
       return;
     }
-    this.flag |= 16 /* DIRTY */;
+    if (!("_oldValue" in this)) {
+      this._oldValue = this._rawValue;
+    }
+    const flags = this.flag;
+    this.flag = flags | 16 /* DIRTY */;
     this._rawValue = value;
     const shallow = this["_IS_SHALLOW" /* IS_SHALLOW */];
     if (shallow) {
@@ -1002,7 +976,12 @@ var SignalImpl = class {
   // Check if the value should be update
   shouldUpdate() {
     this.flag &= -17 /* DIRTY */;
-    return shared.hasChanged(this._oldValue, this._oldValue = this._rawValue);
+    if (!("_oldValue" in this)) {
+      return true;
+    }
+    const changed = shared.hasChanged(this._oldValue, this._rawValue);
+    this._oldValue = this._rawValue;
+    return changed;
   }
   // Get current value without triggering dependency tracking
   peek() {
@@ -1040,11 +1019,6 @@ function signal(value) {
 }
 function shallowSignal(value) {
   if (isSignal(value)) {
-    {
-      shared.warn(
-        "Creating a shallow signal with another signal is not recommended. The value will be unwrapped."
-      );
-    }
     value = value.peek();
   }
   return new SignalImpl(value, true);
@@ -1063,8 +1037,8 @@ function nextTick(fn) {
         try {
           fn();
           resolve();
-        } catch (error6) {
-          reject(error6);
+        } catch (error5) {
+          reject(error5);
         }
       });
     });
@@ -1147,17 +1121,13 @@ function startBatch() {
 function endBatch() {
   if (batchDepth === 0) {
     shared.warn(
-      "[Batch] endBatch() called without matching startBatch(). This may indicate unbalanced batch calls in your code."
+      "[Batch] endBatch() called without matching startBatch(). This indicates unbalanced batch calls in your code. Make sure every startBatch() has a corresponding endBatch(), or use the batch() function which handles this automatically."
     );
     return;
   }
-  if (--batchDepth === 0) {
+  batchDepth--;
+  if (batchDepth === 0) {
     flushJobs();
-    if (batchDepth !== 0) {
-      shared.error(
-        `[Batch] Batch depth is not zero after endBatch(). Current depth: ${batchDepth}. This indicates a bug in batch management.`
-      );
-    }
   }
 }
 function isBatching() {
@@ -1181,7 +1151,7 @@ var EffectImpl = class {
     this.flag = 2 /* WATCHING */ | 16 /* DIRTY */;
     // @ts-ignore
     this[_a2] = true;
-    // ===== State management =====
+    //  State management
     this._active = true;
     this.fn = fn;
     if (options) {
@@ -1299,9 +1269,9 @@ var EffectImpl = class {
     const prevSub = startTracking(this);
     try {
       return this.fn();
-    } catch (error6) {
+    } catch (error5) {
       this.flag |= 16 /* DIRTY */;
-      throw error6;
+      throw error5;
     } finally {
       this.flag &= -1025 /* STOP */;
       endTracking(this, prevSub);
@@ -1427,7 +1397,7 @@ function memoEffect(fn, initialState, options) {
   };
   return effect(effectFn, options);
 }
-var NO_VALUE = Symbol("computed-no-value");
+var NO_VALUE = /* @__PURE__ */ Symbol("computed-no-value");
 var _a3;
 _a3 = "_IS_COMPUTED" /* IS_COMPUTED */;
 var ComputedImpl = class {
@@ -1443,7 +1413,7 @@ var ComputedImpl = class {
     this.flag = 1 /* MUTABLE */ | 16 /* DIRTY */;
     //@ts-ignore
     this[_a3] = true;
-    // ===== Cache =====
+    //  Cache
     // Use symbol sentinel to distinguish "no value" from undefined/null values
     this._value = NO_VALUE;
     this.getter = getter;
@@ -1517,6 +1487,7 @@ var ComputedImpl = class {
     try {
       const newValue = this.getter();
       const flags = this.flag;
+      const subs = this.subLink;
       const clearMask = ~(16 /* DIRTY */ | 32 /* PENDING */);
       const valueChanged = !hadValue || shared.hasChanged(oldValue, newValue);
       if (valueChanged) {
@@ -1531,17 +1502,19 @@ var ComputedImpl = class {
             newValue
           });
         }
-        if (this.subLink) {
-          shallowPropagate(this.subLink);
+        if (subs) {
+          shallowPropagate(subs);
         }
       } else {
         this.flag = flags & clearMask;
       }
     } catch (_error) {
-      this.flag &= -49;
+      const clearMask = -49;
+      this.flag &= clearMask;
+      this.flag |= 16 /* DIRTY */;
       {
         shared.error(
-          "[Computed] Error occurred while computing value. Check your getter function for errors.",
+          "[Computed] Error occurred while computing value.\nThe computed will retry on next access.\nCommon causes:\n  - Accessing undefined properties\n  - Circular dependencies\n  - Exceptions in getter function\nCheck your getter function for errors.",
           _error
         );
       }
@@ -1617,6 +1590,10 @@ function createOptionsStore(options) {
   const reactiveState = reactive(state);
   const subscriptions = /* @__PURE__ */ new Set();
   const actionCallbacks = /* @__PURE__ */ new Set();
+  const notifySubscribers = (state2) => {
+    subscriptions.forEach((callback) => callback(state2));
+    actionCallbacks.forEach((callback) => callback(state2));
+  };
   const defaultActions = {
     patch$(payload) {
       if (!payload) {
@@ -1626,8 +1603,7 @@ function createOptionsStore(options) {
       batch(() => {
         Object.assign(reactiveState, payload);
       });
-      subscriptions.forEach((callback) => callback(reactiveState));
-      actionCallbacks.forEach((callback) => callback(reactiveState));
+      notifySubscribers(reactiveState);
     },
     subscribe$(callback) {
       if (!callback) {
@@ -1650,8 +1626,7 @@ function createOptionsStore(options) {
       batch(() => {
         Object.assign(reactiveState, initState);
       });
-      subscriptions.forEach((callback) => callback(reactiveState));
-      actionCallbacks.forEach((callback) => callback(reactiveState));
+      notifySubscribers(reactiveState);
     }
   };
   const store = __spreadValues(__spreadProps(__spreadValues({}, reactiveState), {
@@ -1661,8 +1636,21 @@ function createOptionsStore(options) {
     for (const key in getters) {
       const getter = getters[key];
       if (getter) {
+        let accessCount = 0;
+        let lastWarnTime = 0;
         Object.defineProperty(store, key, {
           get() {
+            {
+              accessCount++;
+              const now = Date.now();
+              if (accessCount > 100 && now - lastWarnTime > 1e3) {
+                shared.warn(
+                  `Getter '${key}' has been accessed ${accessCount} times. Consider caching the result if the value is used frequently. Note: Getters are computed properties that recalculate on every access.`
+                );
+                lastWarnTime = now;
+                accessCount = 0;
+              }
+            }
             return computed(() => getter.call(store, reactiveState)).value;
           },
           enumerable: true,
@@ -1696,11 +1684,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);
         };
@@ -1720,13 +1708,13 @@ function createStore(storeDefinition) {
   }
   return () => {
     let options;
-    if (typeof storeDefinition === "function") {
+    if (shared.isFunction(storeDefinition)) {
       options = createClassStore(storeDefinition);
     } else {
       options = storeDefinition;
     }
     const store = createOptionsStore(options);
-    if (typeof storeDefinition === "function") {
+    if (shared.isFunction(storeDefinition)) {
       Object.keys(options.actions || {}).forEach((key) => {
         store[key] = options.actions[key].bind(store);
       });
@@ -1768,15 +1756,9 @@ var RefImpl = class extends (_b = SignalImpl, _a4 = "_IS_REF" /* IS_REF */, _b)
 };
 function ref(value = void 0) {
   if (isRef(value)) {
-    {
-      shared.info("Creating a ref with another ref is not recommended. The value will be unwrapped.");
-    }
     return value;
   }
   if (isSignal(value)) {
-    {
-      shared.info("Creating a ref with a signal is not recommended. The value will be unwrapped.");
-    }
     return new RefImpl(value.peek());
   }
   return new RefImpl(value);