src/Projects/BKDesign/Sources/Components/BKLoading/BKLoadingIndicator.swift (2)

3-4: 불필요한 import(SnapKit) 제거 제안

이 파일에서 SnapKit을 사용하지 않습니다. 불필요한 의존성 노출을 줄이기 위해 import를 제거하는 편이 깔끔합니다.

-import SnapKit
 import UIKit

---

29-30: 지연 값 방어 코드(클램프) 추가 제안

음수 지연이 들어오면 즉시 실행되지만, 의도 명확화를 위해 0으로 클램프하는 편이 안전합니다.

-            DispatchQueue.main.asyncAfter(deadline: .now() + delay, execute: workItem)
+            DispatchQueue.main.asyncAfter(deadline: .now() + max(0, delay), execute: workItem)

추가로, 새 show(delay:)가 더 짧은 지연으로 호출되면 기존 예약을 취소하고 재예약하는 정책(옵션)을 고려해 보세요.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/NoteCompletionViewController.swift (2)

79-81: 연속 에러 이벤트 시 중복 present 방지 가드 제안

빠르게 연속 에러가 발행되면 이미 모달이 떠 있는 상태에서 또 present 시도할 수 있습니다. 간단한 가드로 방지하는 편이 안전합니다.

다음과 같이 변경 제안:

-            .sink { [weak self] _ in
-                self?.presentErrorDialog()
-            }
+            .sink { [weak self] _ in
+                guard let self = self, self.presentedViewController == nil else { return }
+                self.presentErrorDialog()
+            }

---

167-181: BKDialog 카피/버튼 라벨 UX 보정

현재 버튼 라벨이 본문 메시지 역할을 하고 있어 UX가 어색합니다. 타이틀은 메시지, 버튼은 “확인”으로 정리하는 편이 자연스럽습니다. 중첩 [weak self]도 한 번으로 단순화 가능합니다.

-        let dialog = BKDialog(
-            title: "오류",
+        let dialog = BKDialog(
+            title: "기록을 불러올 수 없습니다.",
             config: .init(
-                leftButtonTitle: "기록을 불러올 수 없습니다.",
-                leftButtonAction: { [weak self] in
-                    self?.dismiss(animated: true) { [weak self] in
-                        self?.viewModel.send(.errorHandled)
-                    }
-                }
+                leftButtonTitle: "확인",
+                leftButtonAction: { [weak self] in
+                    guard let self else { return }
+                    self.dismiss(animated: true) {
+                        self.viewModel.send(.errorHandled)
+                    }
+                }
             )
         )

참고: 과거 러닝에 따라 에러 상세 카피는 추후로 미루셔도 OK입니다.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#134
File: src/Projects/BKPresentation/Sources/MainFlow/Note/ViewModel/NoteViewModel.swift:66-74
Timestamp: 2025-08-01T20:35:39.806Z
Learning: clxxrlove는 NoteViewModel에서 UI 에러 처리를 추후 작업으로 계획하며, 현재 단계에서는 기본 기능 구현에 집중하는 것을 선호한다고 명시했다.

Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#120
File: src/Projects/BKPresentation/Sources/AppCoordinator.swift:31-41
Timestamp: 2025-07-30T11:32:20.533Z
Learning: Reed-iOS 프로젝트의 OnboardingCheckUseCase.execute() 메서드는 AnyPublisher<Bool, Never>를 반환하므로 실패할 수 없도록 설계되었다. Never 타입은 에러가 발생하지 않음을 보장하므로 에러 처리 로직이 불필요하다.