Skip to content

feat: 지역상생 일정 추천 구현 - #19

Merged
Heyaaz merged 3 commits into
mainfrom
feature/regional-recommendation-sync
May 20, 2026
Merged

feat: 지역상생 일정 추천 구현#19
Heyaaz merged 3 commits into
mainfrom
feature/regional-recommendation-sync

Conversation

@Heyaaz

@Heyaaz Heyaaz commented May 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • 지역상생 기반 일정 추천 및 외부 인기도 지표 연동

Changes

  • Google Places·Naver DataLab 기반 외부 인기도 수집 클라이언트 및 배치 서비스 추가
  • 지역상생 추천 점수 반영, 장소 사진/상세 응답 확장
  • 여행방 날짜 범위 저장 및 일정 응답 매핑 보강
  • 외부 인기도 지표·여행방 날짜 마이그레이션 추가
  • 후보 장소 부족 시 옵션 간 후보 재사용 fallback 처리
  • 단일 일정 내부 중복 장소 선택 방지 및 LLM 중복 선택 검증 추가
  • 합의 엔진, LLM 응답 검증, 일정 응답 매퍼 테스트 보강

Verification

  • Not run
  • Build
  • Test: ./gradlew test --tests 'com.tripsync.application.consensus.ConsensusServiceTest' --no-daemon
  • Test: ./gradlew test --tests 'com.tripsync.application.consensus.ConsensusServiceTest' --tests 'com.tripsync.infrastructure.llm.OpenAiClientTest' --no-daemon
  • Test: ./gradlew cleanTest test --no-daemon
  • Manual

Notes

@Heyaaz Heyaaz left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nogada PR Review

Verdict: ✅ No blocking issues found.

Checked scope:

  • 지역상생 추천을 위한 외부 인기도 지표 수집/저장 흐름
  • 일정 후보 필터링, 지역/숨은명소/인기도 기반 랭킹, 슬롯 중복 방지 로직
  • 다일자 여행방 날짜 범위 추가 및 응답 매핑 변경
  • 공개 사진 프록시 엔드포인트와 SecurityConfig 허용 범위
  • 신규/변경 테스트 및 DB migration

Verification:

  • ./gradlew clean test --no-daemon --no-build-cache 성공
  • Test XML summary: 59 tests, 0 failures, 0 errors, 0 skipped
  • git diff --check origin/main...pr-19에서 V9__add_trip_room_date_range.sql EOF blank line 1건만 확인됨. 동작 영향 없는 포맷 이슈라 non-blocking으로 봤습니다.

Notes:

  • Testcontainers 종료 시점의 Hikari/PostgreSQL shutdown warning은 있었지만 Gradle 빌드와 XML 결과는 성공입니다.

[Heyaaz의 Nogada]

@Heyaaz
Heyaaz merged commit e27789d into main May 20, 2026
1 check passed
@Heyaaz
Heyaaz deleted the feature/regional-recommendation-sync branch May 20, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant