@calmo/task-runner 1.2.3 → 3.0.0

package/src/WorkflowExecutor.ts

@@ -1,124 +1,120 @@
 import { TaskStep } from "./TaskStep.js";
 import { TaskResult } from "./TaskResult.js";
 import { EventBus } from "./EventBus.js";
+import { TaskStateManager } from "./TaskStateManager.js";
+import { IExecutionStrategy } from "./strategies/IExecutionStrategy.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.
+   * @param stateManager Manages execution state.
+   * @param strategy Execution strategy.
    */
   constructor(
     private context: TContext,
-    private eventBus: EventBus<TContext>
+    private eventBus: EventBus<TContext>,
+    private stateManager: TaskStateManager<TContext>,
+    private strategy: IExecutionStrategy<TContext>
   ) {}
 
   /**
    * Executes the given steps.
    * @param steps The list of steps to execute.
+   * @param signal Optional AbortSignal for cancellation.
    * @returns A Promise that resolves to a map of task results.
    */
-  async execute(steps: TaskStep<TContext>[]): Promise<Map<string, TaskResult>> {
+  async execute(
+    steps: TaskStep<TContext>[],
+    signal?: AbortSignal
+  ): Promise<Map<string, TaskResult>> {
     this.eventBus.emit("workflowStart", { context: this.context, steps });
+    this.stateManager.initialize(steps);
+
+    // Check if already aborted
+    if (signal?.aborted) {
+      this.stateManager.cancelAllPending("Workflow cancelled before execution started.");
+      const results = this.stateManager.getResults();
+      this.eventBus.emit("workflowEnd", { context: this.context, results });
+      return results;
+    }
 
-    const results = new Map<string, TaskResult>();
     const executingPromises = new Set<Promise<void>>();
-    const pendingSteps = new Set(steps);
 
-    // Initial pass
-    this.processQueue(pendingSteps, results, executingPromises);
+    const onAbort = () => {
+      // Mark all pending tasks as cancelled
+      this.stateManager.cancelAllPending("Workflow cancelled.");
+    };
 
-    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);
+    if (signal) {
+       signal.addEventListener("abort", onAbort);
     }
 
-    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;
+    try {
+      // Initial pass
+      this.processLoop(executingPromises, signal);
 
-      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;
+      while (
+        this.stateManager.hasPendingTasks() ||
+        this.stateManager.hasRunningTasks()
+      ) {
+        // Safety check: if no tasks are running and we still have pending tasks,
+        // it means we are stuck (e.g. cycle or unhandled dependency).
+        // Since valid graphs shouldn't have this, we break to avoid infinite loop.
+        if (executingPromises.size === 0) {
           break;
+        } else {
+          // Wait for the next task to finish
+          await Promise.race(executingPromises);
         }
-      }
 
-      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);
+        if (signal?.aborted) {
+           this.stateManager.cancelAllPending("Workflow cancelled.");
+        } else {
+           // After a task finishes, check for new work
+           this.processLoop(executingPromises, signal);
+        }
       }
-    }
 
-    // Cleanup pending set
-    for (const step of toRemove) {
-      pendingSteps.delete(step);
-    }
+      // Ensure everything is accounted for (e.g. if loop exited early)
+      this.stateManager.cancelAllPending("Workflow cancelled.");
 
-    // Execute ready tasks
-    for (const step of toRun) {
-      const taskPromise = this.runStep(step, results).then(() => {
-        executingPromises.delete(taskPromise);
-      });
-      executingPromises.add(taskPromise);
+      const results = this.stateManager.getResults();
+      this.eventBus.emit("workflowEnd", { context: this.context, results });
+      return results;
+    } finally {
+      if (signal) {
+        signal.removeEventListener("abort", onAbort);
+      }
     }
   }
 
   /**
-   * Handles the lifecycle of a single task execution.
+   * Logic to identify tasks that can be started and run them.
    */
-  private async runStep(step: TaskStep<TContext>, results: Map<string, TaskResult>): Promise<void> {
-    this.running.add(step.name);
-    this.eventBus.emit("taskStart", { step });
+  private processLoop(
+    executingPromises: Set<Promise<void>>,
+    signal?: AbortSignal
+  ): void {
+    const toRun = this.stateManager.processDependencies();
 
-    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 });
+    // Execute ready tasks
+    for (const step of toRun) {
+      this.stateManager.markRunning(step);
+
+      const taskPromise = this.strategy.execute(step, this.context, signal)
+        .then((result) => {
+            this.stateManager.markCompleted(step, result);
+        })
+        .finally(() => {
+             executingPromises.delete(taskPromise);
+        });
+
+      executingPromises.add(taskPromise);
     }
   }
 }

package/src/index.ts

@@ -1,4 +1,8 @@
 export { TaskRunner } from "./TaskRunner.js";
+export { TaskRunnerBuilder } from "./TaskRunnerBuilder.js";
+export { TaskStateManager } from "./TaskStateManager.js";
+export { StandardExecutionStrategy } from "./strategies/StandardExecutionStrategy.js";
+export type { IExecutionStrategy } from "./strategies/IExecutionStrategy.js";
 export type { TaskStep } from "./TaskStep.js";
 export type { TaskResult } from "./TaskResult.js";
 export type { TaskStatus } from "./TaskStatus.js";

package/src/strategies/IExecutionStrategy.ts

@@ -0,0 +1,21 @@
+import { TaskStep } from "../TaskStep.js";
+import { TaskResult } from "../TaskResult.js";
+
+/**
+ * Interface for task execution strategies.
+ * Allows different execution behaviors (e.g., standard, dry-run, debugging).
+ */
+export interface IExecutionStrategy<TContext> {
+  /**
+   * Executes a single task step.
+   * @param step The task step to execute.
+   * @param context The shared context.
+   * @param signal Optional abort signal.
+   * @returns A promise resolving to the task result.
+   */
+  execute(
+    step: TaskStep<TContext>,
+    context: TContext,
+    signal?: AbortSignal
+  ): Promise<TaskResult>;
+}

package/src/strategies/StandardExecutionStrategy.ts

@@ -0,0 +1,35 @@
+import { IExecutionStrategy } from "./IExecutionStrategy.js";
+import { TaskStep } from "../TaskStep.js";
+import { TaskResult } from "../TaskResult.js";
+
+/**
+ * Standard execution strategy that runs the task's run method.
+ */
+export class StandardExecutionStrategy<TContext>
+  implements IExecutionStrategy<TContext>
+{
+  async execute(
+    step: TaskStep<TContext>,
+    context: TContext,
+    signal?: AbortSignal
+  ): Promise<TaskResult> {
+    try {
+      return await step.run(context, signal);
+    } catch (e) {
+      // Check if error is due to abort
+      if (
+        signal?.aborted &&
+        (e instanceof Error && e.name === "AbortError" || signal.reason === e)
+      ) {
+        return {
+          status: "cancelled",
+          message: "Task cancelled during execution",
+        };
+      }
+      return {
+        status: "failure",
+        error: e instanceof Error ? e.message : String(e),
+      };
+    }
+  }
+}

package/test-report.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
-<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">
+<testsuites name="vitest tests" tests="66" failures="0" errors="0" time="0.734267979">
+    <testsuite name="tests/ComplexScenario.test.ts" timestamp="2026-01-18T16:53:50.036Z" hostname="runnervmmtnos" tests="2" failures="0" errors="0" skipped="0" time="0.012401489">
+        <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.007855447">
             <system-out>
 Running StepA
 Running StepB
@@ -15,85 +15,163 @@ 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.00138321">
+        <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.002288234">
         </testcase>
     </testsuite>
-    <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">
+    <testsuite name="tests/EventBus.test.ts" timestamp="2026-01-18T16:53:50.037Z" hostname="runnervmmtnos" tests="1" failures="0" errors="0" skipped="0" time="0.021934861">
+        <testcase classname="tests/EventBus.test.ts" name="EventBus &gt; should handle async listeners throwing errors without crashing" time="0.019540929">
         </testcase>
     </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">
+    <testsuite name="tests/TaskGraphValidator.test.ts" timestamp="2026-01-18T16:53:50.038Z" hostname="runnervmmtnos" tests="9" failures="0" errors="0" skipped="0" time="0.009766435">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should be instantiated" time="0.002042233">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should return valid result for empty graph" time="0.001089122">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should return valid result for empty graph" time="0.001433754">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect duplicate tasks" time="0.001244392">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect duplicate tasks" time="0.001395153">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect missing dependencies" time="0.000319586">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect missing dependencies" time="0.000435706">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect cycles" time="0.000454157">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect cycles" time="0.000776113">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should return valid for a correct graph" time="0.000401499">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should return valid for a correct graph" time="0.000369241">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should not detect cycles if missing dependencies are present" time="0.000238875">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should not detect cycles if missing dependencies are present" time="0.000350526">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect more complex cycles" time="0.000207848">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should detect more complex cycles" time="0.000261449">
         </testcase>
-        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should handle cycle detection with no cycles but shared dependencies" time="0.000228516">
+        <testcase classname="tests/TaskGraphValidator.test.ts" name="TaskGraphValidator &gt; should handle cycle detection with no cycles but shared dependencies" time="0.000339385">
         </testcase>
     </testsuite>
-    <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">
+    <testsuite name="tests/TaskGraphValidatorDoS.test.ts" timestamp="2026-01-18T16:53:50.040Z" hostname="runnervmmtnos" tests="1" failures="0" errors="0" skipped="0" time="0.037493531">
+        <testcase classname="tests/TaskGraphValidatorDoS.test.ts" name="TaskGraphValidator - Deep Recursion &gt; should handle deep graphs without stack overflow" time="0.035728917">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle an empty list of tasks gracefully" time="0.000317112">
+    </testsuite>
+    <testsuite name="tests/TaskRunner.test.ts" timestamp="2026-01-18T16:53:50.040Z" hostname="runnervmmtnos" tests="10" failures="0" errors="0" skipped="0" time="0.111490101">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should run tasks in the correct sequential order" time="0.003037407">
+        </testcase>
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle an empty list of tasks gracefully" time="0.000322775">
+        </testcase>
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should run independent tasks in parallel" time="0.100688769">
+        </testcase>
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should skip dependent tasks if a root task fails" time="0.000418764">
+        </testcase>
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should throw an error for &apos;circular dependency&apos;" time="0.00237658">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should run independent tasks in parallel" time="0.10065072">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should throw an error for &apos;missing dependency&apos;" time="0.00042754">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should skip dependent tasks if a root task fails" time="0.000465769">
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle tasks that throw an error during execution" time="0.000402213">
+        </testcase>
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should skip tasks whose dependencies are skipped" time="0.000513091">
+        </testcase>
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle tasks that throw a non-Error object during execution" time="0.000398907">
+        </testcase>
+        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle duplicate steps where one gets skipped due to failed dependency" time="0.00050197">
+        </testcase>
+    </testsuite>
+    <testsuite name="tests/TaskRunnerEvents.test.ts" timestamp="2026-01-18T16:53:50.042Z" hostname="runnervmmtnos" tests="7" failures="0" errors="0" skipped="0" time="0.01911567">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire all lifecycle events in a successful run" time="0.00764296">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should throw an error for &apos;circular dependency&apos;" time="0.002694306">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire taskSkipped event when dependency fails" time="0.005221716">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should throw an error for &apos;missing dependency&apos;" time="0.00041795">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should not crash if a listener throws an error" time="0.001525195">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle tasks that throw an error during execution" time="0.000376783">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire workflow events even for empty step list" time="0.00055575">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should skip tasks whose dependencies are skipped" time="0.000786016">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should handle unsubscribe correctly" time="0.00070472">
         </testcase>
-        <testcase classname="tests/TaskRunner.test.ts" name="TaskRunner &gt; should handle tasks that throw a non-Error object during execution" time="0.000431165">
+        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should safely handle off() when no listeners exist" time="0.00037408">
         </testcase>
-        <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 classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should support multiple listeners for the same event" time="0.000612918">
         </testcase>
     </testsuite>
-    <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">
+    <testsuite name="tests/TaskRunnerRefactor.test.ts" timestamp="2026-01-18T16:53:50.043Z" hostname="runnervmmtnos" tests="6" failures="0" errors="0" skipped="0" time="0.009369642">
+        <testcase classname="tests/TaskRunnerRefactor.test.ts" name="TaskRunner Refactor Coverage &gt; should allow setting execution strategy" time="0.002330794">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire taskSkipped event when dependency fails" time="0.003792205">
+        <testcase classname="tests/TaskRunnerRefactor.test.ts" name="TaskRunner Refactor Coverage &gt; should allow unsubscribing from events" time="0.001040117">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should not crash if a listener throws an error" time="0.001438714">
+        <testcase classname="tests/TaskRunnerRefactor.test.ts" name="TaskRunner Refactor Coverage &gt; TaskStateManager.cancelAllPending should handle non-existing tasks gracefully" time="0.000549209">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should fire workflow events even for empty step list" time="0.000282257">
+        <testcase classname="tests/TaskRunnerRefactor.test.ts" name="TaskRunner Refactor Coverage &gt; TaskRunnerBuilder should build a TaskRunner" time="0.000695793">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should handle unsubscribe correctly" time="0.000443788">
+        <testcase classname="tests/TaskRunnerRefactor.test.ts" name="TaskRunner Refactor Coverage &gt; TaskRunnerBuilder should build a TaskRunner with no listeners" time="0.000276137">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should safely handle off() when no listeners exist" time="0.000271827">
+        <testcase classname="tests/TaskRunnerRefactor.test.ts" name="TaskRunner Refactor Coverage &gt; WorkflowExecutor should break infinite loop if no progress can be made" time="0.002033066">
         </testcase>
-        <testcase classname="tests/TaskRunnerEvents.test.ts" name="TaskRunner Events &gt; should support multiple listeners for the same event" time="0.000377915">
+    </testsuite>
+    <testsuite name="tests/WorkflowExecutor.test.ts" timestamp="2026-01-18T16:53:50.044Z" hostname="runnervmmtnos" tests="3" failures="0" errors="0" skipped="0" time="0.056872202">
+        <testcase classname="tests/WorkflowExecutor.test.ts" name="WorkflowExecutor &gt; should execute steps sequentially when dependencies exist" time="0.003496687">
+        </testcase>
+        <testcase classname="tests/WorkflowExecutor.test.ts" name="WorkflowExecutor &gt; should skip dependent steps if dependency fails" time="0.000477163">
+        </testcase>
+        <testcase classname="tests/WorkflowExecutor.test.ts" name="WorkflowExecutor &gt; should run independent steps in parallel" time="0.051018852">
         </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">
+    <testsuite name="tests/cancellation.test.ts" timestamp="2026-01-18T16:53:50.045Z" hostname="runnervmmtnos" tests="10" failures="0" errors="0" skipped="0" time="0.160536321">
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should execute tasks normally without cancellation" time="0.003288668">
+        </testcase>
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should cancel workflow via AbortSignal" time="0.011297793">
+        </testcase>
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should cancel workflow via global timeout" time="0.04959712">
+        </testcase>
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should handle pre-aborted signal" time="0.00046518">
+        </testcase>
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should propagate cancellation to running task" time="0.010460564">
+        </testcase>
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should respect timeout over signal if timeout happens first" time="0.050041281">
+        </testcase>
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should handle case where timeout is set AND signal is already aborted" time="0.000498924">
+        </testcase>
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should cancel workflow via AbortSignal when timeout is also set" time="0.010598674">
         </testcase>
-        <testcase classname="tests/WorkflowExecutor.test.ts" name="WorkflowExecutor &gt; should skip dependent steps if dependency fails" time="0.000479825">
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should fail task if it throws non-abort error during cancellation" time="0.01120543">
         </testcase>
-        <testcase classname="tests/WorkflowExecutor.test.ts" name="WorkflowExecutor &gt; should run independent steps in parallel" time="0.05083559">
+        <testcase classname="tests/cancellation.test.ts" name="TaskRunner Cancellation &gt; should cancel task if it rejects with signal.reason" time="0.010730942">
         </testcase>
     </testsuite>
-    <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">
+    <testsuite name="tests/integration.test.ts" timestamp="2026-01-18T16:53:50.047Z" hostname="runnervmmtnos" tests="3" failures="0" errors="0" skipped="0" time="0.005559619">
+        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for duplicate tasks" time="0.003324235">
         </testcase>
-        <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 classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for missing dependencies" time="0.000340878">
         </testcase>
-        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for cycles" time="0.000287436">
+        <testcase classname="tests/integration.test.ts" name="TaskRunner Validation Integration &gt; should throw validation error with clear message for cycles" time="0.000324869">
+        </testcase>
+    </testsuite>
+    <testsuite name="tests/integration-tests/basic-structure.test.ts" timestamp="2026-01-18T16:53:50.047Z" hostname="runnervmmtnos" tests="4" failures="0" errors="0" skipped="0" time="0.13078604">
+        <testcase classname="tests/integration-tests/basic-structure.test.ts" name="Integration: Basic Structure &gt; Scenario 1: Basic linear workflow (A -&gt; B -&gt; C)" time="0.034031066">
+        </testcase>
+        <testcase classname="tests/integration-tests/basic-structure.test.ts" name="Integration: Basic Structure &gt; Scenario 2: Branching workflow (A -&gt; [B, C] -&gt; D)" time="0.032156516">
+        </testcase>
+        <testcase classname="tests/integration-tests/basic-structure.test.ts" name="Integration: Basic Structure &gt; Scenario 12: Complex &apos;Diamond&apos; dependency graph (A -&gt; B -&gt; D, A -&gt; C -&gt; D)" time="0.041686058">
+        </testcase>
+        <testcase classname="tests/integration-tests/basic-structure.test.ts" name="Integration: Basic Structure &gt; Scenario 14: Zero-dependency parallel burst" time="0.020922221">
+        </testcase>
+    </testsuite>
+    <testsuite name="tests/integration-tests/concurrency-timing.test.ts" timestamp="2026-01-18T16:53:50.048Z" hostname="runnervmmtnos" tests="3" failures="0" errors="0" skipped="0" time="0.126419406">
+        <testcase classname="tests/integration-tests/concurrency-timing.test.ts" name="Integration: Concurrency and Timing &gt; Scenario 6: Mixed duration tasks (verifying parallel efficiency)" time="0.053041689">
+        </testcase>
+        <testcase classname="tests/integration-tests/concurrency-timing.test.ts" name="Integration: Concurrency and Timing &gt; Scenario 7: Cancellation via AbortSignal" time="0.021208639">
+        </testcase>
+        <testcase classname="tests/integration-tests/concurrency-timing.test.ts" name="Integration: Concurrency and Timing &gt; Scenario 8: Global timeout interrupting long tasks" time="0.050030744">
+        </testcase>
+    </testsuite>
+    <testsuite name="tests/integration-tests/context-state.test.ts" timestamp="2026-01-18T16:53:50.049Z" hostname="runnervmmtnos" tests="3" failures="0" errors="0" skipped="0" time="0.019135968">
+        <testcase classname="tests/integration-tests/context-state.test.ts" name="Integration: Context and State &gt; Scenario 4: Shared context mutation (A writes, B reads)" time="0.005077797">
+        </testcase>
+        <testcase classname="tests/integration-tests/context-state.test.ts" name="Integration: Context and State &gt; Scenario 9: Dynamic context validation" time="0.000954216">
+        </testcase>
+        <testcase classname="tests/integration-tests/context-state.test.ts" name="Integration: Context and State &gt; Scenario 13: Tasks with side-effects" time="0.010823344">
+        </testcase>
+    </testsuite>
+    <testsuite name="tests/integration-tests/error-handling.test.ts" timestamp="2026-01-18T16:53:50.049Z" hostname="runnervmmtnos" tests="3" failures="0" errors="0" skipped="0" time="0.008227594">
+        <testcase classname="tests/integration-tests/error-handling.test.ts" name="Integration: Error Handling &gt; Scenario 3: Task failure and downstream skipping" time="0.003512046">
+        </testcase>
+        <testcase classname="tests/integration-tests/error-handling.test.ts" name="Integration: Error Handling &gt; Scenario 10: Circular dependency detection" time="0.002290789">
+        </testcase>
+        <testcase classname="tests/integration-tests/error-handling.test.ts" name="Integration: Error Handling &gt; Scenario 11: Missing dependency handling" time="0.000406341">
+        </testcase>
+    </testsuite>
+    <testsuite name="tests/integration-tests/stress.test.ts" timestamp="2026-01-18T16:53:50.050Z" hostname="runnervmmtnos" tests="1" failures="0" errors="0" skipped="0" time="0.0051591">
+        <testcase classname="tests/integration-tests/stress.test.ts" name="Integration: Stress Tests &gt; Scenario 5: Large graph execution (e.g., 20+ nodes)" time="0.003673567">
         </testcase>
     </testsuite>
 </testsuites>

package/GEMINI.md

@@ -1,46 +0,0 @@
-# task-runner Development Guidelines
-
-Auto-generated from all feature plans. Last updated: 2026-01-17
-
-## Active Technologies
-- TypeScript 5.9.3 + vitest 4.0.17 (for testing) (004-pre-execution-validation)
-- TypeScript 5.9.3 + vitest 4.0.17, AbortSignal/AbortController (standard Web APIs) (002-task-cancellation)
-- N/A (in-memory context object) (002-task-cancellation)
-- N/A (in-memory queue for tasks) (005-concurrency-control)
-
-- TypeScript 5.9.3 + vitest 4.0.17 (003-refactor-file-structure)
-
-- (001-generic-task-runner)
-
-## Project Structure
-
-```text
-src/
-tests/
-```
-
-## Commands
-
-# Add commands for
-
-## Code Style
-
-: Follow standard conventions
-
-## Recent Changes
-- 005-concurrency-control: Added TypeScript 5.9.3 + vitest 4.0.17
-- 002-task-cancellation: Added TypeScript 5.9.3 + vitest 4.0.17, AbortSignal/AbortController (standard Web APIs)
-- 004-pre-execution-validation: Added TypeScript 5.9.3 + vitest 4.0.17 (for testing)
-
-
-
-<!-- MANUAL ADDITIONS START -->
-Before marking a task as concluded, YOU MUST:
-
-1. run pnpm install
-2. run pnpm build
-3. run pnpm test
-4. run pnpm lint
-
-If any of those command fail, review your changes.
-<!-- MANUAL ADDITIONS END -->

package/openspec/changes/add-external-task-cancellation/tasks.md

@@ -1,10 +0,0 @@
-## Implementation
-- [ ] 1.1 Update `TaskRunner.execute` signature to accept an optional config object.
-- [ ] 1.2 Implement logic within `TaskRunner` to listen for `AbortSignal` and initiate cancellation.
-- [ ] 1.3 Implement global timeout mechanism using `AbortController` and `setTimeout`.
-- [ ] 1.4 Propagate `AbortSignal` to individual `TaskStep` executions.
-- [ ] 1.5 Ensure `TaskRunner` correctly handles tasks that are already aborted before execution.
-- [ ] 1.6 Update `TaskResult` and `RunnerEvents` to reflect cancellation status.
-- [ ] 1.7 Add unit tests for `AbortSignal` cancellation scenarios.
-- [ ] 1.8 Add unit tests for global timeout scenarios.
-- [ ] 1.9 Add integration tests covering both `AbortSignal` and timeout interactions.