review-loop

리뷰에서 발견한 것 중 확실한 것들만 자동 수정하고, 다시 리뷰를 돌립니다. 판단이 필요한 항목은 사람에게 넘깁니다.

자동 수정 가능 목록 (auto-fixable)

아래 항목만 자동 수정합니다. 나머지는 전부 사람에게 넘깁니다.

자동 수정 하지 않는 것:

• 로직, 알고리즘 변경
• 필러 문구 (Note that, This ensures that 등) — 문장 구조 판단 필요
• Javadoc 축약 — WHY 존재 여부 판단 필요
• 변수명, 함수명 변경

Phase 1 — 대상 파일 수집

git diff --name-only $(git merge-base HEAD main)...HEAD

변경된 파일 목록을 수집합니다. 리뷰 대상은 이 파일들로 한정합니다.

Phase 2 — 초기 리뷰 실행

review 스킬을 실행해 전체 findings를 수집합니다.

findings를 두 버킷으로 분류합니다:

• auto-fixable: 위 목록에 해당하는 항목 (severity: fix)
• needs-human: 나머지 전부 (severity: consider, 로직 관련, 판단 필요)

auto-fixable이 0개면 Phase 5로 바로 이동합니다.

Phase 3 — 자동 수정 실행

auto-fixable 항목을 파일별로 묶어 순서대로 수정합니다.

수정 원칙:

• 한 파일씩 처리합니다. 여러 패턴이 같은 파일에 있으면 한 번에 수정합니다.
• 수정 전 원본 내용을 메모리에 보관합니다 (롤백 대비).
• 각 수정 후 [수정] <파일>:<줄> — <원문> → <수정안> 형식으로 로그를 남깁니다.

Phase 4 — 재검토 루프

수정 완료 후 Phase 2로 돌아갑니다.

종료 조건:

• auto-fixable findings가 0개 → 루프 종료
• 반복 횟수가 5회 도달 → 루프 강제 종료, 남은 항목 리포트

5회 제한은 무한루프 방지를 위한 안전장치입니다. 정상적인 코드베이스라면 1-2회 안에 클린 상태가 됩니다.

Phase 5 — 최종 리포트

## review-loop 결과

반복 횟수: N회
자동 수정 항목: M개

### 수정 내역
[수정] <파일>:<줄> — <원문> → <수정안>
...

### 사람이 확인해야 할 항목 (자동 수정 안 함)
[<severity>] <category>: <요약>
  Where: <파일>:<줄>
  Why: <이유>
  Fix 제안: <제안>
...

### 결과
클린 (auto-fixable 0개) | 5회 초과로 중단 — N개 잔여

needs-human 항목이 있으면 목록을 그대로 출력합니다. 없으면 "자동 수정 완료 — 잔여 이슈 없음" 을 출력합니다.

Non-goals

• 로직, 비즈니스 코드를 변경하지 않습니다.
• 테스트를 실행하지 않습니다.
• 커밋은 하지 않습니다. 수정 후 사용자가 직접 커밋합니다.
• review-loop 자체가 PR을 열거나 닫지 않습니다.