코드 리뷰 기본기

1. 정확성 (Correctness)     ← 가장 중요
   "이 코드가 의도한 대로 동작하는가?"

2. 보안 (Security)
   "취약점이 있는가?"

3. 에러 처리 (Error Handling)
   "실패하면 어떻게 되는가?"

4. 설계 (Design)
   "구조가 적절한가?"

5. 가독성 (Readability)
   "다른 사람이 이해할 수 있는가?"

6. 스타일 (Style)           ← 린터에 맡기기
   "규칙을 따르는가?"

역할: "시니어 백엔드 개발자로서 이 코드를 리뷰해주세요."

맥락: "이 PR은 결제 API의 재시도 로직을 리팩토링합니다.
      현재 타임아웃 시 무한 재시도 버그가 있어서 수정합니다."

초점: "다음을 중점적으로 봐주세요:
      1. 재시도 로직의 정확성
      2. 동시 요청 처리 시 race condition
      3. 에러 응답 코드의 적절성"

출력: "각 이슈를 심각도로 분류해주세요:
      🔴 Critical: 반드시 수정
      🟡 Warning: 수정 권장
      🔵 Info: 개선 제안"

코드 리뷰 체크리스트:

□ 정확성
  - 의도한 기능을 정확히 수행하는가?
  - edge case 처리가 되어 있는가?
  - 기존 테스트가 깨지지 않는가?

□ 보안
  - 사용자 입력을 검증하는가?
  - SQL 인젝션, XSS 등 취약점은 없는가?
  - 인증/인가 체크가 적절한가?
  - 민감한 정보가 로그에 남지 않는가?

□ 에러 처리
  - 외부 API 호출 실패 시 대응이 있는가?
  - null/undefined 참조 가능성은?
  - 에러 메시지가 디버깅에 도움이 되는가?

□ 설계
  - 단일 책임 원칙(SRP)을 따르는가?
  - 함수/클래스가 너무 크지 않은가?
  - 적절한 추상화 수준인가?

□ 가독성
  - 변수/함수 이름이 의도를 설명하는가?
  - 복잡한 로직에 주석이 있는가?
  - 제3자가 이해하기 쉬운가?

# ❌ happy path만 처리
def get_user(user_id):
    response = api.fetch(f"/users/{user_id}")
    return response.json()["data"]

# ✅ 실패 경로도 처리
def get_user(user_id):
    try:
        response = api.fetch(f"/users/{user_id}", timeout=5)
        response.raise_for_status()
        data = response.json()
        return data.get("data")
    except Timeout:
        logger.warning(f"User API timeout: {user_id}")
        raise ServiceUnavailableError("사용자 서비스 응답 지연")
    except HTTPError as e:
        logger.error(f"User API error: {e.status_code}")
        raise