Review checklist
Code review standards and checklist for the code-reviewer agentFrom its SKILL.md
npx -y skills add zachjxyz/jvn --skill review-checklistAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 2 stars2 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.
SKILL.md
1.6 KB, 352 tokens by cl100k_base, as published. Nobody here has run it
Code Review Checklist
Use this checklist when reviewing implementation changes.
Correctness
- Logic matches specification requirements
- Edge cases handled (null, empty, boundary values)
- Error paths return appropriate responses
- No off-by-one errors in loops/pagination
Type Safety
- No
anytypes - No
@ts-ignoreor@ts-expect-error - Drizzle query types properly inferred (not manually typed)
- API response types match contract
Security
- No secrets or credentials in source code
- Auth checks on all protected routes and server actions
- User input validated at system boundaries
- SQL injection prevented (Drizzle parameterized queries)
- XSS prevented (no
dangerouslySetInnerHTMLwithout sanitization)
Performance
- No N+1 query patterns
- Indexes on columns used in WHERE, JOIN, ORDER BY
- Independent data fetches use
Promise.all() - Heavy components use
next/dynamicfor code splitting - Suspense boundaries wrap async server components
Testing
- Tests written BEFORE implementation (TDD)
- Happy path covered
- Error cases covered
- Edge cases covered
- Integration tests for API routes
Maintainability
- Files under ~200 lines (split if larger)
- Descriptive variable and function names
- Early returns for guard clauses
- No dead code (unused imports, variables, functions)
- No commented-out code
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.