Lead Review (Discourse Pattern)

Overview

Discourse 조율 패턴으로 다관점 리뷰를 수행하는 Lead 스킬이다. 두 가지 모드를 지원한다:

tech-spec 모드: Tech Spec 문서를 리뷰하고 사람 개입 ①-b (최종 승인)를 위한 결과를 생성한다.
epic-review 모드: 에픽 전체 구현을 통합 리뷰하고, 자기 완결적 review.md를 생성한 뒤 종료한다. 사람과 직접 소통하지 않는다.

When to Use

tech-spec 모드: /bf-spec이 Tech Spec 작성 후 자동 스폰
epic-review 모드: bf-lead-orchestrate가 E2E 통과 후 자동 스폰
직접 호출하지 않는다.

Prerequisites

tech-spec 모드: docs/tech-specs/{TICKET}-tech-spec.md 존재
epic-review 모드: 해당 에픽의 모든 Story status: done 또는 skipped, E2E passed 또는 skipped 또는 escalated 또는 max-regression-cycles

Error Handling

tech-spec 파일 미존재: "error: tech-spec not found" 신호를 스폰한 상위 에이전트에 전달 후 종료
conventions.md 미존재 (epic-review 모드): Convention Guard를 "conventions.md가 없으므로 일반 코드 품질 기준으로 검사" 모드로 실행. 결과에 "conventions.md 미존재로 기본 규칙 적용" 명시
git diff 추출 실패: 에픽 첫 Story 커밋을 찾을 수 없으면 HEAD~{story_count}..HEAD 범위로 fallback. 그래도 실패 시 "error: diff extraction failed" 보고

Instructions

1. 모드 감지

스폰 시 전달받은 파라미터로 모드를 결정한다:

mode: "tech-spec" + tech-spec 경로
mode: "epic-review" + epic ID + tech-spec 경로

2. 초기 로딩

yq 전제조건 체크 (최초 1회, epic-review 모드):

command -v yq >/dev/null 2>&1 || { echo "❌ yq not installed. Install: brew install yq"; exit 1; }

모드	읽는 파일
tech-spec	tech-spec.md, conventions.md
epic-review	tech-spec.md, conventions.md, 에픽 전체 git diff (`git log --oneline --grep="({STORY-ID-1}\|{STORY-ID-2}...)" --reverse`로 에픽 첫 Story 커밋을 찾고, 그 부모 커밋 ~ HEAD를 diff 범위로 사용)

3. 모델 선택

모드	모델
tech-spec	항상 Opus (프로젝트 전체 의사결정)
epic-review	에픽 내 L/XL Story 포함 시 Opus, S/M만이면 Sonnet

4. 리뷰어 팀 구성

tech-spec 모드:

Lead가 Tech Spec을 읽고 프로젝트 특성에 맞는 3가지 리뷰 페르소나를 추천한다.
- 예: Architecture Reviewer, Security Reviewer, Performance Reviewer
- 또는: Backend Engineer, Frontend Engineer, QA Engineer
추천된 페르소나 3명을 teammate로 생성한다 (model: opus).

epic-review 모드:

<HARD-GATE> Convention Guard 리뷰어는 epic-review에서 절대 생략할 수 없다. conventions.md가 비어있거나 짧아도 Convention Guard를 포함해야 한다. "컨벤션이 아직 없으니 생략해도 된다"는 이 게이트를 우회하는 전형적인 합리화이다. </HARD-GATE>

Lead가 에픽 변경 범위를 분석하여 2~3명의 리뷰어를 구성한다:
- Convention Guard (필수): docs/conventions.md 기준 컨벤션 준수 여부
- 추가 1~2명: 프로젝트 특성에 따라 Architecture, Security, Performance 중 선택
모델은 3단계에서 결정한 모델과 동일하게 적용한다.

5. 독립 분석

<HARD-GATE> 모든 리뷰어의 독립 분석이 완료될 때까지 discourse(교차 검증)를 시작하지 않는다. 독립 분석 없이 바로 토론에 들어가면 첫 번째 의견에 앵커링되어 다관점 리뷰의 가치가 사라진다. </HARD-GATE>

각 리뷰어가 독립적으로 분석을 수행한다:

tech-spec 모드 관점:

아키텍처 정합성: 기존 패턴과의 일관성, 모듈 경계 위반
보안: 인증/인가, 데이터 노출, 입력 검증
성능: 쿼리 최적화, 캐싱 전략
테스트 가능성: AC가 테스트 가능한 형태인지, 엣지 케이스 누락
변경 영향도: 사이드이펙트, 하위 호환성

epic-review 모드 관점:

Convention Guard: docs/conventions.md 기준 준수 여부
스토리 간 중복/불일치: 중복 유틸 함수, 네이밍 불일치
아키텍처 드리프트: 개별로는 맞지만 합치면 드러나는 구조 문제
테스트 커버리지: AC 대비 테스트 누락
보안: 인젝션, 인증 우회, 데이터 노출

6. Discourse (교차 검증) — 쟁점 해소 프로토콜

다음 우선순위로 쟁점을 해소한다:

① 리뷰어 직접 대화:

Lead가 각 리뷰어의 발견사항을 수집하여 공유한다.
리뷰어끼리 SendMessage로 직접 challenge/agree/보완한다.
합의되면 → Lead에 결론만 보고.

② 미합의 시 Lead 중재:

Lead가 프로젝트 방향성(tech-spec, conventions)을 기준으로 판단한다.
Lead 결정으로 확정.

③ 그래도 미합의 → 버린다 (기록):

최종 결과에 "미합의 쟁점"으로 포함한다.
더 이상 토큰을 쓰지 않는다.

7. 리뷰 결과 통합

발견사항을 분류한다:

🔴 Blocker (반드시 수정)
🟡 Recommended (권장 수정)
🟢 Confirmed (확인 완료)
미합의 쟁점 (쟁점 + 각 리뷰어 의견 + Lead 최종 판단 + 근거)

8. 모드별 출력 및 후속 처리

tech-spec 모드:

docs/reviews/{TICKET}-tech-spec-review.md에 저장한다.
- docs/reviews/ 디렉토리가 없으면 생성
- 재리뷰 시에는 ## Re-review ({날짜}) 섹션을 append
git commit하지 않는다 — docs/ 산출물은 Phase 4 Archive에서 일괄 커밋한다.
스폰한 상위 에이전트에 전달:
- "done" + review.md 경로
종료 (컨텍스트 소멸).

epic-review 모드:

docs/reviews/{EPIC-ID}-review.md에 저장한다.
- 수정 재실행(modification)으로 재리뷰하는 경우: 기존 파일을 덮어쓴다 (이전 리뷰는 git 이력에 보존됨). sprint-status.yaml의 모든 done Story의 review_blockers/review_recommended도 새로 계산하여 덮어쓴다.
sprint-status.yaml 업데이트 (CLAUDE.md의 Read-yq-Verify 프로토콜, yq -i 명령어 사용):
- 에픽 내 각 Story의 review_blockers, review_recommended 건수 기록
- cross-story 귀속 규칙: 여러 Story에 걸친 발견 사항(네이밍 불일치, 중복 유틸 등)은 가장 관련 높은 Story에 귀속한다. 특정 Story에 귀속할 수 없는 경우 에픽의 첫 번째 done Story에 귀속한다. 모든 Story가 skipped인 에픽에서는 review_blockers/review_recommended를 sprint-status.yaml에 기록하지 않는다 (귀속 대상 없음, review.md에만 기록).
```
yq -i '
  .<TICKET>.<EPIC>.<STORY>.review_blockers = 2 |
  .<TICKET>.<EPIC>.<STORY>.review_recommended = 5
' docs/sprint-status.yaml
```
git commit하지 않는다 — docs/ 산출물은 Phase 4 Archive에서 일괄 커밋한다.
Blocker 판정 및 종료:
- Blocker 0건: 에픽 내 모든 status: done Story의 review: approved로 업데이트 (status: skipped Story는 bf-execute/bf-resume이 사람 "진행" 선택 시 처리):
```
yq -i '.<TICKET>.<EPIC>.<STORY>.review = "approved"' docs/sprint-status.yaml
```
  스폰한 상위 에이전트에 전달: "done: approved" + review.md 경로
- Blocker 1건 이상: review 상태를 유지 (pending) 스폰한 상위 에이전트에 전달: "done: blockers" + review.md 경로
종료 (컨텍스트 소멸).

Output Format

review.md 구조

# {TICKET 또는 EPIC-ID} 리뷰

## 리뷰 개요
- 모드: {tech-spec | epic-review}
- 리뷰어: {페르소나 목록}
- 날짜: {YYYY-MM-DD}

## 🔴 Blocker (N건)
- [{위치}] 설명 + 수정 방안

## 🟡 Recommended (N건)
- [{위치}] 설명 + 권장 수정

## 🟢 Confirmed (N건)
- 확인 완료 항목 요약

## 미합의 쟁점
| 쟁점 | 입장 A | 입장 B | Lead 최종 판단 | 근거 |
|------|--------|--------|---------------|------|
| ... | ... | ... | ... | ... |

## 권장 조치 (epic-review 모드만)

### Blocker (즉시 수정 필요)
- {STORY-ID}: {구체적 수정 내용 + 파일:라인 위치}

### Recommended (권장)
- {항목}

Fallback

Agent Teams 생성 실패 시: 메인 세션(또는 스폰한 상위 에이전트)에서 단일 Opus로 리뷰 수행
Teammate 무응답 (10분 이상): 해당 역할을 제외하고 나머지 결과로 진행, 제외된 역할을 결과에 명시

bf-lead-review

Safety Notice

Copy this and send it to your AI assistant to learn