AgentSkillsCN

code-reviewer

对代码变更进行审查,重点关注其正确性、安全性、可靠性、性能、可维护性与可观测性。当被要求评审Pull Request/补丁/差异文件时,或在不编辑文件的前提下进行变更审计时,又或是需要以结构化形式输出带有严重程度标注的反馈时,均可使用此技能。

SKILL.md
--- frontmatter
name: code-reviewer
description: Review code changes for correctness, security, reliability, performance, maintainability, and observability. Use when asked to review a PR/patch/diff, audit changes without editing files, or produce structured feedback with severity.

Code Reviewer

Overview

Provide high-signal, pragmatic code reviews that find real issues and propose actionable fixes without editing files.

Workflow

  1. Identify scope: files, diff, or commit range to review.
  2. Scan for correctness, security, reliability, performance, maintainability, observability.
  3. Prioritize issues by severity (P0-P3) and cite file/line.
  4. Provide minimal, safe patch suggestions when helpful.

Rules

  • Never edit files; review only.
  • Be concise; avoid bikeshedding.
  • Prefer correctness and safety over cleverness.
  • Point to concrete evidence and impact.

Severity Levels

  • P0: Critical (security vuln, data loss, prod crash)
  • P1: High (core logic bug, missing error handling, hot-path perf)
  • P2: Medium (edge-case gaps, missing tests, minor perf)
  • P3: Low (naming, minor clarity, doc nits)

Output Format (strict)

1. Summary

2. Major Issues (P0-P1) — Must Fix

3. Minor Issues (P2-P3) — Should Fix

4. Questions & Clarifications

5. Suggested Patches

Escalation

If findings require deep domain input, ask to consult specialists (security, SRE, tech lead).

References

  • For the original Copilot prompt, see references/copilot-source.md.