core/common/src/main/kotlin/com/ninecraft/booket/core/common/utils/HandleException.kt (1)

60-94: 에러 메시지들을 string resource로 관리해야 합니다.

로직과 구조는 훌륭하지만, 다이얼로그 타이틀과 메시지들이 하드코딩되어 있어 i18n과 유지보수 측면에서 개선이 필요합니다:

• Line 62: "네트워크 연결이 불안정합니다.\n인터넷 연결을 확인해주세요"
• Line 68: "알 수 없는 문제가 발생했어요.\n다시 시도해주세요"
• Line 72: "로그인 오류"
• Line 73: "예기치 않은 오류가 발생했습니다.\n다시 로그인 해주세요."
• Line 77: "세션이 만료되었어요.\n다시 로그인 해주세요"
• Line 83: "알 수 없는 문제가 발생했어요.\n다시 시도해주세요"

이러한 문자열들을 strings.xml로 이동하면 다국어 지원과 일관된 문구 관리가 용이해집니다.

예시:

+// strings.xml에 추가
+<string name="error_network_connection">네트워크 연결이 불안정합니다.\n인터넷 연결을 확인해주세요</string>
+<string name="error_login_title">로그인 오류</string>
+<string name="error_login_message">예기치 않은 오류가 발생했습니다.\n다시 로그인 해주세요.</string>
+<string name="error_session_expired">세션이 만료되었어요.\n다시 로그인 해주세요</string>
+<string name="error_unknown">알 수 없는 문제가 발생했어요.\n다시 시도해주세요</string>

그런 다음 코드에서는:

fun postErrorDialog(
    context: Context, // Context 파라미터 추가 필요
    errorScope: ErrorScope,
    exception: Throwable,
    @StringRes buttonLabelResId: Int = R.string.confirm,
    action: () -> Unit = {},
) {
    val (title, message) = when {
        exception.isNetworkError() -> {
            null to context.getString(R.string.error_network_connection)
        }
        exception is HttpException -> {
            when (errorScope) {
                ErrorScope.GLOBAL -> {
                    null to context.getString(R.string.error_unknown)
                }
                ErrorScope.LOGIN -> {
                    context.getString(R.string.error_login_title) to 
                        context.getString(R.string.error_login_message)
                }
                ErrorScope.AUTH_SESSION_EXPIRED -> {
                    null to context.getString(R.string.error_session_expired)
                }
            }
        }
        else -> {
            null to context.getString(R.string.error_unknown)
        }
    }
    // ...
}

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomePresenter.kt (1)

62-68: 방어적 코딩을 위해 Guest 체크 추가를 권장합니다.

에러 발생 시 handleException을 호출하여 401 에러를 처리하는 로직이 추가되었습니다.

다음 사항들을 검토해주세요:

1. Guest 체크 누락: LibraryPresenter와 달리 명시적인 guest 체크가 없습니다. 현재는 RememberedEffect (lines 109-113)에서 non-guest만 loadHomeContent()를 호출하므로 문제가 없지만, 나중에 다른 경로에서 이 함수가 호출될 경우 guest 사용자에게도 handleException이 실행될 수 있습니다. 일관성과 방어적 코딩을 위해 다음과 같은 체크를 추가하는 것을 고려해보세요:

                    .onFailure { exception ->
                        uiState = UiState.Error(exception)

+                       if (userState !is UserState.Guest) {
                            handleException(
                                exception = exception,
                                onError = {},
                                onLoginRequired = {
                                    navigator.resetRoot(LoginScreen())
                                },
                            )
+                       }
                    }

2. 빈 onError 람다: LibraryPresenter와 마찬가지로 onError = {}로 전달되어 401이 아닌 다른 에러들은 다이얼로그로 표시되지 않습니다. 기존 UI 에러 표시 (line 60)를 유지하면서 401만 다이얼로그로 처리하려는 의도로 보이는데, 이것이 의도한 동작이 맞는지 확인해주세요.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: easyhooon
PR: YAPP-Github/Reed-Android#174
File: feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/book/BookSearchPresenter.kt:128-133
Timestamp: 2025-08-28T12:25:54.058Z
Learning: In BookSearchPresenter.kt, when a guest user tries to register a book and is redirected to login, the bottom sheet (isBookRegisterBottomSheetVisible) and selection state (selectedBookIsbn, selectedBookStatus) are intentionally kept open/preserved so that when the user returns from login, they can continue from where they left off without losing context.

core/common/src/main/kotlin/com/ninecraft/booket/core/common/utils/HandleException.kt (1)

60-94: 에러 다이얼로그 처리 로직이 명확하게 개선되었습니다.

title과 message를 직접 계산하여 ErrorDialogSpec을 생성하는 구조가 간결하고 이해하기 쉽습니다. 다만 유지보수성 향상을 위해 다음 사항들을 고려해보시기 바랍니다:

1. 메시지 중복 제거: 68번과 82번 라인의 "알 수 없는 문제가 발생했어요.\n다시 시도해주세요" 메시지가 중복됩니다. 상수로 추출하면 향후 문구 수정 시 일관성을 유지할 수 있습니다.

2. 문장 부호 일관성: 72번 라인 LOGIN 스코프 메시지만 마침표("해주세요.")로 끝나고, 나머지 메시지들은 마침표 없이 끝납니다. 사용자 경험 일관성을 위해 통일하는 것을 권장합니다.

+private const val UNKNOWN_ERROR_MESSAGE = "알 수 없는 문제가 발생했어요.\n다시 시도해주세요"
+
 fun postErrorDialog(
     errorScope: ErrorScope,
     exception: Throwable,
     @StringRes buttonLabelResId: Int = R.string.confirm,
     action: () -> Unit = {},
 ) {
     val (title, message) = when {
         exception.isNetworkError() -> {
             null to "네트워크 연결이 불안정합니다.\n인터넷 연결을 확인해주세요"
         }

         exception is HttpException -> {
             when (errorScope) {
                 ErrorScope.GLOBAL -> {
-                    null to "알 수 없는 문제가 발생했어요.\n다시 시도해주세요"
+                    null to UNKNOWN_ERROR_MESSAGE
                 }

                 ErrorScope.LOGIN -> {
-                    "로그인 오류" to "예기치 않은 오류가 발생했습니다.\n다시 로그인 해주세요."
+                    "로그인 오류" to "예기치 않은 오류가 발생했습니다.\n다시 로그인 해주세요"
                 }

                 ErrorScope.AUTH_SESSION_EXPIRED -> {
                     null to "세션이 만료되었어요.\n다시 로그인 해주세요"
                 }
             }
         }

         else -> {
-            null to "알 수 없는 문제가 발생했어요.\n다시 시도해주세요"
+            null to UNKNOWN_ERROR_MESSAGE
         }
     }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro