Skip to content

Refactor/#33 modul migration#39

Merged
YeoSeongil merged 59 commits intodevelopfrom
refactor/#33-modulMigration
Feb 6, 2026
Merged

Refactor/#33 modul migration#39
YeoSeongil merged 59 commits intodevelopfrom
refactor/#33-modulMigration

Conversation

@YeoSeongil
Copy link
Member

✨ What’s this PR?

📌 관련 이슈 (Related Issue)


🧶 주요 변경 내용 (Summary)

  • 공용으로 사용할 Shared 모듈 추가 (Dependency, Extension, 상수, 매니저 등) 및 마이그레이션
  • Feature 모듈별 분리 및 마이그레이션
  • Domain -> Core로 모델명 변경 : 보통 Domain하면 클린아키텍처의 도메인 계층을 의미하기 때문에 오해의 소지가 있다고 판단하여 Core로 모델명 변경했습니다.

의존성 구조

graph TD
    Core
    Shared
    Feature

    Shared --> Core
    Feature --> Shared
    Feature --> Core
Loading
graph TD
    DesignSystem
    Feature

    Feature --> DesignSystem
Loading

각 피쳐는 shared와 core, designSystem에 의존합니다


📸 스크린샷 (Optional)


🧪 테스트 / 검증 내역

  • 실기기 동작 확인

💬 기타 공유 사항

  1. 원래는 Feature별로 Dependency 관리하는 모듈도 다 분리하려고 했는데, 너무 오버 엔지니어링 같아서 롤백 했습니다.
    스크린샷 2026-02-02 오후 1 04 48
  • 프로젝트 규모에 비해 오버 엔지니어링 같음
  • 아래 사진 처럼 굳이 같은 메소드를 따로 나눠서 관리해야할까? 하는 생각도 듦. 중복 메소드를 따로 분리해서 재구현 하는 것보다는 차라리 한 곳에서 관리하는게 더 편할듯 (SwiftDataClient 같은)..
스크린샷 2026-02-02 오후 1 05 18 스크린샷 2026-02-02 오후 1 05 24
  1. Shared 모듈
    스크린샷 2026-02-03 오전 1 51 28

    Shared 모듈은 앱 공용으로 사용하는 Extension, 상수, Dependency 등을 관리합니다. Sheet랑 TopAppBar는 후에 삭제 될 예정이지만 일단 공용으로 사용하고 있기 때문에 Shared 모듈에 넣어놨습니다.

  2. 각 Feature별 모듈
    스크린샷 2026-02-03 오전 1 51 35

    훌륭합니다.. 예제 앱, 개별 빌드, 개별 단위 테스팅 가능합니다. + 빌드속도 많이 개선 됨.

  3. 규모가 커서 점진적으로 리팩토링 합시다. 일단 릴리즈가 먼저니까 앱스토어 업데이트 하고 후에 네비게이션 수정, 디펜던시나 자잘한 부분 수정합시다.

앞으로 수정해야할 사항 (Todo)

  1. LinkNavigator 제거
  2. Reducer에 직접 구현 되어있는 것들 삭제(UserDefault가 직접 구현되어 있는 등)

🙇🏻‍♀️ 리뷰 가이드 (선택)

SwiftData, UserDefaults 디펜던시 구현
이 부분 추후 수정될 것 같아서 일단 Share 모듈에 넣어놨습니다.
이 부분 추후 수정될 것 같아서 일단 Share 모듈에 넣어놨습니다.
@YeoSeongil YeoSeongil self-assigned this Feb 6, 2026
@YeoSeongil YeoSeongil added ♻️ Refactor 코드 리팩토링 🗂️ Tuist Tuist 수정 labels Feb 6, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

Important

Review skipped

Too many files!

This PR contains 251 files, which is 101 over the limit of 150.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/#33-modulMigration

Comment @coderabbitai help to get the list of available commands and usage tips.

@YeoSeongil YeoSeongil merged commit 285dd8e into develop Feb 6, 2026
1 check passed
@dbsghdz1 dbsghdz1 deleted the refactor/#33-modulMigration branch February 10, 2026 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ Refactor 코드 리팩토링 🗂️ Tuist Tuist 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

♻️ Refactor: 각 모듈별 마이그레이션

2 participants