simon-code-review

simon-code-review

PR 기반 코드 리뷰 스킬. Draft PR을 생성하고, 논리적 변경 단위별 인라인 코드 리뷰를 작성하며, CI 모니터링과 사용자 피드백 루프를 반복한다.

Mode Detection

스킬 시작 시 모드를 자동 감지한다:

1. .claude/memory/review-sequence.md 존재 확인
2. 존재 → CONNECTED 모드 (simon/grind에서 넘어온 것)
   • .claude/memory/branch-name.md 읽기
   • .claude/reports/{feature-name}-report.md 확인 → PR description에 활용
   • .claude/memory/plan-summary.md 확인 → 계획 매핑용
   • .claude/memory/pr-info.md 확인 → 이미 PR이 있으면 재사용
3. 미존재 → STANDALONE 모드
   • git diff {base-branch}...HEAD로 변경사항 분석
   • Agent team으로 review-sequence.md 자체 생성
   • Report도 자체 생성
4. 이후 워크플로는 양 모드 동일

Shared Protocols: ~/.claude/skills/_shared/preamble.md 읽기 — Session Isolation, Error Resilience, Forbidden Rules, Agent Teams, Cognitive Independence 공통 프로토콜 포함.

Session Isolation Protocol (확장)

SESSION_DIR 결정: CONNECTED 모드에서는 simon-dev이 전달한 {SESSION_DIR}을 사용한다. STANDALONE 모드에서는 현재 브랜치명으로 직접 결정한다 (preamble의 SESSION_DIR 결정 bash 참조).

Workflow

[Mode Detection]
     ↓
[STANDALONE only] → Step 0: Diff 분석 + review-sequence 생성
     ↓
Step 1: Draft PR 생성
     ↓
Step 2: 인라인 코드 리뷰 작성
     ↓
Step 3: CI Watch (background) + 사용자에게 리뷰 안내
     ↓
Step 4: 피드백 루프 ──────────────────────┐
  4-A: 사용자 피드백 수집 (PR 코멘트)      │
  4-B: [GATE] 전문가 검증 → 코드 수정      │
  4-C: 수정된 부분에 인라인 리뷰 재작성     │
  4-D: [GATE] CI Watch 재시작 (코드 변경 시 필수) │
  4-E: 사용자에게 수정 완료 안내            │
       ↓                                   │
  사용자 승인? ─── No ─────────────────────┘
       │
      Yes
       ↓
Step 5: 마무리

중단 복구 프로토콜

어떤 Step에서든 외부 요인(push 실패, API 오류, 사용자 개입 등)으로 흐름이 중단되었다가 재개될 때, 반드시 아래 체크리스트를 따른다:

1. 현재 Step의 완료 여부 확인 — 부분 완료 상태면 미완료 하위 단계부터 재개
2. 잔여 워크플로 전체를 확인 — 현재 Step 완료 후 남은 Step(특히 Step 2 인라인 리뷰)을 순서대로 실행
3. "사용자에게 안내"는 흐름의 종료가 아니라 일시 정지 — 안내 후에도 대기 상태를 유지하고, 문제 해결 시 자동 재개

금지 패턴: push/PR 생성 등 하나의 하위 작업 해결 후 "완료"로 간주하고 후속 Step을 누락하는 것. 특히 Step 2(인라인 리뷰)는 이 스킬의 핵심 가치이므로 절대 누락해서는 안 된다.

Step 0: Standalone 분석 (STANDALONE 모드만)

STANDALONE 모드에서는 simon-dev의 Step 18-A/B에 해당하는 분석을 자체적으로 수행한다. simon-dev이 전문가 패널을 거쳐 생성하는 review-sequence와 동등한 품질을 위해 agent team을 구성한다.