20260105 #713 하이브리드 검색 결과 관련성 필터링 로직 개선

[Yooonjeong]

✨ 변경 사항

---

+) 한글을 지원하는 모델(text-multilingual-embedding-002)로 변경되었습니다.
+) DB 내 임베딩 데이터를 유효한 값으로 최신화했습니다.

✅ 테스트

---

• 수동 테스트 완료
• 테스트 코드 완료

Summary by CodeRabbit

주요 변경사항

• 새로운 기능

  • 읽지 않은 메시지 알림에 채팅방 ID, 미읽 메시지 수, 마지막 메시지 정보 등 상세 정보 추가
  • 메시지 유형별 미리보기 기능 추가

• 개선 사항

  • 임베딩 API 오류 발생 시 명확한 에러 처리 강화
  • 검색 결과에 유사도 필터링(≤ 0.90) 추가로 더 정확한 검색 결과 제공

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)

Validation error: Unrecognized key(s) in object: 'ignored_branch'

⚙️ Configuration instructions

• Please see the configuration documentation for more information.
• You can also validate your configuration using the online YAML validator.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

Redis 캐싱 주석 제거, 벡터 유사도 필터링 임계값 추가, 채팅 메시지 알림 페이로드 빌더 재구성, 오류 처리 개선 및 검색 저장소 포맷팅 조정을 포함한 백엔드 서비스 개선.

Changes

Cohort / File(s)	Summary
AI 임베딩 서비스 ticketmate-ai/src/main/java/com/ticketmate/backend/ai/application/service/VertexAiEmbeddingService.java	Redis/TTL 캐싱 주석 제거; generateEmbedding에서 ClientException 처리 시 오류 로깅 및 CustomException 명시적 발생; 메서드 호출 포맷팅 조정
임베딩 저장소 ticketmate-ai/src/main/java/com/ticketmate/backend/ai/infrastructure/repository/EmbeddingRepository.java	벡터 유사도 쿼리에 임계값 필터 추가 (≤ 0.90); 콘서트 및 에이전트 임베딩 조회 모두에 적용; 메서드 시그니처 변경 없음
채팅 메시지 서비스 ticketmate-chat/src/main/java/com/ticketmate/backend/chat/application/service/ChatMessageService.java	읽지 않은 메시지 알림용 재사용 가능한 페이로드 빌더 도입; toPreview() 헬퍼 메서드 추가; 메시지 타입별 미리보기 렌더링 로직 구현; LinkedHashMap 임포트 추가
검색 저장소 ticketmate-search/src/main/java/com/ticketmate/backend/search/infrastructure/repository/SearchRepositoryImpl.java	세 개의 쿼리 정의에 걸친 포맷팅 및 들여쓰기 조정; 기능적 변경 없음

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Test #446: 임베딩 캐싱 동작 수정 (주 PR에서 VertexAiEmbeddingService.fetchOrGenerateEmbedding의 캐시 주석 제거, 검색된 PR에서 Redis TTL 변경 및 테스트 추가).
• 20250719 #376 #377 기능추가 공연 대리인 임베딩 기반 하이브리드 검색 구현 #436: 임베딩 생성/조회 및 벡터 유사도 저장소 동작 수정 (서비스 메서드 및 임베딩 저장소 쿼리/캐싱 모두 영향).

Suggested labels

enhancement

Suggested reviewers

• mr6208
• Chuseok22

Poem

🐰 임베딩 캐시를 날려 버리고
유사도 필터로 정교함을 더하며
알림 페이로드를 단정하게 정리한,
뒷발로 도약하는 업데이트 🎯
검색 정렬까지 맞춘 신발끈!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 하이브리드 검색 결과 관련성 필터링 로직 개선이라는 주요 변경사항을 명확하게 요약하고 있으며, 실제 변경 내용(임베딩 모델 변경, 캐싱 제거, 유사도 임계값 추가)과 일치합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

ticketmate-ai/src/main/java/com/ticketmate/backend/ai/application/service/VertexAiEmbeddingService.java (1)

33-43: Javadoc 주석이 실제 구현과 불일치합니다.

Line 36-37의 주석에 "TTL 없이 영구 캐시 적용"이라고 되어 있지만, 캐싱 어노테이션이 제거되어 더 이상 캐시가 적용되지 않습니다. 주석을 현재 동작에 맞게 업데이트해야 합니다.

📝 제안된 수정

  /**
   * Vertex AI 임베딩 요청
   * 검색용 임베딩 생성/조회
-  * TTL 없이 영구 캐시 적용
   *
   * @param targetId      공연/대리인 ID (검색어의 경우 null)

ticketmate-ai/src/main/java/com/ticketmate/backend/ai/infrastructure/repository/EmbeddingRepository.java (1)

26-36: 거리 메트릭 및 임계값 개선 필요 - 코사인 거리로 변경 권장

현재 L2 거리 연산자(<->)를 사용 중이지만, 텍스트 임베딩을 통한 의미적 유사도 검색에서는 코사인 거리(또는 내적)가 표준입니다. L2는 공간 또는 비전 데이터에 적합하며, 의미 검색의 경우 방향성이 중요하므로 코사인 거리로 변경하는 것이 권장됩니다. 또한 현재 하드코딩된 0.90 임계값은 설정 가능하도록 분리하여 모델이나 거리 메트릭 변경 시 유연하게 조정할 수 있도록 개선하기 바랍니다.

변경이 필요한 부분

// lines 28, 47 - 현재 코드
AND (e.embedding_vector <-> CAST(:vector AS vector)) <= 0.90

// 변경 권장
AND (e.embedding_vector <=> CAST(:vector AS vector)) >= 0.80  // 코사인 거리로 변경

🧹 Nitpick comments (2)

ticketmate-ai/src/main/java/com/ticketmate/backend/ai/application/service/VertexAiEmbeddingService.java (1)

19-20: 사용되지 않는 import 제거 필요.

캐싱 기능이 제거되었으므로 @Cacheable과 @Caching import가 더 이상 필요하지 않습니다.

🧹 제안된 수정

-import org.springframework.cache.annotation.Cacheable;
-import org.springframework.cache.annotation.Caching;

ticketmate-chat/src/main/java/com/ticketmate/backend/chat/application/service/ChatMessageService.java (1)

106-137: 동일한 메시지를 중복 조회합니다.

Line 106-107에서 ack.getLastReadMessageId()로 메시지를 조회하고, Line 136-137의 afterCommit() 콜백에서 동일한 ID로 다시 조회합니다. 첫 번째 조회 결과를 변수로 캡처하여 재사용하면 불필요한 DB 조회를 줄일 수 있습니다.

♻️ 제안된 수정

    ChatMessage lastReadMessage = chatMessageRepository.findById(ack.getLastReadMessageId())
      .orElseThrow(() -> new CustomException(ErrorCode.MESSAGE_NOT_FOUND));

    String preview = toPreview(lastReadMessage);
    String formattedSendDate = formattingSendDate(TimeUtil.toLocalDateTime(lastReadMessage.getSendDate()));
+   final String lastReadMessageId = lastReadMessage.getChatMessageId();
+   final UUID senderId = lastReadMessage.getSenderId();
+   final LocalDateTime readDate = TimeUtil.toLocalDateTime(lastReadMessage.getSendDate());

    // ... 중간 코드 생략 ...

    TransactionSynchronizationManager.registerSynchronization(
      new TransactionSynchronization() {
        @Override
        public void afterCommit() {
-         ChatMessage lastMessage = chatMessageRepository.findById(ack.getLastReadMessageId())
-           .orElseThrow(() -> new CustomException(ErrorCode.MESSAGE_NOT_FOUND));

          ReadAckResponse readAckResponse = ReadAckResponse.builder()
            .chatRoomId(chatRoomId)
            .readerId(reader.getMemberId())
-           .senderId(lastMessage.getSenderId())
-           .lastReadMessageId(ack.getLastReadMessageId())
-           .readDate(TimeUtil.toLocalDateTime(lastMessage.getSendDate()))
+           .senderId(senderId)
+           .lastReadMessageId(lastReadMessageId)
+           .readDate(readDate)
            .build();

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6cdb09d and b5d43b3.

📒 Files selected for processing (4)

• ticketmate-ai/src/main/java/com/ticketmate/backend/ai/application/service/VertexAiEmbeddingService.java
• ticketmate-ai/src/main/java/com/ticketmate/backend/ai/infrastructure/repository/EmbeddingRepository.java
• ticketmate-chat/src/main/java/com/ticketmate/backend/chat/application/service/ChatMessageService.java
• ticketmate-search/src/main/java/com/ticketmate/backend/search/infrastructure/repository/SearchRepositoryImpl.java

🧰 Additional context used

🧠 Learnings (1)

📓 Common learnings

Learnt from: Chuseok22
Repo: Team-TicketMate/ticketmate-server PR: 494
File: ticketmate-chat/src/main/java/com/ticketmate/backend/chat/application/service/ChatMessageService.java:236-236
Timestamp: 2025-09-01T01:27:07.177Z
Learning: Chuseok22는 PR 간 충돌을 방지하기 위해 채팅 관련 로직 수정을 PR #493 머지 후로 연기하는 것을 선호한다.

🔇 Additional comments (7)

ticketmate-search/src/main/java/com/ticketmate/backend/search/infrastructure/repository/SearchRepositoryImpl.java (3)

52-90: LGTM!

포맷팅 및 들여쓰기 조정만 포함되어 있으며, 쿼리 로직에는 변경이 없습니다.

---

104-118: LGTM!

대리인 상세 정보 조회 쿼리의 포맷팅 조정입니다. 기능적 변경 없음.

---

131-146: LGTM!

키워드 검색 쿼리의 들여쓰기 조정입니다.

ticketmate-ai/src/main/java/com/ticketmate/backend/ai/application/service/VertexAiEmbeddingService.java (1)

73-84: LGTM! 오류 처리 개선.

ClientException 발생 시 로깅 후 CustomException으로 래핑하여 전파하는 방식이 적절합니다. 호출자가 일관된 예외 타입을 처리할 수 있게 됩니다.

ticketmate-chat/src/main/java/com/ticketmate/backend/chat/application/service/ChatMessageService.java (2)

445-456: LGTM! 재사용 가능한 페이로드 빌더.

LinkedHashMap을 사용하여 키 순서를 보장하는 것이 좋습니다. null 값에 대한 빈 문자열 처리도 적절합니다.

---

459-468: LGTM! 메시지 타입별 미리보기 생성.

모든 ChatMessageType 케이스를 exhaustive하게 처리하여 컴파일 타임에 누락을 방지합니다.

ticketmate-ai/src/main/java/com/ticketmate/backend/ai/infrastructure/repository/EmbeddingRepository.java (1)

45-49: LGTM!

대리인 임베딩에도 동일한 거리 임계값이 적용되어 일관성이 유지됩니다.