@calmo/task-runner 1.2.1 → 1.2.3

package/src/TaskRunner.ts

@@ -2,48 +2,12 @@ import { TaskStep } from "./TaskStep.js";
 import { TaskResult } from "./TaskResult.js";
 import { TaskGraphValidator } from "./TaskGraphValidator.js";
 import { TaskGraph } from "./TaskGraph.js";
+import { RunnerEventPayloads, RunnerEventListener } from "./contracts/RunnerEvents.js";
+import { EventBus } from "./EventBus.js";
+import { WorkflowExecutor } from "./WorkflowExecutor.js";
 
-/**
- * Define the payload for every possible event in the lifecycle.
- */
-export interface RunnerEventPayloads<TContext> {
-  workflowStart: {
-    context: TContext;
-    steps: TaskStep<TContext>[];
-  };
-  workflowEnd: {
-    context: TContext;
-    results: Map<string, TaskResult>;
-  };
-  taskStart: {
-    step: TaskStep<TContext>;
-  };
-  taskEnd: {
-    step: TaskStep<TContext>;
-    result: TaskResult;
-  };
-  taskSkipped: {
-    step: TaskStep<TContext>;
-    result: TaskResult;
-  };
-}
-
-/**
- * A generic listener type that maps the event key to its specific payload.
- */
-export type RunnerEventListener<
-  TContext,
-  K extends keyof RunnerEventPayloads<TContext>,
-> = (data: RunnerEventPayloads<TContext>[K]) => void | Promise<void>;
-
-/**
- * Helper type for the listeners map to avoid private access issues in generic contexts.
- */
-type ListenerMap<TContext> = {
-  [K in keyof RunnerEventPayloads<TContext>]?: Set<
-    RunnerEventListener<TContext, K>
-  >;
-};
+// Re-export types for backward compatibility
+export { RunnerEventPayloads, RunnerEventListener };
 
 /**
  * The main class that orchestrates the execution of a list of tasks
@@ -51,8 +15,7 @@ type ListenerMap<TContext> = {
  * @template TContext The shape of the shared context object.
  */
 export class TaskRunner<TContext> {
-  private running = new Set<string>();
-  private listeners: ListenerMap<TContext> = {};
+  private eventBus = new EventBus<TContext>();
   private validator = new TaskGraphValidator();
 
   /**
@@ -69,15 +32,7 @@ export class TaskRunner<TContext> {
     event: K,
     callback: RunnerEventListener<TContext, K>
   ): void {
-    if (!this.listeners[event]) {
-      // Type assertion needed because TypeScript cannot verify that the generic K
-      // matches the specific key in the mapped type during assignment.
-      this.listeners[event] = new Set() as unknown as ListenerMap<TContext>[K];
-    }
-    // Type assertion needed to tell TS that this specific Set matches the callback type
-    (this.listeners[event] as Set<RunnerEventListener<TContext, K>>).add(
-      callback
-    );
+    this.eventBus.on(event, callback);
   }
 
   /**
@@ -89,38 +44,7 @@ export class TaskRunner<TContext> {
     event: K,
     callback: RunnerEventListener<TContext, K>
   ): void {
-    if (this.listeners[event]) {
-      (this.listeners[event] as Set<RunnerEventListener<TContext, K>>).delete(
-        callback
-      );
-    }
-  }
-
-  /**
-   * Emit an event to all subscribers.
-   * @param event The event name.
-   * @param data The payload for the event.
-   */
-  private emit<K extends keyof RunnerEventPayloads<TContext>>(
-    event: K,
-    data: RunnerEventPayloads<TContext>[K]
-  ): void {
-    const listeners = this.listeners[event] as
-      | Set<RunnerEventListener<TContext, K>>
-      | undefined;
-    if (listeners) {
-      for (const listener of listeners) {
-        try {
-          listener(data);
-        } catch (error) {
-          // Prevent listener errors from bubbling up
-          console.error(
-            `Error in event listener for ${String(event)}:`,
-            error
-          );
-        }
-      }
-    }
+    this.eventBus.off(event, callback);
   }
 
   /**
@@ -141,119 +65,10 @@ export class TaskRunner<TContext> {
 
     const validationResult = this.validator.validate(taskGraph);
     if (!validationResult.isValid) {
-      // Construct error message compatible with legacy tests
-      const affectedTasks = new Set<string>();
-      const errorDetails: string[] = [];
-
-      for (const error of validationResult.errors) {
-        errorDetails.push(error.message);
-        switch (error.type) {
-          case "cycle": {
-            // details is { cyclePath: string[] }
-            const path = (error.details as { cyclePath: string[] }).cyclePath;
-            // The last element duplicates the first in the path representation, so valid unique tasks are slice(0, -1) or just all as Set handles uniq
-            path.forEach((t) => affectedTasks.add(t));
-            break;
-          }
-          case "missing_dependency": {
-            // details is { taskId: string, missingDependencyId: string }
-            const d = error.details as { taskId: string };
-            affectedTasks.add(d.taskId);
-            break;
-          }
-          case "duplicate_task": {
-            const d = error.details as { taskId: string };
-            affectedTasks.add(d.taskId);
-            break;
-          }
-        }
-      }
-
-      // Legacy error format: "Circular dependency or missing dependency detected. Unable to run tasks: A, B"
-      const taskList = Array.from(affectedTasks).join(", ");
-      const legacyMessage = `Circular dependency or missing dependency detected. Unable to run tasks: ${taskList}`;
-      const detailedMessage = `Task graph validation failed: ${errorDetails.join("; ")}`;
-
-      // Combine them to satisfy both legacy tests (checking for legacy message) and new requirements (clear details)
-      throw new Error(`${legacyMessage} | ${detailedMessage}`);
-    }
-
-    this.emit("workflowStart", { context: this.context, steps });
-
-    const results = new Map<string, TaskResult>();
-    const executingPromises = new Set<Promise<void>>();
-
-    // Helper to process pending steps and launch ready ones
-    const processPendingSteps = () => {
-      const pendingSteps = steps.filter(
-        (step) => !results.has(step.name) && !this.running.has(step.name)
-      );
-
-      // 1. Identify and mark skipped tasks
-      for (const step of pendingSteps) {
-        const deps = step.dependencies ?? [];
-        const failedDep = deps.find(
-          (dep) => results.has(dep) && results.get(dep)?.status !== "success"
-        );
-        if (failedDep) {
-          const result: TaskResult = {
-            status: "skipped",
-            message: `Skipped due to failed dependency: ${failedDep}`,
-          };
-          results.set(step.name, result);
-          this.emit("taskSkipped", { step, result });
-        }
-      }
-
-      // Re-filter pending steps as some might have been skipped above
-      const readySteps = steps.filter((step) => {
-        if (results.has(step.name) || this.running.has(step.name)) return false;
-        const deps = step.dependencies ?? [];
-        return deps.every(
-          (dep) => results.has(dep) && results.get(dep)?.status === "success"
-        );
-      });
-
-      // 2. Launch ready tasks
-      for (const step of readySteps) {
-        this.running.add(step.name);
-        this.emit("taskStart", { step });
-
-        const taskPromise = (async () => {
-          try {
-            const result = await step.run(this.context);
-            results.set(step.name, result);
-          } catch (e) {
-            results.set(step.name, {
-              status: "failure",
-              error: e instanceof Error ? e.message : String(e),
-            });
-          } finally {
-            this.running.delete(step.name);
-            const result = results.get(step.name)!;
-            this.emit("taskEnd", { step, result });
-          }
-        })();
-
-        // Wrap the task promise to ensure we can track it in the Set
-        const trackedPromise = taskPromise.then(() => {
-          executingPromises.delete(trackedPromise);
-        });
-        executingPromises.add(trackedPromise);
-      }
-    };
-
-    // Initial check to start independent tasks
-    processPendingSteps();
-
-    while (results.size < steps.length && executingPromises.size > 0) {
-      // Wait for the next task to finish
-      await Promise.race(executingPromises);
-      // After a task finishes, check for new work
-      processPendingSteps();
+      throw new Error(this.validator.createErrorMessage(validationResult));
     }
 
-    this.emit("workflowEnd", { context: this.context, results });
-    return results;
+    const executor = new WorkflowExecutor(this.context, this.eventBus);
+    return executor.execute(steps);
   }
 }

package/src/WorkflowExecutor.ts

@@ -0,0 +1,124 @@
+import { TaskStep } from "./TaskStep.js";
+import { TaskResult } from "./TaskResult.js";
+import { EventBus } from "./EventBus.js";
+
+/**
+ * Handles the execution of the workflow steps.
+ * @template TContext The shape of the shared context object.
+ */
+export class WorkflowExecutor<TContext> {
+  private running = new Set<string>();
+
+  /**
+   * @param context The shared context object.
+   * @param eventBus The event bus to emit events.
+   */
+  constructor(
+    private context: TContext,
+    private eventBus: EventBus<TContext>
+  ) {}
+
+  /**
+   * Executes the given steps.
+   * @param steps The list of steps to execute.
+   * @returns A Promise that resolves to a map of task results.
+   */
+  async execute(steps: TaskStep<TContext>[]): Promise<Map<string, TaskResult>> {
+    this.eventBus.emit("workflowStart", { context: this.context, steps });
+
+    const results = new Map<string, TaskResult>();
+    const executingPromises = new Set<Promise<void>>();
+    const pendingSteps = new Set(steps);
+
+    // Initial pass
+    this.processQueue(pendingSteps, results, executingPromises);
+
+    while (results.size < steps.length && executingPromises.size > 0) {
+      // Wait for the next task to finish
+      await Promise.race(executingPromises);
+      // After a task finishes, check for new work
+      this.processQueue(pendingSteps, results, executingPromises);
+    }
+
+    this.eventBus.emit("workflowEnd", { context: this.context, results });
+    return results;
+  }
+
+  /**
+   * Logic to identify tasks that can be started or must be skipped.
+   * Iterate only over pending steps to avoid O(N^2) checks on completed tasks.
+   */
+  private processQueue(
+    pendingSteps: Set<TaskStep<TContext>>,
+    results: Map<string, TaskResult>,
+    executingPromises: Set<Promise<void>>
+  ): void {
+    const toRemove: TaskStep<TContext>[] = [];
+    const toRun: TaskStep<TContext>[] = [];
+
+    for (const step of pendingSteps) {
+      const deps = step.dependencies ?? [];
+      let blocked = false;
+      let failedDep: string | undefined;
+
+      for (const dep of deps) {
+        const depResult = results.get(dep);
+        if (!depResult) {
+          // Dependency not finished yet
+          blocked = true;
+        } else if (depResult.status !== "success") {
+          failedDep = dep;
+          break;
+        }
+      }
+
+      if (failedDep) {
+        const result: TaskResult = {
+          status: "skipped",
+          message: `Skipped due to failed dependency: ${failedDep}`,
+        };
+        results.set(step.name, result);
+        this.eventBus.emit("taskSkipped", { step, result });
+        toRemove.push(step);
+      } else if (!blocked) {
+        toRun.push(step);
+        toRemove.push(step);
+      }
+    }
+
+    // Cleanup pending set
+    for (const step of toRemove) {
+      pendingSteps.delete(step);
+    }
+
+    // Execute ready tasks
+    for (const step of toRun) {
+      const taskPromise = this.runStep(step, results).then(() => {
+        executingPromises.delete(taskPromise);
+      });
+      executingPromises.add(taskPromise);
+    }
+  }
+
+  /**
+   * Handles the lifecycle of a single task execution.
+   */
+  private async runStep(step: TaskStep<TContext>, results: Map<string, TaskResult>): Promise<void> {
+    this.running.add(step.name);
+    this.eventBus.emit("taskStart", { step });
+
+    try {
+      const result = await step.run(this.context);
+      results.set(step.name, result);
+    } catch (e) {
+      results.set(step.name, {
+        status: "failure",
+        error: e instanceof Error ? e.message : String(e),
+      });
+    } finally {
+      this.running.delete(step.name);
+      const result = results.get(step.name)!;
+      this.eventBus.emit("taskEnd", { step, result });
+    }
+  }
+}

package/src/contracts/ITaskGraphValidator.ts

@@ -11,4 +11,11 @@ export interface ITaskGraphValidator {
      * @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;
 }

package/src/contracts/RunnerEvents.ts

@@ -0,0 +1,44 @@
+import { TaskStep } from "../TaskStep.js";
+import { TaskResult } from "../TaskResult.js";
+
+/**
+ * Define the payload for every possible event in the lifecycle.
+ */
+export interface RunnerEventPayloads<TContext> {
+  workflowStart: {
+    context: TContext;
+    steps: TaskStep<TContext>[];
+  };
+  workflowEnd: {
+    context: TContext;
+    results: Map<string, TaskResult>;
+  };
+  taskStart: {
+    step: TaskStep<TContext>;
+  };
+  taskEnd: {
+    step: TaskStep<TContext>;
+    result: TaskResult;
+  };
+  taskSkipped: {
+    step: TaskStep<TContext>;
+    result: TaskResult;
+  };
+}
+
+/**
+ * A generic listener type that maps the event key to its specific payload.
+ */
+export type RunnerEventListener<
+  TContext,
+  K extends keyof RunnerEventPayloads<TContext>,
+> = (data: RunnerEventPayloads<TContext>[K]) => void | Promise<void>;
+
+/**
+ * Helper type for the listeners map.
+ */
+export type ListenerMap<TContext> = {
+  [K in keyof RunnerEventPayloads<TContext>]?: Set<
+    RunnerEventListener<TContext, K>
+  >;
+};

package/test-report.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
-<testsuites name="vitest tests" tests="31" failures="0" errors="0" time="0.154711609">
-    <testsuite name="tests/ComplexScenario.test.ts" timestamp="2026-01-18T05:09:17.318Z" hostname="runnervmmtnos" tests="2" failures="0" errors="0" skipped="0" time="0.009405379">
-        <testcase classname="tests/ComplexScenario.test.ts" name="Complex Scenario Integration Tests &gt; 1. All steps execute when all succeed and context is hydrated" time="0.005631667">
+<testsuites name="vitest tests" tests="35" failures="0" errors="0" time="0.22304865">
+    <testsuite name="tests/ComplexScenario.test.ts" timestamp="2026-01-18T14:25:37.497Z" hostname="runnervmmtnos" tests="2" failures="0" errors="0" skipped="0" time="0.010231018">
+        <testcase classname="tests/ComplexScenario.test.ts" name="Complex Scenario Integration Tests &gt; 1. All steps execute when all succeed and context is hydrated" time="0.006235483">
             <system-out>
 Running StepA
 Running StepB
@@ -15,73 +15,85 @@ Running StepG
 
             </system-out>
         </testcase>
-        <testcase classname="tests/ComplexScenario.test.ts" name="Complex Scenario Integration Tests &gt; 2. The &apos;skip&apos; propagation works as intended if something breaks up in the tree" time="0.001163698">
+        <testcase classname="tests/ComplexScenario.test.ts" name="Complex Scenario Integration Tests &gt; 2. The &apos;skip&apos; propagation works as intended if something breaks up in the tree" time="0.00138321">
         </testcase>
     </testsuite>
-    <testsuite name="tests/TaskGraphValidator.test.ts" timestamp="2026-01-18T05:09:17.319Z" hostname="runnervmmtnos" tests="9" failures="0" errors="0" skipped="0" time="0.009209222">
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should be instantiated" time="0.001860142">
+    <testsuite name="tests/EventBus.test.ts" timestamp="2026-01-18T14:25:37.498Z" hostname="runnervmmtnos" tests="1" failures="0" errors="0" skipped="0" time="0.018978727">
+        <testcase classname="tests/EventBus.test.ts" name="EventBus &gt; should handle async listeners throwing errors without crashing" time="0.016583628">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should return valid result for empty graph" time="0.001425479">
+    </testsuite>
+    <testsuite name="tests/TaskGraphValidator.test.ts" timestamp="2026-01-18T14:25:37.499Z" hostname="runnervmmtnos" tests="9" failures="0" errors="0" skipped="0" time="0.007340745">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should be instantiated" time="0.001387989">
+        </testcase>
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should return valid result for empty graph" time="0.001089122">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect duplicate tasks" time="0.001669406">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect duplicate tasks" time="0.001244392">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect missing dependencies" time="0.000439783">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect missing dependencies" time="0.000319586">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect cycles" time="0.000420086">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect cycles" time="0.000454157">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should return valid for a correct graph" time="0.00036332">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should return valid for a correct graph" time="0.000401499">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should not detect cycles if missing dependencies are present" time="0.000300223">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should not detect cycles if missing dependencies are present" time="0.000238875">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect more complex cycles" time="0.000247734">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect more complex cycles" time="0.000207848">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should handle cycle detection with no cycles but shared dependencies" time="0.00022428">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should handle cycle detection with no cycles but shared dependencies" time="0.000228516">
         </testcase>
     </testsuite>
-    <testsuite name="tests/TaskRunner.test.ts" timestamp="2026-01-18T05:09:17.321Z" hostname="runnervmmtnos" tests="10" failures="0" errors="0" skipped="0" time="0.112022405">
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should run tasks in the correct sequential order" time="0.003955215">
+    <testsuite name="tests/TaskRunner.test.ts" timestamp="2026-01-18T14:25:37.501Z" hostname="runnervmmtnos" tests="10" failures="0" errors="0" skipped="0" time="0.111638951">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should run tasks in the correct sequential order" time="0.002716698">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle an empty list of tasks gracefully" time="0.000433943">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle an empty list of tasks gracefully" time="0.000317112">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should run independent tasks in parallel" time="0.100725915">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should run independent tasks in parallel" time="0.10065072">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should skip dependent tasks if a root task fails" time="0.000326772">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should skip dependent tasks if a root task fails" time="0.000465769">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should throw an error for &apos;circular dependency&apos;" time="0.002189108">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should throw an error for &apos;circular dependency&apos;" time="0.002694306">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should throw an error for &apos;missing dependency&apos;" time="0.000373049">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should throw an error for &apos;missing dependency&apos;" time="0.00041795">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle tasks that throw an error during execution" time="0.000375122">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle tasks that throw an error during execution" time="0.000376783">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should skip tasks whose dependencies are skipped" time="0.000419075">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should skip tasks whose dependencies are skipped" time="0.000786016">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle tasks that throw a non-Error object during execution" time="0.000377657">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle tasks that throw a non-Error object during execution" time="0.000431165">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle duplicate steps where one gets skipped due to failed dependency" time="0.000463308">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle duplicate steps where one gets skipped due to failed dependency" time="0.000468434">
         </testcase>
     </testsuite>
-    <testsuite name="tests/TaskRunnerEvents.test.ts" timestamp="2026-01-18T05:09:17.324Z" hostname="runnervmmtnos" tests="7" failures="0" errors="0" skipped="0" time="0.016480165">
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire all lifecycle events in a successful run" time="0.005054182">
+    <testsuite name="tests/TaskRunnerEvents.test.ts" timestamp="2026-01-18T14:25:37.503Z" hostname="runnervmmtnos" tests="7" failures="0" errors="0" skipped="0" time="0.012381248">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire all lifecycle events in a successful run" time="0.00382187">
+        </testcase>
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire taskSkipped event when dependency fails" time="0.003792205">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire taskSkipped event when dependency fails" time="0.005669685">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should not crash if a listener throws an error" time="0.001438714">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should not crash if a listener throws an error" time="0.001869079">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire workflow events even for empty step list" time="0.000282257">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire workflow events even for empty step list" time="0.000419856">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should handle unsubscribe correctly" time="0.000443788">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should handle unsubscribe correctly" time="0.00057675">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should safely handle off() when no listeners exist" time="0.000271827">
+        </testcase>
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should support multiple listeners for the same event" time="0.000377915">
+        </testcase>
+    </testsuite>
+    <testsuite name="tests/WorkflowExecutor.test.ts" timestamp="2026-01-18T14:25:37.504Z" hostname="runnervmmtnos" tests="3" failures="0" errors="0" skipped="0" time="0.057149901">
+        <testcase classname="tests/WorkflowExecutor.test.ts" name="WorkflowExecutor &gt; should execute steps sequentially when dependencies exist" time="0.003667222">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should safely handle off() when no listeners exist" time="0.0002619">
+        <testcase classname="tests/WorkflowExecutor.test.ts" name="WorkflowExecutor &gt; should skip dependent steps if dependency fails" time="0.000479825">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should support multiple listeners for the same event" time="0.000414686">
+        <testcase classname="tests/WorkflowExecutor.test.ts" name="WorkflowExecutor &gt; should run independent steps in parallel" time="0.05083559">
         </testcase>
     </testsuite>
-    <testsuite name="tests/integration.test.ts" timestamp="2026-01-18T05:09:17.325Z" hostname="runnervmmtnos" tests="3" failures="0" errors="0" skipped="0" time="0.007594438">
-        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for duplicate tasks" time="0.004519361">
+    <testsuite name="tests/integration.test.ts" timestamp="2026-01-18T14:25:37.505Z" hostname="runnervmmtnos" tests="3" failures="0" errors="0" skipped="0" time="0.00532806">
+        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for duplicate tasks" time="0.003098401">
         </testcase>
-        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for missing dependencies" time="0.000476853">
+        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for missing dependencies" time="0.000340456">
         </testcase>
-        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for cycles" time="0.000463618">
+        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for cycles" time="0.000287436">
         </testcase>
     </testsuite>
 </testsuites>