@calmo/task-runner 3.4.0 → 3.5.0

package/src/WorkflowExecutor.ts

@@ -41,7 +41,9 @@ export class WorkflowExecutor<TContext> {
 
     // Check if already aborted
     if (signal?.aborted) {
-      this.stateManager.cancelAllPending("Workflow cancelled before execution started.");
+      this.stateManager.cancelAllPending(
+        "Workflow cancelled before execution started."
+      );
       const results = this.stateManager.getResults();
       this.eventBus.emit("workflowEnd", { context: this.context, results });
       return results;
@@ -55,7 +57,7 @@ export class WorkflowExecutor<TContext> {
     };
 
     if (signal) {
-       signal.addEventListener("abort", onAbort);
+      signal.addEventListener("abort", onAbort);
     }
 
     try {
@@ -64,7 +66,8 @@ export class WorkflowExecutor<TContext> {
 
       while (
         this.stateManager.hasPendingTasks() ||
-        this.stateManager.hasRunningTasks()
+        this.stateManager.hasRunningTasks() ||
+        executingPromises.size > 0
       ) {
         // Safety check: if no tasks are running and we still have pending tasks,
         // it means we are stuck (e.g. cycle or unhandled dependency).
@@ -77,10 +80,10 @@ export class WorkflowExecutor<TContext> {
         }
 
         if (signal?.aborted) {
-           this.stateManager.cancelAllPending("Workflow cancelled.");
+          this.stateManager.cancelAllPending("Workflow cancelled.");
         } else {
-           // After a task finishes, check for new work
-           this.processLoop(executingPromises, signal);
+          // After a task finishes, check for new work
+          this.processLoop(executingPromises, signal);
         }
       }
 
@@ -122,17 +125,62 @@ export class WorkflowExecutor<TContext> {
 
       const step = this.readyQueue.shift()!;
 
-      this.stateManager.markRunning(step);
+      const taskPromise = (async () => {
+        try {
+          if (step.condition) {
+            const check = step.condition(this.context);
+            const shouldRun = check instanceof Promise ? await check : check;
+
+            if (signal?.aborted) {
+              this.stateManager.markCompleted(step, {
+                status: "cancelled",
+                message: "Cancelled during condition evaluation.",
+              });
+              return;
+            }
+
+            if (!shouldRun) {
+              const result: TaskResult = {
+                status: "skipped",
+                message: "Skipped by condition evaluation.",
+              };
+              this.stateManager.markSkipped(step, result);
+              return;
+            }
+          }
+        } catch (error) {
+          const result: TaskResult = {
+            status: "failure",
+            message:
+              error instanceof Error
+                ? error.message
+                : "Condition evaluation failed",
+            error: error instanceof Error ? error.message : String(error),
+          };
+          this.stateManager.markCompleted(step, result);
+          return;
+        }
+
+        if (signal?.aborted) {
+          this.stateManager.markCompleted(step, {
+            status: "cancelled",
+            message: "Cancelled before execution started.",
+          });
+          return;
+        }
+
+        this.stateManager.markRunning(step);
 
-      const taskPromise = this.strategy.execute(step, this.context, signal)
-        .then((result) => {
+        await this.strategy
+          .execute(step, this.context, signal)
+          .then((result) => {
             this.stateManager.markCompleted(step, result);
-        })
-        .finally(() => {
-             executingPromises.delete(taskPromise);
-             // When a task finishes, we try to run more
-             this.processLoop(executingPromises, signal);
-        });
+          });
+      })().finally(() => {
+        executingPromises.delete(taskPromise);
+        // When a task finishes, we try to run more
+        this.processLoop(executingPromises, signal);
+      });
 
       executingPromises.add(taskPromise);
     }

package/src/contracts/ITaskGraphValidator.ts

@@ -5,17 +5,17 @@ import { ValidationResult } from "./ValidationResult.js";
  * Defines the interface for a task graph validator.
  */
 export interface ITaskGraphValidator {
-    /**
-     * Validates a given task graph for structural integrity.
-     * @param taskGraph The task graph to validate.
-     * @returns A ValidationResult object indicating the outcome of the validation.
-     */
-    validate(taskGraph: TaskGraph): ValidationResult;
+  /**
+   * Validates a given task graph for structural integrity.
+   * @param taskGraph The task graph to validate.
+   * @returns A ValidationResult object indicating the outcome of the validation.
+   */
+  validate(taskGraph: TaskGraph): ValidationResult;
 
-    /**
-     * Creates a human-readable error message from a validation result.
-     * @param result The validation result containing errors.
-     * @returns A formatted error string.
-     */
-    createErrorMessage(result: ValidationResult): string;
+  /**
+   * Creates a human-readable error message from a validation result.
+   * @param result The validation result containing errors.
+   * @returns A formatted error string.
+   */
+  createErrorMessage(result: ValidationResult): string;
 }

package/src/contracts/ValidationError.ts

@@ -2,10 +2,10 @@
  * Describes a specific validation error found in the task graph.
  */
 export interface ValidationError {
-    /** The type of validation error. */
-    type: "cycle" | "missing_dependency" | "duplicate_task";
-    /** A human-readable message describing the error. */
-    message: string;
-    /** Optional detailed information about the error, e.g., the cycle path, or the task with a missing dependency. */
-    details?: unknown;
+  /** The type of validation error. */
+  type: "cycle" | "missing_dependency" | "duplicate_task";
+  /** A human-readable message describing the error. */
+  message: string;
+  /** Optional detailed information about the error, e.g., the cycle path, or the task with a missing dependency. */
+  details?: unknown;
 }

package/src/contracts/ValidationResult.ts

@@ -4,8 +4,8 @@ import { ValidationError } from "./ValidationError.js";
  * The result of a task graph validation operation.
  */
 export interface ValidationResult {
-    /** True if the graph is valid, false otherwise. */
-    isValid: boolean;
-    /** An array of ValidationError objects if the graph is not valid. Empty if isValid is true. */
-    errors: ValidationError[];
+  /** True if the graph is valid, false otherwise. */
+  isValid: boolean;
+  /** An array of ValidationError objects if the graph is not valid. Empty if isValid is true. */
+  errors: ValidationError[];
 }

package/src/strategies/DryRunExecutionStrategy.ts

@@ -5,9 +5,9 @@ import { TaskResult } from "../TaskResult.js";
 /**
  * Execution strategy that simulates task execution without running the actual logic.
  */
-export class DryRunExecutionStrategy<TContext>
-  implements IExecutionStrategy<TContext>
-{
+export class DryRunExecutionStrategy<
+  TContext,
+> implements IExecutionStrategy<TContext> {
   /**
    * Simulates execution by returning a success result immediately.
    * @param step The task step (ignored).

package/src/strategies/RetryingExecutionStrategy.ts

@@ -5,7 +5,9 @@ import { TaskResult } from "../TaskResult.js";
 /**
  * Execution strategy that retries tasks upon failure based on their retry configuration.
  */
-export class RetryingExecutionStrategy<TContext> implements IExecutionStrategy<TContext> {
+export class RetryingExecutionStrategy<
+  TContext,
+> implements IExecutionStrategy<TContext> {
   constructor(private innerStrategy: IExecutionStrategy<TContext>) {}
 
   async execute(
@@ -30,7 +32,11 @@ export class RetryingExecutionStrategy<TContext> implements IExecutionStrategy<T
 
       const result = await this.innerStrategy.execute(step, context, signal);
 
-      if (result.status === "success" || result.status === "cancelled" || result.status === "skipped") {
+      if (
+        result.status === "success" ||
+        result.status === "cancelled" ||
+        result.status === "skipped"
+      ) {
         return result;
       }
 
@@ -51,13 +57,13 @@ export class RetryingExecutionStrategy<TContext> implements IExecutionStrategy<T
       try {
         await this.sleep(delay, signal);
       } catch (e) {
-      if (signal?.aborted) {
-        return {
-          status: "cancelled",
-          message: "Task cancelled during retry delay",
-        };
-      }
-      throw e;
+        if (signal?.aborted) {
+          return {
+            status: "cancelled",
+            message: "Task cancelled during retry delay",
+          };
+        }
+        throw e;
       }
     }
   }

package/src/strategies/StandardExecutionStrategy.ts

@@ -5,9 +5,9 @@ import { TaskResult } from "../TaskResult.js";
 /**
  * Standard execution strategy that runs the task's run method.
  */
-export class StandardExecutionStrategy<TContext>
-  implements IExecutionStrategy<TContext>
-{
+export class StandardExecutionStrategy<
+  TContext,
+> implements IExecutionStrategy<TContext> {
   async execute(
     step: TaskStep<TContext>,
     context: TContext,
@@ -19,7 +19,7 @@ export class StandardExecutionStrategy<TContext>
       // Check if error is due to abort
       if (
         signal?.aborted &&
-        (e instanceof Error && e.name === "AbortError" || signal.reason === e)
+        ((e instanceof Error && e.name === "AbortError") || signal.reason === e)
       ) {
         return {
           status: "cancelled",