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 타입 안전성 개선

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

1. 공백이나 특수문자가 포함된 회사명은 네비게이션 경로를 깨뜨립니다.
2. 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,
         ))
     }
 }

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,
+                                // 확장된 상태에서는 전체 텍스트 표시
                            )

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

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

22-27: companion object의 mutable static state를 제거해야 합니다.

이전 리뷰에서 지적된 critical 이슈가 여전히 해결되지 않았습니다. companion object의 가변 정적 변수들은 다음 문제를 야기합니다:

• 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 {

---

29-31: init에서 getLocalYears()를 호출해야 합니다.

getLocalYears()가 호출되지 않아 ReviewsFilterState.years가 항상 빈 리스트로 유지됩니다. 이는 년도 필터 UI가 제대로 작동하지 않는 원인이 됩니다.

다음 diff를 적용하세요:

     init {
         fetchCodes()
+        getLocalYears()
     }

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

20-29: JSON 역직렬화 오류 처리 확인 필요

it.getReviewData()(라인 25)가 JSON 파싱 실패 시 예외를 던질 수 있습니다. 이전 리뷰에서 지적된 것처럼, toReviewData() 구현에 try-catch 블록이 없는 경우 앱이 크래시될 수 있습니다.

getReviewData() 및 toReviewData() 구현에서 JSON 파싱 실패와 URL 디코딩 실패를 처리하고, 실패 시 적절한 에러 UI를 표시하거나 안전하게 뒤로 이동하도록 개선하세요.

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

38-41: 예외 메시지 누락 및 오류 처리 미흡

라인 39에서 메시지 없이 NullPointerException()을 던지는 것은 디버깅을 어렵게 만듭니다. 또한 toReviewData() 호출 시 JSON 파싱 실패를 처리하지 않아 앱 크래시 위험이 있습니다.

다음과 같이 개선하세요:

 internal fun NavBackStackEntry.getReviewData(): PostReviewData {
-    val reviewData = arguments?.getString(ResourceKeys.REVIEW_DATA) ?: throw NullPointerException()
-    return reviewData.toReviewData()
+    val raw = requireNotNull(arguments?.getString(ResourceKeys.REVIEW_DATA)) {
+        "Missing '${ResourceKeys.REVIEW_DATA}' argument for $NAVIGATION_POST_NEXT_REVIEW"
+    }
+    return try {
+        raw.toReviewData()
+    } catch (e: Exception) {
+        throw IllegalArgumentException("Failed to parse review data: ${e.message}", e)
+    }
 }

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

24-29: 에러 처리 누락으로 실패 시 사용자 피드백 없음

fetchReviewDetailsUseCase가 실패할 경우 아무런 상태 변화도 없어서 사용자는 로딩이 끝났는지조차 알 수 없습니다. 최소한 실패 플래그를 상태에 기록해 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, hasError = false) }
+                }
+                .onFailure {
+                    setState { state.value.copy(hasError = true) }
+                }
         }
     }

 internal data class ReviewDetailsState(
+    val hasError: Boolean,
     val selectedTabIndex: Int,
     val reviewId: Long,
     val reviewDetail: FetchReviewDetailEntity,
 ) {
     companion object {
         fun getInitialState() = ReviewDetailsState(
+            hasError = false,
             selectedTabIndex = 0,
             reviewId = 0L,
             reviewDetail = FetchReviewDetailEntity(

---

45-59: 초기 상태가 실제 데이터처럼 보이는 기본값 사용

초기 상태에서 InterviewType.INDIVIDUAL과 InterviewLocation.GYEONGGI를 넣으면 데이터 로딩 전에도 실제 값처럼 렌더링돼 사용자가 혼동할 수 있습니다. 중립값(OTHER)으로 바꾸거나 로딩 상태를 분리해 주세요.

                 type = InterviewType.INDIVIDUAL,
-                location = InterviewLocation.GYEONGGI,
+                type = InterviewType.OTHER,
+                location = InterviewLocation.OTHER,

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

173-177: 마지막 페이지 인덱스 하드코딩으로 흐름 깨짐

질문 수가 3개가 아니면 pagerState.currentPage != 2 조건이 바로 틀어져서 다음/완료 전환이 망가집니다. 마지막 페이지를 동적으로 계산하세요.

                 JobisButton(
                     text = stringResource(id = R.string.next),
                     color = ButtonColor.Primary,
                     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()
+                            }
                         }
                     },
                 )

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

55-58: 콜백 시그니처 불일치 확인 필요

navigator::navigateToPostReview가 메서드 참조로 전달되고 있으나, navigateToPostReview는 (companyName: String, companyId: Long) 파라미터를 필요로 합니다. 이는 파라미터가 없는 콜백으로 전달될 수 없습니다.

다음 스크립트로 실제 호출 방식을 확인하여 컴파일 오류 여부를 검증하세요:

#!/bin/bash
# root 함수의 onPostReviewClick 파라미터 타입 확인
ast-grep --pattern $'fun NavGraphBuilder.root(
  $$$
  onPostReviewClick: $TYPE,
  $$$
)'

# RootScreen에서 onPostReviewClick 호출 방식 확인
rg -n 'onPostReviewClick' --type=kt -A 2 -B 2

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

95-103: 데이터 유효성 검증 권장

year와 interviewerCount를 검증 없이 toString()으로 변환하고 있습니다. 서버에서 0이나 음수 값이 전달될 경우 의미 없는 문자열이 UI에 표시될 수 있습니다.

도메인 계층(FetchReviewDetailEntity)에 유효성 검증을 추가하는 것을 권장합니다. 다음 스크립트로 현재 검증 로직 여부를 확인하세요:

#!/bin/bash
# FetchReviewDetailEntity에 유효성 검증 어노테이션이 있는지 확인
rg -n '@field:|@Min|@Max|@Positive' core/domain/src/main/java/team/retum/usecase/entity/FetchReviewDetailEntity.kt

# 응답 모델에서 유효성 검증 확인
fd FetchReviewDetailResponse.kt --exec cat {}

---

60-64: TODO 주석 제거 필요

프로덕션 코드에 TODO 주석이 남아있습니다. 실제 데이터 처리가 완료되었다면 주석을 제거하고, 미완료 작업이라면 구현을 완료해야 합니다.

다음과 같이 TODO 주석을 제거하세요:

 LaunchedEffect(reviewId) {
-    // TODO : 실 값 들어왔을 때 UI 호출
     reviewDetailsViewModel.setReviewId(reviewId)
     reviewDetailsViewModel.fetchReviewDetails()
 }

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

159-177: 면접관 수 파싱 시 예외 처리 누락

count.toInt()가 빈 문자열이나 숫자가 아닌 입력을 받으면 NumberFormatException이 발생합니다. 안전한 파싱과 유효성 검증을 추가해야 합니다.

다음 diff를 적용하여 안전한 파싱을 구현하세요:

 internal fun onNextClick() {
     with(state.value) {
-        if (interviewType != null && interviewLocation != null) {
-            postSideEffect(
-                PostReviewSideEffect.MoveToNext(
-                    companyId = companyId,
-                    interviewerCount = count.toInt(),
-                    jobCode = selectedTech ?: 0,
-                    interviewType = interviewType,
-                    location = interviewLocation,
-                ),
-            )
+        val parsedCount = count.toIntOrNull()
+        if (parsedCount == null || parsedCount <= 0) {
+            postSideEffect(PostReviewSideEffect.BadRequest)
+            return
+        }
+        if (interviewType != null && interviewLocation != null) {
+            postSideEffect(
+                PostReviewSideEffect.MoveToNext(
+                    companyId = companyId,
+                    interviewerCount = parsedCount,
+                    jobCode = selectedTech ?: 0,
+                    interviewType = interviewType,
+                    location = interviewLocation,
+                ),
+            )
         } else {
             postSideEffect(
                 PostReviewSideEffect.SelectTechAndCount
             )
         }
     }
 }

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

15-16: 미사용 import를 제거하거나 LazyColumn 사용을 고려하세요.

LazyColumn과 items를 import했지만 실제로는 forEach를 사용하고 있습니다.

---

198-205: 이전 NPE 위험이 해결되었습니다. 성능 최적화를 위해 LazyColumn 사용을 고려하세요.

reviewableItem을 로컬 변수로 캡처하여 이전 리뷰에서 지적된 NPE 위험이 해결되었습니다.

다만 Compose에서 forEach를 사용하면 각 항목이 개별적으로 추적되지 않아 재구성 시 비효율적일 수 있습니다. 목록이 길어질 경우 이미 import한 LazyColumn과 items()를 사용하는 것을 권장합니다.

적용 가능한 diff:

-            state.reviewableCompany?.let { reviewable ->
-                reviewable.forEach { reviewableItem ->
-                    WriteInterviewReview(
-                        companyName = reviewableItem.name,
-                        onClick = { onPostReviewClick(reviewableItem.name, reviewableItem.id) },
-                    )
-                }
-            }
+            state.reviewableCompany?.let { reviewable ->
+                LazyColumn {
+                    items(
+                        items = reviewable,
+                        key = { it.id }
+                    ) { reviewableItem ->
+                        WriteInterviewReview(
+                            companyName = reviewableItem.name,
+                            onClick = { onPostReviewClick(reviewableItem.name, reviewableItem.id) },
+                        )
+                    }
+                }
+            }

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

29-45: 사용하지 않는 파라미터 제거 고려

onBackPressed 파라미터가 정의되어 있지만(라인 31) 실제로 사용되지 않습니다. 이 파라미터가 향후 사용 예정이 아니라면 제거를 고려하세요.

 internal fun PostReviewComplete(
-    onBackPressed: () -> Unit,
     navigateToMyPage: () -> Unit,
     postReviewViewModel: PostReviewViewModel = hiltViewModel(),
 )

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

66-88: 예외 처리 범위 확장 권장

postReview 메서드가 BadRequestException만 처리하고 있습니다. 네트워크 오류나 기타 예외 상황에 대한 처리를 추가하면 사용자 경험이 개선됩니다.

다음과 같이 일반 예외 처리를 추가하는 것을 고려해보세요:

 internal fun postReview(reviewData: PostReviewData) {
     viewModelScope.launch(Dispatchers.IO) {
         postReviewUseCase(
             postReviewRequest = PostReviewEntity(
                 interviewType = reviewData.interviewType,
                 location = reviewData.location,
                 companyId = reviewData.companyId,
                 jobCode = reviewData.jobCode,
                 interviewerCount = reviewData.interviewerCount,
                 qnaElements = reviewData.qnaElements.map { it.toEntity() },
                 question = reviewData.question,
                 answer = reviewData.answer,
             ),
         ).onSuccess {
             postSideEffect(PostReviewSideEffect.Success)
         }.onFailure {
             when (it) {
                 is BadRequestException -> {
                     postSideEffect(PostReviewSideEffect.BadRequest)
                 }
+                else -> {
+                    postSideEffect(PostReviewSideEffect.BadRequest)
+                }
             }
         }
     }
 }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro