-
Notifications
You must be signed in to change notification settings - Fork 214
feat: add --slack-full-width for full-width Slack alerts with markdown test result tables #2133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
devin-ai-integration
wants to merge
9
commits into
master
from
devin/1772365421-slack-full-width-with-fixes
Closed
Changes from all commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
b4977bd
feat: add --slack-full-width option where test results are shown as a…
michrzan 1945d18
chore: refactor slack message builder to use super()
michrzan e4d81f8
chore: update docs
michrzan bf14655
feat: unit tests
michrzan 99cb783
chore: docs
michrzan 68ddb44
fix: sync emojis
michrzan 61a99a1
Merge branch 'master' into slack-rick-text-refactor
haritamar 659064f
fix: address CodeRabbit review issues on slack-full-width PR
devin-ai-integration[bot] db96f8e
fix: enforce max_length guarantee when single row exceeds limit
devin-ai-integration[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🌐 Web query:
Slack API maximum blocks per message limit documentation💡 Result:
Slack’s Block Kit documentation states that a message’s
blocksarray can include up to 50 blocks per message (and up to 100 blocks in modals or Home tabs). [1]Docs: Slack Developer Docs → “Blocks” (Block Kit reference). [1]
Sources
[1] Slack Developer Docs — “Blocks” (Block Kit reference): https://docs.slack.dev/reference/block-kit/blocks/
🌐 Web query:
Slack mrkdwn section text field maximum character length limit💡 Result:
In Slack Block Kit, a
sectionblock’stextfield (amrkdwntext object) has a maximum length of 3000 characters. [1][2]If you’re using
section.fields(an array of text objects), each field’stextmax is 2000 characters (max 10 fields). [1]Run end-to-end Slack integration test with large sample payloads before release.
This path combines multiple section blocks with code-fenced sample tables. Slack enforces a 50-block limit per message and 3000-character limit per mrkdwn section text field. Validate that near-limit payloads (with large test result samples) do not trigger
invalid_blocksormsg_too_longerrors against the actual Slack API.🤖 Prompt for AI Agents