mirror of https://github.com/penpot/penpot.git synced 2026-10-29 05:50:35 +00:00

Andrey Antukh ea45507ac8 🔧 Raise commit message limits to 80 chars

Relax the subject limit from 70 to 80 characters and the body
wrap limit from 76 to 80. Merge/Revert/Reapply commits skip the
subject length check since git prepends those prefixes to an
existing subject. Update the checker, its tests, and all related
docs and memories accordingly.

AI-assisted-by: muse-spark-1.3-contributor
Signed-off-by: Andrey Antukh <niwi@niwi.nz>

2026-10-14 18:35:15 +02:00

2.9 KiB

Raw Permalink Blame History

name, description, slash

name	description	slash
create-commit	Stage, review, and commit files following Penpot commit conventions.	true

Skill: create-commit

Produce a git commit that follows Penpot's commit message conventions. This skill owns the commit format, staging review, and safety checks — it does not implement features or push.

When to Use

After code changes are complete and files need to be committed
When delegated by a workflow step (e.g. implement-plan) to handle the commit

Required Reading

Before drafting any commit, read mem:workflow/creating-commits end-to-end. It is the authoritative source for the commit message format, the emoji menu, subject/body limits, and the AI-assisted-by trailer. Follow it exactly.

Iron Rules (non-negotiable)

Wrap every body line at 80 characters or fewer. Count characters, do not eyeball. Exceptions: Signed-off-by: / AI-assisted-by: trailers and lines carrying a URL. This is the rule agents skip most often.
Subject ≤80 chars, imperative, capitalized, no trailing period. Merge/Revert/Reapply commits skip the length check.
Blank line between subject and body.
Run ./scripts/check-commit and require exit code 0. It mechanically checks rules 1–3. A non-zero exit is a hard blocker: fix the message and re-commit. Never report the commit as done with a failing checker.

Workflow

Stage the files specified by the calling context. Do not ask for confirmation.
Run git diff --staged to review the content. If you see secrets (API keys, tokens, passwords, private keys, .env values), debug prints, or anything that does not match the stated intent, STOP and tell the user before committing.
Draft the message following the format in the memory doc, wrapping the body at 80 characters per line, and run:
```
git commit -m "<subject>" -m "<body>"
```
(or git commit -F - if the body has unusual characters).
Verify the message with the checker:
```
./scripts/check-commit
```
If it fails, amend the message (git commit --amend) until it passes. Do not finish with a failing checker.
The AI-assisted-by trailer value is provided by the calling context — use it verbatim.

Constraints

Do not push. Pushing is a separate workflow handled by the user.
Do not run git reset, git checkout, git restore, git clean, or rm.
Do not pass --author. Author identity comes from the local git config.
Do not amend a commit you did not create in this session, unless explicitly asked.
Do not bypass pre-commit hooks (--no-verify) unless explicitly asked.
Never add a Co-authored-by: trailer. It is forbidden and fails ./scripts/check-commit.
Do not add untracked files that were not created in this session.
Do not skip the scripts/check-commit verification step (Iron Rule 4).

2.9 KiB Raw Permalink Blame History Unescape Escape