Skip to content

Test: 50 commits with Block Kit and section splitting#131

Closed
duckdum wants to merge 53 commits into
mainfrom
testing/50-commits-block-kit
Closed

Test: 50 commits with Block Kit and section splitting#131
duckdum wants to merge 53 commits into
mainfrom
testing/50-commits-block-kit

Conversation

@duckdum

@duckdum duckdum commented Mar 26, 2026

Copy link
Copy Markdown
Collaborator

Slack message_ts: 1774562553.678649

E2E test for Block Kit output with sort-commits=true, 50 commits across 8 scopes, verifying section splitting for large scopes.

Eduardo de Paula added 30 commits March 26, 2026 21:38
…rade to Block Kit

- Fix regex to allow spaces in multi-scope commits (e.g. "chore(companion, bs, frontend):")
- Place multi-scope commits under first (primary) scope only
- Filter out merge commits (Merge branch, Merge remote-tracking, Merge pull request)
- Add Block Kit formatted output when sort-commits is enabled:
  - Scopes sorted alphabetically with headers
  - Types sorted alphabetically within each scope
  - Commits sorted by author within each type
  - Chunking to respect Slack's 50-block limit
- Handle edge case when all commits are filtered out
- Add error logging for failed Slack block posts
- Enable sort-commits in workflow
The categorizeCommits JSDoc claimed multi-scope commits get an
"also: X, Y" indicator, but this was never implemented.
…imit

Slack section blocks have a 3000-character text limit. When a scope has
many commits, the joined text could exceed this and get silently
truncated. Now splits into multiple section blocks when approaching
the limit.
Eduardo de Paula added 23 commits March 26, 2026 22:01
@duckdum duckdum closed this Mar 26, 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

Development

Successfully merging this pull request may close these issues.

1 participant