feat/#180: hilt -> koin #182

jinukeu · 2025-05-11T16:40:07Z

📌 PR 요약

🌱 작업한 내용

hilt -> koin으로 마이그레이션 했습니다.
테스트는 많이 못해봤습니다. (일단 실행은 됨)

🌱 PR 포인트

나중에 코드 많이 고쳐야할 것 같습니다. 일단 CMP가 잘 되는지 확인하는게 우선이므로 코드 퀄리티는 챙기지 않았습니다.

📸 스크린샷

스크린샷
파일첨부바람

📮 관련 이슈

Resolved: [Feat] Hilt -> Koin Migration #180

Summary by CodeRabbit

새로운 기능
- 앱 전반에 Koin 기반의 의존성 주입 프레임워크가 도입되었습니다.
- 주요 ViewModel 및 도메인/데이터 모듈에 Koin DI 모듈이 추가되었습니다.
버그 수정
- 기존 Hilt 기반 의존성 주입 코드와 관련 어노테이션이 모두 제거되었습니다.
- 일부 ViewModel 및 데이터 소스의 생성자에서 불필요한 주입 관련 어노테이션이 삭제되었습니다.
기타 변경사항
- Android SDK 최소/타겟/컴파일 버전이 상향되었습니다.
- 타임테이블 위젯 및 관련 기능이 제거되었습니다.
- 일부 종속성 범위가 implementation에서 api로 변경되어 모듈 간 접근성이 개선되었습니다.

coderabbitai · 2025-05-11T16:40:20Z

Walkthrough

이 변경 사항은 프로젝트의 전체 의존성 주입 프레임워크를 Dagger Hilt에서 Koin으로 전환합니다. 각 모듈의 DI 설정, ViewModel, 데이터 소스, 레포지토리, 데이터베이스, DataStore, 네트워크 모듈 등이 Koin 스타일로 재작성되었으며, 관련 Hilt/Dagger 어노테이션과 모듈은 모두 제거되었습니다. 일부 위젯 관련 파일은 삭제되었습니다.

Changes

파일/경로 요약	변경 요약
`app-compose/build.gradle.kts`, `data/openmajor/build.gradle.kts`, `data/timetable/build.gradle.kts`, `domain/timetable/build.gradle.kts`, `local/common/build.gradle.kts`, `local/timetable/build.gradle.kts`, `remote/common/build.gradle.kts`, `remote/openmajor/build.gradle.kts`, `remote/timetable/build.gradle.kts`	각 모듈에 Koin 의존성 추가 (`koin.core`, `koin.android` 등)
`build-logic/convention/src/main/java/com/kunize/convention/Const.kt`	SDK 버전 상향: compile/target 34→35, min 26→28
`gradle/libs.versions.toml`	Koin 버전 및 라이브러리 정의 추가
`app-compose/src/main/java/com/kunize/uswtimetable/SuwikiApplication.kt`	`@HiltAndroidApp` 제거, Koin 초기화 코드 추가
`presentation/navigator/build.gradle.kts`, `presentation/openmajor/build.gradle.kts`, `presentation/timetable/build.gradle.kts`	의존성 가시성 조정(`implementation`→`api`), Koin Compose 관련 의존성 추가
`domain/timetable/src/main/java/com/suwiki/domain/timetable/timetableDomainModule.kt`, `presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/di/OpenMajorModule.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/di/TimetableModule.kt`	Koin 모듈 신규 추가: 도메인/프리젠테이션 레이어의 DI 설정
`data/openmajor/src/main/java/com/suwiki/data/openmajor/di/RepositoryModule.kt`, `data/timetable/src/main/java/com/suwiki/data/timetable/di/RepositoryModule.kt`, `local/common/src/main/java/com/suwiki/local/common/database/di/DaoModule.kt`, `local/common/src/main/java/com/suwiki/local/common/database/di/DatabaseModule.kt`, `local/common/src/main/java/com/suwiki/local/common/datastore/di/DataStoreModule.kt`, `local/timetable/src/main/java/com.suwiki.local.timetable/di/LocalDataSourceModule.kt`, `remote/common/src/main/java/com/suwiki/remote/common/di/FirebaseDatabaseModule.kt`, `remote/common/src/main/java/com/suwiki/remote/common/di/NetworkModule.kt`, `remote/openmajor/src/main/java/com/suwiki/remote/openmajor/di/ApiModule.kt`, `remote/timetable/src/main/java/com.suwiki.remote.timetable/di/RemoteDataSourceModule.kt`	Hilt/Dagger 모듈 삭제 및 동일 기능의 Koin 모듈로 대체
`presentation/navigator/src/main/java/com/suwiki/presentation/navigator/MainActivity.kt`	`@AndroidEntryPoint` 제거
`presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/OpenMajorScreen.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/celleditor/CellEditorScreen.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/openlecture/OpenLectureScreen.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetable/TimetableScreen.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetableeditor/TimetableEditorScreen.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetablelist/TimetableListScreen.kt`	ViewModel 주입 방식 Hilt→Koin으로 변경 (`hiltViewModel()`→`koinViewModel()`)
`presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/OpenMajorViewModel.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/celleditor/CellEditorViewModel.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/openlecture/OpenLectureViewModel.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetable/TimetableViewModel.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetableeditor/TimetableEditorViewModel.kt`, `presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetablelist/TimetableListViewModel.kt`	ViewModel의 Hilt 어노테이션(`@HiltViewModel`, `@Inject`) 제거
`domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/...`	모든 UseCase 클래스의 `@Inject` 생성자 어노테이션 제거
`data/openmajor/src/main/java/com/suwiki/data/openmajor/repository/OpenMajorRepositoryImpl.kt`, `data/timetable/src/main/java/com/suwiki/data/timetable/repository/OpenLectureRepositoryImpl.kt`, `data/timetable/src/main/java/com/suwiki/data/timetable/repository/TimetableRepositoryImpl.kt`, `remote/timetable/src/main/java/com.suwiki.remote.timetable/datasource/RemoteOpenLectureDataSourceImpl.kt`	생성자에서 `@Inject` 어노테이션 제거
`local/openmajor/src/main/java/com/suwiki/local/openmajor/datasource/LocalOpenMajorDataSourceImpl.kt`, `local/timetable/src/main/java/com.suwiki.local.timetable/datasource/LocalOpenLectureDatasourceImpl.kt`, `local/timetable/src/main/java/com.suwiki.local.timetable/datasource/LocalTimetableDatasourceImpl.kt`	DI를 통한 dispatcher 주입 제거, 내부에서 `Dispatchers.IO` 직접 할당
`presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/`, `remote/timetable/src/main/java/com.suwiki.remote.timetable/api/OpenLectureApi.kt`, `remote/timetable/src/main/java/com.suwiki.remote.timetable/di/ApiModule.kt`	위젯 관련 파일 및 OpenLectureApi, Hilt 모듈 등 다수 파일 삭제

Sequence Diagram(s)

sequenceDiagram
    participant App as SuwikiApplication
    participant Koin as Koin
    participant View as Composable/ViewModel
    participant Repo as Repository/DataSource

    App->>Koin: startKoin { modules(...) }
    View->>Koin: koinViewModel()/get()
    Koin->>Repo: 생성자 주입 및 싱글톤/팩토리 제공
    Repo-->>View: 의존성 주입된 인스턴스 반환

Assessment against linked issues

Objective	Addressed	Explanation
Hilt → Koin으로 DI 프레임워크 전환 (#180)	✅
모든 모듈의 DI 설정 Koin으로 통합 (#180)	✅
ViewModel, Repository, DataSource 등 DI 어노테이션 및 모듈 Hilt→Koin 변경 (#180)	✅
Hilt 관련 코드 및 어노테이션 완전 제거 (#180)	✅

Suggested labels

Android

Poem

🥕
코드는 바뀌었네, Hilt는 안녕!
Koin 토끼가 hop-hop 등장,
의존성 주입도 훨씬 간편,
ViewModel도, 데이터도 쏙쏙,
토끼 귀처럼 깔끔하게 정리!
모두 함께 Koin과 뛰어보자,
코드는 더 가볍고, 마음은 더 즐겁게!
🐇

Tip

⚡️ Faster reviews with caching

CodeRabbit now supports caching for code and dependencies, helping speed up reviews. This means quicker feedback, reduced wait times, and a smoother review experience overall. Cached data is encrypted and stored securely. This feature will be automatically enabled for all accounts on May 16th. To opt out, configure Review - Disable Cache at either the organization or repository level. If you prefer to disable all data retention across your organization, simply turn off the Data Retention setting under your Organization Settings.

Enjoy the performance boost—your workflow just got faster.

✨ 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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Generate unit testing code for this file.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR.
@coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

coderabbitai

Actionable comments posted: 10

🔭 Outside diff range comments (5)

remote/common/build.gradle.kts (1)

4-4: ⚠️ Potential issue

Hilt 플러그인 제거 필요
alias(libs.plugins.suwiki.android.hilt)는 Dagger Hilt 사용 중단 후 더 이상 필요하지 않습니다. Koin 전환 과정에서 불필요한 플러그인이므로 삭제해주세요.

local/timetable/build.gradle.kts (1)

4-4: ⚠️ Potential issue

Hilt 플러그인 제거 필요
alias(libs.plugins.suwiki.android.hilt)는 더 이상 사용되지 않으므로 제거해주세요.
presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/OpenMajorScreen.kt (1)
32-33: 💡 Verification agent
🧩 Analysis chain

미사용 import 제거 및 Koin ViewModel 통합 확인

Hilt에서 Koin으로 마이그레이션하면서 더 이상 사용하지 않는 import 문이 남아있습니다:

line 33: androidx.hilt.navigation.compose.hiltViewModel

또한, ViewModel 주입 방식이 hiltViewModel()에서 koinViewModel()로 변경되었습니다.

다음과 같이 미사용 import를 제거해주세요:
 import androidx.compose.ui.text.style.TextAlign
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
-import androidx.hilt.navigation.compose.hiltViewModel
 import com.suwiki.presentation.common.designsystem.component.appbar.SuwikiAppBarWithTitle
Also applies to: 49-57

🏁 Script executed:
#!/bin/bash
# Verify if OpenMajorViewModel is properly registered in a Koin module
rg "openMajor.*Module" -A 20 | grep -i "OpenMajorViewModel"
Length of output: 188
미사용 Hilt import 제거 및 Koin ViewModel 통합 확인

OpenMajorScreen.kt에서 더 이상 사용되지 않는 Hilt 관련 import를 제거해주세요:
-import androidx.hilt.navigation.compose.hiltViewModel
Koin 모듈(OpenMajorModule.kt)에 viewModelOf(::OpenMajorViewModel)로 ViewModel이 등록된 것을 확인했습니다. koinViewModel() 주입 방식을 사용 중이라면 추가 작업 없이 완료됩니다.

파일: presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/OpenMajorScreen.kt

라인: 32–33
data/timetable/src/main/java/com/suwiki/data/timetable/di/RepositoryModule.kt (1)
7-11: 🛠️ Refactor suggestion

사용하지 않는 Dagger 임포트를 제거해야 합니다.

Hilt에서 Koin으로 마이그레이션이 완료되었으나, 더 이상 사용되지 않는 Dagger/Hilt 관련 임포트가 남아있습니다.

다음 임포트를 제거하세요:
-import dagger.Binds
-import dagger.Module
-import dagger.hilt.InstallIn
-import dagger.hilt.components.SingletonComponent
-import javax.inject.Singleton
remote/common/src/main/java/com/suwiki/remote/common/di/NetworkModule.kt (1)
8-12: ⚠️ Potential issue

불필요한 Dagger·javax import 제거가 필요합니다
Hilt → Koin 마이그레이션 후에도 dagger.*, javax.inject.* 관련 import가 그대로 남아 있어 전부 사용되지 않습니다. ktlint / detekt 규칙에 따라 컴파일이 실패하거나 CI 파이프라인이 깨질 수 있으므로 반드시 삭제해 주세요.
-import dagger.Module
-import dagger.Provides
-import dagger.hilt.InstallIn
-import dagger.hilt.components.SingletonComponent
 ...
-import javax.inject.Qualifier
-import javax.inject.Singleton
Also applies to: 20-21

🧹 Nitpick comments (33)

presentation/navigator/build.gradle.kts (1)

12-13: api 활용 전파 범위 및 순환 의존성 검토 필요

openmajor 및 timetable 모듈을 api로 변경해 해당 모듈의 소비자까지 의존성이 전파됩니다. Koin DI 설정을 공유하기 위한 조치라면 적절하지만, 내부 API가 불필요하게 노출되거나 순환 의존성이 발생하지 않는지 검토해 주세요.

domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetMainTimetableUseCase.kt (1)

6-8: javax.inject.Inject 임포트 및 Hilt 의존성 제거 필요
현재 @Inject 어노테이션이 제거되었으나 javax.inject.Inject 임포트가 여전히 남아 있습니다. 사용되지 않는 임포트를 제거하고, timetableDomainModule에 GetMainTimetableUseCase가 factory로 정상 등록되었는지 확인하세요.
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateOpenLectureIfNeedUseCase.kt (1)
5-5: 사용하지 않는 import 제거 필요

javax.inject.Inject는 더 이상 사용되지 않으므로 import 문을 제거하는 것이 좋습니다.
-import javax.inject.Inject
data/timetable/src/main/java/com/suwiki/data/timetable/repository/TimetableRepositoryImpl.kt (1)
11-11: 사용하지 않는 import 제거 필요

javax.inject.Inject는 더 이상 사용되지 않으므로 import 문을 제거하는 것이 좋습니다.
-import javax.inject.Inject
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateTimetableUseCase.kt (1)
5-5: 사용하지 않는 import 제거 필요

javax.inject.Inject는 더 이상 사용되지 않으므로 import 문을 제거하는 것이 좋습니다.
-import javax.inject.Inject
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetOpenLectureListUseCase.kt (1)
6-6: 사용하지 않는 import 제거 필요

javax.inject.Inject를 import하고 있지만 코드에서 더 이상 사용되지 않습니다. 사용하지 않는 import는 제거하는 것이 좋습니다.
-import javax.inject.Inject
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateTimetableCellUseCase.kt (1)
6-6: 사용하지 않는 import 제거 필요

javax.inject.Inject를 import하고 있지만 코드에서 더 이상 사용되지 않습니다. 사용하지 않는 import는 제거하는 것이 좋습니다.
-import javax.inject.Inject
remote/timetable/src/main/java/com.suwiki.remote.timetable/datasource/RemoteOpenLectureDataSourceImpl.kt (1)
8-8: 사용하지 않는 import 제거 필요

javax.inject.Inject를 import하고 있지만 코드에서 더 이상 사용되지 않습니다. 사용하지 않는 import는 제거하는 것이 좋습니다.
-import javax.inject.Inject
data/timetable/src/main/java/com/suwiki/data/timetable/repository/OpenLectureRepositoryImpl.kt (1)
16-16: 사용하지 않는 import 제거 필요

javax.inject.Inject를 import하고 있지만 코드에서 더 이상 사용되지 않습니다. 사용하지 않는 import는 제거하는 것이 좋습니다.
-import javax.inject.Inject
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetTimetableCellTypeUseCase.kt (1)
5-5: 사용하지 않는 Import 제거 필요

javax.inject.Inject는 더 이상 사용되지 않으므로 import 문을 제거해주세요. Hilt에서 Koin으로의 마이그레이션에서 남겨진 것으로 보입니다.
-import javax.inject.Inject
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetAllTimetableUseCase.kt (1)
6-6: 사용하지 않는 Import 제거 필요

javax.inject.Inject는 더 이상 사용되지 않으므로 import 문을 제거해주세요. Hilt에서 Koin으로의 마이그레이션에서 남겨진 것으로 보입니다.
-import javax.inject.Inject
presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/OpenMajorViewModel.kt (1)
16-16: 사용하지 않는 Import 제거 필요

Hilt 마이그레이션 과정에서 더 이상 사용하지 않는 Import 문이 남아있습니다:

dagger.hilt.android.lifecycle.HiltViewModel

javax.inject.Inject

코드 정리를 위해 제거해주세요.
-import dagger.hilt.android.lifecycle.HiltViewModel
-import javax.inject.Inject
Also applies to: 29-29
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/SetTimetableCellTypeUseCase.kt (1)
5-5: 사용하지 않는 Import 제거 필요

javax.inject.Inject는 더 이상 사용되지 않으므로 import 문을 제거해주세요. Hilt에서 Koin으로의 마이그레이션에서 남겨진 것으로 보입니다.
-import javax.inject.Inject
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/InsertTimetableCellUseCase.kt (1)
1-16: 미사용 import 제거 필요

Hilt에서 Koin으로의 마이그레이션 과정에서 @Inject 어노테이션은 제거되었지만, 6번 라인의 javax.inject.Inject import는 여전히 남아있습니다. 이 import는 더 이상 사용되지 않으므로 제거해주세요.
package com.suwiki.domain.timetable.usecase

import com.suwiki.common.model.timetable.TimetableCell
import com.suwiki.domain.common.runCatchingIgnoreCancelled
import com.suwiki.domain.timetable.repository.TimetableRepository
-import javax.inject.Inject

class InsertTimetableCellUseCase(
  private val timetableRepository: TimetableRepository,
) {
  suspend operator fun invoke(cellList: List<TimetableCell>): Result<Unit> = runCatchingIgnoreCancelled {
    timetableRepository.insertTimetableCell(
      cellList = cellList,
    )
  }
}
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetable/TimetableViewModel.kt (1)
1-25: 미사용 import 제거 필요

Hilt에서 Koin으로의 마이그레이션 과정에서 @HiltViewModel과 @Inject 어노테이션은 제거되었지만, 관련 import들이 여전히 남아있습니다. 11번 라인의 dagger.hilt.android.lifecycle.HiltViewModel과 18번 라인의 javax.inject.Inject import는 더 이상 사용되지 않으므로 제거해주세요.
package com.suwiki.presentation.timetable.timetable

import androidx.lifecycle.ViewModel
import com.suwiki.common.model.timetable.TimetableCell
import com.suwiki.domain.timetable.usecase.DeleteTimetableCellUseCase
import com.suwiki.domain.timetable.usecase.GetMainTimetableUseCase
import com.suwiki.domain.timetable.usecase.GetTimetableCellTypeUseCase
import com.suwiki.domain.timetable.usecase.SetTimetableCellTypeUseCase
import com.suwiki.presentation.timetable.navigation.argument.toCellEditorArgument
import com.suwiki.presentation.timetable.timetable.component.timetable.cell.TimetableCellType
-import dagger.hilt.android.lifecycle.HiltViewModel
import org.orbitmvi.orbit.Container
import org.orbitmvi.orbit.ContainerHost
import org.orbitmvi.orbit.syntax.simple.intent
import org.orbitmvi.orbit.syntax.simple.postSideEffect
import org.orbitmvi.orbit.syntax.simple.reduce
import org.orbitmvi.orbit.viewmodel.container
-import javax.inject.Inject
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetablelist/TimetableListViewModel.kt (1)
1-24: 미사용 import 제거 필요

Hilt에서 Koin으로의 마이그레이션 과정에서 @HiltViewModel과 @Inject 어노테이션은 제거되었지만, 관련 import들이 여전히 남아있습니다. 9번 라인의 dagger.hilt.android.lifecycle.HiltViewModel과 18번 라인의 javax.inject.Inject import는 더 이상 사용되지 않으므로 제거해주세요.
package com.suwiki.presentation.timetable.timetablelist

import androidx.lifecycle.ViewModel
import com.suwiki.common.model.timetable.Timetable
import com.suwiki.domain.timetable.usecase.DeleteTimetableUseCase
import com.suwiki.domain.timetable.usecase.GetAllTimetableUseCase
import com.suwiki.domain.timetable.usecase.SetMainTimetableCreateTime
import com.suwiki.presentation.timetable.navigation.argument.toTimetableEditorArgument
-import dagger.hilt.android.lifecycle.HiltViewModel
import kotlinx.collections.immutable.minus
import kotlinx.collections.immutable.toPersistentList
import org.orbitmvi.orbit.Container
import org.orbitmvi.orbit.ContainerHost
import org.orbitmvi.orbit.syntax.simple.intent
import org.orbitmvi.orbit.syntax.simple.postSideEffect
import org.orbitmvi.orbit.syntax.simple.reduce
import org.orbitmvi.orbit.viewmodel.container
-import javax.inject.Inject
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetableeditor/TimetableEditorViewModel.kt (1)
10-20: 미사용 import 제거 필요

Hilt에서 Koin으로 마이그레이션하면서 더 이상 사용하지 않는 import 문이 남아있습니다:

line 10: dagger.hilt.android.lifecycle.HiltViewModel

line 20: javax.inject.Inject

다음과 같이 수정해주세요:
-import dagger.hilt.android.lifecycle.HiltViewModel
 import kotlinx.serialization.json.Json
 import org.orbitmvi.orbit.Container
 import org.orbitmvi.orbit.ContainerHost
 import org.orbitmvi.orbit.annotation.OrbitExperimental
 import org.orbitmvi.orbit.syntax.simple.blockingIntent
 import org.orbitmvi.orbit.syntax.simple.intent
 import org.orbitmvi.orbit.syntax.simple.postSideEffect
 import org.orbitmvi.orbit.syntax.simple.reduce
 import org.orbitmvi.orbit.viewmodel.container
-import javax.inject.Inject
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/DeleteTimetableCellUseCase.kt (1)
7-11: 미사용 import 제거 필요

Hilt에서 Koin으로 마이그레이션하면서 더 이상 사용하지 않는 import 문이 남아있습니다:

line 7: javax.inject.Inject

다음과 같이 수정해주세요:
 import com.suwiki.common.model.timetable.TimetableCell
 import com.suwiki.domain.common.runCatchingIgnoreCancelled
 import com.suwiki.domain.timetable.repository.TimetableRepository
-import javax.inject.Inject
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/SetMainTimetableCreateTime.kt (1)
5-7: Hilt에서 Koin으로 전환 시 미사용 import 삭제 필요

@Inject 어노테이션은 제거되었지만, 해당 import는 여전히 남아있습니다. 미사용 import는 제거하여 코드의 명확성을 높이는 것이 좋습니다.
-import javax.inject.Inject
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/celleditor/CellEditorViewModel.kt (1)
15-28: 미사용 Hilt 관련 import 제거 필요

Hilt에서 Koin으로 마이그레이션하면서 Hilt 관련 어노테이션이 제거되었지만, 관련 import는 여전히 남아있습니다. 미사용 import는 제거하는 것이 좋습니다.
-import dagger.hilt.android.lifecycle.HiltViewModel
-import javax.inject.Inject
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/openlecture/OpenLectureViewModel.kt (1)
16-33: Hilt 관련 미사용 import 제거 필요

Koin으로 전환하면서 Hilt 관련 어노테이션이 제거되었지만, 해당 import는 여전히 남아있습니다. 미사용 import 문을 제거하는 것이 코드 품질 향상에 도움이 됩니다.
-import dagger.hilt.android.lifecycle.HiltViewModel
-import javax.inject.Inject
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetableeditor/TimetableEditorScreen.kt (1)
16-16: 사용하지 않는 Hilt import를 제거하세요.

hiltViewModel()에서 koinViewModel()으로 마이그레이션했기 때문에 이 import는 더 이상 필요하지 않습니다. 코드 정리를 위해 제거하는 것이 좋습니다.
-import androidx.hilt.navigation.compose.hiltViewModel
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/celleditor/CellEditorScreen.kt (1)
38-38: 사용하지 않는 Hilt import를 제거하세요.

hiltViewModel()에서 koinViewModel()으로 마이그레이션했기 때문에 이 import는 더 이상 필요하지 않습니다. 코드 정리를 위해 제거하는 것이 좋습니다.
-import androidx.hilt.navigation.compose.hiltViewModel
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetablelist/TimetableListScreen.kt (1)
18-18: 사용하지 않는 Hilt import를 제거하세요.

hiltViewModel()에서 koinViewModel()으로 마이그레이션했기 때문에 이 import는 더 이상 필요하지 않습니다. 코드 정리를 위해 제거하는 것이 좋습니다.
-import androidx.hilt.navigation.compose.hiltViewModel
app-compose/src/main/java/com/kunize/uswtimetable/SuwikiApplication.kt (1)
4-12: Koin 설정이 올바르게 구현되었습니다.

Hilt에서 Koin으로의 마이그레이션이 Application 클래스에서 제대로 이루어졌습니다. 필요한 모든 모듈들이 등록되어 있는 것으로 보입니다.

다음 사항을 고려해보세요:

모듈을 계층별로 그룹화하거나 주석을 추가하여 구성을 명확히 하는 것이 좋습니다.

모듈 간 의존성이 있는 경우 순서가 중요할 수 있으므로 확인이 필요합니다.

모듈을 계층별로 그룹화하는 예시:
  modules(
+    // Remote data sources
     remoteTimetableDataSourceModule,
+    // Local data sources
     localTimetableDataSourceModule,
+    // Repositories
     timetableRepositoryModule,
     openMajorRepositoryModule,
+    // Domain
     timetableDomainModule,
+    // Presentation
     timetableModule,
     openMajorModule,
  )
Also applies to: 20-31
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetable/TimetableScreen.kt (1)
18-18: 사용되지 않는 import 제거 필요

hiltViewModel에 대한 import가 더 이상 사용되지 않습니다. 코드는 이제 Koin을 사용하므로 이 import 문을 제거해야 합니다.
-import androidx.hilt.navigation.compose.hiltViewModel
local/openmajor/src/main/java/com/suwiki/local/openmajor/datasource/LocalOpenMajorDataSourceImpl.kt (1)
16-28: Koin 마이그레이션은 성공적이나 미사용 import 정리 및 Dispatcher 관련 개선이 필요합니다.

Dagger Hilt에서 Koin으로의 마이그레이션 작업이 정상적으로 수행되었습니다. 그러나 다음과 같은 개선 사항이 있습니다:

미사용 import 문이 남아있습니다 (Dispatcher, SuwikiDispatchers, NormalDataStore, Inject 등)

ioDispatcher를 하드코딩하면 테스트 시 Dispatcher를 대체하기 어려워집니다.

불필요한 import 제거 및 Dispatcher 주입 방식 개선:
-import com.suwiki.common.android.Dispatcher
-import com.suwiki.common.android.SuwikiDispatchers
-import com.suwiki.local.common.datastore.di.NormalDataStore
-import javax.inject.Inject
향후 테스트 용이성을 위해 Dispatcher를 생성자를 통해 주입받는 방식으로 변경할 수도 있습니다:
class LocalOpenMajorDataSourceImpl(
  private val dataStore: DataStore<Preferences>,
  private val db: OpenMajorDatabase,
+ private val ioDispatcher: CoroutineDispatcher = Dispatchers.IO
) : LocalOpenMajorDataSource {

- private val ioDispatcher = Dispatchers.IO
local/timetable/src/main/java/com.suwiki.local.timetable/datasource/LocalTimetableDatasourceImpl.kt (1)
17-28: Koin 마이그레이션은 성공적이나 미사용 import 정리 및 Dispatcher 주입 방식 개선이 필요합니다.

Dagger Hilt에서 Koin으로의 마이그레이션이 정상적으로 이루어졌습니다. 그러나 다음과 같은 개선 사항이 있습니다:

미사용 import 문이 남아있습니다 (Dispatcher, SuwikiDispatchers, NormalDataStore, Inject 등)

ioDispatcher를 클래스 내부에서 직접 초기화하면 테스트 시 대체하기 어려워집니다.

불필요한 import 제거:
-import com.suwiki.common.android.Dispatcher
-import com.suwiki.common.android.SuwikiDispatchers
-import com.suwiki.local.common.datastore.di.NormalDataStore
-import javax.inject.Inject
향후 테스트 용이성을 위해 Dispatcher를 생성자를 통해 주입받는 방식으로 변경할 수도 있습니다:
class LocalTimetableDatasourceImpl(
  private val dataStore: DataStore<Preferences>,
  private val timetableDatabase: TimetableDatabase,
+ private val ioDispatcher: CoroutineDispatcher = Dispatchers.IO
) : LocalTimetableDataSource {
- private val ioDispatcher = Dispatchers.IO
remote/openmajor/src/main/java/com/suwiki/remote/openmajor/di/ApiModule.kt (1)
5-8: 사용하지 않는 Hilt/Dagger 임포트 제거 필요

Koin으로 마이그레이션 했으므로 더 이상 사용하지 않는 Dagger/Hilt 관련 임포트들(dagger.Module, dagger.Provides, dagger.hilt.InstallIn, dagger.hilt.components.SingletonComponent, javax.inject.Singleton)을 제거해주세요.
-import dagger.Module
-import dagger.Provides
-import dagger.hilt.InstallIn
-import dagger.hilt.components.SingletonComponent
 import org.koin.core.qualifier.named
 import org.koin.dsl.module
 import retrofit2.Retrofit
-import javax.inject.Singleton
Also applies to: 12-12
local/common/src/main/java/com/suwiki/local/common/database/di/DaoModule.kt (1)
9-12: 사용하지 않는 Hilt/Dagger 임포트 제거 필요

Koin으로 마이그레이션 했으므로 더 이상 사용하지 않는 Dagger/Hilt 관련 임포트들(dagger.Module, dagger.Provides, dagger.hilt.InstallIn, dagger.hilt.components.SingletonComponent)을 제거해주세요.
 import com.suwiki.local.common.database.dao.TimeTableDao
 import com.suwiki.local.common.database.database.OpenLectureDatabase
-import dagger.Module
-import dagger.Provides
-import dagger.hilt.InstallIn
-import dagger.hilt.components.SingletonComponent
 import org.koin.dsl.module
local/timetable/src/main/java/com.suwiki.local.timetable/datasource/LocalOpenLectureDatasourceImpl.kt (2)
7-8: 사용하지 않는 임포트 제거 필요

Koin으로 마이그레이션 했으므로 더 이상 사용하지 않는 @Dispatcher, SuwikiDispatchers 및 javax.inject.Inject 관련 임포트들을 제거해주세요.
-import com.suwiki.common.android.Dispatcher
-import com.suwiki.common.android.SuwikiDispatchers
 import com.suwiki.common.model.timetable.OpenLecture
 import com.suwiki.data.timetable.datasource.LocalOpenLectureDataSource
 import com.suwiki.local.common.database.database.OpenLectureDatabase
 import com.suwiki.local.common.datastore.di.NormalDataStore
 import com.suwiki.local.timetable.converter.toEntity
 import com.suwiki.local.timetable.converter.toModel
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.map
 import kotlinx.coroutines.withContext
-import javax.inject.Inject
Also applies to: 20-20

13-13: NormalDataStore 어노테이션 확인 필요

NormalDataStore 어노테이션이 아직 임포트되어 있지만 사용되지 않습니다. 이 어노테이션을 통한 식별자가 더 이상 필요 없다면 임포트를 제거해주세요.
 import com.suwiki.common.model.timetable.OpenLecture
 import com.suwiki.data.timetable.datasource.LocalOpenLectureDataSource
 import com.suwiki.local.common.database.database.OpenLectureDatabase
-import com.suwiki.local.common.datastore.di.NormalDataStore
 import com.suwiki.local.timetable.converter.toEntity
 import com.suwiki.local.timetable.converter.toModel
remote/common/src/main/java/com/suwiki/remote/common/di/NetworkModule.kt (1)
67-68: toMediaTypeOrNull() 대신 toMediaType() 사용 권장
"application/json".toMediaTypeOrNull()!! 형태는 널 강제 해제(!!)가 포함돼 불필요한 NPE 위험을 남깁니다. 상수 MIME 타입은 null 이 될 가능성이 없으므로 toMediaType()를 바로 사용하는 편이 안전하고 간결합니다.
-    .addConverterFactory(get<Json>().asConverterFactory("application/json".toMediaTypeOrNull()!!))
+    .addConverterFactory(get<Json>().asConverterFactory("application/json".toMediaType()))
Also applies to: 87-88

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7f463ff and 8ebd72b.

📒 Files selected for processing (73)

app-compose/build.gradle.kts (1 hunks)
app-compose/src/main/java/com/kunize/uswtimetable/SuwikiApplication.kt (1 hunks)
build-logic/convention/src/main/java/com/kunize/convention/Const.kt (1 hunks)
data/openmajor/build.gradle.kts (1 hunks)
data/openmajor/src/main/java/com/suwiki/data/openmajor/di/RepositoryModule.kt (1 hunks)
data/openmajor/src/main/java/com/suwiki/data/openmajor/repository/OpenMajorRepositoryImpl.kt (1 hunks)
data/timetable/build.gradle.kts (1 hunks)
data/timetable/src/main/java/com/suwiki/data/timetable/di/RepositoryModule.kt (1 hunks)
data/timetable/src/main/java/com/suwiki/data/timetable/repository/OpenLectureRepositoryImpl.kt (1 hunks)
data/timetable/src/main/java/com/suwiki/data/timetable/repository/TimetableRepositoryImpl.kt (1 hunks)
domain/timetable/build.gradle.kts (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/timetableDomainModule.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/DeleteTimetableCellUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/DeleteTimetableUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetAllTimetableUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetMainTimetableUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetOpenLectureListUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetTimetableCellTypeUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/InsertTimetableCellUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/InsertTimetableUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/SetMainTimetableCreateTime.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/SetTimetableCellTypeUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateOpenLectureIfNeedUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateTimetableCellUseCase.kt (1 hunks)
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateTimetableUseCase.kt (1 hunks)
gradle/libs.versions.toml (2 hunks)
local/common/build.gradle.kts (1 hunks)
local/common/src/main/java/com/suwiki/local/common/database/di/DaoModule.kt (1 hunks)
local/common/src/main/java/com/suwiki/local/common/database/di/DatabaseModule.kt (1 hunks)
local/common/src/main/java/com/suwiki/local/common/datastore/di/DataStoreModule.kt (1 hunks)
local/openmajor/src/main/java/com/suwiki/local/openmajor/datasource/LocalOpenMajorDataSourceImpl.kt (1 hunks)
local/timetable/build.gradle.kts (1 hunks)
local/timetable/src/main/java/com.suwiki.local.timetable/datasource/LocalOpenLectureDatasourceImpl.kt (1 hunks)
local/timetable/src/main/java/com.suwiki.local.timetable/datasource/LocalTimetableDatasourceImpl.kt (1 hunks)
local/timetable/src/main/java/com.suwiki.local.timetable/di/LocalDataSourceModule.kt (1 hunks)
presentation/navigator/build.gradle.kts (1 hunks)
presentation/navigator/src/main/java/com/suwiki/presentation/navigator/MainActivity.kt (0 hunks)
presentation/openmajor/build.gradle.kts (1 hunks)
presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/OpenMajorScreen.kt (1 hunks)
presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/OpenMajorViewModel.kt (1 hunks)
presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/di/OpenMajorModule.kt (1 hunks)
presentation/timetable/build.gradle.kts (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/celleditor/CellEditorScreen.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/celleditor/CellEditorViewModel.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/di/TimetableModule.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/openlecture/OpenLectureScreen.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/openlecture/OpenLectureViewModel.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetable/TimetableScreen.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetable/TimetableViewModel.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetableeditor/TimetableEditorScreen.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetableeditor/TimetableEditorViewModel.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetablelist/TimetableListScreen.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetablelist/TimetableListViewModel.kt (1 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/SendWidgetUpdateCommand.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/TimetableWidget.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/TimetableWidgetReceiver.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/Common.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/GlanceTimetable.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/cell/GlanceClassCell.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/cell/GlanceELearningCell.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/cell/GlanceEmptyCell.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/column/GlanceClassColumn.kt (0 hunks)
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/column/GlanceTimeColumn.kt (0 hunks)
remote/common/build.gradle.kts (1 hunks)
remote/common/src/main/java/com/suwiki/remote/common/di/FirebaseDatabaseModule.kt (1 hunks)
remote/common/src/main/java/com/suwiki/remote/common/di/NetworkModule.kt (1 hunks)
remote/openmajor/build.gradle.kts (1 hunks)
remote/openmajor/src/main/java/com/suwiki/remote/openmajor/di/ApiModule.kt (1 hunks)
remote/timetable/build.gradle.kts (1 hunks)
remote/timetable/src/main/java/com.suwiki.remote.timetable/api/OpenLectureApi.kt (0 hunks)
remote/timetable/src/main/java/com.suwiki.remote.timetable/datasource/RemoteOpenLectureDataSourceImpl.kt (1 hunks)
remote/timetable/src/main/java/com.suwiki.remote.timetable/di/ApiModule.kt (0 hunks)
remote/timetable/src/main/java/com.suwiki.remote.timetable/di/RemoteDataSourceModule.kt (1 hunks)

💤 Files with no reviewable changes (13)

presentation/navigator/src/main/java/com/suwiki/presentation/navigator/MainActivity.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/SendWidgetUpdateCommand.kt
remote/timetable/src/main/java/com.suwiki.remote.timetable/api/OpenLectureApi.kt
remote/timetable/src/main/java/com.suwiki.remote.timetable/di/ApiModule.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/column/GlanceTimeColumn.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/Common.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/TimetableWidget.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/cell/GlanceELearningCell.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/cell/GlanceClassCell.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/cell/GlanceEmptyCell.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/GlanceTimetable.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/TimetableWidgetReceiver.kt
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/widget/timetable/column/GlanceClassColumn.kt

⏰ Context from checks skipped due to timeout of 90000ms (1)

GitHub Check: build

🔇 Additional comments (49)

remote/timetable/build.gradle.kts (1)

17-19: Koin 의존성 추가 확인
libs.koin.core와 libs.koin.android가 추가되어 Koin DI가 활성화되었습니다. remoteTimetableDataSourceModule 등 Koin 모듈이 올바르게 등록되었는지 검증하세요.

remote/common/build.gradle.kts (1)

38-40: Koin 디펜던시 추가 확인
libs.koin.core와 libs.koin.android가 올바르게 추가되어 Koin DI 설정이 반영되었습니다.

domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/InsertTimetableUseCase.kt (1)

8-8: @Inject 어노테이션 제거 반영됨
클래스 선언에서 @Inject 제거가 정상 적용되었습니다.

domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/DeleteTimetableUseCase.kt (1)

8-8: @Inject 어노테이션 제거 반영됨
클래스 선언부에서 @Inject 제거가 올바르게 적용되었습니다.

remote/openmajor/build.gradle.kts (1)

17-19: Koin 디펜던시 추가 확인
libs.koin.core 및 libs.koin.android가 정상적으로 추가되어 Koin DI 통합이 완료되었습니다.

local/timetable/build.gradle.kts (1)

31-32: Koin 디펜던시 추가 확인
libs.koin.core와 libs.koin.android가 올바르게 추가되어 로컬 데이터소스 DI 설정에 반영되었습니다.

data/timetable/build.gradle.kts (1)

13-15: Koin 의존성 추가가 적절하게 이루어졌습니다.

Hilt에서 Koin으로의 마이그레이션을 위해 필요한 핵심 의존성이 잘 추가되었습니다. 코어 및 안드로이드 특화 모듈을 모두 포함한 것이 좋습니다.

presentation/timetable/build.gradle.kts (2)

12-12: 의존성 변경 (implementation → api)에 대한 검토가 필요합니다.

domain.timetable 모듈을 implementation에서 api로 변경하면 presentation 모듈을 사용하는 다른 모듈에서도 domain.timetable의 클래스들에 직접 접근할 수 있게 됩니다. 이것이 의도적인 설계 결정인지 확인해주세요. 아키텍처 계층 분리에 영향을 줄 수 있습니다.

17-19: Koin Compose 의존성 추가가 적절합니다.

Compose UI와 함께 Koin을 사용하기 위한 필수 의존성이 모두 잘 추가되었습니다:

koin.compose: 기본 Compose 통합

koin.compose.viewmodel: Compose에서 ViewModel 사용을 위한 확장

koin.compose.viewmodel.navigation: 네비게이션과 함께 사용하기 위한 기능

이는 hiltViewModel()에서 koinViewModel()으로의 마이그레이션을 지원합니다.

data/openmajor/build.gradle.kts (1)

12-13: Koin 의존성 추가가 적절합니다.

OpenMajor 데이터 모듈에 Koin 의존성이 올바르게 추가되었습니다. Hilt에서 Koin으로의 마이그레이션에 필요한 핵심 의존성이 모두 포함되어 있습니다.

domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateOpenLectureIfNeedUseCase.kt (1)

7-9: 코드 변경이 적절합니다

Hilt에서 Koin으로 DI 프레임워크를 마이그레이션하는 과정에서 @Inject 어노테이션을 제거한 것은 올바른 접근입니다. Koin에서는 생성자 주입 시 어노테이션이 필요하지 않습니다.

data/timetable/src/main/java/com/suwiki/data/timetable/repository/TimetableRepositoryImpl.kt (1)

13-15: 코드 변경이 적절합니다

Hilt에서 Koin으로 DI 프레임워크를 마이그레이션하는 과정에서 @Inject 어노테이션을 제거한 것은 올바른 접근입니다. Koin에서는 생성자 주입 시 어노테이션이 필요하지 않습니다.
build-logic/convention/src/main/java/com/kunize/convention/Const.kt (1)

6-8:
Details

❓ Verification inconclusive

SDK 버전 업데이트 검토 필요

SDK 버전이 상당히 높아졌습니다:

compileSdk와 targetSdk가 35로 증가 (Android 15 개발자 프리뷰 대상)

minSdk가 26에서 28로 증가 (Android 8.x Oreo 지원 중단)

특히 minSdk 증가는 지원하는 기기 범위에 영향을 줄 수 있으므로, 이 변경이 의도적인지, 그리고 사용자 기반에 미치는 영향을 고려했는지 확인해야 합니다.

🏁 Script executed:
#!/bin/bash
# Android 버전별 사용자 분포를 확인하기 위한 정보 출력
echo "Google Play Console에서 현재 앱의 Android 버전별 사용자 분포를 확인하세요."
echo "Android 8.x (API 26-27) 사용자 비율이 중요합니다."
echo "참고: https://developer.android.com/about/dashboards"
Length of output: 316
SDK 버전 업데이트 검토 요청

minSdk를 28로 상향 조정하면 Android 8.x(API 26–27) 지원이 중단되므로, 이 변경이 의도된 것인지와 사용자 기반에 미치는 영향을 반드시 확인해주세요.
파일: build-logic/convention/src/main/java/com/kunize/convention/Const.kt (6–8행)
변경된 SDK 버전:
const val compileSdk = 35
const val minSdk = 28
const val targetSdk = 35
확인 방법:
• Google Play Console > Android 버전별 사용자 분포 (특히 API 26–27) 확인
• API 26–27 사용자 비율이 충분히 낮아 변경 영향이 허용 가능한지 검토
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateTimetableUseCase.kt (1)

7-9: 코드 변경이 적절합니다

Hilt에서 Koin으로 DI 프레임워크를 마이그레이션하는 과정에서 @Inject 어노테이션을 제거한 것은 올바른 접근입니다. Koin에서는 생성자 주입 시 어노테이션이 필요하지 않습니다.

domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetOpenLectureListUseCase.kt (1)

8-10: 코드 변경 승인 - Hilt에서 Koin으로의 마이그레이션

생성자에서 @Inject 어노테이션이 제거된 것이 확인됩니다. 이는 Koin으로의 마이그레이션에 맞게 적절히 변경되었습니다.

domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/UpdateTimetableCellUseCase.kt (1)

8-10: 코드 변경 승인 - Hilt에서 Koin으로의 마이그레이션

생성자에서 @Inject 어노테이션이 제거된 것이 확인됩니다. 이는 Koin으로의 마이그레이션에 맞게 적절히 변경되었습니다.

remote/timetable/src/main/java/com.suwiki.remote.timetable/datasource/RemoteOpenLectureDataSourceImpl.kt (1)

10-12: 코드 변경 승인 - Hilt에서 Koin으로의 마이그레이션

생성자에서 @Inject 어노테이션이 제거된 것이 확인됩니다. 이는 Koin으로의 마이그레이션에 맞게 적절히 변경되었습니다.

data/timetable/src/main/java/com/suwiki/data/timetable/repository/OpenLectureRepositoryImpl.kt (1)

18-21: 코드 변경 승인 - Hilt에서 Koin으로의 마이그레이션

생성자에서 @Inject 어노테이션이 제거된 것이 확인됩니다. 이는 Koin으로의 마이그레이션에 맞게 적절히 변경되었습니다.
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetTimetableCellTypeUseCase.kt (1)

7-9:
Details

❓ Verification inconclusive

Hilt에서 Koin으로 마이그레이션 확인됨

생성자에서 @Inject 애노테이션이 제거되어 Koin 스타일로 변경되었습니다. 이것은 PR 목적에 맞게 올바르게 변경되었습니다.

적절한 Koin 모듈에서 이 UseCase를 제공하고 있는지 확인하세요.

🏁 Script executed:
#!/bin/bash
# timetableDomainModule에서 GetTimetableCellTypeUseCase가 등록되어 있는지 확인합니다.
echo "Looking for GetTimetableCellTypeUseCase registration in Koin modules..."
fd -e kt | xargs grep -l "timetableDomainModule" | xargs cat | grep -A 10 "timetableDomainModule" | grep "GetTimetableCellTypeUseCase"
Length of output: 283

[Koin 모듈 내 UseCase 등록 재확인 스크립트 실행]
Koin DSL에서 GetTimetableCellTypeUseCase가 실제로 single 또는 factory로 등록되어 있는지 추가로 검색합니다.

🏁 Script executed:
#!/bin/bash
echo "Koin DSL에서 GetTimetableCellTypeUseCase 등록 검색 중..."
# single이나 factory로 등록된 부분을 검색합니다.
rg -n "single<.*GetTimetableCellTypeUseCase" --glob "*.kt" || rg -n "factory<.*GetTimetableCellTypeUseCase" --glob "*.kt"
Length of output: 231
GetTimetableCellTypeUseCase Koin 모듈 등록 확인 필요
현재 코드베이스에서 GetTimetableCellTypeUseCase를 Koin DSL(single 또는 factory)로 등록한 부분이 검색되지 않았습니다.
– timetableDomainModule 또는 다른 Koin 모듈에 해당 UseCase가 제공(provide)되고 있는지 수동으로 확인해주세요.
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/GetAllTimetableUseCase.kt (1)

8-10:
Details

✅ Verification successful

Hilt에서 Koin으로 마이그레이션 확인됨

생성자에서 @Inject 애노테이션이 제거되어 Koin 스타일로 변경되었습니다. 이것은 PR 목적에 맞게 올바르게 변경되었습니다.

적절한 Koin 모듈에서 이 UseCase를 제공하고 있는지 확인하세요.

🏁 Script executed:
#!/bin/bash
# timetableDomainModule에서 GetAllTimetableUseCase가 등록되어 있는지 확인합니다.
echo "Looking for GetAllTimetableUseCase registration in Koin modules..."
fd -e kt | xargs grep -l "timetableDomainModule" | xargs cat | grep -A 10 "timetableDomainModule" | grep "GetAllTimetableUseCase"
Length of output: 308
Koin 모듈 등록 확인됨

timetableDomainModule에 factoryOf(::GetAllTimetableUseCase)로 GetAllTimetableUseCase가 정상적으로 등록되어 있음을 확인했습니다. 추가 조치가 필요 없습니다.
presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/OpenMajorViewModel.kt (1)

31-38:
Details

✅ Verification successful

ViewModel의 DI 프레임워크 마이그레이션 확인됨

@HiltViewModel 및 @Inject 애노테이션이 제거되어 Koin 스타일의 ViewModel 주입으로 변경되었습니다. PR 목적에 맞게 올바르게 변경되었습니다.

적절한 Koin 모듈에서 이 ViewModel을 제공하고 있는지 확인하세요.

🏁 Script executed:
#!/bin/bash
# openMajorModule에서 OpenMajorViewModel이 등록되어 있는지 확인합니다.
echo "Looking for OpenMajorViewModel registration in Koin modules..."
fd -e kt | xargs grep -l "openMajorModule" | xargs cat | grep -A 10 "openMajorModule" | grep "OpenMajorViewModel"

# Koin viewModelOf를 사용하는지 확인합니다.
echo "Checking if viewModelOf is used for OpenMajorViewModel..."
fd -e kt | xargs grep -l "OpenMajorViewModel" | xargs grep "viewModelOf"
Length of output: 802
ViewModel Koin 등록 확인 완료

OpenMajorModule.kt에서 viewModelOf(::OpenMajorViewModel)을 통해 OpenMajorViewModel이 정상적으로 등록된 것이 확인되었습니다. Koin DI 설정이 올바르게 구성되어 있어 추가 검증이나 수정이 필요하지 않습니다.
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/SetTimetableCellTypeUseCase.kt (1)

7-9:
Details

❓ Verification inconclusive

Hilt에서 Koin으로 마이그레이션 확인됨

생성자에서 @Inject 애노테이션이 제거되어 Koin 스타일로 변경되었습니다. 이것은 PR 목적에 맞게 올바르게 변경되었습니다.

적절한 Koin 모듈에서 이 UseCase를 제공하고 있는지 확인하세요.

🏁 Script executed:
#!/bin/bash
# timetableDomainModule에서 SetTimetableCellTypeUseCase가 등록되어 있는지 확인합니다.
echo "Looking for SetTimetableCellTypeUseCase registration in Koin modules..."
fd -e kt | xargs grep -l "timetableDomainModule" | xargs cat | grep -A 10 "timetableDomainModule" | grep "SetTimetableCellTypeUseCase"
Length of output: 283
SetTimetableCellTypeUseCase의 Koin 모듈 등록 여부 확인 필요
생성자에서 @Inject 애노테이션 제거를 통해 Koin 스타일로 변경된 것은 적절합니다.
다만, 해당 UseCase가 실제로 Koin 모듈에 제공(provide)되고 있는지 아래 항목을 직접 확인해 주세요:

timetableDomainModule 또는 관련 DI 모듈 파일 내에 SetTimetableCellTypeUseCase가 등록되어 있는지
(예: single { SetTimetableCellTypeUseCase(get()) } 형태)
presentation/openmajor/build.gradle.kts (1)

10-16:
Details

❓ Verification inconclusive

api 의존성 레벨 사용에 대한 확인 필요

domain 모듈들이 implementation에서 api로 변경되었습니다. 이는 이 모듈을 사용하는 다른 모듈에게 해당 의존성을 전이시키는 영향이 있습니다. Koin을 사용할 때 필요한 설정일 수 있지만, 모듈 간의 결합도가 높아질 수 있습니다.

이러한 변경이 의도적인지, 그리고 이 모듈을 사용하는 다른 모듈에서 이러한 domain 모듈에 직접 접근해야 하는 요구사항이 있는지 확인해주세요.

🏁 Script executed:
#!/bin/bash
# 이 모듈을 사용하는 다른 모듈들을 확인합니다
grep -r "implementation(projects.presentation.openmajor)" --include="*.kts" .
Length of output: 79

아래 명령어로 presentation/openmajor 모듈 참조를 더 포괄적으로 검색해 주세요:

🏁 Script executed:
#!/bin/bash
# presentation/openmajor 모듈을 참조하는 모든 build.gradle.kts 파일을 검색합니다.
rg "projects\\.presentation\\.openmajor|project\$\":presentation:openmajor\"\$" -t kts
Length of output: 112

아래 명령어로 .gradle.kts 파일 전체에서 presentation/openmajor 모듈 참조를 다시 확인해 주세요:

🏁 Script executed:
#!/bin/bash
# 모든 .gradle.kts 파일을 대상으로 presentation/openmajor 모듈 참조 검색
rg "projects\\.presentation\\.openmajor|project\$\":presentation:openmajor\"\$" --glob "*.gradle.kts" -n .
Length of output: 188
api 의존성 전이 영향 확인 요청

presentation/openmajor 모듈에서 domain.openmajor·domain.timetable을 implementation→api로 변경함에 따라, 이를 참조하는 다른 모듈에 도메인 모듈이 전이됩니다. 현재 확인된 참조는 다음과 같습니다:
presentation/navigator/build.gradle.kts (12줄):
api(projects.presentation.openmajor)
이로 인해 presentation/navigator 모듈에서도 domain.openmajor 및 domain.timetable 패키지에 직접 접근이 가능해집니다.
해당 변경이 의도된 것인지, 그리고 navigator 모듈에서 도메인 모듈의 API를 직접 사용해야 하는지 확인 부탁드립니다.
필요하지 않다면 결합도를 낮추기 위해 api → implementation 으로 되돌리는 것을 검토해 주세요.
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetableeditor/TimetableEditorViewModel.kt (1)

22-26:
Details

✅ Verification successful

Hilt에서 Koin으로 마이그레이션 확인

ViewModel 생성자에서 @Inject 어노테이션이 제거되었습니다. 이는 Koin 모듈에서 이 ViewModel이 올바르게 등록되어 있는지 확인해야 합니다.

🏁 Script executed:
#!/bin/bash
# Verify if TimetableEditorViewModel is properly registered in a Koin module
rg "timetable.*Module" -A 20 | grep -i "TimetableEditorViewModel"
Length of output: 200
TimetableEditorViewModel Koin 등록 확인 완료

TimetableModule.kt 파일에서 viewModelOf(::TimetableEditorViewModel)로 올바르게 등록되어 있어 추가 검증이나 수정이 필요하지 않습니다.

• 위치:

presentation/timetable/src/main/java/com/suwiki/presentation/timetable/di/TimetableModule.kt
domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/DeleteTimetableCellUseCase.kt (1)

9-11:
Details

✅ Verification successful

Koin 모듈 등록 확인 필요

UseCase 생성자에서 @Inject 어노테이션이 제거되었습니다. 이 UseCase가 Koin 모듈에 올바르게 등록되어 있는지 확인해야 합니다.

🏁 Script executed:
#!/bin/bash
# Verify if DeleteTimetableCellUseCase is properly registered in a Koin module
rg "timetable.*Module" -A 40 | grep -i "DeleteTimetableCellUseCase"
Length of output: 195
Koin 모듈 등록 확인 완료
DeleteTimetableCellUseCase가 domain/timetable/src/main/java/com/suwiki/domain/timetable/timetableDomainModule.kt 내
factoryOf(::DeleteTimetableCellUseCase)
로 정상 등록되어 있으므로 추가 조치가 필요 없습니다.
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/openlecture/OpenLectureScreen.kt (1)

43-71:
Details

✅ Verification successful

미사용 import 제거 및 Koin ViewModel 통합 확인

Hilt에서 Koin으로 마이그레이션하면서 더 이상 사용하지 않는 import 문이 남아있습니다:

line 43: androidx.hilt.navigation.compose.hiltViewModel

또한, ViewModel 주입 방식이 hiltViewModel()에서 koinViewModel()로 변경되었습니다.

다음과 같이 미사용 import를 제거해주세요:
 import androidx.compose.ui.text.style.TextAlign
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
-import androidx.hilt.navigation.compose.hiltViewModel
 import com.suwiki.common.model.timetable.OpenLecture
Also applies to: 77-77

🏁 Script executed:
#!/bin/bash
# Verify if OpenLectureViewModel is properly registered in a Koin module
rg "timetable.*Module" -A 40 | grep -i "OpenLectureViewModel"

# Check the application initialization of Koin
rg -A 20 "startKoin"
Length of output: 2320
미사용 import 제거 및 Koin ViewModel 주입 확인 완료

Koin 모듈(TimetableModule.kt)에서 OpenLectureViewModel이 정상 등록되어 있으며, SuwikiApplication의 startKoin 초기화에도 timetableModule이 포함되어 있음이 확인되었습니다.
아래 미사용 import만 제거해주세요:
-import androidx.hilt.navigation.compose.hiltViewModel
presentation/openmajor/src/main/java/com/suwiki/presentation/openmajor/di/OpenMajorModule.kt (1)

1-9: Koin 모듈 구현이 적절하게 작성되었습니다.

openMajorModule 모듈이 Koin의 권장 방식대로 구현되었으며, viewModelOf DSL을 사용하여 OpenMajorViewModel을 등록한 것이 적절합니다.
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/openlecture/OpenLectureViewModel.kt (1)

1-194:
Details

✅ Verification successful

Koin 의존성 주입으로의 마이그레이션이 일관되게 적용되었는지 확인

Hilt에서 Koin으로 전환되었으며 ViewModel 기능은 그대로 유지되었습니다. 다만, 이 클래스가 timetableModule에 제대로 등록되었는지 확인해보세요.

🏁 Script executed:
#!/bin/bash
# 모든 Koin 모듈 파일을 찾아 OpenLectureViewModel이 등록되어 있는지 확인

echo "Checking for OpenLectureViewModel registration in Koin modules..."
rg "OpenLectureViewModel" --type kotlin -A 2 -B 2
Length of output: 2950
OpenLectureViewModel Koin 모듈 등록 확인

presentation/timetable/src/main/java/com/suwiki/presentation/timetable/di/TimetableModule.kt에서
val timetableModule = module {
  viewModelOf(::OpenLectureViewModel)
  …
}
으로 OpenLectureViewModel이 정상 등록되어 있으며, OpenLectureRoute에서도 koinViewModel()을 통해 올바르게 주입되고 있습니다.
추가 검토할 사항이 없습니다.
presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetableeditor/TimetableEditorScreen.kt (1)

26-26: Koin 마이그레이션이 올바르게 구현되었습니다.

Hilt에서 Koin으로의 ViewModel 주입 변경이 올바르게 이루어졌습니다. TimetableEditorViewModel이 Koin 모듈에 제대로 등록되어 있는지 확인하세요.

Also applies to: 32-32

presentation/timetable/src/main/java/com/suwiki/presentation/timetable/celleditor/CellEditorScreen.kt (1)

53-53: Koin 마이그레이션이 올바르게 구현되었습니다.

Hilt에서 Koin으로의 ViewModel 주입 변경이 올바르게 이루어졌습니다. CellEditorViewModel이 Koin 모듈에 제대로 등록되어 있는지 확인하세요.

Also applies to: 59-59

presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetablelist/TimetableListScreen.kt (1)

28-28: Koin 마이그레이션이 올바르게 구현되었습니다.

Hilt에서 Koin으로의 ViewModel 주입 변경이 올바르게 이루어졌습니다. TimetableListViewModel이 Koin 모듈에 제대로 등록되어 있는지 확인하세요.

Also applies to: 34-34

app-compose/src/main/java/com/kunize/uswtimetable/SuwikiApplication.kt (1)

15-32: DI 프레임워크 마이그레이션 후 철저한 테스트가 필요합니다.

Hilt에서 Koin으로의 전체 마이그레이션은 앱의 모든 부분에 영향을 미칠 수 있는 중요한 변경사항입니다. PR 설명에 따르면 광범위한 테스트가 아직 수행되지 않았으므로, 다음 사항을 확인하세요:

모든 화면과 기능이 제대로 작동하는지 수동으로 테스트

각 ViewModel과 의존성이 올바르게 Koin 모듈에 등록되었는지 확인

특히 싱글톤 범위와 팩토리 범위가 이전 Hilt 설정과 일치하는지 검증

테스트 케이스가 있다면 모든 테스트를 실행하여 기존 기능이 손상되지 않았는지 확인하세요.

presentation/timetable/src/main/java/com/suwiki/presentation/timetable/timetable/TimetableScreen.kt (1)

31-31: DI 프레임워크 마이그레이션 확인

Hilt에서 Koin으로 마이그레이션이 올바르게 이루어졌습니다. ViewModel 주입 방식이 hiltViewModel()에서 koinViewModel()로 적절하게 변경되었습니다.

Also applies to: 38-38

presentation/timetable/src/main/java/com/suwiki/presentation/timetable/di/TimetableModule.kt (1)

1-15: Koin 모듈 구현 승인

Koin 모듈이 올바르게 구현되었습니다. viewModelOf DSL을 사용하여 각 ViewModel을 등록하는 방식이 잘 구성되어 있습니다. 이 모듈은 Hilt에서 Koin으로의 마이그레이션 과정에서 필요한 의존성 주입 설정을 적절히 제공합니다.

domain/timetable/src/main/java/com/suwiki/domain/timetable/timetableDomainModule.kt (1)

1-26: 도메인 모듈 구현 승인

도메인 계층의 Koin 모듈이 잘 구성되어 있습니다. 주석을 통해 use case들을 카테고리별로 명확하게 구분하여 가독성을 높였습니다. factoryOf DSL을 사용하여 각 use case를 적절하게 등록하고 있습니다.

remote/timetable/src/main/java/com.suwiki.remote.timetable/di/RemoteDataSourceModule.kt (1)

4-6: Remote 데이터 소스 모듈 구현 승인

Koin으로의 마이그레이션이 올바르게 이루어졌습니다. includes(firebaseDatabaseModule)를 통해 Firebase 모듈을 포함시키고, RemoteOpenLectureDataSource를 싱글톤으로 적절하게 등록했습니다. Dagger Hilt에서 Koin으로의 변환이 정확하게 수행되었습니다.

Also applies to: 8-11

remote/common/src/main/java/com/suwiki/remote/common/di/FirebaseDatabaseModule.kt (1)

4-10: Hilt에서 Koin으로 의존성 주입 프레임워크 마이그레이션이 잘 되었습니다.

Dagger Hilt에서 Koin으로의 마이그레이션이 적절하게 수행되었습니다. FirebaseDatabase 인스턴스 제공 로직이 동일하게 유지되면서 의존성 주입 프레임워크만 교체되었습니다.

data/openmajor/src/main/java/com/suwiki/data/openmajor/di/RepositoryModule.kt (1)

5-10: 리포지토리 모듈의 Koin 마이그레이션이 올바르게 구현되었습니다.

Dagger Hilt의 추상 모듈 및 @BINDS 어노테이션 기반 바인딩에서 Koin의 module DSL 및 single 함수를 사용한 방식으로 적절하게 변경되었습니다. get() 함수를 통해 OpenMajorRepositoryImpl의 의존성을 주입하는 방식도 올바르게 구현되었습니다.

remote/openmajor/src/main/java/com/suwiki/remote/openmajor/di/ApiModule.kt (2)

9-10: 적절한 Koin 임포트 추가됨

Hilt에서 Koin으로 마이그레이션을 위한 필요한 임포트들이 추가되었습니다.

14-18: Koin 모듈 정의가 올바르게 구현됨

Hilt 모듈에서 Koin 모듈로 변환이 잘 이루어졌습니다. single<MajorApi> 정의를 통해 싱글톤으로 MajorApi를 제공하고 있으며, named 한정자를 사용하여 적절히 "authRetrofit" Retrofit 인스턴스를 가져오고 있습니다.

local/common/src/main/java/com/suwiki/local/common/database/di/DatabaseModule.kt (2)

8-9: Koin 임포트 추가 적절함

Koin으로의 마이그레이션을 위한 필요한 임포트들이 추가되었습니다.

11-40: 데이터베이스 모듈이 Koin으로 잘 변환됨

Hilt에서 Koin으로 데이터베이스 모듈 변환이 잘 이루어졌습니다. 세 개의 데이터베이스(OpenMajor, Timetable, OpenLecture)가 모두 싱글톤으로 적절하게 제공되고 있으며, 기존 설정(마이그레이션, fallback 정책 등)도 잘 유지되었습니다. Hilt의 @ApplicationContext 대신 Koin의 androidContext()를 사용하여 컨텍스트를 가져오는 것도 적절합니다.

local/common/src/main/java/com/suwiki/local/common/database/di/DaoModule.kt (2)

13-13: Koin 임포트 추가됨

Koin으로의 마이그레이션을 위한 필요한 임포트가 추가되었습니다.

15-19: DAO 모듈이 Koin으로 잘 변환됨

Hilt에서 Koin으로 DAO 모듈 변환이 잘 이루어졌습니다. 세 개의 DAO(OpenMajorDao, TimeTableDao, OpenLectureDao)가 모두 싱글톤으로 적절하게 제공되고 있으며, 각 DAO를 해당 데이터베이스에서 가져오는 방식도 올바르게 구현되었습니다.

local/timetable/src/main/java/com.suwiki.local.timetable/datasource/LocalOpenLectureDatasourceImpl.kt (1)

16-16: Dispatchers 임포트 추가됨

Dispatchers.IO를 직접 사용하기 위한 임포트가 추가되었습니다.

local/timetable/src/main/java/com.suwiki.local.timetable/di/LocalDataSourceModule.kt (1)

1-23: Hilt에서 Koin으로의 전환이 적절하게 이루어졌습니다.

Dagger Hilt에서 Koin으로 의존성 주입 프레임워크 마이그레이션이 잘 수행되었습니다. 모듈 선언, 의존성 제공 및 인터페이스 바인딩이 Koin의 DSL을 활용하여 적절하게 구현되었습니다.

local/common/src/main/java/com/suwiki/local/common/datastore/di/DataStoreModule.kt (1)

18-35: DataStore 모듈이 Koin 방식으로 적절히 변환되었습니다.

Koin의 module DSL을 사용하여 DataStore 인스턴스들을 제공하는 구현이 잘 되었습니다. androidContext()를 통한 컨텍스트 접근과 named 퀄리파이어를 사용한 식별도 적절합니다.

gradle/libs.versions.toml (1)

209-213: Koin 라이브러리 의존성이 적절히 추가되었습니다.

Koin 관련 모든 필수 라이브러리(core, android, compose 관련 모듈)가 적절히 정의되었습니다.

data/timetable/src/main/java/com/suwiki/data/timetable/di/RepositoryModule.kt (1)

16-19: Repository 모듈의 Koin 변환이 잘 되었습니다.

singleOf와 bind DSL을 사용한 Repository 구현 및 인터페이스 바인딩이 적절합니다.

coderabbitai · 2025-05-11T16:43:27Z

local/common/build.gradle.kts

+  implementation(libs.koin.core)
+  implementation(libs.koin.android)
+


🛠️ Refactor suggestion

Koin 코어 및 Android 의존성 추가
libs.koin.core와 libs.koin.android가 추가되어 Koin DI 구성이 지원됩니다. 동시에 기존 Hilt 관련 플러그인(alias(libs.plugins.suwiki.android.hilt)) 및 의존성이 남아 있는지 제거 여부를 검토하세요.

coderabbitai · 2025-05-11T16:43:27Z

domain/timetable/build.gradle.kts

  implementation(libs.hilt.core)
+
+  implementation(libs.koin.core)


🛠️ Refactor suggestion

Hilt 코어 의존성 제거 및 Koin 의존성 전환
implementation(libs.koin.core)가 추가되었습니다. 중복된 DI 구성을 방지하기 위해 implementation(libs.hilt.core) 의존성을 제거하세요.

coderabbitai · 2025-05-11T16:43:27Z

app-compose/build.gradle.kts

+  implementation(libs.koin.core)
+  implementation(libs.koin.android)
+


💡 Verification agent

🧩 Analysis chain

Koin 의존성 추가 및 Hilt 플러그인 제거 확인
libs.koin.core와 libs.koin.android가 추가되어 앱 수준 DI가 Koin으로 전환되었습니다. alias(libs.plugins.suwiki.android.hilt) 플러그인이 남아 있는지 확인 후 제거하고, SuwikiApplication에서 Koin 모듈이 정상 초기화되는지 검증하세요.

🏁 Script executed:

#!/bin/bash echo "app-compose/build.gradle.kts에서 Hilt 플러그인 alias 확인" grep -R "alias(libs.plugins.suwiki.android.hilt)" -n app-compose/build.gradle.kts || echo "Hilt 플러그인 alias 없음" echo "SuwikiApplication에서 Koin 초기화(startKoin) 확인" grep -R "startKoin" -n .

Length of output: 563

🏁 Script executed:

#!/bin/bash file="app-compose/src/main/java/com/kunize/uswtimetable/SuwikiApplication.kt" echo "SuwikiApplication의 startKoin 호출 및 모듈 설정 확인" grep -R "startKoin" -n "$file" -A10 || echo "startKoin 호출 없음" echo "---" echo "modules 등록 확인" grep -R "modules(" -n "$file" || echo "modules 호출 없음"

Length of output: 1026

Hilt 플러그인 alias 제거 필요

app-compose/build.gradle.kts 5행의 alias(libs.plugins.suwiki.android.hilt)를 삭제하세요.

SuwikiApplication에서는 이미 startKoin(20행)과 modules(…)(22행~)가 정상 호출되고 있습니다.

삭제 후 Gradle 빌드 및 앱 실행 시 Koin 기반 DI가 의도대로 동작하는지 검증해 주세요.

coderabbitai · 2025-05-11T16:43:27Z

domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/InsertTimetableUseCase.kt

 import com.suwiki.common.model.timetable.Timetable
 import com.suwiki.domain.common.runCatchingIgnoreCancelled
 import com.suwiki.domain.timetable.repository.TimetableRepository
 import javax.inject.Inject


🛠️ Refactor suggestion

미사용 import 제거 권장
javax.inject.Inject가 더 이상 사용되지 않으므로 삭제하여 가독성과 빌드 경고를 제거해주세요.

coderabbitai · 2025-05-11T16:43:27Z

domain/timetable/src/main/java/com/suwiki/domain/timetable/usecase/DeleteTimetableUseCase.kt

 import com.suwiki.common.model.timetable.Timetable
 import com.suwiki.domain.common.runCatchingIgnoreCancelled
 import com.suwiki.domain.timetable.repository.TimetableRepository
 import javax.inject.Inject


🛠️ Refactor suggestion

미사용 import 제거 권장
javax.inject.Inject가 더 이상 참조되지 않습니다. import 문을 삭제해주세요.

coderabbitai · 2025-05-11T16:43:27Z

data/openmajor/src/main/java/com/suwiki/data/openmajor/repository/OpenMajorRepositoryImpl.kt

 import javax.inject.Inject

-class OpenMajorRepositoryImpl @Inject constructor(
+class OpenMajorRepositoryImpl(


💡 Verification agent

🧩 Analysis chain

@Inject 애노테이션 제거는 적절하나 추가 정리가 필요합니다.

Hilt에서 Koin으로 전환하면서 @Inject 애노테이션을 제거한 것은 적절합니다. 그러나 다음 사항을 확인해 주세요:

10번 줄의 javax.inject.Inject import는 더 이상 사용되지 않으므로 제거해야 합니다.

이 구현체가 Koin 모듈에 적절히 등록되었는지 확인하세요 (일반적으로 RepositoryModule.kt 파일).

-import javax.inject.Inject

🏁 Script executed:

#!/bin/bash # RepositoryModule 파일에서 OpenMajorRepositoryImpl이 등록되었는지 확인 rg "OpenMajorRepositoryImpl" --type kotlin | grep -i module

Length of output: 357

불필요한 javax.inject.Inject import 제거 필요

Koin DI 모듈(RepositoryModule.kt)에서 OpenMajorRepositoryImpl이 이미 등록된 것을 확인했습니다. 아래와 같이 import만 제거해주세요.

파일: data/openmajor/src/main/java/com/suwiki/data/openmajor/repository/OpenMajorRepositoryImpl.kt (10번 줄)

-import javax.inject.Inject

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

import javax.inject.Inject

class OpenMajorRepositoryImpl @Inject constructor(

class OpenMajorRepositoryImpl(

class OpenMajorRepositoryImpl(

// …

)

coderabbitai · 2025-05-11T16:43:27Z

...etable/src/main/java/com.suwiki.local.timetable/datasource/LocalOpenLectureDatasourceImpl.kt

+  private val dataStore: DataStore<Preferences>,
  private val openLectureDatabase: OpenLectureDatabase,
 ) : LocalOpenLectureDataSource {
+  private val ioDispatcher = Dispatchers.IO


🛠️ Refactor suggestion

디스패처 하드코딩 주의

주입되던 CoroutineDispatcher를 Dispatchers.IO로 하드코딩하였습니다. 이는 테스트가 어려워질 수 있습니다(테스트용 디스패처로 쉽게 대체할 수 없음). 가능하다면 Koin을 통해 디스패처를 주입받는 방식으로 변경하는 것을 고려해보세요.

- private val ioDispatcher = Dispatchers.IO + private val ioDispatcher: CoroutineDispatcher = get()

그리고 Koin 모듈에 다음과 같이 디스패처를 제공하는 코드를 추가하세요:

// LocalTimetableModule.kt 파일에 추가 single<CoroutineDispatcher> { Dispatchers.IO }

coderabbitai · 2025-05-11T16:43:28Z

...etable/src/main/java/com.suwiki.local.timetable/datasource/LocalOpenLectureDatasourceImpl.kt

+class LocalOpenLectureDatasourceImpl(
+  private val dataStore: DataStore<Preferences>,
  private val openLectureDatabase: OpenLectureDatabase,


💡 Verification agent

🧩 Analysis chain

생성자 주입 방식 변경

Hilt의 @Inject 대신 Koin의 생성자 주입 방식으로 변경되었습니다. @NormalDataStore 어노테이션도 제거되었는데, 여러 DataStore 인스턴스가 있는 경우 Koin에서 어떻게 구분하는지 확인이 필요합니다.

Koin 설정에서 DataStore 인스턴스가 어떻게 제공되는지 다음 스크립트로 확인해보세요:

🏁 Script executed:

#!/bin/bash # 프로젝트에서 DataStore 관련 Koin 모듈 설정 확인 echo "DataStore 관련 Koin 모듈 설정 확인:" rg "DataStore.*module" -l # DataStore 인스턴스 제공 방식 확인 echo "DataStore 인스턴스 제공 방식:" rg "single.*DataStore" -A 2 -B 2

Length of output: 744

Koin 모듈: DataStore qualifier 명시 검토 필요

DataStoreModule 에서 이미 named("normalDataStore") qualifier 로 인스턴스를 제공하고 있으므로, LocalOpenLectureDatasourceImpl 생성자에 주입할 때도 동일한 qualifier 를 명시해야 합니다. 그렇지 않으면 여러 DataStore 인스턴스 중 어떤 것을 주입할지 알 수 없어 DI 오류가 발생할 수 있습니다.

검토사항:

local/common/src/main/java/com/suwiki/local/common/datastore/di/DataStoreModule.kt 에서
single(qualifier = named("normalDataStore")) { … }
로 정의됨을 확인

LocalOpenLectureDatasourceImpl 을 바인딩하는 Koin 모듈(예: local/timetable/src/main/java/.../di/TimetableModule.kt)에서
single<LocalOpenLectureDatasource> { LocalOpenLectureDatasourceImpl( get(named("normalDataStore")), // qualifier 명시 get() ) }
와 같이 수정

coderabbitai · 2025-05-11T16:43:28Z

gradle/libs.versions.toml

 kotlin-test = "2.0.20"
 core = "1.6.1"

+koin = "4.1.0-Beta10"


💡 Verification agent

🧩 Analysis chain

Koin 버전 추가가 적절합니다.

Koin 4.1.0-Beta10 버전을 추가했습니다. 베타 버전이므로 안정적인 릴리스로 업그레이드 계획을 고려하시는 것이 좋습니다.

다음 스크립트를 실행하여 최신 안정 버전을 확인해보세요:

🏁 Script executed:

#!/bin/bash curl -s https://search.maven.org/solrsearch/select?q=g:%22io.insert-koin%22+AND+a:%22koin-core%22&rows=20&wt=json | jq -r '.response.docs | sort_by(.v) | reverse | .[0:5] | .[] | .v'

Length of output: 2125

Koin 버전 업데이트 제안: 최신 베타(4.1.0-Beta11) 또는 안정(4.0.0) 선택

현재 Maven Central에는 4.1.0-Beta10 대신 4.1.0-Beta11이 등록되어 있습니다.

베타 채널을 유지할 경우 → 4.1.0-Beta11로 업그레이드 권장

안정성을 최우선으로 할 경우 → 현재 안정 릴리스인 4.0.0으로 고정 권장

수정 위치:

파일: gradle/libs.versions.toml (라인 73)
- koin = "4.1.0-Beta10" + koin = "4.1.0-Beta11" # 최신 베타 # 또는 + koin = "4.0.0" # 안정 버전

Committable suggestion skipped: line range outside the PR's diff.

coderabbitai · 2025-05-11T16:43:28Z

remote/common/src/main/java/com/suwiki/remote/common/di/NetworkModule.kt

+        val loggingInterceptor = HttpLoggingInterceptor { message ->
+            when {
+                !message.isJsonObject() && !message.isJsonArray() ->
+                    Timber.tag("RETROFIT_TAG").d("CONNECTION INFO -> $message")

-        else -> kotlin.runCatching {
-          json.encodeToString(Json.parseToJsonElement(message))
-        }.onSuccess {
-          Timber.tag(RETROFIT_TAG).d(it)
-        }.onFailure {
-          Timber.tag(RETROFIT_TAG).d(message)
+                else -> kotlin.runCatching {
+                    json.encodeToString(Json.parseToJsonElement(message))
+                }.onSuccess {
+                    Timber.tag("RETROFIT_TAG").d(it)
+                }.onFailure {
+                    Timber.tag("RETROFIT_TAG").d(message)
+                }
+            }
        }
-      }
+        loggingInterceptor.level = HttpLoggingInterceptor.Level.BODY
+        loggingInterceptor
+    }


🛠️ Refactor suggestion

릴리스 빌드에서 과도한 로그 노출 방지
HttpLoggingInterceptor의 레벨을 항상 BODY로 고정하면 민감한 요청/응답 데이터가 릴리스 빌드에서도 노출될 위험이 있습니다. BuildConfig.DEBUG 플래그로 분기하여 디버그 빌드에만 BODY, 릴리스 빌드에는 NONE(또는 BASIC)을 설정해 주세요.

-loggingInterceptor.level = HttpLoggingInterceptor.Level.BODY +loggingInterceptor.level = if (BuildConfig.DEBUG) { + HttpLoggingInterceptor.Level.BODY +} else { + HttpLoggingInterceptor.Level.NONE +}

BEEEAM-J · 2025-05-12T12:28:16Z

domain/timetable/src/main/java/com/suwiki/domain/timetable/timetableDomainModule.kt

+
+val timetableDomainModule = module {
+    // Timetable management use cases
+    factoryOf(::DeleteTimetableUseCase)


UseCase들을 single 대신에 factory로 생성하는 이유 알 수 있을까요?

factory로 생성하게 되면 UseCase들 호출할 때마다 객체를 계속 새로 생성하게 될텐데 성능에 영향이 있을거 같아서요

아녀 별 이유 없습니다 ... !
일단 cmp 실현 가능 확인이 우선이라 생각해서 막 짰어요..ㅜ

넵, 그럼 이 부분은 추후에 더 검토해보고 수정할지 결정하시죠

BEEEAM-J

koin 적용 코드 확인 했습니다.

고생하셨습니다 :)

feat/#180: hilt -> koin

8ebd72b

jinukeu requested a review from BEEEAM-J May 11, 2025 16:40

jinukeu self-assigned this May 11, 2025

coderabbitai bot reviewed May 11, 2025

View reviewed changes

BEEEAM-J reviewed May 12, 2025

View reviewed changes

BEEEAM-J approved these changes May 12, 2025

View reviewed changes

jinukeu merged commit 61df506 into develop May 12, 2025
1 check failed

jinukeu deleted the feature/koin-setting branch May 12, 2025 12:35

coderabbitai bot mentioned this pull request May 14, 2025

#181 refactor: hilt 의존성 제거 #183

Merged

		implementation(libs.koin.core)
		implementation(libs.koin.android)

		implementation(libs.hilt.core)

		implementation(libs.koin.core)

feat/#180: hilt -> koin #182

feat/#180: hilt -> koin #182

Uh oh!

Conversation

jinukeu commented May 11, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 PR 요약

📸 스크린샷

📮 관련 이슈

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented May 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Assessment against linked issues

Suggested labels

Poem

Chat

Support

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 11, 2025

Choose a reason for hiding this comment

Uh oh!

BEEEAM-J May 12, 2025

Choose a reason for hiding this comment

Uh oh!

jinukeu May 12, 2025

Choose a reason for hiding this comment

Uh oh!

BEEEAM-J May 12, 2025

Choose a reason for hiding this comment

Uh oh!

BEEEAM-J left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

jinukeu commented May 11, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented May 11, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)