[Fix] 친구 추가시 알림 버그 수정

[david-parkk]

#️⃣ 연관된 이슈

closes #377

📝작업 내용

• 친구 추가시 알림 버그 수정
• 작업2
• 작업3

작업 상세 내용

상세 내용을 입력해주세요.

💬리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

Summary by CodeRabbit

릴리스 노트

• 버그 수정
  • 친구 요청 알람 알림이 보낸 사람과 받는 사람의 정보를 모두 포함하도록 개선되었습니다. 이를 통해 알람 알림 기능이 더욱 정확하게 작동합니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

친구 요청 알림 기능에서 수신자(toUser) 정보만 전달하던 것을 발신자(fromUser)와 함께 전달하도록 수정했습니다. 이에 따라 AlarmService의 notifyAlarm 메서드 시그니처가 단일 User 파라미터에서 두 개의 User 파라미터로 변경되었습니다.

Changes

Cohort / File(s)	변경 사항
친구 요청 알림 알람 서비스 통합 src/main/java/ku_rum/backend/domain/friend/application/FriendManageService.java	notifyAlarm 메서드 호출 시 수신자(toUser) 외에 발신자(fromUser)를 추가 파라미터로 전달하도록 수정

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• [Feat] 알림 추가 & 키워드 관련 API #337: 이 변경사항의 기반이 되는 AlarmService 및 AlarmMessageHandler 추가와 User 파라미터를 받는 notifyAlarm 메서드 노출

Suggested reviewers

• kmw10693

Poem

🐰 친구 요청에 알림이 솟아나고,
발신자, 수신자 모두 담아서,
버그는 사라지고 알람이 반짝,
우리의 메시지는 이제 완벽해! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	PR 설명이 템플릿 구조는 따르지만, '작업 상세 내용' 섹션이 공란으로 남아있어 변경사항에 대한 자세한 설명이 부족합니다.	'작업 상세 내용' 섹션에 알림 버그 수정 내용(fromUser 파라미터 추가 등)을 구체적으로 작성해주세요.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 친구 추가시 알림 버그 수정에 관한 변경사항을 명확하게 요약하고 있으며, 실제 코드 변경과 일치합니다.
Linked Issues check	✅ Passed	코드 변경(notifyAlarm 메서드 호출시 fromUser 파라미터 추가)이 issue #377의 알림 엔티티 생성 문제 해결 목표와 일치합니다.
Out of Scope Changes check	✅ Passed	모든 코드 변경이 issue #377의 친구 요청시 알림 생성 버그 수정 범위 내에 있으며, 범위를 벗어난 변경사항은 없습니다.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dfc32aa and 5860413.

📒 Files selected for processing (1)

• src/main/java/ku_rum/backend/domain/friend/application/FriendManageService.java

⏰ 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: build

🔇 Additional comments (1)

src/main/java/ku_rum/backend/domain/friend/application/FriendManageService.java (1)

44-44: 알림 버그 수정이 올바르게 적용되었습니다.

친구 요청 발신자(fromUser)와 수신자(toUser)를 정확한 순서로 전달하여 Entity must not be null 예외를 해결했습니다. FriendRequestHandler는 첫 번째 매개변수(object)를 발신자로, 두 번째 매개변수(user)를 알림 수신자로 사용하므로 현재 구현이 의도와 일치합니다.

---

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

[kmw10693]

빠른 버그 해결 감사합니다!