GitHub PR의 긴 함수를 책임 단위로 분리하는 코드리뷰 프롬프트
변수 입력3개
언제 사용하나
GitHub PR의 함수가 검증, 계산, 저장, 알림처럼 여러 책임을 가져 리뷰와 테스트가 어려울 때 사용합니다.
준비할 정보
{{FILE_AND_FUNCTION}}: 파일 경로와 대상 함수{{PR_DIFF}}: 현재 PR diff{{BEHAVIOR_TESTS}}: 반드시 보존할 동작과 테스트 명령
복사해서 쓰는 프롬프트
아래 GitHub PR의 긴 함수를 책임 단위로 분리하는 리뷰안을 작성해줘.
대상: {{FILE_AND_FUNCTION}}
PR diff: {{PR_DIFF}}
보존할 동작과 테스트: {{BEHAVIOR_TESTS}}
1. 함수의 현재 책임을 입력·계산·side effect·오류 처리로 구분해.
2. 응집도가 높은 추출 함수와 의도가 드러나는 이름을 제안해.
3. 원래 함수는 흐름을 조율하되 인자 폭증과 숨은 상태를 만들지 마.
4. GitHub review comment 형식으로 중요한 제안부터 파일/라인 근거를 붙여줘.
5. 동작 보존 테스트와 작은 commit 분리 순서를 제시해.검증 방법
분리 전후 동일 입력과 오류 조건에 같은 결과가 나오는지 확인합니다. PR Files changed 화면에서 책임별 commit이 독립적으로 리뷰·revert 가능한지도 검수합니다.
주의사항
함수 길이만 줄이기 위해 이름 없는 wrapper를 늘리지 않습니다. public API와 transaction 경계가 바뀌면 리팩터링이 아닌 기능 변경으로 분리해 리뷰합니다.
댓글 0개
아직 댓글이 없습니다.
