feat: 설정 화면에 액션 연결 by clxxrlove · Pull Request #102 · YAPP-Github/Reed-iOS

clxxrlove · 2025-07-22T15:27:24Z

🔗 관련 이슈

Close [BOOK-158/feat] 설정 화면 Sheet, Dialog 연결 #96

📘 작업 유형

✨ Feature (기능 추가)
🐞 Bugfix (버그 수정)
🔧 Refactor (코드 리팩토링)
⚙️ Chore (환경 설정)
📝 Docs (문서 작성 및 수정)
✅ Test (기능 테스트)
🎨 style (코드 스타일 수정)

📙 작업 내역

BottomSheet 수정 - 1
- Leading, Centered 스타일에 상관 없이 주입되는 View의 위치가 다를 수 있음
- 디자인 시스템에서는 Centered에서 항상 upper지만, 실제로 사용되는 경우에서는 lower (Title 기준)
- -> 주입되는 View의 위치를 지정할 수 있도록 변경
BottomSheet 수정 - 2
- BottomSheetTitleView의 Label이 두 줄 이상 출력되지 않는 현상 수정
ButtonGroup 수정
- 버튼이 1개, 2개인 경우 Primary Action에 해당하는 버튼이 있음
- 특정 조건에 따라 이 버튼이 비활성화되어야 하는데, 기존에는 해당 기능이 없음
- PrimaryButton을 명시적으로 지정하여 비활성화 할 수 있는 기능 추가
BKCheckBoxLabel 추가
- CheckBox + Label의 컴포넌트가 여러 곳에서 사용되나, 스타일이 제각각이므로 사용에 유의
- 특정 경우(예시; 약관 동의의 iconOnly 버튼)에서는, 버튼의 사이즈보다 더 큰 영역에서 탭이 가능해야 함
- 위 기능도 구현
Coordinator에 기능 추가
- 로그아웃 등의 사유로 세션(토큰)이 만료되는 경우, AuthFlow로 돌아가도록 기능 추가
컴포넌트 디렉토리 정리

🧪 테스트 내역

브라우저/기기에서 동작 확인
엣지 케이스 테스트 완료
기존 기능 영향 없음

🎨 스크린샷 또는 시연 영상 (선택)

ScreenRecording_07-22-2025.23-39-55_1.MP4

💬 추가 설명 or 리뷰 포인트 (선택)

추가된 기능들 코드 잘 읽어주세요.
기능이 좀 모호하면 말씀주세요. 주석 추가해드릴게요.

Summary by CodeRabbit

신규 기능
- 체크박스와 라벨이 결합된 새로운 UI 컴포넌트 추가.
- 바텀시트에 컨텐츠 위치(상단/하단)를 지정하는 옵션 추가.
- 바텀시트에 동의 체크박스 및 버튼이 포함된 출금 시트 팩토리 메서드 도입.
- 버튼 그룹에서 주요 버튼 활성/비활성 상태 제어 기능 추가.
- 세션 만료 감지 및 알림 프로토콜 추가, 세션 만료 시 플로우 전환 처리 지원.
- 설정 화면에서 로그아웃 기능 및 로그아웃 상태 관리 추가.
개선 사항
- 바텀시트 타이틀/서브타이틀 레이아웃을 StackView로 통일하여 UI 일관성 및 유지보수성 향상.
- 체크박스 크기 및 레이아웃을 명확하게 지정하여 UI 안정성 개선.
- 설정 화면 컬렉션뷰 및 셀의 배경색 일관성 개선.
- 설정 화면에서 아이템 하이라이트(선택) 효과 활성화.
버그 수정
- 설정 셀의 하이라이트 상태에서 배경색이 올바르게 유지되도록 개선.
기타
- 내부 구조 리팩토링 및 불필요한 코드 제거.

(cherry picked from commit f565c76)

(cherry picked from commit d03a2e1)

(cherry picked from commit 4c91572)

(cherry picked from commit e484fdb)

(cherry picked from commit ee83b84)

(cherry picked from commit f2df487)

…Group (cherry picked from commit 1bb7bcb)

…ttom sheet (cherry picked from commit b8afd52)

(cherry picked from commit 04081f4)

…protocol (cherry picked from commit 506d9a4)

(cherry picked from commit 7ebe093)

(cherry picked from commit 0b99a11)

(cherry picked from commit 23ec72e)

(cherry picked from commit aa829c3)

…n logic (cherry picked from commit 7df90d6)

coderabbitai · 2025-07-22T15:27:31Z

"""

Walkthrough

이번 변경은 설정 화면에서 로그아웃 및 회원탈퇴 기능을 구현하고, 관련 다이얼로그 및 시트를 연결합니다. BottomSheet 컴포넌트의 suppliedContent 처리 방식이 enum 기반으로 구조화되었으며, 체크박스와 라벨을 결합한 신규 UI 컴포넌트가 추가되었습니다. Coordinator 계층에는 세션 만료 알림 및 처리 프로토콜이 도입되어 플로우 간 세션 만료 전파가 가능해졌습니다.

Changes

파일/경로 그룹	변경 요약
.../BKBottomSheetTestMenuViewController.swift	BottomSheet 프레젠테이션 메서드에서 suppliedContent → suppliedContentStyle(.upper/.lower)로 변경, 멀티라인 텍스트 적용
.../BKCheckBox.swift	intrinsicContentSize 오버라이드 제거, 이미지뷰에 고정 크기 제약 적용
.../BottomSheet/BKBottomSheetStyle.swift	UIKit import 제거
.../BottomSheet/BKBottomSheetTitleView.swift	Subtitle/CloseButton 설정 및 레이아웃 로직을 StackView 기반으로 리팩토링
.../BottomSheet/BKBottomSheetViewController.swift	suppliedContent → suppliedContentStyle(enum)로 구조 변경, 레이아웃 로직 개선, WithdrawalSheet 팩토리 메서드 추가
.../BottomSheet/SuppliedContentStyle.swift	SuppliedContentStyle enum 신설 (upper/lower)
.../Button/BKButtonGroup.swift	primaryButton 프로퍼티 및 상태 제어 메서드 추가, 초기화시 primaryButton 지정
.../Label/BKCheckBoxLabel.swift	체크박스+라벨 결합 신규 UIView(BKCheckBoxLabel) 추가
.../AppCoordinator.swift	mainFlowCoordinator 종료 시 startAuthFlow()로 복귀하도록 onFinish 핸들러 추가
.../Common/Coordinator/Coordinator.swift	firstAncestor(ofType:) 메서드 추가
.../Common/Coordinator/SessionExpiredHandling.swift	SessionExpirationHandling 프로토콜 신설
.../Common/Coordinator/SessionExpiredNotifying.swift	SessionExpirationNotifying 프로토콜 및 notifyParentSessionExpired() 메서드 추가
.../MainFlow/MainFlowCoordinator.swift	FinishNotifying, SessionExpirationHandling 프로토콜 채택 및 구현, onFinish 프로퍼티 추가
.../MainFlow/Setting/Coordinator/SettingCoordinator.swift	SessionExpirationNotifying 프로토콜 채택
.../MainFlow/Setting/View/SettingCell.swift	isHighlighted 시 백그라운드 컬러 지정 로직 추가
.../MainFlow/Setting/View/SettingView.swift	컬렉션뷰 및 섹션 배경색 지정, 아이템 하이라이트 활성화
.../MainFlow/Setting/View/SettingViewController.swift	로그아웃/탈퇴 버튼 이벤트 처리, 다이얼로그·시트 연결, 세션 만료시 coordinator 알림 추가
.../MainFlow/Setting/ViewModel/SettingViewModel.swift	로그아웃 상태/에러 관리, 액션/사이드이펙트/유스케이스 추가 및 리듀서/핸들러 확장

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant SettingView
    participant SettingViewController
    participant SettingViewModel
    participant Coordinator

    User->>SettingView: 로그아웃 버튼 탭
    SettingView->>SettingViewController: eventPublisher(.logoutTapped)
    SettingViewController->>SettingViewController: presentLogoutDialog()
    User->>SettingViewController: 다이얼로그에서 "로그아웃" 선택
    SettingViewController->>SettingViewModel: send(.logoutButtonTapped)
    SettingViewModel->>SettingViewModel: reduce(.logoutButtonTapped) → .logout side effect
    SettingViewModel->>SettingViewModel: handle(.logout) → logoutUseCase.execute()
    SettingViewModel->>SettingViewModel: onSuccess: send(.logoutSuccessed)
    SettingViewModel->>SettingViewModel: state.isLoggedOut = true
    SettingViewModel->>SettingViewController: state 업데이트
    SettingViewController->>Coordinator: 세션 만료 알림 (handleSessionExpired)
    Coordinator->>Coordinator: 플로우 종료 및 AuthFlow 진입

Estimated code review effort

4 (~90분)

Possibly related PRs

feat: 입력 관련 컴포넌트 추가, 디자인 시스템 일부 수정 #69: BottomSheet suppliedContentStyle enum 도입 및 체크박스 UI와의 관련성
fix: 설정 화면 터치 피드백 제거 #89: SettingView의 shouldHighlightItemAt 반환값 변경(아이템 하이라이트 활성화/비활성화)
feat: 바텀시트 컴포넌트 개발 #80: BottomSheetViewController 및 SuppliedContentStyle enum 신설 및 사용 관련

Suggested labels

✨ feat

Suggested reviewers

doyeonk429
"""

📜 Recent review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eb74253 and bbce898.

📒 Files selected for processing (2)

src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift (8 hunks)
src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingViewController.swift (4 hunks)

🚧 Files skipped from review as they are similar to previous changes (2)

src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingViewController.swift
src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

GitHub Check: build

✨ Finishing Touches

📝 Generate Docstrings

🧪 Generate unit tests

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch BOOK-158-feature/#96

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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Explain this complex logic.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai explain this code block.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and explain its main purpose.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR.
@coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
@coderabbitai generate unit tests to generate unit tests for this PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary or @coderabbitai 요약 to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

coderabbitai

Actionable comments posted: 2

🔭 Outside diff range comments (2)

src/Projects/BKDesign/Sources/Components/Button/BKButtonGroup.swift (2)
81-86: primaryButton 참조 정리 필요

버튼이 제거될 때 primaryButton 참조가 남아있을 수 있습니다. 제거되는 버튼이 primaryButton인 경우 nil로 설정해야 합니다.
 public func removeButton(_ button: BKButton) {
     if let index = buttons.firstIndex(of: button) {
         buttons.remove(at: index)
+        if button === primaryButton {
+            primaryButton = nil
+        }
         updateButtons()
     }
 }
92-95: 모든 버튼 제거 시 primaryButton 초기화 필요

모든 버튼이 제거될 때 primaryButton 참조도 함께 정리되어야 합니다.
 public func removeAllButtons() {
     buttons.removeAll()
+    primaryButton = nil
     updateButtons()
 }

🧹 Nitpick comments (2)

src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingViewController.swift (1)
114-117: 회원 탈퇴 기능 구현 필요

회원 탈퇴 확인 시 실제 액션이 주석 처리되어 있습니다. 향후 구현이 필요함을 명시하는 것이 좋습니다.
 confirmAction: { [weak self] in
     Log.debug("[WithdrawalSheet] confirmed", logger: AppLogger.ui)
+    // TODO: 회원 탈퇴 기능 구현 필요
     // self?.viewModel.send()
 }
src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift (1)
273-284: 비율 계산 로직에서 타입 캐스팅을 개선해보세요.

UIImageView로의 타입 캐스팅이 하드코딩되어 있습니다. 향후 다른 타입의 뷰에서도 비율 계산이 필요할 수 있으니, 프로토콜 기반 접근을 고려해보세요.

다음과 같은 프로토콜 기반 접근을 제안합니다:
protocol AspectRatioProvider {
    var aspectRatio: CGFloat? { get }
}

extension UIImageView: AspectRatioProvider {
    var aspectRatio: CGFloat? {
        guard let image = image else { return nil }
        return image.size.height / image.size.width
    }
}
이후 calculateRatioIfNeeded에서:
if let aspectRatioProvider = view as? AspectRatioProvider {
    contentAspectRatio = aspectRatioProvider.aspectRatio
}

📜 Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ebf5be1 and eb74253.

📒 Files selected for processing (18)

src/Projects/BKDesign/PreviewApp/Sources/View/BKBottomSheetTestMenuViewController.swift (2 hunks)
src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (1 hunks)
src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetStyle.swift (0 hunks)
src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetTitleView.swift (1 hunks)
src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift (9 hunks)
src/Projects/BKDesign/Sources/Components/BottomSheet/SuppliedContentStyle.swift (1 hunks)
src/Projects/BKDesign/Sources/Components/Button/BKButtonGroup.swift (4 hunks)
src/Projects/BKDesign/Sources/Components/Label/BKCheckBoxLabel.swift (1 hunks)
src/Projects/BKPresentation/Sources/AppCoordinator.swift (1 hunks)
src/Projects/BKPresentation/Sources/Common/Coordinator/Coordinator.swift (1 hunks)
src/Projects/BKPresentation/Sources/Common/Coordinator/SessionExpiredHandling.swift (1 hunks)
src/Projects/BKPresentation/Sources/Common/Coordinator/SessionExpiredNotifying.swift (1 hunks)
src/Projects/BKPresentation/Sources/MainFlow/MainFlowCoordinator.swift (2 hunks)
src/Projects/BKPresentation/Sources/MainFlow/Setting/Coordinator/SettingCoordinator.swift (1 hunks)
src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingCell.swift (2 hunks)
src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingView.swift (3 hunks)
src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingViewController.swift (4 hunks)
src/Projects/BKPresentation/Sources/MainFlow/Setting/ViewModel/SettingViewModel.swift (4 hunks)

🧠 Learnings (13)

📓 Common learnings

Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#79
File: src/Projects/BKPresentation/Sources/MainFlow/Setting/ViewModel/SettingViewModel.swift:34-35
Timestamp: 2025-07-11T12:13:49.406Z
Learning: clxxrlove prefers to comment out incomplete features in SettingViewModel (like logout and withdrawal functionality) rather than removing them entirely, keeping the architecture ready for future implementation. This approach allows for easier connection of functionality when ready to implement.

Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.

Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#56
File: src/Projects/BKPresentation/Sources/Constant/PresentationKeys.swift:3-5
Timestamp: 2025-07-07T13:53:13.006Z
Learning: clxxrlove prefers using enum over struct for constants namespace due to better runtime performance, memory layout, and compiler optimizations. Enum prevents instantiation at compile time without needing private init().

Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#94
File: src/Projects/BKData/Sources/Repository/DefaultBookRepository.swift:15-40
Timestamp: 2025-07-21T05:16:18.481Z
Learning: clxxrlove는 Reed-iOS 프로젝트에서 기능 구현 시 우선순위를 명확하게 설정한다. 기본 기능 구현을 먼저 완료하고, 에러 처리나 안정성 개선은 이후 단계에서 진행하는 것을 선호한다.

src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingView.swift (1)

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.

src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingCell.swift (1)

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.

src/Projects/BKPresentation/Sources/MainFlow/Setting/ViewModel/SettingViewModel.swift (3)

Learnt from: clxxrlove
PR: #79
File: src/Projects/BKPresentation/Sources/MainFlow/Setting/ViewModel/SettingViewModel.swift:34-35
Timestamp: 2025-07-11T12:13:49.406Z
Learning: clxxrlove prefers to comment out incomplete features in SettingViewModel (like logout and withdrawal functionality) rather than removing them entirely, keeping the architecture ready for future implementation. This approach allows for easier connection of functionality when ready to implement.

Learnt from: clxxrlove
PR: #56
File: src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift:109-111
Timestamp: 2025-07-07T13:47:32.784Z
Learning: The LoginViewModel's authenticateWithToken case in src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift is test code and not important for implementation according to the user.

Learnt from: clxxrlove
PR: #79
File: src/Projects/BKPresentation/Sources/MainFlow/Setting/ViewModel/SettingViewModel.swift:78-85
Timestamp: 2025-07-11T12:12:48.911Z
Learning: AppVersionUseCase.execute() returns AnyPublisher<String, Never> where the Failure type is Never, making error handling unnecessary. The app version is retrieved from the main bundle which is always available, and the implementation already handles missing version by defaulting to "-".

src/Projects/BKDesign/Sources/Components/BottomSheet/SuppliedContentStyle.swift (2)

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.

Learnt from: clxxrlove
PR: #80
File: src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift:148-148
Timestamp: 2025-07-14T05:23:04.937Z
Learning: Reed-iOS 프로젝트의 BKBottomSheetViewController에서 버튼 영역 높이(84)는 디자인 시스템에 기반한 의도적인 고정값입니다. clxxrlove에 따르면 작은 기기에서도 동일한 크기가 유지되어야 하며, 동적 계산보다는 상수화만 필요합니다.

src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (2)

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.

Learnt from: clxxrlove
PR: #80
File: src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift:148-148
Timestamp: 2025-07-14T05:23:04.937Z
Learning: Reed-iOS 프로젝트의 BKBottomSheetViewController에서 버튼 영역 높이(84)는 디자인 시스템에 기반한 의도적인 고정값입니다. clxxrlove에 따르면 작은 기기에서도 동일한 크기가 유지되어야 하며, 동적 계산보다는 상수화만 필요합니다.

src/Projects/BKPresentation/Sources/MainFlow/MainFlowCoordinator.swift (1)

Learnt from: clxxrlove
PR: #56
File: src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift:109-111
Timestamp: 2025-07-07T13:47:32.784Z
Learning: The LoginViewModel's authenticateWithToken case in src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift is test code and not important for implementation according to the user.

src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingViewController.swift (5)

Learnt from: clxxrlove
PR: #56
File: src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift:109-111
Timestamp: 2025-07-07T13:47:32.784Z
Learning: The LoginViewModel's authenticateWithToken case in src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift is test code and not important for implementation according to the user.

Learnt from: clxxrlove
PR: #79
File: src/Projects/BKPresentation/Sources/MainFlow/Setting/ViewModel/SettingViewModel.swift:34-35
Timestamp: 2025-07-11T12:13:49.406Z
Learning: clxxrlove prefers to comment out incomplete features in SettingViewModel (like logout and withdrawal functionality) rather than removing them entirely, keeping the architecture ready for future implementation. This approach allows for easier connection of functionality when ready to implement.

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.

src/Projects/BKDesign/PreviewApp/Sources/View/BKBottomSheetTestMenuViewController.swift (2)

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.

Learnt from: clxxrlove
PR: #80
File: src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift:148-148
Timestamp: 2025-07-14T05:23:04.937Z
Learning: Reed-iOS 프로젝트의 BKBottomSheetViewController에서 버튼 영역 높이(84)는 디자인 시스템에 기반한 의도적인 고정값입니다. clxxrlove에 따르면 작은 기기에서도 동일한 크기가 유지되어야 하며, 동적 계산보다는 상수화만 필요합니다.

src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift (3)

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.

Learnt from: clxxrlove
PR: #80
File: src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift:148-148
Timestamp: 2025-07-14T05:23:04.937Z
Learning: Reed-iOS 프로젝트의 BKBottomSheetViewController에서 버튼 영역 높이(84)는 디자인 시스템에 기반한 의도적인 고정값입니다. clxxrlove에 따르면 작은 기기에서도 동일한 크기가 유지되어야 하며, 동적 계산보다는 상수화만 필요합니다.

Learnt from: doyeonk429
PR: #68
File: src/Projects/BKDesign/Sources/Components/Button/BKButtonGroup.swift:86-92
Timestamp: 2025-07-10T08:21:16.126Z
Learning: BKButtonGroup의 horizontal 레이아웃에서 stackView.distribution을 .fillProportionally로 설정한 것은 의도된 디자인입니다. doyeonk429는 버튼들이 콘텐츠 길이에 따라 다른 너비를 가지는 것을 선호합니다.

src/Projects/BKDesign/Sources/Components/Button/BKButtonGroup.swift (1)

Learnt from: doyeonk429
PR: #68
File: src/Projects/BKDesign/Sources/Components/Button/BKButtonGroup.swift:86-92
Timestamp: 2025-07-10T08:21:16.126Z
Learning: BKButtonGroup의 horizontal 레이아웃에서 stackView.distribution을 .fillProportionally로 설정한 것은 의도된 디자인입니다. doyeonk429는 버튼들이 콘텐츠 길이에 따라 다른 너비를 가지는 것을 선호합니다.

src/Projects/BKPresentation/Sources/AppCoordinator.swift (1)

Learnt from: clxxrlove
PR: #56
File: src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift:109-111
Timestamp: 2025-07-07T13:47:32.784Z
Learning: The LoginViewModel's authenticateWithToken case in src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift is test code and not important for implementation according to the user.

src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetTitleView.swift (2)

Learnt from: doyeonk429
PR: #63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.

Learnt from: clxxrlove
PR: #80
File: src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift:148-148
Timestamp: 2025-07-14T05:23:04.937Z
Learning: Reed-iOS 프로젝트의 BKBottomSheetViewController에서 버튼 영역 높이(84)는 디자인 시스템에 기반한 의도적인 고정값입니다. clxxrlove에 따르면 작은 기기에서도 동일한 크기가 유지되어야 하며, 동적 계산보다는 상수화만 필요합니다.

🧬 Code Graph Analysis (8)

src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingView.swift (1)

src/Projects/BKDesign/Sources/Extensions/UIColor+.swift (1)

bkBaseColor (91-102)

src/Projects/BKPresentation/Sources/Common/Coordinator/SessionExpiredNotifying.swift (2)

src/Projects/BKPresentation/Sources/Common/Coordinator/Coordinator.swift (1)

firstAncestor (62-71)

src/Projects/BKPresentation/Sources/MainFlow/MainFlowCoordinator.swift (1)

handleSessionExpired (28-31)

src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingCell.swift (1)

src/Projects/BKDesign/Sources/Extensions/UIColor+.swift (1)

bkBaseColor (91-102)

src/Projects/BKDesign/Sources/Components/Label/BKCheckBoxLabel.swift (1)

src/Projects/BKDesign/Sources/Extensions/UIColor+.swift (1)

bkContentColor (52-63)

src/Projects/BKPresentation/Sources/MainFlow/Setting/ViewModel/SettingViewModel.swift (1)

src/Projects/BKData/Sources/Repository/DefaultAuthRepository.swift (1)

logout (54-69)

src/Projects/BKPresentation/Sources/MainFlow/MainFlowCoordinator.swift (1)

src/Projects/BKPresentation/Sources/Common/Coordinator/Coordinator.swift (1)

didFinish (18-25)

src/Projects/BKPresentation/Sources/Common/Coordinator/SessionExpiredHandling.swift (1)

src/Projects/BKPresentation/Sources/MainFlow/MainFlowCoordinator.swift (1)

handleSessionExpired (28-31)

src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetViewController.swift (1)

src/Projects/BKDesign/Sources/Components/Button/BKButtonGroup.swift (2)

twoButtonGroup (140-160)

setPrimaryButtonState (88-90)

🪛 SwiftLint (0.57.0)

src/Projects/BKPresentation/Sources/MainFlow/Setting/View/SettingViewController.swift

[Warning] 75-75: Unused parameter in a closure should be replaced with _

(unused_closure_parameter)

💤 Files with no reviewable changes (1)

src/Projects/BKDesign/Sources/Components/BottomSheet/BKBottomSheetStyle.swift

🧰 Additional context used

🧠 Learnings (13)

📓 Common learnings

Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#79
File: src/Projects/BKPresentation/Sources/MainFlow/Setting/ViewModel/SettingViewModel.swift:34-35
Timestamp: 2025-07-11T12:13:49.406Z
Learning: clxxrlove prefers to comment out incomplete features in SettingViewModel (like logout and withdrawal functionality) rather than removing them entirely, keeping the architecture ready for future implementation. This approach allows for easier connection of functionality when ready to implement.

Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.

Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#56
File: src/Projects/BKPresentation/Sources/Constant/PresentationKeys.swift:3-5
Timestamp: 2025-07-07T13:53:13.006Z
Learning: clxxrlove prefers using enum over struct for constants namespace due to better runtime performance, memory layout, and compiler optimizations. Enum prevents instantiation at compile time without needing private init().

Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#94
File: src/Projects/BKData/Sources/Repository/DefaultBookRepository.swift:15-40
Timestamp: 2025-07-21T05:16:18.481Z
Learning: clxxrlove는 Reed-iOS 프로젝트에서 기능 구현 시 우선순위를 명확하게 설정한다. 기본 기능 구현을 먼저 완료하고, 에러 처리나 안정성 개선은 이후 단계에서 진행하는 것을 선호한다.