refactor: 채팅 도메인 응답의 사용자 관련 id를 siteUserId로 통일

[whqtker]

관련 이슈

• resolves: refactor: 채팅 도메인 응답의 사용자 id를 siteUserId로 통일한다. #662

작업 내용

특이 사항

리뷰 요구사항 (선택)

[coderabbitai]

Walkthrough

채팅 관련 DTO와 서비스의 식별자 필드명을 일관되게 변경하는 리팩토링이 진행되었습니다.

1. ChatMessageResponse와 ChatParticipantResponse 업데이트

   • 기존 senderId/partnerId 필드를 siteUserId로 이름 변경
   • 정적 팩토리 메서드의 매개변수명도 함께 수정

2. ChatMessageSendResponse 메서드 시그니처 변경

   • from(ChatMessage) 팩토리 메서드를 of(ChatMessage, long siteUserId)로 교체
   • 외부에서 siteUserId를 명시적으로 제공하도록 변경

3. ChatService 호출 지점 업데이트

   • 기존 ChatMessageSendResponse.from(chatMessage) 호출을 새로운 메서드 시그니처에 맞게 변경
   • 총 2개 호출 지점 수정

4. 테스트 코드 동기화

   • 테스트에서 참조하는 게터명을 partnerId()/senderId()에서 siteUserId()로 변경
   • 총 7개 라인의 어설션 업데이트

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested reviewers

• wibaek
• Hexeong
• JAEHEE25
• lsy1307
• sukangpunch
• Gyuhyeok99

🚥 Pre-merge checks | ✅ 3 | ❌ 3

❌ Failed checks (3 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	PR 설명이 템플릿만 있고 작업 내용, 특이 사항, 리뷰 요구사항의 구체적인 상세 내용이 모두 미기재되어 있습니다.	PR 본문의 '작업 내용', '특이 사항', '리뷰 요구사항' 섹션에 구체적인 설명을 추가하여 변경사항과 영향범위를 명확히 기재해주시기 바랍니다.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection	⚠️ Warning	❌ Merge conflicts detected (15 files): ⚔️ src/main/java/com/example/solidconnection/chat/dto/ChatMessageResponse.java (content) ⚔️ src/main/java/com/example/solidconnection/chat/dto/ChatMessageSendResponse.java (content) ⚔️ src/main/java/com/example/solidconnection/chat/dto/ChatParticipantResponse.java (content) ⚔️ src/main/java/com/example/solidconnection/chat/service/ChatService.java (content) ⚔️ src/main/java/com/example/solidconnection/mentor/controller/MentorController.java (content) ⚔️ src/main/java/com/example/solidconnection/mentor/dto/MatchedMentorResponse.java (content) ⚔️ src/main/java/com/example/solidconnection/mentor/dto/MentorDetailResponse.java (content) ⚔️ src/main/java/com/example/solidconnection/mentor/dto/MentorMyPageResponse.java (content) ⚔️ src/main/java/com/example/solidconnection/mentor/dto/MentorPreviewResponse.java (content) ⚔️ src/main/java/com/example/solidconnection/mentor/service/MentorQueryService.java (content) ⚔️ src/main/java/com/example/solidconnection/mentor/service/MentoringCommandService.java (content) ⚔️ src/test/java/com/example/solidconnection/chat/service/ChatServiceTest.java (content) ⚔️ src/test/java/com/example/solidconnection/mentor/service/MentorQueryServiceTest.java (content) ⚔️ src/test/java/com/example/solidconnection/mentor/service/MentoringCommandServiceTest.java (content) ⚔️ src/test/java/com/example/solidconnection/mentor/service/MentoringQueryServiceTest.java (content) These conflicts must be resolved before merging into develop.	Resolve conflicts locally and push changes to this branch.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목은 변경사항의 핵심을 명확하게 설명하고 있습니다: 채팅 도메인 응답의 사용자 관련 id를 siteUserId로 통일하는 리팩터링입니다.
Linked Issues check	✅ Passed	코드 변경사항은 이슈 #662의 목표인 채팅 도메인 응답의 사용자 id 필드명 통일(partnerId/senderId → siteUserId)을 올바르게 구현했습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 이슈 #662의 범위 내에서 이루어졌으며, 채팅 도메인 DTO와 서비스 레이어의 필드명 통일 및 테스트 업데이트만 포함되어 있습니다.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

⚔️ Resolve merge conflicts (beta)

• Auto-commit resolved conflicts to branch refactor/662-refactor-chat-response-related-site-user-id
• Post resolved changes as copyable diffs in a comment

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

src/main/java/com/example/solidconnection/chat/service/ChatService.java (1)

143-157: 내부 변수명도 siteUserId로 통일하면 더 일관적입니다.

buildChatMessageResponses에서 사용하는 externalSenderId (Line 153-154)와 toChatMessageResponse의 파라미터 externalSenderId (Line 187)를 siteUserId로 변경하면 이번 리팩터링의 취지에 더 부합합니다. 물론 private 메서드라 급하진 않습니다. 😊

♻️ 제안하는 변경

                     if (senderParticipant == null) {
                         throw new CustomException(CHAT_PARTICIPANT_NOT_FOUND);
                     }
-                    long externalSenderId = senderParticipant.getSiteUserId();
-                    return toChatMessageResponse(message, externalSenderId);
+                    long siteUserId = senderParticipant.getSiteUserId();
+                    return toChatMessageResponse(message, siteUserId);

-    private ChatMessageResponse toChatMessageResponse(ChatMessage message, long externalSenderId) {
+    private ChatMessageResponse toChatMessageResponse(ChatMessage message, long siteUserId) {

         return ChatMessageResponse.of(
                 message.getId(),
                 message.getContent(),
-                externalSenderId,
+                siteUserId,
                 message.getCreatedAt(),
                 attachments
         );

---

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.}

[whqtker]

@coderabbitai
리뷰

[coderabbitai]

@whqtker, PR 변경사항을 리뷰하겠습니다. 채팅 도메인 응답의 사용자 관련 id를 siteUserId로 통일하는 리팩토링 작업을 검토하겠습니다.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.