code-review

Code Review

PR 요청부터 피드백 대응까지의 코드 리뷰 통합 스킬입니다.

Part 1: Requesting Review

PR 생성 전 체크리스트

• 테스트 통과 확인
• 린트 에러 없음
• 변경 범위 명확
• 커밋 메시지 규격 준수

PR 설명 템플릿

## What
[변경 사항 한 줄 요약]

## Why
[변경 이유]

## How
[구현 방법]

## Testing
- [ ] 단위 테스트 추가
- [ ] 수동 테스트 완료

## Screenshots (UI 변경 시)
[스크린샷]

Part 2: Receiving Review

피드백 대응 원칙

대응 패턴

# 수용 시
"좋은 지적이에요. 수정했습니다. [커밋 링크]"

# 설명 필요 시
"이렇게 한 이유는 [근거]입니다. 더 나은 방법이 있을까요?"

# 동의 안 할 때
"이해합니다만, [이유]로 현재 방식을 유지하고 싶습니다. 어떻게 생각하세요?"

Best Practices

리뷰 요청자

• 변경을 작게 유지 (300줄 이하)
• 명확한 컨텍스트 제공
• 셀프 리뷰 먼저

리뷰어

• 건설적인 피드백
• 중요도 표시 (Must/Should/Could)
• 칭찬도 함께

Checklist

• PR 설명 완성
• 테스트 통과
• 피드백 모두 대응
• 최종 승인 확인