[fix] Code Time 추천 로직 버그 수정 (시그널 제외 및 순서 문제)#387
Merged
Conversation
- 추천 세션 일관성 유지를 위해 실시간 필터링에서 시그널 체크 제거 - 차단 관계만 즉시 제외, 시그널은 다음 세션부터 제외 - DailyCodeMatchingService와 동작 통일 Resolves #386
- DESC를 ASC로 변경하여 생성 순서 유지 - 버킷 정책 우선순위(B1→B2→B3→B4) 보존 - Daily Code Matching과 정렬 방향 통일 Resolves #386
- 추천 세션 일관성 검증 (시그널 제외 로직) - 차단 관계 즉시 제외 검증 - 추천 순서 유지 검증 - WITHDRAWN 회원 자동 필터링 검증 - mockito-kotlin 의존성 추가 - PreVerificationStrategyTest asyncNotificationService 파라미터 추가 Resolves #386 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.
Summary
Code Time 추천 로직의 두 가지 핵심 버그를 수정했습니다:
문제 1: 시그널 제외 로직 문제
filterExcludedMembers()에서 실시간 필터링 시getRecentSignalMemberIds()호출문제 2: 추천 순서 역전 문제
findCodeTimeIdsByTimeRange()쿼리에서ORDER BY DESC사용ORDER BY ASC로 변경하여 생성 순서 유지변경 사항
수정된 파일
CodeTimeService.kt:154-186- 시그널 제외 로직 제거RecommendationHistoryJpaRepository.kt:130- 정렬 방향 수정 (DESC → ASC)테스트 추가
CodeTimeServiceTest.kt- 8개 단위 테스트 (모두 통과 ✅)의존성 추가
mockito-kotlin 5.1.0- Kotlin 테스트를 위한 Mockito wrapperTest Plan
참고 문서
docs/recommendation/feature#386.md