Skip to content

feat(hook): Deliver events to integrations - #607

Open
prathshenoy wants to merge 1 commit into
prath.shenoy/hook-event-contractfrom
prath.shenoy/hook-dispatch
Open

feat(hook): Deliver events to integrations#607
prathshenoy wants to merge 1 commit into
prath.shenoy/hook-event-contractfrom
prath.shenoy/hook-dispatch

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

What:

  • Consume lifecycle events from a durable queue and hand each one to every integration wired for the deployment, retrying and then dead-lettering the ones that keep failing.
  • Record every dead-lettered event with its full payload and failure reason, and count it on a metric meant to page.

Why:

  • Keep a slow or broken integration from stalling or failing the work that triggered it.
  • Make a lost notification recoverable and impossible to miss, since nothing else in the system notices when one goes missing.

Test Plan

  • Add unit tests.

Revert Plan

  • Revert this PR. No service constructs the dispatcher or subscribes to the hook topic.

Issues

Stack

  1. feat(hook): Add contract for lifecycle events #606
  2. @ feat(hook): Deliver events to integrations #607
  3. feat(orch): Consume hook events #608

@prathshenoy
prathshenoy force-pushed the prath.shenoy/hook-event-contract branch from b5394eb to 837e0d8 Compare August 17, 2026 16:54
@prathshenoy
prathshenoy force-pushed the prath.shenoy/hook-dispatch branch from a09dcab to b6aef89 Compare August 17, 2026 16:56
@prathshenoy
prathshenoy marked this pull request as ready for review August 17, 2026 16:56
@prathshenoy
prathshenoy force-pushed the prath.shenoy/hook-event-contract branch from 837e0d8 to b5394eb Compare August 18, 2026 00:27
@prathshenoy
prathshenoy force-pushed the prath.shenoy/hook-dispatch branch from b6aef89 to a09dcab Compare August 18, 2026 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant