Reviewer Skill
리뷰 원칙
- •비판이 아닌 개선 방향 제시
- •문제 제기 시 대안 제시
- •우선순위 태그 사용
우선순위 태그
| 태그 | 의미 |
|---|---|
| [Critical] | 반드시 수정 (버그, 보안) |
| [Major] | 수정 권장 (성능, 설계) |
| [Minor] | 제안 (가독성, 스타일) |
체크리스트
버그 & 로직
- • Null/Undefined 처리
- • 경계 조건 (빈 배열, 0 등)
- • 비동기 에러 핸들링
- • useEffect 의존성
보안
- • XSS (dangerouslySetInnerHTML 사용 여부)
- • Server Action 데이터 노출
- • 인증/인가 체크
- • 민감 정보 클라이언트 노출
성능
- • 불필요한 리렌더링
- • N+1 쿼리
- • 대용량 데이터 처리
아키텍처 (Kent Beck Style)
- • SRP 준수 (하나의 함수/클래스 = 하나의 책임)
- • 순수 함수 사용 (사이드 이펙트 격리)
- • 불변성 패턴 (객체 수정 대신 새 객체)
- • 명확한 의도 (이름만으로 역할 파악)
- • YAGNI/KISS 준수 (불필요한 추상화 없음)
- • 확장 포인트 적절 (인터페이스 활용)
컨벤션
- • 서버/클라이언트 분리 적절
- • Repository 패턴 준수
- • 기존 패턴과 일관성
리뷰 템플릿
markdown
## 요약 [변경 사항 요약] ## 좋은 점 - ... ## 개선 필요 ### [Critical] 제목 - 위치: `파일:라인` - 문제: ... - 제안: ...
상세 패턴은 코드베이스의 기존 구현 참조