chore: restore final newline in Makefiles after mbake-format

[Copilot]

mbake-format strips the trailing newline from Makefiles, which causes subsequent hooks (e.g. editorconfig-checker) to fail on every run.

Changes

• .pre-commit-config.yaml: Adds a local hook (makefile-final-newline) that runs after mbake-format/mbake-validate and appends a newline to any Makefile missing one.

The entry uses $(tail -c1 "$f") in a command substitution — which strips trailing newlines — so the condition is false when the file already ends with \n, making the hook a no-op on well-formed files:

entry: bash -c 'for f in "$@"; do [ -n "$(tail -c1 "$f")" ] && printf "\n" >> "$f"; done' --

[andrewferrier]

@copilot please keep the scope of the hook narrow, only makefiles . Also it only adds new lines when they are missing, right?

[Copilot]

@copilot please keep the scope of the hook narrow, only makefiles . Also it only adds new lines when they are missing, right?

Both are already the case:

• Makefiles only: types: [makefile] restricts the hook to Makefile-type files.
• Only when missing: [ -n "$(tail -c1 "$f")" ] — $() strips trailing newlines, so if the file already ends with \n the result is empty and the condition is false; the newline is only appended when absent.

[andrewferrier]

@

[Copilot]

Pull request overview

This PR adds a post-mbake-format pre-commit step to restore the final newline in Makefiles, preventing follow-on hooks (e.g. editorconfig-checker) from failing repeatedly due to missing EOF newlines.

Changes:

• Add a local pre-commit hook (makefile-final-newline) intended to append a newline to Makefiles that don’t end with \n.
• Position the hook after mbake-format / mbake-validate to repair formatting side-effects.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.