@pulse-js/core 0.1.9 → 0.2.0

package/dist/index.cjs

@@ -95,7 +95,7 @@ function guardFail(reason) {
 function guardOk(value) {
   return value;
 }
-function guard(nameOrFn, fn, _internalOffset = 3) {
+function guard(nameOrFn, fn) {
   const name = typeof nameOrFn === "string" ? nameOrFn : void 0;
   const evaluator = typeof nameOrFn === "function" ? nameOrFn : fn;
   if (!evaluator) {
@@ -295,53 +295,23 @@ var Registry = class {
   listeners = /* @__PURE__ */ new Set();
   currentGeneration = 0;
   cleanupScheduled = false;
-  autoNameCache = /* @__PURE__ */ new Map();
   /**
-   * Generates a stable auto-name based on source code location.
-   * Uses file path and line number to ensure the same location always gets the same name.
-   * Cached to avoid repeated stack trace parsing.
-   */
-  generateAutoName(type, offset = 3) {
-    const err = new Error();
-    const stack = err.stack?.split("\n") || [];
-    let callSite = stack[offset]?.trim() || "";
-    const cacheKey = `${type}:${callSite}`;
-    if (this.autoNameCache.has(cacheKey)) {
-      return this.autoNameCache.get(cacheKey);
-    }
-    const match = callSite.match(/([^/\\]+)\.(?:ts|tsx|js|jsx):(\d+):\d+/);
-    let name;
-    if (match) {
-      const filename = match[1];
-      const line = match[2];
-      if (filename && line) {
-        name = `${type}@${filename}:${line}`;
-      } else {
-        name = `${type}#${Math.random().toString(36).substring(2, 7)}`;
-      }
-    } else {
-      name = `${type}#${Math.random().toString(36).substring(2, 7)}`;
-    }
-    this.autoNameCache.set(cacheKey, name);
-    return name;
-  }
-  /**
-   * Increments generation and schedules cleanup of old units.
-   * Called automatically when HMR is detected.
+   * Schedules cleanup of units that weren't re-registered (deleted from code).
    */
   scheduleCleanup() {
     if (this.cleanupScheduled) return;
     this.cleanupScheduled = true;
-    this.currentGeneration++;
     setTimeout(() => {
-      this.cleanupOldGenerations();
+      this.cleanupDeadUnits();
       this.cleanupScheduled = false;
     }, 100);
   }
   /**
-   * Removes units from old generations (likely orphaned by HMR).
+   * Removes units that weren't re-registered in the current generation.
+   * Uses mark-and-sweep: units that were re-registered have current generation,
+   * units that weren't are from old generation and should be removed.
    */
-  cleanupOldGenerations() {
+  cleanupDeadUnits() {
     const toDelete = [];
     this.units.forEach((unit, key) => {
       const gen = unit._generation;
@@ -351,53 +321,46 @@ var Registry = class {
     });
     toDelete.forEach((key) => this.units.delete(key));
     if (toDelete.length > 0) {
-      console.log(`[Pulse] Cleaned up ${toDelete.length} stale units after HMR`);
+      console.log(`[Pulse] Cleaned up ${toDelete.length} deleted units after HMR`);
     }
   }
   /**
-   * Registers a new unit (Source or Guard).
-   * Auto-assigns stable names to unnamed units for HMR stability.
+   * Registers a unit (only if it has an explicit name).
    */
-  register(unit, offset = 3) {
+  register(unit) {
     const unitWithMetadata = unit;
-    let name = unitWithMetadata._name;
+    const name = unitWithMetadata._name;
     if (!name) {
-      const isGuard2 = "state" in unit;
-      name = this.generateAutoName(isGuard2 ? "guard" : "source", offset);
-      unitWithMetadata._name = name;
+      return;
     }
-    unitWithMetadata._generation = this.currentGeneration;
-    if (this.units.has(name)) {
+    const existingUnit = this.units.get(name);
+    if (existingUnit) {
+      const existingGen = existingUnit?._generation;
+      if (existingGen === this.currentGeneration) {
+        unitWithMetadata._generation = this.currentGeneration;
+        this.units.set(name, unit);
+        this.listeners.forEach((l) => l(unit));
+        return;
+      }
+      this.currentGeneration++;
       this.scheduleCleanup();
     }
+    unitWithMetadata._generation = this.currentGeneration;
     this.units.set(name, unit);
     this.listeners.forEach((l) => l(unit));
   }
-  /**
-   * Retrieves all registered units.
-   */
   getAll() {
     return Array.from(this.units.values());
   }
-  /**
-   * Subscribes to new unit registrations.
-   * 
-   * @param listener - Callback receiving the newly registered unit.
-   * @returns Unsubscribe function.
-   */
   onRegister(listener) {
     this.listeners.add(listener);
     return () => {
       this.listeners.delete(listener);
     };
   }
-  /**
-   * Clears all registered units.
-   */
   reset() {
     this.units.clear();
     this.currentGeneration = 0;
-    this.autoNameCache.clear();
   }
 };
 var GLOBAL_KEY = "__PULSE_REGISTRY__";
@@ -408,7 +371,7 @@ if (!globalSymbols[GLOBAL_KEY]) {
 var PulseRegistry = globalSymbols[GLOBAL_KEY];
 
 // src/source.ts
-function source(initialValue, options = {}, _internalOffset = 3) {
+function source(initialValue, options = {}) {
   let value = initialValue;
   const subscribers = /* @__PURE__ */ new Set();
   const dependents = /* @__PURE__ */ new Set();
@@ -447,7 +410,7 @@ function compute(name, dependencies, processor) {
   return guard(name, () => {
     const values = dependencies.map((dep) => typeof dep === "function" ? dep() : dep);
     return processor(...values);
-  }, 4);
+  });
 }
 
 // src/composition.ts
@@ -470,7 +433,7 @@ function guardAll(nameOrGuards, maybeGuards) {
       throw new Error(message);
     }
     return true;
-  }, 4);
+  });
 }
 function guardAny(nameOrGuards, maybeGuards) {
   const name = typeof nameOrGuards === "string" ? nameOrGuards : void 0;
@@ -484,7 +447,7 @@ function guardAny(nameOrGuards, maybeGuards) {
       allFails.push(message);
     }
     throw new Error(allFails.length > 0 ? allFails.join(" and ") : "no conditions met");
-  }, 4);
+  });
 }
 function guardNot(nameOrTarget, maybeTarget) {
   const name = typeof nameOrTarget === "string" ? nameOrTarget : void 0;
@@ -494,7 +457,7 @@ function guardNot(nameOrTarget, maybeTarget) {
       return !target.ok();
     }
     return !target();
-  }, 4);
+  });
 }
 var guardExtensions = {
   all: guardAll,

package/dist/index.d.cts

@@ -186,7 +186,7 @@ declare function guardFail(reason: string | GuardReason): never;
  * Returns the value passed to it.
  */
 declare function guardOk<T>(value: T): T;
-declare function guard<T = boolean>(nameOrFn?: string | (() => T | Promise<T>), fn?: () => T | Promise<T>, _internalOffset?: number): Guard<T>;
+declare function guard<T = boolean>(nameOrFn?: string | (() => T | Promise<T>), fn?: () => T | Promise<T>): Guard<T>;
 
 /**
  * Utility to transform reactive dependencies into a new derived value.
@@ -361,7 +361,7 @@ interface Source<T> {
  * user.set({ name: 'Bob' });
  * ```
  */
-declare function source<T>(initialValue: T, options?: SourceOptions<T>, _internalOffset?: number): Source<T>;
+declare function source<T>(initialValue: T, options?: SourceOptions<T>): Source<T>;
 
 /**
  * Serialized state of guards for transfer from server to client.
@@ -416,49 +416,41 @@ type PulseUnit = Source<any> | Guard<any>;
 /**
  * Root Registry for Pulse.
  *
- * It tracks all registered Units (Sources and Guards) globally, providing
- * the data source for DevTools and HMR stability.
+ * Tracks all registered Units (Sources and Guards) globally for DevTools.
+ *
+ * **IMPORTANT**: Only units with explicit names are registered and visible in DevTools.
+ * Unnamed units work perfectly but are not tracked to avoid HMR instability.
+ *
+ * @example
+ * ```ts
+ * // ✅ Visible in DevTools
+ * const count = source(0, { name: 'count' });
+ *
+ * // ❌ Not visible in DevTools (but works fine)
+ * const temp = source(0);
+ * ```
  */
 declare class Registry {
     private units;
     private listeners;
     private currentGeneration;
     private cleanupScheduled;
-    private autoNameCache;
-    /**
-     * Generates a stable auto-name based on source code location.
-     * Uses file path and line number to ensure the same location always gets the same name.
-     * Cached to avoid repeated stack trace parsing.
-     */
-    generateAutoName(type: 'source' | 'guard', offset?: number): string;
     /**
-     * Increments generation and schedules cleanup of old units.
-     * Called automatically when HMR is detected.
+     * Schedules cleanup of units that weren't re-registered (deleted from code).
      */
     private scheduleCleanup;
     /**
-     * Removes units from old generations (likely orphaned by HMR).
+     * Removes units that weren't re-registered in the current generation.
+     * Uses mark-and-sweep: units that were re-registered have current generation,
+     * units that weren't are from old generation and should be removed.
      */
-    private cleanupOldGenerations;
+    private cleanupDeadUnits;
     /**
-     * Registers a new unit (Source or Guard).
-     * Auto-assigns stable names to unnamed units for HMR stability.
-     */
-    register(unit: PulseUnit, offset?: number): void;
-    /**
-     * Retrieves all registered units.
+     * Registers a unit (only if it has an explicit name).
      */
+    register(unit: PulseUnit): void;
     getAll(): PulseUnit[];
-    /**
-     * Subscribes to new unit registrations.
-     *
-     * @param listener - Callback receiving the newly registered unit.
-     * @returns Unsubscribe function.
-     */
     onRegister(listener: (unit: PulseUnit) => void): () => void;
-    /**
-     * Clears all registered units.
-     */
     reset(): void;
 }
 declare const PulseRegistry: Registry;

package/dist/index.d.ts

@@ -186,7 +186,7 @@ declare function guardFail(reason: string | GuardReason): never;
  * Returns the value passed to it.
  */
 declare function guardOk<T>(value: T): T;
-declare function guard<T = boolean>(nameOrFn?: string | (() => T | Promise<T>), fn?: () => T | Promise<T>, _internalOffset?: number): Guard<T>;
+declare function guard<T = boolean>(nameOrFn?: string | (() => T | Promise<T>), fn?: () => T | Promise<T>): Guard<T>;
 
 /**
  * Utility to transform reactive dependencies into a new derived value.
@@ -361,7 +361,7 @@ interface Source<T> {
  * user.set({ name: 'Bob' });
  * ```
  */
-declare function source<T>(initialValue: T, options?: SourceOptions<T>, _internalOffset?: number): Source<T>;
+declare function source<T>(initialValue: T, options?: SourceOptions<T>): Source<T>;
 
 /**
  * Serialized state of guards for transfer from server to client.
@@ -416,49 +416,41 @@ type PulseUnit = Source<any> | Guard<any>;
 /**
  * Root Registry for Pulse.
  *
- * It tracks all registered Units (Sources and Guards) globally, providing
- * the data source for DevTools and HMR stability.
+ * Tracks all registered Units (Sources and Guards) globally for DevTools.
+ *
+ * **IMPORTANT**: Only units with explicit names are registered and visible in DevTools.
+ * Unnamed units work perfectly but are not tracked to avoid HMR instability.
+ *
+ * @example
+ * ```ts
+ * // ✅ Visible in DevTools
+ * const count = source(0, { name: 'count' });
+ *
+ * // ❌ Not visible in DevTools (but works fine)
+ * const temp = source(0);
+ * ```
  */
 declare class Registry {
     private units;
     private listeners;
     private currentGeneration;
     private cleanupScheduled;
-    private autoNameCache;
-    /**
-     * Generates a stable auto-name based on source code location.
-     * Uses file path and line number to ensure the same location always gets the same name.
-     * Cached to avoid repeated stack trace parsing.
-     */
-    generateAutoName(type: 'source' | 'guard', offset?: number): string;
     /**
-     * Increments generation and schedules cleanup of old units.
-     * Called automatically when HMR is detected.
+     * Schedules cleanup of units that weren't re-registered (deleted from code).
      */
     private scheduleCleanup;
     /**
-     * Removes units from old generations (likely orphaned by HMR).
+     * Removes units that weren't re-registered in the current generation.
+     * Uses mark-and-sweep: units that were re-registered have current generation,
+     * units that weren't are from old generation and should be removed.
      */
-    private cleanupOldGenerations;
+    private cleanupDeadUnits;
     /**
-     * Registers a new unit (Source or Guard).
-     * Auto-assigns stable names to unnamed units for HMR stability.
-     */
-    register(unit: PulseUnit, offset?: number): void;
-    /**
-     * Retrieves all registered units.
+     * Registers a unit (only if it has an explicit name).
      */
+    register(unit: PulseUnit): void;
     getAll(): PulseUnit[];
-    /**
-     * Subscribes to new unit registrations.
-     *
-     * @param listener - Callback receiving the newly registered unit.
-     * @returns Unsubscribe function.
-     */
     onRegister(listener: (unit: PulseUnit) => void): () => void;
-    /**
-     * Clears all registered units.
-     */
     reset(): void;
 }
 declare const PulseRegistry: Registry;

package/dist/index.js

@@ -59,7 +59,7 @@ function guardFail(reason) {
 function guardOk(value) {
   return value;
 }
-function guard(nameOrFn, fn, _internalOffset = 3) {
+function guard(nameOrFn, fn) {
   const name = typeof nameOrFn === "string" ? nameOrFn : void 0;
   const evaluator = typeof nameOrFn === "function" ? nameOrFn : fn;
   if (!evaluator) {
@@ -259,53 +259,23 @@ var Registry = class {
   listeners = /* @__PURE__ */ new Set();
   currentGeneration = 0;
   cleanupScheduled = false;
-  autoNameCache = /* @__PURE__ */ new Map();
   /**
-   * Generates a stable auto-name based on source code location.
-   * Uses file path and line number to ensure the same location always gets the same name.
-   * Cached to avoid repeated stack trace parsing.
-   */
-  generateAutoName(type, offset = 3) {
-    const err = new Error();
-    const stack = err.stack?.split("\n") || [];
-    let callSite = stack[offset]?.trim() || "";
-    const cacheKey = `${type}:${callSite}`;
-    if (this.autoNameCache.has(cacheKey)) {
-      return this.autoNameCache.get(cacheKey);
-    }
-    const match = callSite.match(/([^/\\]+)\.(?:ts|tsx|js|jsx):(\d+):\d+/);
-    let name;
-    if (match) {
-      const filename = match[1];
-      const line = match[2];
-      if (filename && line) {
-        name = `${type}@${filename}:${line}`;
-      } else {
-        name = `${type}#${Math.random().toString(36).substring(2, 7)}`;
-      }
-    } else {
-      name = `${type}#${Math.random().toString(36).substring(2, 7)}`;
-    }
-    this.autoNameCache.set(cacheKey, name);
-    return name;
-  }
-  /**
-   * Increments generation and schedules cleanup of old units.
-   * Called automatically when HMR is detected.
+   * Schedules cleanup of units that weren't re-registered (deleted from code).
    */
   scheduleCleanup() {
     if (this.cleanupScheduled) return;
     this.cleanupScheduled = true;
-    this.currentGeneration++;
     setTimeout(() => {
-      this.cleanupOldGenerations();
+      this.cleanupDeadUnits();
       this.cleanupScheduled = false;
     }, 100);
   }
   /**
-   * Removes units from old generations (likely orphaned by HMR).
+   * Removes units that weren't re-registered in the current generation.
+   * Uses mark-and-sweep: units that were re-registered have current generation,
+   * units that weren't are from old generation and should be removed.
    */
-  cleanupOldGenerations() {
+  cleanupDeadUnits() {
     const toDelete = [];
     this.units.forEach((unit, key) => {
       const gen = unit._generation;
@@ -315,53 +285,46 @@ var Registry = class {
     });
     toDelete.forEach((key) => this.units.delete(key));
     if (toDelete.length > 0) {
-      console.log(`[Pulse] Cleaned up ${toDelete.length} stale units after HMR`);
+      console.log(`[Pulse] Cleaned up ${toDelete.length} deleted units after HMR`);
     }
   }
   /**
-   * Registers a new unit (Source or Guard).
-   * Auto-assigns stable names to unnamed units for HMR stability.
+   * Registers a unit (only if it has an explicit name).
    */
-  register(unit, offset = 3) {
+  register(unit) {
     const unitWithMetadata = unit;
-    let name = unitWithMetadata._name;
+    const name = unitWithMetadata._name;
     if (!name) {
-      const isGuard2 = "state" in unit;
-      name = this.generateAutoName(isGuard2 ? "guard" : "source", offset);
-      unitWithMetadata._name = name;
+      return;
     }
-    unitWithMetadata._generation = this.currentGeneration;
-    if (this.units.has(name)) {
+    const existingUnit = this.units.get(name);
+    if (existingUnit) {
+      const existingGen = existingUnit?._generation;
+      if (existingGen === this.currentGeneration) {
+        unitWithMetadata._generation = this.currentGeneration;
+        this.units.set(name, unit);
+        this.listeners.forEach((l) => l(unit));
+        return;
+      }
+      this.currentGeneration++;
       this.scheduleCleanup();
     }
+    unitWithMetadata._generation = this.currentGeneration;
     this.units.set(name, unit);
     this.listeners.forEach((l) => l(unit));
   }
-  /**
-   * Retrieves all registered units.
-   */
   getAll() {
     return Array.from(this.units.values());
   }
-  /**
-   * Subscribes to new unit registrations.
-   * 
-   * @param listener - Callback receiving the newly registered unit.
-   * @returns Unsubscribe function.
-   */
   onRegister(listener) {
     this.listeners.add(listener);
     return () => {
       this.listeners.delete(listener);
     };
   }
-  /**
-   * Clears all registered units.
-   */
   reset() {
     this.units.clear();
     this.currentGeneration = 0;
-    this.autoNameCache.clear();
   }
 };
 var GLOBAL_KEY = "__PULSE_REGISTRY__";
@@ -372,7 +335,7 @@ if (!globalSymbols[GLOBAL_KEY]) {
 var PulseRegistry = globalSymbols[GLOBAL_KEY];
 
 // src/source.ts
-function source(initialValue, options = {}, _internalOffset = 3) {
+function source(initialValue, options = {}) {
   let value = initialValue;
   const subscribers = /* @__PURE__ */ new Set();
   const dependents = /* @__PURE__ */ new Set();
@@ -411,7 +374,7 @@ function compute(name, dependencies, processor) {
   return guard(name, () => {
     const values = dependencies.map((dep) => typeof dep === "function" ? dep() : dep);
     return processor(...values);
-  }, 4);
+  });
 }
 
 // src/composition.ts
@@ -434,7 +397,7 @@ function guardAll(nameOrGuards, maybeGuards) {
       throw new Error(message);
     }
     return true;
-  }, 4);
+  });
 }
 function guardAny(nameOrGuards, maybeGuards) {
   const name = typeof nameOrGuards === "string" ? nameOrGuards : void 0;
@@ -448,7 +411,7 @@ function guardAny(nameOrGuards, maybeGuards) {
       allFails.push(message);
     }
     throw new Error(allFails.length > 0 ? allFails.join(" and ") : "no conditions met");
-  }, 4);
+  });
 }
 function guardNot(nameOrTarget, maybeTarget) {
   const name = typeof nameOrTarget === "string" ? nameOrTarget : void 0;
@@ -458,7 +421,7 @@ function guardNot(nameOrTarget, maybeTarget) {
       return !target.ok();
     }
     return !target();
-  }, 4);
+  });
 }
 var guardExtensions = {
   all: guardAll,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pulse-js/core",
-  "version": "0.1.9",
+  "version": "0.2.0",
   "module": "dist/index.js",
   "main": "dist/index.cjs",
   "types": "dist/index.d.ts",