🐛 [버그 수정] : 스케줄러 및 메시지 샌딩 오류 수정

[passionryu]

Summary by CodeRabbit

• 신규 기능

  • 뉴스 이력 응답에 피드백 데이터(키워드 반영도, 콘텐츠 품질) 포함.
  • 유효한 설정만 조회하는 기능 추가 및 서비스 반영.

• 버그 수정

  • 설정이 삭제된 경우 발생하던 오류 방지.

• 기능 개선

  • 뉴스 이력 페이지 기본 크기 5 → 3으로 변경.
  • 사용자별 뉴스 추천 로직 개선(설정별 개별 처리로 중복 방지).
  • 스케줄러에서 설정 미존재 시 예외 대신 경고만 출력.

• 문서 및 스타일

  • 불필요한 주석 및 코드 스타일 정리.
  • docker-compose.yml 주석 정렬 개선.

• 환경설정

  • 프로젝트 모듈 및 컴파일러 설정 파일 업데이트.

[coderabbitai]

Walkthrough

이번 변경 사항은 피드백 데이터 통합, 설정 조회 로직 개선, 뉴스 이력 응답 필드 확장, 카카오 메시지 뉴스 검색 방식 수정, 스케줄러 예외 처리 완화, 일부 주석 및 로깅 추가, 프로젝트 설정 파일 수정 등 다양한 영역에 걸쳐 이루어졌습니다. 코드의 핵심 동작 흐름에는 일부 영향이 있습니다.

Changes

파일/경로 요약	변경 내용 요약
.idea/compiler.xml, .idea/modules.xml	모듈명 및 모듈 참조 변경 등 프로젝트 설정 파일 수정
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/service/HotTopicService.java	캐시 조회 전 임시 주석 및 로그 추가
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Kakao/service/KakaoMessageService.java	뉴스 검색 로직을 설정별 개별 검색 방식으로 수정, 기존 메서드 주석 처리 및 교체, 관련 메서드 호출부 변경
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/Repository/SettingRepository.java	유효한 Setting 및 days를 조회하는 새 메서드 추가
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java	전체 Setting 조회 시 삭제되지 않은 유효 데이터만 반환하도록 로직 변경
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/Controller/MoreNewsController.java	뉴스 이력 조회 기본 페이지 사이즈 5 → 3으로 변경, 주석 추가
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/DTO/NewsHistoryResponse.java	Feedback 기반 필드 2개 추가, from 메서드 시그니처 및 구현 변경, 불필요 필드 제거
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/Service/MoreNewsService.java	FeedbackRepository 의존성 추가, groupedNewsHistory 조회 시 Feedback 데이터 통합, 응답 생성 방식 변경
SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/ElasticSearch/configuration/ElasticsearchClientConfig.java	Elasticsearch 로컬 접속용 주석 추가
SpringBoot/src/main/java/Baemin/News_Deliver/Global/Scheduler/SchedulerInitializer.java	설정 미존재 시 예외 발생 대신 경고 로그만 출력하도록 변경
docker-compose.yml	mysqld-exporter 서비스 블록 주석 들여쓰기 스타일 수정(기능 변화 없음)
(merge commit message template)	병합 커밋 메시지 템플릿 추가(코드 변화 없음)

Sequence Diagram(s)

sequenceDiagram
    participant Controller as MoreNewsController
    participant Service as MoreNewsService
    participant Repo as HistoryRepository
    participant FeedbackRepo as FeedbackRepository

    Controller->>Service: getGroupedNewsHistory(page, size)
    Service->>Repo: findAllByUserId(userId)
    Repo-->>Service: List<History>
    Service->>FeedbackRepo: findByHistoryIdIn(List<historyIds>)
    FeedbackRepo-->>Service: List<Feedback>
    Service->>Service: 그룹핑 및 NewsHistoryResponse 생성 (Feedback 포함)
    Service-->>Controller: List<GroupedNewsHistoryResponse>

Loading

sequenceDiagram
    participant KakaoService as KakaoMessageService
    participant SettingRepo as SettingRepository
    participant NewsRepo as NewsRepository

    KakaoService->>SettingRepo: findByUserId(userId)
    SettingRepo-->>KakaoService: List<Setting>
    loop 각 Setting 별
        KakaoService->>NewsRepo: searchNews(keywords, blockKeywords)
        NewsRepo-->>KakaoService: List<NewsEsDocument>
        KakaoService->>KakaoService: 뉴스 5개 제한, 히스토리 저장
    end
    KakaoService-->>Caller: 전체 뉴스 리스트 반환

Loading

Possibly related PRs

• News-Deliver/Server#89: 피드백 API 및 관련 엔티티/레포지토리 구현. 본 PR의 피드백 데이터 통합과 직접적으로 연결됨.
• News-Deliver/Server#101: HotTopicService의 Redis 캐싱 구현. 본 PR의 캐시 관련 로깅 추가와 기능적으로 연관됨.
• News-Deliver/Server#86: MyPage 뉴스 이력 조회, MoreNewsController/Service 초기 구현. 본 PR의 이력 조회 및 피드백 통합과 연결됨.

Suggested reviewers

• daumi125
• moonjun1
• wjkim9

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Gitleaks (8.27.2)

'

/bin/bash: -c: line 1: unexpected EOF while looking for matching `''

✨ Finishing Touches

• 📝 Generate Docstrings

---

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Explain this complex logic.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai explain this code block.
  • @coderabbitai modularize this function.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read src/utils.ts and explain its main purpose.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
  • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai full review to do a full review from scratch and review all the files again.
• @coderabbitai summary to regenerate the summary of the PR.
• @coderabbitai generate docstrings to generate docstrings for this PR.
• @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
• @coderabbitai help to get help.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• 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

Documentation and Community

• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[coderabbitai]

Actionable comments posted: 3

🔭 Outside diff range comments (1)

.idea/compiler.xml (1)

10-18: 모듈 이름 불일치 가능성

annotationProcessing 프로필은 Baemin.News-Deliver.main으로 변경했지만, 같은 파일의 JavacSettings 섹션(16–18행)에는

• Baemin.News-Deliver
• Baemin.News-Deliver.main
• Baemin.News-Deliver.test

세 가지 모듈이 동시에 지정돼 있습니다. modules.xml에서 Baemin.News-Deliver.test.iml가 제거되었다면 이 섹션도 함께 정리하지 않으면 IDE가 존재하지 않는 모듈에 대한 컴파일 옵션을 시도해 경고가 발생합니다. 필요 없는 항목은 삭제하거나 실제 모듈 파일을 유지할지 결정해 주세요.

🧹 Nitpick comments (5)

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/service/HotTopicService.java (1)

85-86: 디버깅 로그는 DEBUG 레벨 사용 권장

타임스탬프 계산을 추적하기 위한 로그는 유용하지만, 프로덕션 환경에서는 INFO 레벨보다 DEBUG 레벨이 더 적절합니다.

다음과 같이 변경하는 것을 권장합니다:

-        log.info("startOfYesterday : {}", startOfYesterday);
-        log.info("endOfYesterday : {}", endOfYesterday);
+        log.debug("startOfYesterday : {}", startOfYesterday);
+        log.debug("endOfYesterday : {}", endOfYesterday);

또는 스케줄러 관련 버그 수정이 완료되면 이 로그들을 제거하는 것도 고려해보세요.

docker-compose.yml (1)

155-165: mysqld-exporter 계속 주석 처리할지 검토 필요

모니터링 지표 수집을 위해 mysqld-exporter가 필요한 경우가 많습니다.
현재 완전히 비활성화돼 있는데,

1. 실제 운영/스테이징 환경에서 사용하지 않는지,
2. 불필요하다면 파일에서 완전히 제거할지
   검토 부탁드립니다.

.idea/modules.xml (1)

5-5: IDE 전용 설정 파일은 VCS 제외 고려

.idea 하위 XML은 개발자별 경로·플러그인 상태에 민감해 잦은 충돌을 유발합니다. 팀원 간 환경이 동일하지 않다면 .gitignore에 추가해 버전 관리 대상에서 제외하는 방안을 재검토해 보시길 권장드립니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Global/Scheduler/SchedulerInitializer.java (1)

64-68: 스케줄러 예외 처리를 완화하여 안정성을 높였습니다.

설정이 없을 때 예외를 발생시키는 대신 경고 로그만 출력하도록 변경한 것은 좋은 개선입니다. 하지만 빈 리스트에 대한 처리를 더 명확히 할 수 있습니다.

다음과 같이 early return을 추가하는 것을 고려해보세요:

 if (settings == null || settings.isEmpty()) {
     log.warn("[SchedulerInit] 등록할 Setting이 없어 스케줄러를 실행하지 않습니다.");
     // throw new KakaoException(ErrorCode.SETTING_NOT_FOUND);
+    return;
 }

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Kakao/service/KakaoMessageService.java (1)

258-303: 사용하지 않는 코드 제거 권장

주석 처리된 이전 구현은 Git 히스토리에서 확인할 수 있으므로 제거하는 것이 코드 가독성 향상에 도움이 됩니다.

버전 관리 시스템에서 이전 코드를 확인할 수 있으므로, 주석 처리된 getNewsEsDocumentList 메서드 전체를 삭제하시기 바랍니다.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f053e64 and dd7505b.

📒 Files selected for processing (13)

• ' (1 hunks)
• .idea/compiler.xml (1 hunks)
• .idea/modules.xml (1 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/service/HotTopicService.java (2 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Kakao/service/KakaoMessageService.java (4 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/Repository/SettingRepository.java (1 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java (1 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/Controller/MoreNewsController.java (1 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/DTO/NewsHistoryResponse.java (2 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/Service/MoreNewsService.java (3 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/ElasticSearch/configuration/ElasticsearchClientConfig.java (1 hunks)
• SpringBoot/src/main/java/Baemin/News_Deliver/Global/Scheduler/SchedulerInitializer.java (1 hunks)
• docker-compose.yml (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: test

🔇 Additional comments (11)

docker-compose.yml (1)

155-165: 들여쓰기 정렬로 가독성 향상 완료

주석 블록의 들여쓰기를 다른 서비스와 맞춰 YAML 구조를 일관성 있게 유지했습니다. 기능적 변화는 없으며, 가독성이 향상되었습니다.

.idea/modules.xml (1)

5-5: 모듈 통합이 실제 프로젝트 설정과 모두 일치하는지 확인 필요

Baemin.News-Deliver.main.iml 하나로 통합한 의도는 이해됩니다. 다만

1. Gradle settings.gradle(또는 settings.gradle.kts)의 include 항목,
2. CI 설정(예: build.gradle 모듈 dependency),
3. 다른 IDE 설정 파일(*.iml, .idea/compiler.xml 등)

모두에서 동일한 모듈명이 사용되고 있는지 점검해 주세요. 불일치 시 IDE import 충돌이나 Gradle sync 오류가 발생할 수 있습니다.

.idea/compiler.xml (1)

10-10: SpringBoot 모듈 레거시 참조 여부 확인

modules.xml에는 여전히 SpringBoot.News-Deliver.main.iml 엔트리가 남아있습니다. 새로 바뀐 Baemin 네임스페이스와 중복되거나 불필요한 레거시 설정일 수 있으므로, 실제 코드·Gradle 모듈이 사용 중인지 다시 한 번 확인해 주세요.

' (1)

1-7: 머지 커밋 메시지 템플릿입니다.

이 파일은 Git 머지 프로세스의 일부인 커밋 메시지 템플릿으로, 실제 코드 변경사항이 아니므로 리뷰가 필요하지 않습니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/Repository/SettingRepository.java (1)

51-66: 유효한 설정 조회 메서드가 올바르게 구현되었습니다.

새로운 findAllValidSettingsWithDays 메서드는 다음과 같은 장점이 있습니다:

• JOIN FETCH를 사용하여 N+1 문제를 방지
• DISTINCT로 조인으로 인한 중복 제거
• 삭제된 설정과 만료된 설정을 적절히 필터링
• NPE 방지를 위한 적절한 조건 처리

스케줄러 오류 수정에 핵심적인 역할을 하는 개선사항입니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java (1)

254-272: 스케줄러 NPE 오류를 효과적으로 수정했습니다.

getAllSettings() 메서드의 변경사항이 올바릅니다:

• 삭제된 설정으로 인한 NPE 문제 해결
• 유효한 설정만 조회하여 스케줄러 안정성 향상
• 상세한 주석으로 변경 이유와 배경을 명확히 설명

버그 수정의 핵심 부분이며, 스케줄러 오류를 근본적으로 해결하는 접근법입니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/Controller/MoreNewsController.java (2)

65-65: 페이지 크기 변경에 대한 확인이 필요합니다.

기본 페이지 크기를 5에서 3으로 변경했습니다. 이 변경이 UI/UX 요구사항에 따른 것인지 확인이 필요합니다.

---

67-67: 서비스 레이어 호출에 대한 주석 추가가 좋습니다.

코드 가독성 향상을 위한 주석 추가는 좋은 개선사항입니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/Service/MoreNewsService.java (1)

171-178: 효율적인 피드백 데이터 조회 구현

히스토리 ID를 수집하여 피드백 데이터를 일괄 조회하는 방식은 N+1 쿼리 문제를 방지하는 좋은 접근입니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/SubServices/MoreNews/DTO/NewsHistoryResponse.java (1)

31-32: 적절한 null 처리 구현

피드백이 없는 경우를 고려한 null 체크 구현이 적절합니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Kakao/service/KakaoMessageService.java (1)

145-178: 설정별 뉴스 검색 로직 개선 확인

각 설정을 개별적으로 처리하여 키워드가 섞이는 문제를 해결한 것이 적절합니다. 설정당 최대 5개 뉴스로 제한하는 것도 합리적입니다.

[coderabbitai]

🛠️ Refactor suggestion

임시 해결책 대신 근본 원인 해결 필요

임시 주석으로 캐싱 문제를 해결하는 것보다는 근본 원인을 파악하고 적절한 해결책을 적용하는 것이 좋겠습니다. 현재 주석만으로는 어떤 캐싱 문제가 발생했는지 명확하지 않습니다.

다음과 같은 조치를 권장합니다:

• 캐싱 문제의 구체적인 원인 분석
• 이슈 트래커에 해당 문제 등록
• 임시 해결책에 대한 명확한 제거 일정 수립

이 임시 해결책을 적절한 수정으로 교체하는 코드를 생성해드릴까요?

🤖 Prompt for AI Agents

In
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/service/HotTopicService.java
at line 74, there is a temporary comment disabling caching to avoid an issue.
Instead of leaving this temporary comment, investigate and identify the root
cause of the caching problem. Remove the temporary comment and implement a
proper fix for the caching issue. Additionally, document the problem and fix in
the issue tracker and establish a clear timeline for removing any temporary
workarounds.

[coderabbitai]

⚠️ Potential issue

문법 오류 수정 필요

예외를 던지지 않고 세미콜론이 중복되어 있습니다.

-if (newsList == null) new KakaoException(ErrorCode.NO_NEWS_DATA);;
+if (newsList == null) throw new KakaoException(ErrorCode.NO_NEWS_DATA);

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	if (newsList == null) new KakaoException(ErrorCode.NO_NEWS_DATA);;
	if (newsList == null) throw new KakaoException(ErrorCode.NO_NEWS_DATA);

🤖 Prompt for AI Agents

In
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Kakao/service/KakaoMessageService.java
at line 102, the code incorrectly creates a new KakaoException without throwing
it and has an extra semicolon. Fix this by adding the 'throw' keyword before
'new KakaoException(ErrorCode.NO_NEWS_DATA)' and remove the redundant semicolon
to properly throw the exception.

[coderabbitai]

💡 Verification agent

❓ Verification inconclusive

호스트/포트 하드코딩 대신 환경 설정으로 외부화하세요
개발(로컬)‧운영 환경마다 주석을 켜고 끄는 방식은 실수 위험이 높고 CI 파이프라인에서 자동화가 어렵습니다. application-*.yml 프로파일이나 @Value 주입으로 호스트·포트를 외부화하면 깔끔하게 해결됩니다.

예시 diff:

+@Value("${elasticsearch.host:elasticsearch}")
+private String esHost;
+
+@Value("${elasticsearch.port:9200}")
+private int esPort;
...
-RestClient restClient = RestClient.builder(
-        new HttpHost("elasticsearch", 9200)
-        // 성열 로컬용 임시 Host
-        //new HttpHost("localhost", 9200)
-).build();
+RestClient restClient = RestClient.builder(
+        new HttpHost(esHost, esPort)
+).build();

이렇게 하면 프로파일별 application-dev.yml, application-prod.yml에서 값만 바꿔주면 되므로 소스 코드 변경·커밋 없이 환경 전환이 가능합니다.

---

호스트/포트 하드코딩을 제거하고 설정 파일로 외부화하세요

현재 ElasticsearchClientConfig.java에 new HttpHost("elasticsearch", 9200)처럼 하드코딩된 호스트와 포트를 사용 중입니다. 개발(로컬)·운영 환경 전환 시마다 주석을 켜고 끄는 방식은 오류를 유발하기 쉽고, CI/CD 자동화에도 적합하지 않습니다.

다음과 같이 @Value를 이용해 application-*.yml 프로파일에서 값을 주입 받도록 변경하세요:

// 클래스 필드에 추가
+    @Value("${elasticsearch.host:elasticsearch}")
+    private String esHost;
+
+    @Value("${elasticsearch.port:9200}")
+    private int esPort;

    @Bean
    public RestClient elasticsearchClient() {
-       RestClient restClient = RestClient.builder(
-           new HttpHost("elasticsearch", 9200)
-           // 성열 로컬용 임시 Host
-           //new HttpHost("localhost", 9200)
-       ).build();
+       RestClient restClient = RestClient.builder(
+           new HttpHost(esHost, esPort)
+       ).build();
        return restClient;
    }

– application-dev.yml, application-prod.yml 등 프로파일별 설정 파일에 elasticsearch.host와 elasticsearch.port만 정의하면 코드 변경 없이 환경 전환이 가능합니다.
– CI/CD 파이프라인에서도 별도 스크립트 수정 없이 자동 배포가 용이해집니다.

🤖 Prompt for AI Agents

In
SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/ElasticSearch/configuration/ElasticsearchClientConfig.java
around lines 50 to 53, the Elasticsearch host and port are hardcoded, which is
error-prone and not suitable for environment switching or CI/CD automation.
Refactor the code to use @Value annotations to inject the host and port values
from the application-*.yml configuration files (e.g., application-dev.yml,
application-prod.yml). Define elasticsearch.host and elasticsearch.port in these
profile-specific YAML files so the code reads these values dynamically without
needing manual changes for different environments.