lsm / neokai / 25616140083 / 25
82%
dev: 82%

DEFAULT BRANCH: dev
Ran
Files 280
Run time 7s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 23.795% (-0.04%) from 23.83%
25616140083.25

push

github

web-flow 
fix(space): keep open tasks open when a dependency fails (#1834)

* fix(space): keep open tasks open when a dependency fails

`blockDependentTasks` previously cascaded `dependency_failed` to all `open`
tasks that pointed to the failed task. This breaks daemon restart recovery:
a stalled `in_progress` run gets flagged `blocked`, and every open task that
was simply waiting on it is also marked `blocked` and never auto-starts when
the dependency eventually completes.

The runtime already skips open tasks with unmet dependencies via
`areDependenciesMet()`, so the cascade only needs to abort tasks that
already have a workflow run attached — i.e. `in_progress` tasks. Open tasks
should stay `open` and be picked up naturally on the next tick.

Updates the cascade tests to cover the new semantics and adds a regression
test for the daemon-restart scenario.

* fix(space): split cascade by parent status; skip blocked in completion path

Addresses two review concerns on the dependency-cascade fix:

- Cancellation propagation (P2): cancelled parents are terminal-but-
  restartable, so dependents must transition out of `open`/`in_progress` —
  otherwise they wait forever on a parent that will never reach `done`.
  Adds `cancelDependentTasks` (cascades `cancelled`) alongside the existing
  `blockDependentTasks` (cascades `blocked`); `updateTaskAndEmit` now picks
  the right cascade based on the parent's terminal status.

- Completion-path safety for cascade-blocked tasks (P1): a task that was
  cascade-blocked while its workflow run was still alive could be pushed
  through `dispatchPostApproval` on run completion, which attempts an
  invalid `blocked → approved` transition. Adds `blocked` to the
  already-resolved skip set in both the active-tick guard
  (`taskAlreadyResolved`) and the terminal-run reconciler.

Tests: new cascade tests for `cancelDependentTasks`; new runtime
completion test asserting a `blocked` canonical task is left alone when
its run finishes.

*... (continued)

17311 of 72751 relevant lines covered (23.79%)

12.31 hits per line

Source Files on job daemon-online-rpc-4 - 25616140083.25