tree 84c8d40e7c0545c63e7a8ea56a71fa109e91f028
parent 7fbc8e43ff52b6f5dc69fdd26ad7388bdac70435
author Josh Bleecher Snyder <josharian@gmail.com> 1748015382 +0000
committer Josh Bleecher Snyder <josharian@gmail.com> 1748548241 -0700

loop: ensure proper blank line before git commit trailers

The prepare-commit-msg hook was incorrectly adding only a single newline
before trailers in some cases, resulting in missing blank lines between
the commit message body and trailers like Co-Authored-By and Change-ID.

This was particularly noticeable after rebasing where commit messages
would have:
- Subject line
- Single newline (no blank line)
- Trailers

The fix improves the logic to:
- Check if file ends with newline using tail -c 1
- If no trailing newline: add two newlines (complete line + blank line)
- If trailing newline exists: check if last line is empty
- If last line has content: add one newline for blank line
- If last line is empty: add nothing (blank line already exists)

This ensures there's always exactly one blank line before trailers,
following proper git commit message formatting conventions.

Co-Authored-By: sketch <hello@sketch.dev>
Change-ID: s796b26b31c71b2e2k