X-Git-Url: https://git.giacomolli.uk/onechat/blobdiff_plain/a50886435ab974bee6f96ddc0d92210549a01221..9346d20fe4297dce1aa16171749a7ef3e41ce7bd:/.agents/skills/commit/SKILL.md?ds=inline diff --git a/.agents/skills/commit/SKILL.md b/.agents/skills/commit/SKILL.md index 14ea4b3..11b5eb2 100644 --- a/.agents/skills/commit/SKILL.md +++ b/.agents/skills/commit/SKILL.md @@ -5,24 +5,33 @@ description: Commit procedure and message rules. Use for every commit. # Commit +The procedure applies to normal repository commits, including code and +documentation commits. Each must use the configured signing key. +Internal git-bug commits for issues, comments, status changes, and other +tracker metadata are exempt from signing and may be unsigned. This +exception does not permit unsigned code or documentation commits. + ## Procedure 1. Confirm that the configured signing key is available. If signing is unavailable or fails, stop and report the problem. Never disable signing, bypass it with a command or configuration override, or - create an unsigned commit. + create an unsigned normal repository commit. 2. Do the `check` skill. All checks must pass before you commit. 3. Stage only the files that belong to this change. Keep unrelated changes in separate commits. -4. Write the message to a file or a here-document and sign the commit - with the configured key, using `git commit -S -F`. Never put `\n` in - `git commit -m`. -5. When `.githooks/commit-msg` rejects the message, fix the message and - commit again. Do not skip the hook with `--no-verify`. -6. Cryptographically verify each new commit's signature against the - configured key before reporting completion. A signature header alone - is not verification. If verification fails, stop and report it. Do - not claim completion or publish the commit. +4. Write the complete message to a file, including all trailers. Add + trailers with `git interpret-trailers --trailer` before the check, + not after it. Never put `\n` in `git commit -m`. +5. Run `sh .githooks/commit-msg MESSAGE_FILE` on that final file before + `git commit`. Fix every failure and run the check again. Sign with + the configured key, using `git commit -S -F MESSAGE_FILE` on the same + file. Do not add or change trailers after the check. The normal + commit hook still runs. Never skip it with `--no-verify`. +6. Cryptographically verify each new normal commit's signature against + the configured key before reporting completion. A signature header + alone is not verification. If verification fails, stop and report it. + Do not claim completion or publish the commit. ## Size @@ -46,7 +55,9 @@ description: Commit procedure and message rules. Use for every commit. - Separate the subject from the body with exactly one blank line. - Always write a body. Wrap each line at 72 characters. Fill each line - of a paragraph before you wrap it. + of a paragraph before you wrap it. `bin/reflow` formats source + comments at 80 columns, not commit prose. Do not use it to format a + commit message. - Write in the present-tense declarative mood. - Start with a direct summary of the complete change. Do not only repeat the subject. @@ -60,8 +71,9 @@ description: Commit procedure and message rules. Use for every commit. - When the change belongs to an issue, end the message with one `Issue:` trailer. Use the shortest unique git-bug issue ID in the `Issue:` trailer, with a minimum of seven characters. Check uniqueness against - all issues, not only open issues. Add it with `git commit --trailer`. - Never use a closing keyword such as "Fixes" or "Closes". + all issues, not only open issues. Assemble it with + `git interpret-trailers --trailer` before the message check. Never use + a closing keyword such as "Fixes" or "Closes". - Link the bug report when a change fixes a reported bug. - Do not add an AI tool as co-author. Do not add attribution trailers.