@calmo/task-runner 1.2.2 → 1.2.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (43) hide show
  1. package/.gemini/commands/openspec/apply.toml +21 -0
  2. package/.gemini/commands/openspec/archive.toml +25 -0
  3. package/.gemini/commands/openspec/proposal.toml +26 -0
  4. package/AGENTS.md +18 -0
  5. package/CHANGELOG.md +16 -0
  6. package/GEMINI.md +8 -0
  7. package/coverage/coverage-final.json +4 -4
  8. package/coverage/index.html +9 -9
  9. package/coverage/lcov-report/index.html +9 -9
  10. package/coverage/lcov-report/src/EventBus.ts.html +4 -4
  11. package/coverage/lcov-report/src/TaskGraphValidator.ts.html +35 -5
  12. package/coverage/lcov-report/src/TaskRunner.ts.html +7 -109
  13. package/coverage/lcov-report/src/WorkflowExecutor.ts.html +166 -109
  14. package/coverage/lcov-report/src/contracts/RunnerEvents.ts.html +1 -1
  15. package/coverage/lcov-report/src/contracts/index.html +1 -1
  16. package/coverage/lcov-report/src/index.html +16 -16
  17. package/coverage/lcov.info +120 -144
  18. package/coverage/src/EventBus.ts.html +4 -4
  19. package/coverage/src/TaskGraphValidator.ts.html +35 -5
  20. package/coverage/src/TaskRunner.ts.html +7 -109
  21. package/coverage/src/WorkflowExecutor.ts.html +166 -109
  22. package/coverage/src/contracts/RunnerEvents.ts.html +1 -1
  23. package/coverage/src/contracts/index.html +1 -1
  24. package/coverage/src/index.html +16 -16
  25. package/dist/TaskGraphValidator.d.ts +6 -0
  26. package/dist/TaskGraphValidator.js +9 -0
  27. package/dist/TaskGraphValidator.js.map +1 -1
  28. package/dist/TaskRunner.js +1 -32
  29. package/dist/TaskRunner.js.map +1 -1
  30. package/dist/WorkflowExecutor.d.ts +9 -0
  31. package/dist/WorkflowExecutor.js +74 -54
  32. package/dist/WorkflowExecutor.js.map +1 -1
  33. package/dist/contracts/ITaskGraphValidator.d.ts +6 -0
  34. package/openspec/AGENTS.md +456 -0
  35. package/openspec/changes/add-external-task-cancellation/proposal.md +14 -0
  36. package/openspec/changes/add-external-task-cancellation/tasks.md +10 -0
  37. package/openspec/project.md +31 -0
  38. package/package.json +1 -1
  39. package/src/TaskGraphValidator.ts +10 -0
  40. package/src/TaskRunner.ts +1 -35
  41. package/src/WorkflowExecutor.ts +81 -62
  42. package/src/contracts/ITaskGraphValidator.ts +7 -0
  43. package/test-report.xml +47 -39
package/coverage/lcov-report/src/TaskGraphValidator.ts.html CHANGED
@@ -25,7 +25,7 @@
25
25
  <div class='fl pad1y space-right2'>
26
26
  <span class="strong">100% </span>
27
27
  <span class="quiet">Statements</span>
28
- <span class='fraction'>43/43</span>
28
+ <span class='fraction'>46/46</span>
29
29
  </div>
30
30
 
31
31
 
@@ -39,14 +39,14 @@
39
39
  <div class='fl pad1y space-right2'>
40
40
  <span class="strong">100% </span>
41
41
  <span class="quiet">Functions</span>
42
- <span class='fraction'>3/3</span>
42
+ <span class='fraction'>5/5</span>
43
43
  </div>
44
44
 
45
45
 
46
46
  <div class='fl pad1y space-right2'>
47
47
  <span class="strong">100% </span>
48
48
  <span class="quiet">Lines</span>
49
- <span class='fraction'>42/42</span>
49
+ <span class='fraction'>44/44</span>
50
50
  </div>
51
51
 
52
52
 
@@ -189,7 +189,17 @@
189
189
  <a name='L124'></a><a href='#L124'>124</a>
190
190
  <a name='L125'></a><a href='#L125'>125</a>
191
191
  <a name='L126'></a><a href='#L126'>126</a>
192
- <a name='L127'></a><a href='#L127'>127</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
192
+ <a name='L127'></a><a href='#L127'>127</a>
193
+ <a name='L128'></a><a href='#L128'>128</a>
194
+ <a name='L129'></a><a href='#L129'>129</a>
195
+ <a name='L130'></a><a href='#L130'>130</a>
196
+ <a name='L131'></a><a href='#L131'>131</a>
197
+ <a name='L132'></a><a href='#L132'>132</a>
198
+ <a name='L133'></a><a href='#L133'>133</a>
199
+ <a name='L134'></a><a href='#L134'>134</a>
200
+ <a name='L135'></a><a href='#L135'>135</a>
201
+ <a name='L136'></a><a href='#L136'>136</a>
202
+ <a name='L137'></a><a href='#L137'>137</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
193
203
  <span class="cline-any cline-neutral">&nbsp;</span>
194
204
  <span class="cline-any cline-neutral">&nbsp;</span>
195
205
  <span class="cline-any cline-neutral">&nbsp;</span>
@@ -290,6 +300,16 @@
290
300
  <span class="cline-any cline-neutral">&nbsp;</span>
291
301
  <span class="cline-any cline-neutral">&nbsp;</span>
292
302
  <span class="cline-any cline-neutral">&nbsp;</span>
303
+ <span class="cline-any cline-yes">6x</span>
304
+ <span class="cline-any cline-yes">6x</span>
305
+ <span class="cline-any cline-neutral">&nbsp;</span>
306
+ <span class="cline-any cline-neutral">&nbsp;</span>
307
+ <span class="cline-any cline-neutral">&nbsp;</span>
308
+ <span class="cline-any cline-neutral">&nbsp;</span>
309
+ <span class="cline-any cline-neutral">&nbsp;</span>
310
+ <span class="cline-any cline-neutral">&nbsp;</span>
311
+ <span class="cline-any cline-neutral">&nbsp;</span>
312
+ <span class="cline-any cline-neutral">&nbsp;</span>
293
313
  <span class="cline-any cline-neutral">&nbsp;</span>
294
314
  <span class="cline-any cline-yes">51x</span>
295
315
  <span class="cline-any cline-yes">51x</span>
@@ -409,6 +429,16 @@ export class TaskGraphValidator implements ITaskGraphValidator {
409
429
  errors
410
430
  };
411
431
  }
432
+ &nbsp;
433
+ /**
434
+ * Creates a human-readable error message from a validation result.
435
+ * @param result The validation result containing errors.
436
+ * @returns A formatted error string.
437
+ */
438
+ createErrorMessage(result: ValidationResult): string {
439
+ const errorDetails = result.errors.map(e =&gt; e.message);
440
+ return `Task graph validation failed: ${errorDetails.join("; ")}`;
441
+ }
412
442
  &nbsp;
413
443
  private detectCycle(
414
444
  taskId: string,
@@ -448,7 +478,7 @@ export class TaskGraphValidator implements ITaskGraphValidator {
448
478
  <div class='footer quiet pad2 space-top1 center small'>
449
479
  Code coverage generated by
450
480
  <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
451
- at 2026-01-18T05:44:30.766Z
481
+ at 2026-01-18T14:25:37.520Z
452
482
  </div>
453
483
  <script src="../prettify.js"></script>
454
484
  <script>
package/coverage/lcov-report/src/TaskRunner.ts.html CHANGED
@@ -25,28 +25,28 @@
25
25
  <div class='fl pad1y space-right2'>
26
26
  <span class="strong">100% </span>
27
27
  <span class="quiet">Statements</span>
28
- <span class='fraction'>30/30</span>
28
+ <span class='fraction'>12/12</span>
29
29
  </div>
30
30
 
31
31
 
32
32
  <div class='fl pad1y space-right2'>
33
33
  <span class="strong">100% </span>
34
34
  <span class="quiet">Branches</span>
35
- <span class='fraction'>7/7</span>
35
+ <span class='fraction'>4/4</span>
36
36
  </div>
37
37
 
38
38
 
39
39
  <div class='fl pad1y space-right2'>
40
40
  <span class="strong">100% </span>
41
41
  <span class="quiet">Functions</span>
42
- <span class='fraction'>6/6</span>
42
+ <span class='fraction'>5/5</span>
43
43
  </div>
44
44
 
45
45
 
46
46
  <div class='fl pad1y space-right2'>
47
47
  <span class="strong">100% </span>
48
48
  <span class="quiet">Lines</span>
49
- <span class='fraction'>29/29</span>
49
+ <span class='fraction'>12/12</span>
50
50
  </div>
51
51
 
52
52
 
@@ -137,41 +137,7 @@
137
137
  <a name='L72'></a><a href='#L72'>72</a>
138
138
  <a name='L73'></a><a href='#L73'>73</a>
139
139
  <a name='L74'></a><a href='#L74'>74</a>
140
- <a name='L75'></a><a href='#L75'>75</a>
141
- <a name='L76'></a><a href='#L76'>76</a>
142
- <a name='L77'></a><a href='#L77'>77</a>
143
- <a name='L78'></a><a href='#L78'>78</a>
144
- <a name='L79'></a><a href='#L79'>79</a>
145
- <a name='L80'></a><a href='#L80'>80</a>
146
- <a name='L81'></a><a href='#L81'>81</a>
147
- <a name='L82'></a><a href='#L82'>82</a>
148
- <a name='L83'></a><a href='#L83'>83</a>
149
- <a name='L84'></a><a href='#L84'>84</a>
150
- <a name='L85'></a><a href='#L85'>85</a>
151
- <a name='L86'></a><a href='#L86'>86</a>
152
- <a name='L87'></a><a href='#L87'>87</a>
153
- <a name='L88'></a><a href='#L88'>88</a>
154
- <a name='L89'></a><a href='#L89'>89</a>
155
- <a name='L90'></a><a href='#L90'>90</a>
156
- <a name='L91'></a><a href='#L91'>91</a>
157
- <a name='L92'></a><a href='#L92'>92</a>
158
- <a name='L93'></a><a href='#L93'>93</a>
159
- <a name='L94'></a><a href='#L94'>94</a>
160
- <a name='L95'></a><a href='#L95'>95</a>
161
- <a name='L96'></a><a href='#L96'>96</a>
162
- <a name='L97'></a><a href='#L97'>97</a>
163
- <a name='L98'></a><a href='#L98'>98</a>
164
- <a name='L99'></a><a href='#L99'>99</a>
165
- <a name='L100'></a><a href='#L100'>100</a>
166
- <a name='L101'></a><a href='#L101'>101</a>
167
- <a name='L102'></a><a href='#L102'>102</a>
168
- <a name='L103'></a><a href='#L103'>103</a>
169
- <a name='L104'></a><a href='#L104'>104</a>
170
- <a name='L105'></a><a href='#L105'>105</a>
171
- <a name='L106'></a><a href='#L106'>106</a>
172
- <a name='L107'></a><a href='#L107'>107</a>
173
- <a name='L108'></a><a href='#L108'>108</a>
174
- <a name='L109'></a><a href='#L109'>109</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
140
+ <a name='L75'></a><a href='#L75'>75</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
175
141
  <span class="cline-any cline-neutral">&nbsp;</span>
176
142
  <span class="cline-any cline-neutral">&nbsp;</span>
177
143
  <span class="cline-any cline-neutral">&nbsp;</span>
@@ -238,40 +204,6 @@
238
204
  <span class="cline-any cline-neutral">&nbsp;</span>
239
205
  <span class="cline-any cline-yes">21x</span>
240
206
  <span class="cline-any cline-yes">21x</span>
241
- <span class="cline-any cline-neutral">&nbsp;</span>
242
- <span class="cline-any cline-yes">6x</span>
243
- <span class="cline-any cline-yes">6x</span>
244
- <span class="cline-any cline-neutral">&nbsp;</span>
245
- <span class="cline-any cline-yes">6x</span>
246
- <span class="cline-any cline-yes">6x</span>
247
- <span class="cline-any cline-yes">6x</span>
248
- <span class="cline-any cline-neutral">&nbsp;</span>
249
- <span class="cline-any cline-neutral">&nbsp;</span>
250
- <span class="cline-any cline-yes">2x</span>
251
- <span class="cline-any cline-neutral">&nbsp;</span>
252
- <span class="cline-any cline-yes">6x</span>
253
- <span class="cline-any cline-yes">2x</span>
254
- <span class="cline-any cline-neutral">&nbsp;</span>
255
- <span class="cline-any cline-neutral">&nbsp;</span>
256
- <span class="cline-any cline-neutral">&nbsp;</span>
257
- <span class="cline-any cline-yes">2x</span>
258
- <span class="cline-any cline-yes">2x</span>
259
- <span class="cline-any cline-yes">2x</span>
260
- <span class="cline-any cline-neutral">&nbsp;</span>
261
- <span class="cline-any cline-neutral">&nbsp;</span>
262
- <span class="cline-any cline-yes">2x</span>
263
- <span class="cline-any cline-yes">2x</span>
264
- <span class="cline-any cline-yes">2x</span>
265
- <span class="cline-any cline-neutral">&nbsp;</span>
266
- <span class="cline-any cline-neutral">&nbsp;</span>
267
- <span class="cline-any cline-neutral">&nbsp;</span>
268
- <span class="cline-any cline-neutral">&nbsp;</span>
269
- <span class="cline-any cline-neutral">&nbsp;</span>
270
- <span class="cline-any cline-yes">6x</span>
271
- <span class="cline-any cline-yes">6x</span>
272
- <span class="cline-any cline-yes">6x</span>
273
- <span class="cline-any cline-neutral">&nbsp;</span>
274
- <span class="cline-any cline-neutral">&nbsp;</span>
275
207
  <span class="cline-any cline-yes">6x</span>
276
208
  <span class="cline-any cline-neutral">&nbsp;</span>
277
209
  <span class="cline-any cline-neutral">&nbsp;</span>
@@ -346,41 +278,7 @@ export class TaskRunner&lt;TContext&gt; {
346
278
  &nbsp;
347
279
  const validationResult = this.validator.validate(taskGraph);
348
280
  if (!validationResult.isValid) {
349
- // Construct error message compatible with legacy tests
350
- const affectedTasks = new Set&lt;string&gt;();
351
- const errorDetails: string[] = [];
352
- &nbsp;
353
- for (const error of validationResult.errors) {
354
- errorDetails.push(error.message);
355
- switch (error.type) {
356
- case "cycle": {
357
- // details is { cyclePath: string[] }
358
- const path = (error.details as { cyclePath: string[] }).cyclePath;
359
- // 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
360
- path.forEach((t) =&gt; affectedTasks.add(t));
361
- break;
362
- }
363
- case "missing_dependency": {
364
- // details is { taskId: string, missingDependencyId: string }
365
- const d = error.details as { taskId: string };
366
- affectedTasks.add(d.taskId);
367
- break;
368
- }
369
- case "duplicate_task": {
370
- const d = error.details as { taskId: string };
371
- affectedTasks.add(d.taskId);
372
- break;
373
- }
374
- }
375
- }
376
- &nbsp;
377
- // Legacy error format: "Circular dependency or missing dependency detected. Unable to run tasks: A, B"
378
- const taskList = Array.from(affectedTasks).join(", ");
379
- const legacyMessage = `Circular dependency or missing dependency detected. Unable to run tasks: ${taskList}`;
380
- const detailedMessage = `Task graph validation failed: ${errorDetails.join("; ")}`;
381
- &nbsp;
382
- // Combine them to satisfy both legacy tests (checking for legacy message) and new requirements (clear details)
383
- throw new Error(`${legacyMessage} | ${detailedMessage}`);
281
+ throw new Error(this.validator.createErrorMessage(validationResult));
384
282
  }
385
283
  &nbsp;
386
284
  const executor = new WorkflowExecutor(this.context, this.eventBus);
@@ -394,7 +292,7 @@ export class TaskRunner&lt;TContext&gt; {
394
292
  <div class='footer quiet pad2 space-top1 center small'>
395
293
  Code coverage generated by
396
294
  <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
397
- at 2026-01-18T05:44:30.766Z
295
+ at 2026-01-18T14:25:37.520Z
398
296
  </div>
399
297
  <script src="../prettify.js"></script>
400
298
  <script>
package/coverage/lcov-report/src/WorkflowExecutor.ts.html CHANGED
@@ -25,28 +25,28 @@
25
25
  <div class='fl pad1y space-right2'>
26
26
  <span class="strong">100% </span>
27
27
  <span class="quiet">Statements</span>
28
- <span class='fraction'>43/43</span>
28
+ <span class='fraction'>48/48</span>
29
29
  </div>
30
30
 
31
31
 
32
32
  <div class='fl pad1y space-right2'>
33
33
  <span class="strong">100% </span>
34
34
  <span class="quiet">Branches</span>
35
- <span class='fraction'>20/20</span>
35
+ <span class='fraction'>14/14</span>
36
36
  </div>
37
37
 
38
38
 
39
39
  <div class='fl pad1y space-right2'>
40
40
  <span class="strong">100% </span>
41
41
  <span class="quiet">Functions</span>
42
- <span class='fraction'>9/9</span>
42
+ <span class='fraction'>5/5</span>
43
43
  </div>
44
44
 
45
45
 
46
46
  <div class='fl pad1y space-right2'>
47
47
  <span class="strong">100% </span>
48
48
  <span class="quiet">Lines</span>
49
- <span class='fraction'>42/42</span>
49
+ <span class='fraction'>48/48</span>
50
50
  </div>
51
51
 
52
52
 
@@ -168,7 +168,26 @@
168
168
  <a name='L103'></a><a href='#L103'>103</a>
169
169
  <a name='L104'></a><a href='#L104'>104</a>
170
170
  <a name='L105'></a><a href='#L105'>105</a>
171
- <a name='L106'></a><a href='#L106'>106</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
171
+ <a name='L106'></a><a href='#L106'>106</a>
172
+ <a name='L107'></a><a href='#L107'>107</a>
173
+ <a name='L108'></a><a href='#L108'>108</a>
174
+ <a name='L109'></a><a href='#L109'>109</a>
175
+ <a name='L110'></a><a href='#L110'>110</a>
176
+ <a name='L111'></a><a href='#L111'>111</a>
177
+ <a name='L112'></a><a href='#L112'>112</a>
178
+ <a name='L113'></a><a href='#L113'>113</a>
179
+ <a name='L114'></a><a href='#L114'>114</a>
180
+ <a name='L115'></a><a href='#L115'>115</a>
181
+ <a name='L116'></a><a href='#L116'>116</a>
182
+ <a name='L117'></a><a href='#L117'>117</a>
183
+ <a name='L118'></a><a href='#L118'>118</a>
184
+ <a name='L119'></a><a href='#L119'>119</a>
185
+ <a name='L120'></a><a href='#L120'>120</a>
186
+ <a name='L121'></a><a href='#L121'>121</a>
187
+ <a name='L122'></a><a href='#L122'>122</a>
188
+ <a name='L123'></a><a href='#L123'>123</a>
189
+ <a name='L124'></a><a href='#L124'>124</a>
190
+ <a name='L125'></a><a href='#L125'>125</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
172
191
  <span class="cline-any cline-neutral">&nbsp;</span>
173
192
  <span class="cline-any cline-neutral">&nbsp;</span>
174
193
  <span class="cline-any cline-neutral">&nbsp;</span>
@@ -177,15 +196,15 @@
177
196
  <span class="cline-any cline-neutral">&nbsp;</span>
178
197
  <span class="cline-any cline-neutral">&nbsp;</span>
179
198
  <span class="cline-any cline-neutral">&nbsp;</span>
180
- <span class="cline-any cline-yes">15x</span>
199
+ <span class="cline-any cline-yes">18x</span>
181
200
  <span class="cline-any cline-neutral">&nbsp;</span>
182
201
  <span class="cline-any cline-neutral">&nbsp;</span>
183
202
  <span class="cline-any cline-neutral">&nbsp;</span>
184
203
  <span class="cline-any cline-neutral">&nbsp;</span>
185
204
  <span class="cline-any cline-neutral">&nbsp;</span>
186
205
  <span class="cline-any cline-neutral">&nbsp;</span>
187
- <span class="cline-any cline-yes">15x</span>
188
- <span class="cline-any cline-yes">15x</span>
206
+ <span class="cline-any cline-yes">18x</span>
207
+ <span class="cline-any cline-yes">18x</span>
189
208
  <span class="cline-any cline-neutral">&nbsp;</span>
190
209
  <span class="cline-any cline-neutral">&nbsp;</span>
191
210
  <span class="cline-any cline-neutral">&nbsp;</span>
@@ -194,83 +213,102 @@
194
213
  <span class="cline-any cline-neutral">&nbsp;</span>
195
214
  <span class="cline-any cline-neutral">&nbsp;</span>
196
215
  <span class="cline-any cline-neutral">&nbsp;</span>
197
- <span class="cline-any cline-yes">15x</span>
216
+ <span class="cline-any cline-yes">18x</span>
198
217
  <span class="cline-any cline-neutral">&nbsp;</span>
199
- <span class="cline-any cline-yes">15x</span>
200
- <span class="cline-any cline-yes">15x</span>
218
+ <span class="cline-any cline-yes">18x</span>
219
+ <span class="cline-any cline-yes">18x</span>
220
+ <span class="cline-any cline-yes">18x</span>
201
221
  <span class="cline-any cline-neutral">&nbsp;</span>
202
222
  <span class="cline-any cline-neutral">&nbsp;</span>
203
- <span class="cline-any cline-yes">15x</span>
204
- <span class="cline-any cline-yes">35x</span>
205
- <span class="cline-any cline-yes">98x</span>
223
+ <span class="cline-any cline-yes">18x</span>
224
+ <span class="cline-any cline-neutral">&nbsp;</span>
225
+ <span class="cline-any cline-yes">18x</span>
226
+ <span class="cline-any cline-neutral">&nbsp;</span>
227
+ <span class="cline-any cline-yes">25x</span>
228
+ <span class="cline-any cline-neutral">&nbsp;</span>
229
+ <span class="cline-any cline-yes">25x</span>
230
+ <span class="cline-any cline-neutral">&nbsp;</span>
231
+ <span class="cline-any cline-neutral">&nbsp;</span>
232
+ <span class="cline-any cline-yes">18x</span>
233
+ <span class="cline-any cline-yes">18x</span>
206
234
  <span class="cline-any cline-neutral">&nbsp;</span>
207
235
  <span class="cline-any cline-neutral">&nbsp;</span>
208
236
  <span class="cline-any cline-neutral">&nbsp;</span>
209
- <span class="cline-any cline-yes">35x</span>
210
- <span class="cline-any cline-yes">49x</span>
211
- <span class="cline-any cline-yes">49x</span>
212
- <span class="cline-any cline-yes">39x</span>
213
237
  <span class="cline-any cline-neutral">&nbsp;</span>
214
- <span class="cline-any cline-yes">49x</span>
215
- <span class="cline-any cline-yes">6x</span>
216
238
  <span class="cline-any cline-neutral">&nbsp;</span>
217
239
  <span class="cline-any cline-neutral">&nbsp;</span>
218
240
  <span class="cline-any cline-neutral">&nbsp;</span>
219
- <span class="cline-any cline-yes">6x</span>
220
- <span class="cline-any cline-yes">6x</span>
221
241
  <span class="cline-any cline-neutral">&nbsp;</span>
222
242
  <span class="cline-any cline-neutral">&nbsp;</span>
223
243
  <span class="cline-any cline-neutral">&nbsp;</span>
224
244
  <span class="cline-any cline-neutral">&nbsp;</span>
225
- <span class="cline-any cline-yes">35x</span>
226
- <span class="cline-any cline-yes">98x</span>
227
245
  <span class="cline-any cline-yes">43x</span>
228
- <span class="cline-any cline-yes">98x</span>
229
- <span class="cline-any cline-yes">29x</span>
246
+ <span class="cline-any cline-yes">43x</span>
230
247
  <span class="cline-any cline-neutral">&nbsp;</span>
248
+ <span class="cline-any cline-yes">43x</span>
249
+ <span class="cline-any cline-yes">57x</span>
250
+ <span class="cline-any cline-yes">57x</span>
231
251
  <span class="cline-any cline-neutral">&nbsp;</span>
232
252
  <span class="cline-any cline-neutral">&nbsp;</span>
253
+ <span class="cline-any cline-yes">57x</span>
254
+ <span class="cline-any cline-yes">43x</span>
255
+ <span class="cline-any cline-yes">43x</span>
233
256
  <span class="cline-any cline-neutral">&nbsp;</span>
234
- <span class="cline-any cline-yes">35x</span>
235
- <span class="cline-any cline-yes">26x</span>
236
- <span class="cline-any cline-yes">26x</span>
257
+ <span class="cline-any cline-yes">23x</span>
258
+ <span class="cline-any cline-yes">20x</span>
259
+ <span class="cline-any cline-yes">7x</span>
260
+ <span class="cline-any cline-yes">7x</span>
237
261
  <span class="cline-any cline-neutral">&nbsp;</span>
238
- <span class="cline-any cline-yes">26x</span>
239
- <span class="cline-any cline-yes">26x</span>
240
- <span class="cline-any cline-yes">26x</span>
241
- <span class="cline-any cline-yes">24x</span>
242
262
  <span class="cline-any cline-neutral">&nbsp;</span>
243
- <span class="cline-any cline-yes">2x</span>
244
263
  <span class="cline-any cline-neutral">&nbsp;</span>
264
+ <span class="cline-any cline-yes">57x</span>
265
+ <span class="cline-any cline-yes">7x</span>
245
266
  <span class="cline-any cline-neutral">&nbsp;</span>
246
267
  <span class="cline-any cline-neutral">&nbsp;</span>
247
268
  <span class="cline-any cline-neutral">&nbsp;</span>
248
- <span class="cline-any cline-yes">26x</span>
249
- <span class="cline-any cline-yes">26x</span>
250
- <span class="cline-any cline-yes">26x</span>
269
+ <span class="cline-any cline-yes">7x</span>
270
+ <span class="cline-any cline-yes">7x</span>
271
+ <span class="cline-any cline-yes">7x</span>
272
+ <span class="cline-any cline-yes">50x</span>
273
+ <span class="cline-any cline-yes">31x</span>
274
+ <span class="cline-any cline-yes">31x</span>
251
275
  <span class="cline-any cline-neutral">&nbsp;</span>
252
276
  <span class="cline-any cline-neutral">&nbsp;</span>
253
277
  <span class="cline-any cline-neutral">&nbsp;</span>
254
278
  <span class="cline-any cline-neutral">&nbsp;</span>
255
- <span class="cline-any cline-yes">26x</span>
256
- <span class="cline-any cline-yes">26x</span>
279
+ <span class="cline-any cline-yes">43x</span>
280
+ <span class="cline-any cline-yes">38x</span>
257
281
  <span class="cline-any cline-neutral">&nbsp;</span>
258
- <span class="cline-any cline-yes">26x</span>
259
282
  <span class="cline-any cline-neutral">&nbsp;</span>
260
283
  <span class="cline-any cline-neutral">&nbsp;</span>
284
+ <span class="cline-any cline-yes">43x</span>
285
+ <span class="cline-any cline-yes">31x</span>
286
+ <span class="cline-any cline-yes">31x</span>
261
287
  <span class="cline-any cline-neutral">&nbsp;</span>
288
+ <span class="cline-any cline-yes">31x</span>
262
289
  <span class="cline-any cline-neutral">&nbsp;</span>
263
- <span class="cline-any cline-yes">15x</span>
264
290
  <span class="cline-any cline-neutral">&nbsp;</span>
265
- <span class="cline-any cline-yes">15x</span>
266
291
  <span class="cline-any cline-neutral">&nbsp;</span>
267
- <span class="cline-any cline-yes">20x</span>
268
292
  <span class="cline-any cline-neutral">&nbsp;</span>
269
- <span class="cline-any cline-yes">20x</span>
270
293
  <span class="cline-any cline-neutral">&nbsp;</span>
271
294
  <span class="cline-any cline-neutral">&nbsp;</span>
272
- <span class="cline-any cline-yes">15x</span>
273
- <span class="cline-any cline-yes">15x</span>
295
+ <span class="cline-any cline-neutral">&nbsp;</span>
296
+ <span class="cline-any cline-yes">31x</span>
297
+ <span class="cline-any cline-yes">31x</span>
298
+ <span class="cline-any cline-neutral">&nbsp;</span>
299
+ <span class="cline-any cline-yes">31x</span>
300
+ <span class="cline-any cline-yes">31x</span>
301
+ <span class="cline-any cline-yes">29x</span>
302
+ <span class="cline-any cline-neutral">&nbsp;</span>
303
+ <span class="cline-any cline-yes">2x</span>
304
+ <span class="cline-any cline-neutral">&nbsp;</span>
305
+ <span class="cline-any cline-neutral">&nbsp;</span>
306
+ <span class="cline-any cline-neutral">&nbsp;</span>
307
+ <span class="cline-any cline-neutral">&nbsp;</span>
308
+ <span class="cline-any cline-yes">31x</span>
309
+ <span class="cline-any cline-yes">31x</span>
310
+ <span class="cline-any cline-yes">31x</span>
311
+ <span class="cline-any cline-neutral">&nbsp;</span>
274
312
  <span class="cline-any cline-neutral">&nbsp;</span>
275
313
  <span class="cline-any cline-neutral">&nbsp;</span>
276
314
  <span class="cline-any cline-neutral">&nbsp;</span></td><td class="text"><pre class="prettyprint lang-js">import { TaskStep } from "./TaskStep.js";
@@ -303,79 +341,98 @@ export class WorkflowExecutor&lt;TContext&gt; {
303
341
  &nbsp;
304
342
  const results = new Map&lt;string, TaskResult&gt;();
305
343
  const executingPromises = new Set&lt;Promise&lt;void&gt;&gt;();
344
+ const pendingSteps = new Set(steps);
306
345
  &nbsp;
307
- // Helper to process pending steps and launch ready ones
308
- const processPendingSteps = () =&gt; {
309
- const pendingSteps = steps.filter(
310
- (step) =&gt; !results.has(step.name) &amp;&amp; !this.running.has(step.name)
311
- );
346
+ // Initial pass
347
+ this.processQueue(pendingSteps, results, executingPromises);
312
348
  &nbsp;
313
- // 1. Identify and mark skipped tasks
314
- for (const step of pendingSteps) {
315
- const deps = step.dependencies ?? [];
316
- const failedDep = deps.find(
317
- (dep) =&gt; results.has(dep) &amp;&amp; results.get(dep)?.status !== "success"
318
- );
319
- if (failedDep) {
320
- const result: TaskResult = {
321
- status: "skipped",
322
- message: `Skipped due to failed dependency: ${failedDep}`,
323
- };
324
- results.set(step.name, result);
325
- this.eventBus.emit("taskSkipped", { step, result });
326
- }
327
- }
349
+ while (results.size &lt; steps.length &amp;&amp; executingPromises.size &gt; 0) {
350
+ // Wait for the next task to finish
351
+ await Promise.race(executingPromises);
352
+ // After a task finishes, check for new work
353
+ this.processQueue(pendingSteps, results, executingPromises);
354
+ }
328
355
  &nbsp;
329
- // Re-filter pending steps as some might have been skipped above
330
- const readySteps = steps.filter((step) =&gt; {
331
- if (results.has(step.name) || this.running.has(step.name)) return false;
332
- const deps = step.dependencies ?? [];
333
- return deps.every(
334
- (dep) =&gt; results.has(dep) &amp;&amp; results.get(dep)?.status === "success"
335
- );
336
- });
356
+ this.eventBus.emit("workflowEnd", { context: this.context, results });
357
+ return results;
358
+ }
337
359
  &nbsp;
338
- // 2. Launch ready tasks
339
- for (const step of readySteps) {
340
- this.running.add(step.name);
341
- this.eventBus.emit("taskStart", { step });
360
+ /**
361
+ * Logic to identify tasks that can be started or must be skipped.
362
+ * Iterate only over pending steps to avoid O(N^2) checks on completed tasks.
363
+ */
364
+ private processQueue(
365
+ pendingSteps: Set&lt;TaskStep&lt;TContext&gt;&gt;,
366
+ results: Map&lt;string, TaskResult&gt;,
367
+ executingPromises: Set&lt;Promise&lt;void&gt;&gt;
368
+ ): void {
369
+ const toRemove: TaskStep&lt;TContext&gt;[] = [];
370
+ const toRun: TaskStep&lt;TContext&gt;[] = [];
342
371
  &nbsp;
343
- const taskPromise = (async () =&gt; {
344
- try {
345
- const result = await step.run(this.context);
346
- results.set(step.name, result);
347
- } catch (e) {
348
- results.set(step.name, {
349
- status: "failure",
350
- error: e instanceof Error ? e.message : String(e),
351
- });
352
- } finally {
353
- this.running.delete(step.name);
354
- const result = results.get(step.name)!;
355
- this.eventBus.emit("taskEnd", { step, result });
356
- }
357
- })();
372
+ for (const step of pendingSteps) {
373
+ const deps = step.dependencies ?? [];
374
+ let blocked = false;
375
+ let failedDep: string | undefined;
358
376
  &nbsp;
359
- // Wrap the task promise to ensure we can track it in the Set
360
- const trackedPromise = taskPromise.then(() =&gt; {
361
- executingPromises.delete(trackedPromise);
362
- });
363
- executingPromises.add(trackedPromise);
377
+ for (const dep of deps) {
378
+ const depResult = results.get(dep);
379
+ if (!depResult) {
380
+ // Dependency not finished yet
381
+ blocked = true;
382
+ } else if (depResult.status !== "success") {
383
+ failedDep = dep;
384
+ break;
385
+ }
364
386
  }
365
- };
366
387
  &nbsp;
367
- // Initial check to start independent tasks
368
- processPendingSteps();
388
+ if (failedDep) {
389
+ const result: TaskResult = {
390
+ status: "skipped",
391
+ message: `Skipped due to failed dependency: ${failedDep}`,
392
+ };
393
+ results.set(step.name, result);
394
+ this.eventBus.emit("taskSkipped", { step, result });
395
+ toRemove.push(step);
396
+ } else if (!blocked) {
397
+ toRun.push(step);
398
+ toRemove.push(step);
399
+ }
400
+ }
369
401
  &nbsp;
370
- while (results.size &lt; steps.length &amp;&amp; executingPromises.size &gt; 0) {
371
- // Wait for the next task to finish
372
- await Promise.race(executingPromises);
373
- // After a task finishes, check for new work
374
- processPendingSteps();
402
+ // Cleanup pending set
403
+ for (const step of toRemove) {
404
+ pendingSteps.delete(step);
375
405
  }
376
406
  &nbsp;
377
- this.eventBus.emit("workflowEnd", { context: this.context, results });
378
- return results;
407
+ // Execute ready tasks
408
+ for (const step of toRun) {
409
+ const taskPromise = this.runStep(step, results).then(() =&gt; {
410
+ executingPromises.delete(taskPromise);
411
+ });
412
+ executingPromises.add(taskPromise);
413
+ }
414
+ }
415
+ &nbsp;
416
+ /**
417
+ * Handles the lifecycle of a single task execution.
418
+ */
419
+ private async runStep(step: TaskStep&lt;TContext&gt;, results: Map&lt;string, TaskResult&gt;): Promise&lt;void&gt; {
420
+ this.running.add(step.name);
421
+ this.eventBus.emit("taskStart", { step });
422
+ &nbsp;
423
+ try {
424
+ const result = await step.run(this.context);
425
+ results.set(step.name, result);
426
+ } catch (e) {
427
+ results.set(step.name, {
428
+ status: "failure",
429
+ error: e instanceof Error ? e.message : String(e),
430
+ });
431
+ } finally {
432
+ this.running.delete(step.name);
433
+ const result = results.get(step.name)!;
434
+ this.eventBus.emit("taskEnd", { step, result });
435
+ }
379
436
  }
380
437
  }
381
438
  &nbsp;</pre></td></tr></table></pre>
@@ -385,7 +442,7 @@ export class WorkflowExecutor&lt;TContext&gt; {
385
442
  <div class='footer quiet pad2 space-top1 center small'>
386
443
  Code coverage generated by
387
444
  <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
388
- at 2026-01-18T05:44:30.766Z
445
+ at 2026-01-18T14:25:37.520Z
389
446
  </div>
390
447
  <script src="../prettify.js"></script>
391
448
  <script>