♻️ Toast 에러 처리 개선

Author: shp7724

SNUTT/CLAUDE.md

@@ -29,6 +29,11 @@ tuist build "SNUTT Dev"    # Build development configuration (RECOMMENDED)
 tuist build "SNUTT Prod"   # Build production configuration
 tuist build "SNUTT Widget" # Build app + widget extension
 
+# ⚠️ CRITICAL: ALWAYS run `tuist build` synchronously (NOT as a background job)!
+# Never use run_in_background=true or append '&' to build commands.
+# Wait for the build to complete before proceeding with other tasks.
+# This ensures proper error detection and prevents build state corruption.
+
 # Module-Specific Builds (Preview Schemes)
 # Use these for faster builds when working on a single module:
 tuist build "Timetable Preview"          # Build only Timetable module

SNUTT/Modules/Feature/APIClientInterface/Resources/Base.lproj/Localizable.strings

@@ -129,14 +129,6 @@
 "error.failureReason.duplicateThemeName" = "테마 이름이 중복됩니다.";
 "error.recoverySuggestion.duplicateThemeName" = "다른 테마 이름을 사용해주세요.";
 
-"error.description.deeplinkNotFound" = "딥링크 오류";
-"error.failureReason.deeplinkNotFound" = "딥링크 정보를 찾을 수 없습니다.";
-"error.recoverySuggestion.deeplinkNotFound" = "딥링크 정보를 확인하고 다시 시도해주세요.";
-
-"error.description.deeplinkProcessFailed" = "딥링크 처리 오류";
-"error.failureReason.deeplinkProcessFailed" = "딥링크 처리에 실패했습니다.";
-"error.recoverySuggestion.deeplinkProcessFailed" = "딥링크 정보를 확인하고 다시 시도해주세요.";
-
 "error.description.alreadyVerified" = "인증 오류";
 "error.failureReason.alreadyVerified" = "이미 인증된 계정입니다.";
 

SNUTT/Modules/Feature/APIClientInterface/Resources/en.lproj/Localizable.strings

@@ -129,14 +129,6 @@
 "error.failureReason.duplicateThemeName" = "The theme name is duplicated.";
 "error.recoverySuggestion.duplicateThemeName" = "Please use a different theme name.";
 
-"error.description.deeplinkNotFound" = "Deeplink Error";
-"error.failureReason.deeplinkNotFound" = "The deeplink information cannot be found.";
-"error.recoverySuggestion.deeplinkNotFound" = "Please check the deeplink information and try again.";
-
-"error.description.deeplinkProcessFailed" = "Deeplink Processing Error";
-"error.failureReason.deeplinkProcessFailed" = "Failed to process the deeplink.";
-"error.recoverySuggestion.deeplinkProcessFailed" = "Please check the deeplink information and try again.";
-
 "error.description.alreadyVerified" = "Verification Error";
 "error.failureReason.alreadyVerified" = "The account is already verified.";
 

SNUTT/Modules/Feature/APIClientInterface/Sources/Error/APIClientError.swift

@@ -25,3 +25,9 @@ extension ClientError: APIClientError {
         underlyingError as? ClientUnknownServerError
     }
 }
+
+extension APIClientError {
+    public var failureReason: String? {
+        serverError?.failureReason ?? localizedCode?.failureReason
+    }
+}

SNUTT/Modules/Feature/APIClientInterface/Sources/Error/LocalizedErrorCode.swift

@@ -72,10 +72,6 @@ public enum LocalizedErrorCode: Int, LocalizedError {
     case invalidLectureTime = 0x5002
     case invalidRnBundle = 0x5003
     case duplicateThemeName = 40904
-    case deeplinkLectureNotFound = 0x5004
-    case deeplinkTimetableNotFound = 0x5005
-    case deeplinkBookmarkNotFound = 0x5006
-    case deeplinkProcessFailed = 0x5007
 
     case alreadyVerifiedAccount = 0x9000
     case alreadyVerifiedEmail = 0x9001
@@ -154,10 +150,6 @@ public enum LocalizedErrorCode: Int, LocalizedError {
             APIClientInterfaceStrings.errorDescriptionInvalidRnBundle
         case .duplicateThemeName:
             APIClientInterfaceStrings.errorDescriptionDuplicateThemeName
-        case .deeplinkLectureNotFound, .deeplinkTimetableNotFound, .deeplinkBookmarkNotFound:
-            APIClientInterfaceStrings.errorDescriptionDeeplinkNotFound
-        case .deeplinkProcessFailed:
-            APIClientInterfaceStrings.errorDescriptionDeeplinkProcessFailed
         case .alreadyVerifiedAccount, .alreadyVerifiedEmail:
             APIClientInterfaceStrings.errorDescriptionAlreadyVerified
         case .duplicateVacancyNotification:
@@ -241,10 +233,6 @@ public enum LocalizedErrorCode: Int, LocalizedError {
             APIClientInterfaceStrings.errorFailureReasonInvalidRnBundle
         case .duplicateThemeName:
             APIClientInterfaceStrings.errorFailureReasonDuplicateThemeName
-        case .deeplinkLectureNotFound, .deeplinkTimetableNotFound, .deeplinkBookmarkNotFound:
-            APIClientInterfaceStrings.errorFailureReasonDeeplinkNotFound
-        case .deeplinkProcessFailed:
-            APIClientInterfaceStrings.errorFailureReasonDeeplinkProcessFailed
         case .alreadyVerifiedAccount, .alreadyVerifiedEmail:
             APIClientInterfaceStrings.errorFailureReasonAlreadyVerified
         case .duplicateVacancyNotification:
@@ -328,10 +316,6 @@ public enum LocalizedErrorCode: Int, LocalizedError {
             APIClientInterfaceStrings.errorRecoverySuggestionInvalidRnBundle
         case .duplicateThemeName:
             APIClientInterfaceStrings.errorRecoverySuggestionDuplicateThemeName
-        case .deeplinkLectureNotFound, .deeplinkTimetableNotFound, .deeplinkBookmarkNotFound:
-            APIClientInterfaceStrings.errorRecoverySuggestionDeeplinkNotFound
-        case .deeplinkProcessFailed:
-            APIClientInterfaceStrings.errorRecoverySuggestionDeeplinkProcessFailed
         case .alreadyVerifiedAccount, .alreadyVerifiedEmail:
             nil
         case .duplicateVacancyNotification:

SNUTT/Modules/Feature/Friends/Sources/UI/FriendsScene.swift

@@ -13,9 +13,21 @@ import ThemesInterface
 import TimetableInterface
 
 public struct FriendsScene: View {
+    struct SelectedLecture: Identifiable {
+        let id: String
+        let lecture: Lecture
+        let quarter: Quarter
+
+        init(lecture: Lecture, quarter: Quarter) {
+            self.id = lecture.id
+            self.lecture = lecture
+            self.quarter = quarter
+        }
+    }
+
     @AppStorage("isNewToFriendsService") private var isNewToFriendsService: Bool = true
     @State var viewModel: FriendsViewModel = .init()
-    @State private var selectedLecture: Lecture?
+    @State private var selectedLecture: SelectedLecture?
     @Environment(\.timetableUIProvider) private var timetableUIProvider
     @Environment(\.themeViewModel) private var themeViewModel
     @Environment(\.errorAlertHandler) private var errorAlertHandler
@@ -72,8 +84,12 @@ public struct FriendsScene: View {
             .sheet(isPresented: $viewModel.isRequestSheetPresented) {
                 FriendRequestOptionSheet(friendsViewModel: viewModel)
             }
-            .sheet(item: $selectedLecture) { lecture in
-                timetableUIProvider.makeLectureDetailPreview(lecture: lecture, options: [.showDismissButton])
+            .sheet(item: $selectedLecture) { selected in
+                timetableUIProvider.makeLectureDetailPreview(
+                    lecture: selected.lecture,
+                    quarter: selected.quarter,
+                    options: [.showDismissButton]
+                )
             }
         }
     }
@@ -135,7 +151,8 @@ public struct FriendsScene: View {
         .environment(
             \.lectureTapAction,
             LectureTapAction(action: { lecture in
-                selectedLecture = lecture
+                guard let quarter = friendContent?.selectedQuarter else { return }
+                selectedLecture = SelectedLecture(lecture: lecture, quarter: quarter)
             })
         )
         .id(friendContent?.friend.id)

SNUTT/Modules/Feature/Timetable/Resources/Base.lproj/Localizable.strings

@@ -133,3 +133,8 @@
 "toast.action.view" = "보기";
 
 "edit.goToTimetable" = "해당 시간표로 이동";
+
+"navigation.error.lecture_not_found" = "시간표에서 삭제된 강의입니다.";
+"navigation.error.timetable_not_found" = "존재하지 않는 시간표입니다.";
+"navigation.error.bookmark_not_found" = "관심강좌에서 삭제된 강의입니다.";
+"navigation.error.unknown" = "올바르지 않은 정보입니다.";

SNUTT/Modules/Feature/Timetable/Resources/en.lproj/Localizable.strings

@@ -132,3 +132,8 @@
 "toast.action.view" = "View";
 
 "edit.goToTimetable" = "Go to Timetable";
+
+"navigation.error.lecture_not_found" = "Lecture has been deleted from timetable.";
+"navigation.error.timetable_not_found" = "Timetable does not exist.";
+"navigation.error.bookmark_not_found" = "Lecture has been deleted from bookmarks.";
+"navigation.error.unknown" = "Invalid information.";

SNUTT/Modules/Feature/Timetable/Sources/Presentation/LectureEditDetailViewModel.swift

@@ -32,7 +32,9 @@ public final class LectureEditDetailViewModel {
     @ObservationIgnored
     @Dependency(\.analyticsLogger) private var analyticsLogger
 
+    let displayMode: DisplayMode
     private let parentTimetable: Timetable?
+    private let quarter: Quarter
 
     var entryLecture: Lecture
     var editableLecture: Lecture
@@ -57,8 +59,19 @@ public final class LectureEditDetailViewModel {
         buildings.allSatisfy { $0.campus == .GWANAK }
     }
 
-    init(parentTimetable: Timetable?, entryLecture: Lecture) {
-        self.parentTimetable = parentTimetable
+    init(displayMode: DisplayMode, entryLecture: Lecture) {
+        self.displayMode = displayMode
+
+        // Extract timetable and quarter from displayMode
+        switch displayMode {
+        case .normal(let timetable), .create(let timetable):
+            self.parentTimetable = timetable
+            self.quarter = timetable.quarter
+        case .preview(_, let quarter):
+            self.parentTimetable = nil
+            self.quarter = quarter
+        }
+
         self.entryLecture = entryLecture
         editableLecture = entryLecture
         lectureID = entryLecture.id
@@ -206,12 +219,10 @@ public final class LectureEditDetailViewModel {
     }
 
     func fetchSyllabusURL() async -> URL? {
-        guard let parentTimetable,
-            !entryLecture.isCustom
-        else { return nil }
+        guard !entryLecture.isCustom else { return nil }
 
-        let year = parentTimetable.quarter.year
-        let semester = parentTimetable.quarter.semester.rawValue
+        let year = quarter.year
+        let semester = quarter.semester.rawValue
 
         do {
             let syllabus = try await courseBookRepository.fetchSyllabusURL(
@@ -226,6 +237,63 @@ public final class LectureEditDetailViewModel {
     }
 }
 
+extension LectureEditDetailViewModel {
+    public enum DisplayMode {
+        /// 내가 추가한 강의 상세
+        case normal(timetable: Timetable)
+        /// 새로운 강의 추가
+        case create(timetable: Timetable)
+        /// 내가 추가하지 않은 강의 상세 (검색 결과, 북마크 등)
+        case preview(LectureDetailPreviewOptions, quarter: Quarter)
+
+        public var isPreview: Bool {
+            if case .preview = self {
+                return true
+            }
+            return false
+        }
+
+        public var isCreate: Bool {
+            if case .create = self {
+                return true
+            }
+            return false
+        }
+
+        public var isNormal: Bool {
+            if case .normal = self {
+                return true
+            }
+            return false
+        }
+
+        public var previewOptions: LectureDetailPreviewOptions? {
+            if case let .preview(options, _) = self {
+                return options
+            }
+            return nil
+        }
+
+        public var timetable: Timetable? {
+            switch self {
+            case .normal(let timetable), .create(let timetable):
+                return timetable
+            case .preview:
+                return nil
+            }
+        }
+
+        public var quarter: Quarter {
+            switch self {
+            case .normal(let timetable), .create(let timetable):
+                return timetable.quarter
+            case .preview(_, let quarter):
+                return quarter
+            }
+        }
+    }
+}
+
 extension Lecture {
     var quotaDescription: String? {
         get {

SNUTT/Modules/Feature/Timetable/Sources/Presentation/LectureListViewModel.swift

@@ -70,12 +70,9 @@ extension LectureListViewModel {
     }
 
     func selectLecture(_ lecture: Lecture) {
+        guard let currentTimetable = timetableViewModel.currentTimetable else { return }
         timetableViewModel.paths.append(
-            .lectureDetail(
-                lecture,
-                parentTimetable: timetableViewModel.currentTimetable,
-                belongsToOtherTimetable: false
-            )
+            .lectureDetail(lecture, parentTimetable: currentTimetable)
         )
         analyticsLogger.logScreen(
             AnalyticsScreen.lectureDetail(.init(lectureID: lecture.referenceID, referrer: .lectureList))