Skip to content

Fix commit-message hook edge cases#129

Open
marcus wants to merge 1 commit intomainfrom
feat/commit-message-normalizer-v15
Open

Fix commit-message hook edge cases#129
marcus wants to merge 1 commit intomainfrom
feat/commit-message-normalizer-v15

Conversation

@marcus
Copy link
Copy Markdown
Owner

@marcus marcus commented Apr 11, 2026

Summary

  • add a td-native commit-message normalizer command and reusable internal parser/rewriter
  • preserve merge and autosquash subjects while keeping commit bodies and trailers untouched
  • make make install-hooks safe from linked worktrees and align contributor docs on the canonical workflow

Testing

  • go test ./internal/git
  • go test ./cmd -run 'TestCommitMessage|TestInstallHooks'
  • go test ./...
    • note: this still hits the pre-existing failure in test/e2e/TestChaosSync with seed 1775903130835843000, reproduced on clean main as well
  • manual td commit-message --file checks for normalized, fixup, and merge subjects

Supersedes the earlier v14 attempt / PR #128 by addressing the linked-worktree, autosquash, and merge-message regressions from review.


Automated by nightshift

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