@calmo/task-runner 3.8.2 → 3.8.3

package/coverage/src/index.html

@@ -25,28 +25,28 @@
             <div class='fl pad1y space-right2'>
                 <span class="strong">100% </span>
                 <span class="quiet">Statements</span>
-                <span class='fraction'>257/257</span>
+                <span class='fraction'>261/261</span>
             </div>
         
             
             <div class='fl pad1y space-right2'>
                 <span class="strong">100% </span>
                 <span class="quiet">Branches</span>
-                <span class='fraction'>97/97</span>
+                <span class='fraction'>99/99</span>
             </div>
         
             
             <div class='fl pad1y space-right2'>
                 <span class="strong">100% </span>
                 <span class="quiet">Functions</span>
-                <span class='fraction'>51/51</span>
+                <span class='fraction'>50/50</span>
             </div>
         
             
             <div class='fl pad1y space-right2'>
                 <span class="strong">100% </span>
                 <span class="quiet">Lines</span>
-                <span class='fraction'>254/254</span>
+                <span class='fraction'>258/258</span>
             </div>
         
             
@@ -93,6 +93,21 @@
 	<td data-value="17" class="abs high">17/17</td>
 	</tr>
 
+<tr>
+	<td class="file high" data-value="ExecutionConstants.ts"><a href="ExecutionConstants.ts.html">ExecutionConstants.ts</a></td>
+	<td data-value="100" class="pic high">
+	<div class="chart"><div class="cover-fill cover-full" style="width: 100%"></div><div class="cover-empty" style="width: 0%"></div></div>
+	</td>
+	<td data-value="100" class="pct high">100%</td>
+	<td data-value="1" class="abs high">1/1</td>
+	<td data-value="100" class="pct high">100%</td>
+	<td data-value="0" class="abs high">0/0</td>
+	<td data-value="100" class="pct high">100%</td>
+	<td data-value="0" class="abs high">0/0</td>
+	<td data-value="100" class="pct high">100%</td>
+	<td data-value="1" class="abs high">1/1</td>
+	</tr>
+
 <tr>
 	<td class="file high" data-value="TaskGraphValidationError.ts"><a href="TaskGraphValidationError.ts.html">TaskGraphValidationError.ts</a></td>
 	<td data-value="100" class="pic high">
@@ -189,13 +204,13 @@
 	<div class="chart"><div class="cover-fill cover-full" style="width: 100%"></div><div class="cover-empty" style="width: 0%"></div></div>
 	</td>
 	<td data-value="100" class="pct high">100%</td>
-	<td data-value="66" class="abs high">66/66</td>
+	<td data-value="69" class="abs high">69/69</td>
 	<td data-value="100" class="pct high">100%</td>
-	<td data-value="35" class="abs high">35/35</td>
+	<td data-value="37" class="abs high">37/37</td>
 	<td data-value="100" class="pct high">100%</td>
-	<td data-value="8" class="abs high">8/8</td>
+	<td data-value="7" class="abs high">7/7</td>
 	<td data-value="100" class="pct high">100%</td>
-	<td data-value="65" class="abs high">65/65</td>
+	<td data-value="68" class="abs high">68/68</td>
 	</tr>
 
 </tbody>
@@ -206,7 +221,7 @@
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-01-21T01:16:02.338Z
+                at 2026-01-21T01:20:30.153Z
             </div>
         <script src="../prettify.js"></script>
         <script>

package/coverage/src/strategies/DryRunExecutionStrategy.ts.html

@@ -163,7 +163,7 @@ export class DryRunExecutionStrategy<
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-01-21T01:16:02.338Z
+                at 2026-01-21T01:20:30.153Z
             </div>
         <script src="../../prettify.js"></script>
         <script>

package/coverage/src/strategies/RetryingExecutionStrategy.ts.html

@@ -358,7 +358,7 @@ export class RetryingExecutionStrategy<
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-01-21T01:16:02.338Z
+                at 2026-01-21T01:20:30.153Z
             </div>
         <script src="../../prettify.js"></script>
         <script>

package/coverage/src/strategies/StandardExecutionStrategy.ts.html

@@ -175,7 +175,7 @@ export class StandardExecutionStrategy<
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-01-21T01:16:02.338Z
+                at 2026-01-21T01:20:30.153Z
             </div>
         <script src="../../prettify.js"></script>
         <script>

package/coverage/src/strategies/index.html

@@ -131,7 +131,7 @@
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-01-21T01:16:02.338Z
+                at 2026-01-21T01:20:30.153Z
             </div>
         <script src="../../prettify.js"></script>
         <script>

package/dist/ExecutionConstants.d.ts

@@ -0,0 +1,12 @@
+/**
+ * Constants used in the execution engine.
+ */
+export declare const ExecutionConstants: {
+    CANCELLED_BEFORE_START: string;
+    TASK_CANCELLED_BEFORE_START: string;
+    CANCELLED_DURING_CONDITION: string;
+    CONDITION_EVALUATION_FAILED: string;
+    SKIPPED_BY_CONDITION: string;
+    EXECUTION_STRATEGY_FAILED: string;
+    WORKFLOW_CANCELLED: string;
+};

package/dist/ExecutionConstants.js

@@ -0,0 +1,13 @@
+/**
+ * Constants used in the execution engine.
+ */
+export const ExecutionConstants = {
+    CANCELLED_BEFORE_START: "Workflow cancelled before execution started.",
+    TASK_CANCELLED_BEFORE_START: "Cancelled before execution started.",
+    CANCELLED_DURING_CONDITION: "Cancelled during condition evaluation.",
+    CONDITION_EVALUATION_FAILED: "Condition evaluation failed",
+    SKIPPED_BY_CONDITION: "Skipped by condition evaluation.",
+    EXECUTION_STRATEGY_FAILED: "Execution strategy failed.",
+    WORKFLOW_CANCELLED: "Workflow cancelled.",
+};
+//# sourceMappingURL=ExecutionConstants.js.map

package/dist/ExecutionConstants.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ExecutionConstants.js","sourceRoot":"","sources":["../src/ExecutionConstants.ts"],"names":[],"mappings":"AAAA;;GAEG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAAG;IAChC,sBAAsB,EAAE,8CAA8C;IACtE,2BAA2B,EAAE,qCAAqC;IAClE,0BAA0B,EAAE,wCAAwC;IACpE,2BAA2B,EAAE,6BAA6B;IAC1D,oBAAoB,EAAE,kCAAkC;IACxD,yBAAyB,EAAE,4BAA4B;IACvD,kBAAkB,EAAE,qBAAqB;CAC1C,CAAC"}

package/dist/WorkflowExecutor.d.ts

@@ -33,4 +33,8 @@ export declare class WorkflowExecutor<TContext> {
      * Logic to identify tasks that can be started and run them.
      */
     private processLoop;
+    /**
+     * Executes a single task step, handling conditions and status updates.
+     */
+    private executeTaskStep;
 }

package/dist/WorkflowExecutor.js

@@ -1,3 +1,4 @@
+import { ExecutionConstants } from "./ExecutionConstants.js";
 /**
  * Handles the execution of the workflow steps.
  * @template TContext The shape of the shared context object.
@@ -34,7 +35,7 @@ export class WorkflowExecutor {
         this.stateManager.initialize(steps);
         // Check if already aborted
         if (signal?.aborted) {
-            this.stateManager.cancelAllPending("Workflow cancelled before execution started.");
+            this.stateManager.cancelAllPending(ExecutionConstants.CANCELLED_BEFORE_START);
             const results = this.stateManager.getResults();
             this.eventBus.emit("workflowEnd", { context: this.context, results });
             return results;
@@ -42,7 +43,7 @@ export class WorkflowExecutor {
         const executingPromises = new Set();
         const onAbort = () => {
             // Mark all pending tasks as cancelled
-            this.stateManager.cancelAllPending("Workflow cancelled.");
+            this.stateManager.cancelAllPending(ExecutionConstants.WORKFLOW_CANCELLED);
         };
         if (signal) {
             signal.addEventListener("abort", onAbort);
@@ -64,7 +65,7 @@ export class WorkflowExecutor {
                     await Promise.race(executingPromises);
                 }
                 if (signal?.aborted) {
-                    this.stateManager.cancelAllPending("Workflow cancelled.");
+                    this.stateManager.cancelAllPending(ExecutionConstants.WORKFLOW_CANCELLED);
                 }
                 else {
                     // After a task finishes, check for new work
@@ -72,7 +73,7 @@ export class WorkflowExecutor {
                 }
             }
             // Ensure everything is accounted for (e.g. if loop exited early)
-            this.stateManager.cancelAllPending("Workflow cancelled.");
+            this.stateManager.cancelAllPending(ExecutionConstants.WORKFLOW_CANCELLED);
             const results = this.stateManager.getResults();
             this.eventBus.emit("workflowEnd", { context: this.context, results });
             return results;
@@ -101,58 +102,70 @@ export class WorkflowExecutor {
                 break;
             }
             const step = this.readyQueue.shift();
-            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 = {
-                                status: "skipped",
-                                message: "Skipped by condition evaluation.",
-                            };
-                            this.stateManager.markSkipped(step, result);
-                            return;
-                        }
-                    }
-                }
-                catch (error) {
-                    const result = {
-                        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;
-                }
+            const taskPromise = this.executeTaskStep(step, signal)
+                .finally(() => {
+                executingPromises.delete(taskPromise);
+                // When a task finishes, we try to run more
+                this.processLoop(executingPromises, signal);
+            });
+            executingPromises.add(taskPromise);
+        }
+    }
+    /**
+     * Executes a single task step, handling conditions and status updates.
+     */
+    async executeTaskStep(step, signal) {
+        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 before execution started.",
+                        message: ExecutionConstants.CANCELLED_DURING_CONDITION,
                     });
                     return;
                 }
-                this.stateManager.markRunning(step);
-                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);
+                if (!shouldRun) {
+                    const result = {
+                        status: "skipped",
+                        message: ExecutionConstants.SKIPPED_BY_CONDITION,
+                    };
+                    this.stateManager.markSkipped(step, result);
+                    return;
+                }
+            }
+        }
+        catch (error) {
+            const result = {
+                status: "failure",
+                message: error instanceof Error
+                    ? error.message
+                    : ExecutionConstants.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: ExecutionConstants.TASK_CANCELLED_BEFORE_START,
             });
-            executingPromises.add(taskPromise);
+            return;
+        }
+        this.stateManager.markRunning(step);
+        try {
+            const result = await this.strategy.execute(step, this.context, signal);
+            this.stateManager.markCompleted(step, result);
+        }
+        catch (error) {
+            const result = {
+                status: "failure",
+                message: ExecutionConstants.EXECUTION_STRATEGY_FAILED,
+                error: error instanceof Error ? error.message : String(error),
+            };
+            this.stateManager.markCompleted(step, result);
         }
     }
 }

package/dist/WorkflowExecutor.js.map

@@ -1 +1 @@
-{"version":3,"file":"WorkflowExecutor.js","sourceRoot":"","sources":["../src/WorkflowExecutor.ts"],"names":[],"mappings":"AAMA;;;GAGG;AACH,MAAM,OAAO,gBAAgB;IAWjB;IACA;IACA;IACA;IACA;IAdF,UAAU,GAAyB,EAAE,CAAC;IAE9C;;;;;;OAMG;IACH,YACU,OAAiB,EACjB,QAA4B,EAC5B,YAAwC,EACxC,QAAsC,EACtC,WAAoB;QAJpB,YAAO,GAAP,OAAO,CAAU;QACjB,aAAQ,GAAR,QAAQ,CAAoB;QAC5B,iBAAY,GAAZ,YAAY,CAA4B;QACxC,aAAQ,GAAR,QAAQ,CAA8B;QACtC,gBAAW,GAAX,WAAW,CAAS;IAC3B,CAAC;IAEJ;;;;;OAKG;IACH,KAAK,CAAC,OAAO,CACX,KAA2B,EAC3B,MAAoB;QAEpB,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,eAAe,EAAE,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,KAAK,EAAE,CAAC,CAAC;QACtE,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC;QAEpC,2BAA2B;QAC3B,IAAI,MAAM,EAAE,OAAO,EAAE,CAAC;YACpB,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAChC,8CAA8C,CAC/C,CAAC;YACF,MAAM,OAAO,GAAG,IAAI,CAAC,YAAY,CAAC,UAAU,EAAE,CAAC;YAC/C,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,aAAa,EAAE,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,OAAO,EAAE,CAAC,CAAC;YACtE,OAAO,OAAO,CAAC;QACjB,CAAC;QAED,MAAM,iBAAiB,GAAG,IAAI,GAAG,EAAiB,CAAC;QAEnD,MAAM,OAAO,GAAG,GAAG,EAAE;YACnB,sCAAsC;YACtC,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAAC,qBAAqB,CAAC,CAAC;QAC5D,CAAC,CAAC;QAEF,IAAI,MAAM,EAAE,CAAC;YACX,MAAM,CAAC,gBAAgB,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;QAC5C,CAAC;QAED,IAAI,CAAC;YACH,eAAe;YACf,IAAI,CAAC,WAAW,CAAC,iBAAiB,EAAE,MAAM,CAAC,CAAC;YAE5C,OACE,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE;gBACnC,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE;gBACnC,iBAAiB,CAAC,IAAI,GAAG,CAAC,EAC1B,CAAC;gBACD,yEAAyE;gBACzE,8DAA8D;gBAC9D,2EAA2E;gBAC3E,IAAI,iBAAiB,CAAC,IAAI,KAAK,CAAC,EAAE,CAAC;oBACjC,MAAM;gBACR,CAAC;qBAAM,CAAC;oBACN,mCAAmC;oBACnC,MAAM,OAAO,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;gBACxC,CAAC;gBAED,IAAI,MAAM,EAAE,OAAO,EAAE,CAAC;oBACpB,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAAC,qBAAqB,CAAC,CAAC;gBAC5D,CAAC;qBAAM,CAAC;oBACN,4CAA4C;oBAC5C,IAAI,CAAC,WAAW,CAAC,iBAAiB,EAAE,MAAM,CAAC,CAAC;gBAC9C,CAAC;YACH,CAAC;YAED,iEAAiE;YACjE,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAAC,qBAAqB,CAAC,CAAC;YAE1D,MAAM,OAAO,GAAG,IAAI,CAAC,YAAY,CAAC,UAAU,EAAE,CAAC;YAC/C,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,aAAa,EAAE,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,OAAO,EAAE,CAAC,CAAC;YACtE,OAAO,OAAO,CAAC;QACjB,CAAC;gBAAS,CAAC;YACT,IAAI,MAAM,EAAE,CAAC;gBACX,MAAM,CAAC,mBAAmB,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;YAC/C,CAAC;QACH,CAAC;IACH,CAAC;IAED;;OAEG;IACK,WAAW,CACjB,iBAAqC,EACrC,MAAoB;QAEpB,MAAM,UAAU,GAAG,IAAI,CAAC,YAAY,CAAC,mBAAmB,EAAE,CAAC;QAE3D,qCAAqC;QACrC,KAAK,MAAM,IAAI,IAAI,UAAU,EAAE,CAAC;YAC9B,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7B,CAAC;QAED,4DAA4D;QAC5D,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,QAAQ,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,QAAQ,IAAI,CAAC,CAAC,CAAC,CAAC;QAEtE,yDAAyD;QACzD,OAAO,IAAI,CAAC,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAClC,IACE,IAAI,CAAC,WAAW,KAAK,SAAS;gBAC9B,iBAAiB,CAAC,IAAI,IAAI,IAAI,CAAC,WAAW,EAC1C,CAAC;gBACD,MAAM;YACR,CAAC;YAED,MAAM,IAAI,GAAG,IAAI,CAAC,UAAU,CAAC,KAAK,EAAG,CAAC;YAEtC,MAAM,WAAW,GAAG,CAAC,KAAK,IAAI,EAAE;gBAC9B,IAAI,CAAC;oBACH,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;wBACnB,MAAM,KAAK,GAAG,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;wBAC3C,MAAM,SAAS,GAAG,KAAK,YAAY,OAAO,CAAC,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC;wBAEjE,IAAI,MAAM,EAAE,OAAO,EAAE,CAAC;4BACpB,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE;gCACpC,MAAM,EAAE,WAAW;gCACnB,OAAO,EAAE,wCAAwC;6BAClD,CAAC,CAAC;4BACH,OAAO;wBACT,CAAC;wBAED,IAAI,CAAC,SAAS,EAAE,CAAC;4BACf,MAAM,MAAM,GAAe;gCACzB,MAAM,EAAE,SAAS;gCACjB,OAAO,EAAE,kCAAkC;6BAC5C,CAAC;4BACF,IAAI,CAAC,YAAY,CAAC,WAAW,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;4BAC5C,OAAO;wBACT,CAAC;oBACH,CAAC;gBACH,CAAC;gBAAC,OAAO,KAAK,EAAE,CAAC;oBACf,MAAM,MAAM,GAAe;wBACzB,MAAM,EAAE,SAAS;wBACjB,OAAO,EACL,KAAK,YAAY,KAAK;4BACpB,CAAC,CAAC,KAAK,CAAC,OAAO;4BACf,CAAC,CAAC,6BAA6B;wBACnC,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;qBAC9D,CAAC;oBACF,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;oBAC9C,OAAO;gBACT,CAAC;gBAED,IAAI,MAAM,EAAE,OAAO,EAAE,CAAC;oBACpB,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE;wBACpC,MAAM,EAAE,WAAW;wBACnB,OAAO,EAAE,qCAAqC;qBAC/C,CAAC,CAAC;oBACH,OAAO;gBACT,CAAC;gBAED,IAAI,CAAC,YAAY,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC;gBAEpC,MAAM,IAAI,CAAC,QAAQ;qBAChB,OAAO,CAAC,IAAI,EAAE,IAAI,CAAC,OAAO,EAAE,MAAM,CAAC;qBACnC,IAAI,CAAC,CAAC,MAAM,EAAE,EAAE;oBACf,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;gBAChD,CAAC,CAAC,CAAC;YACP,CAAC,CAAC,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE;gBAChB,iBAAiB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;gBACtC,2CAA2C;gBAC3C,IAAI,CAAC,WAAW,CAAC,iBAAiB,EAAE,MAAM,CAAC,CAAC;YAC9C,CAAC,CAAC,CAAC;YAEH,iBAAiB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;QACrC,CAAC;IACH,CAAC;CACF"}
+{"version":3,"file":"WorkflowExecutor.js","sourceRoot":"","sources":["../src/WorkflowExecutor.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,kBAAkB,EAAE,MAAM,yBAAyB,CAAC;AAE7D;;;GAGG;AACH,MAAM,OAAO,gBAAgB;IAWjB;IACA;IACA;IACA;IACA;IAdF,UAAU,GAAyB,EAAE,CAAC;IAE9C;;;;;;OAMG;IACH,YACU,OAAiB,EACjB,QAA4B,EAC5B,YAAwC,EACxC,QAAsC,EACtC,WAAoB;QAJpB,YAAO,GAAP,OAAO,CAAU;QACjB,aAAQ,GAAR,QAAQ,CAAoB;QAC5B,iBAAY,GAAZ,YAAY,CAA4B;QACxC,aAAQ,GAAR,QAAQ,CAA8B;QACtC,gBAAW,GAAX,WAAW,CAAS;IAC3B,CAAC;IAEJ;;;;;OAKG;IACH,KAAK,CAAC,OAAO,CACX,KAA2B,EAC3B,MAAoB;QAEpB,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,eAAe,EAAE,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,KAAK,EAAE,CAAC,CAAC;QACtE,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC;QAEpC,2BAA2B;QAC3B,IAAI,MAAM,EAAE,OAAO,EAAE,CAAC;YACpB,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAChC,kBAAkB,CAAC,sBAAsB,CAC1C,CAAC;YACF,MAAM,OAAO,GAAG,IAAI,CAAC,YAAY,CAAC,UAAU,EAAE,CAAC;YAC/C,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,aAAa,EAAE,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,OAAO,EAAE,CAAC,CAAC;YACtE,OAAO,OAAO,CAAC;QACjB,CAAC;QAED,MAAM,iBAAiB,GAAG,IAAI,GAAG,EAAiB,CAAC;QAEnD,MAAM,OAAO,GAAG,GAAG,EAAE;YACnB,sCAAsC;YACtC,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAAC,kBAAkB,CAAC,kBAAkB,CAAC,CAAC;QAC5E,CAAC,CAAC;QAEF,IAAI,MAAM,EAAE,CAAC;YACX,MAAM,CAAC,gBAAgB,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;QAC5C,CAAC;QAED,IAAI,CAAC;YACH,eAAe;YACf,IAAI,CAAC,WAAW,CAAC,iBAAiB,EAAE,MAAM,CAAC,CAAC;YAE5C,OACE,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE;gBACnC,IAAI,CAAC,YAAY,CAAC,eAAe,EAAE;gBACnC,iBAAiB,CAAC,IAAI,GAAG,CAAC,EAC1B,CAAC;gBACD,yEAAyE;gBACzE,8DAA8D;gBAC9D,2EAA2E;gBAC3E,IAAI,iBAAiB,CAAC,IAAI,KAAK,CAAC,EAAE,CAAC;oBACjC,MAAM;gBACR,CAAC;qBAAM,CAAC;oBACN,mCAAmC;oBACnC,MAAM,OAAO,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;gBACxC,CAAC;gBAED,IAAI,MAAM,EAAE,OAAO,EAAE,CAAC;oBACpB,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAChC,kBAAkB,CAAC,kBAAkB,CACtC,CAAC;gBACJ,CAAC;qBAAM,CAAC;oBACN,4CAA4C;oBAC5C,IAAI,CAAC,WAAW,CAAC,iBAAiB,EAAE,MAAM,CAAC,CAAC;gBAC9C,CAAC;YACH,CAAC;YAED,iEAAiE;YACjE,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAAC,kBAAkB,CAAC,kBAAkB,CAAC,CAAC;YAE1E,MAAM,OAAO,GAAG,IAAI,CAAC,YAAY,CAAC,UAAU,EAAE,CAAC;YAC/C,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,aAAa,EAAE,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,EAAE,OAAO,EAAE,CAAC,CAAC;YACtE,OAAO,OAAO,CAAC;QACjB,CAAC;gBAAS,CAAC;YACT,IAAI,MAAM,EAAE,CAAC;gBACX,MAAM,CAAC,mBAAmB,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;YAC/C,CAAC;QACH,CAAC;IACH,CAAC;IAED;;OAEG;IACK,WAAW,CACjB,iBAAqC,EACrC,MAAoB;QAEpB,MAAM,UAAU,GAAG,IAAI,CAAC,YAAY,CAAC,mBAAmB,EAAE,CAAC;QAE3D,qCAAqC;QACrC,KAAK,MAAM,IAAI,IAAI,UAAU,EAAE,CAAC;YAC9B,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7B,CAAC;QAED,4DAA4D;QAC5D,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,QAAQ,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,QAAQ,IAAI,CAAC,CAAC,CAAC,CAAC;QAEtE,yDAAyD;QACzD,OAAO,IAAI,CAAC,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAClC,IACE,IAAI,CAAC,WAAW,KAAK,SAAS;gBAC9B,iBAAiB,CAAC,IAAI,IAAI,IAAI,CAAC,WAAW,EAC1C,CAAC;gBACD,MAAM;YACR,CAAC;YAED,MAAM,IAAI,GAAG,IAAI,CAAC,UAAU,CAAC,KAAK,EAAG,CAAC;YAEtC,MAAM,WAAW,GAAG,IAAI,CAAC,eAAe,CAAC,IAAI,EAAE,MAAM,CAAC;iBACnD,OAAO,CAAC,GAAG,EAAE;gBACZ,iBAAiB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;gBACtC,2CAA2C;gBAC3C,IAAI,CAAC,WAAW,CAAC,iBAAiB,EAAE,MAAM,CAAC,CAAC;YAC9C,CAAC,CAAC,CAAC;YAEL,iBAAiB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;QACrC,CAAC;IACH,CAAC;IAED;;OAEG;IACK,KAAK,CAAC,eAAe,CAC3B,IAAwB,EACxB,MAAoB;QAEpB,IAAI,CAAC;YACH,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;gBACnB,MAAM,KAAK,GAAG,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;gBAC3C,MAAM,SAAS,GAAG,KAAK,YAAY,OAAO,CAAC,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC;gBAEjE,IAAI,MAAM,EAAE,OAAO,EAAE,CAAC;oBACpB,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE;wBACpC,MAAM,EAAE,WAAW;wBACnB,OAAO,EAAE,kBAAkB,CAAC,0BAA0B;qBACvD,CAAC,CAAC;oBACH,OAAO;gBACT,CAAC;gBAED,IAAI,CAAC,SAAS,EAAE,CAAC;oBACf,MAAM,MAAM,GAAe;wBACzB,MAAM,EAAE,SAAS;wBACjB,OAAO,EAAE,kBAAkB,CAAC,oBAAoB;qBACjD,CAAC;oBACF,IAAI,CAAC,YAAY,CAAC,WAAW,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;oBAC5C,OAAO;gBACT,CAAC;YACH,CAAC;QACH,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,MAAM,GAAe;gBACzB,MAAM,EAAE,SAAS;gBACjB,OAAO,EACL,KAAK,YAAY,KAAK;oBACpB,CAAC,CAAC,KAAK,CAAC,OAAO;oBACf,CAAC,CAAC,kBAAkB,CAAC,2BAA2B;gBACpD,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;aAC9D,CAAC;YACF,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;YAC9C,OAAO;QACT,CAAC;QAED,IAAI,MAAM,EAAE,OAAO,EAAE,CAAC;YACpB,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE;gBACpC,MAAM,EAAE,WAAW;gBACnB,OAAO,EAAE,kBAAkB,CAAC,2BAA2B;aACxD,CAAC,CAAC;YACH,OAAO;QACT,CAAC;QAED,IAAI,CAAC,YAAY,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC;QAEpC,IAAI,CAAC;YACH,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,EAAE,IAAI,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;YACvE,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;QAChD,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,MAAM,GAAe;gBACzB,MAAM,EAAE,SAAS;gBACjB,OAAO,EAAE,kBAAkB,CAAC,yBAAyB;gBACrD,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;aAC9D,CAAC;YACF,IAAI,CAAC,YAAY,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;QAChD,CAAC;IACH,CAAC;CACF"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@calmo/task-runner",
-  "version": "3.8.2",
+  "version": "3.8.3",
   "description": "A lightweight, type-safe, and domain-agnostic task orchestration engine. It resolves a Directed Acyclic Graph (DAG) of steps, executes independent tasks in parallel, and manages a shared context across the pipeline.",
   "repository": {
     "type": "git",

package/src/ExecutionConstants.ts

@@ -0,0 +1,12 @@
+/**
+ * Constants used in the execution engine.
+ */
+export const ExecutionConstants = {
+  CANCELLED_BEFORE_START: "Workflow cancelled before execution started.",
+  TASK_CANCELLED_BEFORE_START: "Cancelled before execution started.",
+  CANCELLED_DURING_CONDITION: "Cancelled during condition evaluation.",
+  CONDITION_EVALUATION_FAILED: "Condition evaluation failed",
+  SKIPPED_BY_CONDITION: "Skipped by condition evaluation.",
+  EXECUTION_STRATEGY_FAILED: "Execution strategy failed.",
+  WORKFLOW_CANCELLED: "Workflow cancelled.",
+};

package/src/WorkflowExecutor.ts

@@ -3,6 +3,7 @@ import { TaskResult } from "./TaskResult.js";
 import { EventBus } from "./EventBus.js";
 import { TaskStateManager } from "./TaskStateManager.js";
 import { IExecutionStrategy } from "./strategies/IExecutionStrategy.js";
+import { ExecutionConstants } from "./ExecutionConstants.js";
 
 /**
  * Handles the execution of the workflow steps.
@@ -42,7 +43,7 @@ export class WorkflowExecutor<TContext> {
     // Check if already aborted
     if (signal?.aborted) {
       this.stateManager.cancelAllPending(
-        "Workflow cancelled before execution started."
+        ExecutionConstants.CANCELLED_BEFORE_START
       );
       const results = this.stateManager.getResults();
       this.eventBus.emit("workflowEnd", { context: this.context, results });
@@ -53,7 +54,7 @@ export class WorkflowExecutor<TContext> {
 
     const onAbort = () => {
       // Mark all pending tasks as cancelled
-      this.stateManager.cancelAllPending("Workflow cancelled.");
+      this.stateManager.cancelAllPending(ExecutionConstants.WORKFLOW_CANCELLED);
     };
 
     if (signal) {
@@ -80,7 +81,9 @@ export class WorkflowExecutor<TContext> {
         }
 
         if (signal?.aborted) {
-          this.stateManager.cancelAllPending("Workflow cancelled.");
+          this.stateManager.cancelAllPending(
+            ExecutionConstants.WORKFLOW_CANCELLED
+          );
         } else {
           // After a task finishes, check for new work
           this.processLoop(executingPromises, signal);
@@ -88,7 +91,7 @@ export class WorkflowExecutor<TContext> {
       }
 
       // Ensure everything is accounted for (e.g. if loop exited early)
-      this.stateManager.cancelAllPending("Workflow cancelled.");
+      this.stateManager.cancelAllPending(ExecutionConstants.WORKFLOW_CANCELLED);
 
       const results = this.stateManager.getResults();
       this.eventBus.emit("workflowEnd", { context: this.context, results });
@@ -128,64 +131,79 @@ export class WorkflowExecutor<TContext> {
 
       const step = this.readyQueue.shift()!;
 
-      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;
-        }
+      const taskPromise = this.executeTaskStep(step, signal)
+        .finally(() => {
+          executingPromises.delete(taskPromise);
+          // When a task finishes, we try to run more
+          this.processLoop(executingPromises, signal);
+        });
+
+      executingPromises.add(taskPromise);
+    }
+  }
+
+  /**
+   * Executes a single task step, handling conditions and status updates.
+   */
+  private async executeTaskStep(
+    step: TaskStep<TContext>,
+    signal?: AbortSignal
+  ): Promise<void> {
+    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 before execution started.",
+            message: ExecutionConstants.CANCELLED_DURING_CONDITION,
           });
           return;
         }
 
-        this.stateManager.markRunning(step);
+        if (!shouldRun) {
+          const result: TaskResult = {
+            status: "skipped",
+            message: ExecutionConstants.SKIPPED_BY_CONDITION,
+          };
+          this.stateManager.markSkipped(step, result);
+          return;
+        }
+      }
+    } catch (error) {
+      const result: TaskResult = {
+        status: "failure",
+        message:
+          error instanceof Error
+            ? error.message
+            : ExecutionConstants.CONDITION_EVALUATION_FAILED,
+        error: error instanceof Error ? error.message : String(error),
+      };
+      this.stateManager.markCompleted(step, result);
+      return;
+    }
 
-        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);
+    if (signal?.aborted) {
+      this.stateManager.markCompleted(step, {
+        status: "cancelled",
+        message: ExecutionConstants.TASK_CANCELLED_BEFORE_START,
       });
+      return;
+    }
 
-      executingPromises.add(taskPromise);
+    this.stateManager.markRunning(step);
+
+    try {
+      const result = await this.strategy.execute(step, this.context, signal);
+      this.stateManager.markCompleted(step, result);
+    } catch (error) {
+      const result: TaskResult = {
+        status: "failure",
+        message: ExecutionConstants.EXECUTION_STRATEGY_FAILED,
+        error: error instanceof Error ? error.message : String(error),
+      };
+      this.stateManager.markCompleted(step, result);
     }
   }
 }