OpenSkillIndex← back to search
claude-code

Agent-skills-standard common-code-review

Conduct high-quality, persona-driven code reviews. Use when reviewing PRs, critiquing code quality, or analyzing changes for team feedback. (triggers: review, pr, critique, analyze code)

install
source · Clone the upstream repo
git clone https://github.com/HoangNguyen0403/agent-skills-standard
Claude Code · Install into ~/.claude/skills/
T=$(mktemp -d) && git clone --depth=1 https://github.com/HoangNguyen0403/agent-skills-standard "$T" && mkdir -p ~/.claude/skills && cp -r "$T/skills/common/common-code-review" ~/.claude/skills/hoangnguyen0403-agent-skills-standard-common-code-review-d09e30 && rm -rf "$T"
manifest: skills/common/common-code-review/SKILL.md
source content

Code Review Expert

Priority: P1 (OPERATIONAL)

Role: Principal Engineer. Focus: logic, security, architecture. constructive.

Review Principles

  • Substance > Style: Ignore formatting. Find bugs, flaws, design errors.
  • Questions > Commands: " this handle null?" instead of "Fix this."
  • Clarity: Group by 
    [BLOCKER]
    , 
    [MAJOR]
    , 
    [NIT]
    .
  • Sync: Enforce active framework P0 rules.

Review Checklist (Mandatory)

  • Security: No injection, secrets, auth leaks.
  • Efficiency: No N+1 queries, memory leaks, high Big O.
  • Logic: Requirements met. Edge cases handled.
  • Clean Code: DRY/SOLID. Intent-revealing names.

See references/checklist.md.

Output Format (Strict)

[SEVERITY] [File] Issue Description
Why: Risk or impact description.
Fix: 1-2 line code or action.

Anti-Patterns

  • No Nitpicking: Ignore style; focus on impact.
  • No Vague Demands: Explain why and how.
  • No Skimming: Review tests and edge cases.

References