| name | code-review |
| description | This skill should be used when the user wants to "review code", "check code quality", "run code review", or needs local code quality verification before git push. Runs lint, typecheck, and code review on changed files. |
| version | 0.1.0 |
로컬 코드 품질 검증
git push 전에 변경된 코드의 품질을 검증한다.
프로세스
Step 1: 변경 파일 확인
git diff --name-only로 변경된 파일 목록 확인
Step 2: 정적 분석
yarn lint 실행하여 린트 에러 확인yarn typecheck 실행하여 타입 에러 확인
Step 3: 코드 리뷰
변경된 파일을 읽고 아래 기준에 따라 분석
검토 기준
- 보안 취약점
- console.log 존재 여부
- 반복 코드, 유틸 함수화 가능 여부
- 경계값 처리: 빈 값, 음수, 최대값 등 엣지 케이스
- 스타일: lint 규칙 준수, 일관된 네이밍
- 타입 정의: interface/type 정의, any 타입 남용
- null 처리: optional chaining (
?.), nullish coalescing (??) 사용
- dead code 여부
- locale 파일은 리뷰하지 않음
Side-Effect 분석
- 삭제된 코드 분석: 삭제된 로직이 어떤 역할을 했는지, 그 기능이 다른 곳에서 대체되었는지 확인
- 기존 동작 보존 여부: 변경 전후로 동일한 입력에 대해 동일한 결과가 나오는지 검증
- 엣지 케이스 시나리오: 정상 케이스뿐 아니라 취소, 실패, 빈 값 등 예외 상황에서의 동작 검토
- 콜백/함수 호출 추적: 파라미터로 받은 콜백이 실제로 호출되는지, Dead Code가 없는지 확인
- 타입 변경 영향: 시그니처 변경 시 호출하는 모든 곳에서 호환되는지 검토
리뷰 결과 형식
## 검증 결과
### 1. ESLint
- [ ] 통과 / 실패 (에러 N개)
### 2. TypeScript
- [ ] 통과 / 실패 (에러 N개)
### 3. 코드 리뷰
- 문제점: ...
- 개선 제안: ...
### 결론 및 분류 기준
| 이모지 | 등급 | 판별 기준 |
|--------|------|--------------------------------------------------------|
| 🔴 | P2 | 반드시 수정 필요 - Critical한 이슈만 해당 |
| 🟡 | P3 | 수정 권장 - 높은 확률로 개선해야 하는 이슈 |
| 🟢 | P4 | 의견 - 개선하면 좋지만 안해도 큰 문제 없는 이슈 |
| 🟢 | P5 | 참고/칭찬 - 단순 의견과 개선점, 그리고 칭찬 |
옵션
빌드 검증이 필요하면 "빌드도 검증해줘"라고 요청하면 yarn build도 실행한다.
