Conversation
- CLAUDE.md
- main 브랜치 설명을 prod 승격 전용으로 교체
- 원인: main 은 promote.yml 의 승격 대상
- worktree 생성·진입·정리 절차를 로비 규칙과 /session-close 참조로 축약
- 원인: git -C 격리·EnterWorktree name= 서술이 로비 규칙과 충돌
- 도메인 예외 예시를 가상 Order 예시로 교체
- 원인: TournamentException.alreadyCompleted·complete(winnerWishItemId) 가 코드에 없음
- 반복 서술 삭제: null 금지 반복 줄·한 줄 외울 것·self-invocation 일반론·SLF4J 설명·ItemSnapshot 컬럼 나열
- 매퍼 분기 참조를 분기 위치 결정 트리 (3) 으로 교체
- 응답 래퍼 예외에 SSE 구독 추가
- 웹 요청 경계 훅 서술에 hop 세션 직접 준수 추가
- 원인: 로비에서 hop 한 세션에는 프로젝트 훅이 돌지 않음
- .claude/rules/domain-exception.md
- 예외 생성 예시를 ErrorCode 기반 WishException 모양으로 교체
- 원인: ErrorCode 이관 전 생성자 모양 잔존
- 없는 팩토리 alreadyExists 를 notFound 로 교체
- 나쁜 예 이름을 invalidValue 로 바꿈
- 원인: 실존 팩토리 untrustworthyValue 와 이름 겹침
- 자동 로드 파일 나열을 frontmatter paths 참조로 교체
- 에픽 #728 참조 삭제
- .claude/rules/openapi-controller.md
- 어드민 컨트롤러 나열을 admin/** 전체 지칭으로 교체
- 원인: 나열 밖 어드민 컨트롤러 존재
- SSE 구독의 SseEmitter 직접 반환 예외 추가
- .claude/rules/db-schema.md
- 엔티티 개수 수치 삭제
- 원인: 22개 중 6개 수치 불일치
- paths 에 metrics 패키지 추가
- 원인: metrics/registration/WishExternalEntry 가 엔티티
- .claude/rules/testing-convention.md
- stub 예시를 StubProductLinkExtractor 실물 타입으로 교체
- 원인: StubProductExtractor·ProductExtractor·Product 타입 없음
- 게이트 우회 구멍 서술을 관리자 머지 시 체크 건너뜀으로 교체
- 원인: ruleset 이 dev 직접 push 를 막음
- Kotest·Strikt 도입 검토 문구 삭제
- .claude/rules/web-request-boundary.md
- 훅 차단 항목에 hop 세션 직접 준수 추가
- 이슈 번호 괄호 삭제
- src/main/resources/db/migration/CLAUDE.md
- out-of-order 근거를 additive 한정으로 교체
- 원인: 순서 의존 마이그레이션도 존재
- squash 예외 문장 삭제
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: TeamPiKi/core/.coderabbit.yml Review profile: CHILL Plan: Advanced Run ID:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
- CLAUDE.md 트랜잭션 분리 예시를 AsyncItemParsingWorker·ItemParsingService 로 교체 - 원인: WishlistService.registerFromUrl 은 추출 없이 itemRegistrar.accept 후 persist 만 수행 - CLAUDE.md worktree 절의 로비 규칙 재서술 제거 - 원인: 바로 위 문단과 중복 - CLAUDE.md 정리 불릿에 session-close 대상 범위 추가·타이머 문장 삭제 - CLAUDE.md 테스트 절의 infra 정본 목차 괄호 삭제 - 원인: 정본 목차 복제 - openapi-controller.md·CLAUDE.md openapi 절의 glob 나열을 frontmatter paths 참조로 교체 - openapi-controller.md 어드민 예외 서술을 클래스 @hidden 기준으로 정정 - 원인: DiscordAccessController 는 화면이 아닌 Discord 인터랙션 엔드포인트 - db-schema.md 본문의 glob 재나열을 frontmatter paths 참조로 교체 - db/migration/CLAUDE.md 순서 의존 변경 문장 삭제 - 원인: 다음 불릿과 중복 - web-request-boundary.md 훅 절 도입문 갱신 - 원인: hop 세션에는 차단 메시지 자체가 없음 - testing-convention.md 원칙 목차 복제 삭제 - terraform/iam.tf 주석의 배포 공통화 등급 표기 삭제 - 원인: infra 규약에서 등급 개념 삭제
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Situation
설정 문서 감사 결과
.claude/rules/5개·마이그레이션 CLAUDE.md 를 현재 코드와 대조settings.json훅이 돌지 않음Task
낡은 서술 정리
Action
코드와 다른 예시·수치 교체
domain-exception.md예시WishException(message, category, httpStatus)·alreadyExists()*ErrorCode엔트리 하나에서 파생하는 실제 모양·notFound()testing-convention.mdstub 예시StubProductExtractor·ProductExtractorStubProductLinkExtractor·ProductLinkExtractor·ProductSnapshotdb-schema.mdpaths포인터,paths에metrics/**추가openapi-controller.md어드민 범위admin/**컨트롤러 전부(클래스에@Hidden)ApiResponseBody예외WishlistService.registerFromUrl(이제 추출 안 함)AsyncItemParsingWorker에서ItemParsingService로 위임TournamentException.alreadyCompleted()사실이 바뀐 서술 교체
main브랜치promote.yml)git -C격리·name=경고origin/dev, 진입 방식은 분리 질문에서 확인/session-close(clean·머지된 현재 자리만), 수동도--force없이paths포인터삭제
#728미배정 현황 ·#986·#988·create_init_schemasquash 예외 ·iam.tf주석의 "배포 공통화 등급 C"ItemSnapshot필드 목록Result
범위
require의 status 서술은 보류GlobalExceptionHandler는 400. 문서와 핸들러 중 어느 쪽을 맞출지 결정 대기terraform/iam.tf는 주석 한 줄만 수정,terraform fmt -check통과WishErrorCode.kt의#728참조StubProductLinkExtractor.kt의 옛 절 위치 참조ExtractionModelProbe.kt의 extractordocs/api-contract.md참조연관 이슈