agentsclimarketplace
Sign in

Code review checklist

Skill subhansh-dev/agent-maxxing/engineering/code-review-checklist

Ship-ready code review checklist covering correctness, readability, security, performance, and frontend/backend specifics.From its SKILL.md

Install
npx -y skills add subhansh-dev/agent-maxxing --skill code-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.

SKILL.md

1.7 KB, 361 tokens by cl100k_base, as published. Nobody here has run it

Code Review Checklist

Before shipping any code, run through this.

Does It Work?

  • Runs without errors
  • Handles the happy path
  • Handles edge cases (null, empty, 0, negative, very large)
  • Handles errors gracefully (try/catch, validation)
  • No console.logs or debug code left in

Is It Readable?

  • Variable names are descriptive (not x, temp, data2)
  • Functions do one thing
  • No magic numbers (use constants)
  • Comments explain "why", not "what"
  • Consistent style with existing codebase

Is It Safe?

  • No SQL injection / XSS / path traversal
  • No hardcoded credentials
  • No secrets in logs or error messages
  • Input validation on user-facing endpoints
  • Rate limiting on public APIs

Is It Performant?

  • No N+1 queries
  • No unnecessary re-renders (React)
  • No blocking operations on main thread
  • Lazy loading for heavy resources
  • Pagination for large datasets

Is It Maintainable?

  • No code duplication (DRY)
  • Clear separation of concerns
  • Error messages are helpful
  • Configuration is externalized
  • Dependencies are pinned

Frontend-Specific

  • Accessible (ARIA labels, keyboard nav, contrast)
  • Responsive (mobile, tablet, desktop)
  • Loading states for async operations
  • Error states with recovery paths
  • Empty states that guide the user
  • No layout shift (CLS)
  • 60fps animations (transform + opacity only)

Backend-Specific

  • Database migrations are reversible
  • API responses are consistent
  • Rate limiting is in place
  • Logging is sufficient for debugging
  • Health check endpoint exists
  • Graceful shutdown handling

What ships with it

Read from the repository

Just SKILL.md. No reference files, no scripts.

Keep looking

Skills are one crate of 325,949. 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.