AgentSkillsCN

code-review

对代码的质量、安全性与可维护性进行全面审查。当您需要评审Pull Request、检视代码变更、开展代码审查、执行质量保证测试,或为复杂任务进行架构评审时,此技能将为您提供专业指导。其中包含QA检查清单与CTO评审模板。

SKILL.md
--- frontmatter
name: code-review
description: Review code for quality, security, and maintainability. Use when reviewing pull requests, examining code changes, doing code review, QA checks, or architecture review for COMPLEX tasks. Includes QA checklist and CTO review template.

Code Review

QA Checklist (every review)

Code Quality

  • 0 linter errors
  • 0 TypeScript errors
  • No console.log (except debug)
  • Tests pass
  • Edge cases handled
  • Error cases handled

Naming Standards

TypeConventionExample
Files (components)PascalCaseComponentName.tsx
Files (utils)camelCaseutilName.ts
FunctionscamelCasefunctionName()
ComponentsPascalCaseComponentName
ConstantsUPPER_SNAKECONSTANT_NAME

JTBD Verification (for user-facing features)

  • Job Story implemented
  • UI text about benefits, not features
  • Minimum steps to result
  • No confusing terminology

Feedback Format

  • CRITICAL: Must fix before merge
  • SUGGESTION: Consider improving
  • NICE TO HAVE: Optional enhancement

CTO Review (for COMPLEX tasks)

For architecture decisions, use the full template in CTO-REVIEW.md.

Apply CTO Review when:

  • Change affects >5 files
  • API change used by others
  • Data migration
  • New architectural concept