Agent skill

code-review

Use when reviewing code. Triggers: 'review my code', 'check my work', 'look over this', 'review PR #X', 'PR comments to address', 'reviewer said', 'address feedback', 'self-review before PR', 'audit this code'. Modes: --self (pre-PR self-review), --feedback (process received review comments), --give (review someone else's code/PR), --audit (deep single-pass analysis). For heavyweight multi-phase analysis, use advanced-code-review instead.

Stars 163
Forks 31

Install this agent skill to your Project

npx add-skill https://github.com/majiayu000/claude-skill-registry/tree/main/skills/other/other/code-review-axiomantic-spellbook

SKILL.md

Code Review

Invariant Principles

  1. Evidence Over Assertion - Every finding needs file:line reference
  2. Severity Honesty - Critical=security/data loss; Important=correctness; Minor=style
  3. Context Awareness - Same code may warrant different severity in different contexts
  4. Respect Time - False positives erode trust; prioritize signal

Inputs

Input Required Description
args Yes Mode flags and targets
git diff Auto Changed files
PR data If --pr PR metadata via GitHub

Outputs

Output Type Description
findings List Issues with severity, file:line
status Enum PASS/WARN/FAIL or APPROVE/REQUEST_CHANGES

Mode Router

Flag Mode Command File
--self, -s, (default) Pre-PR self-review (inline below)
--feedback, -f Process received feedback code-review-feedback
--give <target> Review someone else's code code-review-give
--audit [scope] Multi-pass deep-dive (inline below)

Modifiers: --tarot (roundtable dialogue via code-review-tarot), --pr <num> (PR source)


MCP Tool Integration

Tool Purpose
pr_fetch(num_or_url) Fetch PR metadata and diff
pr_diff(raw_diff) Parse diff into FileDiff objects
pr_match_patterns(files, root) Heuristic pre-filtering
pr_files(pr_result) Extract file list

Principle: MCP tools for read/analyze. gh CLI for write operations (posting reviews, replies).

Fallback: MCP unavailable -> gh CLI -> local diff -> manual paste.


Self Mode (--self)

Workflow:

  1. Get diff: git diff $(git merge-base origin/main HEAD)..HEAD
  2. Multi-pass: Logic > Integration > Security > Style
  3. Generate findings with severity, file:line, description
  4. Gate: Critical=FAIL, Important=WARN, Minor only=PASS

Audit Mode (--audit [scope])

Scopes: (none)=branch changes, file.py, dir/, security, all

Passes: Correctness > Security > Performance > Maintainability > Edge Cases

Output: Executive Summary, findings by category, Risk Assessment (LOW/MEDIUM/HIGH/CRITICAL)


Self-Check

  • Correct mode identified
  • All findings have file:line
  • Severity based on impact, not effort
  • Output matches mode spec

Expand your agent's capabilities with these related and highly-rated skills.

Didn't find tool you were looking for?

Be as detailed as possible for better results