# [[Code Review Foundations (코드 리뷰 기초)|Code Review Foundations (코드 리뷰 기초]]

## 📌 Brief Summary
코드 리뷰(Code Review)는 한 명 이상의 개발자가 다른 개발자가 작성한 소스 코드를 검토하여 버그를 찾고, 품질을 높이며, 지식을 공유하는 협업 프로세스입니다 [1]. 이는 소프트웨어 개발 생명주기(SDLC)에서 결함을 조기에 발견하여 수정 비용을 절감하고 시스템의 아키텍처적 일관성을 유지하는 핵심 방어선 역할을 합니다 [2, 3]. 리뷰 방식은 실시간으로 진행되는 '동기식(Synchronous)'과 PR/MR 도구를 활용하는 '비동기식(Asynchronous)'으로 나뉘며, 조직의 규모와 목적에 따라 상호 보완적으로 활용됩니다 [4, 5].

## 📖 Core Content
*   **코드 리뷰의 주요 목적:**
    *   **품질 보증 및 버그 발견:** 기능적 오류, 엣지 케이스 누락, 성능 병목, 보안 취약점 등을 배포 전 조기에 식별함 [1, 2].
    *   **지식 공유 및 멘토링:** 코드베이스에 대한 팀의 공동 소유권을 강화하고, 시니어의 사고방식을 주니어에게 전수하며 기술적 부채를 방지함 [2, 4].
    *   **일관성 유지:** 팀의 코딩 컨벤션, 설계 원칙, 아키텍처 가이드라인을 준수하도록 강제함 [3].
*   **비동기식 코드 리뷰 (Asynchronous Review):**
    *   **정의:** Pull Request(PR) 또는 Merge Request(MR) 도구를 통해 리뷰어가 편한 시간에 서면으로 피드백을 남기는 방식 [4].
    *   **장점:** 개발자의 몰입(Focus Time)을 방해하지 않으며, 전 세계 어디서든 시간대와 관계없이 협업이 가능하고 논의 과정이 자동으로 기록됨 [4, 10].
    *   **단점:** 피드백 루프가 길어질 수 있고(Ping-pong), 텍스트 기반 소통으로 인해 의도가 오해받을 위험이 있음 [8].
*   **동기식 코드 리뷰 (Synchronous Review):**
    *   **정의:** 페어 프로그래밍(Pair Programming), 몹 프로그래밍(Mob Programming), 대면 워크스루 등을 통해 실시간으로 코드를 검토함 [1, 3].
    *   **장점:** 즉각적인 피드백과 심층적인 논의가 가능하여 복잡한 설계 문제나 긴급한 핫픽스 처리에 탁월함 [4, 6].
    *   **단점:** 참여자 간의 일정 조율 오버헤드가 크고, 기록을 별도로 남기지 않으면 추적성(Traceability)을 잃기 쉬움 [1, 10].
*   **베스트 프랙티스:**
    *   **시간 제한(Time-boxing):** 인지 부하를 줄이기 위해 리뷰 세션은 60~90분 이내로 제한하고, 작은 단위(예: 400 LOC 이하)로 나누어 리뷰함 [6].
    *   **문서화:** 동기식으로 합의된 내용이라도 미래의 자신과 동료를 위해 결정 사항을 PR 코멘트나 ADR로 기록해야 함 [10].

## ⚖️ Trade-offs & Caveats
*   **속도 vs 철저함:** 빠른 배포를 위해 리뷰를 생략하거나 피상적으로 훑으면 보안 결함과 기술 부채가 쌓이며, 반대로 사소한 스타일 지적(Nit-picking)에 집착하면 배포 병목이 발생함 [6, 8].
*   **심리적 안전감:** 리뷰 프로세스는 비판이 아닌 개선을 위한 협업이어야 함. 공격적인 어조나 '에고(Ego)'가 개입될 경우 개발자의 동기부여를 저해하고 팀 결속력을 해칠 수 있음 [9].
*   **분산 팀의 제약:** 시차가 큰 글로벌 팀에서 동기식 리뷰를 강제하면 특정 인원이 소외될 수 있으므로, 비동기 방식을 기본으로 하되 필요 시에만 짧은 동기 세션을 결합하는 하이브리드 전략이 필요함.

## 🔗 Knowledge Connections

### Related Concepts
*   **Pair Programming**: 코드를 작성하면서 실시간으로 리뷰가 완료되는 가장 강력한 동기식 협업 기법입니다.
*   **Pull Request (PR) Workflow**: 비동기식 리뷰가 이루어지는 현대적인 표준 개발 워크플로우입니다.
*   **[[DORA-Metrics|DORA Metrics]]**: 리뷰 속도와 효율성이 팀의 소프트웨어 전달 성과에 미치는 영향을 측정하는 지표 체계입니다.
*   **Shift-Left Security**: 보안 검토를 코드 리뷰 단계로 앞당겨 수정 비용을 절감하려는 전략적 접근입니다.

### Deeper Research Questions
*   비동기 리뷰 중 댓글 대화가 몇 회 이상 지속될 때 동기식 회의로 전환하는 것이 팀의 생산성 ROI 측면에서 가장 유리한가?
*   주니어 개발자의 온보딩 속도를 극대화하기 위해 동기식과 비동기식 리뷰를 어떤 비율로 배분하는 것이 가장 효과적인가?
*   코드 리뷰에서 결정된 주요 설계 변경 사항을 자동으로 문서화(Architecture Decision Records)로 변환해주는 도구 체계는 어떻게 구축하는가?
*   리뷰어의 피로(Review Fatigue)를 정량화하고 이를 완화하기 위한 지능형 리뷰어 할당(Reviewer Assignment) 알고리즘은 무엇인가?
*   문화적 차이나 언어 장벽이 있는 글로벌 팀에서 텍스트 기반 비동기 리뷰의 오해를 줄이기 위한 커뮤니케이션 프로토콜은 어떻게 설계해야 하는가?

### Practical Application Contexts
*   **Implementation:** 일상적인 변경은 PR 기반의 비동기 리뷰로 진행하고, 핵심 아키텍처 변경이나 난해한 버그 수정 시에는 15~30분의 짧은 동기식 워크스루를 병행합니다.
*   **System Design:** 새로운 서비스 설계안을 확정하기 전, 관계자 전원이 참여하는 몹 리뷰(Mob Review)를 통해 설계의 사각지대를 실시간으로 제거합니다.
*   **Operation / Maintenance:** 운영 장애 복구 과정에서 작성된 긴급 패치 코드는 반드시 전문가와 실시간 동기 리뷰를 거쳐 2차 장애를 방지합니다.
*   **Learning Path:** 신입 사원의 첫 커밋부터 일정 기간은 시니어와 페어 프로그래밍을 진행하여 팀의 기술 부채 방지 원칙과 코딩 철학을 전수합니다.
*   **My Project Relevance:** 스타일 지적 등 기계적인 검증은 자동화 도구(CI/CD)에 맡기고, 리뷰어는 비즈니스 로직과 설계의 적합성 검증이라는 본연의 가치에 집중합니다.

### Adjacent Topics
*   **Code Review Communication Etiquette**: 효율적인 피드백 전달을 위한 어조, 기술, 감정 관리 등 소프트 스킬 영역으로 확장됩니다.
*   **Egoless Programming**: 개인의 자존심을 버리고 팀의 코드를 최우선으로 생각하는 개발 철학입니다.

---
*Last updated: 2026-05-02*
