Skip to content

chore: 코드래빗 리뷰 설정 파일 추가 (#98) - #99

Merged
chjpooh2002 merged 1 commit into
developfrom
chore/98-coderabbit-config
Nov 13, 2026
Merged

chjpooh2002 merged 1 commit into
developfrom
chore/98-coderabbit-config

Conversation

@chjpooh2002

Copy link
Copy Markdown
Contributor

관련 이슈

작업 내용

레포 최상단에 코드래빗 설정 파일 .coderabbit.yaml을 추가했습니다. 공개 레포 오픈소스 무료 플랜에서 되는 기능만 씁니다.

  • 한국어 리뷰, 지적 강도 chill, 병합 차단(request_changes_workflow) 끔
  • develop·main 대상 PR 자동 리뷰 (초안, 제목에 WIP·[skip review]가 있는 PR 제외)
  • 경로별 리뷰 지시문 + 정적 분석 도구(gitleaks, actionlint, hadolint, checkov)

초안에서 고친 부분

고친 부분 이유
기술 스택을 "스프링 부트 3"에서 스프링 부트 4.1, 자바 21, JPA + QueryDSL로 바꾸고 잭슨 3(tools.jackson) 규칙 추가 실제 스택과 달라 예전 예제 기준으로 지적할 수 있음
src/main/resources/**의 스키마 규칙을 src/main/resources/db/migration/** 별도 항목으로 분리 플라이웨이 마이그레이션이 이 경로에 있음. 병합된 파일 수정 금지, 파일 이름 규칙, 인덱스 접두어도 같이 넣음. 설정 파일 항목에는 ddl-auto 규칙을 넣음
예외 처리 지시문에 BusinessException + {도메인}ErrorCode(ResultCode) → GlobalExceptionHandler → ApiResponse 명시 공통 예외 응답 형식의 실제 구현 이름
소유자 확인 지시문에 @LoginUser Long userId, TemplateRepository.findByIdAndUserId·findTrashedByIdAndUserIdForUpdate, 남의 리소스는 403이 아니라 404 명시 실제 소유 조건 조회 방식과 CLAUDE.md 규칙
"갤러리 자원 소유자 확인"을 갤러리 공개 캐시 응답에 회원별 값 금지로 바꿈 갤러리 템플릿은 공개 데이터라 소유자가 없음
트랜잭션 지시문에 Query/Command 서비스 트랜잭션 규칙, UserCommandService.lockUser 첫 조회, @Modifying의 updated_at, 외부 호출 클래스 이름(LanguageModelClient, GoogleAuthCodeClient 등) 추가 이 프로젝트에서 실제로 틀리기 쉬운 부분
암호화 지시문에 TemplateContent의 @Convert(EncryptedStringConverter), AI 원문·결과 로그 금지 추가 암호화 대상 필드의 실제 구현
Clock 빈, UsageLimits 규칙 추가 CLAUDE.md 규칙
테스트 지시문에 H2·@WithMockUser 금지, ArchUnit 테스트 완화 금지 추가 CLAUDE.md 테스트 규칙
path_filters는 초안 그대로 (build/ 주석에 QueryDSL Q클래스 포함 명시) 자동 생성 파일은 Q클래스뿐이고 build/ 아래에 생성됨
**/*.tf, **/Dockerfile 넓은 패턴 유지, .github/workflows/** 유지 develop에는 아직 테라폼·Dockerfile이 없음(#97, #89~#91에서 루트 Dockerfile, infra/terraform/로 추가 예정이며 넓은 패턴이 둘 다 잡음)

profile, request_changes_workflow, 도구 목록, chat 설정은 초안 그대로입니다.

명세와 달라진 점

없음

테스트

  • 파이썬 yaml.safe_load로 파싱 확인
  • 코드래빗 스키마(https://coderabbit.ai/integrations/schema.v2.json)로 jsonschema 검증: 오류 0개
  • 지시문에 넣은 클래스·메서드 이름이 실제 코드에 있는지 확인

확인 체크리스트

  • ./gradlew build 통과 (설정 파일만 추가, 코드 변경 없음)
  • 테스트 포함 (해당 없음)
  • 변경 400줄 안팎
  • 병합된 마이그레이션 수정 안 함
  • 비밀 값·개인정보 없음
  • global 공통 코드를 바꿨다면 디스코드에 공유함 (해당 없음)

리뷰어에게

  • 코드래빗은 PR이 향하는 브랜치의 설정을 읽으므로 이 PR 자체에는 이 설정이 적용되지 않습니다.
  • 병합 후 확인할 것
    • 깃허브 앱 설치 (관리자)
    • 다음 PR에서 한국어 리뷰가 달리는지 확인
    • 병합을 막지 않는지(Request changes가 아닌 코멘트로만 다는지) 확인

@chjpooh2002 chjpooh2002 added chore 설정·빌드·의존성 infra AWS, 테라폼, 배포, CI labels Nov 12, 2026
@chjpooh2002 chjpooh2002 self-assigned this Nov 13, 2026

@oTAETAEo oTAETAEo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니당 👍

@chjpooh2002
chjpooh2002 merged commit b41bcd5 into develop Nov 13, 2026
1 check passed
@chjpooh2002
chjpooh2002 deleted the chore/98-coderabbit-config branch November 13, 2026 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 설정·빌드·의존성 infra AWS, 테라폼, 배포, CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[chore] 코드래빗 리뷰 설정 파일(.coderabbit.yaml) 추가

2 participants