Skip to content

Skip paths archive sibling messages but never terminalize sibling step_tasks rows, leaving status='started' tasks on completed runs - #649

Open
jumski wants to merge 1 commit into
mainfrom
issue-638-terminalize-skipped-tasks
Open

Skip paths archive sibling messages but never terminalize sibling step_tasks rows, leaving status='started' tasks on completed runs#649
jumski wants to merge 1 commit into
mainfrom
issue-638-terminalize-skipped-tasks

Conversation

@jumski

@jumski jumski commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add skipped as a terminal step_tasks.status value.
  • Terminalize queued and started task rows when fail_task or _cascade_force_skip_steps skips their parent step, while preserving message archival, events, counters, callbacks, and return values.
  • Backfill existing active task rows under skipped steps, document the status, and add an @pgflow/core patch changeset.

Checks

  • Six focused pgTAP regression files pass: 40 assertions total.
  • pnpm nx verify-migrations core --skip-nx-cache
  • pnpm nx gen-types core --skip-nx-cache with no generated type diff
  • pnpm nx verify-gen-types core --skip-nx-cache
  • pnpm nx test:pgtap core --skip-nx-cache — 279 files, 1,259 tests
  • Migration backfill fixture — failed, completed, started, queued becomes failed, completed, skipped, skipped
  • /usr/bin/git diff-tree --check HEAD^ HEAD

pnpm nx fix-sql core hits the existing local toolchain error sqruff: error: unexpected argument '--force' found; direct Sqruff formatting reported no changes.

Closes #638

@changeset-bot

changeset-bot Bot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5ec80bf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@pgflow/core Patch
pgflow Patch
@pgflow/client Patch
@pgflow/edge-worker Patch
@pgflow/dsl Patch

Not sure what this means? Click here to learn what changesets are.

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

jumski commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

@jumski jumski changed the title fix(core): terminalize skipped tasks Skip paths archive sibling messages but never terminalize sibling step_tasks rows, leaving status='started' tasks on completed runs Aug 21, 2026
@nx-cloud

nx-cloud Bot commented Aug 21, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 5ec80bf

Command Status Duration Result
nx run edge-worker:e2e:portable-runtimes ✅ Succeeded 1m 2s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-08-22 00:29:57 UTC

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Preview Deployment: Website

Deployment successful!

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

📝 Details:

  • Branch: issue-638-terminalize-skipped-tasks
  • Commit: c36c7dc25ef2a8b44e58718e6a8ced0734a16c9f
  • View Logs

_Last updated: _

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skip paths archive sibling messages but never terminalize sibling step_tasks rows, leaving status='started' tasks on completed runs

1 participant