code-review

Code Review

Purpose

사용자가 /dev 파이프라인 밖에서 직접 코드 리뷰를 요청할 때 사용한다. code-reviewer 에이전트를 호출하는 래퍼 스킬.

Argument Parsing

(인수 없음)              → 워킹 디렉토리 변경(unstaged) + 스테이징된 변경(staged) 리뷰
"src/auth/"              → 특정 경로의 최근 변경 리뷰
"--staged"               → git staged 변경만 리뷰
"--branch feature/auth"  → 현재 브랜치와 main/master 분기점 이후 전체 diff 리뷰

Workflow

Step 1: 리뷰 대상 Diff 수집

인수에 따라 적절한 git diff 명령어를 실행한다:

# 인수 없음 — unstaged + staged 변경 모두 수집
# 먼저 커밋이 존재하는지 확인한다
git rev-parse HEAD 2>/dev/null
# 커밋이 있으면:
git diff HEAD
# 커밋이 없으면 (BRAND_NEW repo):
git diff          # unstaged 변경
git diff --staged # staged 변경
# 두 결과를 합쳐서 리뷰 대상으로 한다

# 특정 경로
git diff HEAD -- <path>
# 커밋이 없으면: git diff -- <path> + git diff --staged -- <path>

# --staged
git diff --staged

# --branch
git merge-base main HEAD  # 분기점 찾기
git diff <merge-base>..HEAD

변경된 파일이 없으면 "변경사항이 없습니다" 메시지 출력 후 종료한다.

Step 2: PLAN 문서 탐색 (선택)

ls docs/plans/PLAN_*.md 2>/dev/null

PLAN이 존재하면, 변경된 파일이 어떤 PLAN/Phase에 해당하는지 매핑한다:

• 매핑되면 PLAN 문서를 code-reviewer에게 컨텍스트로 전달한다
• 매핑되지 않으면 PLAN 없이 리뷰한다

Step 3: code-reviewer Subagent 실행

사용하는 Subagent: agents/code-reviewer.md

• PLAN이 있고 특정 Phase에 매핑되면 phase 모드로 실행한다
• PLAN이 있지만 특정 Phase에 매핑되지 않거나, --branch로 넓은 범위를 리뷰하면 integration 모드로 실행한다
• PLAN이 없으면 PLAN 컨텍스트 없이 리뷰한다 (code-reviewer에 PLAN 경로를 전달하지 않음)
• 전달 항목: diff 범위, PLAN 경로 (있으면), git ref (있으면)

Step 4: 결과 출력

code-reviewer의 결과를 사용자에게 표시한다.

REQUEST_CHANGES인 경우, AskUserQuestion으로 확인:

1. 자동 수정 — 리뷰 이슈를 반영하여 코드 수정
2. 건너뛰기 — 이번에는 수정하지 않음

Step 5: 수정 적용 (선택)

자동 수정 선택 시:

1. 이슈별로 코드를 수정한다
2. 수정 후 다시 code-reviewer를 실행하여 이슈가 해결되었는지 확인한다

Related Files

File	Purpose
agents/code-reviewer.md	Subagent: 코드 리뷰 실행
docs/plans/PLAN_*.md	컨텍스트: PLAN 대비 충실도 평가용