Skip to content

feat(triage): add a built-in conventional commits rule - #379

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-10919-add-a-built-in-check-for-conventional-commit-titles-and
Oct 19, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-10919-add-a-built-in-check-for-conventional-commit-titles-and

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Closes CL-10919

What changed

  • New rule check: conventional-commits (no model). The pull request title and each commit's first line (first 100 commits) must read type(scope)!: subject with an allowed type (case-insensitive, per the spec); merge commits and GitHub's Revert "…" subjects are skipped, in the title and the commits. Options: types to override the default list (feat, fix, refactor, perf, docs, test, chore, ci, build, style, revert) and fields: [title, commits] (default both).
  • A failure names up to three offenders, e.g. "Add tests for live run stream isn't a conventional commit", then "and N more".
  • No new host calls: GitHub and Bitbucket Data Center already read commits.
  • The setup skill's phrase-based conventional-title recipe is replaced by the built-in rule; PRODUCT and the user manual's Rules section updated.

How it was verified

  • bun run check; contracts 113, triage-workflows 194, web lib 197, policy-check-action 2.
  • Tests through the real policy file: scope with !, merge commits skipped, bad type, missing colon or space, custom types, title only. End-to-end test shaped like feat(chat): stream live run progress and render saved inference traces workbench#1125 gives Fail naming the three commits.
  • Judge requests unchanged: the 54 eval cases (186 requests) are byte-identical on main and here.

@linear-code

linear-code Bot commented Oct 19, 2026

Copy link
Copy Markdown

CL-10919

@TheGreatAxios
TheGreatAxios force-pushed the cl-10919-add-a-built-in-check-for-conventional-commit-titles-and branch from ffb6f27 to e081a81 Compare October 19, 2026 18:55
@TheGreatAxios
TheGreatAxios marked this pull request as ready for review October 19, 2026 18:56
@corbits-triage

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Triage will approve.
Waiting on a maintainer to approve: request review.

@TheGreatAxios
TheGreatAxios merged commit 1436a9e into main Oct 19, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant