Now liveThe Skillselion MCP - thousands of ranked skills, loaded into your agent mid-task. No install.Get it →
travisjneuman avatar

Generic Code Reviewer

  • 124 installs
  • 86 repo stars
  • Updated July 17, 2026
  • travisjneuman/.claude

Use generic-code-reviewer for development tasks

About

generic-code-reviewer: A skill for development. This provides functionality for development workflows.

  • generic-code-reviewer

Generic Code Reviewer by the numbers

  • 124 all-time installs (skills.sh)
  • +3 installs in the week ending Aug 5, 2026 (Skillselion tracking)
  • Ranked #2,799 of 4,347 Backend & APIs skills by installs in the Skillselion catalog
  • Data as of Aug 5, 2026 (Skillselion catalog sync)
npx skills add https://github.com/travisjneuman/.claude --skill generic-code-reviewer

Add your badge

Show developers this skill is listed on Skillselion. Paste this into your README.

Listed on Skillselion
Installs124
repo stars86
Last updatedJuly 17, 2026
Repositorytravisjneuman/.claude

What it does

Use generic-code-reviewer for development tasks

Files

SKILL.mdMarkdownGitHub ↗

Generic Code Reviewer

Review code against production quality standards. Adapts to any project's tech stack.

Full Standards: See Code Review Standards

CLAUDE.md Compliance

Always check the project's CLAUDE.md for specific rules.

Before ANY Commit:

  • Task file exists in tasks/[feature-name].md
  • All tests passing
  • Type checking passing (if TypeScript)
  • No console errors/warnings
  • Bundle size within limits

Tech Stack Detection

DetectionStackKey Checks
React in package.jsonReact/TSComponents, hooks, state
Next.js in package.jsonNext.jsSSR, API routes
NestJS in package.jsonNestJSGuards, DTOs, services
.html files, no buildVanillaSemantic HTML, minimal JS
.py filesPythonType hints, validation

P0 Issues (Block Merge)

Security - Frontend

  • Sanitize input (textContent, not innerHTML)
  • unknown type for external data
  • No exposed API keys
  • HTTPS for external requests

Security - Backend

  • Input validation on all endpoints
  • Auth on protected routes
  • Parameterized queries (no raw SQL)
  • Secrets in environment variables

Correctness

  • Logic errors that break functionality
  • Type errors in strict mode
  • Unhandled promise rejections

P1 Issues (Should Fix)

Performance

Project TypeTarget
Static site< 50KB (excluding media)
SPA/React< 100KB gzipped initial
Full-stack< 200KB gzipped initial

Animation:

  • GPU-accelerated only (transform, opacity)
  • 60fps target
  • Use requestAnimationFrame

Accessibility (WCAG AA)

  • Focus indicators on interactive elements
  • Keyboard navigation (Tab, Enter, Escape)
  • Color contrast >= 4.5:1
  • ARIA labels on icon-only buttons
  • Alt text for meaningful images
  • Semantic HTML structure

P2 Issues (Nice to Have)

Code Quality

  • DRY, Single Responsibility
  • No magic numbers/strings
  • Self-documenting code
  • Follow existing patterns
  • No commented-out/dead code

Review Output Format

Only report issues found (don't list empty categories):

Blocking Issues (P0):

  • [Only if found - security, correctness issues]

Should Fix (P1):

  • [Only if found - performance, accessibility issues]

Consider (P2):

  • [Only if found - code quality polish]

If no issues: "Code review passed. Ready to merge."

Judgment Calls

When user asks "Is this OK?", consider:

ContextStricterMore Lenient
EnvironmentProductionPrototype/POC
PathHot path, frequently executedOne-time setup, admin only
VisibilityPublic API, external interfaceInternal helper, private method
TimelineFeature completeActive iteration

Adjust severity accordingly. P0 security issues are never lenient.

Quick Checklist

Pre-Commit:

  • [ ] Tests pass
  • [ ] Type/lint pass
  • [ ] Build succeeds
  • [ ] No console errors

Before Merge:

  • [ ] All P0 issues resolved
  • [ ] P1 issues addressed or tracked
  • [ ] P2 issues noted for future

See Also

  • Code Review Standards - Full requirements
  • Design Patterns - UI consistency
  • Project CLAUDE.md - Workflow rules

Related skills

Backend & APIsbackendintegrations

This week in AI coding

Five minutes, every Monday - the tools, releases and tactics for developers.

unsubscribe anytime.