External pr review

외부 기여자 PR 검토

트리거

• 작업지시자가 "PR #N 리뷰" 또는 "외부 PR 검토"를 명시 지시한 경우
• 본 SKILL을 직접 호출한 경우

사전 조건

• 검토 대상 PR이 외부 기여자 fork에서 본 저장소 {BASE_BRANCH}(또는 합의된 base)로 열린 상태
• 내부 타스크 PR(publish/task{N})에는 본 SKILL 사용 금지 — 내부 타스크는 일반 단계 절차로 검토
• gh CLI 인증

절차

1. PR 메타 수집

   gh pr view {N} --json number,title,state,baseRefName,headRefName,headRepository,mergeable,mergeStateStatus,reviewDecision,labels,body
   gh pr diff {N} | head -200
   gh pr checks {N}
   

   • 이슈 연결, base/head, mergeable, CI 상태 모두 확인
2. 검토 문서 작성: mydocs/pr/pr_{N}_review.md
   • 중앙 템플릿 mydocs/_templates/external_pr_review.md를 기준으로 작성한다.
   • 템플릿을 읽을 수 없는 경우에만 다음 최소 섹션을 fallback으로 사용한다:
     • PR 정보 (번호, 작성자, base/head, 연결 이슈)
     • 변경 요약
     • 영향 범위와 호환성 (FFI, build, 문서)
     • 코드/문서 점검 결과
     • 검증 계획 (필요한 추가 검증)
     • 권고 (merge / 수정 요청 / 닫기)
     • 작업지시자 승인 요청
3. 작업지시자 승인 요청 (검토 방향 결정)
4. 필요 시 수정·검증 계획 문서 작성: mydocs/pr/pr_{N}_review_impl.md
   • 중앙 템플릿 mydocs/_templates/external_pr_review_impl.md를 기준으로 작성한다.
   • 본 저장소에서 추가 검증을 직접 수행할 때 사용
   • 작성 후 작업지시자 승인 요청
5. 검증 수행 (해당하는 경우만)
   • 검증은 변경 유형에 따라 {PROJECT_VALIDATION_GUIDE} 정책 적용
6. 최종 보고서 작성: mydocs/pr/pr_{N}_report.md
   • 중앙 템플릿 mydocs/_templates/external_pr_report.md를 기준으로 작성한다.
   • 검토 결과, 검증 결과, 최종 권고, GitHub PR 코멘트 본문(또는 링크)
7. 작업지시자 승인 후 GitHub PR에 코멘트/리뷰 등록 (merge 결정은 작업지시자가 수행)
8. 처리 완료 시 문서 보관 이동

   git mv mydocs/pr/pr_{N}_review.md mydocs/pr/archives/
   git mv mydocs/pr/pr_{N}_review_impl.md mydocs/pr/archives/  # 존재 시
   git mv mydocs/pr/pr_{N}_report.md mydocs/pr/archives/
   

9. 단일 또는 단계별 커밋 (외부 PR 검토는 내부 단계 형식 강제 아님)

   git commit -m "PR #{N} 검토: {요약}"
   

검증

• mydocs/pr/pr_{N}_review.md가 mydocs/_templates/external_pr_review.md의 필수 섹션을 채움
• mydocs/pr/pr_{N}_review_impl.md를 작성했다면 mydocs/_templates/external_pr_review_impl.md의 필수 섹션을 채움
• mydocs/pr/pr_{N}_report.md가 mydocs/_templates/external_pr_report.md의 필수 섹션을 채움
• 권고 결정이 명시됨 (merge / 수정 / 닫기 중 하나)
• 처리 완료 후 작성된 PR 검토 문서가 mydocs/pr/archives/에 존재

절대 하지 말 것

• 내부 타스크 PR(publish/task{N})에 본 SKILL 적용
• 외부 PR을 작업지시자 승인 없이 merge 또는 close
• 외부 기여자 fork의 코드를 본 저장소에 직접 cherry-pick (PR 절차 생략)
• 내부 단계 절차(_stage{N}.md, _report.md) 형식을 외부 PR 문서에 강제 적용

호출 방법

• Codex: $external-pr-review 또는 /skills 메뉴
• Claude Code: /external-pr-review