🔗 :: (#468) 면접후기 기능 리팩토링 #473

uson1004 · 2025-10-28T07:09:23Z

개요

https://drive.google.com/file/d/1kPvDZ2Ta0tqk69P0ZkUZwY8VD3ph81UC/view?usp=share_link
면접 후기 작성 플로우 영상입니다
용량 제한으로 구글 드라이브로 올립니다

작업 내용

리뷰 작성
리뷰 조회, 리뷰 검색, 리뷰 필터링을 개발하였습니다
모듈을 생성하였습니다.
데이터클래스 형태의 인자를 주고받을 수 있도록 구현하였습니다
바텀 네비게이션을 추가하였습니다
ReviewState, Proccess 이넘 클래스 추가

할 말

compose ui에서 개선할 수 있는 부분 봐주시면 좋을 거 같습니다

Summary by CodeRabbit

릴리스 노트

새로운 기능
- 후기 작성 기능 추가 (인터뷰 유형, 위치, 기술, 면접관 수 기록)
- 후기 필터링 기능 (전공, 연도, 인터뷰 유형, 지역별)
- 후기 검색 기능 추가
- 하단 네비게이션에 후기 메뉴 추가
개선사항
- 후기 상세 정보 조회 화면 개선
- 지원 상태 업데이트 (새 상태 추가)
- API 필터링 옵션 확대

coderabbitai · 2025-10-28T07:10:51Z

📝 Walkthrough

🐰 코드 리뷰 요약

워크스루

post-review 기능 모듈을 새로 추가하고 리뷰 시스템을 전반적으로 리팩토링했습니다. 기존의 리뷰 상세 조회 및 작성 기능을 여러 화면으로 분리했으며, 인터뷰 타입과 위치 등 새로운 필터링 옵션을 추가하고 네비게이션 구조를 개편했습니다.

변경 사항

공통 요소	요약
앱 모듈 기본 설정 `app/build.gradle.kts`, `app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt`	post-review 기능 모듈 의존성 추가, 네비게이션 메서드 시그니처 업데이트 (companyName, companyId 추가, reviewId 타입 String → Long 변경)
네비게이션 리팩토링 `app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt`, `app/src/main/java/team/retum/jobisandroidv2/root/RootNavigation.kt`, `app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt`	4개의 post-review 목적지 추가, onPostReviewClick 서명 변경, 리뷰 필터/검색 콜백 추가
UI 리소스 `app/src/main/java/team/retum/jobisandroidv2/ui/BottomMenu.kt`, `app/src/main/java/team/retum/jobisandroidv2/ui/BottomNavigationBar.kt`, `app/src/main/res/drawable/ic_review.xml`, `app/src/main/res/values/strings.xml`	Review 메뉴 아이템 추가, 리뷰 아이콘 및 문자열 리소스 추가
공통 라이브러리 - Enum `core/common/src/main/java/team/retum/common/enums/*.kt`	ApplyStatus 상수 수정, InterviewLocation/InterviewType/ReviewProcess 새로운 enum 추가
공통 라이브러리 - 리소스 키 `core/common/src/main/java/team/retum/common/utils/ResourceKeys.kt`	REVIEW_DATA 상수 추가
데이터 계층 `core/data/src/main/java/team/retum/data/repository/review/ReviewRepository.kt`, `core/data/src/main/java/team/retum/data/repository/review/ReviewRepositoryImpl.kt`	fetchReviews 메서드 확장 (페이지네이션, 위치/타입 필터링 추가), fetchQuestions/fetchReviewsCount/fetchMyReviews 메서드 추가
디자인 시스템 `core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisColor.kt`, `core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisIcon.kt`, `core/design-system/src/main/res/drawable/ic_*.xml`	색상 값 수정, Asterisk 아이콘 추가, ic_success.xml 새로 추가
도메인 계층 - Entity `core/domain/src/main/java/team/retum/usecase/entity/*.kt`	FetchReviewDetailEntity/FetchReviewsEntity 필드 재정의, MyReviews/QuestionsEntity/ReviewsCountEntity 새로 추가, PostReviewEntity 확장
도메인 계층 - UseCase `core/domain/src/main/java/team/retum/usecase/usecase/review/*.kt`	FetchMyReviewUseCase/FetchQuestionsUseCase/FetchReviewsCountUseCase 추가, FetchReviewsUseCase 파라미터 확장
네트워크 계층 `core/network/src/main/java/team/retum/network/api/ReviewApi.kt`, `core/network/src/main/java/team/retum/network/datasource/review/*.kt`, `core/network/src/main/java/team/retum/network/di/RequestUrls.kt`	fetchReviews 쿼리 파라미터 기반으로 변경, 3개의 새로운 API 엔드포인트 추가, URL 경로 수정
네트워크 계층 - 응답 모델 `core/network/src/main/java/team/retum/network/model/request/PostReviewRequest.kt`, `core/network/src/main/java/team/retum/network/model/response/*.kt`	PostReviewRequest 필드 확장, FetchMyReviewResponse/FetchQuestionsResponse/FetchReviewsCountResponse 새로 추가, 기존 응답 모델 필드 수정
기능 모듈 - Company `feature/company/src/main/java/team/retum/company/navigation/CompanyDetailsNavigation.kt`, `feature/company/src/main/java/team/retum/company/ui/CompanyDetailsScreen.kt`, `feature/company/src/main/java/team/retum/company/viewmodel/CompanyDetailsViewModel.kt`	navigateToReviewDetails 서명 변경 (String,String → Long), fetchReviews 호출 파라미터 확장
기능 모듈 - Home `feature/home/src/main/java/team/retum/home/ui/ApplyCompanyItem.kt`	SEND/PROCESSING 상태에 대한 색상 매핑 추가
기능 모듈 - MyPage `feature/mypage/src/main/java/team/retum/jobis/navigation/MyPageNavigation.kt`, `feature/mypage/src/main/java/team/retum/jobis/ui/MyPageScreen.kt`	onPostReviewClick 서명 변경 (Long → String,Long)
기능 모듈 - Post-Review (신규) `feature/post-review/*`	전체 post-review 기능 모듈 신규 추가 (빌드 설정, 모델, 네비게이션, UI 화면, ViewModel, 리소스)
기능 모듈 - Review (리팩토링) `feature/review/build.gradle.kts`, `feature/review/src/main/java/team/retum/review/navigation/.kt`, `feature/review/src/main/java/team/retum/review/ui/.kt`, `feature/review/src/main/java/team/retum/review/viewmodel/*.kt`	네비게이션 구조 개편 (PostReviewNavigation 제거, ReviewFilter/SearchReviews/Review 추가), UI 화면 리팩토링 (PostReviewScreen/ReviewsScreen 제거, ReviewFilterScreen/SearchReviewScreen 추가), ViewModel 교체 (ReviewsViewModel → ReviewViewModel/SearchReviewsViewModel/ReviewFilterViewModel)
설정 파일 `settings.gradle.kts`	`:feature:post-review` 모듈 추가

시퀀스 다이어그램

sequenceDiagram
    actor User
    participant App as App Layer
    participant Nav as Navigation
    participant PostReview as Post-Review Feature
    participant Domain as Domain Layer
    participant Network as Network Layer

    User->>App: 리뷰 작성 시작
    App->>Nav: navigateToPostReview(companyName, companyId)
    Nav->>PostReview: PostReview 화면 표시
    
    User->>PostReview: 인터뷰 타입 선택
    PostReview->>PostReview: onNextClick()
    
    User->>PostReview: 위치/기술/개수 입력
    PostReview->>PostReview: 선택사항 수집
    
    User->>PostReview: 질문/답변 입력
    PostReview->>PostReview: 최종 데이터 수집
    
    User->>PostReview: 제출 버튼 클릭
    PostReview->>Domain: PostReviewUseCase 호출
    Domain->>Network: POST /reviews
    Network-->>Domain: Success/BadRequest
    
    alt 성공
        Domain-->>PostReview: Success
        PostReview->>Nav: navigateToPostReviewComplete()
        Nav->>PostReview: 완료 화면 표시 (1.5초 후)
        PostReview->>Nav: navigateToPostReview()
    else 실패
        Domain-->>PostReview: BadRequest
        PostReview->>User: 에러 토스트 표시
    end

예상 코드 리뷰 소요 시간

🎯 4 (복잡함) | ⏱️ ~60분

추가 검토 필요 영역:

네비게이션 흐름: MainNavigation, RootNavigation, RootScreen의 콜백 전파 구조와 post-review 모듈의 새로운 4단계 네비게이션 흐름 검증
데이터 모델 변경: ReviewProcess enum 상수 완전 교체로 기존 코드와의 호환성, FetchReviewDetailEntity 필드 대폭 재정의의 영향도 검토
API 변경: fetchReviews 메서드가 경로 기반에서 쿼리 파라미터 기반으로 전환되며 모든 호출부 검증 필요
Post-Review 모듈: 새로운 모듈의 전체 구조, PostReviewViewModel 상태 관리, 4개의 네비게이션 목적지 간 데이터 전달 메커니즘
Review 모듈 리팩토링: ReviewsScreen/PostReviewScreen 제거와 새로운 ReviewScreen/SearchReviewScreen/ReviewFilterScreen의 기능 동등성 확인
타입 변경: reviewId String→Long, navigateToReviewDetails 서명 변경(String,String→Long) 등의 광범위한 타입 변경으로 인한 일관성 검증

시

🐇 후기 기능이 피어났네요,

네 단계로 나뉜 화면들이 춤을 춘다네요 ✨

질문에 답변하고, 기술 고르고,

위치와 개수까지 담아내니

면접 후기가 더욱 자세해졌어요! 🎉

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.57% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.
Description check	⚠️ Warning	PR 설명이 필수 템플릿 구조를 따르지 않으며, 요구되는 정보가 불완전합니다.	PR 설명을 템플릿 형식에 맞춰 작성하세요: '개요' 섹션에 변경사항 요약 추가, '작업 내용' 섹션을 구조화하고, 필요시 '할 말' 섹션을 보강하세요.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title Check	✅ Passed	PR 제목 "🔗 :: (#468) 면접후기 기능 리팩토링"은 변경사항의 주요 목적을 명확하게 전달합니다. raw_summary 분석 결과, 이 PR은 새로운 feature/post-review 모듈 생성, 리뷰 작성/조회/검색/필터링 기능 구현, 기존 리뷰 화면 재구조화 등을 포함하고 있으며, 제목의 "면접후기 기능 리팩토링"이 이러한 변경사항을 적절히 요약합니다. 제목은 간결하고 구체적이며, 팀원이 히스토리를 스캔할 때 PR의 주요 변경 내용을 충분히 이해할 수 있습니다.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feature/468-면접후기-기능-리팩토링

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

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai · 2025-10-28T07:19:47Z

Note

Docstrings generation - SKIPPED
Skipped regeneration as there are no new commits. Docstrings already generated for this pull request at #474.

coderabbitai · 2025-10-28T07:20:01Z

Note

Unit test generation is an Early Access feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Generating unit tests... This may take up to 20 minutes.

coderabbitai

Actionable comments posted: 29

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt (1)

22-24: reviewId 타입 불일치(Long vs String) 명확히 해야 함

내비게이션/UI 계층에서는 Long 사용(ReviewItems, ReviewDetailsNavigation), ViewModel과 네트워크 계층에서는 String 사용합니다. ReviewDetailsScreen 62줄에서 .toString() 변환이 발생하며, 형변환 오류 가능성이 있습니다.

도메인 전역으로 Long으로 통일하거나, 경계 변환 로직을 명확히 정의하는 것을 권장합니다.

Also applies to: 39-61, feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt:62
app/src/main/java/team/retum/jobisandroidv2/root/RootNavigation.kt (1)
37-41: navArgument 설정 오타로 인한 런타임 크래시 소지(필수 수정)

navArgument(APPLICATION_ID) { NavType.StringType }는 type = ...이 누락되었습니다. 또한 실제로 Long으로 사용하므로 LongType으로 일치시키는 편이 안전합니다.
-        arguments = listOf(navArgument(APPLICATION_ID) { NavType.StringType }),
+        arguments = listOf(navArgument(APPLICATION_ID) { type = NavType.LongType }),
@@
-            applicationId = it.arguments?.getString(APPLICATION_ID)?.toLong(),
+            applicationId = it.arguments?.getLong(APPLICATION_ID),

🧹 Nitpick comments (55)

core/network/src/main/java/team/retum/network/api/InterestsApi.kt (1)

1-3: 빈 인터페이스의 용도를 명확히 해주세요.

이 인터페이스가 현재 비어있어 실제 기능이 없습니다. PR의 목적이 면접 후기 기능 리팩토링인데, InterestsApi라는 네이밍과의 연관성도 불분명합니다.

다음 사항을 확인해주세요:

이것이 의도적인 플레이스홀더 코드인지 확인

추후 구현 예정이라면 TODO 주석이나 문서화 추가

이 API가 PR의 리뷰 기능과 어떻게 연관되는지 설명

향후 구현할 API 엔드포인트를 정의하는 데 도움이 필요하시면 말씀해주세요.
core/network/src/main/java/team/retum/network/model/response/FetchReviewDetailResponse.kt (1)
18-18: 리스트 프로퍼티 네이밍 가독성 개선 제안

List 타입이므로 qnaResponse → qnaResponses(또는 qnas)로의 복수형 네이밍을 권장합니다. 외부 파급이 크면 보류해도 됩니다.

선호 시 최소 변경 diff:
-    @Json(name = "qn_as") val qnaResponse: List<QnAs>,
+    @Json(name = "qn_as") val qnaResponses: List<QnAs>,
feature/post-review/.gitignore (1)

1-1: 모듈 .gitignore의 /build 규칙은 루트에서 이미 무시되고 있습니다

검증 결과, 루트 .gitignore(3줄)의 build/ 규칙이 모든 수준의 build 디렉토리를 이미 무시하고 있으므로, 모듈 단위 /build 규칙 21개 모두 중복입니다. 이 파일의 /build 추가는 기술적으로는 문제없지만 불필요합니다. 선택적으로 제거하거나, 모듈별 명시성이 필요하다면 유지해도 무방합니다.

core/network/src/main/java/team/retum/network/model/response/FetchMyReviewResponse.kt (1)

6-14: reviewId 타입 합치기 (Long vs String) 권장

네트워크 모델은 Long(reviewId)인데, feature/review의 ViewModel 상태는 String(reviewId)입니다. 경계에서 변환을 강제하거나, 전역적으로 Long으로 통일하는 쪽이 안전합니다. 타입 혼재는 내비게이션/UseCase 파라미터에서 런타임 이슈를 유발할 수 있습니다.

또한 Moshi codegen이 core:network 모듈에서 KSP/kapt로 활성화돼 있는지 확인 부탁드립니다(리플렉션 회피 성능 이점).

옵션: 네트워크 일시 필드 누락에 견고하게 하려면 기본값을 부여하는 것도 고려 가능합니다.
core/design-system/src/main/res/drawable/ic_success.xml (1)
6-8: 아이콘 색상 하드코딩 지양 (틴트 기반 제어 권장)

디자인시스템에서 일관된 테마 틴트를 쓰려면 path의 fillColor를 무채색(예: #000000)으로 두고 사용처(Icon/Image)에서 tint를 적용하는 편이 좋습니다.

예시 변경:
-      android:fillColor="#2F53FF"/>
+      android:fillColor="#000000"/>
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/review/ReviewContent.kt (1)
48-51: 아이콘 접근성: contentDescription 처리

해당 화살표가 장식 목적이면 contentDescription = null 권장. 의미가 있다면 stringResource로 지역화된 문자열을 사용하세요.

예시:
-                contentDescription = "long arrow",
+                contentDescription = null,
feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt (1)
26-32: 실패/로딩 상태 처리 추가 권장

onFailure 처리와 로딩 플래그를 상태에 포함해 UI가 적절히 대응하도록 해주세요(스낵바, 에러 뷰 등).

간단 예시:
     internal fun fetchReviewDetails() {
         viewModelScope.launch(Dispatchers.IO) {
-            fetchReviewDetailsUseCase(state.value.reviewId).onSuccess {
-                setState { state.value.copy(reviewDetail = it) }
-            }
+            fetchReviewDetailsUseCase(state.value.reviewId)
+                .onSuccess { setState { state.value.copy(reviewDetail = it /*, isLoading = false*/) } }
+                .onFailure { /* setState { state.value.copy(error = it, isLoading = false) } */ }
         }
     }
feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (2)
45-49: 중첩 컴포저블에 외부 modifier 재사용 지양

내부 Row에서는 Modifier를 새로 시작하는 편이 안전합니다(예상치 못한 제약/weight 전파 방지).
-        Row(
-            modifier = modifier
+        Row(
+            modifier = Modifier
             .weight(1f)
             .clickable(onClick = { onReviewDetailClick(reviewId) }),
51-77: a11y 및 토큰 사용 개선: contentDescription/색상

AsyncImage의 contentDescription은 stringResource로 지역화하거나 장식이면 null 처리 권장.

보조 텍스트 색은 onPrimary보다 onSurfaceVariant가 토큰 의미에 맞습니다.
-                contentDescription = "company profile",
+                contentDescription = null, // 또는 stringResource(R.string.cd_company_profile)

...
-                    color = JobisTheme.colors.onPrimary,
+                    color = JobisTheme.colors.onSurfaceVariant,
feature/review/src/main/java/team/retum/review/viewmodel/SearchReviewsViewModel.kt (3)
33-42: 검색 디바운스 파이프라인: flatMapLatest로 이전 요청 취소

현재는 매 키워드마다 별도 코루틴을 띄워 오래 걸린 이전 요청 결과가 나중에 덮어쓸 수 있습니다. flatMapLatest 기반으로 취소 가능 파이프라인으로 전환을 권장합니다. 공백 트리밍도 포함했습니다.
-    @OptIn(FlowPreview::class)
-    private fun debounceName() {
-        viewModelScope.launch {
-            state.map { it.keyword }.distinctUntilChanged().debounce(SEARCH_DEBOUNCE_MILLIS).collect {
-                if (!it.isNullOrBlank()) {
-                    fetchReviews()
-                }
-            }
-        }
-    }
+    @OptIn(FlowPreview::class, kotlinx.coroutines.ExperimentalCoroutinesApi::class)
+    private fun debounceName() {
+        viewModelScope.launch {
+            state
+                .map { it.keyword?.trim() }
+                .distinctUntilChanged()
+                .debounce(SEARCH_DEBOUNCE_MILLIS)
+                .flatMapLatest { kw ->
+                    kotlinx.coroutines.flow.flow {
+                        if (kw.isNullOrBlank()) {
+                            // 빈 키워드: 결과 초기화
+                            setState { state.value.copy(reviews = emptyList(), showRecruitmentsEmptyContent = false) }
+                            return@flow
+                        }
+                        val result = fetchReviewsUseCase(
+                            companyId = null, page = null, location = null,
+                            interviewType = null, keyword = kw, year = null, code = null
+                        )
+                        emit(result)
+                    }
+                }
+                .collect { res ->
+                    res.onSuccess {
+                        setState {
+                            state.value.copy(
+                                showRecruitmentsEmptyContent = it.reviews.isEmpty(),
+                                reviews = it.reviews,
+                            )
+                        }
+                    }
+                    // TODO: onFailure 처리도 추가 권장
+                }
+        }
+    }
56-61: 빈 키워드 초기화 및 실패 처리 추가 권장

키워드가 비워졌을 때 리스트 초기화, 실패 시 에러 상태 반영이 필요합니다. 위 flatMapLatest 예시에 포함했습니다.

70-73: 네이밍 정합성: showRecruitmentsEmptyContent → showReviewsEmptyContent

리뷰 검색 화면이라면 프로퍼티명을 리뷰 맥락으로 맞추는 것이 가독성에 좋습니다. 전파 영향이 크니 후속 PR에서 점진 적용을 권장합니다.
feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (3)
172-189: 항목 렌더링을 개선할 수 있습니다.

forEachIndexed를 사용한 항목 렌더링이 작동하지만, Compose에서는 Row와 items.joinToString()을 사용하거나 items.flatMapIndexed로 구분자를 추가하는 방식이 더 선언적입니다.

다음과 같이 리팩토링할 수 있습니다:
Row(
    modifier = Modifier
        .fillMaxWidth()
        .padding(top = 10.dp),
    horizontalArrangement = Arrangement.spacedBy(8.dp),
) {
    items.forEachIndexed { index, item ->
        JobisText(
            text = item,
            style = JobisTypography.SubBody,
            color = JobisTheme.colors.inverseOnSurface,
        )
        if (index < items.size - 1) {
            JobisText(
                text = "•",
                style = JobisTypography.SubBody,
                color = JobisTheme.colors.inverseOnSurface,
            )
        }
    }
}
현재 코드도 정상 작동하므로 필수는 아닙니다.

266-349: 리스트 렌더링에 안정적인 키를 사용하는 것을 권장합니다.

forEachIndexed로 리스트를 렌더링할 때 각 항목의 고유 식별자를 키로 사용하지 않고 있습니다. 리스트가 동적으로 변경될 경우 상태 보존 문제가 발생할 수 있습니다. 가능하다면 LazyColumn과 items() 함수를 사용하거나, 각 항목에 key 파라미터를 제공하는 것을 권장합니다.

FetchReviewDetailEntity.QnAs에 고유 ID가 있다면 다음과 같이 사용할 수 있습니다:
// LazyColumn 사용 예시
LazyColumn {
    items(
        items = review,
        key = { it.id } // 고유 ID가 있다고 가정
    ) { reviewItem ->
        // 카드 렌더링 로직
    }
}
60-64: ViewModel에서 ID를 Long 타입으로 처리하도록 리팩터링하세요.

검증 결과, ReviewDetailsViewModel.setReviewId()는 String을 기대하지만, ReviewDetails 함수의 reviewId 매개변수는 Long입니다. 이로 인해 62번째 줄에서 불필요한 toString() 변환이 발생합니다.

ID는 본질적으로 Long 타입이므로, ViewModel의 상태 관리도 String 대신 Long을 사용하도록 수정하는 것이 좋습니다. 이렇게 하면 타입 안정성이 개선되고, 불필요한 문자열 변환을 제거할 수 있습니다.

변경 사항:

ReviewDetailsViewModel.setReviewId() 메서드의 매개변수를 String에서 Long으로 변경

상태의 reviewId 필드도 Long 타입으로 변경

ReviewDetailsScreen.kt 62번째 줄에서 reviewId.toString() 제거
feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (1)

204-204: TODO 항목이 추가되었습니다.

지원 후 홈 화면 진입 시 UI 즉시 반영과 관련된 TODO 항목입니다. 이 기능 구현이 필요한지 확인해 주세요.

이 TODO 항목 해결을 위한 이슈 생성이나 구현을 도와드릴 수 있습니다. 필요하신가요?
core/domain/src/main/java/team/retum/usecase/entity/QuestionsEntity.kt (2)
16-16: TODO 주석 처리 필요

이름 통일에 대한 TODO가 남아있습니다. 네이밍 일관성 문제를 해결하거나, 다른 이슈로 추적이 필요하다면 이슈를 생성해주세요.

이슈 생성을 원하시면 도움을 드릴 수 있습니다.

6-14: @Immutable 어노테이션 일관성 개선 권장

중첩된 QuestionEntity에만 @Immutable 어노테이션이 적용되어 있습니다. 부모 클래스 QuestionsEntity도 불변 객체이므로 일관성을 위해 동일한 어노테이션을 추가하는 것을 권장합니다.

다음 diff를 적용하여 일관성을 개선하세요:
+@Immutable
 data class QuestionsEntity(
     val questions: List<QuestionEntity>,
 ) {
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchReviewsUseCase.kt (1)
12-14: 다수의 nullable 파라미터를 데이터 클래스로 리팩토링 권장

7개의 nullable 파라미터를 가진 함수는 유지보수가 어렵고 호출 시 실수하기 쉽습니다. 필터 파라미터를 묶는 데이터 클래스(예: ReviewFilters)를 사용하는 것을 권장합니다.

다음과 같이 리팩토링을 고려해보세요:
data class ReviewFilters(
    val page: Int? = null,
    val location: InterviewLocation? = null,
    val interviewType: InterviewType? = null,
    val keyword: String? = null,
    val year: Int? = null,
    val companyId: Long? = null,
    val code: Long? = null,
)

class FetchReviewsUseCase @Inject constructor(
    private val reviewRepository: ReviewRepository,
) {
    suspend operator fun invoke(filters: ReviewFilters = ReviewFilters()) = runCatching {
        reviewRepository.fetchReviews(
            filters.page,
            filters.location,
            filters.interviewType,
            filters.keyword,
            filters.year,
            filters.companyId,
            filters.code
        ).toEntity()
    }
}
feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt (2)
77-89: 리스트 키 지정 및 리스트 기반 items 사용 권장

인덱스 기반 items(size)는 항목 변경 시 상태 재사용 문제가 생길 수 있습니다. reviewId로 key를 지정하고 리스트 기반 DSL을 쓰세요.
-        LazyColumn {
-            items(state.reviews.size) {
-                val review = state.reviews[it]
-                ReviewItems(
-                    companyImageUrl = review.companyLogoUrl,
-                    companyName = review.companyName,
-                    reviewId = review.reviewId,
-                    writer = review.writer,
-                    major = review.major,
-                    onReviewDetailClick = onReviewDetailClick,
-                )
-            }
-        }
+        LazyColumn {
+            items(
+                items = state.reviews,
+                key = { it.reviewId },
+            ) { review ->
+                ReviewItems(
+                    companyImageUrl = review.companyLogoUrl,
+                    companyName = review.companyName,
+                    reviewId = review.reviewId,
+                    writer = review.writer,
+                    major = review.major,
+                    onReviewDetailClick = onReviewDetailClick,
+                )
+            }
+        }
필요 시 import 추가:
import androidx.compose.foundation.lazy.items
65-75: a11y/i18n: contentDescription 리소스화 권장

"filter", "search" 하드코딩 대신 string 리소스를 사용해 접근성과 현지화를 보완하세요.
-            JobisIconButton(
-                drawableResId = JobisIcon.Filter,
-                contentDescription = "filter",
+            JobisIconButton(
+                drawableResId = JobisIcon.Filter,
+                contentDescription = stringResource(R.string.cd_filter),
                 onClick = onReviewFilterClick,
                 tint = JobisTheme.colors.onPrimary,
             )
-            JobisIconButton(
-                drawableResId = JobisIcon.Search,
-                contentDescription = "search",
+            JobisIconButton(
+                drawableResId = JobisIcon.Search,
+                contentDescription = stringResource(R.string.cd_search),
                 onClick = onSearchReviewClick,
             )
(리소스가 없다면 추가 필요)
feature/review/src/main/java/team/retum/review/ui/SearchReviewScreen.kt (2)
52-65: 리스트는 items(list, key) 사용으로 안정 키 제공 및 간결화 권장

현재 index 기반 items(size) 사용은 스크롤/애니메이션 시 재조합 비용이 커집니다. reviewId를 key로 사용하세요.

적용 예시:
@@
-        if (!state.showRecruitmentsEmptyContent) {
-            LazyColumn {
-                items(state.reviews.size) {
-                    val review = state.reviews[it]
-                    ReviewItems(
-                        companyImageUrl = review.companyLogoUrl,
-                        companyName = review.companyName,
-                        reviewId = review.reviewId,
-                        writer = review.writer,
-                        major = review.major,
-                        onReviewDetailClick = onReviewDetailClick,
-                    )
-                }
-            }
+        if (!state.showRecruitmentsEmptyContent) {
+            LazyColumn {
+                items(
+                    items = state.reviews,
+                    key = { it.reviewId },
+                ) { review ->
+                    ReviewItems(
+                        companyImageUrl = review.companyLogoUrl,
+                        companyName = review.companyName,
+                        reviewId = review.reviewId,
+                        writer = review.writer,
+                        major = review.major,
+                        onReviewDetailClick = onReviewDetailClick,
+                    )
+                }
+            }
추가: 아래 import 필요
import androidx.compose.foundation.lazy.items
52-52: 프로퍼티 네이밍 불일치

showRecruitmentsEmptyContent는 리뷰 화면 맥락과 맞지 않습니다. showReviewsEmptyContent 등 의미 맞는 이름으로 정리하면 가독성이 좋아집니다.
core/domain/src/main/java/team/retum/usecase/entity/FetchReviewsEntity.kt (1)

10-18: @immutable를 상위 엔티티에도 부여 고려

UI에서 이 엔티티를 직접 소비한다면, FetchReviewsEntity에도 @Immutable을 부여하면 Compose 최적화에 도움됩니다. 현재 내부 Review에만 지정되어 있어 일관성이 떨어집니다.

feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt (2)

22-27: Companion object에 선택값 저장은 전역 사이드이펙트

전역 var(code/year/interviewType/location)는 프로세스 생명주기/복원 시 예기치 않은 상태 누수를 유발합니다. SavedStateHandle, 공유 ViewModel, 또는 단일 AppliedFilters 상태로 관리하는 방식으로 전환을 권장합니다.

33-47: 에러 핸들링/빈 상태 처리 추가 권장

fetchCodeUseCase 실패 시 사용자 피드백/리트라이가 없습니다. .onFailure 처리 및 로딩/에러 상태 반영을 고려해 주세요.

core/domain/src/main/java/team/retum/usecase/entity/MyReviews.kt (1)

3-13: Immutable 어노테이션 일관성

이 파일은 javax.annotation.concurrent.Immutable, 다른 파일은 androidx.compose.runtime.Immutable를 사용합니다. 모듈 전반 일관된 어노테이션(가능하면 Compose의 @Immutable) 사용을 권장합니다.
feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt (3)
81-83: 하드코딩 문자열 -> stringResource로 i18n 정리

여러 UI 텍스트가 하드코딩되어 있습니다. 다국어/접근성/일관성 위해 stringResource로 이전하세요.

예시(패턴 동일 적용):
- title = "필터 설정",
+ title = stringResource(R.string.review_filter_title),
@@
- text = "전공",
+ text = stringResource(R.string.major),
@@
- title = "개인 면접",
+ title = stringResource(R.string.interview_type_individual),
필요한 키를 strings.xml에 추가해 주세요.

Also applies to: 133-136, 167-170, 198-201, 230-233, 203-216, 235-253

106-118: 하단 고정 버튼 시스템 인셋/오버랩 처리

스크롤 콘텐츠와 버튼이 겹칠 수 있습니다. navigationBarsPadding()과 여백을 주고, 스크롤 영역 하단에 Spacer를 넣어 가려짐을 방지하세요.
-        JobisButton(
+        JobisButton(
             text = stringResource(id = R.string.appliance),
             onClick = {
@@
-            modifier = Modifier.align(Alignment.BottomCenter),
+            modifier = Modifier
+                .align(Alignment.BottomCenter)
+                .navigationBarsPadding()
+                .padding(bottom = 16.dp),
             color = ButtonColor.Primary,
         )
스크롤 영역 끝에 여유 공간도 추가:
// ReviewFilterScreen 내부 Column 맨 아래
Spacer(modifier = Modifier.height(96.dp))
109-113: UI에서 ViewModel.Companion에 직접 쓰기 지양

UI가 ReviewFilterViewModel.Companion.*에 직접 접근해 외부 상태를 수정합니다. 적용 로직은 ViewModel 메서드(예: applyAndClose(onBackPressed))로 캡슐화하세요. 테스트 용이성과 추적성이 좋아집니다.

간단 예시:
// ViewModel
fun apply(onApplied: () -> Unit) = setState {
    // 필요 시 영속화/저장 후
    state.value // no-op; 혹은 저장 로직
}.also { onApplied() }

// UI
onClick = { reviewFilterViewModel.apply(onBackPressed) }
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (2)
67-73: 초기 answers 길이 하드코딩(3) 제거 권장

질문 수와 무관하게 3개로 고정되어 있어 이후 단계에서 불일치가 발생할 수 있습니다. 초기값은 빈 리스트로 두고, fetch 성공 시 질문 수에 맞춰 동기화하세요.
         fun getInitialState() = PostNextReviewState(
             questions = emptyList(),
             buttonEnabled = false,
             answer = "",
-            answers = listOf("", "", ""),
+            answers = emptyList(),
             qnaElements = emptyList(),
         )
59-65: PostNextReviewState에서 unused 필드 answer 제거 필요

검증 결과, PostNextReviewState.answer 필드는 PostNextReviewViewModel의 어느 메서드에서도 사용되지 않습니다. 초기화만 되고(line 70) 읽거나 수정되지 않으므로 제거하세요.

관련 코드:

fetchQuestions(), setQuestion(), onNextClick() 모두 answer 필드 미사용

대신 answers: List<String> 필드가 실제 답변 관리에 사용됨
feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt (1)
26-28: 중복 내비게이션 방지 옵션 추가

탭 재선택 등에서 동일 목적지 중복 쌓임을 방지하려면 launchSingleTop(및 필요 시 restoreState)을 사용하는 것이 안전합니다.
-fun NavController.navigateToReview() {
-    navigate(NAVIGATION_REVIEW)
-}
+fun NavController.navigateToReview() {
+    navigate(NAVIGATION_REVIEW) {
+        launchSingleTop = true
+        restoreState = true
+    }
+}
feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt (1)
62-66: contentDescription 하드코딩

i18n/접근성 일관성을 위해 stringResource로 관리해주세요.
-            Image(
-                painter = painterResource(team.retum.design_system.R.drawable.ic_success),
-                contentDescription = "review_make_success",
-            )
+            Image(
+                painter = painterResource(team.retum.design_system.R.drawable.ic_success),
+                contentDescription = stringResource(R.string.content_description_review_make_success),
+            )
core/network/src/main/java/team/retum/network/api/ReviewApi.kt (1)

36-38: ID 타입 일관성

reviewId: String은 다른 API에서 Long을 주로 쓰는 패턴과 상이할 수 있습니다. 서버 스키마에 맞다면 유지, 아니라면 Long으로 통일을 검토해주세요.

core/data/src/main/java/team/retum/data/repository/review/ReviewRepositoryImpl.kt (1)

20-37: 파라미터 패스스루 LGTM

데이터소스에 네이밍 아규먼트로 정확히 위임되고 있습니다. 유지보수성을 위해 도메인/네트워크 계층과 파라미터 순서를 맞추면 가독성이 조금 더 좋아집니다.
feature/post-review/src/main/java/team/retum/post/review/model/PostReviewData.kt (2)
30-38: URL 인코딩에서 문자셋 상수 사용

문자열 리터럴 대신 표준 상수를 사용하면 안전합니다.
+import java.nio.charset.StandardCharsets
@@
-    return URLEncoder.encode(json, "UTF-8")
+    return URLEncoder.encode(json, StandardCharsets.UTF_8.toString())
@@
-    val decoded = URLDecoder.decode(this, "UTF-8")
+    val decoded = URLDecoder.decode(this, StandardCharsets.UTF_8.toString())
또한 Json 설정을 하나로 통일(encodeDefaults/ignoreUnknownKeys 등)하면 버전 변경 시 안정적입니다. 예) private val json = Json { encodeDefaults = true }.

19-27: 네이밍 일관성(qnaElements vs qnas)

네트워크 모델은 qnas, UI 모델은 qnaElements로 혼재되어 혼동을 유발합니다. 한쪽으로 맞추는 것을 권장합니다.
feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt (3)
576-582: onValueChange 시그니처 호환성 확인

setKeyword가 (String?) -> Unit, onValueChange는 (String) -> Unit입니다. 메서드 레퍼런스가 타입 추론에 실패할 수 있어 람다로 감싸는 편이 안전합니다.
-            onValueChange = setKeyword,
+            onValueChange = { setKeyword(it) },
362-373: 하드코딩된 텍스트 i18n 처리

"답변"과 같은 문자열은 string 리소스로 분리하세요. 접근성 번역/일관성에 유리합니다.
-                JobisText(
-                    text = "답변",
+                JobisText(
+                    text = stringResource(id = R.string.answer),
Also applies to: 466-477, 691-702

169-191: 리스트 렌더링: items(size) 대신 items(data) + key 권장

성능/재구성 안정성을 위해 key를 제공하세요.
-            if (state.myReview.isNotEmpty()) {
-                items(state.myReview.size) {
-                    val myReview = state.myReview[it]
+            if (state.myReview.isNotEmpty()) {
+                items(
+                    items = state.myReview,
+                    key = { it.id } // 가용한 고유 키가 있다면
+                ) { myReview ->
feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt (2)
151-158: answers 인덱싱 안전화 및 VM 위임

answers[page] 직접 인덱싱은 크래시 위험이 있습니다. 안전 접근을 쓰거나(권장), TODO대로 VM로 상태를 옮기세요.

안전 접근 예시:
-                    value = { answers[page] }, // TODO :: viewModel로 로직 이동
+                    value = { answers.getOrElse(page) { "" } }, // TODO :: viewModel로 로직 이동
                     onValueChange = { // TODO :: ''
-                        answers[page] = it
+                        if (page < answers.size) {
+                            answers[page] = it
+                        }
                     },
131-134: 문자열 하드코딩(i18n) 제거 제안

"Q. ", "다음"은 stringResource로 분리하는 것이 좋습니다. 리소스가 없으면 추가해주세요.

예시:
-                        text = "Q. " + questions[page].question,
+                        text = stringResource(R.string.prefix_question) + " " + questions[page].question,
...
-                    text = "다음",
+                    text = stringResource(R.string.next),
Also applies to: 161-163
core/domain/src/main/java/team/retum/usecase/entity/FetchReviewDetailEntity.kt (1)

16-24: 중복/모호한 필드 네이밍(question/answer)

최상위 question/answer와 qnaResponse[].question/answer가 공존해 의미가 모호합니다. 도메인 용어를 반영해 이름을 구체화(예: overallQuestion, overallAnswer)하거나 KDoc으로 의미를 명확히 해주세요.

Also applies to: 41-45
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (3)
69-92: postReview 실패 처리 범위 확장 제안

BadRequestException 외 예외는 무시됩니다. 네트워크/서버 오류 등 공통 에러를 구분 처리하거나 로깅이 필요합니다.

간단 예시:
.onFailure {
    when (it) {
        is BadRequestException -> postSideEffect(PostReviewSideEffect.BadRequest)
        else -> postSideEffect(PostReviewSideEffect.BadRequest) // 혹은 별도 SideEffect 추가
    }
}
94-103: 주석 처리된 dead code 제거

사용하지 않는 addReview 블록은 제거해 주세요. 히스토리는 Git이 보존합니다.

220-238: 선택형 필드의 초기값 표현 개선 제안

interviewLocation이 기본값 DAEJEON으로 시작하면 “미선택” 표현이 어렵습니다. InterviewLocation? = null로 바꾸고 UI에서 명시적 선택을 유도하는 방안을 고려해 주세요.
feature/review/src/main/java/team/retum/review/navigation/ReviewDetailsNavigation.kt (1)
22-27: 유효하지 않은 reviewId 가드 추가 권장

nullable 시 0으로 진행하면 잘못된 상세 진입이 발생할 수 있습니다. 가드 후 뒤로가기 처리로 안전화하세요.
-        val reviewId = it.arguments?.getLong(ResourceKeys.REVIEW_ID) ?: 0
+        val reviewId = it.arguments?.getLong(ResourceKeys.REVIEW_ID) ?: run {
+            onBackPressed()
+            return@composable
+        }
app/src/main/java/team/retum/jobisandroidv2/root/RootNavigation.kt (2)

27-31: 콜백 시그니처 명확성

onPostReviewClick: (String, Long) 파라미터 의미가 불분명합니다(예: companyName, companyId?). 추후 유지보수를 위해 타입 alias 또는 data class로 명시하는 것을 권장합니다.

Also applies to: 54-58

35-41: 라우트 패턴 변경 시 3개 위치 동기화 필요

현재 라우트 패턴 "root{applicationId}"는 작동하지만 구분자 없이 매개변수를 붙이므로 가독성이 낮습니다. 제안된 대로 "root/{applicationId}"로 개선할 경우 다음 3개 위치를 동시에 수정해야 합니다:

RootNavigation.kt 36줄: route = "$NAVIGATION_ROOT/{$APPLICATION_ID}"

RootNavigation.kt 66줄: navigate("$NAVIGATION_ROOT/$applicationId")

JobisNavigator.kt 235줄: navController.currentDestination?.route == "$NAVIGATION_ROOT/{$APPLICATION_ID}"

모두 함께 수정하지 않으면 라우트 매칭이 실패합니다.
core/data/src/main/java/team/retum/data/repository/review/ReviewRepository.kt (2)
15-23: 옵셔널 파라미터 기본값 제공으로 호출부 간소화

다수가 nullable인 만큼 기본값을 = null로 제공하면 호출부 가독성이 좋아집니다(특히 필터 일부만 전달 시).
-    suspend fun fetchReviews(
-        page: Int?,
-        location: InterviewLocation?,
-        interviewType: InterviewType?,
-        keyword: String?,
-        year: Int?,
-        companyId: Long?,
-        code: Long?,
-    ): FetchReviewsResponse
+    suspend fun fetchReviews(
+        page: Int? = null,
+        location: InterviewLocation? = null,
+        interviewType: InterviewType? = null,
+        keyword: String? = null,
+        year: Int? = null,
+        companyId: Long? = null,
+        code: Long? = null,
+    ): FetchReviewsResponse
25-32: 리뷰 ID 타입 일관화 제안

네비게이션/화면에서 reviewId: Long로 사용 중입니다. fetchReviewDetail(reviewId: String)도 Long으로 통일하면 변환 비용/오류를 줄일 수 있습니다.
-    suspend fun fetchReviewDetail(reviewId: String): FetchReviewDetailResponse
+    suspend fun fetchReviewDetail(reviewId: Long): FetchReviewDetailResponse
또한 code는 의미가 모호하니 jobCode 등 도메인 명시로 교체를 고려해 주세요.
feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt (2)
92-97: 하드코딩 텍스트를 stringResource로 이전

국제화/번역/테스트 편의성을 위해 문자열 리소스로 이전해 주세요.

예시:
-            text = "받았던 면접 질문을 추가해주세요!",
+            text = stringResource(R.string.post_expect_title),
@@
-                text = "질문",
+                text = stringResource(R.string.question),
@@
-            hint = "받았던 질문을 작성해 주세요.",
+            hint = stringResource(R.string.post_expect_question_hint),
@@
-            hint = "예상 질문 답변을 성심성의껏 작성해 주세요!",
+            hint = stringResource(R.string.post_expect_answer_hint),
@@
-            text = "건너뛸래요.",
+            text = stringResource(R.string.skip),
@@
-            text = "완료",
+            text = stringResource(R.string.complete),
(리소스 키는 예시입니다.)

Also applies to: 101-110, 111-116, 120-129, 130-136, 138-147, 149-153

44-57: sideEffect 수집 패턴 정리(선택)

두 개의 LaunchedEffect(Unit)를 하나로 합치거나 collectLatest/수명주기 인지 수집으로 일원화하면 가독성과 안전성이 좋아집니다.
LaunchedEffect(Unit) {
    launch { postExpectReviewViewModel.sideEffect.collect { /* ... */ } }
    launch { postReviewViewModel.sideEffect.collect { /* ... */ } }
}
Also applies to: 59-65
app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (1)
213-218: 불용 파라미터를 제거하세요

검증 결과, JobisNavigator.navigateToReview(companyId, companyName)의 매개변수는 정말로 사용되지 않습니다. 기저 함수 NavController.navigateToReview()는 인자를 받지 않으며, 래퍼 함수는 이 매개변수들을 전달하지 않습니다.

호출 사이트에서 2개 인자를 전달하는 패턴도 발견되지 않았으므로, **옵션 A(파라미터 삭제)**를 진행하면 됩니다.
-    fun navigateToReview(
-        companyId: Long,
-        companyName: String,
-    ) {
-        navController.navigateToReview()
-    }
+    fun navigateToReview() {
+        navController.navigateToReview()
+    }

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ac5b061 and 6f55f18.

⛔ Files ignored due to path filters (1)

feature/bookmark/src/main/res/drawable/ic_empty_bookmark.png is excluded by !**/*.png

📒 Files selected for processing (88)

app/build.gradle.kts (1 hunks)
app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (3 hunks)
app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (3 hunks)
app/src/main/java/team/retum/jobisandroidv2/root/RootNavigation.kt (2 hunks)
app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt (5 hunks)
app/src/main/java/team/retum/jobisandroidv2/ui/BottomMenu.kt (2 hunks)
app/src/main/java/team/retum/jobisandroidv2/ui/BottomNavigationBar.kt (1 hunks)
app/src/main/res/drawable/ic_review.xml (1 hunks)
app/src/main/res/values/strings.xml (1 hunks)
build.gradle.kts (1 hunks)
core/common/src/main/java/team/retum/common/enums/ApplyStatus.kt (1 hunks)
core/common/src/main/java/team/retum/common/enums/InterviewLocation.kt (1 hunks)
core/common/src/main/java/team/retum/common/enums/InterviewType.kt (1 hunks)
core/common/src/main/java/team/retum/common/enums/ReviewProcess.kt (1 hunks)
core/common/src/main/java/team/retum/common/utils/ResourceKeys.kt (1 hunks)
core/data/src/main/java/team/retum/data/repository/review/ReviewRepository.kt (1 hunks)
core/data/src/main/java/team/retum/data/repository/review/ReviewRepositoryImpl.kt (2 hunks)
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisColor.kt (1 hunks)
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisIcon.kt (1 hunks)
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/review/ReviewContent.kt (1 hunks)
core/design-system/src/main/res/drawable/ic_asterisk.xml (1 hunks)
core/design-system/src/main/res/drawable/ic_success.xml (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/FetchReviewDetailEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/FetchReviewsEntity.kt (2 hunks)
core/domain/src/main/java/team/retum/usecase/entity/MyReviews.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/PostReviewEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/QuestionsEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/ReviewsCountEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchMyReviewUseCase.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchQuestionsUseCase.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchReviewsCountUseCase.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchReviewsUseCase.kt (1 hunks)
core/network/src/main/java/team/retum/network/api/InterestsApi.kt (1 hunks)
core/network/src/main/java/team/retum/network/api/ReviewApi.kt (2 hunks)
core/network/src/main/java/team/retum/network/datasource/review/ReviewDataSource.kt (1 hunks)
core/network/src/main/java/team/retum/network/datasource/review/ReviewDataSourceImpl.kt (2 hunks)
core/network/src/main/java/team/retum/network/di/RequestUrls.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/request/PostReviewRequest.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchMyReviewResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchQuestionsResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchReviewDetailResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchReviewsCountResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchReviewsResponse.kt (1 hunks)
feature/bookmark/src/main/java/team/retum/bookmark/ui/BookmarkScreen.kt (1 hunks)
feature/company/src/main/java/team/retum/company/navigation/CompanyDetailsNavigation.kt (1 hunks)
feature/company/src/main/java/team/retum/company/ui/CompanyDetailsScreen.kt (3 hunks)
feature/company/src/main/java/team/retum/company/viewmodel/CompanyDetailsViewModel.kt (1 hunks)
feature/home/src/main/java/team/retum/home/ui/ApplyCompanyItem.kt (1 hunks)
feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (1 hunks)
feature/mypage/src/main/java/team/retum/jobis/navigation/MyPageNavigation.kt (1 hunks)
feature/mypage/src/main/java/team/retum/jobis/ui/MyPageScreen.kt (3 hunks)
feature/post-review/.gitignore (1 hunks)
feature/post-review/build.gradle.kts (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/model/PostReviewData.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewCompleteNavigation.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostExpectReviewViewModel.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (1 hunks)
feature/post-review/src/main/res/values/strings.xml (1 hunks)
feature/review/build.gradle.kts (2 hunks)
feature/review/src/main/java/team/retum/review/navigation/PostReviewNavigation.kt (0 hunks)
feature/review/src/main/java/team/retum/review/navigation/ReviewDetailsNavigation.kt (1 hunks)
feature/review/src/main/java/team/retum/review/navigation/ReviewFilterNavigation.kt (1 hunks)
feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt (1 hunks)
feature/review/src/main/java/team/retum/review/navigation/ReviewsNavigation.kt (0 hunks)
feature/review/src/main/java/team/retum/review/navigation/SearchReviewsNavigation.kt (1 hunks)
feature/review/src/main/java/team/retum/review/ui/PostReviewScreen.kt (0 hunks)
feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (2 hunks)
feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt (1 hunks)
feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt (1 hunks)
feature/review/src/main/java/team/retum/review/ui/ReviewsScreen.kt (0 hunks)
feature/review/src/main/java/team/retum/review/ui/SearchReviewScreen.kt (1 hunks)
feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/PostReviewViewModel.kt (0 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt (2 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt (1 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt (1 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/ReviewsViewModel.kt (0 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/SearchReviewsViewModel.kt (1 hunks)
feature/review/src/main/res/values/strings.xml (1 hunks)
settings.gradle.kts (1 hunks)

💤 Files with no reviewable changes (6)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewsViewModel.kt
feature/review/src/main/java/team/retum/review/navigation/ReviewsNavigation.kt
feature/review/src/main/java/team/retum/review/viewmodel/PostReviewViewModel.kt
feature/review/src/main/java/team/retum/review/ui/PostReviewScreen.kt
feature/review/src/main/java/team/retum/review/ui/ReviewsScreen.kt
feature/review/src/main/java/team/retum/review/navigation/PostReviewNavigation.kt

🧰 Additional context used

🧬 Code graph analysis (30)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/review/ReviewContent.kt (1)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/card/JobisCard.kt (1)

JobisCard (43-81)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewCompleteNavigation.kt (1)

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt (1)

PostReviewComplete (29-47)

feature/review/src/main/java/team/retum/review/navigation/ReviewFilterNavigation.kt (1)

feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt (1)

ReviewFilter (47-66)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt (1)

feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt (1)

PostExpectReview (34-76)

feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt (4)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt (4)

setYear (27-29)

setCode (23-25)

setLocation (35-37)

clearReview (39-44)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisLargeTopAppBar (153-181)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisIconButton.kt (1)

JobisIconButton (74-93)

feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1)

ReviewItems (25-81)

feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt (6)

app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (1)

navigateToPostExpectReview (149-151)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt (1)

navigateToPostExpectReview (31-33)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/textfield/JobisTextField.kt (1)

JobisTextField (355-436)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisButton.kt (1)

JobisButton (322-340)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (1)

setState (60-64)

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt (2)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (1)

navigateToPostReview (35-37)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

feature/review/src/main/java/team/retum/review/navigation/SearchReviewsNavigation.kt (1)

feature/review/src/main/java/team/retum/review/ui/SearchReviewScreen.kt (1)

SearchReview (19-33)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (1)

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt (1)

PostReview (66-135)

feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt (1)

feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt (1)

Review (24-50)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (2)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (2)

setState (60-64)

postSideEffect (106-110)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostExpectReviewViewModel.kt (1)

setButtonEnabled (22-27)

feature/review/src/main/java/team/retum/review/viewmodel/SearchReviewsViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (1)

setState (60-64)

feature/post-review/build.gradle.kts (1)

buildSrc/src/main/kotlin/GradlePluginExtensions.kt (2)

android (8-10)

implementation (15-17)

app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt (1)

feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt (1)

review (10-24)

feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt (4)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/textfield/JobisTextField.kt (1)

JobisTextField (355-436)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisButton.kt (1)

JobisButton (322-340)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostExpectReviewViewModel.kt (3)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (2)

setState (60-64)

postSideEffect (106-110)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (2)

setButtonEnabled (130-140)

onNextClick (187-197)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (1)

onNextClick (47-55)

app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (7)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (1)

postReview (15-33)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1)

postNextReview (17-31)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt (1)

postExpectReview (15-29)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewCompleteNavigation.kt (1)

postReviewComplete (10-22)

feature/review/src/main/java/team/retum/review/navigation/ReviewDetailsNavigation.kt (1)

reviewDetails (13-29)

feature/review/src/main/java/team/retum/review/navigation/ReviewFilterNavigation.kt (1)

reviewFilter (10-18)

feature/review/src/main/java/team/retum/review/navigation/SearchReviewsNavigation.kt (1)

searchReview (10-22)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (1)

setState (60-64)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (1)

setState (60-64)

feature/review/src/main/java/team/retum/review/navigation/ReviewDetailsNavigation.kt (1)

feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (1)

ReviewDetails (47-78)

feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt (4)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisButton.kt (1)

JobisButton (322-340)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/checkbox/CheckBox.kt (1)

JobisCheckBox (21-66)

app/build.gradle.kts (1)

buildSrc/src/main/kotlin/GradlePluginExtensions.kt (1)

implementation (15-17)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1)

feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt (1)

PostNextReview (45-82)

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt (9)

app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (1)

navigateToPostNextReview (141-143)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1)

navigateToPostNextReview (33-35)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisLargeTopAppBar (153-181)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisButton.kt (1)

JobisButton (322-340)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (6)

setButtonClear (179-185)

setInterviewType (151-158)

setInterviewLocation (160-167)

setChecked (142-149)

setKeyword (59-67)

setSelectedTech (104-105)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisIconButton.kt (1)

JobisIconButton (74-93)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/textfield/JobisTextField.kt (1)

JobisTextField (355-436)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/checkbox/CheckBox.kt (1)

JobisCheckBox (21-66)

feature/review/build.gradle.kts (1)

buildSrc/src/main/kotlin/GradlePluginExtensions.kt (1)

implementation (15-17)

feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (2)

setState (60-64)

postSideEffect (106-110)

feature/review/src/main/java/team/retum/review/ui/SearchReviewScreen.kt (4)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/textfield/JobisTextField.kt (1)

JobisTextField (355-436)

feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1)

ReviewItems (25-81)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/empty/EmptyContent.kt (1)

EmptyContent (21-56)

feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (4)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/tab/TabBar.kt (1)

TabBar (21-68)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/card/JobisCard.kt (1)

JobisCard (43-81)

🪛 detekt (1.23.8)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt

[warning] 38-38: A call to the default constructor of an exception was detected. Instead one of the constructor overloads should be called. This allows to provide more meaningful exceptions.

(detekt.exceptions.ThrowingExceptionsWithoutMessageOrCause)

app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt

core/common/src/main/java/team/retum/common/enums/ApplyStatus.kt

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisColor.kt

core/network/src/main/java/team/retum/network/datasource/review/ReviewDataSource.kt

feature/post-review/src/main/res/values/strings.xml

feature/review/build.gradle.kts

feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt

feature/review/src/main/res/values/strings.xml

@uson1004

Docstrings generation was requested by @uson1004. * #473 (comment) The following files were modified: * `app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt` * `app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt` * `app/src/main/java/team/retum/jobisandroidv2/root/RootNavigation.kt` * `app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt` * `core/data/src/main/java/team/retum/data/repository/review/ReviewRepository.kt` * `core/data/src/main/java/team/retum/data/repository/review/ReviewRepositoryImpl.kt` * `core/design-system/src/main/java/team/retum/jobisdesignsystemv2/review/ReviewContent.kt` * `core/domain/src/main/java/team/retum/usecase/entity/FetchReviewDetailEntity.kt` * `core/domain/src/main/java/team/retum/usecase/entity/FetchReviewsEntity.kt` * `core/domain/src/main/java/team/retum/usecase/entity/MyReviews.kt` * `core/domain/src/main/java/team/retum/usecase/entity/PostReviewEntity.kt` * `core/domain/src/main/java/team/retum/usecase/entity/QuestionsEntity.kt` * `core/domain/src/main/java/team/retum/usecase/entity/ReviewsCountEntity.kt` * `core/domain/src/main/java/team/retum/usecase/usecase/review/FetchMyReviewUseCase.kt` * `core/domain/src/main/java/team/retum/usecase/usecase/review/FetchQuestionsUseCase.kt` * `core/domain/src/main/java/team/retum/usecase/usecase/review/FetchReviewsCountUseCase.kt` * `core/domain/src/main/java/team/retum/usecase/usecase/review/FetchReviewsUseCase.kt` * `core/network/src/main/java/team/retum/network/api/ReviewApi.kt` * `core/network/src/main/java/team/retum/network/datasource/review/ReviewDataSource.kt` * `core/network/src/main/java/team/retum/network/datasource/review/ReviewDataSourceImpl.kt` * `feature/bookmark/src/main/java/team/retum/bookmark/ui/BookmarkScreen.kt` * `feature/company/src/main/java/team/retum/company/navigation/CompanyDetailsNavigation.kt` * `feature/company/src/main/java/team/retum/company/ui/CompanyDetailsScreen.kt` * `feature/company/src/main/java/team/retum/company/viewmodel/CompanyDetailsViewModel.kt` * `feature/home/src/main/java/team/retum/home/ui/ApplyCompanyItem.kt` * `feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt` * `feature/mypage/src/main/java/team/retum/jobis/navigation/MyPageNavigation.kt` * `feature/mypage/src/main/java/team/retum/jobis/ui/MyPageScreen.kt` * `feature/post-review/src/main/java/team/retum/post/review/model/PostReviewData.kt` * `feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt` * `feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt` * `feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewCompleteNavigation.kt` * `feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt` * `feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt` * `feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt` * `feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt` * `feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt` * `feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostExpectReviewViewModel.kt` * `feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt` * `feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt` * `feature/review/src/main/java/team/retum/review/navigation/ReviewDetailsNavigation.kt` * `feature/review/src/main/java/team/retum/review/navigation/ReviewFilterNavigation.kt` * `feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt` * `feature/review/src/main/java/team/retum/review/navigation/SearchReviewsNavigation.kt` * `feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt` * `feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt` * `feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt` * `feature/review/src/main/java/team/retum/review/ui/SearchReviewScreen.kt` * `feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt` * `feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt` * `feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt` * `feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt` * `feature/review/src/main/java/team/retum/review/viewmodel/SearchReviewsViewModel.kt`

coderabbitai

Actionable comments posted: 17

♻️ Duplicate comments (18)

core/common/src/main/java/team/retum/common/enums/ApplyStatus.kt (1)

12-13: DOC_FAILED 상태의 UI 처리 누락

과거 리뷰에서 이미 지적되었듯이, 새로 추가된 DOC_FAILED 상태가 ApplyCompanyItem.kt의 색상 매핑에서 명시적으로 처리되지 않습니다. PROCESSING 상태는 UI에서 올바르게 처리되고 있으나, DOC_FAILED는 else 분기로 빠져 의도하지 않은 onPrimary 색상이 할당됩니다.
feature/home/src/main/java/team/retum/home/ui/ApplyCompanyItem.kt (1)
48-54: DOC_FAILED 상태가 여전히 처리되지 않습니다

과거 리뷰에서 지적된 것처럼, ApplyStatus.DOC_FAILED가 when 표현식에서 명시적으로 처리되지 않아 else 분기를 통해 onPrimary 색상이 할당됩니다. DOC_FAILED("서류 탈락")는 의미상 FAILED("탈락")와 유사하므로 error 색상으로 매핑되어야 합니다.

다음과 같이 수정하세요:
-        ApplyStatus.FAILED, ApplyStatus.REJECTED -> JobisTheme.colors.error
+        ApplyStatus.FAILED, ApplyStatus.DOC_FAILED, ApplyStatus.REJECTED -> JobisTheme.colors.error
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostExpectReviewViewModel.kt (2)
12-27: 중첩 setState 호출을 제거해야 합니다.
setAnswer와 setQuestion이 setState 블록 안에서 다시 setButtonEnabled를 호출하면서 또 다른 setState를 트리거하고 있습니다. BaseViewModel.setState는 MutableStateFlow.update를 사용하므로 이 패턴은 마지막 호출이 값을 덮어써 상태 플리커나 레이스를 일으킬 수 있습니다. 한 번의 setState 안에서 필드와 buttonEnabled를 동시에 계산·반영하도록 정리해 주세요.
-    internal fun setAnswer(answer: String) = setState {
-        setButtonEnabled(answer = answer)
-        state.value.copy(answer = answer)
-    }
-
-    internal fun setQuestion(question: String) = setState {
-        setButtonEnabled(question = question)
-        state.value.copy(question = question)
-    }
-
-    private fun setButtonEnabled(
-        answer: String = state.value.answer,
-        question: String = state.value.question,
-    ) = setState {
-        state.value.copy(buttonEnabled = answer.isNotBlank() && question.isNotBlank())
-    }
+    internal fun setAnswer(answer: String) {
+        val question = state.value.question
+        setState {
+            state.value.copy(
+                answer = answer,
+                buttonEnabled = answer.isNotBlank() && question.isNotBlank(),
+            )
+        }
+    }
+
+    internal fun setQuestion(question: String) {
+        val answer = state.value.answer
+        setState {
+            state.value.copy(
+                question = question,
+                buttonEnabled = answer.isNotBlank() && question.isNotBlank(),
+            )
+        }
+    }
29-36: setEmpty가 상태를 실제로 비우지 못합니다.
setEmpty가 copy만 호출하고 setState로 적용하지 않아 onNextClick이 이전 질문/답변 값을 그대로 전파합니다. 버튼 비활성화도 되지 않습니다. 먼저 상태를 비우고 버튼도 꺼지게 만든 뒤 사이드 이펙트를 보내도록 수정해 주세요.
-    internal fun setEmpty() {
-        with(state.value) {
-            copy(
-                question = "",
-                answer = "",
-            )
-        }
-        onNextClick()
-    }
+    internal fun setEmpty() {
+        setState {
+            state.value.copy(
+                question = "",
+                answer = "",
+                buttonEnabled = false,
+            )
+        }
+        onNextClick()
+    }
feature/post-review/src/main/res/values/strings.xml (1)

30-30: 이전 리뷰 코멘트가 이미 반영되었습니다.

"잘못된 요청이에요"로 올바르게 수정되어 있습니다.
feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (3)
22-23: navArgument 타입 명시 필요 및 companyId는 LongType 사용

이전 리뷰에서 지적된 문제가 아직 해결되지 않았습니다. navArgument 블록에 type =을 명시하고, companyId는 LongType으로 변경하여 수동 파싱을 제거해야 합니다.

다음 diff를 적용하세요:
-            navArgument(ResourceKeys.COMPANY_NAME) { NavType.StringType },
-            navArgument(ResourceKeys.COMPANY_ID) { NavType.StringType },
+            navArgument(ResourceKeys.COMPANY_NAME) { type = NavType.StringType },
+            navArgument(ResourceKeys.COMPANY_ID) { type = NavType.LongType },
29-30: companyName URI 인코딩 및 companyId 타입 안전성 개선

이전 리뷰에서 지적된 문제가 아직 해결되지 않았습니다:

공백이나 특수문자가 포함된 회사명은 네비게이션 경로를 깨뜨립니다.

companyId를 String으로 받아 파싱하면서 0L 디폴트를 사용하면 실패를 숨깁니다.

다음 diff를 적용하세요:
+import android.net.Uri
-import kotlin.text.toLongOrNull
@@
-            companyName = it.arguments?.getString(ResourceKeys.COMPANY_NAME) ?: "",
-            companyId = it.arguments?.getString(ResourceKeys.COMPANY_ID)?.toLongOrNull() ?: 0L,
+            companyName = requireNotNull(it.arguments).getString(ResourceKeys.COMPANY_NAME).orEmpty(),
+            companyId = requireNotNull(it.arguments).getLong(ResourceKeys.COMPANY_ID),
35-39: navigateToPostReview에서 companyName URI 인코딩 필요

이전 리뷰에서 지적된 문제가 아직 해결되지 않았습니다. companyName을 URI 인코딩하지 않으면 공백이나 특수문자로 인해 네비게이션이 실패할 수 있습니다.

다음 diff를 적용하세요:
+import android.net.Uri
@@
 fun NavController.navigateToPostReview(companyName: String, companyId: Long) {
-    navigate("$NAVIGATION_POST_REVIEW/$companyName/$companyId") {
+    navigate("$NAVIGATION_POST_REVIEW/${Uri.encode(companyName)}/$companyId") {
         popUpTo(NAVIGATION_POST_NEXT_REVIEW) { inclusive = false }
         popUpTo(NAVIGATION_POST_EXPECT_REVIEW) { inclusive = false }
     }
 }
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (2)
19-29: 질문 로드 실패 처리 누락

이전 리뷰에서 지적된 문제가 아직 해결되지 않았습니다. 네트워크 실패 시 상태가 갱신되지 않아 UX가 멈춘 것처럼 보일 수 있습니다.

다음과 같이 실패 분기를 추가하는 것을 권장합니다:
         viewModelScope.launch(Dispatchers.IO) {
             fetchQuestionsUseCase().onSuccess {
                 setState {
                     state.value.copy(
                         questions = it.questions,
+                        answers = List(it.questions.size) { "" },
+                        qnaElements = emptyList(),
+                        buttonEnabled = false,
                     )
                 }
+            }.onFailure {
+                setState {
+                    state.value.copy(
+                        questions = emptyList(),
+                        answers = emptyList(),
+                        qnaElements = emptyList(),
+                        buttonEnabled = false,
+                    )
+                }
+                // TODO: 실패 사이드이펙트/토스트 노출
             }
         }
47-61: answers 길이 검증 및 버튼 활성화 로직 누락

이전 리뷰에서 지적된 것처럼, 질문 수와 answers 길이가 불일치하면 zip으로 데이터가 잘려나가 데이터 유실이 발생합니다. 또한 모든 답변이 채워졌을 때만 버튼을 활성화해야 합니다.

다음과 같이 길이 검증과 버튼 활성화를 함께 반영해주세요:
     internal fun setQuestion() {
         setState {
             with(state.value) {
-                val updatedQuestions = questions.map { it.id }
-                copy(
-                    qnaElements = updatedQuestions.zip(answers).map { (q, a) ->
-                        PostReviewContent(
-                            question = q,
-                            answer = a,
-                        )
-                    },
-                )
+                val ids = questions.map { it.id }
+                val pairs = ids.zip(answers)
+                val allAnswered = answers.size == ids.size && answers.all { it.isNotBlank() }
+                copy(
+                    qnaElements = pairs.map { (q, a) -> PostReviewContent(question = q, answer = a) },
+                    answers = answers,
+                    buttonEnabled = allAnswered,
+                )
             }
         }
     }
feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt (2)
55-55: answers 리스트 고정 크기로 인한 크래시 가능성

이전 리뷰에서 지적된 문제가 아직 해결되지 않았습니다. 질문 수(state.questions.size)와 answers 크기가 불일치하면 answers[page]에서 IndexOutOfBoundsException이 발생합니다.

다음 diff를 적용하세요:
-    val answers = remember { mutableStateListOf("", "", "") }
+    val answers = remember(state.questions) {
+        mutableStateListOf(*Array(state.questions.size) { "" })
+    }
또는 toMutableStateList() 사용:
+import androidx.compose.runtime.toMutableStateList
@@
-    val answers = remember { mutableStateListOf("", "", "") }
+    val answers = remember(state.questions) {
+        List(state.questions.size) { "" }.toMutableStateList()
+    }
166-166: 마지막 페이지 하드코딩 제거 필요

이전 리뷰에서 지적된 문제가 아직 해결되지 않았습니다. 페이지 수가 3이 아닐 경우 흐름이 깨집니다. 마지막 인덱스를 동적으로 계산해야 합니다.

다음 diff를 적용하세요:
                     onClick = {
                         setQuestion()
                         coroutineScope.launch {
-                            if (pagerState.currentPage != 2) pagerState.animateScrollToPage(pagerState.currentPage + 1) else onPostExpectReviewClick()
+                            val lastPageIndex = pagerState.pageCount - 1
+                            if (pagerState.currentPage < lastPageIndex) {
+                                pagerState.animateScrollToPage(pagerState.currentPage + 1)
+                            } else {
+                                onPostExpectReviewClick()
+                            }
                         }
                     },
feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt (1)
88-154: 하단 배치가 깨지지 않도록 Column 에 전체 높이를 부여하세요

Spacer(weight = 1f) 로 버튼을 하단에 붙이려면 부모 Column 이 가용 높이를 차지해야 합니다. 현재 Modifier.fillMaxSize() 가 없어 버튼이 중간에 머무릅니다. 기존 지적과 동일하니 아래와 같이 수정해 주세요.
-import androidx.compose.foundation.layout.Column
+import androidx.compose.foundation.layout.Column
+import androidx.compose.foundation.layout.fillMaxSize
@@
-    Column {
+    Column(
+        modifier = Modifier.fillMaxSize(),
+    ) {
feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1)
33-39: 라우트 인코딩 누락으로 인한 네비게이션 실패

JSON 안에 /, ?, % 등이 포함되면 경로가 분리되어 목적지 매칭이 실패하고, 예외를 메시지 없이 던져 디버깅도 어렵습니다. Uri.encode/Uri.decode와 명시적인 예외 메시지를 추가해 주세요.
+import android.net.Uri
@@
 fun NavController.navigateToPostNextReview(reviewData: PostReviewData) {
-    navigate("$NAVIGATION_POST_NEXT_REVIEW/${reviewData.toJsonString()}")
+    val payload = Uri.encode(reviewData.toJsonString())
+    navigate("$NAVIGATION_POST_NEXT_REVIEW/$payload")
 }
 
 internal fun NavBackStackEntry.getReviewData(): PostReviewData {
-    val reviewData = arguments?.getString(ResourceKeys.REVIEW_DATA) ?: throw NullPointerException()
-    return reviewData.toReviewData()
+    val encoded = requireNotNull(arguments?.getString(ResourceKeys.REVIEW_DATA)) {
+        "Missing '${ResourceKeys.REVIEW_DATA}' argument for $NAVIGATION_POST_NEXT_REVIEW"
+    }
+    return Uri.decode(encoded).toReviewData()
 }
feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt (1)
704-708: 힌트 문자열을 필드 용도에 맞게 교체해주세요.

면접관 수 입력 필드에 여전히 search 문자열을 사용하고 있어 UX가 어색합니다. 면접관 수에 맞는 새로운 string 리소스를 추가한 뒤 해당 리소스를 사용해주세요.
-            hint = stringResource(id = R.string.search),
+            hint = stringResource(id = R.string.hint_interviewer_count),
※ hint_interviewer_count 리소스 추가도 함께 반영해야 합니다.
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (3)
107-116: [DUPLICATE] Compose SnapshotStateList를 IO 스레드에서 변경하는 스레드 안전성 문제

Line 114에서 techs.addAll(it.codes)를 Dispatchers.IO에서 호출하면 스냅샷 충돌 및 크래시 위험이 있습니다. 이 문제는 이전 리뷰에서 이미 지적되었습니다.

메인 스레드에서 변경하거나 StateFlow로 변경하세요:
 internal fun fetchCodes(keyword: String?) =
     viewModelScope.launch(Dispatchers.IO) {
         fetchCodeUseCase(
             keyword = keyword,
             type = CodeType.JOB,
             parentCode = null,
         ).onSuccess {
-            techs.addAll(it.codes)
+            withContext(Dispatchers.Main) {
+                techs.addAll(it.codes)
+            }
         }
     }
130-140: [DUPLICATE] 버튼 활성화 로직 역전 및 NPE 위험

Line 137에서 !state.value.keyword?.isNotEmpty()!!는 키워드가 입력되면 버튼을 비활성화하는 역전된 로직이며, keyword가 null일 경우 NPE 위험이 있습니다. 이 문제는 이전 리뷰에서 이미 지적되었습니다.

수정 적용:
 else -> {
-   setState { state.value.copy(buttonEnabled = !state.value.keyword?.isNotEmpty()!!) }
+   setState {
+       state.value.copy(
+           buttonEnabled = !state.value.keyword.isNullOrEmpty()
+       )
+   }
 }
187-197: [DUPLICATE] interviewerCount 파싱 시 크래시 위험

Line 191에서 count.toInt()는 빈 문자열이나 숫자가 아닌 입력 시 NumberFormatException을 발생시킵니다. 이 문제는 이전 리뷰에서 이미 지적되었습니다.

안전한 파싱 적용:
 internal fun onNextClick() {
     with(state.value) {
+        val parsedCount = count.toIntOrNull()
+        if (parsedCount == null || parsedCount <= 0) {
+            postSideEffect(PostReviewSideEffect.BadRequest)
+            return
+        }
         postSideEffect(PostReviewSideEffect.MoveToNext(
             companyId = companyId,
-            interviewerCount = count.toInt(),
+            interviewerCount = parsedCount,
             jobCode = selectedTech ?: 0,
             interviewType = interviewType,
             location = interviewLocation,
         ))
     }
 }

🧹 Nitpick comments (10)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt (1)
33-47: 에러 핸들링 추가를 고려하세요.

fetchCodeUseCase 실패 시 처리가 없어 사용자가 빈 목록만 보게 됩니다. 에러 상태를 추가하거나 로깅을 고려하세요.
             fetchCodeUseCase(
                 keyword = null,
                 type = CodeType.JOB,
                 parentCode = null,
             ).onSuccess {
                 setState {
                     state.value.copy(
                         majorList = it.codes,
                     )
                 }
+            }.onFailure {
+                // 로깅 또는 에러 상태 처리 고려
             }
feature/post-review/src/main/java/team/retum/post/review/model/PostReviewData.kt (1)
30-38: deprecated API 사용을 피하세요.

URLEncoder.encode(String, String) 및 URLDecoder.decode(String, String)는 deprecated API입니다. StandardCharsets를 사용하여 현대적이고 안전한 API로 변경하는 것을 권장합니다.

다음과 같이 수정하세요:
+import java.nio.charset.StandardCharsets
+
 internal fun PostReviewData.toJsonString(): String {
     val json = Json.encodeToString(this)
-    return URLEncoder.encode(json, "UTF-8")
+    return URLEncoder.encode(json, StandardCharsets.UTF_8)
 }
 
 internal fun String.toReviewData(): PostReviewData {
-    val decoded = URLDecoder.decode(this, "UTF-8")
+    val decoded = URLDecoder.decode(this, StandardCharsets.UTF_8)
     return Json.decodeFromString<PostReviewData>(decoded)
 }
feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt (1)
77-89: items() 확장 함수 대신 itemsIndexed() 사용을 권장합니다.

items(state.reviews.size)와 인덱스 접근 대신, LazyColumn의 itemsIndexed() 확장 함수를 사용하면 더 간결하고 안전합니다.

다음과 같이 리팩토링할 수 있습니다:
+import androidx.compose.foundation.lazy.itemsIndexed
+
-        LazyColumn {
-            items(state.reviews.size) {
-                val review = state.reviews[it]
+        LazyColumn {
+            itemsIndexed(state.reviews) { _, review ->
                 ReviewItems(
                     companyImageUrl = review.companyLogoUrl,
                     companyName = review.companyName,
                     reviewId = review.reviewId,
                     writer = review.writer,
                     major = review.major,
                     onReviewDetailClick = onReviewDetailClick,
                 )
             }
         }
feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt (1)

38-38: TODO 주석: API 요청 최적화 필요

빈번한 API 요청 문제를 해결해야 합니다. 캐싱, 요청 중복 제거, 또는 상태 관리 개선 등의 방법을 고려해보세요.

이 문제를 해결하는 구현을 생성하거나 새로운 이슈를 열어드릴까요?
feature/review/src/main/java/team/retum/review/viewmodel/SearchReviewsViewModel.kt (1)
33-60: 검색어가 비면 결과도 초기화해 주세요

검색어를 모두 지우면 fetchReviews() 는 호출되지 않는데, 이전 검색 결과가 그대로 남아 있어 화면과 검색어가 불일치합니다. 공백일 때는 리스트와 빈 상태 플래그를 초기화하는 편이 자연스럽습니다.
         viewModelScope.launch {
-            state.map { it.keyword }.distinctUntilChanged().debounce(SEARCH_DEBOUNCE_MILLIS).collect {
-                if (!it.isNullOrBlank()) {
-                    fetchReviews()
-                }
+            state.map { it.keyword }.distinctUntilChanged().debounce(SEARCH_DEBOUNCE_MILLIS).collect { keyword ->
+                if (keyword.isNullOrBlank()) {
+                    setState {
+                        state.value.copy(
+                            reviews = emptyList(),
+                            showRecruitmentsEmptyContent = false,
+                        )
+                    }
+                } else {
+                    fetchReviews()
                 }
             }
         }
feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (5)
113-122: 파라미터 수가 많아 데이터 클래스 사용 권장

8개의 파라미터를 개별적으로 전달하고 있습니다. 유지보수성과 가독성 향상을 위해 데이터 클래스로 그룹화하는 것을 고려해보세요.

다음과 같이 데이터 클래스를 사용할 수 있습니다:
data class StudentInfoData(
    val writer: String,
    val major: String,
    val year: Int,
    val companyName: String,
    val location: InterviewLocation,
    val type: InterviewType,
    val interviewerCount: Int,
    val selectedTabIndex: Int,
)

@Composable
private fun StudentInfo(data: StudentInfoData) {
    // implementation
}
133-140: 복잡한 문자열 포맷팅 로직 단순화 권장

중첩된 stringResource 호출과 조건문이 가독성을 저해합니다. 로직을 별도로 추출하는 것을 고려해보세요.

다음과 같이 리팩토링할 수 있습니다:
val reviewType = if (selectedTabIndex != 0) {
    stringResource(id = R.string.reviewed_question)
} else {
    stringResource(id = R.string.interview_review)
}

JobisText(
    text = stringResource(
        id = R.string.review_writer_title,
        writer,
        reviewType,
    ),
    style = JobisTypography.PageTitle,
)
254-258: 텍스트 오버플로우 처리 명시 권장

maxLines = 3을 설정했지만 JobisText의 기본 overflow 속성에 의존하고 있습니다. 명시적으로 TextOverflow.Ellipsis를 지정하는 것이 좋습니다.

JobisText의 기본값이 Ellipsis이므로 현재는 문제없지만, 명시성을 위해 다음과 같이 작성할 수 있습니다:
Text(
    text = buildAnnotatedString { /* ... */ },
    style = JobisTypography.Description,
    modifier = Modifier
        .padding(top = 12.dp)
        .fillMaxWidth(0.5f),
    maxLines = 3,
    overflow = TextOverflow.Ellipsis, // 명시적 지정
)
274-274: 사용하지 않는 index 변수 제거 권장

forEachIndexed를 사용하지만 index 변수를 사용하지 않습니다. forEach로 변경하는 것을 고려하세요.
-    review.forEachIndexed { index, reviewItem ->
+    review.forEach { reviewItem ->
344-350: 답변 텍스트 오버플로우 처리 검토

maxLines = 3을 설정했지만 긴 답변이 잘릴 경우 사용자가 전체 내용을 볼 방법이 없습니다. 확장 상태에서는 전체 텍스트를 표시하거나, 최소한 말줄임표가 표시되도록 해야 합니다.

확장 상태에서는 전체 답변을 표시하는 것을 고려하세요:
                            JobisText(
                                text = reviewItem.answer,
                                color = JobisTheme.colors.inverseOnSurface,
                                style = JobisTypography.Description,
                                textAlign = TextAlign.Center,
-                                maxLines = 3,
+                                // 확장된 상태에서는 전체 텍스트 표시
                            )

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ac5b061 and 13a317c.

⛔ Files ignored due to path filters (1)

feature/bookmark/src/main/res/drawable/ic_empty_bookmark.png is excluded by !**/*.png

📒 Files selected for processing (86)

app/build.gradle.kts (1 hunks)
app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (3 hunks)
app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (3 hunks)
app/src/main/java/team/retum/jobisandroidv2/root/RootNavigation.kt (2 hunks)
app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt (5 hunks)
app/src/main/java/team/retum/jobisandroidv2/ui/BottomMenu.kt (2 hunks)
app/src/main/java/team/retum/jobisandroidv2/ui/BottomNavigationBar.kt (1 hunks)
app/src/main/res/drawable/ic_review.xml (1 hunks)
app/src/main/res/values/strings.xml (1 hunks)
core/common/src/main/java/team/retum/common/enums/ApplyStatus.kt (1 hunks)
core/common/src/main/java/team/retum/common/enums/InterviewLocation.kt (1 hunks)
core/common/src/main/java/team/retum/common/enums/InterviewType.kt (1 hunks)
core/common/src/main/java/team/retum/common/enums/ReviewProcess.kt (1 hunks)
core/common/src/main/java/team/retum/common/utils/ResourceKeys.kt (1 hunks)
core/data/src/main/java/team/retum/data/repository/review/ReviewRepository.kt (1 hunks)
core/data/src/main/java/team/retum/data/repository/review/ReviewRepositoryImpl.kt (2 hunks)
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisColor.kt (1 hunks)
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisIcon.kt (1 hunks)
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/review/ReviewContent.kt (1 hunks)
core/design-system/src/main/res/drawable/ic_asterisk.xml (1 hunks)
core/design-system/src/main/res/drawable/ic_success.xml (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/FetchReviewDetailEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/FetchReviewsEntity.kt (2 hunks)
core/domain/src/main/java/team/retum/usecase/entity/MyReviews.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/PostReviewEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/QuestionsEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/ReviewsCountEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchMyReviewUseCase.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchQuestionsUseCase.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchReviewsCountUseCase.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/review/FetchReviewsUseCase.kt (1 hunks)
core/network/src/main/java/team/retum/network/api/ReviewApi.kt (2 hunks)
core/network/src/main/java/team/retum/network/datasource/review/ReviewDataSource.kt (1 hunks)
core/network/src/main/java/team/retum/network/datasource/review/ReviewDataSourceImpl.kt (2 hunks)
core/network/src/main/java/team/retum/network/di/RequestUrls.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/request/PostReviewRequest.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchMyReviewResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchQuestionsResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchReviewDetailResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchReviewsCountResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/FetchReviewsResponse.kt (1 hunks)
feature/bookmark/src/main/java/team/retum/bookmark/ui/BookmarkScreen.kt (1 hunks)
feature/company/src/main/java/team/retum/company/navigation/CompanyDetailsNavigation.kt (1 hunks)
feature/company/src/main/java/team/retum/company/ui/CompanyDetailsScreen.kt (3 hunks)
feature/company/src/main/java/team/retum/company/viewmodel/CompanyDetailsViewModel.kt (1 hunks)
feature/home/src/main/java/team/retum/home/ui/ApplyCompanyItem.kt (1 hunks)
feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (1 hunks)
feature/mypage/src/main/java/team/retum/jobis/navigation/MyPageNavigation.kt (1 hunks)
feature/mypage/src/main/java/team/retum/jobis/ui/MyPageScreen.kt (3 hunks)
feature/post-review/.gitignore (1 hunks)
feature/post-review/build.gradle.kts (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/model/PostReviewData.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewCompleteNavigation.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostExpectReviewViewModel.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (1 hunks)
feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (1 hunks)
feature/post-review/src/main/res/values/strings.xml (1 hunks)
feature/review/build.gradle.kts (2 hunks)
feature/review/src/main/java/team/retum/review/navigation/PostReviewNavigation.kt (0 hunks)
feature/review/src/main/java/team/retum/review/navigation/ReviewDetailsNavigation.kt (1 hunks)
feature/review/src/main/java/team/retum/review/navigation/ReviewFilterNavigation.kt (1 hunks)
feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt (1 hunks)
feature/review/src/main/java/team/retum/review/navigation/ReviewsNavigation.kt (0 hunks)
feature/review/src/main/java/team/retum/review/navigation/SearchReviewsNavigation.kt (1 hunks)
feature/review/src/main/java/team/retum/review/ui/PostReviewScreen.kt (0 hunks)
feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (2 hunks)
feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt (1 hunks)
feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt (1 hunks)
feature/review/src/main/java/team/retum/review/ui/ReviewsScreen.kt (0 hunks)
feature/review/src/main/java/team/retum/review/ui/SearchReviewScreen.kt (1 hunks)
feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/PostReviewViewModel.kt (0 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt (2 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt (1 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt (1 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/ReviewsViewModel.kt (0 hunks)
feature/review/src/main/java/team/retum/review/viewmodel/SearchReviewsViewModel.kt (1 hunks)
feature/review/src/main/res/values/strings.xml (1 hunks)
settings.gradle.kts (1 hunks)

💤 Files with no reviewable changes (6)

feature/review/src/main/java/team/retum/review/navigation/PostReviewNavigation.kt
feature/review/src/main/java/team/retum/review/viewmodel/PostReviewViewModel.kt
feature/review/src/main/java/team/retum/review/viewmodel/ReviewsViewModel.kt
feature/review/src/main/java/team/retum/review/ui/PostReviewScreen.kt
feature/review/src/main/java/team/retum/review/navigation/ReviewsNavigation.kt
feature/review/src/main/java/team/retum/review/ui/ReviewsScreen.kt

🧰 Additional context used

🧬 Code graph analysis (31)

app/build.gradle.kts (1)

buildSrc/src/main/kotlin/GradlePluginExtensions.kt (1)

implementation (15-17)

feature/review/build.gradle.kts (1)

buildSrc/src/main/kotlin/GradlePluginExtensions.kt (1)

implementation (15-17)

feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt (4)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/textfield/JobisTextField.kt (1)

JobisTextField (355-436)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisButton.kt (1)

JobisButton (322-340)

feature/review/src/main/java/team/retum/review/navigation/SearchReviewsNavigation.kt (1)

feature/review/src/main/java/team/retum/review/ui/SearchReviewScreen.kt (1)

SearchReview (19-33)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewCompleteNavigation.kt (1)

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt (1)

PostReviewComplete (29-47)

feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt (4)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt (4)

setYear (23-25)

setCode (19-21)

setLocation (31-33)

clearReview (35-44)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisLargeTopAppBar (153-181)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisIconButton.kt (1)

JobisIconButton (74-93)

feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1)

ReviewItems (25-81)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/review/ReviewContent.kt (1)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/card/JobisCard.kt (1)

JobisCard (43-81)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (2)

setState (60-64)

postSideEffect (106-110)

feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt (5)

app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (1)

navigateToPostExpectReview (149-151)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt (1)

navigateToPostExpectReview (31-33)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/textfield/JobisTextField.kt (1)

JobisTextField (355-436)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (2)

setAnswer (31-41)

setQuestion (47-61)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (1)

setState (60-64)

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt (2)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (1)

navigateToPostReview (35-40)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt (1)

feature/post-review/src/main/java/team/retum/post/review/ui/PostExpectReviewScreen.kt (1)

PostExpectReview (34-76)

app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt (1)

feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt (1)

review (10-24)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1)

feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt (1)

PostNextReview (45-81)

feature/post-review/build.gradle.kts (1)

buildSrc/src/main/kotlin/GradlePluginExtensions.kt (2)

android (8-10)

implementation (15-17)

feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt (4)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisButton.kt (1)

JobisButton (322-340)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/checkbox/CheckBox.kt (1)

JobisCheckBox (21-66)

feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (5)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/tab/TabBar.kt (1)

TabBar (21-68)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/card/JobisCard.kt (1)

JobisCard (43-81)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/empty/EmptyContent.kt (1)

EmptyContent (21-56)

feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt (1)

feature/review/src/main/java/team/retum/review/ui/ReviewScreen.kt (1)

Review (24-50)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (2)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (2)

setState (60-64)

postSideEffect (106-110)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostExpectReviewViewModel.kt (1)

setButtonEnabled (22-27)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (1)

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt (1)

PostReview (66-135)

feature/review/src/main/java/team/retum/review/navigation/ReviewFilterNavigation.kt (1)

feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt (1)

ReviewFilter (47-66)

feature/review/src/main/java/team/retum/review/ui/SearchReviewScreen.kt (4)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/textfield/JobisTextField.kt (1)

JobisTextField (355-436)

feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1)

ReviewItems (25-81)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/empty/EmptyContent.kt (1)

EmptyContent (21-56)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostExpectReviewViewModel.kt (3)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (2)

setState (60-64)

postSideEffect (106-110)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (2)

setButtonEnabled (130-140)

onNextClick (187-197)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostNextReviewViewModel.kt (1)

onNextClick (63-71)

feature/review/src/main/java/team/retum/review/viewmodel/SearchReviewsViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (1)

setState (60-64)

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewScreen.kt (9)

app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (1)

navigateToPostNextReview (141-143)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1)

navigateToPostNextReview (33-35)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisLargeTopAppBar (153-181)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/text/JobisText.kt (1)

JobisText (24-45)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisButton.kt (1)

JobisButton (322-340)

feature/post-review/src/main/java/team/retum/post/review/viewmodel/PostReviewViewModel.kt (6)

setButtonClear (179-185)

setInterviewType (151-158)

setInterviewLocation (160-167)

setChecked (142-149)

setKeyword (59-67)

setSelectedTech (104-105)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/button/JobisIconButton.kt (1)

JobisIconButton (74-93)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/textfield/JobisTextField.kt (1)

JobisTextField (355-436)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/checkbox/CheckBox.kt (1)

JobisCheckBox (21-66)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (1)

setState (60-64)

feature/review/src/main/java/team/retum/review/navigation/ReviewDetailsNavigation.kt (1)

feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (1)

ReviewDetails (48-75)

feature/company/src/main/java/team/retum/company/viewmodel/CompanyDetailsViewModel.kt (1)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewViewModel.kt (1)

fetchReviewsUseCase (14-63)

feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt (1)

core/common/src/main/java/team/retum/common/base/BaseViewModel.kt (1)

setState (60-64)

app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (7)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewNavigation.kt (1)

postReview (15-33)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt (1)

postNextReview (17-31)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt (1)

postExpectReview (15-29)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostReviewCompleteNavigation.kt (1)

postReviewComplete (10-22)

feature/review/src/main/java/team/retum/review/navigation/ReviewDetailsNavigation.kt (1)

reviewDetails (13-29)

feature/review/src/main/java/team/retum/review/navigation/ReviewFilterNavigation.kt (1)

reviewFilter (10-18)

feature/review/src/main/java/team/retum/review/navigation/SearchReviewsNavigation.kt (1)

searchReview (10-22)

🪛 detekt (1.23.8)

feature/post-review/src/main/java/team/retum/post/review/navigation/PostNextReviewNavigation.kt

[warning] 38-38: A call to the default constructor of an exception was detected. Instead one of the constructor overloads should be called. This allows to provide more meaningful exceptions.

(detekt.exceptions.ThrowingExceptionsWithoutMessageOrCause)

🔇 Additional comments (50)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/review/ReviewContent.kt (2)

22-23: 좋은 리팩토링입니다!

reviewId 타입을 String에서 Long으로 변경하고 onClick 시그니처를 (Long) -> Unit으로 단순화한 것은 훌륭한 개선입니다. ID에 Long 타입을 사용하는 것이 더 표준적이며 타입 안정성을 높여줍니다. 또한 네비게이션에는 리뷰 ID만 필요하므로 API가 더욱 명확해졌습니다.

27-27: 구현이 올바릅니다.

onClick 핸들러에 reviewId만 전달하도록 구현이 정확하게 업데이트되었습니다. writer 파라미터는 여전히 UI 표시에 사용되고 있어 적절합니다.

core/network/src/main/java/team/retum/network/di/RequestUrls.kt (1)

85-90: 리뷰 API 경로 리팩토링 확인 완료
상세·목록·질문·집계·내 리뷰 경로가 공통 path 기반으로 정리되어 새 Retrofit 시그니처와 일관됩니다. GET/POST 경로도 상수 재사용이 가능해 혼선이 줄어들 것 같습니다.

core/common/src/main/java/team/retum/common/enums/ApplyStatus.kt (1)

6-6: SEND 라벨 변경 의도를 확인하세요.

"지원 중"에서 "전송"으로 라벨이 변경되었습니다. 이 변경이 의도된 것이며 모든 UI 텍스트와 사용자 경험상 일관성이 있는지 확인이 필요합니다.

feature/home/src/main/java/team/retum/home/ui/ApplyCompanyItem.kt (1)

51-51: LGTM: SEND와 PROCESSING 상태 처리가 올바르게 추가됨

새로 추가된 SEND와 PROCESSING 상태를 secondary 색상으로 매핑한 것은 적절합니다.

feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt (2)

61-91: LGTM!

필터 선택 토글 로직이 올바르게 구현되었습니다. 동일한 값 재선택 시 선택 해제하는 패턴이 사용자 경험에 적합합니다.

94-113: LGTM!

ReviewsFilterState 데이터 클래스가 적절하게 정의되었습니다. @Immutable 어노테이션과 팩토리 메서드 패턴이 Compose 모범 사례에 부합합니다.

feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt (1)

34-36: LGTM!

setTabIndex 함수는 탭 인덱스를 올바르게 업데이트하며, 구현이 명확합니다.

feature/bookmark/src/main/java/team/retum/bookmark/ui/BookmarkScreen.kt (1)

135-135: 드로어블 리소스 변경 - 시각적 적절성 검증 필요

ic_empty_bookmark에서 설계 시스템의 제네릭 ic_empty로 변경되었습니다. 새 리소스는 디자인 시스템에 존재하며(core/design-system/src/main/res/drawable/ic_empty.png), 이 변경이 BookmarkScreen에만 적용되어 있습니다.

확인 사항:

제네릭 ic_empty가 북마크 빈 상태 컨텍스트에 시각적으로 적절한지 검증

이것이 의도된 디자인 시스템 통합인지 확인

contentDescription은 여전히 "empty bookmark"이므로, 선택된 아이콘이 북마크 컨텍스트와 일치하는지 확인해주세요.

feature/post-review/src/main/res/values/strings.xml (1)

1-58: LGTM!

문자열 리소스가 화면별로 잘 정리되어 있으며, 한국어 표기법도 적절합니다.

app/src/main/java/team/retum/jobisandroidv2/ui/BottomNavigationBar.kt (1)

28-28: LGTM!

하단 네비게이션에 Review 메뉴 아이템이 올바르게 추가되었습니다.

settings.gradle.kts (1)

47-47: LGTM!

새로운 post-review 모듈이 올바르게 추가되었습니다.

app/src/main/res/values/strings.xml (1)

6-6: LGTM!

후기 기능을 위한 문자열 리소스가 올바르게 추가되었습니다.

core/design-system/src/main/res/drawable/ic_asterisk.xml (1)

1-9: LGTM!

필수 입력 표시를 위한 별표 아이콘이 올바르게 정의되었습니다.

feature/company/src/main/java/team/retum/company/viewmodel/CompanyDetailsViewModel.kt (1)

45-53: LGTM!

fetchReviewsUseCase의 확장된 시그니처에 맞게 파라미터가 올바르게 전달되고 있습니다. 회사 상세 화면에서는 companyId만 필요하므로 나머지 파라미터를 null로 전달하는 것이 적절합니다.

app/src/main/java/team/retum/jobisandroidv2/ui/BottomMenu.kt (2)

10-10: LGTM!

NAVIGATION_REVIEW import가 올바르게 추가되었습니다.

29-33: LGTM!

Review 하단 메뉴 아이템이 기존 패턴과 일관되게 잘 정의되었습니다.

core/common/src/main/java/team/retum/common/enums/ReviewProcess.kt (1)

4-8: 검증 완료 - 모든 변경이 올바르게 적용되었습니다.

검증 결과, ReviewProcess 열거형의 상수 변경(QUESTION, TECH, FINISH → INTERVIEW_TYPE, INTERVIEW_LOCATION, TECH_STACK, INTERVIEWER_COUNT, SUMMARY)이 코드베이스 전체에 완전하고 일관되게 적용되었습니다:

제거된 상수에 대한 직접 참조 없음

문자열 기반 참조 없음

모든 사용처에서 새로운 상수로 업데이트됨 (PostReviewViewModel.kt, PostReviewScreen.kt)

breaking change이지만 변경이 완전하게 처리되어 컴파일 오류나 런타임 문제가 발생하지 않습니다.

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisIcon.kt (1)

40-40: 변경 사항이 적절합니다.

기존 패턴을 따라 새로운 아이콘 상수를 추가했습니다.

core/design-system/src/main/res/drawable/ic_success.xml (1)

1-9: 벡터 드로어블이 올바르게 정의되었습니다.

표준 벡터 드로어블 형식을 따르고 있으며, 성공 완료 화면에서 사용될 아이콘으로 적절합니다.

core/network/src/main/java/team/retum/network/model/response/FetchReviewDetailResponse.kt (2)

23-27: 내부 데이터 클래스 리팩토링이 적절합니다.

Detail에서 QnAs로 명명이 개선되었고, area 필드가 제거되어 구조가 단순화되었습니다. 명명이 더 명확해졌습니다.

10-20: 검증 완료 - 모든 변경사항이 올바르게 반영되었습니다.

새로운 필드 구조가 모든 계층에서 일관되게 처리되고 있습니다. JSON 필드명(review_id, company_name, company_name 등)이 모두 적절한 snake_case로 설정되어 있으며, API → DataSource → Repository → UseCase → ViewModel 전체 파이프라인에서 toEntity() 확장함수를 통해 정상적으로 매핑되고 있습니다. QnAs 중첩 클래스도 별도 매핑함수로 지원되고 있습니다.

app/build.gradle.kts (1)

118-118: 모듈 의존성 추가가 적절합니다.

새로운 post-review 기능 모듈을 앱에 통합하는 표준적인 방식입니다.

app/src/main/res/drawable/ic_review.xml (1)

1-9: 벡터 드로어블이 올바르게 정의되었습니다.

하단 네비게이션용 리뷰 아이콘이 적절한 크기(20dp)와 색상으로 정의되었습니다.

core/domain/src/main/java/team/retum/usecase/entity/ReviewsCountEntity.kt (1)

5-11: 엔티티 정의와 매핑이 적절합니다.

단순하고 명확한 엔티티 구조를 따르고 있으며, 네트워크 응답에서 도메인 엔티티로의 매핑이 올바르게 구현되었습니다. totalPageCount에 Long 타입을 사용한 것도 적절합니다.

core/common/src/main/java/team/retum/common/enums/InterviewType.kt (1)

3-7: 백엔드 API 스펙 확인이 필요합니다.

현재 InterviewType 열거형은 명시적인 직렬화 어노테이션이 없으며, Moshi의 기본 동작에 따라 INDIVIDUAL, GROUP, OTHER 형식으로 직렬화됩니다. 코드베이스에서 커스텀 어댑터나 테스트 파일을 찾을 수 없었으므로, 백엔드 API가 실제로 이 형식을 기대하는지 백엔드 스펙을 통해 직접 확인해야 합니다.

core/common/src/main/java/team/retum/common/enums/InterviewLocation.kt (1)

3-8: 백엔드 API 스펙을 통해 enum 직렬화 형식 검증 필요

InterviewLocation enum은 Moshi에 의해 enum 상수명이 그대로 직렬화됩니다 (DAEJEON, SEOUL, GYEONGGI, OTHER). InterviewType과 동일하게 명시적인 직렬화 어노테이션이 없으므로, 백엔드가 정확히 이 형식을 기대하는지 API 문서 또는 백엔드 팀을 통해 확인하세요.

enum 상수는 이미 요청/응답 클래스(PostReviewRequest.kt, FetchReviewDetailResponse.kt)에서 사용 중이므로, 기존 백엔드 계약이 일치하는지 재확인하시기 바랍니다.

core/domain/src/main/java/team/retum/usecase/entity/MyReviews.kt (1)

1-23: LGTM!

도메인 엔티티와 변환 로직이 깔끔하게 구현되어 있습니다. 적절한 가시성 제어자 사용과 불변성 보장이 잘 되어 있습니다.

core/network/src/main/java/team/retum/network/model/response/FetchReviewsCountResponse.kt (1)

1-9: LGTM!

간단하고 명확한 응답 모델입니다. Moshi 어댑터 생성과 JSON 매핑이 적절하게 구성되어 있습니다.

feature/review/src/main/java/team/retum/review/navigation/ReviewNavigation.kt (1)

1-28: LGTM!

Jetpack Compose Navigation을 활용한 깔끔한 네비게이션 구성입니다. 콜백 기반 설계가 적절하며 확장 함수를 통한 API가 사용하기 편리합니다.

core/domain/src/main/java/team/retum/usecase/usecase/review/FetchMyReviewUseCase.kt (1)

1-13: LGTM!

간단하고 명확한 Use Case 구현입니다. runCatching을 통한 에러 처리와 엔티티 변환이 적절하게 구성되어 있습니다.

core/network/src/main/java/team/retum/network/model/response/FetchMyReviewResponse.kt (1)

1-15: LGTM!

네트워크 응답 모델이 깔끔하게 구성되어 있습니다. Moshi 어댑터 생성과 JSON 필드 매핑이 적절합니다.

core/network/src/main/java/team/retum/network/model/response/FetchReviewsResponse.kt (1)

12-17: LGTM!

응답 모델의 필드 업데이트가 적절합니다. reviewId의 타입 변경(String → Long)과 새로운 필드들(companyName, companyLogoUrl, major) 추가가 확장된 기능 요구사항과 잘 맞습니다.

feature/review/src/main/java/team/retum/review/ui/component/ReviewItems.kt (1)

26-81: LGTM!

리뷰 아이템 컴포저블의 구현이 깔끔합니다. 이미지 URL이 비어있을 때의 폴백 처리, 텍스트 오버플로우 처리, 클릭 이벤트 전달 등이 모두 적절하게 구현되어 있습니다.

feature/company/src/main/java/team/retum/company/navigation/CompanyDetailsNavigation.kt (1)

15-15: Long 기반 리뷰 식별자로의 전환이 적절합니다.

리뷰 상세 네비게이션 파라미터를 (String, String)에서 (Long)으로 변경하여 타입 안정성과 일관성이 개선되었습니다.

core/domain/src/main/java/team/retum/usecase/usecase/review/FetchQuestionsUseCase.kt (1)

7-13: LGTM!

UseCase의 구현이 간결하고 적절합니다. runCatching을 사용하여 에러를 Result로 래핑하는 방식이 적절하며, 호출자가 성공/실패를 처리할 수 있도록 되어 있습니다.

core/domain/src/main/java/team/retum/usecase/entity/QuestionsEntity.kt (1)

16-16: 네이밍 패턴 일관성 확인 완료 - 지적 사항 없음

프로젝트 전체의 toEntity() 함수 패턴을 검토한 결과, QuestionsEntity.kt의 함수명과 구조는 이미 프로젝트의 표준 네이밍 규칙과 완벽하게 일치합니다. 18개 이상의 엔티티 파일에서 internal fun ResponseType.toEntity() = EntityType(...) 패턴을 따르고 있으며, QuestionsEntity.kt도 동일한 규칙을 준수하고 있습니다. TODO 주석은 제거해도 무방합니다.

Likely an incorrect or invalid review comment.

core/domain/src/main/java/team/retum/usecase/usecase/review/FetchReviewsUseCase.kt (1)

12-14: 파라미터 위임 검증 완료

확장된 필터 인자가 repository 계층으로 정확히 전달되고 runCatching 으로 오류를 포장하는 흐름이 명확합니다.

core/data/src/main/java/team/retum/data/repository/review/ReviewRepositoryImpl.kt (1)

20-49: 데이터소스 위임이 올바르게 정렬되었습니다

repository 계층에서 추가된 필터 파라미터를 전부 data source 에 네임드 인자로 넘겨주어 매핑 오류 위험이 없습니다.

core/domain/src/main/java/team/retum/usecase/entity/PostReviewEntity.kt (1)

7-21: 도메인 엔티티 구조 변경이 적절합니다.

면접 후기 엔티티에 새로운 필드들(interviewType, location, jobCode, interviewerCount, question, answer)이 추가되었고, PostReviewContentEntity의 question 타입이 String에서 Long으로 변경되어 식별자로서 더 적합한 타입을 사용하게 되었습니다.

core/network/src/main/java/team/retum/network/datasource/review/ReviewDataSource.kt (1)

27-31: 새로운 리뷰 관련 메서드 추가가 적절합니다.

fetchQuestions(), fetchReviewsCount(), fetchMyReviews() 메서드가 추가되어 리뷰 기능의 API 표면이 확장되었습니다.

app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt (1)

59-62: 리뷰 관련 콜백 추가 및 전달이 올바르게 구현되었습니다.

새로운 콜백들(onReviewFilterClick, onSearchReviewClick, onReviewDetailClick)이 Root → RootScreen → review 컴포저블로 일관되게 전달되고 있으며, onPostReviewClick의 시그니처도 추가 파라미터를 받도록 적절히 업데이트되었습니다.

Also applies to: 99-101, 135-138, 185-189

app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (1)

201-203: reviewId 타입이 Long으로 변경된 것이 적절합니다.

이전 String 타입에서 Long으로 변경되어 도메인 모델과 일관성을 갖추게 되었습니다.

app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (1)

118-133: 포스트 리뷰 네비게이션 체인이 올바르게 구성되었습니다.

postReview → postNextReview → postExpectReview → postReviewComplete 흐름이 적절한 콜백과 함께 구성되었습니다. 각 단계가 필요한 데이터를 다음 단계로 전달하고 있습니다.

core/network/src/main/java/team/retum/network/api/ReviewApi.kt (1)

24-47: 리뷰 API 확장이 적절하게 구현되었습니다.

fetchReviews가 페이지네이션 및 다중 필터링을 지원하도록 확장되었고, 새로운 엔드포인트들(fetchQuestions, fetchReviewsCount, fetchMyReviews)이 추가되어 리뷰 기능을 충분히 지원합니다. 모든 쿼리 파라미터가 nullable로 선언되어 선택적 필터링이 가능합니다.

feature/review/src/main/java/team/retum/review/ui/ReviewFilterScreen.kt (1)

47-66: UI 구조와 컴포넌트 구성이 잘 되어 있습니다.

ReviewFilter 진입점부터 세부 섹션들(Skills, Years, InterviewType, Location)까지 잘 구조화되어 있으며, 애니메이션과 상태 관리가 적절하게 구현되었습니다. FlowRow를 사용한 레이아웃과 색상 전환 애니메이션이 좋은 사용자 경험을 제공합니다.

Also applies to: 68-119, 123-373
feature/review/src/main/java/team/retum/review/ui/ReviewDetailsScreen.kt (4)
77-110: 명시적 데이터 플로우로 개선된 구조

ViewModel 상태에 직접 의존하지 않고 명시적인 파라미터를 통한 데이터 전달 방식으로 변경하여 테스트 가능성과 재사용성이 향상되었습니다.

162-172: 열거형 매핑 로직이 올바르게 구현됨

InterviewType과 InterviewLocation 열거형의 모든 케이스를 exhaustive하게 처리하고 있어 안전합니다.

194-201: 깔끔한 위임 패턴

InterviewReview가 ReviewContent로 명확하게 위임하고 있습니다.

207-207: 빈 값 검증 로직 확인 필요

현재 !review.answer.isBlank() || !review.question.isBlank() 로직은 둘 중 하나라도 비어있지 않으면 콘텐츠를 표시합니다. 의도가 "둘 다 존재해야 표시"라면 && 연산자를 사용해야 합니다.

의도한 로직이 다음 중 어느 것인지 확인하세요:

현재: 질문 또는 답변 중 하나라도 있으면 표시

제안: 질문과 답변이 모두 있어야 표시 (&& 사용)
// 제안하는 변경
if (!review.answer.isBlank() && !review.question.isBlank()) {
    // show content
}

app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt

feature/mypage/src/main/java/team/retum/jobis/ui/MyPageScreen.kt

...re/post-review/src/main/java/team/retum/post/review/navigation/PostExpectReviewNavigation.kt

feature/post-review/src/main/java/team/retum/post/review/ui/PostNextReviewScreen.kt

feature/post-review/src/main/java/team/retum/post/review/ui/PostReviewCompleteScreen.kt

feature/review/src/main/java/team/retum/review/viewmodel/ReviewDetailsViewModel.kt

coderabbitai · 2025-11-02T14:48:19Z

feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt

+    companion object {
+        var code: Long? = null
+        var year: Int? = null
+        var interviewType: InterviewType? = null
+        var location: InterviewLocation? = null
+    }


⚠️ Potential issue | 🔴 Critical

companion object의 mutable static state를 제거하세요.

companion object에 var로 선언된 필터 상태는 여러 문제를 야기합니다:

ViewModel 인스턴스 간 상태 공유로 인한 예기치 않은 동작

메모리 누수 위험 (정적 참조가 계속 유지)

단위 테스트 불가능 (테스트 간 격리 불가)

ReviewsFilterState에 동일한 속성이 이미 존재하여 중복

모든 필터 상태는 ReviewsFilterState 인스턴스를 통해 관리되어야 합니다.

다음 diff를 적용하여 companion object를 제거하세요:

- companion object { - var code: Long? = null - var year: Int? = null - var interviewType: InterviewType? = null - var location: InterviewLocation? = null - } - init {

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

🤖 Prompt for AI Agents

In feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt around lines 22 to 27, remove the companion object that defines mutable static vars (code, year, interviewType, location) and stop using static state; instead, add those properties to and manage them via the existing ReviewsFilterState instance held by the ViewModel (initialize/update/read from the state object, expose changes via LiveData/StateFlow as appropriate), eliminate any references to ReviewFilterViewModel.Companion.* across the codebase, and adjust constructors/usage so each ViewModel instance uses its own ReviewsFilterState to avoid shared mutable static state.

feature/review/src/main/java/team/retum/review/viewmodel/ReviewFilterViewModel.kt

feature/review/src/main/res/values/strings.xml

uson1004 added 30 commits August 16, 2025 19:05

feat :: 지원서 서류 상태 추가

d8272ba

feat :: 면접후기 상태 추가

210b44e

refactor :: color system primary200 light theme색깔 적용

df136c3

feat :: 면접 구분 페이지 추가

2860e52

refactor :: 기존 면접후기 페이지 제거

c579233

refactor :: 안쓰이는 코드 부분 주석, 삭제 처리

dadf486

feat :: 면접 구분 선택 페이지 구현

2d23a5c

feat :: 인디케이터 사이즈 이벤트 및 interviewLocation 구현

a3bdd3e

feat :: 기업명 조회 기능 삭제

5fd23f1

refactor :: keyword 검색 모달 복구

da18f22

feat :: setInterviewCount 추가

c0703fc

refactor :: buttonEnabled 방식 변경

d673c5e

feat :: stringresource 추가

a232057

feat :: string 적용

7e8020f

refactor :: 전공 조회 기능 리팩토링

9931037

feat :: asterisk 아이콘 추가

f7a397d

refactor :: postReview modal 크기 고정 설정

99b04e8

refactor :: 후기 리스트 조회 api 수정

0b031f7

refactor ::� 후기 조회 각 레이어 별 프로퍼티 및 엔티티 수정

44e9235

refactor :: 후기 상세 조회 api 수정

664da85

feat :: 리뷰 질문 조회 api 생성

796f944

feat :: reviews count 조회 api 구현

1758710

feat :: review datasource 구현

1bcdbd6

feat : review repository 구현

d04f2ec

feat :: 질문 조회 usecase 구현

aa046dc

feat :: reviews count usecase 구현

90b968a

feat :: 후기 bottomNavigation 구현

8a6a1f2

refactor :: review ui 파일 company로 이전

738e890

refactor :: review navigation 파일 company로 이전

41f78ab

refactor :: reviews 네비게이션 인자 삭제

f6aee92

uson1004 added 3 commits October 28, 2025 13:45

refactor :: 모듈 경로 변경

0829640

refactor :: post-review 모듈 경로 제거

036f30d

chore :: ktlint Check

6f55f18

uson1004 self-assigned this Oct 28, 2025

uson1004 requested a review from a team as a code owner October 28, 2025 07:09

uson1004 added this to return 3기 backlog Oct 28, 2025

uson1004 added 리팩토링 코드를 리팩토링 합니다. 기능 구현 비즈니스 로직을 작성합니다. labels Oct 28, 2025

uson1004 linked an issue Oct 28, 2025 that may be closed by this pull request

면접후기 기능 리팩토링 #468

Open

5 tasks

github-project-automation bot moved this to Todo in return 3기 backlog Oct 28, 2025

coderabbitai bot reviewed Oct 28, 2025

View reviewed changes

coderabbitai bot mentioned this pull request Oct 28, 2025

📝 Add docstrings to feature/468-면접후기-기능-리팩토링 #474

Open

uson1004 added 12 commits October 28, 2025 23:13

chore :: InterestApi 제거

eda496e

feat :: stringResource 추가

2c391bc

feat :: stringResource 적용

3063d5e

feat :: post review 프로세스 popStack

8147da7

refactor :: 비었을 때 empty view로 처리

481e08c

refactor :: tapbar 예상 면접 키워드 수정

db1ea50

refactor :: 필터링 년도 리스트 역순으로 정렬

a2af718

refactor :: searchFilter stringresource 변경

b865cb7

chore :: ktlint Check

3b66c17

refactor :: textfield viewModel 로직 구현

a6f81e5

refactor :: fetchReview 파라미터 통일

7b9981f

chore :: 스트링 맞춤법 수정

13a317c

coderabbitai bot reviewed Nov 2, 2025

View reviewed changes

Team-return deleted a comment from coderabbitai bot Nov 4, 2025

🔗 :: (#468) 면접후기 기능 리팩토링 #473

Are you sure you want to change the base?

🔗 :: (#468) 면접후기 기능 리팩토링 #473

Conversation

uson1004 commented Oct 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

개요

작업 내용

할 말

Summary by CodeRabbit

릴리스 노트

Uh oh!

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

Uh oh!

🐰 코드 리뷰 요약

워크스루

변경 사항

시퀀스 다이어그램

예상 코드 리뷰 소요 시간

추가 검토 필요 영역:

관련 PR

추천 리뷰어

시

Pre-merge checks and finishing touches

Uh oh!

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

Uh oh!

Uh oh!

coderabbitai bot commented Oct 28, 2025

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot Nov 2, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

uson1004 commented Oct 28, 2025 •

edited

Loading

coderabbitai bot commented Oct 28, 2025 •

edited

Loading

coderabbitai bot commented Oct 28, 2025 •

edited

Loading