Fix: ci 관련 코드 수정

[alswlekk]

🔗 관련 이슈

• Close #

📙 작업 설명

📸 스크린샷 또는 시연 영상 (선택)

기능	미리보기	기능	미리보기
기능 설명		기능 설명

💬 추가 설명 or 리뷰 포인트 (선택)

Summary by CodeRabbit

• Style

  • 코드 포매팅 및 일관된 문법 스타일 적용(매개변수 쉼표·콜론 간격·줄 배치 등)
  • 가독성 향상을 위한 소소한 리팩터링 및 불필요 공백 제거

• Chores

  • 사용하지 않는 import 제거
  • 생성자 및 메서드 매개변수 포맷 정리(트레일링 콤마 등)
  • 내부 로깅 문구 추가로 진단 정보 보강

참고: 이번 업데이트로 사용자에게 보이는 기능 변화는 없습니다.

[coderabbitai]

Walkthrough

여러 Kotlin 파일에서 스타일 정렬(후행 쉼표·공백·줄바꿈)과 불필요한 import 제거가 이루어졌고, FCM 토큰 검증/갱신 경로에 추가 로깅이 삽입되었습니다. 공개 API 타입/반환은 변경되지 않았습니다.

Changes

Cohort / File(s)	변경 요약
API 서비스 레이어 app/src/main/java/com/konkuk/medicarecall/data/api/fcm/NotificationService.kt, app/src/main/java/com/konkuk/medicarecall/data/api/fcm/FcmUpdateService.kt	메서드 파라미터 목록 포맷 정리(후행 쉼표 추가, 괄호/줄바꿈 조정). 시그니처 타입·반환 불변.
저장소 인터페이스 및 구현 app/src/main/java/com/konkuk/medicarecall/data/repository/MemberRegisterRepository.kt, app/src/main/java/com/konkuk/medicarecall/data/repositoryimpl/MemberRegisterRepositoryImpl.kt, app/src/main/java/com/konkuk/medicarecall/data/repositoryimpl/FcmRepositoryImpl.kt	미사용 import 제거 및 생성자/메서드/DTO 호출 인자 리스트에 후행 쉼표 추가(포맷팅). FcmRepositoryImpl에 FCM 토큰 검증/갱신 로깅(log) 추가(동작 흐름 자체는 동일).
UI 뷰모델 레이어 app/src/main/java/com/konkuk/medicarecall/ui/feature/login/info/viewmodel/LoginViewModel.kt	생성자 매개변수 포맷(후행 쉼표) 및 단일라인 상태 변경 함수들을 멀티라인으로 확장(로직 불변).
DTO 및 기타 포맷팅 app/src/main/java/com/konkuk/medicarecall/data/dto/request/NotificationStatusRequestDto.kt, app/src/main/java/com/konkuk/medicarecall/data/di/ApiModule.kt, app/src/main/java/com/konkuk/medicarecall/data/repositoryimpl/StatisticsRepositoryImpl.kt	DTO 생성자 필드 공백/쉼표 정리, 불필요 공백 제거 및 소폭 포맷 정리(기능 변화 없음).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 분

• 변경 범위는 포맷팅 중심이나, FCM 관련 로그 추가로 일부 파일에서 동작 관찰 포인트가 늘어났습니다.
• 집중 검토 권장 파일:
  • app/src/main/java/com/konkuk/medicarecall/data/repositoryimpl/FcmRepositoryImpl.kt (추가된 로깅이 민감한 정보 노출 또는 로깅 수준 문제 없는지)
  • app/src/main/java/com/konkuk/medicarecall/data/api/fcm/NotificationService.kt (API 시그니처 포맷 변경이 인터페이스 매칭에 영향 없는지)
  • DTO 변경 (NotificationStatusRequestDto.kt)이 직렬화/역직렬화에 영향이 없는지 간단 빌드/런타임 확인

Possibly related PRs

• [Feat] 케어콜 푸시알림 설정 #110 #122 — 동일 FCM 관련 파일들에 포맷팅 변경 및 로깅 관련 편집이 포함된 PR로 코드 수준 연관성 높음.

Suggested labels

fix

Suggested reviewers

• ProtossManse
• ikseong00

Poem

쉼표 하나로 라인이 숨 고르고,
로그 한 줄로 흐름이 말을 걸어오네.
작지만 깔끔한 손질 끝에,
빌드가 웃음 지을 준비를 하네 ✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	PR 제목이 실제 변경사항과 맞지 않습니다. 제목은 'CI 관련 코드 수정'이지만, 실제 변경사항은 코드 포맷팅(후행 쉼표 추가), 로깅 추가, 불필요한 import 제거 등 다양한 개선사항입니다.	제목을 'Fix: code formatting and logging improvements' 또는 유사한 형태로 변경하여 실제 변경사항을 정확히 반영하도록 수정해주세요.
Docstring Coverage	⚠️ Warning	Docstring coverage is 20.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (1 passed)

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/fcm-alarm-#110

---

📜 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 a321ef8 and 623445f.

📒 Files selected for processing (2)

• app/src/main/java/com/konkuk/medicarecall/data/dto/request/NotificationStatusRequestDto.kt (1 hunks)
• app/src/main/java/com/konkuk/medicarecall/data/repositoryimpl/FcmRepositoryImpl.kt (1 hunks)

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

🔇 Additional comments (2)

app/src/main/java/com/konkuk/medicarecall/data/dto/request/NotificationStatusRequestDto.kt (1)

3-5: DTO 포맷팅 변경만 있어서 그대로 두셔도 될 것 같아요

read 프로퍼티의 공백/쉼표 정리는 코틀린 컨벤션에 맞고, 바이너리/동작에는 전혀 영향 없습니다.

app/src/main/java/com/konkuk/medicarecall/data/repositoryimpl/FcmRepositoryImpl.kt (1)

24-28: 생성자 트레일링 콤마 추가는 깔끔하고 문제 없어 보입니다

파라미터 리스트에 콤마를 붙여두면 이후 파라미터 추가 시 diff가 더 깔끔해져서 유지보수에도 도움이 될 것 같아요.

---

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