Conversation
Walkthrough회원가입 API와 관련된 온보딩 및 OAuth 인증 플로우가 대대적으로 개편되었습니다. 기존의 signUp 방식이 제거되고, 닉네임 중복 체크 및 사용자 정보 수정 API로 분리되었습니다. OAuth 소셜 로그인 연동, JWT 저장, 신규 사용자 분기, DTO/Entity/Repository/RemoteDataSource/Service 등 계층별로 신규 코드가 대량 추가되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
actor User
participant NamingScreen
participant OnBoardingViewModel
participant OnBoardingRepository
participant OnBoardingRemoteDataSource
participant OnBoardingService
User->>NamingScreen: 닉네임 입력
NamingScreen->>OnBoardingViewModel: ChangeInputNickname(intent)
OnBoardingViewModel->>OnBoardingRepository: checkNicknameDuplicate(nickname)
OnBoardingRepository->>OnBoardingRemoteDataSource: checkNicknameDuplicate(nickname)
OnBoardingRemoteDataSource->>OnBoardingService: checkNicknameDuplicate API 호출
OnBoardingService-->>OnBoardingRemoteDataSource: isDuplicated 반환
OnBoardingRemoteDataSource-->>OnBoardingRepository: isDuplicated 반환
OnBoardingRepository-->>OnBoardingViewModel: isDuplicated 반환
OnBoardingViewModel-->>NamingScreen: isNicknameDuplicate 상태 업데이트
sequenceDiagram
actor User
participant OauthScreen
participant OauthViewModel
participant OauthRepository
participant OauthRemoteDataSource
participant OauthService
User->>OauthScreen: 소셜 로그인 성공 (Kakao 등)
OauthScreen->>OauthViewModel: KakaoLoginSuccess(provider, accessToken)
OauthViewModel->>OauthRepository: postSocialLogin(provider, accessToken)
OauthRepository->>OauthRemoteDataSource: socialLogin(provider, accessToken)
OauthRemoteDataSource->>OauthService: postSocialLogin API 호출
OauthService-->>OauthRemoteDataSource: accessToken, refreshToken, isNewUser 반환
OauthRemoteDataSource-->>OauthRepository: SocialLoginResponseDto 반환
OauthRepository-->>OauthViewModel: TokenEntity 반환
OauthViewModel->>OauthRepository: saveJwtToken(accessToken, refreshToken)
OauthRepository-->>OauthViewModel: 저장 결과 반환
OauthViewModel-->>OauthScreen: 신규/기존 사용자 분기(NavigateToOnBoarding or NavigateToPlace)
Assessment against linked issues
Assessment against linked issues: Out-of-scope changes해당 변경사항 중 이슈 #74의 "회원가입 API" 목적과 무관한 기능적 코드 변경은 발견되지 않았습니다. Possibly related PRs
Suggested labels
Suggested reviewers
Poem
✨ 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: 6
🧹 Nitpick comments (7)
data/onboarding/src/main/java/com/teamsolply/solply/onboarding/dto/request/PatchUserInfoRequestDto.kt (1)
7-14: DTO 구조가 적절하게 업데이트되었습니다.
SignUpRequestDto에서PatchUserInfoRequestDto로의 리네이밍과 새로운 속성들(favoriteTown,persona) 추가가 적절합니다. 모든 속성이 올바른 직렬화 어노테이션을 사용하고 있습니다.향후 개선을 위해 다음을 고려해보세요:
- 입력 값 검증 로직 추가 (예: nickname 길이 제한, persona 유효성 검사)
- 문서화 주석 추가로 각 필드의 용도 명시
app/build.gradle.kts (1)
14-14: 키스토어 경로 하드코딩 개선 제안키스토어 파일 경로가 하드코딩되어 있어 유지보수성이 떨어집니다.
gradle.properties에 키스토어 경로를 정의하고 참조하는 방식으로 개선할 수 있습니다:
- storeFile = File("${project.rootDir.absolutePath}/keystore/debug.keystore") + storeFile = File(findProperty("DEBUG_KEYSTORE_PATH") ?: "${project.rootDir.absolutePath}/keystore/debug.keystore")core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/textfield/SolplyTextField.kt (1)
157-157: 중복 검사 타이밍 개선을 고려해보세요.현재
isNicknameDuplicate를 즉시 확인하는데, 사용자가 아직 입력 중이거나 검증이 진행 중일 때의 상태를 더 명확하게 구분하면 좋겠습니다.validationState = - if (isNicknameDuplicate) NickNameValidateState.Duplicate else NickNameValidateState.Valid + if (isNicknameDuplicate) NickNameValidateState.Duplicate + else if (isTyping) NickNameValidateState.Typing + else NickNameValidateState.Validfeature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingViewModel.kt (1)
57-62: TODO 메서드를 완성하거나 제거해주세요.
patchUserInfo메서드가 TODO 상태로 남아있습니다. 구현이 필요하거나 사용하지 않는다면 제거하는 것이 좋겠습니다.구현이 필요하시면 회원가입 API 호출 로직을 작성해드릴 수 있습니다.
feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthScreen.kt (1)
58-63: 하드코딩된 provider 문자열을 상수로 분리해주세요.
"KAKAO"문자열이 하드코딩되어 있습니다. 유지보수성을 위해 상수로 분리하는 것이 좋겠습니다.+private const val KAKAO_PROVIDER = "KAKAO" onSuccess = { accessToken, _ -> viewModel.sendIntent( OauthIntent.KakaoLoginSuccess( - provider = "KAKAO", + provider = KAKAO_PROVIDER, accessToken = accessToken ) ) }feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthViewModel.kt (1)
62-62: 로그 메시지를 개선해야 합니다.디버그 로그 메시지로 "asdasdasd"를 사용하는 것은 전문적이지 않습니다. 의미있는 로그 메시지로 변경하거나 제거해야 합니다.
- Log.d("asdasdasd", it.toString()) + Log.e("OauthViewModel", "Social login failed: ${it.message}", it)feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingScreen.kt (1)
43-43: 디버그 로그 메시지를 개선하거나 제거해야 합니다."asdasdasd"를 태그로 사용하는 것은 전문적이지 않습니다. 의미있는 태그로 변경하거나 프로덕션에서는 제거해야 합니다.
- Log.d("asdasdasd", "${state.selectedTownId} ${state.selectedPersona} ${state.userNickname}") + Log.d("OnBoardingRoute", "State: townId=${state.selectedTownId}, persona=${state.selectedPersona}, nickname=${state.userNickname}")
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (42)
.gitignore(1 hunks)app/build.gradle.kts(2 hunks)core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/textfield/SolplyTextField.kt(4 hunks)core/network/src/main/java/com/teamsolply/solply/network/AccessTokenInterceptor.kt(1 hunks)core/network/src/main/java/com/teamsolply/solply/network/service/TokenRefreshService.kt(1 hunks)data/oauth/src/main/java/com/teamsolply/solply/oauth/dto/request/SocialLoginRequestDto.kt(1 hunks)data/oauth/src/main/java/com/teamsolply/solply/oauth/dto/response/SocialLoginResponseDto.kt(1 hunks)data/oauth/src/main/java/com/teamsolply/solply/oauth/repository/OauthRepositoryImpl.kt(1 hunks)data/oauth/src/main/java/com/teamsolply/solply/oauth/source/OauthRemoteDataSource.kt(1 hunks)data/onboarding/src/main/java/com/teamsolply/solply/onboarding/dto/request/PatchUserInfoRequestDto.kt(1 hunks)data/onboarding/src/main/java/com/teamsolply/solply/onboarding/dto/response/NicknameDuplicateResponseDto.kt(1 hunks)data/onboarding/src/main/java/com/teamsolply/solply/onboarding/dto/response/PatchUserInfoResponseDto.kt(1 hunks)data/onboarding/src/main/java/com/teamsolply/solply/onboarding/repository/OnBoardingRepositoryImpl.kt(1 hunks)data/onboarding/src/main/java/com/teamsolply/solply/onboarding/source/OnBoardingRemoteDataSource.kt(1 hunks)domain/oauth/src/main/java/com/teamsolply/solply/oauth/model/TokenEntity.kt(1 hunks)domain/oauth/src/main/java/com/teamsolply/solply/oauth/repository/OauthRepository.kt(1 hunks)domain/onboarding/src/main/java/com/teamsolply/solply/onboarding/model/SignUpEntity.kt(0 hunks)domain/onboarding/src/main/java/com/teamsolply/solply/onboarding/model/UserInfoEntity.kt(1 hunks)domain/onboarding/src/main/java/com/teamsolply/solply/onboarding/repository/OnBoardingRepository.kt(1 hunks)feature/main/src/main/java/com/teamsolply/solply/main/MainNavigator.kt(1 hunks)feature/main/src/main/java/com/teamsolply/solply/main/MainScreen.kt(1 hunks)feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthContract.kt(1 hunks)feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthScreen.kt(2 hunks)feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthViewModel.kt(2 hunks)feature/oauth/src/main/java/com/teamsolply/solply/oauth/navigation/OauthNavigation.kt(1 hunks)feature/onboarding/build.gradle.kts(1 hunks)feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingContract.kt(2 hunks)feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingScreen.kt(8 hunks)feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingViewModel.kt(2 hunks)feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/component/BackHeader.kt(0 hunks)feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/navigation/OnBoardingNavigation.kt(1 hunks)feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/NamingScreen.kt(3 hunks)feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/SelectPersonaScreen.kt(0 hunks)remote/oauth/.gitignore(1 hunks)remote/oauth/build.gradle.kts(1 hunks)remote/oauth/src/main/java/com/teamsolply/solply/oauth/datasource/OauthRemoteDataSourceImpl.kt(1 hunks)remote/oauth/src/main/java/com/teamsolply/solply/oauth/di/OauthRemoteDataModule.kt(1 hunks)remote/oauth/src/main/java/com/teamsolply/solply/oauth/di/OauthServiceModule.kt(1 hunks)remote/oauth/src/main/java/com/teamsolply/solply/oauth/service/OauthService.kt(1 hunks)remote/onboarding/src/main/java/com/teamsolply/solply/onboarding/datasource/OnBoardingRemoteDataSourceImpl.kt(1 hunks)remote/onboarding/src/main/java/com/teamsolply/solply/onboarding/service/OnBoardingService.kt(1 hunks)settings.gradle.kts(1 hunks)
💤 Files with no reviewable changes (3)
- feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/component/BackHeader.kt
- feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/SelectPersonaScreen.kt
- domain/onboarding/src/main/java/com/teamsolply/solply/onboarding/model/SignUpEntity.kt
🧰 Additional context used
🧬 Code Graph Analysis (4)
feature/oauth/src/main/java/com/teamsolply/solply/oauth/navigation/OauthNavigation.kt (1)
feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthScreen.kt (1)
OauthRoute(43-77)
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingViewModel.kt (1)
core/ui/src/main/java/com/teamsolply/solply/ui/base/BaseViewModel.kt (1)
reduce(44-46)
feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthScreen.kt (1)
feature/main/src/main/java/com/teamsolply/solply/main/MainNavigator.kt (1)
navigateToPlace(79-81)
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingScreen.kt (2)
core/ui/src/main/java/com/teamsolply/solply/ui/lifecycle/LaunchedEffectWithLifecycle.kt (1)
LaunchedEffectWithLifecycle(11-24)feature/main/src/main/java/com/teamsolply/solply/main/MainNavigator.kt (1)
navigateToPlace(79-81)
🔇 Additional comments (49)
remote/oauth/.gitignore (1)
1-1: 표준 관례에 따른 올바른 설정입니다.새로운
remote/oauth모듈에 대한.gitignore파일이/build디렉토리를 무시하도록 적절히 설정되었습니다..gitignore (1)
32-33: 보안을 위한 적절한 키스토어 파일 제외 설정입니다.앱 서명에 사용되는 민감한 키스토어 파일들이 Git 추적에서 제외되도록 명시적으로 설정되었습니다. 이는 보안 모범 사례에 부합합니다.
feature/onboarding/build.gradle.kts (1)
10-10: 클린 아키텍처 원칙에 따른 올바른 의존성 추가입니다.피처 모듈이 도메인 계층의 온보딩 인터페이스를 사용할 수 있도록 의존성이 적절히 추가되었습니다.
settings.gradle.kts (1)
53-53: 새로운 OAuth 모듈의 표준적인 추가입니다.
remote:oauth모듈이 다른 remote 모듈들과 함께 적절한 위치에 추가되었습니다. 멀티모듈 프로젝트의 표준 관례를 따릅니다.remote/oauth/build.gradle.kts (1)
1-13: OAuth 모듈 빌드 설정이 적절합니다.새로운 OAuth 모듈의 Gradle 빌드 스크립트가 올바르게 구성되어 있습니다. 필요한 의존성들(network, model, data.oauth)이 포함되어 있고 네임스페이스도 명확하게 정의되어 있습니다.
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/navigation/OnBoardingNavigation.kt (1)
21-23: 코드 포맷팅 개선이 잘 되었습니다.불필요한 trailing comma를 제거하고 브레이스 위치를 조정하여 코드 가독성이 향상되었습니다.
domain/oauth/src/main/java/com/teamsolply/solply/oauth/model/TokenEntity.kt (1)
5-6: TokenEntity 사용처 확인 완료
- data/oauth/src/main/java/com/teamsolply/solply/oauth/repository/OauthRepositoryImpl.kt (22–26)에서
isNewUser가 정상적으로 매핑되는 것을 확인했습니다.- 추가 검토나 수정 없이 필드 추가를 승인합니다.
domain/onboarding/src/main/java/com/teamsolply/solply/onboarding/model/UserInfoEntity.kt (1)
3-8: 사용자 정보 데이터 클래스가 잘 설계되었습니다.
UserInfoEntity는 온보딩 프로세스에서 필요한 사용자 정보를 적절히 모델링했습니다.favoriteTownId만 nullable로 설정한 것이 적절하며, 모든 필드의 타입과 명명이 명확합니다.data/onboarding/src/main/java/com/teamsolply/solply/onboarding/dto/response/NicknameDuplicateResponseDto.kt (1)
6-10: 구현이 올바르게 되어 있습니다.이 DTO는 간단하고 명확한 구조를 가지고 있으며, Kotlin 직렬화 모범 사례를 잘 따르고 있습니다.
@SerialName어노테이션을 통한 JSON 매핑도 적절합니다.data/oauth/src/main/java/com/teamsolply/solply/oauth/dto/request/SocialLoginRequestDto.kt (1)
6-10: 소셜 로그인 요청 DTO가 올바르게 구현되었습니다.구조가 단순하고 명확하며, 직렬화 어노테이션이 적절히 사용되었습니다. 속성명도 명확하고 이해하기 쉽습니다.
data/oauth/src/main/java/com/teamsolply/solply/oauth/source/OauthRemoteDataSource.kt (1)
5-7: OAuth 원격 데이터 소스 인터페이스가 잘 설계되었습니다.클린 아키텍처 원칙을 잘 따르고 있으며, suspend 함수를 사용하여 비동기 처리가 적절히 고려되었습니다. 함수 시그니처도 소셜 로그인 기능에 적합합니다.
feature/main/src/main/java/com/teamsolply/solply/main/MainScreen.kt (1)
134-142: OAuth 네비게이션 그래프에 Place 화면으로의 네비게이션이 적절히 추가되었습니다.기존 네비게이션 패턴과 일관성을 유지하며, 네비게이션 옵션 설정도 다른 유사한 콜백들과 동일한 구조를 따르고 있습니다.
app/build.gradle.kts (1)
54-54: OAuth 원격 모듈 의존성 추가 승인OAuth 기능 구현을 위한 원격 모듈 의존성이 올바르게 추가되었습니다.
feature/oauth/src/main/java/com/teamsolply/solply/oauth/navigation/OauthNavigation.kt (1)
20-21: 네비게이션 파라미터 추가가 올바르게 구현됨OAuth 플로우에서 Place 화면으로의 네비게이션을 지원하기 위한 파라미터가 올바르게 추가되고 전달되었습니다.
Also applies to: 26-27
remote/oauth/src/main/java/com/teamsolply/solply/oauth/di/OauthServiceModule.kt (1)
12-19: Dagger Hilt 모듈 구성이 올바르게 구현됨OAuth 서비스를 위한 의존성 주입 모듈이 올바르게 구현되었습니다. @NoneAccessToken 어노테이션을 사용하여 토큰 없는 요청을 위한 Retrofit 인스턴스를 사용하는 것이 적절합니다.
data/oauth/src/main/java/com/teamsolply/solply/oauth/dto/response/SocialLoginResponseDto.kt (1)
6-14: 소셜 로그인 응답 DTO가 올바르게 구현됨소셜 로그인 응답을 위한 DTO가 적절한 직렬화 어노테이션과 함께 올바르게 구현되었습니다. 필드명과 구조가 OAuth 응답 형식에 적합합니다.
core/network/src/main/java/com/teamsolply/solply/network/service/TokenRefreshService.kt (1)
8-10: 토큰 새로고침 서비스 구성이 올바르게 업데이트됨API 엔드포인트와 헤더 구성이 백엔드 API 규격에 맞게 올바르게 업데이트되었습니다. "Refresh-Token" 헤더 사용이 적절합니다.
data/onboarding/src/main/java/com/teamsolply/solply/onboarding/dto/response/PatchUserInfoResponseDto.kt (1)
1-16: 깔끔한 Response DTO 구현입니다.Kotlin serialization을 올바르게 사용하고 있으며, 모든 필드에 적절한
@SerialName어노테이션이 적용되어 있습니다. 타입 정의도 명확하고 구조가 간결합니다.feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingContract.kt (2)
23-24: 상태 관리 개선이 좋습니다.
userNickname을 null이 아닌 기본값으로 변경하고isNicknameDuplicate상태를 추가한 것은 닉네임 중복 검사 플로우를 명확하게 지원합니다. 널 안전성도 향상되었습니다.
35-35: Intent 이름 개선이 적절합니다.
ChangeInputNickname으로 이름을 변경한 것이 더 명확하고 의도를 잘 표현합니다.remote/oauth/src/main/java/com/teamsolply/solply/oauth/di/OauthRemoteDataModule.kt (1)
11-17: 표준적인 Dagger Hilt 모듈 구현입니다.SingletonComponent 스코프에 올바르게 설치되어 있고, 인터페이스와 구현체의 바인딩이 적절하게 구성되어 있습니다. 의존성 주입 패턴을 정확히 따르고 있습니다.
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/textfield/SolplyTextField.kt (2)
117-117: UI 컴포넌트 책임 분리가 잘 되었습니다.중복 검사 로직을 UI 컴포넌트에서 분리하여 외부 상태로 관리하는 것이 좋은 아키텍처 개선입니다. 컴포넌트가 더 예측 가능하고 테스트하기 쉬워졌습니다.
140-144: 최소 길이 검증 추가가 좋습니다.
TooShort상태를 추가하여 최소 길이 검증을 구현한 것이 사용자 경험을 개선합니다. 검증 로직이 더 명확해졌습니다.feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthScreen.kt (1)
69-69: Place 화면으로의 네비게이션 처리가 올바르게 구현되었습니다.새로운
NavigateToPlace사이드 이펙트가 적절하게 처리되어 있습니다.domain/oauth/src/main/java/com/teamsolply/solply/oauth/repository/OauthRepository.kt (1)
6-7: 인터페이스 리팩토링이 적절하게 수행되었습니다.소셜 로그인과 JWT 토큰 저장 로직을 분리하여 관심사가 명확하게 분리되었고,
Result타입을 사용한 오류 처리 방식도 적절합니다.remote/onboarding/src/main/java/com/teamsolply/solply/onboarding/datasource/OnBoardingRemoteDataSourceImpl.kt (1)
13-17: 온보딩 데이터 소스 리팩토링이 적절하게 수행되었습니다.단일 회원가입 메서드를 닉네임 중복 검사와 사용자 정보 수정으로 분리하여 더욱 세분화된 API 호출이 가능해졌습니다. 구현 패턴도 일관성 있게 유지되었습니다.
domain/onboarding/src/main/java/com/teamsolply/solply/onboarding/repository/OnBoardingRepository.kt (2)
3-3: 임포트 변경이 적절합니다.
SignUpEntity에서UserInfoEntity로의 임포트 변경이 새로운 도메인 모델 구조와 일치합니다.
6-11: 메소드 분리가 단일 책임 원칙을 잘 따르고 있습니다.기존 signUp 메소드를 닉네임 중복 체크와 사용자 정보 패치로 분리한 것은 좋은 설계입니다. 각 메소드가 명확한 책임을 가지며, 테스트하기 쉬운 구조입니다.
data/onboarding/src/main/java/com/teamsolply/solply/onboarding/source/OnBoardingRemoteDataSource.kt (2)
3-5: 새로운 DTO 임포트가 적절합니다.
PatchUserInfoRequestDto,NicknameDuplicateResponseDto,PatchUserInfoResponseDto의 임포트가 새로운 API 구조와 일치합니다.
8-9: 메소드 시그니처가 적절하게 설계되었습니다.각 메소드가 명확한 입력과 출력을 가지며, 적절한 DTO를 사용하고 있습니다. 비동기 처리를 위한 suspend 함수로 정의된 것도 좋습니다.
data/oauth/src/main/java/com/teamsolply/solply/oauth/repository/OauthRepositoryImpl.kt (4)
5-5: 원격 데이터 소스 의존성 추가가 적절합니다.OAuth 원격 데이터 소스 의존성이 추가되어 소셜 로그인 기능을 지원할 수 있게 되었습니다.
9-10: 생성자 파라미터 순서를 일관되게 유지하세요.로컬 데이터 소스와 원격 데이터 소스의 순서가 일관되게 유지되어 가독성이 좋습니다.
13-27: 소셜 로그인 메소드가 잘 구현되었습니다.
runCatching과mapCatching을 사용한 에러 처리가 적절하며, DTO에서 도메인 엔티티로의 매핑이 올바릅니다.isNewUser필드를 포함한 것도 좋은 설계입니다.
29-35: JWT 토큰 저장 메소드 시그니처 변경이 적절합니다.개별 토큰 파라미터를 받도록 변경한 것이 더 유연하고 사용하기 쉬운 API를 제공합니다.
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/NamingScreen.kt (3)
24-26: 제어형 컴포넌트로의 리팩토링이 훌륭합니다.
inputNickname,isNicknameDuplicate파라미터와changeInputNickname콜백을 추가하여 컴포넌트를 완전히 제어 가능하게 만든 것은 좋은 설계입니다. 이는 상태 관리를 외부화하여 더 예측 가능하고 테스트하기 쉬운 구조를 만듭니다.
50-52: 외부 상태 사용이 적절합니다.
inputNickname과changeInputNickname콜백을 사용하여 상태 관리가 외부화되었습니다.state.isNicknameDuplicate를 사용한 것도 일관된 상태 관리 패턴을 보여줍니다.
64-70: 버튼 활성화 로직이 일관되게 구현되었습니다.
state.userNickname을 사용하여 버튼 활성화 상태와 텍스트 색상을 일관되게 처리한 것이 좋습니다. UI 상태가 명확하고 예측 가능합니다.data/onboarding/src/main/java/com/teamsolply/solply/onboarding/repository/OnBoardingRepositoryImpl.kt (3)
3-4: 새로운 임포트가 적절합니다.
PatchUserInfoRequestDto와UserInfoEntity임포트가 새로운 API 구조와 일치합니다.
11-13: 닉네임 중복 체크 메소드가 간결하게 구현되었습니다.
runCatching을 사용한 에러 처리가 적절하며, 원격 데이터 소스에서 받은isDuplicated값을 직접 반환하는 것이 효율적입니다.
15-34: 사용자 정보 패치 메소드가 잘 구현되었습니다.DTO 생성, 원격 데이터 소스 호출, 도메인 엔티티로의 매핑이 순서대로 적절하게 구현되었습니다.
runCatching과mapCatching을 사용한 에러 처리도 올바릅니다.feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthContract.kt (3)
13-13: OAuth 토큰과 JWT 토큰 처리 분리가 잘 이루어졌습니다.
KakaoLoginSuccess에서refreshToken을 제거하고provider와accessToken만 포함하도록 변경한 것은 OAuth 토큰 교환과 JWT 토큰 저장을 분리하는 좋은 아키텍처 결정입니다.
15-19: 새로운 JWT 토큰 저장 인텐트가 잘 설계되었습니다.
SaveJwtToken인텐트는 필요한 모든 정보(accessToken, refreshToken, isNewUser)를 포함하고 있어 JWT 토큰 저장과 신규 사용자 분기 처리를 명확하게 지원합니다.
25-25: Place 화면 네비게이션을 위한 사이드 이펙트가 적절히 추가되었습니다.
NavigateToPlace사이드 이펙트는 기존 사용자가 로그인 후 메인 화면으로 이동하는 플로우를 명확하게 표현합니다.remote/onboarding/src/main/java/com/teamsolply/solply/onboarding/service/OnBoardingService.kt (2)
14-17: 닉네임 중복 체크 API가 잘 설계되었습니다.GET 메서드를 사용하여 닉네임 중복을 체크하는 것은 REST 원칙에 맞으며, 쿼리 파라미터를 통한 닉네임 전달도 적절합니다.
19-22: 사용자 정보 업데이트 API가 적절히 구현되었습니다.PATCH 메서드를 사용하여 사용자 정보를 부분 업데이트하는 것은 의미적으로 올바르며, 요청 DTO를 통한 데이터 전달도 타입 안전성을 보장합니다.
feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthViewModel.kt (2)
19-22: OAuth 토큰 교환과 JWT 토큰 저장이 잘 분리되었습니다.
KakaoLoginSuccess처리에서postSocialLogin을 호출하여 OAuth 토큰을 JWT 토큰으로 교환하는 단계적 접근 방식은 명확하고 유지보수성이 좋습니다.
28-41: JWT 토큰 저장과 네비게이션 로직이 적절히 구현되었습니다.
SaveJwtToken인텐트 처리에서 신규 사용자 여부에 따라 온보딩 또는 Place 화면으로 분기하는 로직이 명확하고 올바릅니다.feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingScreen.kt (2)
71-73: 닉네임 입력 콜백이 적절히 구현되었습니다.
changeInputNickname콜백을 통해 닉네임 입력을 외부에서 관리하도록 변경한 것은 상태 관리를 ViewModel로 집중화하는 좋은 아키텍처 결정입니다.
176-178: NamingScreen 파라미터 전달이 잘 구현되었습니다.
inputNickname,isNicknameDuplicate,changeInputNickname파라미터를 전달하여 닉네임 중복 체크 기능을 지원하는 것이 적절합니다.
| Log.d(TAG, "Request URL: ${newRequest.url}") | ||
| Log.d( | ||
| TAG, | ||
| "Authorization header: ${if (newRequest.header("Authorization") != null) "Bearer [REDACTED]" else "null"}" | ||
| ) |
There was a problem hiding this comment.
프로덕션 환경에서 민감한 정보 로깅의 보안 위험이 있습니다.
조건부 디버그 로깅을 제거하고 모든 요청에 대해 URL과 인증 헤더 정보를 무조건 로깅하는 것은 보안상 위험합니다. 프로덕션 빌드에서는 민감한 정보가 로그에 노출될 수 있습니다.
다음과 같이 수정하여 디버그 빌드에서만 로깅하도록 개선하세요:
- Log.d(TAG, "Request URL: ${newRequest.url}")
- Log.d(
- TAG,
- "Authorization header: ${if (newRequest.header("Authorization") != null) "Bearer [REDACTED]" else "null"}"
- )
+ if (BuildConfig.DEBUG) {
+ Log.d(TAG, "Request URL: ${newRequest.url}")
+ Log.d(
+ TAG,
+ "Authorization header: ${if (newRequest.header("Authorization") != null) "Bearer [REDACTED]" else "null"}"
+ )
+ }📝 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.
| Log.d(TAG, "Request URL: ${newRequest.url}") | |
| Log.d( | |
| TAG, | |
| "Authorization header: ${if (newRequest.header("Authorization") != null) "Bearer [REDACTED]" else "null"}" | |
| ) | |
| if (BuildConfig.DEBUG) { | |
| Log.d(TAG, "Request URL: ${newRequest.url}") | |
| Log.d( | |
| TAG, | |
| "Authorization header: ${if (newRequest.header("Authorization") != null) "Bearer [REDACTED]" else "null"}" | |
| ) | |
| } |
🤖 Prompt for AI Agents
In
core/network/src/main/java/com/teamsolply/solply/network/AccessTokenInterceptor.kt
around lines 35 to 39, the current code logs request URLs and authorization
headers unconditionally, which risks exposing sensitive information in
production logs. Modify the code to wrap these log statements inside a check
that ensures they only execute in debug builds, such as using a build
configuration flag or a debug mode check, to prevent sensitive data from being
logged in production environments.
| .currentBackStackEntryAsState().value?.destination | ||
|
|
||
| val startDestination = OnBoarding | ||
| val startDestination = Splash |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
시작 화면 변경이 앱 플로우에 미치는 영향을 검증하세요.
시작 화면을 OnBoarding에서 Splash로 변경하는 것은 앱의 초기 사용자 경험을 크게 바꿉니다. 스플래시 화면에서 적절한 시점에 온보딩이나 OAuth 로그인으로 이동하는 로직이 올바르게 구현되어 있는지 확인해야 합니다.
스플래시 화면에서 온보딩/OAuth로의 네비게이션 로직을 확인하세요:
🏁 Script executed:
#!/bin/bash
# 스플래시 화면에서 다른 화면으로의 네비게이션 로직 검색
fd -t f -e kt | xargs grep -l "Splash" | xargs grep -A 10 -B 5 "navigate"Length of output: 17757
Splash 화면에서 온보딩 로직 누락 확인 및 보완 필요
Splash를 시작 화면으로 변경하면서 실제로 OnBoarding으로의 네비게이션이 빠져 있는 것으로 보입니다. MainNavigator에는 여전히 navigateToOnboarding 메서드(import 포함)가 남아 있으나, SplashNavigation 및 SplashScreen 쪽에 온보딩으로 이동하는 콜백이 전달·처리되지 않습니다.
조치가 필요한 위치:
- feature/main/src/main/java/com/teamsolply/solply/main/MainNavigator.kt
•fun navigateToOnboarding(navOptions: NavOptions)존재 - feature/main/src/main/java/com/teamsolply/solply/main/splash/SplashNavigation.kt
•splashNavGraph호출부에 OnBoarding 콜백 전달부 없음 - feature/main/src/main/java/com/teamsolply/solply/main/splash/SplashScreen.kt
•SplashSideEffect에 OnBoarding 관련 케이스가 정의되어 있는지 확인
제안
- 신규 사용자에게만 OnBoarding으로 분기하도록 SplashViewModel에서 SideEffect를 추가하고, SplashNavigation에
navigateToOnboarding콜백을 연결하거나 - 온보딩 플로우 자체를 완전히 제거하려면 MainNavigator 및 관련 import를 정리
위 사항을 반영해 앱 플로우가 의도한 대로 동작하는지 검증 후 수정 바랍니다.
🤖 Prompt for AI Agents
In feature/main/src/main/java/com/teamsolply/solply/main/MainNavigator.kt at
line 29 and related files, the navigation from Splash screen to OnBoarding is
missing despite the presence of navigateToOnboarding method. Fix this by adding
a SideEffect in SplashViewModel to trigger OnBoarding navigation for new users,
passing the navigateToOnboarding callback through SplashNavigation's
splashNavGraph, and handling the OnBoarding case in SplashScreen's
SplashSideEffect. Alternatively, if OnBoarding is to be removed, clean up
MainNavigator and imports accordingly. Verify the app flow works as intended
after these changes.
| release { | ||
| signingConfig = signingConfigs.getByName("debug") | ||
| isMinifyEnabled = false | ||
| proguardFiles(getDefaultProguardFile("proguard-android-optimize.txt"), "proguard-rules.pro") | ||
| } |
There was a problem hiding this comment.
릴리즈 빌드에서 디버그 키스토어 사용 및 난독화 비활성화 문제
릴리즈 빌드에서 디버그 키스토어를 사용하고 있으며, 코드 난독화가 비활성화되어 있습니다. 이는 보안상 위험할 수 있습니다.
다음 수정사항을 적용하세요:
+ create("release") {
+ keyAlias = "release"
+ keyPassword = "your_release_password"
+ storeFile = File("${project.rootDir.absolutePath}/keystore/release.keystore")
+ }
}
buildTypes {
debug {
signingConfig = signingConfigs.getByName("debug")
}
release {
- signingConfig = signingConfigs.getByName("debug")
+ signingConfig = signingConfigs.getByName("release")
- isMinifyEnabled = false
+ isMinifyEnabled = true
+ isShrinkResources = true
proguardFiles(getDefaultProguardFile("proguard-android-optimize.txt"), "proguard-rules.pro")
}🤖 Prompt for AI Agents
In app/build.gradle.kts around lines 22 to 26, the release build is incorrectly
using the debug signingConfig and has minify (code obfuscation) disabled, which
poses security risks. Change the signingConfig to use the proper release
keystore configuration instead of "debug" and enable minify by setting
isMinifyEnabled to true. Also, ensure the proguardFiles are correctly configured
for release builds to enable code obfuscation and optimization.
| @POST("api/auth/social/{soicialPlatform}/login") | ||
| suspend fun postSocialLogin( | ||
| @Path("soicialPlatform") soicialPlatform: String, | ||
| @Body oauthAccessToken: SocialLoginRequestDto | ||
| ): BaseResponse<SocialLoginResponseDto> |
There was a problem hiding this comment.
경로 매개변수명 오타가 있습니다.
@Path("soicialPlatform")에서 "soicialPlatform"은 "socialPlatform"으로 수정되어야 합니다. 매개변수명도 함께 수정하는 것이 좋겠습니다.
- @POST("api/auth/social/{soicialPlatform}/login")
+ @POST("api/auth/social/{socialPlatform}/login")
suspend fun postSocialLogin(
- @Path("soicialPlatform") soicialPlatform: String,
+ @Path("socialPlatform") socialPlatform: String,
@Body oauthAccessToken: SocialLoginRequestDto
): BaseResponse<SocialLoginResponseDto>🤖 Prompt for AI Agents
In
remote/oauth/src/main/java/com/teamsolply/solply/oauth/service/OauthService.kt
lines 11 to 15, there is a typo in the path parameter name "soicialPlatform"
which should be corrected to "socialPlatform". Update both the @Path annotation
and the corresponding function parameter name to "socialPlatform" to ensure
consistency and correct routing.
| provider: String, | ||
| oauthAccessToken: String | ||
| ): SocialLoginResponseDto = oauthService.postSocialLogin( | ||
| soicialPlatform = provider, |
There was a problem hiding this comment.
매개변수 이름의 오타를 수정해주세요.
soicialPlatform → socialPlatform으로 수정해야 합니다.
- soicialPlatform = provider,
+ socialPlatform = provider,📝 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.
| soicialPlatform = provider, | |
| socialPlatform = provider, |
🤖 Prompt for AI Agents
In
remote/oauth/src/main/java/com/teamsolply/solply/oauth/datasource/OauthRemoteDataSourceImpl.kt
at line 16, there is a typo in the parameter name 'soicialPlatform'. Correct the
spelling to 'socialPlatform' to fix the issue.
| viewModelScope.launch { | ||
| onBoardingRepository.checkNicknameDuplicate(nickname = intent.nickname) | ||
| .onSuccess { | ||
| reduce { copy(isNicknameDuplicate = it) } | ||
| } | ||
| } |
There was a problem hiding this comment.
🛠️ Refactor suggestion
비동기 작업의 오류 처리를 추가해주세요.
닉네임 중복 검사에서 실패 케이스에 대한 처리가 누락되었습니다. 네트워크 오류나 서버 오류 시 사용자에게 적절한 피드백을 제공해야 합니다.
viewModelScope.launch {
onBoardingRepository.checkNicknameDuplicate(nickname = intent.nickname)
.onSuccess {
reduce { copy(isNicknameDuplicate = it) }
}
+ .onFailure {
+ // 오류 처리 로직 추가 (예: 에러 메시지 표시)
+ }
}📝 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.
| viewModelScope.launch { | |
| onBoardingRepository.checkNicknameDuplicate(nickname = intent.nickname) | |
| .onSuccess { | |
| reduce { copy(isNicknameDuplicate = it) } | |
| } | |
| } | |
| viewModelScope.launch { | |
| onBoardingRepository.checkNicknameDuplicate(nickname = intent.nickname) | |
| .onSuccess { | |
| reduce { copy(isNicknameDuplicate = it) } | |
| } | |
| .onFailure { | |
| // 오류 처리 로직 추가 (예: 에러 메시지 표시) | |
| } | |
| } |
🤖 Prompt for AI Agents
In
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingViewModel.kt
lines 46 to 51, the coroutine launched for checking nickname duplication handles
only the success case but lacks error handling. Add an onFailure block to handle
errors from checkNicknameDuplicate, and update the state or notify the user
appropriately to provide feedback on network or server errors.
1 participant
📌 PR 요약
🌱 작업한 내용
🌱 PR 포인트
📸 스크린샷
📮 관련 이슈
Summary by CodeRabbit
신규 기능
기능 개선
버그 수정
구성 및 의존성