좋은 코드 리뷰 하는 법: 팀을 성장시키는 리뷰 문화

코드 리뷰는 버그를 잡는 동시에 팀의 지식을 공유하고 코드 표준을 맞추는 과정입니다. 잘하면 팀이 성장하고, 못하면 감정만 상합니다.

리뷰어를 위한 원칙

1) 사람이 아니라 코드를 말한다

"왜 이렇게 짰어요?"가 아니라 "이 부분은 이렇게 하면 더 안전할 것 같아요"로. 비난이 아니라 제안으로.

2) 중요도를 표시한다

모든 코멘트가 같은 무게가 아닙니다. 라벨을 붙이면 작성자가 우선순위를 압니다.

라벨	의미
[must]	반드시 고쳐야 함(버그·보안)
[should]	고치는 게 좋음
[nit]	사소한 취향(선택)
[question]	단순 질문

3) 이유와 대안을 함께

"이거 별로네요"(X) → "이 반복문은 입력이 커지면 O(n²)이라 Set으로 바꾸면 O(n)이 됩니다"(O).

좋은 코멘트 vs 나쁜 코멘트

나쁨: "이건 틀렸어요."
좋음: "[must] 여기서 null이 들어오면 터집니다.
       옵셔널 체이닝(?.)으로 방어하면 어떨까요?"

작성자를 위한 자세

• PR은 작게. 400줄 넘으면 리뷰 품질이 급락합니다
• 설명을 먼저: 무엇을·왜 바꿨는지 PR 본문에 요약
• 방어하지 말고 질문으로 받기: "이 의도였는데, 더 나은 방법이 있을까요?"
• 반영하지 않을 제안은 이유를 남긴다

자동화로 사람의 시간을 아낀다

포맷팅·린트·간단한 규칙은 사람이 아니라 **도구(Prettier·ESLint·CI)**가 잡게 하세요. 리뷰어는 설계·로직·가독성에 집중해야 합니다.

마무리 체크리스트

• 코드를 말하고, 사람을 비난하지 않기
• 코멘트에 중요도 라벨
• 항상 이유 + 대안
• PR은 작게, 설명은 먼저
• 기계가 할 일은 기계에게

좋은 리뷰 문화는 가장 저렴하면서 효과 큰 팀 성장 투자입니다.