코드 리뷰 기본 사항
사용자가 지정한 변경 사항을 검토합니다. 사용자가 지정하지 않은 경우, 현재 브랜치를 베이스 브랜치와 비교하여 검토합니다.
- 베이스 브랜치를 찾습니다:
git merge-base HEAD origin/main을 시도한 후,origin/master,origin/develop순서로 시도합니다. 모두 없으면 커밋되지 않은 변경 사항(git diff HEAD)을 검토합니다. - 전체 diff를 읽은 다음, diff만으로 판단하기 어려운 변경 사항이 있으면 주변 코드를 열어 확인합니다.
체크리스트
각 항목을 확인하세요. 실제 발견 사항이 있는 경우에만 보고합니다.
정확성
- 코드가 의도한 변경을 올바르게 수행하는가? off-by-one 오류, 잘못된 조건, 처리되지 않은
None/null, 무시된 예외를 찾으세요. - 빈 입력, 매우 큰 입력, 동시 호출, 재시도 같은 엣지 케이스가 처리되었는가?
테스트
- 모든 동작 변경 사항이, 그 테스트 없이는 실패할 테스트로 커버되는가?
- 테스트가 구현 세부 사항이 아닌 동작을 검증하는가?
가독성
- 이름이 정확한가? 새로운 팀원이 한 번에 이해할 수 있는가?
- 죽은 코드, 디버그 출력, 주석 처리된 코드가 남아 있는가?
보안 (기본)
- 코드, 설정, 로그에 비밀번호, 토큰, 자격 증명이 포함되어 있지 않은가?
- 사용자 입력이 검증되고, 쿼리는 문자열 연결 없이 매개변수화되었는가?
일관성
CLAUDE.md와 주변 코드의 관례를 따르는가?
출력
심각도별로 발견 사항을 그룹화하세요.
- 반드시 수정: 버그, 보안 문제, 위험한 로직에 대한 누락된 테스트.
- 권장 수정: 나중에 시간을 소모할 유지보수 문제.
- 사소한 개선: 선택적 개선 사항. 몇 개로 제한하세요.
각 발견 사항에 대해 파일:라인, 문제, 구체적인 수정 제안을 제공하세요. 한 줄 평결로 마무리하세요: 병합 가능, 필수 수정 후 병합, 또는 재검토 필요. 특정 범주에서 발견 사항이 없으면 해당 범주는 생략하세요. 공간을 채우기 위해 문제를 지어내지 마세요.
Pro 에디션은 전체 보안, 성능, 접근성, API 디자인 체크리스트와 diff에 적합한 항목을 선택하는
/review명령을 추가로 제공합니다.