palcarazm / batchjs / 28379735443

97%

version/v1: 100%

push

github

refactor: convert executionPromise from Promise to lazy factory function

- Change doRun() return type from executionPromise: Promise<void> to
  executionPromise: () => Promise<void> across Job, Step, and mocks
- Move executionPromise invocation from transitionTo to executeTransition
  to ensure execution starts only after RUNNING state is set
- Refactor Job._executePlan from nested Promise with once("started")
  callback to flat async/await with extracted _executePlanSteps method
- Fix error handling in Job._executePlan and Step.doRun to properly await
  transitionTo(FAILED) before re-throwing errors
- Add early continue for empty parallel step groups to avoid unnecessary
  checkpoint store writes
- Update JSDoc comments to reflect lazy execution semantics

The previous implementation started execution promises immediately, before
the RUNNING state was fully established, causing potential race conditions
with "started" event listeners. The new lazy factory pattern ensures
execution only begins after the state transition completes and all event
listeners are registered.

273 of 296 branches covered (92.23%)

Branch coverage included in aggregate %.

33 of 33 new or added lines in 3 files covered. (100.0%)

778 of 786 relevant lines covered (98.98%)

185.03 hits per line