Skip to content

Added notification batching with per-messenger rate limits - #255

Merged
nt0xa merged 6 commits into
mainfrom
feature/notification-batching
Oct 7, 2026
Merged

nt0xa merged 6 commits into
mainfrom
feature/notification-batching

Conversation

@nt0xa

@nt0xa nt0xa commented Oct 7, 2026

Copy link
Copy Markdown
Owner

Bursts of events per payload are summarized into one message per window via batcher, and each messenger is rate limited according to its API limits.

@codecov

codecov Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 54.28571% with 64 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.06%. Comparing base (40326d9) to head (80469a8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/cmd/server/events.go 50.00% 14 Missing and 1 partial ⚠️
internal/modules/slack/notifier.go 0.00% 15 Missing ⚠️
internal/modules/lark/notifier.go 0.00% 13 Missing ⚠️
internal/modules/telegram/notifier.go 0.00% 13 Missing ⚠️
internal/cmd/server/run.go 0.00% 6 Missing ⚠️
internal/templates/templates.go 77.77% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
+ Coverage   43.65%   44.06%   +0.41%     
==========================================
  Files         216      216              
  Lines        8176     8305     +129     
==========================================
+ Hits         3569     3660      +91     
- Misses       4326     4360      +34     
- Partials      281      285       +4     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nt0xa nt0xa added notifier Affecting notifiers minor Minor changes feature New feature or request and removed feature New feature or request labels Oct 7, 2026
@nt0xa
nt0xa merged commit 581aeed into main Oct 7, 2026
4 checks passed
@nt0xa
nt0xa deleted the feature/notification-batching branch October 7, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Minor changes notifier Affecting notifiers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant