name	review-loop
description	JDSnack 코드 리뷰 핸드오프 루프. Claude가 결정론 게이트(빌드/테스트/lint/diff 줄수)와 code-reviewer 서브에이전트로 5점 채점·판정만 하고, 4점 미만이면 구조화된 변경요청을 Codex에 넘긴다. Codex가 수정·푸시하면 Claude가 재리뷰하며, 4점 또는 5회까지 반복한다. Claude는 코드를 직접 수정하지 않는다.

review-loop (Claude = 게이트키퍼)

역할 분담: Claude는 리뷰·판정·PR·merge만, Codex는 코딩·수정·푸시만 담당합니다. 이 스킬은 Claude 쪽 오케스트레이터입니다. Claude는 소스 코드를 직접 수정하지 않습니다. 4점 미만이면 변경요청을 만들어 Codex에 넘기고, 푸시되면 다시 리뷰합니다. 규칙 정본은 .agent-os/이며 여기서 재서술하지 않습니다.

0. 전제

Codex가 에이전트 코딩을 끝내고 작업 브랜치에 커밋·푸시한 상태여야 함.
위험도·PR 흐름은 .agent-os/operations/pr-automation-loop.md를 따름.

1. 결정론 게이트 (LLM 호출 전 — 실패 시 리뷰 토큰 0)

Claude가 검증 목적으로 실행. 하나라도 실패하면 리뷰를 시작하지 않고 2번 형식의 변경요청을 만들어 Codex에 넘김:

diff 줄수 한도 — 변경 1,000줄 초과면 PR 분할을 Codex에 요구하고 중단. git diff --stat origin/main...HEAD 합계(또는 gh pr diff <N> | wc -l) 확인.
backend 변경 시 — cd backend && ./gradlew test
frontend 변경 시 — cd frontend && npm run lint && npm test
셋 다 통과해야 2단계로 진행. (게이트용 재실행은 Claude가 수행, 코드 수정은 하지 않음)

2. 리뷰 핸드오프 루프 (최대 5회)

attempt = 1로 시작. 각 회차:

diff 추출 — 로컬이면 git diff origin/main...HEAD, PR이면 gh pr diff <N>.
code-reviewer 서브에이전트 호출 — 입력은 오직:
- 위 diff
- 대상 spec의 acceptance-criteria.md / test-scenarios.md 경로
- attempt > 1이면 직전 회차 findings 중 미해결분(델타)만
- 전체 레포·.agent-os 전체·대화 전사를 넣지 말 것.
점수 분기:
- score ≥ 4 → 루프 종료(통과). 3단계로.
- score < 4 → Claude는 수정하지 않는다. code-reviewer의 findings를 그대로 Codex용 변경요청으로 정리해 출력:
```
→ Codex 변경요청 (attempt N)
score: N/5
수정 항목(blocker/major 우선):
- 파일:라인 — 무엇을 어떻게 고쳐야 하는가
```
  이후 Codex가 같은 브랜치에서 수정·커밋·푸시한다. 푸시 완료를 확인하면 attempt += 1 후 1번(결정론 게이트 포함)부터 재리뷰. 재리뷰는 델타만 평가.
시도 소진 — attempt == 5인데도 score < 4면 루프 중단. 자동 merge 금지:
- 마지막 점수·미해결 findings를 사용자에게 에스컬레이션.
- pr-automation-loop.md 기준 실패 Issue 생성 여부 판단.

3. 통과 후 (Claude 담당)

PR 생성/갱신은 Claude가 수행. High-risk PR은 scripts/pr-review-gate.sh <N>와 merge-rules.md 머지 조건을 따름.
통과 점수·회차 수를 PR 본문 검증 섹션에 한 줄로 기록.
merge도 Claude가 수행.

경계 규칙 (반드시 준수)

Claude는 소스 코드를 수정/커밋하지 않는다. 수정의 주체는 항상 Codex.
리뷰어에 diff + 합격기준 + 루브릭만 전달. 레포 전체 주입 금지.
회차 간에는 델타(미해결 findings)만 넘기고 누적 전사 금지.
외부 리뷰 봇(Greptile 등)을 같이 쓰면 이 로컬 루프와 둘 중 하나만 매 회 실행.
리뷰어 출력은 code-reviewer의 구조화 형식만 허용. 산문 리뷰 금지.

review-loop (Claude = 게이트키퍼)

0. 전제

Codex가 에이전트 코딩을 끝내고 작업 브랜치에 커밋·푸시한 상태여야 함.

위험도·PR 흐름은 .agent-os/operations/pr-automation-loop.md를 따름.

1. 결정론 게이트 (LLM 호출 전 — 실패 시 리뷰 토큰 0)

Claude가 검증 목적으로 실행. 하나라도 실패하면 리뷰를 시작하지 않고 2번 형식의 변경요청을 만들어 Codex에 넘김:

diff 줄수 한도 — 변경 1,000줄 초과면 PR 분할을 Codex에 요구하고 중단. git diff --stat origin/main...HEAD 합계(또는 gh pr diff <N> | wc -l) 확인.

backend 변경 시 — cd backend && ./gradlew test

frontend 변경 시 — cd frontend && npm run lint && npm test

셋 다 통과해야 2단계로 진행. (게이트용 재실행은 Claude가 수행, 코드 수정은 하지 않음)

2. 리뷰 핸드오프 루프 (최대 5회)

attempt = 1로 시작. 각 회차:

diff 추출 — 로컬이면 git diff origin/main...HEAD, PR이면 gh pr diff <N>.

code-reviewer 서브에이전트 호출 — 입력은 오직:

위 diff
대상 spec의 acceptance-criteria.md / test-scenarios.md 경로
attempt > 1이면 직전 회차 findings 중 미해결분(델타)만
전체 레포·.agent-os 전체·대화 전사를 넣지 말 것.

점수 분기:

score ≥ 4 → 루프 종료(통과). 3단계로.
score < 4 → Claude는 수정하지 않는다. code-reviewer의 findings를 그대로 Codex용 변경요청으로 정리해 출력:
```
→ Codex 변경요청 (attempt N)
score: N/5
수정 항목(blocker/major 우선):
- 파일:라인 — 무엇을 어떻게 고쳐야 하는가
```
이후 Codex가 같은 브랜치에서 수정·커밋·푸시한다. 푸시 완료를 확인하면 attempt += 1 후 1번(결정론 게이트 포함)부터 재리뷰. 재리뷰는 델타만 평가.

시도 소진 — attempt == 5인데도 score < 4면 루프 중단. 자동 merge 금지:

마지막 점수·미해결 findings를 사용자에게 에스컬레이션.
pr-automation-loop.md 기준 실패 Issue 생성 여부 판단.

3. 통과 후 (Claude 담당)

PR 생성/갱신은 Claude가 수행. High-risk PR은 scripts/pr-review-gate.sh <N>와 merge-rules.md 머지 조건을 따름.

통과 점수·회차 수를 PR 본문 검증 섹션에 한 줄로 기록.

merge도 Claude가 수행.

경계 규칙 (반드시 준수)

Claude는 소스 코드를 수정/커밋하지 않는다. 수정의 주체는 항상 Codex.

리뷰어에 diff + 합격기준 + 루브릭만 전달. 레포 전체 주입 금지.

회차 간에는 델타(미해결 findings)만 넘기고 누적 전사 금지.

외부 리뷰 봇(Greptile 등)을 같이 쓰면 이 로컬 루프와 둘 중 하나만 매 회 실행.

리뷰어 출력은 code-reviewer의 구조화 형식만 허용. 산문 리뷰 금지.