🚑️ [배포 후 긴급 수정] : 메시지 전송 안되는 문제 해결

[passionryu]

Summary by CodeRabbit

• 신규 기능

  • 설정 ID로 단일 설정 정보를 안전하게 조회할 수 있는 기능이 추가되었습니다.
  • 설정 조회 시 연관된 키워드, 차단 키워드, 요일 정보를 함께 불러오는 기능이 추가되었습니다.

• 버그 수정

  • 카카오 메시지 발송 시 사용자의 설정별로 현재 시간(분 단위)에 맞는 설정만 필터링하여 메시지를 전송하도록 개선되었습니다.
  • 각 설정별로 최대 5개의 뉴스를 개별적으로 조회하고, 이력 저장 시 중복을 방지하도록 수정되었습니다.

• 리팩터링/스타일

  • 코드 내 주석 및 포맷이 개선되어 가독성이 향상되었습니다.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

이 변경사항은 카카오 메시지 발송 로직에서 사용자 설정을 현재 시간(분 단위) 기준으로 필터링하여 해당 시간에만 메시지를 보내도록 수정하였고, 각 설정별로 뉴스 검색 및 히스토리 저장을 개별적으로 처리하는 방식으로 전환하였습니다. 또한, 설정 엔티티 조회 시 연관 컬렉션을 fetch join으로 즉시 로딩하는 리포지토리 메서드와, 단일 설정을 예외 처리와 함께 조회하는 서비스 메서드가 추가되었습니다. 스케줄러 관련 클래스에는 주로 주석 및 포매팅 개선이 이루어졌습니다.

Changes

파일/경로 요약	변경 요약
.../Kakao/service/KakaoMessageService.java	- 현재 시간(분 단위)로 사용자 설정 필터링 - 설정별 뉴스 개별 조회 및 히스토리 저장(getNewsEsDocumentList_Fixed 신설) - 메시지 템플릿 ID 변경 - null 처리 및 중복 히스토리 방지 로직 추가 - 기존 방식 주석 처리 및 포매팅 개선
.../Mypage/Repository/SettingRepository.java	- 연관 컬렉션(days, keywords, blockKeywords)을 fetch join으로 즉시 로딩하는 findByIdWithAllAssociations 메서드 추가
.../Mypage/service/SettingService.java	- 단일 설정을 예외 처리와 함께 DTO로 반환하는 getSettingById(Long id) 메서드 추가
.../Global/Scheduler/TaskSchedulerService.java	- 주석 및 포매팅 개선 - fetch join 관련 대안 코드 주석 추가 - 실제 로직 변화 없음

Sequence Diagram(s)

sequenceDiagram
    participant Scheduler
    participant KakaoMessageService
    participant SettingRepository
    participant NewsService
    participant HistoryService

    Scheduler->>KakaoMessageService: sendKakaoMessage(token, userId)
    KakaoMessageService->>SettingRepository: findAllByUserId(userId)
    SettingRepository-->>KakaoMessageService: List<Setting>
    KakaoMessageService->>KakaoMessageService: 현재 시간(분) 기준 설정 필터링
    alt 설정 있음
        loop 각 설정별
            KakaoMessageService->>NewsService: getNewsEsDocumentList_Fixed(setting)
            NewsService-->>KakaoMessageService: List<News>
            KakaoMessageService->>HistoryService: saveHistory(setting, news)
            KakaoMessageService->>KakaoMessageService: sendSingleKakaoMessage(...)
        end
    else 설정 없음
        KakaoMessageService->>Scheduler: false 반환 (로그 기록)
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~18 minutes

Possibly related PRs

• News-Deliver/Server#116: KakaoMessageService에서 뉴스 개별 조회 메서드로 교체하며 메시지 발송 문제를 해결하는 직접적 코드 레벨 연관이 있습니다.
• News-Deliver/Server#132: sendKakaoMessage가 여러 사용자 설정을 개별적으로 처리하도록 변경된 점에서 코드 레벨에서 직접적으로 관련되어 있습니다.

Suggested reviewers

• daumi125
• moonjun1

Note

⚡️ Unit Test Generation is now available in beta!

Learn more here, or try it out under "Finishing Touches" below.

---

📜 Recent review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between a1a14ac and d0ae3cc.

📒 Files selected for processing (4)

• SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Kakao/service/KakaoMessageService.java (11 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/Global/Scheduler/TaskSchedulerService.java (6 hunks)

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/message

---

🪧 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 generate unit tests to generate unit tests for 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.