🛡️ 코드 리뷰 워크플로우

Claude 구현 → Codex 검증으로 이어지는 리뷰 절차를 표준화한 워크플로우입니다.

기본 절차

① Claude가 구현 완료
      ↓
② Codex에 검증 요청
      ↓
③ gate-matrix로 작업 유형 분류
      ↓
④ code-review-guard 스킬로 리뷰
      ↓
   ┌──┴──┐
  PASS  HOLD
   ↓      ↓
  병합   Claude 수정 → 재검증

🗂️ 게이트 매트릭스

작업 유형에 따라 필수 스킬, 선택 스킬, 검증 포커스가 달라집니다. 작업이 여러 유형에 걸칠 때는 가장 위험한 유형을 주 유형으로 잡고 게이트를 합산합니다.

대표 유형별 게이트

작업 유형	필수 스킬	선택 스킬	검증 포커스
기능 구현/리팩터	code-review-guard	vercel-react-best-practices	G1, G2, G4, G5
인증/권한/로그인	code-review-guard	—	G2, G3, G4, G5
/saju 입력 흐름	saju-frontend, code-review-guard	—	G1, G2, G4, G5
/saju/result 결과 조회	saju-frontend, code-review-guard	vercel-react-best-practices	G1, G2, G4, G5
React Query 캐시/무효화	saju-frontend, code-review-guard	vercel-react-best-practices	G1, G2, G4, G5
BFF/API route 변경	saju-frontend, code-review-guard	—	G1, G2, G3, G4, G5
쿠키/세션/민감정보 처리	saju-frontend, code-review-guard	—	G2, G3, G4, G5
UI/디자인 전수 수정	code-review-guard	web-design-guidelines	G1, G4, G5 + 접근성
디자인 시스템/토큰	design-system-hybrid, code-review-guard	web-design-guidelines	G1, G2, G4, G5 + 접근성
성능 개선	code-review-guard	vercel-react-best-practices	G1, G2, G4, G5 + 성능 근거
CI/CD 워크플로우	saju-frontend, code-review-guard	—	G1, G3, G4, G5
docs 사이트	saju-docs, code-review-guard	—	G1, G2, G4, G5
Notion/작업 기록	notion-weekly-worklog	change-summary-report	G5
릴리즈 직전 통합 점검	code-review-guard	vercel-react-best-practices, web-design-guidelines	G1~G5 전체

G1 비즈니스 로직 무결성 / G2 API 계약·타입 안전성 / G3 인증·보안 / G4 상태관리·캐시 일관성 / G5 회귀·운영 안정성

세부 유형과 검증 명령은 .agents/skills/codex-review-workflow/references/gate-matrix.md에서 관리합니다.

📋 필수 점검 항목

리뷰어가 반드시 확인하는 항목들이에요.

• 도메인 플로우 — /saju → /saju/result, /home, /mypage, /compatibility, /location, /login 각 경로의 정상 동작
• 인증 분기 — 로그인/미로그인 접근 제어 누락 여부
• React Query — cache key, invalidation, stale 데이터 노출 위험
• 타입 안전성 — nullable/optional 필드 처리, 실패 응답 처리 누락
• 보안 — localStorage/console에 민감정보 노출 여부
• Dead code — 하드코딩 flag, 임시 우회 분기

📄 리뷰 출력 형식

Codex의 리뷰 결과는 항상 아래 형식을 따릅니다.

### Verdict
MERGE: PASS  또는  MERGE: HOLD
(HOLD인 경우 이유 1문장)

### Findings
[Critical / High / Medium / Low] 제목
- 영향
- 근거: path:line
- 재현 조건
- 수정 제안

### Open Questions
- 요구사항 확인이 필요한 항목

### Summary
- 심각도별 이슈 수
- 병합 가능/보류 판단 근거
- 권장 테스트

:::tip 원칙 모든 이슈에는 파일/라인 근거가 포함돼야 합니다. 추측성 지적은 금지예요. :::

🔄 재검증 모드

수정 후 재요청이 오면 delta-only — 새로 바뀐 부분부터 우선 검증합니다. 단, 이전 Critical/High가 미해결이면 반드시 다시 보고합니다.