How to use
- Copy the installation command and review the source package before installing it in your own agent environment.
- Check the source document for required tools, credentials and supported hosts.
- Describe your input and desired deliverable; review the output against the source material.
npx skills add https://github.com/victorgarciaesgi/regle --skill code-reviewBefore you start
- Installation does not include a model subscription, API allowance or permission to access your files.
- Repository stars and installation counts describe activity, not an independent quality or security test.
Original skill document
Rendered from the source repository. Headings, examples, code, tables, links, and referenced images are preserved.
Code review — pre-commit validation
Apply on every pull request review. These checks are blocking. Do not approve a PR until all pass or the author has addressed failures.
Review checklist (run every time)
| # | Gate | Command | On failure | |
|---|---|---|---|---|
| 1 | Commit messages | `printf '%s' "$MSG" \ | pnpm exec commitlint` per commit | Blocking comment + suggested fix |
| 2 | Workspace | test -f node_modules/.modules.yaml | Request pnpm install | |
| 3 | Lint | pnpm run lint | Blocking; suggest pnpm run lint:fix | |
| 4 | Format | pnpm run fmt:check | Blocking; suggest pnpm run fmt |
Run all commands from the repo root after pnpm install.
Commit messages (commitlint)
Every commit must follow Conventional Commits per commitlint.config.ts (@commitlint/config-conventional).
type(optional-scope): imperative subject
Optional body explaining why.- Types:
feat,fix,docs,style,refactor,perf,test,chore,ci,build - Subject: lowercase, imperative mood, no trailing period
- Blank line between title and body when a body is present
Auto-fix loop
bash
pnpm run lint:fix
pnpm run fmt
pnpm run lint
pnpm run fmt:checkCommon commitlint failures
| Error | Fix |
|---|---|
type-empty / subject-empty | Use type: subject format |
type-enum | Use an allowed conventional type |
subject-case | Keep subject lowercase |
subject-full-stop | Remove trailing . from subject |
