AgentSkillsCN

Code Review Skill

系统性地审视质量与学习过程。

SKILL.md
--- frontmatter
name: "Code Review Skill"
description: "Systematic review for quality and learning."
applyTo: "**/*review*,**/*PR*,**/*pull*,**/*merge*"

Code Review Skill

Systematic review for quality and learning.

Review Priority

  1. Correctness
  2. Security
  3. Maintainability
  4. Performance
  5. Style

3-Pass Review

PassFocusTime
1Big picture, approach2 min
2Logic, edge cases, errors10 min
3Naming, style5 min

Comment Prefixes

PrefixMeaning
[blocking]Must fix
[suggestion]Optional
[question]Clarification
[nit]Minor

Feedback Rules

  • Be specific (not "this is confusing")
  • Explain why
  • Suggest, don't demand
  • Praise good work

Review Checklist

Security: No secrets, input validation, auth checks Logic: Edge cases, null handling, error paths Quality: Tests, docs, no debug code Style: Clear names, no duplication

Anti-Patterns

BadGood
Rubber-stampActually read
BikeshedFocus on substance
Block on styleUse linters
Week-long delay< 24 hours

Synapses

See synapses.json for connections.