Skip to content

fix: persist serialized webhook consumer lifecycle - #15

Merged
09473ZH merged 2 commits into
agent/codex-long-sessions-cleanfrom
fix/durable-webhook-lifecycle-285
Sep 24, 2026
Merged

09473ZH merged 2 commits into
agent/codex-long-sessions-cleanfrom
fix/durable-webhook-lifecycle-285

Conversation

@09473ZH

@09473ZH 09473ZH commented Sep 23, 2026

Copy link
Copy Markdown

Serialized event senders currently cannot distinguish a running consumer from a consumer that ended without business ACK. Lost responses and restarts can consequently restart work or leave the sender polling the wrong state.

Persist admission by route and stable delivery identity before starting work. Authenticated batch status/cancel requests share route authentication and bypass model admission quota. Receiver restart exposes interrupted work; the sender owns explicit recovery for serialized event routes. Cancellation persists a tombstone before stopping the consumer; consumers sharing other obligations remain running until those obligations finish or are cancelled.

Validation: canonical scripts/run_tests.sh -j 1 passed 80 tests across seven webhook/restart test files. Real HTTP replay covers lost responses, repeated admission, busy scopes, normal completion, active cancellation, late delivery and durable restart. The additional shared-consumer cancellation check also passes. New files pass Ruff; changed modules compile. This fork has no pre-commit configuration.

Dependency for CorgiCortex/Qortex-Trading-Harness#285. completed means the consumer ended, not that business ACK or trading execution completed. Production deployment remains paused pending the Harness integration and separate hardware validation.

@09473ZH
09473ZH requested a review from corgi-bot September 23, 2026 16:20
@09473ZH

09473ZH commented Sep 23, 2026

Copy link
Copy Markdown
Author

Harness integration: https://github.com/CorgiCortex/Qortex-Trading-Harness/pull/286 pins this PR's 2b4dc8369fc57f58967c4888f2d80a8fa63217ff and installs the lifecycle ledger module in the source overlay. Production remains paused; merge/deployment and hardware acceptance are separate.

@09473ZH
09473ZH merged commit 762d98a into agent/codex-long-sessions-clean Sep 24, 2026
35 of 37 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

Development

Successfully merging this pull request may close these issues.

1 participant