Menú
Pull Request를 체계적으로 리뷰하여 프로젝트 컨벤션 준수 여부와 코드 품질을 검증합니다
Instalar con Codex o Claude Copia este prompt, pégalo en Codex, Claude u otro asistente, y deja que revise la página de la skill y la instale por ti.
Basado en la clasificación ocupacional SOC
| name | review-pr |
| description | Pull Request를 체계적으로 리뷰하여 프로젝트 컨벤션 준수 여부와 코드 품질을 검증합니다 |
이 skill은 solid-connect-server 프로젝트의 Pull Request를 체계적으로 리뷰합니다.
/review-pr <PR번호>
예제:
/review-pr 666
GitHub CLI로 PR의 기본 정보와 변경사항을 파악합니다.
gh pr view <번호> -R solid-connection/solid-connect-server # PR 기본 정보 조회
gh pr diff <번호> -R solid-connection/solid-connect-server # 변경된 파일과 diff 확인
gh pr checks <번호> -R solid-connection/solid-connect-server # CI/CD 상태 확인
수집할 정보:
도구 우선순위:
Serena MCP (Java 코드 분석에 최적화)
mcp__serena__get_symbols_overview <파일경로> - 파일의 클래스/메서드 구조 파악mcp__serena__find_symbol <심볼명> - 특정 심볼 검색mcp__serena__search_for_pattern <패턴> - 컨벤션 위반 패턴 검색Read/Grep (보조 분석)
Read <파일경로> - 파일 전체 읽기Grep --pattern <패턴> - 패턴 검색아래 체크리스트를 순서대로 확인합니다.
각 항목의 상세 컨벤션은 참조 문서를 확인하세요.
검증 항목:
👉 참고: CLAUDE.md - "아키텍처" 섹션
검증 항목:
/user-profile)from(), 다중 파라미터 of()XXXRequest, XXXResponse 형식어떤_것을_하면_어떤_결과가_나온다() 패턴👉 참고: CLAUDE.md - "네이밍 컨벤션" 섹션
검증 항목:
*) import 금지패턴 검색 예제:
mcp__serena__search_for_pattern "import.*\\*" # 와일드카드 import 검색
👉 참고: CLAUDE.md - "코드 스타일" 섹션
검증 항목:
@Column 어노테이션 존재name 속성으로 컬럼명 명시nullable 속성 명시int, long, boolean)Integer, Long, Boolean)👉 참고: CLAUDE.md - "데이터베이스 접근" 섹션
검증 항목:
V{VERSION}__{DESCRIPTION}.sqlsrc/main/resources/db/migration/👉 참고: CLAUDE.md - "데이터베이스 마이그레이션" 섹션
검증 항목:
@TestContainerSpringBootTest 어노테이션 사용@DisplayName으로 한글 설명 제공@Nested로 기능별 그룹화👉 참고: .claude/skills/test/SKILL.md
검증 항목:
<type>: <description> 형식feat, fix, refactor, test, chore, docs, perf👉 참고: CLAUDE.md - "Git 커밋 컨벤션" 섹션
검증 항목:
@Transactional 적절하게 사용 (읽기 전용: readOnly = true)@AuthorizedUser)👉 참고: CLAUDE.md - "아키텍처", "기술 스택 상세" 섹션
# 파일의 클래스/메서드 구조 파악
mcp__serena__get_symbols_overview src/main/java/.../MentorService.java
# 특정 심볼 검색
mcp__serena__find_symbol "MentorDetailResponse"
# 컨벤션 위반 패턴 검색
mcp__serena__search_for_pattern "import.*\\*"
# PR 정보
gh pr view 666 -R solid-connection/solid-connect-server --json title,body,author,number,url
# 변경사항
gh pr diff 666 -R solid-connection/solid-connect-server --patch
# CI 상태
gh pr checks 666 -R solid-connection/solid-connect-server
# 파일 읽기
Read src/main/java/.../MentorService.java
# 패턴 검색
Grep --pattern "@Column" --glob "*.java" --path src/main/java/.../domain
다음 형식으로 리뷰 결과를 정리하여 제공합니다.
## PR 리뷰 결과: #{번호} - {제목}
**PR 링크:** {GitHub URL}
**관련 이슈:** #{이슈번호}
### 📊 PR 정보 요약
- **작성자:** {작성자}
- **변경 파일:** {숫자}개
- **추가 라인:** +{숫자}, **삭제 라인:** -{숫자}
- **CI/CD 상태:** {통과/실패}
### 주요 변경사항
{PR 설명 요약}
---
### ✅ 통과 항목
- 아키텍처 계층 구조 준수
- 네이밍 컨벤션 준수
- ...
### ⚠️ 개선 권장 항목
- **코드 스타일**: 와일드카드 import 사용
- 파일: `src/main/java/.../MentorService.java:5`
- 개선: 명시적 import로 변경
### ❌ 필수 수정 항목
- **Entity**: @Column 어노테이션 누락
- 파일: `src/main/java/.../domain/Mentor.java:30`
- 수정 방향: 모든 필드에 `@Column` 어노테이션 추가
---
### 💡 종합 의견
{전반적인 리뷰 의견}
**승인 상태:** ✅ 승인 / ⚠️ 조건부 승인 / ❌ 수정 후 재검토
CLAUDE.md - 전체 개발 컨벤션.claude/skills/test/SKILL.md - 테스트 작성 가이드