Component Design Reviewer
React 컴포넌트 설계 원칙에 따른 전문적인 코드 리뷰를 제공하는 Skill입니다. Claude가 직접 코드를 분석하여 5가지 핵심 원칙을 중심으로 상세한 리포트를 생성합니다.
검사 원칙 요약
- •SRP - 하나의 컴포넌트, 하나의 역할
- •Props - drilling 최소화, 명확한 interface
- •Composition - 상속보다 합성
- •Reusability - 공통 요소 추출
- •Custom Hooks - UI/비즈니스 로직 분리
Instructions
- •Read로 대상 컴포넌트 파일 읽기
- •5가지 원칙별로 체크리스트 기반 분석
- •Grep으로 props drilling, hooks 패턴 검색
- •컴포넌트별 리포트 생성 (Critical/Warning/Suggestion)
- •문제마다 "문제 코드" 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의 "리뷰 출력 형식" 섹션을 따를 것