Open
Conversation
- Scouter 에이전트/설정/플러그인 전체 삭제 - Glowroot 0.14.5 Standalone 에이전트 추가 (app/glowroot/) - build.gradle: scouterJvmFlags → glowrootJvmFlags 교체, bootRun 에이전트 적용 - 프로덕션 docker-compose: Glowroot UI 포트 노출 (Blue:4001, Green:4002) + 데이터 볼륨 - deploy.sh: 환경변수 기반 Glowroot 인증 자동 설정 (admin.json 동적 생성) - CD 워크플로우: GLOWROOT_USER/PASSWORD 환경변수 추가 - 로컬 docker-compose: Scouter 서버 컨테이너 제거
- QuizHistory: @softdelete 어노테이션 적용, deleted 필드 수동 관리 제거 - Repository: 불필요한 커스텀 쿼리 제거, Spring Data JPA 메서드 네이밍 통일 - GenerationCommandServiceImpl: Slack 알림 로직을 GenerationSlackNotifier로 분리 - null 체크를 CollectionUtils.isEmpty()로 통일
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
- initHistory: 퀴즈 생성 시 히스토리 초기화 - deleteAllHistory: 전체 히스토리 삭제 - updateHistoryTitle: 히스토리 제목 변경 - title 필드 추가 (QuizHistory, SaveHistoryRequest) - 히스토리 목록 조회 구현 (QuizHistoryMapper)
- Generation 내부 전용 서비스임을 패키지 구조로 명시 - postRefineQuiz 메서드로 셔플/해설 빌드 로직 통합
- InitHistoryRequest DTO 생성 (problemSetId, title) - POST /history/init 엔드포인트 추가 - GenerationCommandServiceImpl에서 QuizHistoryCommandService 의존성 제거 - generation ↔ history 서비스 간 완전 분리
- @softdelete 제거하고 물리 삭제로 변경 - deleteAll 대신 JPQL DELETE 쿼리로 성능 개선 - initHistory를 bulk INSERT 네이티브 쿼리로 최적화
- ChangeHistoryTitleRequest record 생성 (NotBlank, Size 검증 포함) - Map<String, String> 대신 타입 안전한 DTO로 교체
- userId가 null인 경우 NPE 방지를 위한 null 체크 추가 - 예외를 NOT_ENOUGH_ACCESS로 변경
- HistoryIdResponse record 생성 (initHistory, saveHistory 공용) - initHistory 반환 타입을 void → String으로 변경하여 historyId 반환 - Map<String, String> 대신 타입 안전한 DTO로 통일
- ChunkInfo에 boolean fastFirst 필드 추가 - ChunkSplitter.distributeQuizCount()를 첫 청크 1문제 고정 + 나머지 라운드 로빈으로 변경 - 페이지 분배는 기존 균등 방식 유지, 첫 청크만 fastFirst=true
- 3개 GuideLine 프롬프트의 근거/복습 인용 마커 통일 - ExplanationMarkdownBuilder 마커 동기화
- HistoryDetailResponse에 totalTime 필드 추가 - QuizHistoryCommandServiceImpl, QueryServiceImpl 매핑 반영
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
1 participant
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.
📢 설명
변경 1: [제목]
[변경에 대한 설명]
의사 선택과정 (trade-off)
얻은 것
잃은 것
코드 설명: 인라인 코멘트 확인
✅ 체크 리스트
재현 확인
기본