agentsclimarketplace

Pre commit check

Skill SID-SURANGE/cursor-team-ops/skills/core/pre-commit-check

Enforcement & release-hygiene layer for Cursor agents β€” blocking git/DB/license guardrails, commit hygiene, and docs-ops.

Install
npx -y skills add SID-SURANGE/cursor-team-ops --skill pre-commit-check

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

One thing to look at

  • 0 stars0 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

Check staged changes before creating a git commit. Reviews the diff for secrets, unrelated changes, debug code, oversized PRs, and matches the repo's commit message style. Use when the user says "commit", "create a commit", "git commit", "commit this", "commit my changes", "ready to commit", "safe to commit", "check my changes", "stage and commit", "push my work", or "I'm done with this task".

SKILL.md

2.9 KB, 613 tokens by cl100k_base, as published. Nobody here has run it

πŸ” Pre-Commit Check

Run these in parallel

  1. git status β€” list staged and unstaged files.
  2. git diff --cached β€” full staged diff.
  3. git log --oneline -5 β€” recent commit messages to infer style.
  4. git diff --cached --shortstat β€” line and file counts for size check.

Check the staged diff for

IssueAction
.env, *.key, credentials.*, *secret* stagedStop. Warn user. Do not commit.
Secrets or tokens in changed linesStop. Warn user.
Changes to files unrelated to the stated taskFlag β€” do not auto-unstage
Debug code (console.log, print(, debugger, breakpoint())Flag
Nothing stagedDo not create an empty commit. Tell the user.
>400 lines changed or >10 files touchedFlag as oversized β€” suggest splitting by concern
Multiple unrelated concerns in one diffFlag β€” name each concern and suggest separate commits

Commit message

Format

<type>: <subject>          ← max 72 chars, imperative present tense

[optional body]            ← wrap at 72 chars, explains WHY not WHAT

[optional footer]          ← e.g. Closes #123, Co-authored-by: ...

Type prefixes (use the project's convention if one exists)

TypeWhen to use
featNew user-visible feature
fixBug fix
refactorCode restructure, no behaviour change
testAdding or updating tests
docsDocumentation only
choreTooling, config, dependencies
perfPerformance improvement

Subject line rules

  • Imperative present tense: "add user auth" not "added" or "adding"
  • No trailing period
  • No emoji unless the project convention uses them
  • ≀72 characters β€” hard limit

Body (include when the change is non-obvious)

  • Explain why the change was made, not what the diff shows
  • Mention alternatives considered if the choice is non-obvious
  • Reference issue numbers: Closes #42, Refs #17

What to avoid

  • "WIP", "fix stuff", "changes" β€” not searchable or reviewable
  • Giant commits mixing unrelated changes β€” split them
  • Committing without staged content

Always pass the message via HEREDOC (per git-safety rule)

git commit -m "$(cat <<'EOF'
feat: add user authentication

Replaces the previous session-token approach with JWT.
Closes #42
EOF
)"

After commit

Run git status to confirm success before reporting done.

What ships with it

Read from the repository

Just SKILL.md. No reference files, no scripts.

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.