This is a structured framework for doing code reviews that don't suck. It gives you checklists for correctness, maintainability, performance, and security, plus thresholds for when code is too complex (cyclomatic complexity over 20 means refactor required). The best part is the hard data on review effectiveness: PRs under 400 lines catch the most defects, while anything over 800 lines loses 90% effectiveness. It includes a comment severity system (blocking, major, minor, question) so you're not wasting time debating semicolons while missing actual bugs. Works with any language and integrates with github-agile for PR workflows. Honest take: the metrics and thresholds alone make this worth using as a reference, even if you don't follow it religiously.

skills.sh installs
504
GitHub stars
182
last updated
Aug 21, 2026

Install

$ npx -y skills add jwynia/agent-skills --skill code-review --agent claude-code

Installs into .claude/skills of the current project.

Files

Code Review Diagnostic

Systematic code review catches 60-90% of defects before production, reduces maintenance costs by 40%, and serves as effective knowledge transfer. This skill provides structured review guidance for both human reviewers and AI agents.

When to Use This Skill

Use this skill when:

  • Reviewing code before merge
  • Assessing code quality
  • Preparing code for PR submission
  • Self-reviewing before requesting review

Do NOT use this skill when:

  • Writing new code (use implementation skills)
  • Designing architecture (use system-design)
  • Working on requirements (use requirements-analysis)

Core Principle

Review effectiveness degrades sharply with PR size. Under 400 lines: highest defect detection. 400-800 lines: 50% less effective. 800+ lines: 90% less effective.

Quick Reference: Review Effectiveness

FactorOptimalDegraded
PR size< 400 lines> 800 lines
Review time< 60 minutes> 90 minutes
Review speed200-400 LOC/hour> 500 LOC/hour
Reviewers24+ (diminishing returns)

Quality Pyramid

Sponsors

Details

Repository
github.com/jwynia/agent-skills
Categories
Security, Git & Pull Requests, Code Review & Quality, AI & Agent Building, Debugging
First seen
Jun 3, 2026

More from jwynia/agent-skills

All 118

Recommended

More Security
1aj-geddes avatarapi-reference-documentationaj-geddes/useful-ai-promptsCreate comprehensive API reference documentation with OpenAPI/Swagger specs, REST endpoints, authentication, examples, and SDKs. Use when documenting REST APIs,499skills.sh installs
2longbridge avatarlongbridge-basicinfolongbridge/skillsStatic basic information for all Longbridge-tradable securities — stocks, ETFs, options, warrants: company name, listing date, exchange, industry classification497skills.sh installs
3inertia-rails avatarinertia-rails-controllersinertia-rails/skillsALWAYS `render inertia: { key: data }` to pass data as props — instance variables are NOT auto-passed (only alba-inertia does that). Rails controller patterns f496skills.sh installs
4sivaprasadreddy avatarspring-boot-skillsivaprasadreddy/sivalabs-agent-skillsBuild Spring Boot 4.x applications following the best practices. Use this skill: * When developing Spring Boot applications using Spring MVC, Spring Data JPA, S493skills.sh installs
5atlassian avatarsearch-company-knowledgeatlassian/atlassian-mcp-serverSearch across company knowledge bases (Confluence, Jira, internal docs) to find and explain internal concepts, processes, and technical details. When an agent n488skills.sh installs
6aj-geddes avatarzero-trust-architectureaj-geddes/useful-ai-promptsImplement Zero Trust security model with identity verification, microsegmentation, least privilege access, and continuous monitoring. Use when building secure c487skills.sh installs