hierarchical-approval 4.1.2 → 4.1.4

package/CHANGELOG.md

@@ -5,6 +5,28 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+## [4.1.4] - 2026-11-01
+
+### Changed
+
+- `InMemorySchedulerAdapter.scheduleAt` now rejects after `shutdown()` with an `ApprovalError`
+  (code `SCHEDULER_SHUTDOWN`) instead of a plain `Error`; the message is unchanged.
+
+### Fixed
+
+- `weekendCalendar` and `businessHoursCalendar` now report an invalid start date as `Invalid Date`
+  in their `ApprovalValidationError` messages instead of failing with a bare `RangeError`.
+
+## [4.1.3] - 2026-10-28
+
+### Fixed
+
+- `weekendCalendar().addBusinessDays()` now throws an `ApprovalValidationError` for an infinite
+  day count, or a count above 36,600, instead of spinning forever.
+- An `engine.on()` error handler that itself throws no longer aborts `emit()` or skips the
+  remaining listeners; the failure is dropped instead of escaping the operation that emitted
+  the event.
+
 ## [4.1.2] - 2026-10-24
 
 ### Fixed

package/dist/index.cjs

@@ -434,11 +434,11 @@ var EventBus = class {
         const result = listener(args[0]);
         if (typeof result?.then === "function") {
           void Promise.resolve(result).catch((err) => {
-            this.onListenerError?.(err, event);
+            this.report(err, event);
           });
         }
       } catch (err) {
-        this.onListenerError?.(err, event);
+        this.report(err, event);
       } finally {
         if (once) this.forget(event, listener, wrapped);
       }
@@ -456,6 +456,17 @@ var EventBus = class {
     perListener.push(wrapped);
     return wrapped;
   }
+  /**
+   * Hands a listener failure to the configured handler. A throwing handler is
+   * swallowed: it would otherwise escape `emit()` and skip the remaining listeners,
+   * defeating the isolation this bus promises.
+   */
+  report(error, event) {
+    try {
+      this.onListenerError?.(error, event);
+    } catch {
+    }
+  }
   /** Drop the bookkeeping for one specific wrapper, identified by reference. */
   forget(event, listener, wrapped) {
     const perEvent = this.wrappers.get(event);
@@ -5214,6 +5225,10 @@ function parseOrThrow(fn) {
 // src/utils/BusinessCalendar.ts
 var DAY_MS = 864e5;
 var MAX_SCAN_DAYS = 3660;
+var MAX_BUSINESS_DAYS = 36600;
+function describeDate(d) {
+  return Number.isNaN(d.getTime()) ? "Invalid Date" : d.toISOString();
+}
 function dayKey(d) {
   const y = d.getFullYear();
   const m = String(d.getMonth() + 1).padStart(2, "0");
@@ -5227,6 +5242,11 @@ function weekendCalendar(options = {}) {
   return {
     addBusinessDays(from, days) {
       if (days <= 0 || Number.isNaN(days)) return new Date(from.getTime());
+      if (days > MAX_BUSINESS_DAYS) {
+        throw new ApprovalValidationError(
+          `weekendCalendar: cannot add ${days} business days to ${describeDate(from)} \u2014 days must be at most ${MAX_BUSINESS_DAYS}.`
+        );
+      }
       const whole = Math.floor(days);
       const fraction = days - whole;
       const cursor = new Date(from.getTime());
@@ -5241,7 +5261,7 @@ function weekendCalendar(options = {}) {
         }
         if (++idleDays >= MAX_SCAN_DAYS) {
           throw new ApprovalValidationError(
-            `weekendCalendar: no business day found within 10 years of ${from.toISOString()} \u2014 check weekendDays and holidays.`
+            `weekendCalendar: no business day found within 10 years of ${describeDate(from)} \u2014 check weekendDays and holidays.`
           );
         }
       }
@@ -5288,7 +5308,7 @@ function businessHoursCalendar(options = {}) {
       return cursor;
     }
     throw new ApprovalValidationError(
-      `businessHoursCalendar: no working day found within 10 years of ${from.toISOString()} \u2014 check weekendDays and holidays.`
+      `businessHoursCalendar: no working day found within 10 years of ${describeDate(from)} \u2014 check weekendDays and holidays.`
     );
   };
   const base = weekendCalendar(options);