Merged
Conversation
- DataStore 관련 파일들을 datastore/auth 하위 패키지로 이동
- AuthTokenStorageFactory 추가 - TokenCrypto 인터페이스 추가 - core:datastore 모듈의 hilt 및 security 의존성 제거
- TokenProvider 인터페이스를 통해 토큰을 가져오도록 수정 - TokenProvider의 구현체를 제공하는 provideTokenStore 메소드 추가 - core/network 모듈에서 core/datastore 의존성 제거
WalkthroughDI(Dependency Injection) 관련 모듈과 구현체가 core 모듈에서 app 모듈로 이동되었으며, DataStore와 Security 관련 DI 구성이 app 모듈 내에서 통합 관리되도록 리팩토링되었습니다. 또한, Token 관련 인터페이스와 구현체, Serializer 구조가 새롭게 정의되고, 기존의 중복 또는 불필요한 코드가 제거되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant AppModule
participant DataStoreModule
participant Crypto
participant TokenCrypto
participant AuthTokenSerializer
participant AuthTokenDataStore
participant NetworkModule
participant TokenProvider
AppModule->>DataStoreModule: DI 초기화
DataStoreModule->>Crypto: Crypto 인스턴스 주입
DataStoreModule->>TokenCrypto: TokenCrypto 생성(crypto 위임)
DataStoreModule->>AuthTokenSerializer: AuthTokenSerializer 생성(TokenCrypto 주입)
DataStoreModule->>AuthTokenDataStore: AuthTokenDataStore 생성(context, AuthTokenSerializer)
NetworkModule->>TokenProvider: TokenProvider 생성(AuthTokenDataStore 주입)
Assessment against linked issues
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (3)
💤 Files with no reviewable changes (1)
🚧 Files skipped from review as they are similar to previous changes (2)
⏰ Context from checks skipped due to timeout of 90000ms (1)
✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed 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)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (8)
core/security/src/main/java/com/threegap/bitnagil/security/crypto/SecureCrypto.kt (1)
7-7: 클래스 가시성 변경이 DI 구조 개선에 적절합니다
SecureCrypto를 public으로 변경하여 app 모듈의 DI에서 사용할 수 있도록 한 것은 적절합니다. 다만 public API가 되면서 향후 변경 시 하위 호환성을 고려해야 합니다.core/security/src/main/java/com/threegap/bitnagil/security/keystore/AndroidKeyProvider.kt (1)
9-9: 보안 관련 클래스의 가시성 변경이 적절합니다
AndroidKeyProvider를 public으로 변경하여 app 모듈의 DI에서 사용할 수 있도록 한 것은 적절합니다. 보안 관련 클래스이므로 향후 변경 시 보안 영향도를 충분히 검토해야 합니다.core/datastore/src/test/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStoreImplTest.kt (1)
32-56: readFrom 블로킹 I/O 가능성
writeTo는withContext(Dispatchers.IO)로 감싸면서 I/O 스레드를 분리한 반면,readFrom에서는 같은 고려가 없습니다. 테스트 코드라 큰 문제는 아니지만, 구현 예시로 복사될 가능성을 생각하면 일관성을 맞춰 두는 편이 안전합니다.-override suspend fun readFrom(input: InputStream): AuthToken { - return try { - input.bufferedReader().use { - Json.decodeFromString(AuthToken.serializer(), it.readText()) - } - } catch (e: Exception) { - AuthToken() - } -} +override suspend fun readFrom(input: InputStream): AuthToken = + withContext(Dispatchers.IO) { + try { + input.bufferedReader().use { + Json.decodeFromString(AuthToken.serializer(), it.readText()) + } + } catch (e: Exception) { + AuthToken() + } + }core/network/src/main/java/com/threegap/bitnagil/network/token/TokenProvider.kt (1)
3-5: KDoc 주석 추가 제안공통 인터페이스이므로 간단한 KDoc 한 줄만 있어도 사용 의도가 분명해집니다.
/** * 네트워크 계층에서 인증 토큰을 비동기로 제공하기 위한 Provider. */ interface TokenProvider { suspend fun getToken(): String? }core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/serializer/AuthTokenSerializer.kt (1)
1-6: 타입 alias로 단순화 가능추가 메서드가 없으므로 굳이 인터페이스로 감쌀 필요 없이
typealias로 충분합니다. DI 식별자가 필요하다면@Qualifier로 해결할 수 있습니다.-package com.threegap.bitnagil.datastore.auth.serializer - -import androidx.datastore.core.Serializer -import com.threegap.bitnagil.datastore.auth.model.AuthToken - -interface AuthTokenSerializer : Serializer<AuthToken> +package com.threegap.bitnagil.datastore.auth.serializer + +import androidx.datastore.core.Serializer +import com.threegap.bitnagil.datastore.auth.model.AuthToken + +typealias AuthTokenSerializer = Serializer<AuthToken>core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/crypto/TokenCrypto.kt (1)
3-6: CPU 집중 작업의 suspend 처리 고려암·복호화 구현이 CPU-집중 또는 I/O-집중일 가능성이 있다면
suspend키워드를 붙여 코루틴 친화적으로 만드는 방안을 검토해주세요. 현재 형태로도 동작은 하지만, 나중에 무거운 알고리즘으로 교체될 때 호출 지점 수정이 필요해집니다.app/src/main/java/com/threegap/bitnagil/di/core/NetworkModule.kt (1)
114-119: TokenProvider 구현체 제공이 잘 구현되었습니다.AuthTokenDataStore를 TokenProvider로 브릿지하는 구현이 깔끔합니다. 다만
tokenFlow.first()호출이 빈번한 경우 성능에 영향을 줄 수 있으니 필요시 캐싱 전략을 고려해보세요.app/src/main/java/com/threegap/bitnagil/di/core/DataStoreModule.kt (1)
20-27: TokenCrypto 구현을 별도 클래스로 추출 검토익명 객체 대신 명시적인 구현 클래스를 만들면 테스트 용이성과 재사용성이 향상됩니다.
별도 파일에 구현 클래스 생성:
class TokenCryptoImpl @Inject constructor( private val crypto: Crypto ) : TokenCrypto { override fun encrypt(bytes: ByteArray): ByteArray = crypto.encrypt(bytes) override fun decrypt(bytes: ByteArray): ByteArray = crypto.decrypt(bytes) }그리고 모듈에서:
- fun provideTokenCrypto(crypto: Crypto): TokenCrypto = - object : TokenCrypto { - override fun encrypt(bytes: ByteArray): ByteArray = crypto.encrypt(bytes) - - override fun decrypt(bytes: ByteArray): ByteArray = crypto.decrypt(bytes) - } + fun provideTokenCrypto(impl: TokenCryptoImpl): TokenCrypto = impl
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (23)
app/src/main/java/com/threegap/bitnagil/di/core/DataStoreModule.kt(1 hunks)app/src/main/java/com/threegap/bitnagil/di/core/NetworkModule.kt(2 hunks)app/src/main/java/com/threegap/bitnagil/di/core/Qualifier.kt(1 hunks)app/src/main/java/com/threegap/bitnagil/di/core/SecurityModule.kt(2 hunks)core/datastore/build.gradle.kts(0 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/crypto/TokenCrypto.kt(1 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/model/AuthToken.kt(1 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/serializer/AuthTokenSerializer.kt(1 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/serializer/AuthTokenSerializerImpl.kt(1 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStore.kt(1 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStoreImpl.kt(1 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenStorageFactory.kt(1 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/di/DataStoreModule.kt(0 hunks)core/datastore/src/main/java/com/threegap/bitnagil/datastore/serializer/TokenSerializer.kt(0 hunks)core/datastore/src/test/java/com/threegap/bitnagil/datastore/auth/serializer/AuthAuthTokenSerializerImplTest.kt(5 hunks)core/datastore/src/test/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStoreImplTest.kt(1 hunks)core/network/build.gradle.kts(0 hunks)core/network/src/main/java/com/threegap/bitnagil/network/auth/AuthInterceptor.kt(1 hunks)core/network/src/main/java/com/threegap/bitnagil/network/auth/TokenAuthenticator.kt(1 hunks)core/network/src/main/java/com/threegap/bitnagil/network/token/TokenProvider.kt(1 hunks)core/security/build.gradle.kts(0 hunks)core/security/src/main/java/com/threegap/bitnagil/security/crypto/SecureCrypto.kt(1 hunks)core/security/src/main/java/com/threegap/bitnagil/security/keystore/AndroidKeyProvider.kt(1 hunks)
💤 Files with no reviewable changes (5)
- core/datastore/build.gradle.kts
- core/security/build.gradle.kts
- core/network/build.gradle.kts
- core/datastore/src/main/java/com/threegap/bitnagil/datastore/serializer/TokenSerializer.kt
- core/datastore/src/main/java/com/threegap/bitnagil/datastore/di/DataStoreModule.kt
🧰 Additional context used
🧠 Learnings (3)
app/src/main/java/com/threegap/bitnagil/di/core/NetworkModule.kt (1)
Learnt from: wjdrjs00
PR: YAPP-Github/Bitnagil-Android#16
File: core/network/src/main/java/com/threegap/bitnagil/network/auth/TokenAuthenticator.kt:12-46
Timestamp: 2025-07-03T09:05:30.067Z
Learning: 이 프로젝트에서는 네트워크 모듈을 점진적으로 개발하고 있으며, TokenAuthenticator 같은 인증 관련 기능은 실제 API 연동 작업 시점에 NetworkModule에 연결할 예정입니다.
core/network/src/main/java/com/threegap/bitnagil/network/auth/AuthInterceptor.kt (1)
Learnt from: wjdrjs00
PR: YAPP-Github/Bitnagil-Android#16
File: core/network/src/main/java/com/threegap/bitnagil/network/auth/TokenAuthenticator.kt:12-46
Timestamp: 2025-07-03T09:05:30.067Z
Learning: 이 프로젝트에서는 네트워크 모듈을 점진적으로 개발하고 있으며, TokenAuthenticator 같은 인증 관련 기능은 실제 API 연동 작업 시점에 NetworkModule에 연결할 예정입니다.
core/network/src/main/java/com/threegap/bitnagil/network/auth/TokenAuthenticator.kt (1)
Learnt from: wjdrjs00
PR: YAPP-Github/Bitnagil-Android#16
File: core/network/src/main/java/com/threegap/bitnagil/network/auth/TokenAuthenticator.kt:12-46
Timestamp: 2025-07-03T09:05:30.067Z
Learning: 이 프로젝트에서는 네트워크 모듈을 점진적으로 개발하고 있으며, TokenAuthenticator 같은 인증 관련 기능은 실제 API 연동 작업 시점에 NetworkModule에 연결할 예정입니다.
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
🔇 Additional comments (11)
app/src/main/java/com/threegap/bitnagil/di/core/Qualifier.kt (1)
1-1: 패키지 구조 변경이 적절합니다DI 관련 파일들을 app 모듈의
di.core패키지로 통합하여 중앙 집중식 관리를 구현하는 것이 순환 종속성 문제 해결에 적합합니다.core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/model/AuthToken.kt (1)
1-1: 인증 관련 모델의 패키지 재구성이 적절합니다인증 관련 클래스들을
auth서브패키지로 그룹화하여 관심사 분리와 모듈화를 개선했습니다.app/src/main/java/com/threegap/bitnagil/di/core/SecurityModule.kt (2)
1-1: DI 모듈의 중앙 집중화가 적절합니다SecurityModule을 app 모듈로 이동하여 DI 생성 책임을 중앙화한 것은 순환 종속성 문제 해결에 효과적입니다.
16-16: 코드 가독성 개선빈 줄 추가로 코드 가독성이 개선되었습니다.
core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStore.kt (1)
1-16: 패키지 이동 확인 완료패키지 이동 외에 로직 변경 사항이 없으며, DI 재구성 의도와도 잘 맞습니다. 다른 모듈의 import-path 모두 업데이트되었는지 한 번 더 확인만 해주세요.
core/network/src/main/java/com/threegap/bitnagil/network/auth/AuthInterceptor.kt (1)
3-3: 네트워크 모듈의 데이터스토어 의존성 분리가 잘 구현되었습니다.TokenProvider 인터페이스를 통한 추상화로 모듈 간 결합도가 효과적으로 감소되었습니다. 순환 종속성 문제 해결에 기여하는 좋은 설계 변경입니다.
Also applies to: 10-10, 14-14
app/src/main/java/com/threegap/bitnagil/di/core/NetworkModule.kt (1)
1-1: 패키지 구조 개선이 적절합니다.DI 모듈을
di.core패키지로 이동하여 모듈 구조가 더 명확해졌습니다.core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStoreImpl.kt (1)
1-1: 패키지 구조 개선과 DI 책임 분리가 적절합니다.데이터스토어 구현체를
auth.storage패키지로 이동하고@Inject어노테이션을 제거하여 DI 책임을 app 모듈로 성공적으로 이관했습니다.Also applies to: 5-5, 8-8
core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenStorageFactory.kt (1)
10-20: 팩토리 패턴을 활용한 DataStore 생성 로직이 우수합니다.
- 암호화된 토큰 저장을 위한 적절한 파일명 사용 ("auth-token.enc")
ReplaceFileCorruptionHandler로 데이터 손상 시 안전한 복구 메커니즘 제공- 객체 생성 로직을 중앙화하여 유지보수성 향상
팩토리 패턴 적용으로 DI 구조가 더욱 명확해졌습니다.
core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/serializer/AuthTokenSerializerImpl.kt (1)
1-1: 시리얼라이저 리팩토링이 적절하게 수행되었습니다.
- 표준
Serializer<AuthToken>인터페이스 사용으로 일관성 향상TokenCrypto인터페이스 분리로 암호화 책임 명확화@Inject어노테이션 제거로 DI 책임 분리 완료인터페이스 추상화와 네이밍이 더 명확해졌습니다.
Also applies to: 3-5, 13-15
app/src/main/java/com/threegap/bitnagil/di/core/DataStoreModule.kt (1)
29-34: DI 구성이 적절합니다!Factory 패턴을 활용한
AuthTokenDataStore생성이 깔끔하게 구현되었습니다. Context와 Serializer 의존성 주입도 올바르게 처리되었습니다.
core/network/src/main/java/com/threegap/bitnagil/network/auth/TokenAuthenticator.kt
Show resolved
Hide resolved
...test/java/com/threegap/bitnagil/datastore/auth/serializer/AuthAuthTokenSerializerImplTest.kt
Outdated
Show resolved
Hide resolved
l5x5l
reviewed
Jul 6, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[ PR Content ]
core 모듈 간 의존성 연결로 인해 순환 참조 가능성 문제가 발생할 여지가 있어 의존성 생성 책임을 app 모듈로 일괄 이동했습니다.
DI 조립은 app에서 통합 관리하는 구조로 변경했습니다.
Related issue
Screenshot 📸
Work Description
To Reviewers 📢
Summary by CodeRabbit
신규 기능
버그 수정
리팩터링
테스트