리뷰 항목은 기능, 경계, 성능, 유지보수성으로 나누면 감정이 줄어든다
좋은 리뷰는 취향 논쟁이 아니라 변경 코드가 어떤 위험을 만들 수 있는지 근거를 남기는 과정이다.
| 관점 | 확인 질문 | 나쁜 신호 | 좋은 피드백 |
|---|---|---|---|
| 기능 정확성 | 요구사항과 예외 케이스를 만족하나 | 성공 경로만 테스트 | 실패 입력과 빈 상태를 함께 확인 |
| 경계와 보안 | 클라이언트와 서버 책임이 섞이지 않았나 | 비밀값 노출, 검증 누락 | 서버 액션/Route Handler에서 재검증 |
| 성능 영향 | 렌더링, 네트워크, 번들 크기가 늘지 않았나 | 큰 라이브러리 즉시 로드 | 동적 import와 측정 근거 제시 |
| 유지보수성 | 이름, 책임, 테스트 위치가 자연스러운가 | 한 파일에 기능 과밀 | 작은 단위로 분리하고 테스트 보강 |