code-review
Spring Boot / JPA 기반 백엔드 코드를 리뷰할 때 사용한다.
Installer avec Codex ou Claude Copiez ce prompt, collez-le dans Codex, Claude ou un autre assistant, puis laissez-le vérifier la page du skill et l'installer pour vous.
Menu
Spring Boot / JPA 기반 백엔드 코드를 리뷰할 때 사용한다.
Installer avec Codex ou Claude Copiez ce prompt, collez-le dans Codex, Claude ou un autre assistant, puis laissez-le vérifier la page du skill et l'installer pour vous.
Basé sur la classification professionnelle SOC
Write or review behavior-first tests using the classical/classist/Chicago style. Use when Codex needs to add, modify, or critique tests while avoiding implementation-coupled mocks, especially for unit tests around domain logic, integration tests around real module boundaries, mock-heavy test migrations, and requests mentioning "고전파 테스트", "classical testing", "classist", "Chicago TDD", behavior-first tests, or boundary-only mocking.
Spring Boot / JPA 기반 백엔드 코드를 리뷰할 때 사용한다.
GitHub Pull Request 생성 + 스마트 커밋. Trigger: "PR 올려줘", "PR 생성", "create PR", "/create-pr", "/pr" - 미커밋 변경사항을 논리적으로 분리하여 커밋 - dev 브랜치면 자동으로 feature 브랜치 생성 - 기존 PR이 있으면 추가 커밋 반영하여 PR 업데이트
Controller 테스트 기반으로 REST Docs와 Swagger UI 문서를 추가하거나 갱신합니다. Trigger: "swagger ui 문서화", "controller 문서화", "REST Docs 추가", "openapi 추가", "api-docs 추가" - 컨트롤러에 Swagger 어노테이션을 붙이지 않고 테스트에서 문서화 - 지정된 Controller가 없으면 `dev...HEAD` diff 기준으로 변경된 Controller를 대상 선정 - REST Docs snippet, openapi3.yaml, Swagger UI까지 이어지는 흐름으로 작업 - 문서화 대상 API별 Swagger UI 스크린샷을 남기고 PR까지 업데이트
Controller 테스트 기반으로 REST Docs와 Swagger UI 문서를 추가하거나 갱신합니다. Trigger: "swagger ui 문서화", "controller 문서화", "REST Docs 추가", "openapi 추가", "api-docs 추가" - 컨트롤러에 Swagger 어노테이션을 붙이지 않고 테스트에서 문서화 - 지정된 Controller가 없으면 `dev...HEAD` diff 기준으로 변경된 Controller를 대상 선정 - REST Docs snippet, openapi3.yaml, Swagger UI까지 이어지는 흐름으로 작업
Use when you need to review, improve, or refactor Java code for object-oriented design quality — including applying SOLID, DRY, and YAGNI principles, improving class and interface design, fixing OOP concept misuse (encapsulation, inheritance, polymorphism), identifying and resolving code smells (God Class, Feature Envy, Data Clumps), or improving object creation patterns, method design, and exception handling. Part of the skills-for-java project
| name | code-review |
| description | Spring Boot / JPA 기반 백엔드 코드를 리뷰할 때 사용한다. |
이 스킬은 Spring Boot + JPA 기반 백엔드 코드를 리뷰할 때 사용한다.
목표는 단순히 "코드가 동작하는지"를 보는 것이 아니라, 비즈니스 규칙과 상태 변화가 적절한 객체에 배치되어 있는지, 서비스가 과도한 책임을 지고 있지는 않은지, JPA를 객체지향 모델링 도구로 잘 활용하고 있는지를 점검하는 것이다.
리뷰는 다음 원칙을 따른다.
anemic domain model보다 rich domain model에 가까운 방향을 제안한다.리뷰 우선순위는 아래와 같다.
다음을 점검한다.
특히 다음 문제를 주의 깊게 본다.
다음을 점검한다.
특히 아래 신호를 문제로 본다.
애플리케이션 서비스는 유스케이스를 조율해야 한다. 아래와 같은 경우 비대화 신호로 본다.
리뷰 시에는 단순히 "서비스가 길다"로 끝내지 말고, 다음 중 어떤 방향으로 분리 가능한지 제안한다.
다음을 점검한다.
groupId, groupMemberId, groupChallengeId를 직접 조합하면서 흐름을 만드는가특히 아래 패턴을 주의한다.
다음을 점검한다.
table 1개 = repository 1개에만 머물러 있지 않은가다음을 점검한다.
특히 아래 경우를 경계한다.
도메인 객체에는 다음과 같은 내용을 넣지 않는 방향을 권장한다.
ResponseStatusExceptionDataIntegrityViolationException이런 것은 웹 계층, 애플리케이션 서비스, 인프라 계층이 책임져야 한다.
리뷰 시 도메인 로직을 엔티티로 옮기자고 제안하더라도, 인프라 관심사를 같이 밀어 넣는 식의 제안은 하지 않는다.
다음을 점검한다.
좋은 테스트 방향은 다음과 같다.
아래 패턴은 발견 시 우선적으로 지적한다.
리뷰 결과는 아래 형식으로 작성한다.
각 포인트마다 아래 구조를 따른다.
아래 레벨로 구분한다.