비동기 메서드에서 엔티티 객체를 직접 조회하도록 수정하여 트랜잭션 반영 문제 해결

[Hanjaemo]

💡 작업 내용

• summary 메서드의 파라미터를 TalkPick에서 TalkPickId로 변경

💡 자세한 설명

톡픽 생성과 요약은 서로 다른 트랜잭션에서 실행됩니다.
톡픽 생성 트랜잭션에서 가져온 TalkPick 엔티티는 새로운 트랜잭션(톡픽 요약)에서 detached 상태가 되어 영속성 컨텍스트에 의해 관리되지 않습니다.
이로 인해 JPA가 해당 엔티티의 변경을 추적하지 못해 데이터베이스에 반영되지 않는 문제가 발생했습니다.

이를 해결하기 위해 summary 메서드가 TalkPickId를 받아 비동기 트랜잭션 내부에서 엔티티를 조회하도록 수정했습니다.
이 방식으로 summary 메서드 내부에서 조회된 TalkPick 엔티티는 새로운 트랜잭션의 영속성 컨텍스트에 포함되어 변경 사항이 올바르게 반영됩니다.

✅ 셀프 체크리스트

• PR 제목을 형식에 맞게 작성했나요?
• 브랜치 전략에 맞는 브랜치에 PR을 올리고 있나요?
• 이슈는 close 했나요?
• Reviewers, Labels, Projects를 등록했나요?
• 작업 도중 문서 수정이 필요한 경우 잘 수정했나요?
• 테스트는 잘 통과했나요?
• 불필요한 코드는 제거했나요?

closes #836

Summary by CodeRabbit

• 리팩토링
  • 토크픽 요약 로직의 입력 방식 변경
  • 토크픽 ID를 사용하여 요약 프로세스 개선
  • 레포지토리를 통한 토크픽 데이터 조회 메커니즘 업데이트

[coderabbitai]

워크스루

TalkPickService와 TalkPickSummarizer 클래스에서 요약 메서드의 입력 방식이 변경되었습니다. 기존에는 TalkPick 객체를 직접 전달하던 방식에서 Long 타입의 ID를 전달하는 방식으로 수정되었습니다. 이 변경으로 인해 요약 생성 시 리포지토리를 통해 객체를 먼저 조회한 후 요약 로직을 수행하는 방식으로 변경되었습니다.

변경 사항

파일	변경 요약
src/main/java/balancetalk/talkpick/application/TalkPickService.java	createTalkPick와 updateTalkPick 메서드에서 talkPickSummarizer.summary 호출 시 TalkPick 대신 Long ID 전달로 변경
src/main/java/balancetalk/talkpick/domain/TalkPickSummarizer.java	summary 메서드 시그니처 변경: TalkPick → Long 새로운 talkPickRepository 필드 추가

연결된 이슈에 대한 평가

목표	해결 여부	설명
요약 상태 변경 문제 해결 [#836]	❓	트랜잭션 관련 문제 해결 여부를 명확히 확인하기 어려움

관련될 수 있는 PR

• 변경된 요구사항에 따른 톡픽 본문 요약 기능 구현 #829: 요약 로직 통합과 관련된 변경 사항

제안된 라벨

⚙️ chore, 🪄 refactor

제안된 리뷰어

• gywns0417
• jschoi-96

시 (Poem)

🐰 토끼의 요약 마법
ID로 요약을 부르니
데이터는 춤을 추네
리포지토리의 마법사여
요약은 계속되리! 🌟

---

📜 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 33e2d82 and bea0f51.

📒 Files selected for processing (2)

• src/main/java/balancetalk/talkpick/application/TalkPickService.java (2 hunks)
• src/main/java/balancetalk/talkpick/domain/TalkPickSummarizer.java (2 hunks)

🔇 Additional comments (8)

src/main/java/balancetalk/talkpick/domain/TalkPickSummarizer.java (6)

3-3: [임포트된 상수 사용에 대한 의견]
새로 추가된 NOT_FOUND_TALK_PICK 등의 상수를 사용하는 것은 분명한 의도를 전달하기에 좋습니다. 각 예외에 맞는 상수를 잘 정의했으니 유지보수성이 향상될 것으로 보입니다.

---

11-11: [새로운 저장소 연결]
TalkPickRepository 임포트를 통해 도메인 계층에서 직접 엔티티를 조회하도록 수정된 점이 트랜잭션 반영 이슈 해소에 유효해 보입니다.

---

56-56: [저장소 필드 추가]
talkPickRepository 필드가 추가되어, 요약 로직에서 직접 엔티티 로드를 수행할 수 있게 한 것은 의도대로 적절합니다.

---

61-61: [요약 메서드 시그니처 변경]
summary 메서드가 TalkPick 대신 Long talkPickId를 받도록 변경되었습니다. 이로 인해 비동기 트랜잭션에서 실제 엔티티를 다시 조회할 수 있게 되어, 영속성 컨텍스트에서 분리된 문제를 해결하는 데 적합한 접근이라 보입니다.

---

62-63: [엔티티 직접 재조회 로직 확인]
talkPickRepository.findById(talkPickId)를 통해 실제 엔티티를 조회하고 예외를 던지는 구현이 문제없이 동작할 것으로 보입니다. 다만, 추후 필요 시 캐싱 혹은 더 나은 예외 처리를 고려해볼 수도 있겠습니다.

---

64-64: [빈 줄 변경]
이 라인은 의미 있는 변경사항이 없어 보입니다.

src/main/java/balancetalk/talkpick/application/TalkPickService.java (2)

46-46: [엔티티 ID 기반 요약 호출]
summary(savedTalkPickId)로 변경된 호출이 TalkPickSummarizer의 수정된 시그니처와 호환되어 트랜잭션 반영 이슈를 해결하는 데 도움이 됩니다. 업무 흐름상으로도 깔끔하므로 적절한 수정입니다.

---

90-90: [업데이트 로직 내 비동기 요약 호출]
summary(talkPickId)를 통해 업데이트 후에도 동일한 방식으로 요약을 진행합니다. 변경된 구조에 잘 맞춰 반영되었으므로 문제 없어 보입니다.

---

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

🪧 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.
  • Generate unit testing code for this file.
  • 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 generate unit testing code for this file.
  • @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 generate unit testing code.
  • @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.

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. (Beta)
• @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.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud