Conversation
Signed-off-by: Uroš Marolt <uros@marolt.me>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
2 similar comments
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
There was a problem hiding this comment.
Pull request overview
Adjusts Slack notification formatting to better respect Slack’s per-section text limits by splitting long section content into multiple blocks.
Changes:
- Introduces a conservative
MAX_SECTION_TEXTlimit for Slack section blocks. - Updates
buildContentBlocksto split oversized section text at newline boundaries into multiple section blocks. - Simplifies the single-string content path to return a single section block.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Uroš Marolt <uros@marolt.me>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
Note
Low Risk
Low risk: changes only Slack notification block construction to avoid per-section length errors, though it could slightly alter message formatting/truncation for very long lines.
Overview
Fixes Slack Block Kit section length violations by introducing
splitIntoSectionBlocks, which splits long message text into multiplesectionblocks at newline boundaries and hard-truncates any single line that exceeds the per-section character limit.buildContentBlocksnow uses this splitter for both plain-string content and titledSlackMessageSectioncontent, reducing the chance of Slack rejecting oversized section blocks.Reviewed by Cursor Bugbot for commit a4986c1. Bugbot is set up for automated code reviews on this repo. Configure here.