Skip to content

fix(readiness): propagate unmet deps from composite parents to children#10

Merged
santoshkumarradha merged 1 commit into
mainfrom
fix/composite-readiness-ancestor-deps
May 5, 2026
Merged

fix(readiness): propagate unmet deps from composite parents to children#10
santoshkumarradha merged 1 commit into
mainfrom
fix/composite-readiness-ancestor-deps

Conversation

@santoshkumarradha
Copy link
Copy Markdown
Member

Summary

  • refresh task_readiness with a recursive parent_task_id ancestor CTE
  • count unmet blocks/feeds_into deps across the task and its ancestors
  • add regression coverage for child, grandchild, satisfied ancestor deps, and flat no-parent behavior

Verification

  • cargo test
  • cargo build --release

Fixes #9

The task_readiness view previously counted unmet deps only on the task itself, so children of composite parents could be promoted to "ready" before their parent's deps were satisfied. Switch to a recursive CTE that walks the parent_task_id chain and unions all ancestors' unmet deps into the task's effective unmet count.

Tests added: child blocked by parent's deps; child promoted when parent's deps satisfied; multi-level grandchild inheritance; flat no-parent behavior preserved.

Closes: composite child promoting before parent deps met.

Fixes #9
@github-actions github-actions Bot added the fix label May 5, 2026
@santoshkumarradha santoshkumarradha enabled auto-merge (squash) May 5, 2026 19:34
@santoshkumarradha santoshkumarradha merged commit 1cc8dd7 into main May 5, 2026
3 checks passed
@santoshkumarradha santoshkumarradha deleted the fix/composite-readiness-ancestor-deps branch May 5, 2026 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix readiness propagation for children of composite tasks

1 participant