Skip to content

feat(notifier): Slack limits and link escaping, resolved/details in chat formats, Teams format - #355

Merged
Valreb001 merged 1 commit into
Tx-wats:mainfrom
gadst12:feature/302-305-chat-format-improvements
Oct 21, 2026
Merged

Valreb001 merged 1 commit into
Tx-wats:mainfrom
gadst12:feature/302-305-chat-format-improvements

Conversation

@gadst12

@gadst12 gadst12 commented Oct 21, 2026 •

Copy link
Copy Markdown

Summary

Closes #302
Closes #303
Closes #304
Closes #305

Validation

  • rustfmt passes on the changed Rust files.
  • Added tests: oversized Slack inputs, Slack URLs with special characters, resolved headline and new details, and a Teams snapshot including the batch digest.
  • I could not compile or run the tests because main already fails to build. crates/config/src/lib.rs:617 has an unclosed AlertRuleRepr enum, and the test fixture in format.rs repeats several fields. Both problems predate this PR and are left unchanged.

…ormat

- Enforce Slack Block Kit limits (3000/2000 chars, max 10 fields)
- Escape URLs inside Slack link markup
- Add resolved headline variant and source/ledger/memo/severity/
  suppressed/matched-event details to chat formats
- Add format = "teams" rendering an Adaptive Card payload

Closes Tx-wats#302
Closes Tx-wats#303
Closes Tx-wats#304
Closes Tx-wats#305
@drips-wave

drips-wave Bot commented Oct 21, 2026

Copy link
Copy Markdown

@gadst12 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Valreb001
Valreb001 merged commit 5ce990b into Tx-wats:main Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants