hotfix: RepositoryModule 내 불필요한 임포트 제거

[codebidoof]

📄 작업 내용 요약

• RepositoryModule.kt: 사용하지 않는 MyPageRepositoryImpl 클래스 임포트 문 제거

---

📎 Issue 번호

• closed #

---

✅ 작업 목록

• 기능 구현
• 코드 리뷰 반영
• 테스트 코드 작성
• 문서 업데이트

---

📝 기타 참고사항

Summary by CodeRabbit

• 버그 수정
  • FAQ 저장소 연결에 필요한 참조를 정리해 관련 모듈이 정상적으로 컴파일되도록 개선했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f758d9b1-7955-459f-91ba-a3da577cc1a3

📥 Commits

Reviewing files that changed from the base of the PR and between 25b7b47 and 0596e67.

📒 Files selected for processing (1)

• core/data/src/main/java/kr/co/call/data/di/RepositoryModule.kt

💤 Files with no reviewable changes (1)

• core/data/src/main/java/kr/co/call/data/di/RepositoryModule.kt

---

📝 Walkthrough

Walkthrough

RepositoryModule.kt의 저장소 import 목록에 FaqRepository가 추가되어 기존 바인딩 선언의 타입 참조가 컴파일 가능하도록 정리되었습니다. 클래스와 @Binds 함수 시그니처는 변경되지 않았습니다.

Changes

저장소 바인딩

Layer / File(s)	Summary
FaqRepository import 반영 core/data/src/main/java/kr/co/call/data/di/RepositoryModule.kt	FaqRepository import를 추가하여 모듈의 저장소 바인딩 타입 참조를 정리했습니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	제목은 변경 내용과 관련 있지만 Conventional Commits의 허용 type 목록에 없는 hotfix를 사용했습니다.	type을 fix, chore, refactor 중 하나로 바꾸고 ": 설명" 형식을 유지하세요.
모듈 의존성 방향 검증	⚠️ Warning	core/domain/repository/ChatRepository.kt가 androidx.paging.PagingData를 import해 core/domain 순수 Kotlin 원칙을 위반합니다.	PagingData 의존을 core/domain 밖으로 옮기고, domain은 자체 모델/콜백만 노출하도록 수정하세요.

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Ui 변경 시 스크린샷 첨부 확인	✅ Passed	변경 파일은 RepositoryModule.kt 하나뿐이며 Compose UI 파일이 없어 스크린샷 첨부 확인 대상이 아닙니다.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch hotfix/duplicated-import-sentence-problem

---

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

[JiwonLee42]

너무짧아요

[github-actions]

Gemini AI 코드리뷰

안녕하세요, 시니어 Android 개발자입니다. PR 리뷰 진행하겠습니다.

PR 요약:
core/data/src/main/java/kr/co/call/data/di/RepositoryModule.kt 파일에서 중복된 import kr.co.call.data.repositoryImpl.MyPageRepositoryImpl 문을 제거하는 변경사항입니다.

코드 리뷰 (규칙별 적용):

1. Kotlin 코드 리뷰:

   • Coroutine 사용 시 Dispatcher 명시 여부, viewModelScope/lifecycleScope 오남용: 해당 사항 없음 (코루틴 사용 코드 부재)
   • Flow/StateFlow 사용 시 collect 시점 (repeatOnLifecycle 사용 여부): 해당 사항 없음 (Flow/StateFlow 사용 코드 부재)
   • null 안전성 (!! 사용 지양): 해당 사항 없음 (null 안전성 관련 코드 부재)
   • data class에 불필요한 var 사용 여부: 해당 사항 없음 (data class 코드 부재)
   • Hilt DI 시 생성자 주입 원칙 준수 여부: 이 변경은 Hilt 모듈 내 중복 import 제거로, 직접적인 DI 구현 변경은 아닙니다. 하지만 DI 모듈을 깔끔하게 유지하는 것은 좋은 습관입니다. MyPageRepositoryImpl 자체의 의존성 주입 방식(생성자 주입)은 해당 PR의 범위 밖이지만, Hilt 사용 시 생성자 주입 원칙을 지키는 것은 중요합니다.
   • 하드코딩된 문자열/매직 넘버: 해당 사항 없음 (문자열/숫자 코드 부재)

2. Jetpack Compose 코드: 해당 사항 없음 (Jetpack Compose 코드 부재)

3. Repository/DataSource 레이어: 해당 사항 없음 (Repository/DataSource 구현 코드 부재)

   • Retrofit 에러 핸들링 (try-catch, Result 래핑): 이 모듈은 Repository DI를 정의하는 곳이므로, 실제 Repository 구현체에서 Retrofit 에러 핸들링을 Result 래핑 등을 통해 견고하게 처리하는지 확인이 필요합니다.
   • 네트워크 응답과 도메인 모델 매핑 분리 여부: 위와 동일하게 Repository 구현체에서 data 레이어의 네트워크 응답 모델을 domain 레이어의 도메인 모델로 매핑하는 로직이 잘 분리되어 있는지 확인이 필요합니다.
   • 캐싱 전략 (로컬 DB vs 메모리): 위와 동일하게 Repository 구현체에서 캐싱 전략이 적절히 구현되어 있는지 확인이 필요합니다.

4. ViewModel: 해당 사항 없음 (ViewModel 코드 부재)

5. FCM/SSE/실시간 통신 관련 코드: 해당 사항 없음 (실시간 통신 관련 코드 부재)

총평:
이번 PR은 RepositoryModule.kt 파일 내 중복 import 문을 제거하는 간단하고 깔끔한 코드 정리 작업입니다. 코드 베이스의 불필요한 부분을 줄여 유지보수성을 높이는 좋은 변경입니다. 변경된 코드 자체에는 문제가 없으며, 머지해도 좋을 것 같습니다.

이후 더 복잡한 기능 구현 PR에서는 위에 명시된 리뷰 규칙들을 더 면밀하게 적용하여 코드 품질과 안정성을 확보하는 데 중점을 두겠습니다.

고생 많으셨습니다!

[codebidoof]

너무짧아요

그럼 빌드 시키지 마세요 ㅋ