# External Pr Review

> 외부 기여자 PR 검토 절차를 적용한다. PR 정보 수집, mydocs/pr/pr_{N}_review.md 작성, 검증, pr_{N}_report.md 작성, 처리 완료 시 archives/ 이동을 수행한다. 외부 기여자 PR 전용 (내부 타스크에는 사용 금지).

- Skill: `postmelee/external-pr-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add postmelee/external-pr-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/postmelee/external-pr-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: postmelee (https://skillmd.com/u/postmelee)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/postmelee/external-pr-review

---


# 외부 기여자 PR 검토

## 트리거

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

## 사전 조건

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

## 절차

1. PR 메타 수집
   ```bash
   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. 처리 완료 시 문서 보관 이동
   ```bash
   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 검토는 내부 단계 형식 강제 아님)
   ```bash
   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`