review-code

Agent Skills › avibebuilder/claude-prime › review-code

review-code

GitHub

用于对现有代码进行质量、正确性和适配性评估。适用于PR、补丁或文件审查，提供批判性分析而非修复。根据风险调整深度，输出包含判定、关键问题和建议的结构化报告。

.claude/skills/review-code/SKILL.md avibebuilder/claude-prime

Trigger Scenarios

用户请求代码审查询问代码是否良好要求检查现有代码的正确性或质量

Install

npx skills add avibebuilder/claude-prime --skill review-code -g -y

More Options

Non-standard path

npx skills add https://github.com/avibebuilder/claude-prime/tree/main/.claude/skills/review-code -g -y

Use without installing

npx skills use avibebuilder/claude-prime@review-code

指定 Agent (Claude Code)

npx skills add avibebuilder/claude-prime --skill review-code -a claude-code -g -y

安装 repo 全部 skill

npx skills add avibebuilder/claude-prime --all -g -y

预览 repo 内 skill

npx skills add avibebuilder/claude-prime --list

SKILL.md

Frontmatter

{
    "name": "review-code",
    "description": "Review code for quality, correctness, and fit. Use when the user wants judgment on code that already exists — their own changes, a teammate's patch, a PR, branch, commit, diff, staged changes, or one or more files to look over. Activate on requests like review, look over, sanity check, critique, code review, or 'is this good?' The key signal is that the user wants evaluation of existing code and its tradeoffs, not implementation, debugging, or explanation. This skill works independently, but when plans, specs, task artifacts, or prior discussion exist, use them to understand why the code exists before judging it.",
    "argument-hint": "what-to-review"
}

ultrathink

Process

Critique only — no fixes or implementation. Understand why the code exists before judging it.

1. Identify the review target

Prefer in order: explicit <target>, changes made in this conversation, working diff. If unclear, ask.

Then gather enough context (PR/issue text, plans, surrounding code) to avoid misreads. Don't turn every review into archaeology — if a missing detail would materially change judgment, ask briefly.

2. Match depth to risk

Quick — small, local, low-risk; keep output brief
Standard — normal feature or refactor
Deep — risky, cross-cutting, security-sensitive, or surprising

3. Review flow first, then details

Understand how important parts work before commenting line by line. Focus on load-bearing logic, edge cases, scope drift, and intentional tradeoffs.

4. Report

For each finding: file path + line, why it matters, severity.

Critical — correctness, safety, broken edge cases, scope violations. Requires evidence in the code — speculative failure modes belong in Questions.
Suggestion — non-blocking improvements
Question — missing context or unconfirmable concerns

If something looks odd but context supports it, acknowledge rather than flag.

5. Output format

## Code Review: [scope]

**Verdict: {Approve | Request Changes | Reject}**

### Inferred Intent
[Only when context supports it]

### Critical Issues (Must Fix)
- `path/to/file:line` — concise description. Why: [reason]

### Suggestions (Nice to Have)
- `path/to/file:line` — concise description

### Questions
- [Clarifications on intent or tradeoffs]

### Summary
[1-2 sentences]

Include only sections with content. Always include Verdict and Summary.

Boundaries

Read surrounding code when the diff alone is misleading.
Deliberate decisions (inline comments, user-stated tradeoffs) are not defects — raise as Question only with concrete evidence.
Don't judge by personal preference when project fit explains the choice.
Name the right approach ("needs parameterized queries") but don't write corrected code.