8. 코드 리뷰

시작 상태

git log --oneline -5 && git status --short
git log -1 --oneline tutorial-gates

7단계 UI 통합 커밋(feat: 공백 옵션과 남은 글자 표시 통합)이 보이고 git status --short는 아무것도 찍지 않아야 합니다. 통합 커밋이 없거나 트리가 깨끗하지 않으면 7. 병렬 작업으로 돌아갑니다. 세 번째 명령은 3단계에서 남긴 chore: 게이트 설정 커밋을 찍어야 합니다. 아니라면 3. 게이트 먼저로 돌아가 태그 위치를 확인합니다.

이 단계에서 하는 것

서브에이전트에 리뷰를 맡겨 결론만 받습니다. 지적마다 테스트나 코드로 직접 검증한 뒤 수용 또는 기각 사유를 기록으로 남깁니다. 같은 모델의 새 컨텍스트가 같은 곳을 놓치는 이유와 교차 벤더 리뷰가 다른 이유는 Code Review가 다룹니다.

따라하기

리뷰 범위를 먼저 고정합니다. 두 리뷰가 끝날 때까지 코드를 수정하거나 커밋하지 않습니다.

git rev-parse tutorial-gates
git rev-parse HEAD
git diff --stat tutorial-gates HEAD

기준 SHA와 대상 SHA를 적어 둡니다. 리뷰를 서브에이전트에 위임합니다(4번 틀):

`git diff tutorial-gates HEAD`의 변경 전체를 리뷰 서브에이전트에 맡겨 주세요.
기준 SHA와 현재 HEAD SHA를 결과에 포함하세요.
파일을 읽거나 grep해도 되는데, 덤프하지 말고 결론만 정리해서 알려 주세요.

확인할 것:
- countChars 가 스펙의 "글자" 정의를 모든 입력에서 지키는가
- 공백 제외 옵션과 140자 표시가 서로 간섭하지 않는가
- 테스트 이름이 실제로 잠근 것을 정직하게 말하는가

지적은 심각도 순으로 각각 한 문장과 파일:줄만. 수정은 하지 마세요.

선택 경로입니다. 준비: 공식 플러그인과 스킬에서 Codex CLI를 설치해 두었다면, 수정 전 같은 HEAD에서 한 번 더 돌립니다.

codex exec review --base tutorial-gates < /dev/null

codex exec review --help로 이 형태와 --base 지원 여부를 먼저 확인합니다. 지원하지 않으면 이 경로를 생략합니다. 태그 SHA와 HEAD SHA가 1번에서 적어 둔 값과 같은지 확인한 뒤 두 결과를 비교합니다.

각 지적을 테스트나 코드 근거로 검증합니다. 지적이 없으면 없다고 기록합니다. 모두 타당하면 모두 수용하고, 문제가 아닌 지적만 근거와 함께 기각합니다. 수용과 기각 개수를 맞추려고 결론을 지어내지 않습니다.
docs/review.md를 다음 구조로 작성합니다.

# 리뷰 기록

## 비교 범위
- 기준 SHA:
- 대상 SHA:
- 리뷰 방식: 서브에이전트 / 선택 Codex

## 결과
지적이 없으면 "지적 없음"과 확인한 범위를 적는다.

| 지적 | 판정(수용/기각) | 테스트 또는 코드 근거 | 후속 조치 |
|---|---|---|---|

SHA와 결과는 실제 값으로 채웁니다. 수용한 지적만 수정하고 네 게이트를 실행합니다.

npm run lint && npm test && npm run build && npm run e2e

커밋 직전에 끝 비용과 컨텍스트 사용률을 읽어 docs/tutorial-log.md의 “8. 리뷰” 행에 적습니다. 수정이 없으면 리뷰 기록과 기록표만 커밋합니다.

git add docs/review.md docs/tutorial-log.md
git commit -m "docs: 8단계 리뷰 기록"

수정이 있으면 수정한 파일과 두 기록을 함께 커밋합니다.

git add src docs/review.md docs/tutorial-log.md
git commit -m "fix: 리뷰 수용 사항 반영"

커밋 메시지에 기각 문구를 강제로 넣지 않습니다.

완료 확인

test -s docs/review.md
git diff tutorial-gates HEAD --stat
npm run lint && npm test && npm run build && npm run e2e; echo "green exit=$?"

docs/review.md에 기준·대상 SHA와 각 판단의 근거가 있고, green exit=0이어야 합니다. 지적 없음·전부 수용·일부 기각 모두 유효한 완료 상태입니다.

계측 기록

값은 4번에서 이미 적었습니다. 시작 상태를 확인한 직후 시작 비용을, 커밋 직전에 끝 비용과 컨텍스트 사용률을 각각 적고 차액으로 읽습니다. 세션 수는 이 단계에서 연 세션 개수입니다. 서브에이전트 리뷰가 돌아온 직후 /context에서 본 세션의 사용률이 얼마나 움직였는지 보십시오.

어긋났을 때

리뷰가 파일 덤프로 돌아오면 “결론만”을 다시 요구하거나 파일로 쓰게 합니다. 위임 방식은 Subagents에 있습니다.
제안 패치를 그대로 적용했다가 게이트가 깨지면 Claude Code를 못 쓰게 만드는 10가지 습관 7번을 확인합니다.