Menu
Final pre-commit check: run type-check (npx tsc --noEmit), lint (npm run lint), and tests (npm test)
Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.
Based on SOC occupation classification
Implement a numbered testing strategy phase from docs/testing_strategy_phases/phase_$ARGUMENTS_*.md step by step
Implement a numbered phase from docs/phases/phase-$ARGUMENTS.md step by step
Review changed code for correctness, security, and project conventions. Invoke this skill automatically whenever a complete unit of work is done — a full implementation phase, a complete feature (repository + service + route + component all connected), or just before creating a PR. Do NOT invoke after editing a single file or mid-way through a feature. If the user says "done", "finished", "phase complete", "ready to commit", or similar, trigger this review immediately.
Run prisma migrate dev, regenerate client to the correct output path, and verify import paths
Scaffold a Prisma-backed repository for a given Prisma model
Scaffold a service class for a given domain following project conventions
| name | verify |
| description | Final pre-commit check: run type-check (npx tsc --noEmit), lint (npm run lint), and tests (npm test) |
Run the full pre-commit verification suite. Do not skip steps even if earlier ones pass.
npx tsc --noEmit
Fix all type errors before proceeding. Do not use @ts-ignore or as any without inline documentation.
Common causes in this project:
@prisma/client → @/app/generated/prisma/client)prisma generate not run after schema change (run /db-migrate)cookies() callawait on Server Component paramsnpm run lint
Fix all errors. Warnings are acceptable.
npm test
Runs all three Vitest projects (unit, integration, components) with pool: 'forks' and maxWorkers: 1. Do not change these settings.
To debug a single failing file:
npx vitest run tests/unit/services/invoice.service.spec.ts
If a test fails: determine whether the test is wrong (implementation changed) or the code is wrong (regression). Fix the root cause — do not delete failing tests.
grep -rn "from '@prisma/client'" src/ tests/
grep -rn 'from "@prisma/client"' src/ tests/
Fix any hits before committing.
Verify complete:
✓ TypeScript: no errors
✓ ESLint: no errors
✓ Vitest: X passed (Y unit, Z integration, W component)
✓ Import paths: clean
Ready to commit.
List any fixes made during this run.
Note on E2E: Playwright tests are NOT included here — they require a running dev server. Run separately:
npx playwright test
E2E tests are a CI gate on PRs, not a local pre-commit requirement.