[Feature] Tuist 각 레이어 모듈화 #136
Conversation
## Walkthrough
이 변경사항은 RunLog 프로젝트를 모듈화하기 위해 기존 단일 프로젝트 구조를 제거하고, Tuist를 활용한 App, DesignSystem, Domain, Data, Inject, Presentation, Util 등 여러 모듈로 분리된 구조로 재구성합니다. 각 모듈별로 Project.swift, Info.plist, 소스코드, 리소스, 의존성 설정이 도입되었으며, 기존 코드의 접근 제어자(public 등) 및 유틸리티 확장자들도 외부 접근이 가능하도록 변경되었습니다.
## Changes
| 파일/경로 그룹 | 변경 요약 |
|---|---|
| **기존 RunLog 프로젝트 구성 및 엔티티/설정 파일 삭제**<br>`RunLog/Project.swift`, `RunLog/RunLog/Info.plist`, `RunLog/RunLog/Sources/Data/CoreDataContainer.swift`, `RunLog/RunLog/Sources/Domain/Entity/AppConfig.swift`, `RunLog/RunLog/Sources/Domain/Entity/DayLog.swift`, `RunLog/RunLog/Sources/Domain/Entity/Road.swift` | 기존 단일 프로젝트 구성 파일, Info.plist, CoreDataContainer, Entity 구조체 등 삭제 |
| **모듈별 Tuist 프로젝트 구성 파일 신규 생성**<br>`RunLog/Projects/App/Project.swift`, `RunLog/Projects/Data/Project.swift`, `RunLog/Projects/DesignSystem/Project.swift`, `RunLog/Projects/Domain/Project.swift`, `RunLog/Projects/Inject/Project.swift`, `RunLog/Projects/Presentation/Project.swift`, `RunLog/Projects/Util/Project.swift` | 각 모듈별 Tuist 기반 Project.swift 신규 생성 및 의존성 선언 |
| **Info.plist 신규 생성 및 수정**<br>`RunLog/Projects/App/InfoPlists/Info.plist`, `RunLog/Projects/DesignSystem/InfoPlists/Info.plist` | Info.plist 신규 생성, 앱 이름 설정, 폰트 및 런치스크린 설정 변경 |
| **CoreDataContainer 클래스 신규 생성 및 이동**<br>`RunLog/Projects/Data/Sources/DataContainer/CoreDataContainer.swift` | CoreDataContainer 클래스 신규 생성 (Data 모듈로 이동) |
| **Domain 모듈 내 Entity 구조체 및 Error enum 신규 생성 및 public 선언**<br>`RunLog/Projects/Domain/Sources/Entity/*.swift`, `RunLog/Projects/Domain/Sources/Error/*.swift` | AppConfig, DayLog, Section, Point 등 Entity 구조체와 AppConfigError, DataError 등 에러 enum을 public으로 변경 및 신규 생성 |
| **Domain 모듈 내 Repository, UseCase 프로토콜 public 변경**<br>`RunLog/Projects/Domain/Sources/Repository/*.swift`, `RunLog/Projects/Domain/Sources/UseCase/*.swift` | Repository, UseCase 프로토콜을 public으로 변경 |
| **Domain 모듈 내 UseCaseImpl 클래스 및 메서드 public 변경, import 및 에러타입 변경**<br>`RunLog/Projects/Domain/Sources/UseCaseImpl/*.swift` | UseCaseImpl 클래스 및 메서드 public으로 변경, import 추가, 에러타입 변경(CoreDataError → DataError) |
| **Data 모듈 내 클래스, 싱글턴, 메서드 public 변경 및 import 추가, 에러타입 변경**<br>`RunLog/Projects/Data/Sources/Impls/*.swift`, `RunLog/Projects/Data/Sources/Managers/*.swift`, `RunLog/Projects/Data/Sources/Network/OpenWeather/OpenWeatherService.swift` | Data 계층 클래스, 싱글턴, 메서드 public으로 변경, import 추가, 에러타입 변경 |
| **Inject 모듈 내 DIContainer 및 Dependency 프로퍼티 public 변경**<br>`RunLog/Projects/Inject/Sources/DIContainer.swift` | DIContainer 및 Dependency 프로퍼티 public으로 변경 |
| **DesignSystem 모듈 내 UI 컴포넌트 및 유틸 public 변경, 이미지/폰트 로딩 로직 개선, import 추가**<br>`RunLog/Projects/DesignSystem/Sources/*.swift` | RLIcon, RLLabel, FontSystem, IconSystem 등 public으로 변경, 이미지/폰트 로딩 로직 개선, import 추가 |
| **Util 모듈 내 Constants, Extension, Helper 등 public 변경 및 시그니처 명확화**<br>`RunLog/Projects/Util/Sources/Constant/Constants.swift`, `RunLog/Projects/Util/Sources/Extension/*.swift`, `RunLog/Projects/Util/Sources/Helpers/DynamicSize.swift`, `RunLog/Projects/Util/Sources/Extension/UINavigationController+.swift` | Constants, Road, MotivationMessage 등 public으로 변경, icon → iconName(String)으로 변경, UIKit 관련 확장자 public 적용, 시그니처 파라미터화, UINavigationController 확장 메서드 추가 |
| **Presentation 모듈 내 View, ViewModel, Controller import 추가, public 변경, 네비게이션/탭바/버튼 appearance 커스텀화, 이미지 로딩 방식 개선**<br>`RunLog/Projects/Presentation/Sources/**/*` | import/의존성 추가, public 적용, 네비게이션/탭바/버튼 appearance 커스텀화, 이미지 로딩 방식 개선 |
| **기타** | 각 모듈별 의존성 및 리소스 경로 정비, 접근 제어자 일관성 부여, UIKit/Combine 등 import 정비 |
## Sequence Diagram(s)
```mermaid
sequenceDiagram
participant App (RunLog)
participant Domain
participant Data
participant DesignSystem
participant Presentation
participant Util
participant Inject
App->>Inject: registerDependencies()
App->>Util: UIFont.registerFonts()
App->>Presentation: 화면 표시 및 뷰모델 생성
Presentation->>Domain: UseCase 호출
Domain->>Data: Repository 호출
Data->>CoreDataContainer: 데이터 저장/조회
Presentation->>DesignSystem: UI 컴포넌트/이미지/폰트 사용
Presentation->>Util: 확장자/헬퍼 사용Estimated code review effort🎯 4 (Complex) | ⏱️ ~90 minutes Assessment against linked issues
Possibly related PRs
Suggested reviewers
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 9
🔭 Outside diff range comments (1)
RunLog/Projects/Presentation/Sources/Setting/ViewModel/CalUnitViewModel.swift (1)
90-90: Double.formattedString 확장 메서드에 대한 모듈 import 누락
Double.formattedString은RunLog/Projects/Util/Sources/Extension/Double+.swift에 정의된 Util(또는 RLUtil) 모듈의 확장 기능입니다. Presentation 모듈에서 해당 기능을 사용하려면 아래와 같이 import를 추가해주세요.
- 파일:
RunLog/Projects/Presentation/Sources/Setting/ViewModel/CalUnitViewModel.swift- 추가할 내용 (파일 최상단에 삽입)
import Util
🧹 Nitpick comments (15)
RunLog/Projects/Presentation/Sources/DetailLog/View/DetailLogView.swift (1)
16-16: 클래스 선언부 들여쓰기 변경클래스 선언부에 2칸 들여쓰기가 추가되었습니다. 이는 단순한 포맷팅 변경으로 보이며 기능적 영향은 없습니다.
RunLog/Projects/App/Sources/AppDelegate.swift (1)
38-63: DI 등록 로직이 동기 블로킹 코드로 길어짐
registerDependencies()내부에서 객체 생성·레지스터만 수행하지만, 향후 비동기 초기화(예: 네트워크 fetch)가 추가될 경우didFinishLaunching지연이 커질 수 있습니다.
별도StartupService로 분리하여 비동기 처리하도록 설계하면 AppDelegate 책임이 줄어듭니다.RunLog/Projects/Domain/Sources/UseCase/AppConfigUseCase.swift (1)
2-4: 파일명 오타 –Usecase→UseCaseSwift 파일명과 타입명을 일관성 있게 유지하면 탐색성이 좋아집니다. 필요 시 아래와 같이 리네임을 권장합니다.
-// AppConfigUsecase.swift +// AppConfigUseCase.swiftRunLog/Projects/Domain/Sources/Repository/DayLogRepository.swift (1)
10-10: 프로토콜의 public 노출이 적절하며, 문서화 업데이트가 필요합니다.
DayLogRepository프로토콜을public으로 변경하여 의존성 주입과 모듈 간 사용을 가능하게 한 것이 올바른 접근입니다.다만 주석에서
CoreDataError를 언급하고 있는데(13, 19, 25, 30, 35행), 현재 구현에서는DataError를 사용하는 것으로 보입니다. 문서를 최신 에러 타입에 맞게 업데이트하는 것을 고려해보세요.RunLog/Projects/Presentation/Sources/Setting/ViewController/ChangeNicknameViewController.swift (2)
72-98: 네비게이션 바 설정이 더 명시적이고 일관성 있게 개선되었습니다.기존의 일반적인
setupAppearance()호출 대신 구체적인 매개변수를 전달하여 네비게이션 바 외관을 설정하는 방식으로 변경되었습니다. 버튼 생성과 Combine 기반 이벤트 처리도 적절합니다.다만 버튼 생성 부분에서 약간의 개선 여지가 있습니다:
버튼 설정을 더 간결하게 개선할 수 있습니다:
- let navigationButton = UIButton(type: .system) - let buttonTitle = "완료" - navigationButton.setTitle(buttonTitle, for: .normal) - navigationButton.titleLabel?.attributedText = .RLAttributedString( - text: buttonTitle, - font: .Label1, - color: .LightGreen, - align: .center - ) + let navigationButton = UIButton(type: .system).then { + let buttonTitle = "완료" + $0.setTitle(buttonTitle, for: .normal) + $0.titleLabel?.attributedText = .RLAttributedString( + text: buttonTitle, + font: .Label1, + color: .LightGreen, + align: .center + ) + }
72-98: 네비게이션 바 설정 방식 개선명시적인 appearance 설정과 Combine 기반 이벤트 처리로 변경된 것은 좋습니다. 하지만 이런 패턴이 여러 ViewController에서 반복될 가능성이 높아 보입니다.
공통 네비게이션 바 설정을 위한 헬퍼 메서드를 고려해보세요:
extension UIViewController { func setupStandardNavigationBar( title: String, rightButtonTitle: String, rightButtonAction: @escaping () -> Void ) { // 공통 설정 로직 } }RunLog/Projects/Util/Sources/Helpers/DynamicSize.swift (1)
15-15: SwiftLint 경고: 주석 공백 문제를 수정해주세요.주석에서 슬래시 다음에 최소 하나의 공백이 필요합니다.
-///- `SceneDelegate`에서 `setScreenSize(_:)`를 호출하여 초기 화면 크기를 설정해야 함 +/// - `SceneDelegate`에서 `setScreenSize(_:)`를 호출하여 초기 화면 크기를 설정해야 함RunLog/Projects/Domain/Sources/UseCaseImpl/DayLogUseCaseImpl.swift (1)
173-194: 스트릭 업데이트 로직 리팩토링 제안현재
updateStreakIfNeeded메서드의 로직이 복잡하고 이해하기 어렵습니다. 더 명확한 구조로 개선할 수 있습니다.public func updateStreakIfNeeded() async throws { let today = Date().toYearMonthDay - - // case: 오늘 운동을 했는지 안했는지 모르겠는데 마이페이지로 들어온 상황 - // streak = 6 - // 오늘 운동했니? + + // 오늘 운동 기록이 있는지 확인 do { - // yes -> 업데이트 X try await dayLogRepository.readDayLog(date: today) + // 오늘 운동했으면 업데이트 불필요 + return } catch DataError.modelNotFound { - // no -> 어제 운동했니? - let hasYesterdayDayLog = try await hasYesterdayDayLog() - // yes -> 업데이트 X - // no -> streak 0으로 초기화 - if !hasYesterdayDayLog { - var appconfig = try await appConfigRepository.readAppConfig() - appconfig.streakDays = 0 - try await appConfigRepository.updateAppConfig(appconfig) - } + // 오늘 운동 안했으면 어제 운동 여부 확인 + await resetStreakIfNeeded() } - } + +private func resetStreakIfNeeded() async throws { + let hasYesterdayDayLog = try await hasYesterdayDayLog() + + // 어제도 운동 안했으면 스트릭 리셋 + if !hasYesterdayDayLog { + var appconfig = try await appConfigRepository.readAppConfig() + appconfig.streakDays = 0 + try await appConfigRepository.updateAppConfig(appconfig) + } +}RunLog/Projects/Util/Sources/Extension/String+.swift (1)
36-40: 주석 처리된 코드 제거 필요파일 끝의 주석 처리된 코드는 불필요하며 SwiftLint 경고를 발생시킵니다. 이전 기본값들을 보여주는 참고용이지만 코드베이스를 깔끔하게 유지하기 위해 제거해야 합니다.
- - -//highlightText: String, -//baseFont: UIFont = .RLHeadline2, -//baseColor: UIColor = .Gray000, -//highlightFont: UIFont = .RLHeadline2, -//highlightColor: UIColor = .LightGreenRunLog/Projects/Domain/Sources/Entity/DayLog.swift (1)
52-57: 주석 처리된 확장 메서드를 정리하세요.주석 처리된 코드는 혼란을 야기할 수 있습니다. 필요하다면 구현을 완료하거나, 불필요하다면 제거하는 것이 좋습니다.
-//extension DayLog { -// /// DayLog의 각 section을 RecordDetail 배열로 변환 -// func toRecordDetails() -> [RecordDetail] { -// return self.sections.map { RecordDetail(from: $0) } -// } -//}또한 SwiftLint 경고를 해결하기 위해 주석에 공백을 추가하세요:
-//extension DayLog { -// /// DayLog의 각 section을 RecordDetail 배열로 변환 -// func toRecordDetails() -> [RecordDetail] { -// return self.sections.map { RecordDetail(from: $0) } -// } -//} +// extension DayLog { +// /// DayLog의 각 section을 RecordDetail 배열로 변환 +// func toRecordDetails() -> [RecordDetail] { +// return self.sections.map { RecordDetail(from: $0) } +// } +// }RunLog/Projects/DesignSystem/Sources/FontSystem.swift (1)
106-106: TODO 주석 해결 필요enum 이름이 이미
Pretendard로 변경되었으므로 TODO 주석을 제거해야 합니다.- // TODO: Pretendard로 네이밍 변경RunLog/Projects/Util/Sources/Extension/UIViewController+.swift (1)
25-25: 주석 스타일 수정 필요SwiftLint 규칙에 따라 주석 슬래시 뒤에 공백을 추가해야 합니다.
- color: .green, //.LightGreen + color: .green, // .LightGreenRunLog/Projects/Domain/Sources/Entity/AppConfig.swift (1)
10-30: 도메인 엔티티 설계 검토AppConfig 구조체가 잘 정의되어 있습니다. 모든 속성이 public으로 설정되어 모듈 간 접근이 가능합니다.
데이터 무결성을 위해 속성을 불변으로 만드는 것을 고려해보세요:
public struct AppConfig: Equatable { public let nickname: String public let totalDistance: Double public let streakDays: Int public let totalDays: Int public let unitDistance: Double // 업데이트가 필요한 경우 새 인스턴스를 반환하는 메서드 제공 public func updating(nickname: String? = nil, totalDistance: Double? = nil, // ... 다른 속성들 ) -> AppConfig { return AppConfig( nickname: nickname ?? self.nickname, totalDistance: totalDistance ?? self.totalDistance, // ... ) } }RunLog/Projects/Domain/Sources/Entity/Section.swift (1)
10-32: 도메인 엔티티 구조 개선 제안Section과 Point 구조체가 잘 정의되어 있습니다. 특히 지리적 좌표를 나타내는 Point의 경우 불변성을 고려해보세요.
Point의 좌표 데이터는 생성 후 변경되지 않는 것이 일반적이므로 불변으로 만드는 것을 권장합니다:
public struct Point: Equatable { public let latitude: Double public let longitude: Double public let timestamp: Date public init(latitude: Double, longitude: Double, timestamp: Date) { self.latitude = latitude self.longitude = longitude self.timestamp = timestamp } }Section의 경우 필요에 따라 distance와 steps는 let으로, route는 var로 유지할 수 있습니다.
RunLog/Projects/Domain/Sources/Error/DataError.swift (1)
20-29: 다국어화 일관성 고려에러 메시지가 영어로 작성되어 있는데, 프로젝트의 다른 부분과 일관성을 위해 한국어 지원을 고려해보세요.
case .fetchError: return "데이터 조회 오류" case .deleteError: return "데이터 삭제 오류" case .modelNotFound: return "모델을 찾을 수 없음" case .modelAlreadyExist: return "모델이 이미 존재함" case .conversionError: return "데이터 변환 오류"
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (54)
RunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/1024.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/114.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/120.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/180.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/29.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/40.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/57.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/58.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/60.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/80.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/AppIcon.appiconset/87.pngis excluded by!**/*.pngRunLog/Projects/App/Resources/Assets.xcassets/Splash/splashImage.imageset/splashImage.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Fonts/NanumMyeongjo-Regular.ttfis excluded by!**/*.ttfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-Black.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-Bold.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-ExtraBold.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-ExtraLight.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-Light.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-Medium.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-Regular.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-SemiBold.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/Pretendard-Thin.otfis excluded by!**/*.otfRunLog/Projects/DesignSystem/Resources/Fonts/RacingSansOne-Regular.ttfis excluded by!**/*.ttfRunLog/Projects/DesignSystem/Resources/Images.xcassets/Church.imageset/Church.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Church.imageset/Church@2x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Church.imageset/Church@3x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Earth.imageset/Earth.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Earth.imageset/Earth@2x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Earth.imageset/Earth@3x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Flag.imageset/Flag.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Flag.imageset/Flag@2x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Flag.imageset/Flag@3x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Korea.imageset/Korea.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Korea.imageset/Korea@2x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Korea.imageset/Korea@3x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Mandarin.imageset/Mandarin.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Mandarin.imageset/Mandarin@2x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Mandarin.imageset/Mandarin@3x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Medal.imageset/Medal.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Medal.imageset/Medal@2x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Medal.imageset/Medal@3x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Rocket.imageset/Rocket.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Rocket.imageset/Rocket@2x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/Rocket.imageset/Rocket@3x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/UnitExample.imageset/Frame 2087327598.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/UnitExample.imageset/Frame 2087327598@2x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/UnitExample.imageset/Frame 2087327598@3x.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/noneBeat.imageset/noneBeat.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/oneBeat.imageset/oneBeat.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/threeBeats.imageset/threeBeats.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/twoBeats.imageset/twoBeats.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/walkDog.imageset/walkDog.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/walkGirl.imageset/walkGirl.pngis excluded by!**/*.pngRunLog/Projects/DesignSystem/Resources/Images.xcassets/walkMan.imageset/walkMan.pngis excluded by!**/*.png
📒 Files selected for processing (107)
RunLog/Project.swift(0 hunks)RunLog/Projects/App/InfoPlists/Info.plist(2 hunks)RunLog/Projects/App/Project.swift(1 hunks)RunLog/Projects/App/Sources/AppDelegate.swift(2 hunks)RunLog/Projects/App/Sources/SceneDelegate.swift(1 hunks)RunLog/Projects/Data/Project.swift(1 hunks)RunLog/Projects/Data/Sources/DTOs/DataMapper.swift(1 hunks)RunLog/Projects/Data/Sources/DataContainer/CoreDataContainer.swift(1 hunks)RunLog/Projects/Data/Sources/Impls/AppConfigRepositoryImpl.swift(4 hunks)RunLog/Projects/Data/Sources/Impls/DayLogRepositoryImpl.swift(8 hunks)RunLog/Projects/Data/Sources/Managers/DistanceManager.swift(1 hunks)RunLog/Projects/Data/Sources/Managers/DrawingManager.swift(2 hunks)RunLog/Projects/Data/Sources/Managers/LocationManager.swift(4 hunks)RunLog/Projects/Data/Sources/Managers/PedometerManager.swift(1 hunks)RunLog/Projects/Data/Sources/Network/OpenWeather/OpenWeatherService.swift(1 hunks)RunLog/Projects/DesignSystem/InfoPlists/Info.plist(1 hunks)RunLog/Projects/DesignSystem/Project.swift(1 hunks)RunLog/Projects/DesignSystem/Resources/Images.xcassets/Contents.json(1 hunks)RunLog/Projects/DesignSystem/Sources/ButtonSystem.swift(1 hunks)RunLog/Projects/DesignSystem/Sources/ColorSystem.swift(1 hunks)RunLog/Projects/DesignSystem/Sources/FontSystem.swift(5 hunks)RunLog/Projects/DesignSystem/Sources/IconSystem.swift(3 hunks)RunLog/Projects/DesignSystem/Sources/LabelSystem.swift(3 hunks)RunLog/Projects/DesignSystem/Sources/TextFieldSystem.swift(1 hunks)RunLog/Projects/Domain/Project.swift(1 hunks)RunLog/Projects/Domain/Sources/Entity/AppConfig.swift(1 hunks)RunLog/Projects/Domain/Sources/Entity/DayLog.swift(1 hunks)RunLog/Projects/Domain/Sources/Entity/Section.swift(1 hunks)RunLog/Projects/Domain/Sources/Error/AppConfigError.swift(1 hunks)RunLog/Projects/Domain/Sources/Error/DataError.swift(1 hunks)RunLog/Projects/Domain/Sources/Repository/AppConfigRepository.swift(1 hunks)RunLog/Projects/Domain/Sources/Repository/DayLogRepository.swift(1 hunks)RunLog/Projects/Domain/Sources/UseCase/AppConfigUseCase.swift(1 hunks)RunLog/Projects/Domain/Sources/UseCase/DayLogUseCase.swift(1 hunks)RunLog/Projects/Domain/Sources/UseCase/MediaUseCase.swift(1 hunks)RunLog/Projects/Domain/Sources/UseCaseImpl/AppConfigUseCaseImpl.swift(4 hunks)RunLog/Projects/Domain/Sources/UseCaseImpl/DayLogUseCaseImpl.swift(11 hunks)RunLog/Projects/Domain/Sources/UseCaseImpl/MediaUseCaseImpl.swift(3 hunks)RunLog/Projects/Inject/Project.swift(1 hunks)RunLog/Projects/Inject/Sources/DIContainer.swift(2 hunks)RunLog/Projects/Presentation/Project.swift(1 hunks)RunLog/Projects/Presentation/Sources/Common/View/MainTabBarController.swift(2 hunks)RunLog/Projects/Presentation/Sources/DetailLog/Model/DisplayDayLog.swift(1 hunks)RunLog/Projects/Presentation/Sources/DetailLog/Model/RecordDetail.swift(1 hunks)RunLog/Projects/Presentation/Sources/DetailLog/View/DetailLogView.swift(1 hunks)RunLog/Projects/Presentation/Sources/DetailLog/View/DetailLogViewController.swift(2 hunks)RunLog/Projects/Presentation/Sources/DetailLog/View/MovingTrackSheetView.swift(1 hunks)RunLog/Projects/Presentation/Sources/DetailLog/View/MovingTrackSheetViewController.swift(1 hunks)RunLog/Projects/Presentation/Sources/DetailLog/View/RecordDetailView.swift(1 hunks)RunLog/Projects/Presentation/Sources/DetailLog/View/RecordDetailViewCell.swift(1 hunks)RunLog/Projects/Presentation/Sources/DetailLog/ViewModel/DetailLogViewModel.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/Component/RadioButtonCell.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/View/Calendar/CalendarView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/View/Calendar/CalendarViewCell.swift(2 hunks)RunLog/Projects/Presentation/Sources/Log/View/Calendar/CalendarViewController.swift(2 hunks)RunLog/Projects/Presentation/Sources/Log/View/EditLog/EditLogInfoView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/View/EditLog/EditLogInfoViewController.swift(2 hunks)RunLog/Projects/Presentation/Sources/Log/View/LogView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/View/LogViewController.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/View/Timeline/TimelineView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/View/Timeline/TimelineViewCell.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/ViewModel/EditLogInfoViewModel.swift(1 hunks)RunLog/Projects/Presentation/Sources/Log/ViewModel/LogViewModel.swift(1 hunks)RunLog/Projects/Presentation/Sources/Run/RunningDataProvider.swift(1 hunks)RunLog/Projects/Presentation/Sources/Run/View/CardElementView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Run/View/CardView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Run/View/MapBlurView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Run/View/RunHomeViewController.swift(4 hunks)RunLog/Projects/Presentation/Sources/Run/View/RunningViewController.swift(1 hunks)RunLog/Projects/Presentation/Sources/Run/ViewModel/RunHomeViewModel.swift(2 hunks)RunLog/Projects/Presentation/Sources/Setting/Model/ProfileCardType.swift(1 hunks)RunLog/Projects/Presentation/Sources/Setting/Model/UserInfoVO.swift(1 hunks)RunLog/Projects/Presentation/Sources/Setting/View/CalUnitView.swift(2 hunks)RunLog/Projects/Presentation/Sources/Setting/View/ChangeNicknameView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Setting/View/MypageProfileView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Setting/View/ProfileCardView.swift(1 hunks)RunLog/Projects/Presentation/Sources/Setting/View/SettingMenuCell.swift(1 hunks)RunLog/Projects/Presentation/Sources/Setting/ViewController/ChangeCalUnitViewController.swift(2 hunks)RunLog/Projects/Presentation/Sources/Setting/ViewController/ChangeNicknameViewController.swift(2 hunks)RunLog/Projects/Presentation/Sources/Setting/ViewController/MyPageViewController.swift(2 hunks)RunLog/Projects/Presentation/Sources/Setting/ViewModel/CalUnitViewModel.swift(1 hunks)RunLog/Projects/Presentation/Sources/Setting/ViewModel/ChangeNicknameViewModel.swift(1 hunks)RunLog/Projects/Presentation/Sources/Setting/ViewModel/MyPageViewModel.swift(1 hunks)RunLog/Projects/Util/Project.swift(1 hunks)RunLog/Projects/Util/Sources/Constant/Constants.swift(6 hunks)RunLog/Projects/Util/Sources/Extension/Bundle+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/CLPlacemark+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/Date+.swift(3 hunks)RunLog/Projects/Util/Sources/Extension/Double+.swift(2 hunks)RunLog/Projects/Util/Sources/Extension/Int+.swift(2 hunks)RunLog/Projects/Util/Sources/Extension/MKMapView+.swift(2 hunks)RunLog/Projects/Util/Sources/Extension/String+.swift(2 hunks)RunLog/Projects/Util/Sources/Extension/TimeInterval+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/UIButton+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/UIColor+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/UIControl+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/UINavigationController+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/UITapGestureRecognizer+.swift(2 hunks)RunLog/Projects/Util/Sources/Extension/UITextField+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/UIView+.swift(1 hunks)RunLog/Projects/Util/Sources/Extension/UIViewController+.swift(3 hunks)RunLog/Projects/Util/Sources/Helpers/DynamicSize.swift(3 hunks)RunLog/RunLog/Info.plist(0 hunks)RunLog/RunLog/Sources/Data/CoreDataContainer.swift(0 hunks)RunLog/RunLog/Sources/Domain/Entity/AppConfig.swift(0 hunks)RunLog/RunLog/Sources/Domain/Entity/DayLog.swift(0 hunks)RunLog/RunLog/Sources/Domain/Entity/Road.swift(0 hunks)
⛔ Files not processed due to max files limit (6)
- RunLog/RunLog/Sources/Domain/Entity/Section.swift
- RunLog/RunLog/Sources/Util/Extension/DayLog+.swift
- RunLog/RunLog/Sources/Util/Extension/UINavigationController+.swift
- RunLog/Tuist/ProjectDescriptionHelpers/Project+Templete.swift
- RunLog/Tuist/ProjectDescriptionHelpers/ProjectName.swift
- RunLog/Workspace.swift
💤 Files with no reviewable changes (6)
- RunLog/RunLog/Sources/Domain/Entity/Road.swift
- RunLog/Project.swift
- RunLog/RunLog/Sources/Data/CoreDataContainer.swift
- RunLog/RunLog/Sources/Domain/Entity/DayLog.swift
- RunLog/RunLog/Sources/Domain/Entity/AppConfig.swift
- RunLog/RunLog/Info.plist
🧰 Additional context used
🧠 Learnings (26)
📓 Common learnings
Learnt from: Monfi98
PR: ESTSOFT-iOS-01/RunLog#134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/DetailLog/View/RecordDetailViewCell.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/Log/View/LogView.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/DetailLog/View/RecordDetailView.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/DetailLog/Model/DisplayDayLog.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/App/Sources/SceneDelegate.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/App/Sources/AppDelegate.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/Log/ViewModel/EditLogInfoViewModel.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/Log/View/EditLog/EditLogInfoView.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/DetailLog/Model/RecordDetail.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Util/Project.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/DesignSystem/Resources/Images.xcassets/Contents.json (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Data/Project.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Project.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/App/Project.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Domain/Project.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Inject/Project.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/DesignSystem/Project.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/Run/View/RunningViewController.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Domain/Sources/Entity/Section.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/App/InfoPlists/Info.plist (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Domain/Sources/Entity/AppConfig.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Presentation/Sources/Log/ViewModel/LogViewModel.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Domain/Sources/Entity/DayLog.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/DesignSystem/InfoPlists/Info.plist (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
RunLog/Projects/Util/Sources/Constant/Constants.swift (1)
Learnt from: Monfi98
PR: #134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
🧬 Code Graph Analysis (27)
RunLog/Projects/DesignSystem/Sources/ButtonSystem.swift (1)
RunLog/RunLog/Sources/DesignSystem/ButtonSystem.swift (2)
RLButton(12-61)RLButton(64-116)
RunLog/Projects/Presentation/Sources/Setting/Model/ProfileCardType.swift (1)
RunLog/RunLog/Sources/Presentation/Setting/View/ProfileCardView.swift (1)
ProfileCardView(13-108)
RunLog/Projects/Presentation/Sources/DetailLog/View/RecordDetailView.swift (1)
RunLog/RunLog/Sources/Presentation/DetailLog/View/RecordDetailView.swift (1)
RecordDetailView(12-90)
RunLog/Projects/Presentation/Sources/Setting/Model/UserInfoVO.swift (1)
RunLog/RunLog/Sources/Presentation/Setting/Model/UserInfoVO.swift (2)
UserInfoVO(25-40)UserInfoVO(11-23)
RunLog/Projects/App/Sources/SceneDelegate.swift (1)
RunLog/RunLog/Sources/App/SceneDelegate.swift (5)
sceneWillEnterForeground(43-46)SceneDelegate(10-55)sceneWillResignActive(38-41)sceneDidEnterBackground(48-52)sceneDidDisconnect(26-31)
RunLog/Projects/Domain/Sources/Repository/AppConfigRepository.swift (4)
RunLog/RunLog/Sources/Domain/Repository/AppConfigRepository.swift (1)
AppConfigRepository(12-28)RunLog/RunLog/Sources/Data/Impl/AppConfigRepositoryImpl.swift (2)
AppConfigRepositoryImpl(11-128)updateAppConfig(68-97)RunLog/RpTest/Sources/RpTest.swift (3)
AppConfigRepositoryTests(12-91)test_createAndReadAppConfig(27-43)test_updateAppConfig(46-67)RunLog/RunLog/Sources/Domain/UseCaseImpl/AppConfigUseCaseImpl.swift (3)
AppConfigUseCaseImpl(10-89)updateUnitDistance(71-78)updateNickname(80-87)
RunLog/Projects/Util/Sources/Extension/Bundle+.swift (1)
RunLog/RunLog/Sources/Util/Extension/Bundle+.swift (1)
Bundle(9-16)
RunLog/Projects/Util/Project.swift (1)
RunLog/Tuist/ProjectDescriptionHelpers/Project+Templete.swift (2)
project(4-46)framework(66-84)
RunLog/Projects/Domain/Sources/UseCase/AppConfigUseCase.swift (1)
RunLog/RunLog/Sources/Domain/UseCase/AppConfigUseCase.swift (1)
AppConfigUseCase(10-65)
RunLog/Projects/Domain/Sources/UseCase/MediaUseCase.swift (3)
RunLog/RunLog/Sources/Domain/UseCaseImpl/MediaUseCaseImpl.swift (1)
MediaUseCaseImpl(12-145)RunLog/RunLog/Sources/Domain/UseCase/MediaUseCase.swift (1)
MediaUseCase(13-31)RunLog/RunLog/Sources/Domain/Error/MediaUseCaseError.swift (2)
MediaUseCaseError(18-27)MediaUseCaseError(11-16)
RunLog/Projects/Util/Sources/Extension/UIView+.swift (4)
RunLog/RunLog/Sources/Util/Extension/UIView+.swift (2)
UIView(10-15)addSubviews(12-14)RunLog/RunLog/Sources/Presentation/Log/View/LogView.swift (1)
LogView(12-57)RunLog/RunLog/Sources/Presentation/Run/View/CardView.swift (1)
setupUI(39-67)RunLog/RunLog/Sources/Presentation/Log/View/EditLog/EditLogInfoView.swift (1)
EditLogInfoView(12-80)
RunLog/Projects/Presentation/Sources/Setting/View/CalUnitView.swift (2)
RunLog/Projects/DesignSystem/Sources/IconSystem.swift (1)
designSystem(123-125)RunLog/RunLog/Sources/Presentation/Setting/View/CalUnitView.swift (2)
CalUnitView(13-106)setupUI(57-62)
RunLog/Projects/Util/Sources/Extension/CLPlacemark+.swift (1)
RunLog/RunLog/Sources/Util/Extension/CLPlacemark+.swift (2)
placemarksToString(15-64)CLPlacemark(12-65)
RunLog/Projects/DesignSystem/Sources/IconSystem.swift (1)
RunLog/RunLog/Sources/DesignSystem/IconSystem.swift (2)
RLIcon(49-119)RLIcon(11-47)
RunLog/Projects/Domain/Sources/Entity/Section.swift (6)
RunLog/RunLog/Sources/Domain/Entity/Section.swift (2)
Section(10-14)Point(16-20)RunLog/RunLog/Sources/Presentation/DetailLog/Model/RecordDetail.swift (2)
RecordDetail(17-38)RecordDetail(10-15)RunLog/RunLog/Sources/Data/DTO/DataMapper.swift (2)
toDTO(34-43)toEntity(93-102)RunLog/RunLog/Sources/Domain/UseCaseImpl/DayLogUseCaseImpl.swift (1)
addSectionByDate(86-115)RunLog/RunLog/Sources/Domain/UseCaseImpl/MediaUseCaseImpl.swift (1)
convertSectionsToCoordinates(16-29)RunLog/RunLog/Sources/Domain/Entity/DayLog.swift (1)
DayLog(10-23)
RunLog/Projects/Util/Sources/Extension/UITapGestureRecognizer+.swift (1)
RunLog/RunLog/Sources/Util/Extension/UITapGestureRecognizer+.swift (6)
UITapGestureRecognizer(11-66)TapGestureSubscription(30-59)TapPublisher(13-60)receive(23-27)handleTap(55-58)request(45-47)
RunLog/Projects/Util/Sources/Extension/Double+.swift (2)
RunLog/RunLog/Sources/Util/Extension/Double+.swift (1)
Double(10-41)RunLog/RpTest/Sources/DoubleTest.swift (3)
DoubleTest(11-41)testPerformanceExample(35-39)testFormattedStringWithDecimal(24-27)
RunLog/Projects/Presentation/Sources/Log/View/LogViewController.swift (3)
RunLog/Projects/Util/Sources/Extension/UINavigationController+.swift (2)
setupAppearance(14-37)setupTitle(39-41)RunLog/Projects/DesignSystem/Sources/TextSystem.swift (1)
RLAttributedString(36-52)RunLog/RunLog/Sources/Presentation/Log/View/LogViewController.swift (1)
setupNavigationBar(75-78)
RunLog/Projects/Util/Sources/Extension/Date+.swift (1)
RunLog/RunLog/Sources/Util/Extension/Date+.swift (3)
Date(7-27)DateFormatStyle(29-56)formattedString(20-26)
RunLog/Projects/Inject/Sources/DIContainer.swift (2)
RunLog/RunLog/Sources/App/DIContainer.swift (4)
DIContainer(10-30)Dependency(33-40)register(18-21)resolve(23-29)RunLog/RunLog/Sources/App/AppDelegate.swift (1)
registerDependencies(34-60)
RunLog/Projects/Util/Sources/Extension/UIViewController+.swift (1)
RunLog/RunLog/Sources/Util/Extension/UIViewController+.swift (7)
UIViewController(12-91)setupTabBarAppearance(64-73)LoadingIndicator(13-15)startLoading(18-37)stopLoading(40-45)setupNavigationBarAppearance(48-61)setupTapGestureToDismissKeyboard(76-80)
RunLog/Projects/Domain/Sources/UseCaseImpl/MediaUseCaseImpl.swift (4)
RunLog/RunLog/Sources/Domain/UseCaseImpl/MediaUseCaseImpl.swift (4)
MediaUseCaseImpl(12-145)setRouteImage(31-77)makeRouteSizeRegion(127-143)convertSectionsToCoordinates(16-29)RunLog/RunLog/Sources/Domain/UseCase/MediaUseCase.swift (1)
MediaUseCase(13-31)RunLog/RpTest/Sources/MediaUseCaseTests.swift (4)
MediaUseCaseTests(5-73)setUpWithError(10-14)test_createAndSavePolylineImage(22-53)tearDownWithError(16-20)RunLog/RunLog/Sources/Domain/Error/MediaUseCaseError.swift (1)
MediaUseCaseError(11-16)
RunLog/Projects/Util/Sources/Extension/UIControl+.swift (1)
RunLog/RunLog/Sources/Util/Extension/UIControl+.swift (8)
UIControl(11-55)EventPublisher(17-28)controlPublisher(13-15)EventSubscription(30-54)receive(24-27)handleEvent(51-53)cancel(46-49)request(44-44)
RunLog/Projects/Domain/Sources/Entity/DayLog.swift (4)
RunLog/RunLog/Sources/Domain/Entity/DayLog.swift (1)
DayLog(10-23)RunLog/RunLog/Sources/Util/Extension/DayLog+.swift (1)
DayLog(10-15)RunLog/RunLog/Sources/Domain/UseCaseImpl/DayLogUseCaseImpl.swift (2)
addSectionByDate(86-115)DayLogUseCaseImpl(11-194)RunLog/RunLog/Sources/Domain/Entity/Section.swift (1)
Section(10-14)
RunLog/Projects/DesignSystem/InfoPlists/Info.plist (1)
RunLog/RunLog/Sources/DesignSystem/FontSystem.swift (2)
UIFont(94-198)RLFont(34-92)
RunLog/Projects/Util/Sources/Constant/Constants.swift (3)
RunLog/RunLog/Sources/Domain/Entity/Road.swift (1)
Road(10-14)RunLog/RunLog/Sources/Util/Constant/Constants.swift (4)
Constants(10-142)AqiLevel(119-141)MotivationMessage(23-53)WeatherCondition(81-116)RunLog/RunLog/Sources/Presentation/Run/ViewModel/RunHomeViewModel.swift (1)
RunHomeViewModel(99-113)
RunLog/Projects/Presentation/Sources/Common/View/MainTabBarController.swift (4)
RunLog/Projects/Presentation/Sources/Run/View/RunHomeViewController.swift (1)
viewDidLoad(68-79)RunLog/RunLog/Sources/Presentation/Common/View/MainTabBarController.swift (4)
MainTabBarController(53-69)MainTab(10-51)viewDidLoad(65-68)navigationController(42-50)RunLog/RunLog/Sources/Util/Extension/UIViewController+.swift (1)
setupTabBarAppearance(64-73)RunLog/RunLog/Sources/App/SceneDelegate.swift (1)
scene(14-24)
🪛 SwiftLint (0.57.0)
RunLog/Projects/Util/Sources/Extension/UIViewController+.swift
[Warning] 25-25: Prefer at least one space after slashes for comments
(comment_spacing)
RunLog/Projects/Util/Sources/Extension/String+.swift
[Warning] 36-36: Prefer at least one space after slashes for comments
(comment_spacing)
[Warning] 37-37: Prefer at least one space after slashes for comments
(comment_spacing)
[Warning] 38-38: Prefer at least one space after slashes for comments
(comment_spacing)
[Warning] 39-39: Prefer at least one space after slashes for comments
(comment_spacing)
[Warning] 40-40: Prefer at least one space after slashes for comments
(comment_spacing)
RunLog/Projects/Domain/Sources/Entity/DayLog.swift
[Warning] 52-52: Prefer at least one space after slashes for comments
(comment_spacing)
[Warning] 57-57: Prefer at least one space after slashes for comments
(comment_spacing)
RunLog/Projects/DesignSystem/Sources/FontSystem.swift
[Warning] 106-106: TODOs should be resolved (Pretendard로 네이밍 변경)
(todo)
RunLog/Projects/Util/Sources/Helpers/DynamicSize.swift
[Warning] 15-15: Prefer at least one space after slashes for comments
(comment_spacing)
🔇 Additional comments (155)
RunLog/Projects/Presentation/Sources/Setting/View/ChangeNicknameView.swift (1)
7-8: 모듈화에 따른 적절한 import 추가RLDesignSystem과 RLUtil 모듈 import가 추가되어 기존에 사용하던 RLTextField, DynamicSize 등의 컴포넌트와 유틸리티에 대한 명시적 의존성이 설정되었습니다. 모듈화 구조에 맞는 올바른 변경사항입니다.
RunLog/Projects/Presentation/Sources/DetailLog/ViewModel/DetailLogViewModel.swift (1)
7-8: 도메인과 의존성 주입 모듈 import 추가RLDomain 모듈 import로 DayLog 엔티티에 대한 접근이 가능해지고, RLInject 모듈 import로 @dependency 어노테이션을 통한 의존성 주입이 가능해집니다. 프레젠테이션 레이어에서 도메인 레이어와 의존성 주입을 활용하기 위한 적절한 설정입니다.
RunLog/Projects/Presentation/Sources/Log/View/Calendar/CalendarView.swift (1)
7-8: 디자인 시스템과 유틸리티 모듈 import 추가기존 코드에서 사용 중인 RLAttributedString, DynamicSize, RLIcon 등의 디자인 시스템 컴포넌트와 유틸리티에 대한 명시적 모듈 의존성이 설정되었습니다. 모듈화 구조에 맞는 올바른 변경사항입니다.
RunLog/Projects/Presentation/Sources/Setting/Model/UserInfoVO.swift (1)
7-7: 유틸리티 모듈 import 추가코드에서 사용하는
toString(withDecimal:)메서드가 RLUtil 모듈에서 제공되므로 import 추가가 필요합니다. 모듈화에 따른 적절한 의존성 설정입니다.RunLog/Projects/Presentation/Sources/Run/View/MapBlurView.swift (1)
7-7: 유틸리티 모듈 import 추가코드에서 사용하는
DynamicSize.scaledSize()메서드와addSubviews메서드가 RLUtil 모듈에서 제공되므로 import 추가가 필요합니다. 모듈화 구조에 맞는 올바른 변경사항입니다.RunLog/Projects/Presentation/Sources/DetailLog/View/MovingTrackSheetView.swift (1)
7-8: 모듈화에 필요한 import 문 추가가 적절합니다.RLDesignSystem과 RLUtil 모듈 import가 올바르게 추가되었습니다. 파일에서 사용되는 RLLabel, DynamicSize.scaledSize, Date.formattedString 등의 기능들이 이 모듈들에서 제공되므로 필수적인 변경사항입니다.
RunLog/Projects/Presentation/Sources/Run/View/CardView.swift (1)
7-8: 모듈 분리에 따른 필수 import 문이 정확히 추가되었습니다.RLDesignSystem(RLButton 사용)과 RLUtil(DynamicSize.scaledSize 사용) 모듈에 대한 import가 올바르게 추가되어 기존 기능들이 정상 작동할 수 있습니다.
RunLog/Projects/Presentation/Sources/Log/ViewModel/LogViewModel.swift (1)
7-8: 도메인 및 의존성 주입 모듈 import가 올바르게 추가되었습니다.RLDomain(DayLogUseCase, AppConfigUseCase, DayLog 엔티티 사용)과 RLInject(@dependency 프로퍼티 래퍼 사용) 모듈에 대한 import가 적절히 추가되어 기존 기능이 정상 작동할 수 있습니다.
RunLog/Projects/DesignSystem/Sources/ColorSystem.swift (1)
7-7: UIColor hex 초기화를 위한 RLUtil import가 필요합니다.UIColor(hex:) 및 UIColor(hex:alpha:) 초기화 메서드 사용을 위해 RLUtil 모듈 import가 올바르게 추가되었습니다. 이는 컬러 시스템이 정상 작동하기 위해 필수적인 변경사항입니다.
RunLog/Projects/Presentation/Sources/Setting/ViewModel/CalUnitViewModel.swift (1)
7-8: 도메인 및 의존성 주입 모듈 import가 적절히 추가되었습니다.RLDomain(AppConfigUseCase 사용)과 RLInject(@dependency 프로퍼티 래퍼 사용) 모듈에 대한 import가 올바르게 추가되었습니다.
RunLog/Projects/Presentation/Sources/Setting/Model/ProfileCardType.swift (1)
7-7: 적절한 import 추가입니다.
ProfileCardTypeenum에서RLIcon.document.name와RLIcon.streak.name을 사용하므로RLDesignSystem모듈 import가 필수적입니다.RunLog/Projects/Presentation/Sources/DetailLog/Model/DisplayDayLog.swift (1)
7-8: 필요한 모듈 import가 올바르게 추가되었습니다.
RLDomain:DayLog엔티티 사용을 위해 필요RLUtil:toLevelDescription()확장 메서드 사용을 위해 필요두 import 모두 코드에서 실제로 사용되므로 적절합니다.
RunLog/Projects/Presentation/Sources/Log/View/LogView.swift (1)
7-7: 올바른 import 추가입니다.
LogView에서DynamicSize.scaledSize()메서드를 사용하므로RLUtil모듈 import가 필요합니다.RunLog/Projects/Data/Sources/DTOs/DataMapper.swift (1)
7-7: 필수적인 import 추가입니다.
DataMapper가DayLog,Section,Point,AppConfig등의 도메인 엔티티를 사용하므로RLDomain모듈 import가 반드시 필요합니다.RunLog/Projects/Presentation/Sources/Setting/ViewModel/ChangeNicknameViewModel.swift (1)
7-8: 의존성 주입을 위한 적절한 import 추가입니다.
RLDomain:AppConfigUseCase프로토콜 사용을 위해 필요RLInject:@Dependency프로퍼티 래퍼 사용을 위해 필요두 import 모두 의존성 주입 패턴 구현에 필수적입니다.
RunLog/Projects/Presentation/Sources/Log/View/Timeline/TimelineView.swift (1)
7-7: 모듈화에 맞는 적절한 import 추가
DynamicSize.scaledSize()함수 사용을 위해RLUtil모듈 import가 필요하며, 모듈화 구조에 부합하는 변경사항입니다.RunLog/Projects/Presentation/Sources/Run/View/CardElementView.swift (1)
7-7: 디자인 시스템 모듈 import 적절함
RLFont,.RLAttributedString등 디자인 시스템 컴포넌트 사용을 위해RLDesignSystem모듈 import가 필요하며, 모듈 분리 목적에 부합합니다.RunLog/Projects/Presentation/Sources/Setting/View/SettingMenuCell.swift (1)
7-8: 필요한 모듈들의 적절한 import
RLIcon,.RLAttributedString(RLDesignSystem)과DynamicSize.scaledSize()(RLUtil) 사용을 위해 두 모듈 import가 모두 필요하며, 모듈화 아키텍처에 적합한 변경입니다.RunLog/Projects/Presentation/Sources/DetailLog/View/RecordDetailView.swift (1)
7-7: 유틸리티 모듈 import 적절함
DynamicSize.scaledSize()함수 사용을 위해RLUtil모듈 import가 필요하며, 모듈 분리 구조에 맞는 변경사항입니다.RunLog/Projects/Presentation/Sources/DetailLog/Model/RecordDetail.swift (1)
7-7: 도메인 모듈 import 필요함
Section,Point등 도메인 타입 사용을 위해RLDomain모듈 import가 필요하며, 도메인 레이어 분리에 따른 적절한 변경입니다.RunLog/Projects/DesignSystem/Sources/ButtonSystem.swift (1)
7-7: 모듈화에 맞는 적절한 import 추가입니다.
RLUtil모듈 import는RLButton클래스에서 사용하는DynamicSize.scaledSize()등의 유틸리티 함수에 접근하기 위해 필요합니다. 모듈 간 의존성이 명확하게 정의되어 좋습니다.RunLog/Projects/Presentation/Sources/Setting/View/MypageProfileView.swift (1)
7-7: 유틸리티 모듈 접근을 위한 필수 import입니다.
RLUtilimport를 통해DynamicSize.scaledSize(),.RLAttributedString(),.styledText()등의 유틸리티 함수에 접근할 수 있습니다. 모듈화 구조에 적합한 변경사항입니다.RunLog/Projects/Presentation/Sources/Run/RunningDataProvider.swift (1)
7-9: 레이어 분리에 따른 적절한 모듈 import들입니다.
RLDomain: 도메인 엔티티 및 유스케이스 접근용RLData: 데이터 레이어 접근용RLInject:@Dependency프로퍼티 래퍼를 통한 의존성 주입용각 레이어가 명확하게 분리되어 모듈화 목표에 부합합니다.
RunLog/Projects/Presentation/Sources/DetailLog/View/RecordDetailViewCell.swift (1)
7-7: UI 컴포넌트의 유틸리티 접근을 위한 적절한 import입니다.
RLUtilimport를 통해DynamicSize.scaledSize()및 커스텀 폰트 확장 기능(.RLHeadline2,.RLBody1)에 접근할 수 있습니다. 모듈화된 유틸리티 활용에 필요한 변경사항입니다.RunLog/Projects/Presentation/Sources/Setting/View/ProfileCardView.swift (1)
7-7: 프로필 카드 뷰의 유틸리티 기능 활용을 위한 필수 import입니다.
RLUtilimport를 통해DynamicSize.scaledSize(),.RLAttributedString(),.formattedString등의 유틸리티 함수에 접근할 수 있습니다. 모듈화된 구조에서 공통 유틸리티를 효과적으로 활용하는 적절한 변경사항입니다.RunLog/Projects/Presentation/Sources/Log/View/Timeline/TimelineViewCell.swift (1)
7-7: 모듈화에 따른 필수 import 추가가 적절합니다.코드에서 사용하는 유틸리티 확장자들(
DynamicSize.scaledSize(),.toString(),.formattedString(),.RLAttributedString())이 RLUtil 모듈에 정의되어 있어 이 import가 필요합니다.RunLog/Projects/Presentation/Sources/Log/View/EditLog/EditLogInfoView.swift (1)
7-8: 모듈화에 따른 필수 import들이 적절히 추가되었습니다.RLDesignSystem은 RLTextField 컴포넌트를 위해, RLUtil은 유틸리티 확장자들(
.RLAttributedString(),DynamicSize.scaledSize())을 위해 필요합니다.RunLog/Projects/Presentation/Sources/Log/Component/RadioButtonCell.swift (1)
7-7: RLUtil import 불필요
RLAttributedString확장은RLDesignSystem(TextSystem.swift의NSAttributedStringextension)에서 제공되므로 별도의RLUtilimport는 필요하지 않습니다. 원래 제안된RLUtilimport는 제거해 주세요.Likely an incorrect or invalid review comment.
RunLog/Projects/DesignSystem/Sources/TextFieldSystem.swift (1)
7-7: DesignSystem 모듈에서 유틸리티 사용을 위한 적절한 import입니다.코드에서 사용하는 유틸리티 확장자들(
.RLAttributedString(),DynamicSize.scaledSize(),.publisher)이 RLUtil 모듈에서 제공되므로 이 import가 필요합니다.RunLog/Projects/Presentation/Sources/Run/View/RunningViewController.swift (1)
7-8: 모듈화 구조에 맞는 올바른 import 추가입니다.RLDesignSystem은 RLButton과 RLIcon 사용을 위해, RLUtil은 다양한 유틸리티 확장자들(
.RLAttributedString(),DynamicSize.scaledSize(),.publisher,.initZoomLevel()등)을 위해 필요합니다.RunLog/Projects/Presentation/Sources/Setting/ViewModel/MyPageViewModel.swift (1)
7-8: 모듈화에 따른 올바른 import 추가
RLDomain과RLInject모듈 import가 정확하게 추가되었습니다. 코드에서 사용하는@Dependency프로퍼티 래퍼와 도메인 유스케이스들을 위해 필요한 의존성입니다.RunLog/Projects/Presentation/Sources/DetailLog/View/MovingTrackSheetViewController.swift (1)
7-7: 도메인 모듈 import 추가 확인
RLDomainimport가 올바르게 추가되었습니다.DetailLogViewModel에서 사용하는 도메인 모델들에 접근하기 위해 필요한 의존성입니다.RunLog/Projects/Presentation/Sources/Log/ViewModel/EditLogInfoViewModel.swift (1)
7-8: 의존성 주입과 도메인 모듈 import 추가
RLDomain과RLInject모듈 import가 적절히 추가되었습니다.@Dependency어노테이션과DayLogUseCase사용을 위해 필요한 의존성들입니다.RunLog/Projects/App/Sources/SceneDelegate.swift (1)
7-8: 앱 모듈의 올바른 의존성 import
RLPresentation과RLUtil모듈 import가 정확합니다.MainTabBarController접근과DynamicSize유틸리티 사용을 위해 필요한 의존성들입니다.RunLog/Projects/Presentation/Sources/DetailLog/View/DetailLogView.swift (1)
7-8: 디자인 시스템과 유틸리티 모듈 import 추가
RLDesignSystem과RLUtil모듈 import가 올바르게 추가되었습니다. 코드에서 사용하는RLLabel,RLIcon등의 디자인 시스템 컴포넌트와DynamicSize유틸리티를 위해 필요한 의존성입니다.RunLog/Projects/DesignSystem/Resources/Images.xcassets/Contents.json (1)
1-6: 애셋 카탈로그 메타데이터 추가 확인 완료
Contents.json의 기본info필드만 정의되어 있으며 문제될 부분이 없습니다. 향후 실제 이미지 세트가 추가될 때 빈 애셋으로 남지 않도록 주의해 주세요.RunLog/Projects/Domain/Sources/UseCase/MediaUseCase.swift (1)
13-19:Section타입의 접근 수준도public인지 확인 필요프로토콜을
public으로 승격하면서 메서드 시그니처에 노출되는Section역시 외부 모듈에서 볼 수 있어야 컴파일 오류가 없습니다.
Section(및 그 내부 속성)·MediaUseCaseError의 접근 수준을 함께 검토해 주세요.RunLog/Projects/Domain/Sources/UseCase/AppConfigUseCase.swift (1)
10-12: 공개 프로토콜로 변경 시 의존 타입 공개 여부 확인
AppConfigError,DayLog,Section등 프로토콜 메서드에 노출되는 모든 엔티티가public인지 확인해 주세요.
미공개 항목이 있으면 외부 모듈 사용 시 컴파일이 실패합니다.RunLog/Projects/Domain/Sources/UseCase/DayLogUseCase.swift (1)
11-12: 공개 프로토콜 승격 시 관련 엔티티 접근 수준 확인
DayLog,Section등이public이어야 외부 모듈에서 문제없이 호출 가능합니다. 확인 바랍니다.RunLog/Projects/Util/Sources/Extension/CLPlacemark+.swift (1)
14-14: 모듈화를 위한 접근 제어자 변경이 적절합니다.
placemarksToString()메서드를public으로 변경하여 다른 모듈에서 사용할 수 있도록 한 것이 모듈화 목적에 부합합니다. 메서드의 구현 로직은 그대로 유지되어 기존 기능에 영향을 주지 않습니다.RunLog/Projects/Util/Sources/Extension/Bundle+.swift (1)
11-11: API 키 노출에 대한 보안 고려사항을 검토해주세요.
weatherKey프로퍼티를public으로 변경하면 다른 모듈에서 API 키에 접근할 수 있게 됩니다. 모듈화 목적에는 부합하지만, API 키와 같은 민감한 정보의 공개 노출에 대한 보안 검토가 필요합니다.다음 사항들을 확인해주세요:
- API 키가 실제로 필요한 모듈에서만 접근하는지
- 더 제한적인 접근 방식(예: 특정 서비스 클래스를 통한 간접 접근)을 고려할 수 있는지
RunLog/Projects/Inject/Project.swift (1)
1-9: Inject 모듈의 프로젝트 설정이 올바르게 구성되었습니다.의존성 주입을 위한 별도 모듈 생성이 순환 의존성 문제 해결이라는 PR 목표에 부합합니다. 프로젝트 설정이 다른 모듈들과 일관된 패턴을 따르고 있어 좋습니다.
RunLog/Projects/Presentation/Sources/Setting/View/CalUnitView.swift (2)
7-8: 모듈 import가 적절합니다.
RLDesignSystem과RLUtil모듈을 import하여 모듈화된 아키텍처에 맞게 의존성을 명시적으로 선언한 것이 좋습니다.
24-24: DesignSystem 모듈 통합이 올바르게 적용되었습니다.
UIImage(named:)대신.designSystem(named:)를 사용하여 DesignSystem 모듈의 번들에서 이미지를 로드하도록 변경한 것이 모듈화 목적에 부합합니다. 이렇게 하면 이미지 리소스가 올바른 모듈 번들에서 로드됩니다.RunLog/Projects/Util/Sources/Extension/UIButton+.swift (1)
12-12: 모듈화에 적합한 접근 제어자 변경입니다.UIButton의 Combine publisher를 다른 모듈에서 사용할 수 있도록 public으로 변경한 것이 적절합니다. 모듈 간 UI 이벤트 처리를 위해 필요한 변경사항입니다.
RunLog/Projects/Util/Sources/Extension/UITextField+.swift (1)
12-12: 텍스트 필드 이벤트 처리를 위한 적절한 공개 변경입니다.UITextField의 텍스트 변경 이벤트를 다른 모듈에서 구독할 수 있도록 public으로 변경한 것이 모듈화 목적에 부합합니다.
RunLog/Projects/Domain/Sources/Error/AppConfigError.swift (1)
11-11: 도메인 에러 타입의 적절한 공개화입니다.AppConfigError를 public으로 변경하여 다른 모듈에서 앱 설정 관련 에러를 적절히 처리할 수 있도록 한 것이 올바른 접근입니다. 에러 케이스들도 포괄적으로 정의되어 있습니다.
RunLog/Projects/Util/Sources/Extension/TimeInterval+.swift (1)
12-12: 시간 포맷팅 유틸리티의 올바른 공개화입니다.TimeInterval의 시간 문자열 변환 기능을 다른 모듈에서 사용할 수 있도록 public으로 변경한 것이 적절합니다. 한국어 형식("X시간 Y분")으로 제공하는 것도 좋습니다.
RunLog/Projects/Domain/Sources/Repository/AppConfigRepository.swift (1)
12-12: 저장소 프로토콜의 필수적인 공개화 변경입니다.AppConfigRepository 프로토콜을 public으로 변경하여 다른 모듈에서 의존성 주입이 가능하도록 한 것이 모듈화 아키텍처에 필수적입니다. 프로토콜이 명확한 CRUD 인터페이스를 제공하고 있어 좋습니다.
RunLog/Projects/Presentation/Sources/Log/View/Calendar/CalendarViewCell.swift (2)
7-8: 모듈 import 변경 승인DesignSystem과 Util 모듈 분리에 따른 적절한 import 추가입니다. 모듈화를 통해 의존성이 명확해졌습니다.
80-80: DesignSystem 모듈의 중앙화된 이미지 관리 적용 승인
UIImage(named:)에서.designSystem(named:)로 변경하여 DesignSystem 모듈의 중앙화된 이미지 관리 방식을 적용한 것이 좋습니다. 이는 리소스 관리의 일관성을 높이고 유지보수성을 개선합니다.RunLog/Projects/Presentation/Sources/Log/View/Calendar/CalendarViewController.swift (2)
7-8: 모듈 import 변경 승인Presentation 레이어에서 DesignSystem과 Util 모듈을 적절히 import하여 모듈화된 구조에 맞게 의존성을 관리하고 있습니다.
61-61: 속성 접근 방식 일관성 확인 완료전체 코드베이스에서
iconName만 사용되고.icon.name패턴은 더 이상 존재하지 않음을 확인했습니다. 변경 사항이 일관되게 반영되어 있어 추가 조치가 필요 없습니다.
머지 진행 부탁드립니다.RunLog/Projects/Presentation/Sources/Run/ViewModel/RunHomeViewModel.swift (2)
7-9: 모듈화에 따른 적절한 import 추가Presentation 레이어에서 Domain, Util, Inject 모듈을 적절히 import하여 모듈 간 의존성을 명확히 표현하고 있습니다.
135-137: styledText API 개선 적용 승인
styledText메서드에서baseColor와highlightColor파라미터를 명시적으로 지정하도록 변경된 것은 API의 명확성을 높이는 좋은 개선사항입니다. 기본값에 의존하지 않고 명시적으로 색상을 지정하여 의도를 더 명확히 표현하고 있습니다.RunLog/Projects/Data/Project.swift (1)
4-18: Data 모듈 설정 승인Data 모듈이 적절한 의존성 구조로 설정되었습니다:
- Domain, Inject, Util 모듈에 대한 적절한 의존성
- 네트워킹을 위한 Moya 라이브러리 포함
- Core Data 모델 포함으로 데이터 지속성 지원
모듈화를 통해 Data 레이어의 책임이 명확히 분리되고, 외부 라이브러리 의존성도 적절히 관리되고 있습니다.
RunLog/Projects/DesignSystem/InfoPlists/Info.plist (1)
1-37: DesignSystem 모듈의 Info.plist 구성이 적절합니다.프레임워크 패키지 타입(FMWK) 설정과 UIAppFonts 배열의 폰트 파일들이 FontSystem.swift에서 참조하는 폰트들과 일치합니다. 빌드 설정 플레이스홀더 사용도 적절합니다.
RunLog/Projects/Util/Sources/Extension/UIView+.swift (1)
12-12: 모듈화를 위한 접근 제어자 변경이 적절합니다.
addSubviews메서드를 public으로 변경하여 다른 모듈(특히 Presentation 레이어)에서 사용할 수 있도록 한 것이 맞습니다. 관련 코드 스니펫에서 확인된 바와 같이 이 메서드는 여러 UI 클래스에서 활용되고 있습니다.RunLog/Projects/Util/Project.swift (1)
4-11: Util 모듈 프로젝트 구성이 적절합니다.프레임워크 설정과 SnapKit 의존성이 유틸리티 모듈의 목적에 맞게 구성되어 있습니다. Module 상수 사용으로 일관된 네이밍 전략을 유지하고 있습니다.
RunLog/Projects/DesignSystem/Project.swift (1)
4-15: DesignSystem 모듈 프로젝트 구성이 적절합니다.내부 유틸리티 모듈 의존성과 UI 관련 외부 라이브러리들(SnapKit, Then, NVActivityIndicatorView)이 디자인 시스템의 목적에 맞게 구성되어 있습니다. 커스텀 Info.plist 경로와 리소스 설정도 적절합니다.
RunLog/Projects/Presentation/Project.swift (1)
4-18: Presentation 모듈 프로젝트 구성이 클린 아키텍처 원칙을 잘 따르고 있습니다.레이어드 아키텍처에 맞는 의존성 구성이 적절합니다:
- inject, domain, util, designSystem 내부 모듈 의존성
- UI 관련 외부 라이브러리들의 일관된 사용
- Presentation → Domain 방향의 올바른 의존성 방향
모듈화 목표에 부합하는 깔끔한 구성입니다.
RunLog/Projects/Util/Sources/Extension/UIColor+.swift (1)
14-14: 접근 제어자 변경이 적절합니다모듈화 구조에서 다른 모듈들이 이 유틸리티를 사용할 수 있도록
public접근 제어자를 추가한 것이 올바릅니다. hex 문자열을 UIColor로 변환하는 로직도 정확하게 구현되어 있습니다.RunLog/Projects/App/Project.swift (1)
4-16: App 모듈 설정 검증 완료: InfoPlist 경로 및 의존성 정상
- RunLog/Projects/App/InfoPlists/Info.plist 파일이 존재하며,
InfoPlists/info.plist경로가 올바릅니다.- RunLog/Projects/{Data,Domain,Inject,Presentation,Util}/Project.swift 파일이 모두 존재하여 의존성 모듈이 정상입니다.
위 설정은 Tuist 모듈화 및 의존성 구성 가이드라인에 부합하므로 추가 조치가 필요 없습니다.
RunLog/Projects/Presentation/Sources/Run/View/RunHomeViewController.swift (3)
7-8: 모듈 import가 적절합니다모듈화된 구조에 맞게
RLDesignSystem과RLUtil모듈을 import한 것이 올바릅니다.
139-139: 네비게이션 바 설정이 명시적으로 개선되었습니다이전의 파라미터 없는 호출에서 명시적인 폰트와 색상 파라미터를 전달하는 방식으로 변경되어 더 명확하고 유지보수하기 좋은 코드가 되었습니다.
144-154: 탭바 설정과 타이틀 설정이 올바르게 리팩토링되었습니다
setupTabBarAppearance에 명시적인 틴트 컬러를 전달하고, 커스텀 UILabel을 생성하여setupTitle(label:)메서드로 설정하는 방식이 이전의setupLogoTitle()방식보다 더 유연하고 명확합니다.RunLog/Projects/Presentation/Sources/Setting/ViewController/MyPageViewController.swift (2)
7-7: RLUtil 모듈 import가 적절합니다모듈화된 구조에서 네비게이션 바 설정 유틸리티를 사용하기 위한 import가 올바릅니다.
68-85: 네비게이션 바 설정이 명시적으로 개선되었습니다이전의 암시적인 설정 방식을 명시적인 파라미터를 가진
setupAppearance메서드로 변경하고, 커스텀 UILabel을 생성하여setupTitle(label:)메서드로 타이틀을 설정하는 방식이 더 명확하고 유지보수하기 좋습니다.RunLog/Projects/Presentation/Sources/Setting/ViewController/ChangeCalUnitViewController.swift (2)
7-8: 모듈 import가 적절합니다모듈화된 구조에 맞게
RLDesignSystem과RLUtil모듈을 import한 것이 올바릅니다.
73-78: 네비게이션 바 appearance 설정이 개선되었습니다명시적인 파라미터를 통한 네비게이션 바 스타일 설정이 이전의 암시적 방식보다 더 명확하고 유지보수하기 좋습니다.
RunLog/Projects/Presentation/Sources/Setting/ViewController/ChangeNicknameViewController.swift (2)
7-7: 모듈화에 따른 유틸리티 모듈 임포트 추가가 적절합니다.RLUtil 모듈 임포트가 추가되어 네비게이션 바 설정에 필요한 유틸리티 메서드들을 사용할 수 있게 되었습니다.
7-7: 모듈화에 따른 적절한 import 추가RLUtil 모듈의 import가 올바르게 추가되어 UINavigationController 확장 메서드들을 사용할 수 있게 되었습니다.
RunLog/Projects/Presentation/Sources/Log/View/EditLog/EditLogInfoViewController.swift (3)
7-7: 모듈화에 따른 일관된 임포트 추가입니다.다른 뷰 컨트롤러와 동일하게 RLUtil 모듈을 임포트하여 일관성을 유지하고 있습니다.
73-96: 네비게이션 바 설정이 일관된 패턴으로 개선되었습니다.
ChangeNicknameViewController와 동일한 패턴으로 네비게이션 바 설정이 개선되어 코드베이스 전반의 일관성이 유지되고 있습니다. 명시적인 파라미터 전달과 Combine 기반 이벤트 처리가 적절합니다.
7-7: 모듈 import 일관성 확인RLUtil 모듈의 import가 일관되게 적용되어 있습니다.
RunLog/Projects/Domain/Sources/Entity/AppConfig.swift (1)
10-30: 도메인 엔티티 구조가 깔끔하고 적절합니다.앱 설정 정보를 담는 도메인 엔티티로서 필요한 속성들을 잘 정의하고 있습니다. public 접근 제어자는 모듈 간 사용을 위해 적절하며, Equatable 프로토콜 준수도 좋습니다.
RunLog/Projects/Domain/Sources/Entity/Section.swift (2)
10-20: Section 엔티티 구조가 적절합니다.러닝 경로의 구간을 나타내는 엔티티로서 거리, 걸음 수, 경로 포인트 배열을 적절히 포함하고 있습니다. public 접근 제어자와 Equatable 준수가 모듈화에 적합합니다.
22-32: Point 엔티티 구조가 위치 데이터 표현에 적합합니다.위도, 경도, 타임스탬프를 포함한 지리적 좌표를 나타내는 구조체로서 GPS 트래킹에 필요한 모든 정보를 담고 있습니다.
RunLog/Projects/App/InfoPlists/Info.plist (1)
9-10: 앱 구성 설정이 적절하게 업데이트됨CFBundleDisplayName과 UILaunchStoryboardName 추가가 적절합니다. 모듈화 과정에서 폰트를 프로그래매틱 방식으로 등록하는 것으로 변경한 것도 좋은 아키텍처 결정입니다.
Also applies to: 83-84
RunLog/Projects/Util/Sources/Extension/Double+.swift (1)
12-12: 모듈화를 위한 적절한 public 접근 제어자 추가유틸리티 확장 함수들을 public으로 노출하여 다른 모듈에서 사용할 수 있도록 한 것이 적절합니다. 기존 로직은 변경되지 않아 안전하며, 테스트 커버리지도 존재하는 것으로 확인됩니다.
Also applies to: 17-17, 24-24, 37-37, 40-40
RunLog/Projects/Util/Sources/Extension/UITapGestureRecognizer+.swift (1)
13-15: Combine Publisher 인터페이스의 적절한 public 노출TapPublisher와 관련 타입들을 public으로 노출하면서도 내부 구현인 TapGestureSubscription은 private으로 유지한 것이 좋은 캡슐화 설계입니다. 외부에서 사용할 수 있는 인터페이스만 노출하고 구현 세부사항은 숨겨두었습니다.
Also applies to: 23-23, 63-63
RunLog/Projects/Util/Sources/Extension/MKMapView+.swift (1)
18-22: 맵 유틸리티 메서드의 적절한 public 노출지도 카메라 위치 설정과 줌 레벨 초기화 메서드들을 public으로 노출한 것이 적절합니다. 메서드들이 잘 문서화되어 있고 프레젠테이션 레이어에서 사용할 수 있도록 모듈화 목표에 부합합니다.
Also applies to: 54-54
RunLog/Projects/Util/Sources/Extension/Int+.swift (1)
12-12: Int 유틸리티 확장의 적절한 public 노출숫자 포맷팅과 날씨/레벨 설명 변환 메서드들을 public으로 노출한 것이 적절합니다. 다양한 모듈에서 사용될 수 있는 유용한 유틸리티 기능들이며, 적절한 기본값 처리도 포함되어 있습니다.
Also applies to: 19-19, 29-29
RunLog/Projects/Util/Sources/Helpers/DynamicSize.swift (2)
8-8: UIKit import 추가가 적절합니다.모듈화로 인해 CGFloat과 CGRect 타입 사용을 위한 UIKit import가 필요하며, 이는 올바른 변경입니다.
16-16: public 접근 제어자 변경이 모듈화에 적합합니다.유틸리티 모듈의 핵심 기능들을 다른 모듈에서 사용할 수 있도록 public으로 변경한 것이 모듈화 목적에 부합합니다.
Also applies to: 33-33, 63-63
RunLog/Projects/Data/Sources/DataContainer/CoreDataContainer.swift (1)
10-12: public 클래스 선언과 초기화가 모듈화에 적합합니다.Data 모듈의 Core Data 컨테이너를 다른 모듈에서 접근할 수 있도록 public으로 선언한 것이 적절합니다.
RunLog/Projects/Presentation/Sources/Common/View/MainTabBarController.swift (2)
7-7: RLDesignSystem 모듈 import가 적절합니다.RLIcon 타입 사용을 위해 RLDesignSystem 모듈을 import한 것이 올바른 의존성 관리입니다.
54-54: public 접근 제어자 변경이 모듈화 구조에 맞습니다.Presentation 모듈의 MainTabBarController를 App 모듈에서 사용할 수 있도록 public으로 변경한 것이 적절합니다. 기존 로직은 그대로 유지되어 기능상 문제가 없습니다.
Also applies to: 58-58, 66-66
RunLog/Projects/Data/Sources/Managers/PedometerManager.swift (1)
13-13: 매니저 클래스의 public 접근 제어자 변경이 적절합니다.PedometerManager의 클래스, 싱글톤 인스턴스, Input/Output 열거형, 그리고 Subject들을 public으로 변경하여 다른 모듈에서 접근 가능하도록 한 것이 데이터 레이어 모듈화에 적합합니다. 기존 로직은 변경되지 않아 안전합니다.
Also applies to: 16-16, 23-23, 27-27, 30-30, 33-33
RunLog/Projects/Data/Sources/Network/OpenWeather/OpenWeatherService.swift (1)
15-15: 네트워크 서비스의 public 접근 제어자 변경이 모듈화에 적합합니다.OpenWeatherService의 클래스, 싱글톤 인스턴스, Input/Output 열거형, 그리고 Subject들을 public으로 변경하여 외부 모듈에서 날씨 데이터 서비스를 사용할 수 있도록 한 것이 데이터 레이어의 모듈 간 통신에 적절합니다. 네트워크 요청 로직은 그대로 유지되어 기능상 안전합니다.
Also applies to: 18-18, 25-25, 28-28, 31-31, 34-34
RunLog/Projects/Presentation/Sources/Log/View/LogViewController.swift (1)
76-92: 모듈화에 맞춘 네비게이션 바 설정 개선을 승인합니다.새로운
UINavigationController확장 메서드를 사용하여 네비게이션 바를 더 명시적으로 설정하도록 개선되었습니다. 색상, 폰트, 틴트 컬러를 명확히 지정하고, 커스텀 UILabel로 타이틀을 설정하는 방식이 일관성 있고 유지보수하기 좋습니다.RunLog/Projects/DesignSystem/Sources/IconSystem.swift (4)
8-8: UIKit import 추가를 승인합니다.UIImage extension을 위해 UIKit import가 추가되었습니다.
12-12: RLIcon enum의 public 접근 제어자 변경을 승인합니다.모듈화에 따라 external access를 위해 public으로 변경된 것이 적절합니다.
51-51: name 속성의 public 접근 제어자 변경을 승인합니다.RLIcon enum과 일관성을 위해 name 속성도 public으로 변경된 것이 적절합니다.
122-126: UIImage extension의 designSystem 메서드 추가를 승인합니다.모듈별 이미지 로딩을 위한 깔끔한 패턴입니다.
.module을 사용하여 DesignSystem 모듈의 번들에서 올바르게 이미지를 로드합니다.RunLog/Projects/Presentation/Sources/DetailLog/View/DetailLogViewController.swift (3)
7-9: 모듈 import 추가를 승인합니다.모듈화에 따라 RLDomain, RLDesignSystem, RLUtil 모듈들이 올바르게 import되었습니다.
83-88: 네비게이션 바 appearance 설정 개선을 승인합니다.새로운 setupAppearance 메서드를 사용하여 배경색, 전경색, 폰트, 틴트 컬러를 명시적으로 설정하는 방식이 일관성 있고 유지보수하기 좋습니다.
92-111: UIKit 네이티브 메뉴로의 변경을 승인합니다.Combine 기반 메뉴에서 UIKit의 네이티브 UIMenu와 UIAction을 사용하는 방식으로 변경되어 더 간단하고 직관적입니다. 각 액션이 직접 뷰모델 input으로 연결되어 명확합니다.
RunLog/Projects/Util/Sources/Extension/Date+.swift (3)
10-10: toYearMonthDay 속성의 public 접근 제어자 변경을 승인합니다.모듈화에 따라 외부 모듈에서 날짜 유틸리티를 사용할 수 있도록 public으로 변경된 것이 적절합니다.
20-20: formattedString 메서드의 public 접근 제어자 변경을 승인합니다.다른 모듈에서 날짜 포맷팅 기능을 사용할 수 있도록 public으로 변경된 것이 적절합니다.
29-29: DateFormatStyle enum의 public 접근 제어자 변경을 승인합니다.formattedString 메서드와 함께 사용되는 enum이므로 public으로 변경된 것이 일관성 있고 적절합니다.
RunLog/Projects/DesignSystem/Sources/LabelSystem.swift (6)
7-7: RLUtil 모듈 import 추가를 승인합니다.유틸리티 확장 메서드들을 사용하기 위해 RLUtil 모듈이 올바르게 import되었습니다.
14-14: RLLabel 클래스의 public 접근 제어자 변경을 승인합니다.모듈화에 따라 외부 모듈에서 이 UI 컴포넌트를 사용할 수 있도록 public으로 변경된 것이 적절합니다.
16-17: icon, label 속성들의 public 접근 제어자 변경을 승인합니다.외부에서 이 속성들에 접근할 수 있도록 public으로 변경된 것이 적절합니다.
20-20: attributedText 속성의 public 접근 제어자 변경을 승인합니다.외부 모듈에서 attributed text를 설정할 수 있도록 public으로 변경된 것이 적절합니다.
31-31: tintColor override의 public 접근 제어자 변경을 승인합니다.외부 모듈에서 틴트 컬러를 설정할 수 있도록 public으로 변경된 것이 적절합니다.
75-76: 개별 addSubview 호출로의 변경을 승인합니다.
addSubviews에서 개별addSubview호출로 변경되어 더 명시적이고 표준적인 접근 방식입니다.RunLog/Projects/Domain/Sources/UseCaseImpl/DayLogUseCaseImpl.swift (2)
7-7: 모듈화를 위한 public 접근 제어자 추가 승인Domain 레이어의 UseCase 구현체가 다른 모듈에서 사용될 수 있도록 public 접근 제어자를 추가한 것이 적절합니다. RLUtil 모듈 import도 필요한 변경사항입니다.
Also applies to: 12-12, 17-17, 25-25, 65-65, 74-74, 81-81, 87-87, 118-118, 127-127, 138-138, 147-147, 158-158, 173-173
182-182: 에러 타입 추상화 개선 승인
CoreDataError.modelNotFound에서DataError.modelNotFound로 변경한 것은 올바른 추상화입니다. Domain 레이어는 Core Data의 구체적인 구현 세부사항을 알 필요가 없어야 합니다.Also applies to: 209-209
RunLog/Projects/Util/Sources/Extension/UIControl+.swift (1)
13-28: Combine 퍼블리셔 확장의 public 접근성 승인UIControl의 Combine 통합을 위한 확장이 올바르게 public으로 노출되었습니다. EventPublisher 구조체와 관련 타입들의 public 접근 제어자가 모듈 간 사용을 위해 적절히 설정되었습니다.
RunLog/Projects/Data/Sources/Managers/DistanceManager.swift (1)
14-33: 거리 측정 매니저의 모듈화 지원 승인DistanceManager의 public 접근성 변경이 적절합니다. 싱글톤 패턴, Input/Output 열거형, 그리고 Combine Subject들이 올바르게 public으로 노출되어 다른 모듈에서 사용할 수 있도록 구성되었습니다.
RunLog/Projects/Util/Sources/Extension/String+.swift (1)
12-32: 텍스트 스타일링 메서드의 public 접근성과 명시적 매개변수 승인메서드가 public으로 변경되고 기본값을 제거하여 명시적 매개변수를 요구하는 것이 모듈 간 사용에 적합한 API 설계입니다. 인터페이스가 더 명확해졌습니다.
RunLog/Projects/Data/Sources/Impls/AppConfigRepositoryImpl.swift (1)
7-7: 앱 설정 리포지토리의 모듈화 지원 승인AppConfigRepositoryImpl의 public 접근성 변경과 RLDomain 모듈 import가 적절합니다. 모든 CRUD 메서드가 올바르게 public으로 노출되어 Domain 레이어의 UseCase에서 사용할 수 있도록 구성되었습니다. 도메인별 에러 타입 사용과 Core Data 스레드 안전성 처리도 잘 되어 있습니다.
Also applies to: 12-12, 15-15, 19-19, 45-45, 69-69, 100-100
RunLog/Projects/Util/Sources/Extension/UINavigationController+.swift (3)
14-37: 네비게이션 바 스타일 설정 메서드가 잘 구현되었습니다.메서드는 모든 appearance 상태(standard, scrollEdge, compact)를 일관되게 설정하고 있으며, 백 버튼의 제목을 제거하는 등 세심한 처리가 되어 있습니다.
39-41: 타이틀 설정 메서드 구현 확인.간단하고 명확한 구현입니다.
topViewController의 nil 체크는 암시적으로 처리되어 안전합니다.
44-46: 오른쪽 버튼 설정 메서드 구현 확인.구현이 간단하고 적절합니다.
topViewController가 nil일 경우 graceful하게 처리됩니다.RunLog/Projects/Domain/Sources/Entity/DayLog.swift (2)
10-23: DayLog 엔티티 구조가 잘 설계되었습니다.일관된 접근 제어자 사용과 적절한 프로퍼티 분류(불변 vs 가변)가 잘 되어 있습니다.
Equatable준수로 비교 연산도 지원합니다.
24-49: public 초기화 메서드 구현이 적절합니다.모든 프로퍼티를 매개변수로 받는 완전한 초기화 메서드로, 모듈 간 사용에 적합합니다.
RunLog/Projects/Domain/Sources/UseCaseImpl/MediaUseCaseImpl.swift (5)
7-7: RLDesignSystem 임포트가 추가되었습니다.
UIColor.LightGreen사용을 위한 적절한 의존성 추가입니다.
13-15: MediaUseCaseImpl 클래스가 public으로 변경되었습니다.모듈화에 따른 적절한 접근 제어자 변경입니다. 초기화 메서드도 public으로 일관되게 설정되었습니다.
17-30: 좌표 변환 로직이 올바르게 구현되었습니다.섹션별로 타임스탬프 기준 정렬을 통해 경로의 순서를 보장하는 것이 좋은 접근입니다.
32-78: 비동기 이미지 생성 로직이 잘 구현되었습니다.
withCheckedContinuation을 사용한 비동기 처리와 적절한 에러 핸들링이 잘 되어 있습니다. 스냅샷 실패 시 예외 던지기도 적절합니다.
80-90: 파일 저장 메서드가 적절하게 구현되었습니다.Document 디렉터리 접근과 에러 처리가 올바르게 되어 있습니다.
RunLog/Projects/Inject/Sources/DIContainer.swift (3)
10-12: DIContainer 클래스가 public으로 적절하게 변경되었습니다.모듈 간 의존성 주입을 위한 필수적인 변경사항입니다.
final키워드로 상속을 방지한 것도 좋습니다.
18-29: 의존성 등록 및 해결 메서드가 public으로 변경되었습니다.타입 안전성을 위한 제네릭 사용과 적절한 에러 메시지가 포함된 구현입니다. 강제 캐스팅은 DI 컨테이너에서 일반적인 패턴입니다.
34-40: Dependency 프로퍼티 래퍼가 public으로 변경되었습니다.모듈 간 의존성 주입 사용을 위한 적절한 변경입니다. 초기화 시점에서 의존성을 해결하는 방식이 적절합니다.
RunLog/Projects/Data/Sources/Managers/DrawingManager.swift (3)
12-19: DrawingManager 싱글톤 구현이 적절합니다.public 접근 제어자 변경과 함께 초기화 시 bind() 호출로 올바른 설정이 되어 있습니다.
22-32: Input/Output 열거형과 Subject가 public으로 적절하게 변경되었습니다.Combine을 활용한 반응형 프로그래밍 패턴이 잘 구현되어 있습니다. 케이스별로 명확한 의미를 가지고 있습니다.
71-82: MapView 델리게이트 메서드가 public으로 변경되었습니다.폴리라인 렌더링을 위한 적절한 구현입니다. 색상과 선 두께 설정이 명확합니다.
RunLog/Projects/DesignSystem/Sources/FontSystem.swift (4)
7-7: RLUtil 모듈 import 승인DesignSystem 모듈에서 RLUtil 모듈의 유틸리티 기능을 사용하기 위한 import가 적절합니다.
37-37: public 접근 제어자 변경 승인모듈화에 따라 RLFont의
value및lineHeightMultiple속성을 public으로 변경한 것이 적절합니다. 다른 모듈에서 접근할 수 있도록 합니다.Also applies to: 75-75
98-117: CaseIterable 프로토콜 추가 승인폰트 등록 시스템을 위해 각 폰트 enum에 CaseIterable 프로토콜을 추가한 것이 적절합니다. 동적 폰트 등록에 필요한 변경사항입니다.
206-237: 폰트 동적 등록 시스템 우수한 구현CTFontManagerRegisterFontsForURL을 사용한 폰트 동적 등록 구현이 훌륭합니다. 모듈 번들에서 폰트 파일을 찾아 시스템에 등록하는 로직이 견고하며, 오류 처리와 로깅도 적절히 구현되어 있습니다.
주요 장점:
- otf와 ttf 확장자 모두 지원
- 적절한 오류 처리 및 로깅
- 모듈 번들을 통한 리소스 접근
RunLog/Projects/Data/Sources/Managers/LocationManager.swift (4)
14-14: LocationManager 클래스 및 싱글톤 public 접근 승인모듈화에 따라 LocationManager 클래스와 shared 싱글톤 인스턴스를 public으로 변경한 것이 적절합니다. 다른 모듈에서 위치 서비스에 접근할 수 있도록 합니다.
Also applies to: 17-17
30-34: Input/Output enum 및 Subject public 접근 승인Combine 기반의 Input/Output 패턴을 위해 enum과 PassthroughSubject를 public으로 변경한 것이 적절합니다. 외부 모듈에서 LocationManager와 상호작용할 수 있도록 합니다.
Also applies to: 37-41
85-112: 위치 업데이트 delegate 메서드 구현 우수GPS 신호 안정성 검사와 최소 이동 거리 필터링을 통한 위치 업데이트 로직이 훌륭합니다. 불안정한 GPS 신호 필터링과 1m 이하 이동 무시로 배터리 효율성과 정확성을 모두 고려했습니다.
133-138: 권한 변경 delegate 메서드 public 접근 승인CLLocationManagerDelegate 메서드를 public으로 변경한 것이 모듈 간 접근성을 위해 필요한 변경사항입니다.
RunLog/Projects/Util/Sources/Extension/UIViewController+.swift (4)
18-18: UIViewController 확장 메서드 public 접근 승인모듈화에 따라 UIViewController 확장 메서드들을 public으로 변경한 것이 적절합니다. 다른 모듈에서 이러한 유틸리티 메서드들을 사용할 수 있도록 합니다.
Also applies to: 40-40, 76-76, 86-86
25-25: 로딩 인디케이터 색상 변경 확인 필요
.LightGreen에서.green으로 색상이 변경되었습니다. 이것이 의도된 변경인지 확인이 필요합니다. 브랜드 색상 일관성을 위해 기존.LightGreen색상을 유지하는 것이 좋을 수 있습니다.
48-61: 네비게이션 바 설정 매개변수화 우수한 개선
setupNavigationBarAppearance메서드에titleFont와titleColor매개변수를 추가하여 유연성을 크게 향상시켰습니다. 이는 다양한 화면에서 다른 폰트와 색상을 사용할 수 있게 하는 훌륭한 개선사항입니다.
64-73: 탭 바 설정 매개변수화 승인
setupTabBarAppearance메서드에tintColor매개변수를 추가하여 탭 바 색상을 동적으로 설정할 수 있게 한 것이 좋은 개선사항입니다.RunLog/Projects/Data/Sources/Impls/DayLogRepositoryImpl.swift (4)
7-7: RLDomain 모듈 import 승인Data 레이어에서 Domain 모듈의 엔티티와 프로토콜을 사용하기 위한 import가 적절합니다.
12-12: DayLogRepositoryImpl 클래스 public 접근 승인모듈화에 따라 DayLogRepositoryImpl 클래스와 초기화자를 public으로 변경한 것이 적절합니다. Domain 레이어의 UseCase에서 접근할 수 있도록 합니다.
Also applies to: 16-16
32-32: DataError로의 오류 타입 마이그레이션 승인
CoreDataError에서DataError로 오류 타입을 변경한 것이 새로운 모듈 구조에 적합합니다.
modelAlreadyExist: 중복 데이터 생성 시modelNotFound: 데이터 조회/수정/삭제 시 대상 없음conversionError: DTO to Entity 변환 실패 시오류 처리가 일관되고 적절합니다.
Also applies to: 52-52, 55-55, 86-86, 114-114
20-39: CRUD 작업 구현 우수DayLog의 생성, 조회, 수정, 삭제 작업이 올바르게 구현되어 있습니다:
- 비동기 Core Data 작업을 위한
context.perform사용- 적절한 NSPredicate를 통한 데이터 필터링
- DataMapper 사용으로 DTO-Entity 변환 분리
- 중복 생성 방지 로직
모든 메서드가 public으로 적절히 노출되었습니다.
Also applies to: 41-60, 62-73, 75-101, 103-121
RunLog/Projects/Domain/Sources/UseCaseImpl/AppConfigUseCaseImpl.swift (4)
7-7: RLUtil 모듈 import 승인Domain 레이어에서 RLUtil 모듈의 Constants 등 유틸리티 기능을 사용하기 위한 import가 적절합니다.
11-11: AppConfigUseCaseImpl 클래스 public 접근 승인모듈화에 따라 AppConfigUseCaseImpl 클래스와 초기화자를 public으로 변경한 것이 적절합니다. Presentation 레이어나 다른 모듈에서 의존성 주입을 통해 접근할 수 있도록 합니다.
Also applies to: 15-15
28-33: UseCase 메서드 public 접근 승인모든 UseCase 메서드들을 public으로 변경한 것이 적절합니다. 각 메서드는 적절한 비동기 처리와 오류 전파를 수행하고 있습니다:
getUnitDistance(),getNickname(),getUserIndicators()등 조회 메서드updateUnitDistance(),updateNickname()등 수정 메서드Also applies to: 35-40, 42-47, 49-54, 72-79, 81-89
56-70: 거리 지표 계산 로직 우수한 구현
getDistanceIndicators()메서드의 구현이 훌륭합니다:
Constants.allRoads에서 사용자 총 거리의 50% 이상인 도로들을 필터링- 랜덤하게 선택하여 다양성 제공
- 기본값으로 마라톤 거리 사용
사용자에게 의미 있는 거리 비교 지표를 제공하는 좋은 로직입니다.
RunLog/Projects/Util/Sources/Constant/Constants.swift (6)
10-14: Road 구조체의 모듈화가 적절히 구현되었습니다.Domain 엔티티에서 Util 모듈로 이동하면서 public 접근 제어자와 iconName 프로퍼티 변경이 올바르게 적용되었습니다. 기존
icon: String에서iconName: String으로 명명이 더 명확해졌습니다.
29-59: MotivationMessage 열거형의 public 노출이 올바르게 구현되었습니다.아이콘 참조가
RLIcon열거형에서 문자열 기반iconName으로 변경되었고, 모든 멤버가 적절히 public으로 선언되었습니다. 기능적으로는 동일하게 작동합니다.
61-84: LocationMessage 열거형이 모듈 간 사용을 위해 적절히 수정되었습니다.모든 public 멤버가 올바르게 선언되어 다른 모듈에서 접근 가능하며, 기존 기능이 그대로 유지되고 있습니다.
87-122: WeatherCondition 열거형의 모듈화가 완료되었습니다.날씨 조건 매핑 로직과 설명 문자열이 그대로 유지되면서 public 접근성이 추가되어 다른 모듈에서의 사용이 가능합니다.
125-148: AqiLevel 열거형이 올바르게 public으로 노출되었습니다.대기질 수준 매핑과 설명 기능이 보존되면서 모듈 간 접근이 가능하도록 적절히 수정되었습니다.
16-27: DesignSystem RLIcon enum 케이스 일치 여부 확인 필요Constants.swift에서 allRoads 배열의 iconName을 기존
RLIcon.xxx.name호출에서 문자열("Medal","Flag"등)로 변경하셨습니다. 이 값들이 DesignSystem 모듈의 RLIcon enum 케이스명과 완전히 일치하는지 반드시 검증해주세요.검증 항목:
- RLIcon enum에 아래 케이스가 모두 존재하는지 (대소문자 포함 정확히 매칭)
• Medal
• Flag
• Mandarin
• Korea
• Church
• Earth
• Rocket- 향후 RLIcon 케이스 추가·삭제 시, Constants 내 문자열도 함께 업데이트될지 확인
자동화 스크립트로는 외부 모듈을 조회하기 어려우니, DesignSystem 모듈 레포지토리에서 직접 아래 명령어로 확인 바랍니다:
# RLIcon enum 정의 위치 찾기 grep -R "enum RLIcon" -n . # 케이스 목록 추출 grep -R "case " -n <DesignSystem 모듈 경로>/Sources
| import RLDomain | ||
| import RLData | ||
| import RLInject |
There was a problem hiding this comment.
RLDesignSystem 모듈 import 누락으로 컴파일 오류 가능성
UIFont.registerFonts() 확장은 DesignSystem 모듈에서 정의된 것으로 보입니다.
해당 모듈을 re-export 하지 않는 한, 아래와 같이 명시적으로 임포트해 주세요.
import RLDomain
import RLData
import RLInject
+import RLDesignSystemAlso applies to: 19-20
🤖 Prompt for AI Agents
In RunLog/Projects/App/Sources/AppDelegate.swift at lines 7-9 and also lines
19-20, the RLDesignSystem module is missing from the imports, which may cause
compile errors because UIFont.registerFonts() extension is defined there. Add an
explicit import statement for RLDesignSystem at the top of the file alongside
the existing imports to resolve this issue.
| guard let modelURL = Bundle.module.url(forResource: "DTOs", withExtension: "momd"), | ||
| let model = NSManagedObjectModel(contentsOf: modelURL) else { | ||
| fatalError("Failed to load Core Data model named DTOs") | ||
| } |
There was a problem hiding this comment.
fatalError 사용으로 인한 앱 크래시 위험이 있습니다.
Core Data 모델 로딩 실패나 저장소 로딩 실패 시 fatalError로 앱이 강제 종료됩니다. 프로덕션 환경에서는 더 우아한 에러 처리가 필요합니다.
더 안전한 에러 처리 방식을 제안합니다:
-public final class CoreDataContainer {
+public final class CoreDataContainer {
+ public enum CoreDataError: Error {
+ case modelLoadFailed
+ case storeLoadFailed(Error)
+ }
- lazy var persistentContainer: NSPersistentContainer = {
+ lazy var persistentContainer: Result<NSPersistentContainer, CoreDataError> = {
guard let modelURL = Bundle.module.url(forResource: "DTOs", withExtension: "momd"),
let model = NSManagedObjectModel(contentsOf: modelURL) else {
- fatalError("Failed to load Core Data model named DTOs")
+ return .failure(.modelLoadFailed)
}
let container = NSPersistentContainer(name: "DTOs", managedObjectModel: model)
+ var loadError: Error?
- container.loadPersistentStores { _, error in
+ container.loadPersistentStores { _, error in
+ loadError = error
- if let error = error as NSError? {
- fatalError("Unresolved Core Data error: \(error), \(error.userInfo)")
- }
}
+ if let error = loadError {
+ return .failure(.storeLoadFailed(error))
+ }
- return container
+ return .success(container)
}()Also applies to: 22-24
🤖 Prompt for AI Agents
In RunLog/Projects/Data/Sources/DataContainer/CoreDataContainer.swift around
lines 15 to 18 and 22 to 24, the use of fatalError for handling Core Data model
or store loading failures causes the app to crash abruptly. Replace fatalError
with a safer error handling approach by catching the failure, logging the error
details, and returning a failure state or throwing an error to allow the app to
handle it gracefully without crashing.
| public var context: NSManagedObjectContext { | ||
| return persistentContainer.viewContext | ||
| } |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Context 접근 방식을 개선할 수 있습니다.
현재 구조에서는 persistentContainer 로딩 실패 시에도 context에 접근하려고 시도할 수 있습니다.
Context 접근을 더 안전하게 만들 수 있습니다:
- public var context: NSManagedObjectContext {
- return persistentContainer.viewContext
- }
+ public var context: NSManagedObjectContext? {
+ switch persistentContainer {
+ case .success(let container):
+ return container.viewContext
+ case .failure:
+ return nil
+ }
+ }📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| public var context: NSManagedObjectContext { | |
| return persistentContainer.viewContext | |
| } | |
| public var context: NSManagedObjectContext? { | |
| switch persistentContainer { | |
| case .success(let container): | |
| return container.viewContext | |
| case .failure: | |
| return nil | |
| } | |
| } |
🤖 Prompt for AI Agents
In RunLog/Projects/Data/Sources/DataContainer/CoreDataContainer.swift around
lines 29 to 31, the context property directly returns
persistentContainer.viewContext without handling the case where
persistentContainer might fail to load. To improve safety, modify the context
accessor to safely unwrap or check the persistentContainer's availability before
returning its viewContext, possibly returning an optional or throwing an error
if the container is not loaded, to prevent accessing context when
persistentContainer is nil or invalid.
| let project = Project.framework( | ||
| name: Module.domain.name, | ||
| organizationName: Module.organizationName, | ||
| infoPlist: .default, | ||
| dependencies: [ | ||
| Module.util.project, | ||
| Module.designSystem.project | ||
| ] | ||
| ) |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Domain 모듈의 DesignSystem 의존성 검토 제안
Domain 모듈이 DesignSystem 모듈에 의존하고 있는데, 일반적으로 Domain 레이어는 UI 관련 요소에 의존하지 않는 것이 Clean Architecture 원칙에 부합합니다. Domain 레이어에서 실제로 DesignSystem의 어떤 부분을 사용하고 있는지 확인이 필요합니다.
Domain 레이어에서 DesignSystem 사용 현황을 확인해 보겠습니다:
🏁 Script executed:
#!/bin/bash
# Domain 소스에서 DesignSystem 관련 사용 패턴 확인
fd -e swift . RunLog/Projects/Domain/Sources --exec rg -l "RLDesignSystem|DesignSystem" {}Length of output: 160
🏁 Script executed:
#!/bin/bash
# Check for DesignSystem import in the identified file
rg -n "import DesignSystem" RunLog/Projects/Domain/Sources/UseCaseImpl/MediaUseCaseImpl.swift
# Check for RLDesignSystem usage within the file
rg -n "RLDesignSystem" RunLog/Projects/Domain/Sources/UseCaseImpl/MediaUseCaseImpl.swiftLength of output: 208
Domain 레이어에서 RLDesignSystem 의존성 제거 요청
Domain 모듈이 UI/디자인 시스템에 직접 의존하고 있습니다. Clean Architecture 원칙에 따라 Domain 레이어는 순수 비즈니스 로직만 담아야 하므로, 아래 위치에서 확인된 RLDesignSystem import를 제거하거나 추상화 계층으로 분리해 주세요.
• 파일: RunLog/Projects/Domain/Sources/UseCaseImpl/MediaUseCaseImpl.swift
– 라인 7: import RLDesignSystem
제안사항
- 해당 의존성이 실제로 어떤 기능에 쓰이는지 파악
- Domain 레이어에서는 RLDesignSystem API 대신 인터페이스(또는 도메인 모델)만 사용하도록 변경
- 디자인 시스템과의 구체적 연동 로직은 Infrastructure/UI 레이어로 이동
🤖 Prompt for AI Agents
In RunLog/Projects/Domain/Project.swift lines 4 to 12, the Domain module
currently depends on RLDesignSystem, violating Clean Architecture principles. To
fix this, first identify where RLDesignSystem APIs are used, specifically in
RunLog/Projects/Domain/Sources/UseCaseImpl/MediaUseCaseImpl.swift line 7 where
it is imported. Remove this import and replace any direct usage of
RLDesignSystem APIs with domain-specific interfaces or models. Then move any UI
or design system specific implementation details to the Infrastructure or UI
layer, ensuring the Domain layer contains only pure business logic without UI
dependencies.
There was a problem hiding this comment.
@Monfi98, 네! 감사합니다. Domain 레이어를 순수하게 유지하시면 더 깔끔한 아키텍처가 될 것 같아요. 수정 작업 진행하시면서 궁금한 점이 있으시면 언제든 말씀해 주세요! 👍
🧠 Learnings used
Learnt from: Monfi98
PR: ESTSOFT-iOS-01/RunLog#134
File: RunLog/Project.swift:11-16
Timestamp: 2025-07-16T08:01:39.578Z
Learning: RunLog 프로젝트에서 Project.swift 파일은 RunLog/ 디렉토리 안에 위치하며, 상대 경로 참조 시 "RunLog/InfoPlists/..."는 "RunLog/RunLog/InfoPlists/..."를 의미하는 올바른 구조입니다.
| public enum DataError: LocalizedError { | ||
| case fetchError | ||
| case deleteError | ||
| case modelNotFound | ||
| case modelAlreadyExist | ||
| case conversionError | ||
|
|
||
| var errorDescription: String { | ||
| switch self { | ||
| case .fetchError: | ||
| "Fetch Error" | ||
| case .deleteError: | ||
| "Delete Error" | ||
| case .modelNotFound: | ||
| "Model Not Found" | ||
| case .modelAlreadyExist: | ||
| "Model Already Exist" | ||
| case .conversionError: | ||
| "Conversion Error" | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
데이터 에러 열거형이 잘 구성되어 있지만 접근 제어자를 수정해야 합니다.
중앙 집중식 에러 처리를 위한 구조는 적절하나, errorDescription 속성에 public 접근 제어자가 누락되어 다른 모듈에서 접근할 수 없습니다.
다음과 같이 수정해주세요:
- var errorDescription: String {
+ public var errorDescription: String? {또한 한국어 앱이므로 에러 메시지의 한국어 지원도 고려해보시기 바랍니다:
case .fetchError:
- "Fetch Error"
+ "데이터 조회 오류"
case .deleteError:
- "Delete Error"
+ "데이터 삭제 오류"
case .modelNotFound:
- "Model Not Found"
+ "모델을 찾을 수 없음"
case .modelAlreadyExist:
- "Model Already Exist"
+ "모델이 이미 존재함"
case .conversionError:
- "Conversion Error"
+ "데이터 변환 오류"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| public enum DataError: LocalizedError { | |
| case fetchError | |
| case deleteError | |
| case modelNotFound | |
| case modelAlreadyExist | |
| case conversionError | |
| var errorDescription: String { | |
| switch self { | |
| case .fetchError: | |
| "Fetch Error" | |
| case .deleteError: | |
| "Delete Error" | |
| case .modelNotFound: | |
| "Model Not Found" | |
| case .modelAlreadyExist: | |
| "Model Already Exist" | |
| case .conversionError: | |
| "Conversion Error" | |
| } | |
| } | |
| } | |
| public enum DataError: LocalizedError { | |
| case fetchError | |
| case deleteError | |
| case modelNotFound | |
| case modelAlreadyExist | |
| case conversionError | |
| public var errorDescription: String? { | |
| switch self { | |
| case .fetchError: | |
| "데이터 조회 오류" | |
| case .deleteError: | |
| "데이터 삭제 오류" | |
| case .modelNotFound: | |
| "모델을 찾을 수 없음" | |
| case .modelAlreadyExist: | |
| "모델이 이미 존재함" | |
| case .conversionError: | |
| "데이터 변환 오류" | |
| } | |
| } | |
| } |
🤖 Prompt for AI Agents
In RunLog/Projects/Domain/Sources/Error/DataError.swift between lines 11 and 32,
the errorDescription property lacks a public access modifier, preventing access
from other modules. Add the public modifier to the errorDescription computed
property to make it accessible externally. Additionally, update the error
message strings to Korean to support the app's localization requirements.
| var errorDescription: String { | ||
| switch self { | ||
| case .fetchError: | ||
| "Fetch Error" | ||
| case .deleteError: | ||
| "Delete Error" | ||
| case .modelNotFound: | ||
| "Model Not Found" | ||
| case .modelAlreadyExist: | ||
| "Model Already Exist" | ||
| case .conversionError: | ||
| "Conversion Error" | ||
| } | ||
| } |
There was a problem hiding this comment.
에러 설명 접근 제어자 누락
LocalizedError 프로토콜을 올바르게 구현하려면 errorDescription 프로퍼티가 public이어야 합니다.
다음과 같이 수정해주세요:
- var errorDescription: String {
+ public var errorDescription: String? {
switch self {
case .fetchError:
- "Fetch Error"
+ return "Fetch Error"
case .deleteError:
- "Delete Error"
+ return "Delete Error"
case .modelNotFound:
- "Model Not Found"
+ return "Model Not Found"
case .modelAlreadyExist:
- "Model Already Exist"
+ return "Model Already Exist"
case .conversionError:
- "Conversion Error"
+ return "Conversion Error"
}
}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| var errorDescription: String { | |
| switch self { | |
| case .fetchError: | |
| "Fetch Error" | |
| case .deleteError: | |
| "Delete Error" | |
| case .modelNotFound: | |
| "Model Not Found" | |
| case .modelAlreadyExist: | |
| "Model Already Exist" | |
| case .conversionError: | |
| "Conversion Error" | |
| } | |
| } | |
| public var errorDescription: String? { | |
| switch self { | |
| case .fetchError: | |
| return "Fetch Error" | |
| case .deleteError: | |
| return "Delete Error" | |
| case .modelNotFound: | |
| return "Model Not Found" | |
| case .modelAlreadyExist: | |
| return "Model Already Exist" | |
| case .conversionError: | |
| return "Conversion Error" | |
| } | |
| } |
🤖 Prompt for AI Agents
In RunLog/Projects/Domain/Sources/Error/DataError.swift around lines 18 to 31,
the errorDescription property lacks the public access modifier required by the
LocalizedError protocol. Add the public keyword before var errorDescription to
make it publicly accessible and correctly implement the protocol.
| import Foundation | ||
| import UIKit | ||
|
|
||
| protocol DayLogUseCase { | ||
| public protocol DayLogUseCase { | ||
|
|
There was a problem hiding this comment.
🛠️ Refactor suggestion
Domain 계층에서 UIKit 의존 – 아키텍처 분리 고려
Domain 레이어가 UIKit과 UIImage에 직접 의존하면 테스트·플랫폼 독립성이 떨어집니다.
이미지 처리가 필요한 기능은 Application/Presentation 레이어로 옮기고 Domain은 데이터 구조만 다루도록 재설계하는 것이 Clean Architecture에 부합합니다.
🤖 Prompt for AI Agents
In RunLog/Projects/Domain/Sources/UseCase/DayLogUseCase.swift around lines 8 to
12, the Domain layer imports UIKit, creating a dependency on UI components which
reduces testability and platform independence. To fix this, remove the UIKit
import and any UIImage usage from the Domain layer. Refactor image handling
logic to reside in the Application or Presentation layers, ensuring the Domain
layer only manages pure data structures and business logic in line with Clean
Architecture principles.
| self.navigationController?.setupAppearance( | ||
| backgroundColor: .Gray900, | ||
| foregroundColor: .Gray000, | ||
| font: .RLHeadline1, | ||
| tintColor: .LightGreen | ||
| ) | ||
|
|
||
| let navigationButton = UIButton(type: .system) | ||
| let buttonTitle = "완료" | ||
| navigationButton.setTitle(buttonTitle, for: .normal) | ||
| navigationButton.titleLabel?.attributedText = .RLAttributedString( | ||
| text: buttonTitle, | ||
| font: .Label1, | ||
| color: .LightGreen, | ||
| align: .center | ||
| ) | ||
|
|
||
| navigationButton.publisher | ||
| .sink { [weak self] in | ||
| self?.validateAndSaveInfo() | ||
| } | ||
| .store(in: &cancellables) | ||
|
|
||
| self.navigationController?.setupRightButton(navigationButton) |
There was a problem hiding this comment.
🛠️ Refactor suggestion
코드 중복 패턴 발견
이 파일의 네비게이션 바 설정 코드가 ChangeNicknameViewController와 거의 동일합니다. 이런 중복은 유지보수성을 저해할 수 있습니다.
공통 네비게이션 바 설정 로직을 추상화하는 것을 권장합니다:
// UIViewController+Navigation.swift
extension UIViewController {
func setupStandardRightButton(
title: String,
action: @escaping () -> Void,
cancellables: inout Set<AnyCancellable>
) {
let button = UIButton(type: .system)
button.setTitle(title, for: .normal)
button.titleLabel?.attributedText = .RLAttributedString(
text: title,
font: .Label1,
color: .LightGreen,
align: .center
)
button.publisher
.sink { action() }
.store(in: &cancellables)
navigationController?.setupRightButton(button)
}
}🤖 Prompt for AI Agents
In
RunLog/Projects/Presentation/Sources/Log/View/EditLog/EditLogInfoViewController.swift
around lines 73 to 96, the navigation bar setup code duplicates logic found in
ChangeNicknameViewController, reducing maintainability. Refactor by extracting
the common navigation button setup into a UIViewController extension method like
setupStandardRightButton that accepts the button title, action closure, and
cancellables set. Replace the existing button creation and setup code with a
call to this new method to centralize and reuse the navigation bar configuration
logic.
| let navigationButton = UIButton(type: .system) | ||
| let buttonTitle = "완료" | ||
| navigationButton.setTitle(buttonTitle, for: .normal) | ||
| navigationButton.titleLabel?.attributedText = .RLAttributedString( | ||
| text: "완료", | ||
| font: .Label1, | ||
| color: .LightGreen, | ||
| align: .center | ||
| ) | ||
|
|
||
| navigationButton.publisher | ||
| .sink { [weak self] in | ||
| self?.validateAndSaveUnit() | ||
| } | ||
| .store(in: &cancellables) | ||
|
|
||
| navigationController?.setupRightButton(navigationButton) |
There was a problem hiding this comment.
🛠️ Refactor suggestion
네비게이션 버튼 설정을 개선해야 합니다
수동으로 UIButton을 생성하고 Combine을 사용한 이벤트 처리 방식은 좋지만, 버튼 타이틀을 두 번 설정하는 문제가 있습니다.
다음과 같이 수정하여 중복 설정을 제거하세요:
let navigationButton = UIButton(type: .system)
-let buttonTitle = "완료"
-navigationButton.setTitle(buttonTitle, for: .normal)
-navigationButton.titleLabel?.attributedText = .RLAttributedString(
+navigationButton.setAttributedTitle(.RLAttributedString(
text: "완료",
font: .Label1,
color: .LightGreen,
align: .center
-)
+), for: .normal)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| let navigationButton = UIButton(type: .system) | |
| let buttonTitle = "완료" | |
| navigationButton.setTitle(buttonTitle, for: .normal) | |
| navigationButton.titleLabel?.attributedText = .RLAttributedString( | |
| text: "완료", | |
| font: .Label1, | |
| color: .LightGreen, | |
| align: .center | |
| ) | |
| navigationButton.publisher | |
| .sink { [weak self] in | |
| self?.validateAndSaveUnit() | |
| } | |
| .store(in: &cancellables) | |
| navigationController?.setupRightButton(navigationButton) | |
| let navigationButton = UIButton(type: .system) | |
| navigationButton.setAttributedTitle(.RLAttributedString( | |
| text: "완료", | |
| font: .Label1, | |
| color: .LightGreen, | |
| align: .center | |
| ), for: .normal) | |
| navigationButton.publisher | |
| .sink { [weak self] in | |
| self?.validateAndSaveUnit() | |
| } | |
| .store(in: &cancellables) | |
| navigationController?.setupRightButton(navigationButton) |
🤖 Prompt for AI Agents
In
RunLog/Projects/Presentation/Sources/Setting/ViewController/ChangeCalUnitViewController.swift
around lines 80 to 96, the UIButton's title is set twice: once with setTitle and
again with attributedText, causing redundancy. Remove the setTitle call and only
set the attributedText property to define the button's title with the desired
style. This eliminates duplicate title settings while preserving the button's
appearance and functionality.
| let project = Project.app( | ||
| name: Module.app.name, | ||
| organizationName: Module.organizationName, | ||
| infoPlist: .file(path: "InfoPlists/info.plist"), |
There was a problem hiding this comment.
P2) 추후에 config파일 등 넣게 되면, SupportingFile끼리 폴더링하고 그 안에서 infoplist 위치시키는 방향으로 가면 될 것 같습니다
| dependencies: [ | ||
| Module.inject.project, | ||
| Module.domain.project, | ||
| Module.util.project, |
There was a problem hiding this comment.
P0) domain이 util을 가지고 있으면 해당 모듈을 data 모듈에서 사용할 수 있는 걸로 알고 있습니다만 요거 한번 체크해서 graph 한번 다시 구조화해야할 것 같네용
| let project = Project.framework( | ||
| name: Module.inject.name, | ||
| organizationName: Module.organizationName, | ||
| infoPlist: .default | ||
| ) |
There was a problem hiding this comment.
Q) inject 모듈이 분리되어야 하는 이유가 뭘까요? 추후에 DIContainer 외에 다른 파일이 추가될 가능성이 있을까요? 혹은 Util과 Inject의 목적이 다를까요?
There was a problem hiding this comment.
inject 관련은 PR내에 추가사항에 분리되어야하는 이유가 적혀있습니다! 아마 코드레빗이 주구장창써놔가지구 잘 안보이는 것 같습니다
There was a problem hiding this comment.
순환참조가 된다는 문제인거 같은데 그러면,, Inject를 나눠서 양방향으로 graph가 그려지게 만드는 것보다 가장 top module이 App이 되는 방향으로 수정해야할 것 같은데요.. 요거 어렵네요 😢
| Module.inject.project, | ||
| Module.domain.project, | ||
| Module.data.project, // 수정해야함 | ||
| Module.util.project, | ||
| Module.designSystem.project, | ||
| .external(name: "SnapKit"), | ||
| .external(name: "Then"), | ||
| .external(name: "NVActivityIndicatorView"), | ||
| .external(name: "NVActivityIndicatorViewExtended"), |
There was a problem hiding this comment.
P1) 마찬가지로 의존성 간의 graph 한 번 확인해보고 고민하는게 좋을 것 같습니다.
| let project = Project.framework( | ||
| name: Module.util.name, | ||
| organizationName: Module.organizationName, | ||
| infoPlist: .default, | ||
| dependencies: [ | ||
| .external(name: "SnapKit"), | ||
| ] | ||
| ) |
There was a problem hiding this comment.
P2) Util이 Snapkit을 가져야한다면 Util -> DesignSys -> Presentation 순의 의존성 그래프가 잘 작동할 수 있도록 타 모듈에서는 Snapkit 제거하셔도 될 것 같습니다. DesignSys, Presentation에 중복 의존성 그래프가 생기고, Util을 사용하는 domain이나 data에서도 snapkit에 대한 의존성이 추가되는 걸로 알고 있습니다.
doyeonk429
left a comment
doyeonk429
left a comment
There was a problem hiding this comment.
수고하셨습니다 ! 이미 개발한 프로젝트에 모듈화 추가하는게 가장 어려운 일인데 정말 수고하셨어요 !
GeunwoongSim
left a comment
GeunwoongSim
left a comment
There was a problem hiding this comment.
수고하셨습니다!! 공부할게 많네영...ㅋㅋㅋㅋ
|
후하,, 말씀해주신 부분은 제가 Pedometer부분 리팩토링하면서 의존성 한번 걷어보고 보겠습니다 ! |
💡 PR 유형
✏️ 변경 사항
모든 레이어를 모듈화 하였습니다.
🚨 관련 이슈
🧪 테스트
✅ 체크리스트
🔥 추가 설명 or 리뷰 포인트
추가 사항
이번에 모듈화를 진행하면서 기존의 것들 외에 App 안에 있던 DIContainer를 분리하면서 RLInject 모듈을 추가하였습니다.
App에 DIcontainer가 위치한다면 아래 그림처럼 되고,
tuist에서도 generate시에 Found circular dependency between targets 에러가 뜨게 됩니다.
따라서 RLInject로 분리하므로서 다음과 같은 형태를 띌 수 있습니다.(DIContainer 시점)
이외에는 카톡에서 따로 말씀 드렸던 것 처럼 Data Layer ↔ Presentation Layer 간에 의존성을 분리하는 방향으로 수정해주시면 될 것 같습니다.(현재는 Presentation에 import RLData 되어있어요)
Summary by CodeRabbit
신규 기능
버그 수정
리팩터
스타일
문서화
기타