Skip to content

Introduce notification deduplication service#71

Merged
Abd-Standard merged 1 commit into
Core-Foundry:mainfrom
eischideraa-unn:backend-notification-deduplication-service
Jun 19, 2026
Merged

Introduce notification deduplication service#71
Abd-Standard merged 1 commit into
Core-Foundry:mainfrom
eischideraa-unn:backend-notification-deduplication-service

Conversation

@eischideraa-unn

@eischideraa-unn eischideraa-unn commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Introduces a configurable notification deduplication service for the listener backend to prevent identical notifications from being processed repeatedly within a time window.

Changes

  • Added TTL-based notification deduplication using generated fingerprints.
  • Added configurable deduplication settings:
    • NOTIFICATION_DEDUPLICATION_WINDOW_MS
    • NOTIFICATION_DEDUPLICATION_MAX_SIZE
  • Added deduplication metrics for:
    • accepted requests
    • skipped duplicates
    • evicted cache entries
    • expired cache entries
    • current cache size
    • configured deduplication window
  • Updated Discord notification handling so duplicate skips are treated as handled, not retryable failures.
  • Added test coverage for duplicate skipping, TTL expiry, metrics, and config loading.

Acceptance Criteria

  • Duplicate notifications are ignored.
  • Legitimate new requests continue processing.
  • Duplicate notifications are allowed again after the configured deduplication window expires.
  • Metrics accurately report skipped duplicates and cache activity.

closes #66

Verification

  • npm test -- --runInBand
    • 10 test suites passed
    • 109 tests passed
  • npm run build
    • TypeScript build passed

@Abd-Standard Abd-Standard merged commit 85871d9 into Core-Foundry:main Jun 19, 2026
0 of 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

Development

Successfully merging this pull request may close these issues.

[Backend] Introduce Notification Deduplication Service

2 participants