ci(commits): check conventional commit style and dispatch the compaction bot - #755
Merged
Merged
Conversation
Graffioh
force-pushed
the
ci/compact-merge-bot
branch
from
September 23, 2026 12:19
f75eddb to
7051e5c
Compare
4 tasks
Graffioh
force-pushed
the
ci/compact-merge-bot
branch
2 times, most recently
from
September 23, 2026 12:41
8dfe893 to
2824081
Compare
Graffioh
force-pushed
the
ci/compact-merge-bot
branch
2 times, most recently
from
September 23, 2026 13:01
c044724 to
e866091
Compare
Collaborator
Author
|
@cubic-dev-ai review this PR |
Contributor
@Graffioh cubic can't start this review because your workspace has reached its free monthly review limit. cubic has reviewed 162,810 of the 160,000 allowed lines of code this month. Reviews resume on 1 October 2026 (in 8 days). Paid plans include much higher monthly review limits. Upgrade now to resume reviews. To help optimise your usage, you can tune cubic to get the most out of your usage limits:
|
Graffioh
marked this pull request as ready for review
September 23, 2026 13:06
Graffioh
marked this pull request as draft
September 23, 2026 13:07
Graffioh
force-pushed
the
ci/compact-merge-bot
branch
from
September 23, 2026 13:16
e866091 to
7d07322
Compare
Graffioh
force-pushed
the
ci/compact-merge-bot
branch
2 times, most recently
from
September 23, 2026 13:25
274e94c to
be99bdd
Compare
Graffioh
marked this pull request as ready for review
September 23, 2026 13:32
…ion bot Commit style (commit-style.yml) checks every non-merge commit in a pull request. Subjects must look like `type(scope): summary`, with a type from CONTRIBUTING.md plus build and revert and a lowercase scope. They must be at most 100 characters with no trailing period, and a blank line must separate the subject from the body. GitHub's `Revert "..."` subjects pass as they are. Compact commits (compact-commits.yml) forwards a pull request to the commit compaction bot when a maintainer adds the `compact-commits` label; it checks out and runs nothing. The bot folds the commits into a few conventional commits without changing the code, validates the messages with this repository's check_commit_messages.py, force-pushes, and comments the old-to-new mapping. It never merges. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…s label Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Graffioh
force-pushed
the
ci/compact-merge-bot
branch
from
September 23, 2026 14:01
be99bdd to
1c7be1e
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What
Two workflows for commit history.
1. Commit style check:
commit-style.yml(no LLM)It checks every non-merge commit in a PR with a plain regex (
.github/ci/check_commit_messages.py):type(scope): summary, where type is one of feat, fix, perf, refactor, docs, test, bench, build, ci, chore, revert (CONTRIBUTING's list plusbuild/revert, which main already uses).fix(server): .... A barefix: ...fails.Revert "..."subjects and merge commits.These limits come from the last 600 commits on main: 105 subjects exceed 72 characters, 3 exceed 100, none end with a period, and every scope is lowercase. 36 of those 600 have no scope and would now fail. On real history it passes #743 and #733 and flags #738's four
ggml-cuda: .../harness: ...subjects. Failures point to thecompact-commitslabel.2. Compaction bot trigger:
compact-commits.ymlAdding the
compact-commitslabel forwards the PR to the commit compaction bot, which runs outside this repository.permissions: {}.check_commit_messages.py;Setup
LUCEBOX_BOTS_DISPATCH_TOKENandLUCEBOX_BOTS_DISPATCH_REPO(see the maintainers' setup notes).COMPACT_BOT_TOKEN,OPENROUTER_API_KEYandCOMPACT_MODELsecrets.compact-commitslabel.Testing
uv-workspacejob.🤖 Generated with Claude Code