trailofbits/review-pr

PR 리뷰

브랜치의 변경 표면만 검토하고, 전체 저장소는 검토하지 않습니다.

1. diff 생성: git diff <base>...HEAD (작업 트리에 브랜치에 속하는 커밋되지 않은 변경 사항이 있을 경우 git diff <base>도 사용). 기준 브랜치는 호출자가 지정하며, 지정되지 않으면 main으로 대체합니다.
2. 모든 변경된 청크(hunk)의 정확성을 검토: 경계 입력에서의 잘못된 결과, 처리되지 않은 오류 경로, 함수 이름이나 호출자와 모순되는 동작.
3. 변경된 동작에 대한 테스트 커버리지 확인: 새로운 로직이 엣지 케이스를 테스트하지 않으면 결함으로 보고.
4. 변경으로 인해 더 이상 유효하지 않은 문서 확인: README나 문서에서 기준 브랜치에서는 참이었으나 변경 후에는 거짓이 되는 문장.
5. 발견된 모든 결함을 심각도와 함께 보고: 주요, 보조, 정보 등 모든 발견 사항을 포함. 브랜치가 건드리지 않은 파일의 결함은 보고하지 않음 — 해당 파일은 풀 리퀘스트 범위 밖입니다.