version 2.16.3

[rlagkswn00]

Summary by CodeRabbit

릴리스 노트

• Refactor
  • 공지사항 메시지 데이터 처리 로직을 개선하여 시스템의 데이터 무결성과 안정성을 강화했습니다.

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

[coderabbitai]

Walkthrough

NoticeMessageDto 클래스에 Long 타입의 id 필드를 추가했습니다. Builder 생성자를 업데이트하여 id 매개변수를 포함하고 널 체크를 확장했습니다. from(Notice)와 from(DepartmentNotice) 팩토리 메서드를 수정하여 id를 빌더 체인에 추가했습니다.

Changes

Cohort / File(s)	변경 요약
DTO 필드 추가 및 빌더 업데이트 src/main/java/com/kustacks/kuring/message/application/port/out/dto/NoticeMessageDto.java	Long 타입의 id 필드 추가, Builder 생성자의 첫 번째 매개변수로 id 포함, id 널 체크 추가, from(Notice) 및 from(DepartmentNotice) 팩토리 메서드에서 id 전달 구현

Possibly related PRs

• Fix : 공지사항 FCM 알림 DTO에 id 추가 #333: NoticeMessageDto 클래스에 Long id 필드를 추가하고 Builder 생성자와 팩토리 메서드를 동일하게 업데이트한 동일한 변경사항입니다.

Suggested labels

🐛 Bug

Poem

🐰 번호 하나가 누락되었네,
메시지에도 신분증이 필요해!
id를 추가하니 완전해졌고,
빌더도 야무지게 정렬되었네 ✨
알림이 이제 제대로 구분돼!

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	PR 제목 '버전 2.16.3'은 버전 번호만 명시하고 있으며, 실제 변경 내용인 NoticeMessageDto에 id 필드 추가에 대한 정보를 전혀 포함하지 않고 있습니다.	PR 제목을 '버전 2.16.3: NoticeMessageDto에 id 필드 추가' 또는 유사한 형태로 변경하여 주요 변경 사항을 명확히 반영해야 합니다.
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.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/main/java/com/kustacks/kuring/message/application/port/out/dto/NoticeMessageDto.java (1)

27-39: NoticeMessageDto.builder() 호출부에서 id 필드를 설정하지 않아 런타임 실패가 발생합니다.

FirebaseNotificationService.convertDtoFromCommand() 메서드의 builder 호출(195-202줄)이 id 필드를 설정하지 않으므로, 생성자의 Assert.notNull(id, ...) 체크에서 AssertionError가 발생합니다. 이 메서드를 수정하여 id를 설정하거나, 생성자의 null 검증 정책을 검토해주세요.

[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

[github-actions]

Unit Test Results

  74 files  ±0    74 suites  ±0   1m 26s ⏱️ -3s
532 tests ±0  525 ✔️ ±0  7 💤 ±0  0 ❌ ±0 
535 runs  ±0  528 ✔️ ±0  7 💤 ±0  0 ❌ ±0 

Results for commit 686680b. ± Comparison against base commit b653184.