agentsclimarketplace

Commit

Skill athal7/dotfiles/skills/commit

Install
npx -y skills add athal7/dotfiles --skill commit

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

2 things to look at

  • no licenseNo license file was found in the repository. Code published without one is not open source by default, so using it at work is a question for whoever answers licensing questions where you are.
  • 6 stars6 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

Semantic commit format and pre-commit checks

The file declares its own license as MIT. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.

SKILL.md

3.0 KB, as published. Nobody here has run it

STOP — verification gate

Before reading further, confirm in this session you have:

  1. Verified the staged diff.
  2. Applied any findings.
  3. Re-verified until clean.

If any of those is missing, stop now and verify first. A 30-day audit found 60% of commit-time sessions skipped this step despite the precondition being declared. The pattern is the agent skipping past preamble — this gate exists to interrupt that. Do not draft the commit message, do not run git add, do not continue past this section until verification has happened.

Before every commit

Run automatically without asking. Set the bash tool's workdir to the repo root rather than passing git -C <path>. Permission patterns like git push * match parsed argv starting with git push; git -C <path> push injects flags between git and the subcommand and bypasses those patterns silently.

  1. Stage: git add -A.
  2. Unstage in-flight OpenSpec change files: git reset -q -- 'openspec/changes/*' ':(exclude)openspec/changes/archive'. In-flight changes under openspec/changes/<name>/ are intentionally un-ignored so they surface in review, but they are review artifacts, not code — they belong in the durable store and are moved there at archive time. This unstages them while leaving everything else (including any archived specs) staged.
  3. Skip globally-ignored files: git check-ignore <files>. Do NOT stage files in ~/.config/git/ignore (e.g. .talismanrc).
  4. Run the full test suite — unit, integration, e2e, system. Do not commit with failures.
  5. Draft the commit message in the format below, with a Co-Authored-By trailer naming the model used (e.g. anthropic/claude-sonnet-4-6):
    Co-Authored-By: anthropic/claude-sonnet-4-6 <[email protected]>
    

Format

type(scope): description
TypeWhen
featNew feature or capability
fixBug fix
refactorCode change that neither fixes nor adds
docsDocumentation only
testAdding or updating tests
choreBuild, CI, dependencies, tooling

Scope — component/area (fix(auth): ...). Semantic, not ticket numbers.

Description — lowercase, no trailing period, imperative ("add" not "added"), focus on why, under 72 characters.

feat(auth): add password reset flow
fix(auth): prevent token refresh race condition
refactor(api): extract validation middleware
chore(deps): bump lodash to 4.17.21

Squashing

Before first push: squash related commits — multiple feature attempts → single feat, fix+test for same issue → single fix. Keep unrelated changes separate.

After review feedback: add new commits, don't rewrite history. Each round gets its own commit(s) with an appropriate type (e.g. fix(auth): address review — validate token expiry). Never force-push pushed commits — reviewers lose context. Squashing at merge happens via the platform's squash-and-merge.

Keep looking

Skills are one crate of 328,083. Ordering is by how many stacks a row turns up in, so the top of any crate is what has actually been picked rather than what has the most stars.