How do I install the Pre Commit Review skill?

Run npx skillmds@latest add qdhenry/pre-commit-review in your terminal (requires Node.js), paste this page's agent-chat prompt into Claude, Cursor, or any MCP-connected agent, or download the SKILL.md file and copy it into your agent's skills directory.

Is Pre Commit Review safe to use?

This skill has not completed SkillMD's automated safety review yet. SkillMD never runs a skill's scripts for you; review the SKILL.md before installing.

Which AI agents work with Pre Commit Review?

This skill is tagged as working with Claude Code, Claude.ai, OpenAI Codex. SKILL.md is an open format, so most agents that read a skills directory can load it too.

Is Pre Commit Review free to use?

Yes. Installing skills from SkillMD is free, and the skill stays under its author's original license.

Who published Pre Commit Review?

qdhenry (@qdhenry) published this skill. Their other Agent Skills are listed on their SkillMD profile.

Pre Commit Review

Name: Pre Commit Review
Author: qdhenry

Comprehensive review of all staged changes before committing. Routes files to appropriate governance domains and produces a go/no-go recommendation. Use before committing significant changes.

qdhenry 87ac19e 2 files · 2.1 KB Updated Sep 17, 2026

Coding & Dev Tools

Install

Press C to copy

$ npx skillmds@latest add qdhenry/pre-commit-review

Installs into ~/.claude/skills (or .claude/skills inside a project). Restart Claude Code to pick it up.

Download .zip

＋ Add to collection

README badge

[![SkillMD](https://skillmd.com/badge/qdhenry/pre-commit-review.svg)](https://skillmd.com/skills/qdhenry/pre-commit-review)

Trust & safety

Audits pending

Not yet audited · commit 87ac19e

Works with: Claude Code Claude.ai OpenAI Codex +

File contents

Security checks (convex files):

assertOrgAccess or getAuthUser present in query/mutation handlers
.withIndex() used (no bare .filter())
No direct ctx.auth.getUserIdentity()

Convex checks (convex files):

camelCase naming for tables/functions
v.union(v.literal()) for enum fields
Index coverage for new query patterns

Design system checks (src files):

No arbitrary colors (text-[#...], bg-[#...]), no purple/violet
No dark: prefix in className
CSS token usage for spacing/radius

Architecture checks (src files):

PascalCase component files
kebab-case route directories
Awaited params in page.tsx/layout.tsx

GO — No blocking violations found. List any warnings.

NO-GO — Blocking violations found. List each violation with:

File and line number
Rule violated (e.g., SEC-001, DS-002)
Required fix

---name: pre-commit-reviewdescription: Comprehensive review of all staged changes before committing. Routes files to appropriate governance domains and produces a go/no-go recommendation. Use before committing significant changes.---<objective>Review all staged changes before commit. Route each file to the appropriate governance domain for review. Produce a go/no-go recommendation with specific findings and required fixes.</objective><process><step_1>**Get staged files:**Run `git diff --cached --name-only` to list all staged files.If no files are staged, run `git diff --name-only` to show unstaged changes and inform the user.</step_1><step_2>**Classify files by domain:**- `convex/**/*.ts` (not `_generated/`) → Security + Convex patterns- `src/**/*.tsx` → Design system + Architecture- `src/**/*.css` → Design system- `src/app/**/page.tsx` or `layout.tsx` → Architecture (Next.js 15 patterns)</step_2><step_3>**Run domain-specific checks using Grep and Read:****Security checks** (convex files):- `assertOrgAccess` or `getAuthUser` present in query/mutation handlers- `.withIndex()` used (no bare `.filter()`)- No direct `ctx.auth.getUserIdentity()`**Convex checks** (convex files):- camelCase naming for tables/functions- `v.union(v.literal())` for enum fields- Index coverage for new query patterns**Design system checks** (src files):- No arbitrary colors (`text-[#...]`, `bg-[#...]`), no purple/violet- No `dark:` prefix in className- CSS token usage for spacing/radius**Architecture checks** (src files):- PascalCase component files- kebab-case route directories- Awaited params in page.tsx/layout.tsx</step_3><step_4>**Produce recommendation:****GO** — No blocking violations found. List any warnings.**NO-GO** — Blocking violations found. List each violation with:- File and line number- Rule violated (e.g., SEC-001, DS-002)- Required fix</step_4></process><success_criteria>- All staged files classified and checked- Clear GO or NO-GO recommendation- Every blocking violation has a specific fix- Warnings listed separately from blockers</success_criteria>

qdhenry/foundry-oss/tree/main/.claude/governance/skills/pre-commit-review commit 87ac19ec87

Frequently asked questions

npx skillmds@latest add qdhenry/pre-commit-review