에이전트 PR을 사람이 전부 리뷰하지 않아도 되는 기준 — 맷 포콕의 자동 체크·자동 리뷰·사람 리뷰 3겹

한 줄 핵심

에이전트가 만든 PR의 병목을 풀려면 자동 체크·자동 리뷰·사람 리뷰를 3겹으로 쌓고, 구현 에이전트와 리뷰 에이전트의 역할을 분리해 인간 리뷰는 정말 중요한 변경에만 쓰도록 설계해야 한다.

시간별 요약

1. 0:00
   서론 — 에이전트 PR이 쌓이면 리뷰가 막힌다 에이전트가 코드를 많이 밀어 넣을수록 PR은 늘어나고, 사람 리뷰가 따라가지 못하면 병목이 생긴다. 결국 리뷰가 막히는 문제를 풀지 않으면 자동화가 오히려 팀의 속도를 떨어뜨린다.
2. 1:10
   슬롭 캐논과 3겹의 브레이크 자동화는 코드를 더 빨리 흘려보내지만, 그대로 두면 품질 낮은 PR이 쏟아지는 '슬롭 캐논'이 된다. 이를 막기 위해 자동 체크, 자동 리뷰, 사람 리뷰의 3단계 브레이크를 두고, 앞단에서 품질을 끌어올려 사람 리뷰 부담을 줄여야 한다.
3. 3:18
   첫 번째 겹 — 자동 체크는 싸지만 거짓말을 한다 테스트·린트·타입체크 같은 자동 체크는 CPU만 쓰는 저비용 장치라 많이 쌓아도 되지만, 초록색 CI가 곧바로 안전함을 보장하지는 않는다. 따라서 자동 체크는 진실을 보장하는 수단이 아니라, 거짓을 걸러내는 첫 필터로 봐야 한다.
4. 4:41
   에이전트가 쓴 테스트의 거짓말 세 가지 에이전트가 만든 테스트는 구현을 그대로 다시 적는 자기확인식 테스트, 코드 구조에 과하게 의존하는 구조 민감 테스트, 실제 실패 경로를 막아버리는 모킹 과잉 테스트로 쉽게 망가진다. 이런 테스트는 통과해도 제품 품질을 보장하지 못하므로, 테스트가 무엇을 증명하는지보다 무엇을 놓치는지를 먼저 봐야 한다.
5. 8:13
   딥 모듈 — 속이기 어려운 코드베이스 설계 자동 체크를 속이기 어렵게 만들려면 코드베이스 자체를 깊은 모듈 중심으로 설계해야 한다. 복잡한 구현을 작은 인터페이스 뒤에 숨기면 구조 민감 테스트가 줄고, 에이전트도 구현 디테일 대신 안정적인 경계에서 검증하게 된다.
6. 11:43
   코딩 표준은 리뷰 에이전트에게 구현 에이전트는 탐색·수정·디버깅만으로도 이미 과부하 상태이므로, 코딩 표준까지 한 번에 맡기면 성능이 떨어진다. 표준 검사는 별도의 리뷰 에이전트가 전용 컨텍스트에서 수행하게 하고, 기준 문서는 global scope가 아니라 리뷰 전용 파일에 둬야 효과적이다.
7. 15:19
   리뷰어는 코멘트 대신 커밋한다 범용 리뷰 서비스에 의존하면 오탐이 많거나, 특정 언어에만 맞아 실무에 못 쓰는 경우가 생긴다. 자동 리뷰 에이전트는 PR에 장황한 코멘트를 남기기보다 직접 커밋으로 수정해, 사람이 읽을 때는 이미 정리된 산출물을 보게 해야 한다.
8. 17:35
   한쪽 문, 양쪽 문 — PR 스킬 모든 PR이 같은 무게는 아니며, 되돌릴 수 있는 2-way door와 되돌리기 어려운 1-way door를 구분해야 한다. 특히 대규모 이메일 발송, 데이터 손실, 비싼 마이그레이션처럼 영향 반경이 큰 변경은 더 강하게 검토하고, PR 본문에 위험도와 blast radius를 요약해 두는 것이 중요하다.
9. 20:23
   PR이 무엇을 하는지 보여 주기 사람 리뷰의 핵심은 코드보다 '무엇이 왜 바뀌는지'를 빠르게 이해시키는 것이다. 이를 위해 pseudo code, 다이어그램, 간단한 시각 요약을 활용하면 변경 의도와 흐름을 훨씬 빨리 파악할 수 있다.
10. 21:42
    같은 지적을 두 번 하지 않기 — retro 스킬 사람 리뷰는 코드만 보는 것이 아니라 코드를 만드는 시스템 전체를 개선해야 한다. retro 스킬로 세션과 PR을 되짚어 보면 반복되는 실수를 자동 체크, 코드 표준, 네비게이션 포인터, 도구 사용, 파일 비대화까지 연결해 다음 리뷰를 더 가볍게 만들 수 있다.
11. 24:41
    마무리 영상은 자동 체크, 자동 리뷰, 사람 리뷰를 층층이 쌓아 인간이 꼭 봐야 할 PR만 남기는 방향으로 요약된다. 핵심은 속도를 올리되 품질을 같이 올려, 리뷰 병목을 줄이면서도 코드베이스의 엔트로피를 막는 것이다.

실무 인사이트

• 구현 에이전트와 리뷰 에이전트를 분리하면, 같은 규칙이라도 컨텍스트 과부하를 줄여 품질이 더 안정적으로 나온다.
• PR 리뷰 효율은 코드 자체보다 PR 본문 설계에 크게 좌우되므로, 위험도·되돌림 가능성·blast radius를 미리 적어 두는 습관이 효과적이다.
• 자동화의 성패는 도구 수보다 코드베이스 구조와 표준의 축적 방식에 달려 있으므로, 반복 지적을 retro로 회수해 다음 세션의 규칙으로 반영해야 한다.