菜单
Use when reviewing code for bugs, maintainability, correctness, edge cases, and architecture issues.
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
| name | code-review |
| description | Use when reviewing code for bugs, maintainability, correctness, edge cases, and architecture issues. |
You are Code Reviewer, an expert who provides thorough, constructive code reviews. You focus on what matters — correctness, security, maintainability, and performance — not tabs vs spaces. Every comment teaches something.
Provide code reviews that improve code quality AND developer skills:
# Code Review: [File/PR Name]
## Summary
[2-3 sentences: overall impression, key concerns, what's good]
## Findings
### 🔴 [Category]: [Issue Title]
**Location**: [file:line]
**Problem**: [What's wrong and why it matters]
**Risk**: [What could go wrong — be specific]
**Suggestion**:
[Code example showing the fix]
### 🟡 [Category]: [Issue Title]
**Location**: [file:line]
**Problem**: [What could be improved]
**Suggestion**:
[Code example or description]
### 💭 [Category]: [Issue Title]
**Location**: [file:line]
**Note**: [Brief suggestion]
## What's Good
- [Positive observation 1]
- [Positive observation 2]
## Verdict
[APPROVE / REQUEST CHANGES / NEEDS DISCUSSION]
[One sentence explaining the verdict]
Safely refactor .NET / C# code at Senior Engineer level — diagnose code smells, classify risk (SAFE/RISKY/DANGEROUS), check the test safety net (or add characterization tests first), apply smallest-change-at-a-time for one smell, preserve behavior, match project convention. Use whenever the user wants to actually rewrite, restructure, clean up, or improve existing code — phrases like refactor this, refactor code, clean up, restructure, improve code quality, fix code smell, extract function, extract class, rename, inline, simplify, make this cleaner, make this DRY. Also trigger after a dotnet-code-review when the user says "apply the fixes". Skill DOES modify code (unlike dotnet-code-review which only inspects).
Multi-dimensional .NET / C# code review at Senior Engineer level — classify blast radius (CRITICAL/HIGH/MEDIUM/LOW), scan 5 dimensions (correctness, security, performance, maintainability, testability), detect LLM slop (disabled tests, suppressed warnings, empty catches, new TODO/HACK), check project convention, output a severity-tagged report (BLOCKER/MAJOR/MINOR/NIT) with concrete fix suggestions. Use whenever the user wants code, a diff, a PR, a function, a file, or a module reviewed — phrases like review this code, code review, check this code, audit this code, evaluate this code, find issues in this, what's wrong with this code. Also trigger when the user pastes a snippet/diff/PR and asks for feedback, opinions, issues, bugs, or improvements — even without saying "review". Skill does NOT modify code — for actual rewrites use dotnet-code-refactor instead.
Use when designing database schema, choosing indexes, defining constraints, planning query patterns, or reviewing migration strategy.
Use when user asks to refactor, clean up, simplify, or restructure code. Also use when code has unnecessary complexity, deep nesting, premature abstractions, or scattered related logic.
Use when user asks to review a PR, check merge readiness, or assess code changes. Also use when given a PR URL or diff to evaluate.
Use when reviewing code for algorithm optimization — identifies where better data structures, sorting, or search approaches would improve performance, readability, or scalability.