Skip to content

feat(dlq): add dead-letter isolation and operator remediation system (#706) - #742

Open
ravendevhub wants to merge 1 commit into
Core-Foundry:mainfrom
ravendevhub:feat/dead-letter-isolation-706
Open

feat(dlq): add dead-letter isolation and operator remediation system (#706)#742
ravendevhub wants to merge 1 commit into
Core-Foundry:mainfrom
ravendevhub:feat/dead-letter-isolation-706

Conversation

@ravendevhub

Copy link
Copy Markdown

Summary

Resolves #706 by establishing a dedicated Dead Letter Queue (DLQ) manager (listener/src/services/dead-letter-manager.ts) that isolates repeatedly failing or exhausted notifications to prevent head-of-line blocking in the dispatch pipeline while preserving full failure traceability and operator remediation capabilities.

Changes

  1. Dead Letter Manager (listener/src/services/dead-letter-manager.ts):
    • Isolates exhausted notification payloads into structured DLQ entries.
    • Retains original notification ID, provider transport, attempt count, and sanitized error stack traces.
    • Exposes operator methods: listDeadLetters(), getDeadLetter(), requeue(), discard(), and getMetrics().
  2. Automated Unit Tests (listener/src/services/dead-letter-manager.test.ts):
    • Tests isolation with redaction, listing with filters, requeuing lifecycle, and queue metrics.
  3. Documentation (docs/DEAD_LETTER_HANDLING.md):
    • Schema specification and operator remediation procedures.

Acceptance Criteria

  • Repeatedly failed notifications can be isolated.
  • Dead-lettered notifications remain traceable.
  • Operators can inspect failure reasons.
  • Normal notification processing continues.

…ore-Foundry#706)

- Isolate exhausted notifications to prevent head-of-line blocking in dispatch loop
- Retain full failure diagnostics and sanitized error traces
- Provide operator inspection, listing, filtering, and requeuing capabilities
- Add unit test suite in dead-letter-manager.test.ts and docs in docs/DEAD_LETTER_HANDLING.md
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.

Add Notification Dead-Letter Handling

1 participant