fix: correctly decrement remaining_deps for multiple skipped parents

[jumski]

Fix remaining_deps decrement for multiple skipped parents

This PR fixes a bug where child steps with multiple skipped parent steps would not have their remaining_deps decremented correctly. Previously, when multiple parent steps were skipped simultaneously, the child's remaining_deps would only be decremented by 1 regardless of how many parents were skipped.

The fix:

• Added a new CTE skipped_parent_counts that counts how many skipped parents each child step has
• Modified the dependent_updates query to decrement remaining_deps by the actual count of skipped parents
• Added two test cases to verify the fix:
  1. multi_parent_skip_decrements_remaining_deps.test.sql - Tests that when two parent steps are skipped, the child's remaining_deps is decremented by 2
  2. multi_parent_skip_then_complete.test.sql - Tests a more complex scenario with three parents where two are skipped and one completes normally

This ensures that workflows with conditional branches properly advance when multiple parent steps are skipped in the same cascade resolution.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 0b9906b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[jumski]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• fix: correctly decrement remaining_deps for multiple skipped parents #606 👈 (View in Graphite)
• rename when_failed to when_exhausted for clarity #602
• feat: add support for skipped steps in client #601
• feat: add conditional flow execution and retry handling #600
• docs: add conditional steps documentation and skipped step styling #594
• feat: add conditional step execution with skip infrastructure #572 : 1 other dependent PR (#573 )
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[nx-cloud]

View your CI Pipeline Execution ↗ for commit 0b9906b

Command	Status	Duration	Result
nx run edge-worker:test:integration	✅ Succeeded	5m 37s	View ↗
nx run client:e2e	✅ Succeeded	2m 59s	View ↗
nx affected -t verify-exports --base=origin/mai...	✅ Succeeded	4s	View ↗
nx affected -t build --configuration=production...	✅ Succeeded	4s	View ↗
nx affected -t lint typecheck test --parallel -...	✅ Succeeded	2m 27s	View ↗
nx run core:pgtap	✅ Succeeded	1m 55s	View ↗
nx run cli:e2e	✅ Succeeded	5s	View ↗
nx run edge-worker:e2e	✅ Succeeded	41s	View ↗

---

☁️ Nx Cloud last updated this comment at 2026-02-06 12:23:40 UTC

[github-actions]

🔍 Preview Deployment: Website

✅ Deployment successful!

🔗 Preview URL: https://pr-606.pgflow.pages.dev

📝 Details:

• Branch: 02-06-fix_fixed_decrements_for_remaining_deps
• Commit: 1a48a0a5e270dc43772dc5421a1ac57b0254764f
• View Logs

_Last updated: _