wasabeef/claude-code-cookbook/pr-review

# PR 을 포괄적으로 리뷰
gh pr view 123 --comments
"이 PR 을 체계적으로 리뷰해서 코드 품질, 보안, 아키텍처 관점으로 피드백하세요"
 
# 보안 특화 리뷰
gh pr diff 123
"보안 리스크와 취약점에 초점을 맞춰서 리뷰하세요"
 
# 아키텍처 관점의 리뷰
gh pr checkout 123 && find . -name "*.js" | head -10
"레이어 분리, 의존관계, SOLID 원칙 관점으로 아키텍처를 평가하세요"

# 코드 품질의 정량적 평가
find . -name "*.js" -exec wc -l {} + | sort -rn | head -5
"코드의 복잡도, 함수 크기, 중복도를 평가해서 개선점을 지적하세요"
 
# 보안 취약점 체크
grep -r "password\|secret\|token" . --include="*.js" | head -10
"기밀정보 누출, 하드코딩, 인증 우회 리스크를 체크하세요"
 
# 아키텍처 위반 탐지
grep -r "import.*from.*\.\./\.\." . --include="*.js"
"레이어 위반, 순환 의존, 결합도 문제를 평가하세요"

🔴 critical.must: 치명적 문제
├─ 보안 취약점
├─ 데이터 정합성 문제
└─ 시스템 장애 리스크
 
🟡 high.imo: 높은 우선도 개선
├─ 기능 불량 리스크
├─ 성능 문제
└─ 유지보수성 대폭 저하
 
🟢 medium.imo: 중간 우선도 개선
├─ 가독성 향상
├─ 코드 구조 개선
└─ 테스트 품질 향상
 
🟢 low.nits: 경미한 지적
├─ 스타일 통일
├─ 오타 수정
└─ 주석 추가
 
🔵 info.q: 질문·정보 제공
├─ 구현 의도 확인
├─ 설계 판단 배경
└─ 베스트 프랙티스 공유

**critical.must.** [문제점 요약]
 
[구체적인 위험 설명]
 
권장 해결방안:
[해결 방법]

**critical.must.** 패스워드가 평문으로 저장되고 있습니다
 
보안 리스크를 방지하기 위해 해싱화가 필수입니다.
 
권장 해결방안:
const bcrypt = require('bcrypt');
const hashedPassword = await bcrypt.hash(password, 12);

**high.imo.** [성능 문제점]
 
[문제 상황 설명]
 
개선안:
[최적화 방법]

**high.imo.** N+1 쿼리 문제가 발생합니다
 
쿼리 수를 대폭 줄일 수 있습니다.
 
개선안:
// Eager Loading 사용
const users = await User.findAll({ include: [Post] });

**high.must.** [아키텍처 문제점]
 
[위반 내용 설명]
[개선 방향 제시]

**high.must.** 레이어 위반이 발생하고 있습니다
 
도메인 레이어가 인프라 레이어에 직접 의존하고 있습니다.
의존성 역전 원칙으로 인터페이스를 도입하세요.