@rotorsoft/act 1.31.9 → 1.31.11

package/dist/index.js

@@ -95,11 +95,12 @@ function classify_registry(registry, states) {
           set.add(lane_name);
           event_to_lanes.set(name, set);
         }
-        if (!target_lanes.has(target)) {
-          target_lanes.set(target, lane);
-        } else if ((target_lanes.get(target) ?? void 0) !== (lane ?? void 0)) {
+        const recorded_lane = target_lanes.get(target);
+        if (recorded_lane === void 0) {
+          target_lanes.set(target, lane_name);
+        } else if (recorded_lane !== lane_name) {
           throw new Error(
-            `Stream "${target}" has conflicting lane assignments ("${target_lanes.get(target) ?? "default"}" vs "${lane ?? "default"}")`
+            `Stream "${target}" has conflicting lane assignments ("${recorded_lane}" vs "${lane_name}")`
           );
         }
         const key = `${target}|${source ?? ""}`;
@@ -2243,7 +2244,12 @@ async function load(me, target, callback, patch_fn = bare_patch) {
       event_id: event.id,
       patches,
       snaps
-    }).catch((err) => log().error(err));
+    }).catch(
+      (err) => log().warn(
+        err,
+        "cache set skipped; the next load re-folds from the store"
+      )
+    );
   }
   return {
     event,
@@ -2342,7 +2348,12 @@ async function action(me, action2, target, payload, options, patch_fn = bare_pat
         );
       } catch (error) {
         if (error instanceof ConcurrencyError) {
-          cache().invalidate(stream).catch((err) => log().error(err));
+          cache().invalidate(stream).catch(
+            (err) => log().warn(
+              err,
+              "cache invalidate skipped; a stale checkpoint re-folds from its event_id"
+            )
+          );
         }
         throw error;
       }
@@ -2377,9 +2388,19 @@ async function action(me, action2, target, payload, options, patch_fn = bare_pat
             event_id: snap_event?.id ?? last.event.id,
             patches: snap_event ? 0 : last.patches,
             snaps: snap_event ? last.snaps + 1 : last.snaps
-          }).catch((err) => log().error(err));
+          }).catch(
+            (err) => log().warn(
+              err,
+              "cache set skipped; the next load re-folds from the store"
+            )
+          );
         else
-          cache().invalidate(stream).catch((err) => log().error(err));
+          cache().invalidate(stream).catch(
+            (err) => log().warn(
+              err,
+              "cache invalidate skipped; a stale checkpoint re-folds from its event_id"
+            )
+          );
       }
       return snapshots;
     } catch (error) {
@@ -2653,10 +2674,8 @@ async function run_drain_cycle(ops, registry, batch_handlers, misrouted, handle,
   const acked = await ops.ack(submitted);
   const expected = submitted.filter((l) => l.due === void 0).length;
   if (acked.length < expected)
-    log().error(
-      new Error(
-        `drain: ${expected - acked.length} of ${expected} acks were dropped \u2014 the lease was taken by another worker mid-handler. That work will be redelivered (at-least-once), but persistent drops mean leaseMillis is too short for this handler.`
-      )
+    log().warn(
+      `drain: ${expected - acked.length} of ${expected} acks were dropped \u2014 the lease was taken by another worker mid-handler. That work will be redelivered (at-least-once), but persistent drops mean leaseMillis is too short for this handler.`
     );
   const closeable = handled.filter((h) => h.close !== void 0).map((h) => h.close);
   return { leased, fetched, handled, acked, blocked, closeable };