Skip to content

feat: add commit-msg hook and simplify session-commit#47

Open
Cragsmann wants to merge 2 commits into
functions-dev:masterfrom
Cragsmann:SRVOCF-950--commit-msg-hook-and-slash-commands
Open

feat: add commit-msg hook and simplify session-commit#47
Cragsmann wants to merge 2 commits into
functions-dev:masterfrom
Cragsmann:SRVOCF-950--commit-msg-hook-and-slash-commands

Conversation

@Cragsmann

Copy link
Copy Markdown
Collaborator

Summary

  • Add Husky pre-commit (lint) and commit-msg hooks
  • commit-msg hook validates: conventional commit type prefix, authorship trailer (Co-Authored-By or Signed-off-by), no em dashes
  • Merge, fixup, and squash commits are exempt
  • Simplify session-commit: remove progress log and blocker migration steps, add hook failure handling
  • Remove redundant commit-user command

Issue: SRVOCF-950

Test plan

  • Verify tandem commit (with Co-Authored-By) passes the hook
  • Verify autonomous commit (with Signed-off-by) passes the hook
  • Verify commit without authorship trailer is rejected
  • Verify commit without conventional type prefix is rejected
  • Verify commit with em dash is rejected
  • Verify merge commits bypass the hook
  • Run /session-commit and confirm it produces a valid commit

Cragsmann and others added 2 commits June 24, 2026 09:57
Add Husky pre-commit (lint) and commit-msg hooks. The commit-msg
hook validates conventional commit type prefix, authorship trailer
(Co-Authored-By or Signed-off-by), and rejects em dashes. Merge,
fixup, and squash commits are exempt.

Simplify session-commit: remove progress log and blocker migration
steps, add hook failure handling. Remove redundant commit-user
command.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: David Simansky <dsimansk@redhat.com>
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