Code Review Expert

This is a structured code review workflow that runs against your current git diff and flags issues by severity (P0 critical down to P3 style nits). It checks for SOLID violations, security risks like injection or auth gaps, race conditions, and removal candidates, then asks how you want to proceed before touching anything. The review-first approach is solid since you get findings organized by priority with inline comments before committing to fixes. Most useful when you want a senior engineer perspective on a pull request without the back and forth, though the depth of analysis depends on how well it can pattern match against the bundled checklists for security, code quality, and architecture smells.

install copies · 90 days. Times the install command was copied on this site in the last 90 days, bots filtered.: 5
skills.sh installs: 13.8K
GitHub stars: 4K
last updated: Aug 21, 2026

View on GitHub

Install

$ npx -y skills add sanyuan0704/sanyuan-skills --skill code-review-expert --agent claude-code

Installs into .claude/skills of the current project.

Files

Code Review Expert

Overview

Perform a structured review of the current git changes with focus on SOLID, architecture, removal candidates, and security risks. Default to review-only output unless the user asks to implement changes.

Severity Levels

Level	Name	Description	Action
P0	Critical	Security vulnerability, data loss risk, correctness bug	Must block merge
P1	High	Logic error, significant SOLID violation, performance regression	Should fix before merge
P2	Medium	Code smell, maintainability concern, minor SOLID violation	Fix in this PR or create follow-up
P3	Low	Style, naming, minor suggestion	Optional improvement

Workflow

1) Preflight context

Use git status -sb, git diff --stat, and git diff to scope changes.
If needed, use rg or grep to find related modules, usages, and contracts.
Identify entry points, ownership boundaries, and critical paths (auth, payments, data writes, network).

SKILL.md on GitHub