검토 기준표

리뷰는 의견보다 관점 분리가 먼저다

PR을 볼 때는 기능 동작, 실패 경계, 성능 비용, 유지보수성을 분리하면 빠르게 승인할 부분과 더 봐야 할 부분이 명확해집니다.

자동화 스타일과 기본 오류

린트, 포맷, 타입, 테스트는 사람 리뷰 전에 통과시킵니다.

사람 의도와 위험

요구사항 해석과 운영 위험은 맥락을 아는 사람이 확인합니다.

A

기능 정확성

  • 요구사항과 화면 흐름이 일치하는지 확인합니다.
  • 서버 컴포넌트와 클라이언트 컴포넌트 책임이 섞이지 않았는지 봅니다.
B

경계와 오류

  • 빈 데이터, 권한 없음, 네트워크 실패 상태를 처리했는지 점검합니다.
  • 환경 변수와 서버 전용 코드가 클라이언트로 새지 않는지 봅니다.
C

성능 비용

  • 불필요한 클라이언트 번들과 반복 fetch가 생기지 않았는지 확인합니다.
  • 이미지, 폰트, 스크립트 최적화 대상이 빠지지 않았는지 봅니다.
D

유지보수성

  • 파일 위치와 이름이 App Router 구조 안에서 예측 가능한지 확인합니다.
  • 테스트와 설명이 다음 수정자를 충분히 돕는지 살핍니다.
피드백은 수정 요청과 질문을 구분한다
수정

동작 오류, 보안 위험, 회귀 가능성처럼 반드시 바꿔야 하는 항목입니다.

질문

맥락 확인이나 대안 제안처럼 작성자의 판단을 먼저 듣는 항목입니다.