[Refactor] analysis AI/question candidate 정리 (#271) - #274
Conversation
|
Warning Review limit reached
Next review available in: 42 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (9)
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 |
✨ 어떤 이유로 PR를 하셨나요?
📋 세부 내용 - 왜 해당 PR이 필요한지 작업 내용을 자세하게 설명해주세요
analysis 쪽에서 이미 분리된 책임과 맞지 않게 남아 있던 중복 코드와 question candidate 경합 포인트를 정리했습니다.
이번 PR은 기능 추가보다는 불필요한 외부 의존 표면을 줄이고, custom candidate 저장 충돌과 후보 식별자 충돌을 더 안전하게 다루는 데 목적이 있습니다.
변경 내용:
AnalysisAiClient에 남아 있던 예전 prompt 조립 중복 코드, 미사용 상수, 미사용 의존성 제거QuestionCommandService의 custom candidate 저장을REQUIRES_NEWpersistence service로 분리QuestionCandidateResponse에candidateKey추가default:{id}, custom 후보는custom:{id}형식으로 식별자 분리기대 효과:
AnalysisAiClient가 실제 사용하는 협력자만 남아 구조가 더 읽기 쉬워짐📸 작업 화면 스크린샷
🚨 관련 이슈 번호 [ #271 ]