name: review
description: 코드 리뷰, diff 감사, 변경사항 검토 요청 시 자동 활성화됩니다. 변경된 코드의 정확성과 품질을 빠르게 점검합니다.

코드 리뷰어

활성화 조건

• "리뷰해줘", "코드 검토", "diff 봐줘" 요청
• PR 전 변경사항 점검
• 특정 파일/함수에 대한 피드백 요청

qa와의 차이

• review: 변경된 diff만 대상. 빠른 피드백.
• qa: 프로젝트 전체 감사 (보안/성능/접근성). /qa 또는 감사 요청 시.

실행 절차

Step 1: 변경 범위 파악

git diff --staged
git diff HEAD
git log --oneline -5

변경된 파일과 라인만 리뷰 대상으로 한정.

Step 2: 정확성 검토

• 로직 오류, 엣지 케이스 누락
• 변수/함수명 의도와 동작 불일치
• 에러 핸들링 누락

Step 3: 품질 검토

• 불필요한 복잡도, 중복 코드
• 타입 안정성, 네이밍 일관성
• 기존 코드 스타일과의 정합성

Step 4: 안전성 검토

• 변경으로 인한 사이드 이펙트
• 동작 변경에 대한 테스트 유무
• 명백한 보안 문제 (인젝션, 노출)

Thinking Cycle (필수)

모든 작업에 사고 사이클을 적용한다. 상세: ../_shared/resources/thinking-cycle.md

1. 질문: 실행 전 최소 1개 소크라테스 질문 → 답변 전 진행 금지
2. 결정: 트레이드오프 존재 시 선택지 제시 → 근거 있는 선택 요구
3. 실행: Phase 0, 1 완료 후에만 진입
4. 코드 리딩: 변경 코드 이해도 점검 (레벨 C 기본)
5. 회고: 작업 완료 후 사용자 회고 → .claude/reflections/YYYY-MM-DD.md에 기록

핵심 규칙

1. diff에 집중: 변경되지 않은 코드는 리뷰 대상이 아님
2. 모든 발견 사항에 file:line, 영향, 수정 방법 포함
3. 증거 없는 추측성 발견 사항 금지
4. 동작 변경 시 테스트 누락은 발견 사항
5. 칭찬할 것이 있으면 함께 언급

보고 형식

## 리뷰 결과

### 변경 요약
- 변경 파일 N개, +X/-Y 라인

### 발견 사항

| 심각도 | 파일:라인 | 발견 사항 | 수정 방법 |
|--------|-----------|-----------|-----------|
| HIGH | `path:line` | 설명 | 수정안 |

### 종합 의견
- [전체적인 코드 품질 한 줄 평가]