AgentSkillsCN

component-design-reviewer

审查React组件,并提供改进建议。当你需要代码审查、组件审查、结构优化、重构,或组件拆分时,可使用此技能。

SKILL.md
--- frontmatter
name: component-design-reviewer
description: Reviews React components and provides improvement suggestions. Use when you need code review, component review, structure improvement, refactoring, or component separation.
allowed-tools: Read, Grep, Glob

Component Design Reviewer

React 컴포넌트 설계 원칙에 따른 전문적인 코드 리뷰를 제공하는 Skill입니다. Claude가 직접 코드를 분석하여 5가지 핵심 원칙을 중심으로 상세한 리포트를 생성합니다.

검사 원칙 요약

  1. SRP - 하나의 컴포넌트, 하나의 역할
  2. Props - drilling 최소화, 명확한 interface
  3. Composition - 상속보다 합성
  4. Reusability - 공통 요소 추출
  5. Custom Hooks - UI/비즈니스 로직 분리

Instructions

  1. Read로 대상 컴포넌트 파일 읽기
  2. 5가지 원칙별로 체크리스트 기반 분석
  3. Grep으로 props drilling, hooks 패턴 검색
  4. 컴포넌트별 리포트 생성 (Critical/Warning/Suggestion)
  5. 문제마다 "문제 코드" vs "개선 코드" 예시 제공

리뷰 체크리스트

단일 책임 원칙 검사

  • 컴포넌트가 하나의 역할만 수행하는가? (위반 신호: 이름에 "And" 포함)
  • UI 렌더링과 비즈니스 로직이 분리되어 있는가? (위반 신호: 여러 API 호출)
  • 컴포넌트 크기가 적절한가? (권장: 150줄 이하)
  • 조건부 렌더링이 복잡하지 않은가? (권장: 3단계 이내)

Props 설계 검사

  • Props drilling이 3단계 이상인가?
  • Props interface가 명확하게 정의되어 있는가?
  • Default props가 적절히 활용되고 있는가?
  • Props 개수가 적절한가? (권장: 7개 이하)

합성 패턴 검사

  • 상속 대신 합성을 사용했는가?
  • children props를 적절히 활용했는가?
  • 컴포넌트 합성이 유연한가?
  • Render props / Compound components가 필요한 상황인가?

재사용성 검사

  • 공통 UI 요소가 추출되어 있는가?
  • 범용 vs 특화 컴포넌트가 적절히 구분되어 있는가?
  • 컴포넌트가 다른 곳에서 재사용 가능한가?
  • 하드코딩된 값이 없는가?

Custom Hooks 검사

  • 비즈니스 로직이 custom hooks로 분리되어 있는가?
  • useState/useEffect 로직이 컴포넌트 내에 과도하게 있지 않은가?
  • 재사용 가능한 로직이 hooks로 추출되어 있는가?
  • Hooks 명명이 use로 시작하고 명확한가?

출력 형식

examples.md의 "리뷰 출력 형식" 섹션을 따를 것