풀 리퀘스트 검토하기
- diff를 가져옵니다. PR 번호가 있으면
gh pr diff <번호>를 사용하고, 그렇지 않으면git diff origin/main...HEAD를 사용합니다. PR 설명이 있다면 함께 읽습니다 (gh pr view <번호>). - diff만으로 명확하지 않은 경우 변경된 파일 전체를 읽습니다. 읽지 않은 코드는 검토하지 않습니다.
- 다음 순서대로 확인합니다:
- 정확성: 잘못된 조건, off-by-one 오류, 처리되지 않은 오류, 경쟁 조건, null/빈 입력.
- 테스트: 각 동작 변경이 테스트로 커버되는가? 변경 없이도 테스트가 실패하는가?
- 보안: 신뢰할 수 없는 입력이 셸, SQL, 파일 경로, HTML에 도달하는 경우; diff에 포함된 비밀 정보.
- 호환성: 변경된 공개 인터페이스, 마이그레이션, 설정, 환경 변수.
- 가독성: 다음 유지보수 담당자의 작업 속도를 늦출 수 있는 문제만 지적합니다. 린터가 이미 커버하는 순수 스타일 선호사항은 생략합니다.
npm test가 존재하는 경우 실행하고 결과를 정직하게 보고합니다.
출력 형식
## 요약
PR이 무엇을 하는지와 전체 평가(승인 / 변경 요청 / 의견)를 한두 문장으로 요약합니다.
## 수정 필수
- `path/file.js:42` 무엇이 잘못되었는지, 왜 중요한지, 제안하는 수정 방법
## 고려해야 할 점
- `path/file.js:90` ...
## 있으면 좋은 점
- ...
비어 있는 섹션은 생략합니다. 모든 발견 사항에 대해 파일:줄번호를 명시합니다. 문제가 없다면 요약에 그렇게 명시합니다. 사용자가 요청하지 않는 한 GitHub에 댓글을 게시하지 않습니다.