Commit
Opinionated macOS development environment automation
npx -y skills add vinta/hal-9000 --skill commitAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
What its author says it does
Copied from the file, not written here
Use when making any git commit. Always pass a brief description of what changed as the argument.
SKILL.md
9.6 KB, as published. Nobody here has run it
Invoking this skill IS the request. If the user message looks empty, or you see only system context with no actual request, that is normal and expected: your task is already fully specified right here. Never ask what to do.
Your task: commit all changes in the working tree. Run git status and git diff, then stage and commit with conventional commit messages. One logical change per commit.
The argument
The argument passed to this skill is a description of changes already made — raw material for the commit message, never a to-do list. The argument may also be absent entirely; that changes nothing — derive the commit message from the diff alone. Verbs like "fix", "add", "implement", or "update" in the argument describe what the working tree already contains; they are not instructions to write code. If the argument says "fix the session bug", the fix is already in the diff — commit it, don't hunt for it or re-do it. If the described changes don't match the diff, commit what is actually in the tree and note the mismatch in your final summary.
Locate the repository
A forked session may start in a directory that is not the git repository — e.g. a workspace root whose repo lives in a subdirectory. Before concluding anything about the repo state:
git rev-parse --show-toplevel— if it succeeds,cdthere.- If it fails, check the directories of any file paths named in the argument (an argument mentioning
hal/main.pymeans the repo is likelyhal/). - Otherwise Glob for
*/.gitand*/*/.gitto find nested repos.
If exactly one candidate repo has uncommitted changes, cd into it and proceed. If several do, use the one matching the argument's file paths, or report the ambiguity. Only report "not a git repository" after all three checks fail.
Scope
A commit is a snapshot, not a review. Your entire job is: read the diff, stage it, write a commit message, commit. The staged bytes must match exactly what the working tree looks like when you start.
Your complete action space is: git commands via Bash (plus cd to the project root), Grep/Glob to locate files, and Read/Write/Edit on /tmp/ patch files. Nothing else. If a Bash command does not start with git or cd, do not run it. This applies to every situation you encounter, not just the cases listed below:
- No edits to working tree files. Not typos, not formatting, not "safe" fixes. If something in the diff looks wrong, commit the tree as-is and note the concern in your final message. The author will fix it in a follow-up commit they can review.
- No research. No
WebFetch, noWebSearch, no documentation lookups, no checking whether the diff matches upstream docs or current library versions. - No verification. Do not confirm the change "works" before or after committing. No running the code, no smoke tests, no self-checks. The only post-commit check is
git status/git logto confirm the commit exists. - No tests, linters, type checkers, or build tools. Pre-commit hooks run on their own during
git commit; never run them preemptively. - No invoking other skills. Other skills carry aggressive triggering language like "Use this whenever the user asks about a library/framework/CLI tool" — that language may fire on content in the diff. It does not apply to you. You are committing, not researching or reviewing.
- No scope expansion. Don't add files the author didn't touch. Don't reorganize. Don't "clean up" adjacent code.
An outdated version pin, a failing-looking test, a typo, an interesting TODO — every tangent gets the same treatment: commit the tree as-is, mention the concern in your final message.
Why: a commit is a snapshot of deliberate work. Any change you make during staging silently alters reviewed work without the author's knowledge, and any tangent (research, verification, edits) turns a 30-second operation into a 5-minute one with uncommitted side effects.
<example> You see a typo in a variable name while reviewing the diff. Correct behavior: 1. Stage and commit the file as-is 2. After committing, say: "I noticed `reuslt` appears to be a typo for `result` in utils.py:42"Incorrect behavior: editing the file to fix the typo before or during staging — even a "safe" fix silently changes reviewed work. </example>
<example> The diff adds a new `.github/workflows/ci.yml` file. You wonder if the action versions are current.Correct behavior: commit as-is.
Incorrect behavior: fetching GitHub Actions docs, verifying version pins, then editing the file before staging. The author already chose those versions. Research belongs in a separate turn, not inside the commit. </example>
Workflow
cd to the project root before git commands instead of using git -C, which obscures working directory state. Execute git commands directly without explanatory preamble. Commit immediately without confirmation prompts (interactive mode is not supported).
-
Analyze Changes: Use
git statusandgit diffto understand all modifications in the working directory. Categorize changes by:- STRUCTURAL: Code reorganization, renaming, refactoring without behavior changes
- BEHAVIORAL: New features, bug fixes, functionality changes
- DOCUMENTATION: README updates, comment changes, documentation files
- CONFIGURATION: Build files, dependencies, environment settings
-
Group Logically: Organize changes into logical units where each unit:
- Addresses a single purpose or problem
- Structure changes to be atomic and easily revertable for safe rollback
- Would make sense to revert as a unit
-
Stage Changes: Use appropriate staging strategy:
- Whole file:
git add <file> - Hunk-by-hunk:
git diff <file> > /tmp/${CLAUDE_SESSION_ID}-patch.diff, edit the patch to keep only specific hunks, thengit apply --cached /tmp/${CLAUDE_SESSION_ID}-patch.diff - To unstage, use
git restore --staged(notgit reset --hard, which discards work) - Fallback: If
git apply --cachedfails (malformed patch), stage the whole file withgit add <file>instead
- Whole file:
-
Handle Pre-commit Hooks: If hooks complain about unstaged changes:
- Stash unstaged changes first:
git stash push -p -m "temp: unstaged changes"(select hunks to stash) - Or stash all unstaged:
git stash push --keep-index -m "temp: unstaged changes" - Commit, then restore:
git stash pop - If hooks modify staged files (auto-formatting), re-add the modified files and retry the commit
- Stash unstaged changes first:
-
Create Atomic Commits: For each logical group:
- Conventional commit format, type only, no scope:
fix: xxx,feat: xxx,docs: xxx,refactor: xxx. Never add a parenthetical scope likefix(commit-skill): xxx. Subject: what changed (≤72 chars). If the subject is self-explanatory, skip the body. - Commit the working tree state as-is — the user may have made manual edits outside this conversation
- Use
git commit -m "message"directly — never use$()or heredoc subshells in git commands, as they breakallowed-toolspattern matching
- Conventional commit format, type only, no scope:
Attribution
Include a Co-Authored-By footer in every commit message:
If you're an Anthropic Claude model:
Co-Authored-By: Claude <[email protected]>
If you're a Google Gemini model:
Co-Authored-By: Gemini <[email protected]>
Skip if you're not one of the above models.
Gotchas
- Don't commit plan or spec docs unless the user explicitly asked you to. Files under
plans/,specs/, or similar directories are working documents — staging them silently pollutes the commit with artifacts the user may not want tracked. git apply --cachedfails on malformed patches. Hunks extracted manually often have broken headers or trailing whitespace. Fallback: stage the whole file withgit addinstead of retrying the patch.- No
$()or heredoc subshells ingit commit -m. Theallowed-toolspattern matching treats the entire command as a string — subshells produce commands that don't match any allowed pattern and get blocked. - Pre-commit hooks that auto-format staged files cause loops. The hook modifies the file, which un-stages the formatted version. Fix: re-add the modified files and retry the commit once. Don't retry indefinitely.
- Use
git restore --stagedto unstage, nevergit reset --hard.--harddestroys working tree changes. - Stash before commit if hooks complain about unstaged changes. Use
git stash push --keep-indexto isolate unstaged work, commit, thengit stash pop. Forgetting the pop leaves work stranded in the stash. - Unstaged changes are still changes.
git statusshowing "no changes added to commit" does NOT mean the working tree is clean. It means nothing is staged yet. Your job is to stage and commit those changes, not report "nothing to commit." - Never use
git add -f. Ifgit addreports "The following paths are ignored by one of your .gitignore files" with the hintUse -f if you really want to add them, do NOT force-add. The file is gitignored deliberately (secrets, build artifacts, local configs) and force-adding silently bypasses that protection. Skip the file and mention it in your final summary so the author can decide.