Skip to content

Feat: Unified Notification System with Smart Alerting - #52

Merged
akargi merged 10 commits into
Chulilee:mainfrom
memplethee-lab:feat/notification
Aug 22, 2026
Merged

Feat: Unified Notification System with Smart Alerting#52
akargi merged 10 commits into
Chulilee:mainfrom
memplethee-lab:feat/notification

Conversation

@memplethee-lab

Copy link
Copy Markdown
Contributor

Closes #42

Summary

Adds smart notification alerting for blockchain events, including Telegram and Discord delivery, user preference routing, throttling, deduplication, and analytics tracking.

Related Issues

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that changes existing behaviour)
  • Documentation only
  • Refactor / chore (no functional change)

Changes

  • Add Telegram notification provider.
  • Add Discord webhook notification provider.
  • Add Telegram and Discord notification channels.
  • Add configurable blockchain alert rules.
  • Add alert rule evaluation and affected-user resolution.
  • Add notification throttling and event deduplication with Redis.
  • Add notification analytics and delivery metrics.
  • Route alerts through enabled user notification preferences.
  • Register the new providers and alerting services in the notifications module.
  • Add required notification dependencies and type definitions.

Checklist

  • I have read the CONTRIBUTING guide.
  • npm run lint passes.
  • npx tsc --noEmit passes (no type errors).
  • npm run build succeeds.
  • npm test passes, and I added/updated tests for my change where it
    made sense.
  • Any Stellar/Soroban network access goes through the stellar module
    services (no direct Horizon/RPC calls from feature modules).
  • I did not commit secrets, private keys, or mainnet credentials.

Notes for Reviewers

  • Telegram delivery requires TELEGRAM_BOT_TOKEN.
  • Discord delivery requires DISCORD_WEBHOOK_URL.
  • External providers remain disabled when their credentials are not configured.
  • The feature uses Redis for blockchain event subscription, throttling,
    deduplication, and analytics counters.
  • npm run build was verified successfully.
  • Lint, typecheck, and test commands were not run as part of this change.

@memplethee-lab memplethee-lab changed the title Feat/notification Feat: Unified Notification System with Smart Alerting Aug 22, 2026
@akargi
akargi merged commit 4674eef into Chulilee:main Aug 22, 2026
1 check 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.

[Feature]: Unified Notification System with Smart Alerting

2 participants