8장: 코드 리뷰의 심리전

코드 리뷰는 기술적인 과정인 동시에, 사람과 사람 사이의 소통입니다. 리뷰어와 리뷰이 사이에는 눈에 보이지 않는 심리적 역학이 작동합니다.

소프트웨어 공학의 주요 국제 학회인 ESEC/FSE에서 발표된 연구는, fMRI(기능적 MRI)와 아이트래킹을 활용해 코드 리뷰 중의 뇌 활동을 분석했습니다. 그 결과, 리뷰어가 기술적 판단뿐 아니라 감정적 처리도 동시에 하고 있음이 확인되었습니다.

이 장에서는 코드 리뷰에 숨어 있는 심리 편향, 그리고 그것을 활용하거나 방어하는 방법을 풀어냅니다.

후광 효과

후광 효과란, 어떤 특징의 인상이 그것과 무관한 다른 특징의 평가에까지 영향을 미치는 현상입니다.

코드 리뷰에서 후광 효과가 작동하는 전형적인 장면이 있습니다. 시니어 엔지니어가 쓴 코드와 신입이 쓴 코드는, 내용이 같아도 리뷰의 엄격함이 달라집니다. 시니어의 코드는 "분명 무슨 생각이 있겠지"라며 호의적으로 해석되고, 신입의 코드는 "기본이 안 됐다"며 엄하게 평가됩니다.

반대 패턴도 있습니다. 과거에 큰 버그를 낸 엔지니어의 코드는 그 뒤로도 주의 깊게 살펴봅니다. 기술력이 올라갔어도, 과거의 인상이 리뷰 기준에 계속 영향을 미칩니다.

후광 효과를 키우는 요인

GitHub상의 가시적 지표: 커밋 수, PR 빈도, OSS 컨트리뷰션. 이런 것이 많은 사람의 코드는 '능력 있는 사람' 필터를 거쳐 평가됩니다.

소속 팀·기업: 빅테크 기업 출신, 유명 OSS의 커미터 같은 직함이 코드 평가에 영향을 줍니다.

소통 능력: PR 설명이 꼼꼼한 사람의 코드는, 코드 자체의 품질도 높으리라 짐작되기 쉽습니다.

리뷰어로서의 대바카라

코드만 보는 시간을 만든다. PR의 작성자 이름을 의식적으로 무시하고, 코드 자체를 읽는 시간을 맨 먼저 가집니다. '누가 썼는가'를 확인하는 일은, 코드 평가가 굳어진 뒤에 해도 충분합니다.

체크리스트를 쓴다. 보안, 성능, 가독성, 테스트 커버리지처럼 평가 항목을 고정하면, 후광 효과로 인한 주관적인 치우침을 억누를 수 있습니다.

권위 편향과 코드 리뷰

권위 편향은 제3장에서도 다뤘지만, 코드 리뷰라는 맥락에서는 특히 중요합니다.

테크 리드나 아키텍트처럼 팀 안에서 권위 있는 인물의 코드에 대해, 다른 멤버가 Approve를 내기 쉬운 경향이 있습니다. 기술적으로 문제가 없어서 Approve하는 것이 아니라, "내가 틀린 건지도 몰라", "지적하면 어색해진다"는 심리가 작동하는 것입니다.