Git bisect로 회귀가 들어온 범위 좁히기, 리뷰에서 함께 확인할 질문
과거에는 되던 동작이 언제 깨졌는지 모를 때 모든 커밋을 순서대로 읽는 것은 비효율적이다. 명확한 재현 명령이 있으면 변경 범위를 이분 탐색할 수 있다. 코드 밖의 맥락 전달하기를 중심으로 작은 예제와 확인 기준을 정리합니다.
과거에는 되던 동작이 언제 깨졌는지 모를 때 모든 커밋을 순서대로 읽는 것은 비효율적이다. 명확한 재현 명령이 있으면 변경 범위를 이분 탐색할 수 있다.
작성자에게 당연한 전제도 리뷰어에게는 보이지 않을 수 있다. 파일의 차이만 보여 주면 변경 이유와 확인 범위를 추측하게 된다. 이번 기록에서는 결과를 판단하는 데 필요한 맥락을 짧게 전달하고, 질문이 실제 개선으로 이어지도록 확인 순서를 정리한다.
코드 밖의 맥락 전달하기
좋은 기준 커밋과 나쁜 기준 커밋을 지정하고 동일한 재현 조건으로 중간 커밋을 판정한다. 작업 전 변경을 보존하고 별도 작업 사본을 사용한다.
이번에 확인할 질문: 리뷰어가 변경 이유와 남아 있는 제약을 코드만 추측하지 않고 이해할 수 있는가?
작게 확인하는 예제
다음 예제는 핵심 조건을 드러내기 위한 출발점이다. 서비스의 전체 구현을 대신하지 않으므로 실제 입력 범위와 실패 처리 규칙을 함께 읽어야 한다.
git bisect start
git bisect bad <failing-revision>
git bisect good <working-revision>
git bisect run ./scripts/reproduce.sh
git bisect reset
찾은 커밋의 직전과 직후에서 같은 재현 명령을 다시 실행해 차이가 일관적인지 확인한다.
놓치기 쉬운 조건
재현 스크립트는 성공과 실패의 종료 코드를 정확히 구분해야 한다. 환경 변화나 간헐적 실패가 원인이면 커밋 탐색 결과를 곧바로 확정할 수 없다.
| 확인할 부분 | 살펴볼 내용 |
|---|---|
| 기준 동작 | 찾은 커밋의 직전과 직후에서 같은 재현 명령을 다시 실행해 차이가 일관적인지 확인한다. |
| 비교 조건 | 입력과 실행 순서를 고정하고 한 번에 한 조건만 바꾼다. |
| 적용 범위 | 예제가 생략한 의존성과 실제 서비스의 제약을 구분한다. |
다음 변경에 적용할 순서
- 이번 변경이 해결하는 구체적인 상황을 먼저 설명한다.
- 재현 예제와 확인한 결과를 한곳에서 볼 수 있게 남긴다.
- 취향에 대한 제안과 지켜야 할 계약의 위반을 구분한다.
리뷰는 작성자의 지식을 시험하는 자리가 아니라 변경을 함께 책임지는 과정이다. 답변이 반복되는 질문은 설명이 부족한 신호일 수 있다. 대화가 끝난 뒤 가장 중요한 전제를 코드 가까이 남기면 다음 리뷰의 질문도 더 구체적이 된다.
이 이야기가 도움이 되었나요?