Skip to content

feat: process commit detection and revert type support#161

Closed
stasson wants to merge 5 commits intomainfrom
160-process-commits
Closed

feat: process commit detection and revert type support#161
stasson wants to merge 5 commits intomainfrom
160-process-commits

Conversation

@stasson
Copy link
Contributor

@stasson stasson commented Mar 16, 2026

Summary

Closes #160

  • Add is_process_commit() classifier to standard-commit crate — detects merge commits (GitHub, GitLab, Gerrit), reverts, fixups, squash, and initial commits via first-line prefix matching
  • Add revert as 11th default commit type with patch bump rule and "Reverts" changelog section
  • check --range silently skips process commits and reports count separately
  • bump explicitly filters process commits before version calculation
  • Update SPEC.md with §2.3.3 process commit detection and revert type throughout

Test plan

  • All 160+ workspace tests pass (cargo test --workspace)
  • Zero clippy warnings (cargo clippy --workspace -- -D warnings)
  • Manual: git std check --range on a repo with merge commits — merges skipped, not failed
  • Manual: git std check "revert: undo auth changes" — valid

🤖 Generated with Claude Code

stasson and others added 5 commits March 16, 2026 20:09
Add `is_process_commit()` function to detect platform-generated
commits (GitHub/GitLab/Gerrit merges, git revert, fixup, squash,
and initial commits) using simple first-line prefix matching.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add `revert` as the 11th default conventional commit type. Reverts
trigger a patch bump and appear in changelogs under a "Reverts"
section.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Process commits (merges, reverts, fixups) are now silently skipped
in `check --range` instead of being reported as failures. The count
of skipped commits is shown separately. JSON output includes a
`skipped` field for process commits.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Explicitly filter out process commits in both semver and calver
bump paths before parsing, making the filtering intentional rather
than relying on implicit parse failures.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Update SPEC.md with revert in default types, bump rules showing
revert as patch, and new section 2.3.3 documenting process commit
detection patterns and behaviour in check and bump.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@stasson
Copy link
Contributor Author

stasson commented Mar 16, 2026

Off-track — this was meant to be #138 (scheme=patch). Closing.

@stasson stasson closed this Mar 16, 2026
@stasson stasson deleted the 160-process-commits branch March 16, 2026 21:31
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.

Process commit detection and revert type support

1 participant