Diff Review는 어떤 순서로 할까
먼저 git diff --stat 수준에서 변경 file이 envelope 안에 있는지 보고, lockfile, binary, generated file이 예상 없이 생겼는지 확인합니다. 다음으로 line diff에서 public API, error handling, logging의 secret 노출과 broad formatting churn을 봅니다. 마지막으로 test를 깨끗한 상태에서 재실행하고 새 test가 bug를 실제로 재현하는지 확인합니다.
| 검증 | 실패하면 무엇을 의미하나 |
|---|
| 기존 test | regression 또는 environment mismatch |
| 새 regression test | fix가 요구 사례를 다루지 못함 |
| Type, lint, build | local unit test가 놓친 integration 오류 |
| Diff scope | agent가 task를 넘어선 변경 수행 |
| Clean checkout 재현 | 숨은 local state나 untracked file 의존 |
Agent가 말한 “완료”는 제안 상태로 봅니다. Test command, exit code와 미실행 검증을 결과 보고에 포함시키고, migration, security, auth 변경은 독립 reviewer가 봅니다. 실패하면 whole repository를 다시 맡기기보다 가장 작은 failing evidence와 함께 다음 수정을 요청합니다.
도입 효과도 speed만 보지 않습니다. Task당 accepted diff 비율, human review time, 재작업 횟수, regression과 불필요한 file 변경을 기록합니다. 첫 초안이 빨라도 review와 rollback이 더 길다면 automation 이득은 없습니다.