@axlsdk/axl 0.17.3 → 0.17.4

package/dist/index.d.cts

@@ -2597,6 +2597,10 @@ declare class AxlRuntime extends EventEmitter {
      *  the instance so per-event onTrace handlers don't reach into
      *  `this.config.state` on every emission. */
     private readonly maxEventsPerExecution;
+    /** Tracks executionIds for which we've already warned about a malformed
+     *  `events` field — keeps the log to one line per row instead of one per
+     *  rebuild tick. */
+    private warnedMalformedExecutions;
     constructor(config?: AxlConfig);
     /**
      * Whether `config.trace.redact` is enabled on this runtime. A narrow
@@ -2717,6 +2721,14 @@ declare class AxlRuntime extends EventEmitter {
         captureTraces?: boolean;
     }): Promise<unknown>;
     /** Get all execution info (running + completed + historical). */
+    /** Normalize an `ExecutionInfo` loaded from a `StateStore` so the
+     *  `events: AxlEvent[]` type contract holds for downstream consumers
+     *  (Studio aggregators, REST routes, redaction). Custom or legacy stores
+     *  may persist rows without `events` (or with a non-array value) — without
+     *  this guard, a single malformed row crashes every iterator over
+     *  `exec.events` and takes down dependent features. Mutates the row
+     *  in place; logs at most one warning per `executionId`. */
+    private normalizeStoredExecution;
     getExecutions(): Promise<ExecutionInfo[]>;
     /**
      * Create a WorkflowContext for ad-hoc use (evals, tool testing, prototyping).

package/dist/index.d.ts

@@ -2597,6 +2597,10 @@ declare class AxlRuntime extends EventEmitter {
      *  the instance so per-event onTrace handlers don't reach into
      *  `this.config.state` on every emission. */
     private readonly maxEventsPerExecution;
+    /** Tracks executionIds for which we've already warned about a malformed
+     *  `events` field — keeps the log to one line per row instead of one per
+     *  rebuild tick. */
+    private warnedMalformedExecutions;
     constructor(config?: AxlConfig);
     /**
      * Whether `config.trace.redact` is enabled on this runtime. A narrow
@@ -2717,6 +2721,14 @@ declare class AxlRuntime extends EventEmitter {
         captureTraces?: boolean;
     }): Promise<unknown>;
     /** Get all execution info (running + completed + historical). */
+    /** Normalize an `ExecutionInfo` loaded from a `StateStore` so the
+     *  `events: AxlEvent[]` type contract holds for downstream consumers
+     *  (Studio aggregators, REST routes, redaction). Custom or legacy stores
+     *  may persist rows without `events` (or with a non-array value) — without
+     *  this guard, a single malformed row crashes every iterator over
+     *  `exec.events` and takes down dependent features. Mutates the row
+     *  in place; logs at most one warning per `executionId`. */
+    private normalizeStoredExecution;
     getExecutions(): Promise<ExecutionInfo[]>;
     /**
      * Create a WorkflowContext for ad-hoc use (evals, tool testing, prototyping).

package/dist/index.js

@@ -7485,6 +7485,10 @@ var AxlRuntime = class extends EventEmitter2 {
    *  the instance so per-event onTrace handlers don't reach into
    *  `this.config.state` on every emission. */
   maxEventsPerExecution;
+  /** Tracks executionIds for which we've already warned about a malformed
+   *  `events` field — keeps the log to one line per row instead of one per
+   *  rebuild tick. */
+  warnedMalformedExecutions = /* @__PURE__ */ new Set();
   constructor(config) {
     super();
     this.config = resolveConfig(config ?? {});
@@ -7778,12 +7782,31 @@ var AxlRuntime = class extends EventEmitter2 {
     return result;
   }
   /** Get all execution info (running + completed + historical). */
+  /** Normalize an `ExecutionInfo` loaded from a `StateStore` so the
+   *  `events: AxlEvent[]` type contract holds for downstream consumers
+   *  (Studio aggregators, REST routes, redaction). Custom or legacy stores
+   *  may persist rows without `events` (or with a non-array value) — without
+   *  this guard, a single malformed row crashes every iterator over
+   *  `exec.events` and takes down dependent features. Mutates the row
+   *  in place; logs at most one warning per `executionId`. */
+  normalizeStoredExecution(exec) {
+    if (!Array.isArray(exec.events)) {
+      if (!this.warnedMalformedExecutions.has(exec.executionId)) {
+        this.warnedMalformedExecutions.add(exec.executionId);
+        console.warn(
+          `[axl] StateStore returned execution ${exec.executionId} with non-array events (got ${exec.events === null ? "null" : typeof exec.events}); coercing to []`
+        );
+      }
+      exec.events = [];
+    }
+    return exec;
+  }
   async getExecutions() {
     if (!this.historicalExecutionsLoadPromise && this.stateStore.listExecutions) {
       this.historicalExecutionsLoadPromise = this.stateStore.listExecutions().then((stored) => {
         for (const exec of stored) {
           if (!this.executions.has(exec.executionId) && !this.historicalExecutions.has(exec.executionId)) {
-            this.historicalExecutions.set(exec.executionId, exec);
+            this.historicalExecutions.set(exec.executionId, this.normalizeStoredExecution(exec));
           }
         }
       }).catch(() => {
@@ -8154,8 +8177,9 @@ var AxlRuntime = class extends EventEmitter2 {
     if (this.stateStore.getExecution) {
       const stored = await this.stateStore.getExecution(executionId);
       if (stored) {
-        this.historicalExecutions.set(executionId, stored);
-        return stored;
+        const normalized = this.normalizeStoredExecution(stored);
+        this.historicalExecutions.set(executionId, normalized);
+        return normalized;
       }
     }
     return void 0;