Tag: Code_Review
All the articles with the tag "Code_Review".
-
설계까지 다 거치고도, 실행해봐야 보였다 5편 — 되돌리기 버튼 하나에, 버그가 세 겹으로 숨어 있었다
이미 실행된 태그 병합을 되돌리는 버튼을 만들었다. 내가 직접 브리프에 적어준 구현 방법에 버그가 있다는 걸 태스크 리뷰가 잡았고, 그 여파로 계획 문서가 낡아버린 걸 다음 단계를 준비하다가 스스로 알아챘다. 그리고 두 태스크가 전부 리뷰를 통과한 뒤에도, 전체를 다시 훑는 마지막 리뷰가 버그 두 개를 더 찾아냈다.
-
파일쓰기 재설계 이후 4편 — 같은 함수를 리뷰가 세 번 다시 봐야 했다
링크를 고쳐주는 함수(`repairInboundLinks`) 하나를 놓고, 리뷰가 세 번 서로 다른 문제를 잡아냈다. 한 번은 계획에 없던 조건이 맞는지, 한 번은 커밋 메시지의 설명이 사실과 맞는지, 한 번은 고친 코드가 고치려던 문제를 다른 경로로 다시 만들어내고 있는지.
-
파일쓰기 로직 재설계하다가 2편 — 이미 검증된 것도, 합쳐놓은 것도 다시 봐야 했다
서브에이전트가 정확히 고친 코드를 '더 간단하게' 정리하겠다고 손댔다가 같은 버그를 조용히 되살릴 뻔했다. 그리고 두 태스크가 각각 완벽하게 끝난 다음에도, 그 둘을 합쳐서 보는 최종 리뷰에서 진짜 버그가 나오기도 하고 '구조적으로 안전하다'는 게 증명되기도 했다.
-
파일쓰기 로직 재설계하다가 1편 — 옮겨 적을 때마다 뭔가 하나씩 틀렸다
조사 자체는 완벽했는데, 그걸 구두로 요약할 때마다 뭔가 하나씩 빠졌다. 네 번 연속으로. 그리고 첫 하위 프로젝트에서는, 구현이 계획 문서의 코드를 한 글자도 안 틀리고 그대로 옮겼는데도 리뷰에서 결함 3개(로그 중복, 따옴표 유실, 위키링크 퍼지매칭 오탐)가 나왔다 — 원인은 구현이 아니라 계획 문서 자체였다.
-
AstroPaper 수정하다가 2편 — 시리즈 페이지 검증은 다 통과했는데 최종 리뷰가 버그를 잡았다
태스크마다 타입체크와 리뷰를 통과했고, 실제 dev 서버로 열어보고 프로덕션 빌드까지 돌리는 마지막 검증까지 다 통과했다. 그런데 전체 브랜치를 놓고 하는 최종 리뷰에서 진짜 버그(breadcrumb 접기 로직이 series 페이지를 안 챙김)가 하나 나왔다 — 그 검증 자체가 perPage 설정을 잘못 가정하고 있었다.