pr-review-orchestrator

PR Review Orchestrator

목표

변경사항 하나를 네 에이전트 팀으로 검토한다. 오케스트레이터는 직접 코드를 고치지 않는다. 팀을 만들고, 작업 큐를 배치하고, 팀원 간 직접 메시지를 허용하고, 최종 리포트만 통합한다.

입력

• diff_path 또는 PR diff 텍스트
• 변경 파일 목록
• 선택 입력: 리뷰 기준, 금지된 변경 영역, 릴리스 위험도

산출물

• _workspace/pr-review/static.md
• _workspace/pr-review/security.md
• _workspace/pr-review/tests.md
• _workspace/pr-review/final-review.md
• _workspace/pr-review/team-log.jsonl

Phase 0. 사전 조건

1. 변경 파일 목록을 읽고 리뷰 범위를 확정한다.
2. _workspace/pr-review/ 디렉터리 존재를 확인한다.
3. diff가 없거나 변경 파일이 0개면 사용자에게 중단 사유를 보고한다.
4. 파일 수가 40개를 넘으면 먼저 범위를 나누고, 이번 실행에서는 가장 위험도가 높은 묶음 하나만 리뷰한다.

Phase 1. 팀 생성

TeamCreate로 pr-review-team을 만든다. 팀원 네 명을 AgentTool로 스폰한다.

이름	역할	출력
static-reviewer	버그·성능·스타일 검토	_workspace/pr-review/static.md
security-reviewer	입력 검증·권한·비밀 노출 검토	_workspace/pr-review/security.md
test-reviewer	테스트 영향·누락 회귀 검토	_workspace/pr-review/tests.md
review-integrator	세 리포트 병합·중복 제거	_workspace/pr-review/final-review.md

각 팀원 프롬프트에는 공통 규칙을 넣는다.

• 발견 즉시 관련 팀원에게 SendMessage로 알린다.
• 최종 판단은 파일에 저장한다.
• Critical 발견 시 오케스트레이터와 관련 팀원에게 동시에 보고한다.
• 코드 수정은 하지 않는다.

Phase 2. 작업 큐 배치

TaskCreate로 네 작업을 만든다.

1. static-pass: static-reviewer 담당. diff 전체의 버그·성능·스타일 검토.
2. security-pass: security-reviewer 담당. 외부 입력·권한·비밀 노출 검토.
3. test-pass: test-reviewer 담당. 변경 파일과 테스트 매핑, 누락 테스트 검토.
4. integrate-review: review-integrator 담당. 앞 세 작업 완료 후 실행. depends_on = ["static-pass", "security-pass", "test-pass"].

Phase 3. 팀원 간 메시지 규칙

SendMessage는 리더를 거치지 않는다.

• static-reviewer가 데이터 구조(shape) 변경을 발견하면 test-reviewer에게 테스트 영향 확인을 요청한다.
• security-reviewer가 인증·권한 변경을 발견하면 test-reviewer에게 보안 회귀 테스트 존재 여부를 묻는다.
• test-reviewer가 테스트 불가능한 설계를 발견하면 static-reviewer에게 구조 리스크 확인을 요청한다.
• Critical 이슈는 오케스트레이터에도 요약 메시지를 보낸다.

메시지 형식은 고정한다.

{
    "type": "finding",
    "severity": "Critical|High|Medium|Low",
    "file": "path/to/file.ts",
    "line": 42,
    "claim": "문제를 주장하는 한 문장",
    "request": "상대 팀원이 확인할 질문"
}