@rotorsoft/act 1.12.0 → 1.13.0

package/dist/index.js

@@ -19,7 +19,7 @@ import {
   sleep,
   store,
   validate
-} from "./chunk-5PWLGL5E.js";
+} from "./chunk-QCSXT5MB.js";
 import {
   ActorSchema,
   CausationEventSchema,
@@ -271,7 +271,7 @@ var make_restart_candidate_pass = (deps, options) => {
         // names map is sparse — `__snapshot__` key absent when the
         // stream has never been snapshotted (a common case for the
         // restart-candidate signal).
-        snaps: names["__snapshot__"] ?? 0
+        snaps: names.__snapshot__ ?? 0
       });
     },
     drain: () => findings
@@ -333,7 +333,7 @@ var make_snapshot_drift_pass = (deps, options) => {
       candidates.push({
         stream,
         total: count,
-        snaps: names["__snapshot__"] ?? 0
+        snaps: names.__snapshot__ ?? 0
       });
     },
     async finalize(deps2) {
@@ -1618,7 +1618,7 @@ function trace_cycle(logger, leased, fetched, handled, acked, blocked) {
   const detail = leased.map(({ stream, at, retry }) => {
     const f = fetch_by_stream.get(stream);
     const key = f?.source ? `${hue(C_STREAM, stream)}${dim(`<-${f.source}`)}` : hue(C_STREAM, stream);
-    const events = f && f.events.length ? ` ${dim(
+    const events = f?.events.length ? ` ${dim(
       `[${f.events.map(({ id, name }) => `#${id} ${String(name)}`).join(", ")}]`
     )}` : "";
     const acked_at = acked_by_stream.get(stream);
@@ -1968,6 +1968,11 @@ function merge_event_register(target, source) {
     const target_reg = target[event_name];
     if (!target_reg) continue;
     for (const [name, reaction] of source_reg.reactions) {
+      const existing = target_reg.reactions.get(name);
+      if (existing !== void 0 && existing !== reaction)
+        throw new Error(
+          `Duplicate reaction "${name}" for event "${event_name}". Reaction handlers are keyed by function name; rename one of them.`
+        );
       target_reg.reactions.set(name, reaction);
     }
   }
@@ -3418,6 +3423,10 @@ function act() {
           throw new Error(
             `Reaction handler for "${String(event)}" must be a named function`
           );
+        if (registry.events[event].reactions.has(handler.name))
+          throw new Error(
+            `Duplicate reaction "${handler.name}" for event "${String(event)}". Reaction handlers are keyed by function name; rename one of them.`
+          );
         registry.events[event].reactions.set(handler.name, reaction);
         return Object.assign(builder, {
           to(resolver) {
@@ -3534,6 +3543,10 @@ function _projection(target) {
             throw new Error(
               `Projection handler for "${event}" must be a named function`
             );
+          if (register.reactions.has(handler.name))
+            throw new Error(
+              `Duplicate projection handler "${handler.name}" for event "${event}". Projection handlers are keyed by function name; rename one of them.`
+            );
           register.reactions.set(handler.name, reaction);
           const widened = base;
           return Object.assign(widened, {
@@ -3609,6 +3622,10 @@ function slice() {
           throw new Error(
             `Reaction handler for "${String(event)}" must be a named function`
           );
+        if (events[event].reactions.has(handler.name))
+          throw new Error(
+            `Duplicate reaction "${handler.name}" for event "${String(event)}". Reaction handlers are keyed by function name; rename one of them.`
+          );
         events[event].reactions.set(handler.name, reaction);
         return Object.assign(builder, {
           to(resolver) {