agentsclimarketplace

Review checklist

Skill zachjxyz/jvn/template/.claude/skills/review-checklist

Spec-driven development with Claude Code. Named after John von Neumann — the man who wrote the spec that defined computing. Three commands: /spec, /design, /build.

Install
npx -y skills add zachjxyz/jvn --skill review-checklist

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

  • 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.

What its author says it does

Copied from the file, not written here

Code review standards and checklist for the code-reviewer agent

SKILL.md

1.6 KB, 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 any types
  • No @ts-ignore or @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 dangerouslySetInnerHTML without 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/dynamic for 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

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.