jdsnack-agent-os-project: Skill for Claude Code

review-loop (설정된 reviewer = 게이트키퍼)

역할 분담: Codex는 구현·테스트와 현재 기본 PR 리뷰 및 Spec 기획·문서 계획, Claude는 명시적으로 선택된 리뷰 backend 또는 사용자 승인 폴백을 담당합니다. reviewer 선택은 scripts/review-policy.json의 primaryReviewer를 따릅니다. 현재 primary reviewer는 Codex이므로 Claude 호출 없이 바로 읽기 전용 리뷰를 실행합니다. primary가 Claude이고 Claude가 unavailable이거나 구조화 결과를 반환하지 못하면 Codex read-only reviewer로 전환합니다. 유효한 Claude 판정은 Codex 결과로 덮어쓰지 않습니다. 어느 reviewer든 소스 코드를 수정하지 않습니다. 규칙 정본은 .agent-os/이며 여기서 재서술하지 않습니다.

0. 전제

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

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

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

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

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

attempt = 1로 시작. 각 회차:

1. diff 추출 — 로컬이면 git diff origin/main...HEAD, PR이면 gh pr diff <N>.
2. code-reviewer 서브에이전트 호출 — 입력은 오직:
   • 위 diff
   • 대상 spec의 acceptance-criteria.md / test-scenarios.md 경로
   • attempt > 1이면 직전 회차 findings 중 미해결분(델타)만
   • 전체 레포·.agent-os 전체·대화 전사를 넣지 말 것.
3. 점수 분기:
   • score ≥ 4 → 루프 종료(통과). 3단계로.
   • score < 4 → Claude는 수정하지 않는다. code-reviewer의 findings를 그대로 Codex용 변경요청으로 정리해 출력:

     → Codex 변경요청 (attempt N)
     score: N/5
     수정 항목(blocker/major 우선):
     - 파일:라인 — 무엇을 어떻게 고쳐야 하는가
     

     이후 Codex가 같은 브랜치에서 수정·커밋·푸시한다. 푸시 완료를 확인하면 attempt += 1 후 1번(결정론 게이트 포함)부터 재리뷰. 재리뷰는 델타만 평가.
4. 시도 소진 — attempt == 3인데도 score < 4면 루프 중단. 자동 merge 금지:
   • 마지막 점수·미해결 findings를 사용자에게 에스컬레이션.
   • pr-automation-loop.md 기준 실패 Issue 생성 여부 판단.

3. 리뷰 결과 제출과 승인 job

리뷰 job은 구조화된 판정을 만들고, 승인 여부를 정해진 게이트에 맡깁니다. review job 자체에서는 PASS를 승인하거나 머지를 요청하지 않습니다.