Skip to content

fix(workflows): preserve pending result deliveries - #134

Merged
tt-a1i merged 2 commits into
mainfrom
codex/issue-107-result-delivery
Aug 24, 2026
Merged

fix(workflows): preserve pending result deliveries#134
tt-a1i merged 2 commits into
mainfrom
codex/issue-107-result-delivery

Conversation

@tt-a1i

@tt-a1i tt-a1i commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • retain every failed Workflow completion in memory before attempting durable delivery-state persistence
  • drain an additional batch when a completion or lifecycle flush request arrives during an in-flight delivery
  • isolate per-envelope persistence failures so one run cannot drop or block its siblings
  • restore held-inline completions into the retry queue even when the pending-state write fails

The retry loop remains event-driven: a transport failure does not request its own immediate retry, so an unavailable host cannot create a busy loop.

Validation

  • bun run check
  • bun run test: 897 Node tests and 30 Vitest tests passed
  • focused result-delivery regression suite: 8 passed
  • two-axis review: Standards and Spec

Scope

This fixes the shared result-delivery queue defects in #107, #108, and #109. It does not attempt the separate multi-artifact transaction work in #110 or close the broader historical umbrella #71.

Closes #107
Closes #108
Closes #109

@tt-a1i
tt-a1i merged commit 39c7c32 into main Aug 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant