Development record
git revert writes no Assisted-by trailer, and the attribution commit gate (now also on the merge queue and on push to main, iss-280) refuses the message, so a plain revert can only be satisfied by rewriting history. Carried from the duplicate iss-2609100513521322's third acceptance: the trailer should be written when the message is written (a commit template or a prepare-commit-msg hook), not demanded after.
issueresolvedminoriss-2609251125591539
git revert writes no Assisted-by trailer, and the attribution commit gate (now also on the merge queue and on push to main, iss-280) refuses the message, so a plain revert can only be satisfied by rewriting history. Carried from the duplicate iss-2609100513521322's third acceptance: the trailer should be written when the message is written (a commit template or a prepare-commit-msg hook), not demanded after.
Grounds
- pursued: git revert --no-edit HEAD under git -c abcd.assistedBy=<Vendor>:<model> records a commit carrying that trailer once, and check-attribution.sh passes it; a revert, cherry-pick, squash or amend under a configured value that arrives without the trailer, or any trailer written with the key unset, would show it wrong (internal/surface/cli/githook_preparecommitmsg_test.go drives each source).
Frontmatter
| id | iss-2609251125591539 |
|---|---|
| type | issue |
| lifecycle | resolved |
| severity | minor |
.abcd/work/issues/resolved/iss-2609251125591539-git-revert-writes-no-assisted-by-trailer.md
open on GitHub ↗ · commit history ↗
mentions2
- iss-280main's own commit messages are ungated: Makefile's check-attribution runs origin/main..HEAD, whose base is main itself, so a commit that lands on main (squash, web merge) is never checked by any local or CI run afterwards
- iss-2609100513521322`auto-release` cuts the version tag **before** the gate that could refuse the