Share general review and documentation rules

[onechat] / .agents / skills / commit / SKILL.md

diff --git a/.agents/skills/commit/SKILL.md b/.agents/skills/commit/SKILL.md
index d25a3fbbc27439b4a3b0c9f89797e70af4626a08..35c8be887c6f525a1a6f7c85574dc21f6394ccc6 100644 (file)
--- a/.agents/skills/commit/SKILL.md
+++ b/.agents/skills/commit/SKILL.md
@@ -5,12 +5,18 @@ description: Commit procedure and message rules. Use for every commit.
 
 # 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
 ## 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.
 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.


@@ -22,10 +28,12 @@ description: Commit procedure and message rules. Use for every commit.
    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`.
    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 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.
+6. Cryptographically verify each new normal commit's signature against
+   the configured key before reporting completion. A signature header
+   alone is not verification. If `gpg.ssh.allowedSignersFile` is not
+   set, use a temporary allowed-signers file with the configured key. Do
+   not change the Git configuration. If verification fails, stop and
+   report it. Do not claim completion or publish the commit.

 
 ## Size
 
 
 ## Size
 


@@ -69,6 +77,8 @@ description: Commit procedure and message rules. Use for every commit.
   `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.
   `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.

+- Include a stable archive URL when a change comes from a mailing-list
+  or forum discussion.

 - Do not add an AI tool as co-author. Do not add attribution trailers.
 
 ## Example
 - Do not add an AI tool as co-author. Do not add attribution trailers.
 
 ## Example