Skip to content

Icon Composer로 앱 로고를 넣습니다.#267

Merged
Kimyonhae merged 2 commits into
devfrom
refactor/logo-icon-composer
Jun 1, 2026
Merged

Icon Composer로 앱 로고를 넣습니다.#267
Kimyonhae merged 2 commits into
devfrom
refactor/logo-icon-composer

Conversation

@Kimyonhae

@Kimyonhae Kimyonhae commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

✨ 작업 요약

Icon Composer 앱 로고 적용(Tuist 설정) 및 디버그 모드 테스트 코드 임시 주석 처리

📋 구체적인 내용

  • 추가/변경된 동작:
  • ChaGok.icon 폴더를 App/Resources/ 내부로 이동하여 tuist generate 시에도 리소스 폴더가 유실되지 않고 Xcode 내비게이터에
    보존되도록 개선
  • Project.swift Target Settings(base)에 ASSETCATALOG_COMPILER_APPICON_NAME"ChaGok"으로 지정하여 Tuist 재생성 시에도
    앱 아이콘 빌드 설정이 영구 적용되도록 구축
  • Swift Macro(MLXHuggingFaceMacros 등) 및 호스트 아키텍처 이슈 등으로 인해 빌드/테스트가 깨지는 현상을 임시 우회하기 위해,
    디버그 모드 네트워크 실패 및 unknownError 테스트 코드를 주석 처리
  • 영향 받는 화면/모듈:
  • App 모듈 (앱 아이콘 리소스 및 Tuist 빌드 세팅)
  • Data 모듈 (디버그 모드 테스트 코드)

🧩 설계·구현 노트

  • 선택한 방식
  • Tuist 프로젝트 파일(.xcodeproj)이 빌드 시점마다 새로 빌드되는 구조이므로, 수동 설정을 배제하고 Project.swift 설정과 리소스
    디렉토리(Resources/**) 배치를 수정하여 빌드 환경을 자동화하고 견고히 함.
  • 외부 디펜던시의 Swift Macro 컴파일 오류로 빌드가 정체되는 것을 방지하기 위해 디버그 테스트 일부를 안전하게 주석 처리.

✅ 확인 사항

  • 정상 플로우 동작 확인 (tuist generate 후 Xcode 상에서 리소스 유지 확인)
  • 엣지 케이스 / 빈 값·에러 처리 확인
  • (UI 변경 시) 스크린샷 또는 GIF 첨부
  • (로직 추가 시) 테스트 추가 여부

👀 리뷰 포인트

  • 설계·구조 적절성 (Tuist 리소스 경로 분리 및 빌드 세팅 정의)
  • 로직·엣지 케이스
  • 예외/에러 핸들링 (빌드 에러 임시 대응 검토)

Kimyonhae added 2 commits June 1, 2026 23:02
- tuist: app icon Nmae 설정
- 전체 루트에 넣으면 tuist generate와 함께 로고가 사라지므로 ReSource 내부에 저장합니다.
@Kimyonhae Kimyonhae self-assigned this Jun 1, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request configures the app icon name to "ChaGok" in the project settings and adds the corresponding SVG asset and configuration files. Additionally, it comments out the debug error simulation blocks in both MLXModelProvider.swift and WhisperKitProvider.swift. The reviewer recommends removing these commented-out blocks entirely to improve code readability and maintainability, suggesting they be replaced with a TODO comment or controlled via custom compiler flags.

@Kimyonhae Kimyonhae added the 🚀 Enhancement 기존 기능 개선, UX/옵션 추가 등 label Jun 1, 2026
@tuist

tuist Bot commented Jun 1, 2026

Copy link
Copy Markdown

🛠️ Tuist Run Report 🛠️

Tests 🧪

Scheme Status Cache hit rate Tests Skipped Ran Commit
ChaGok-Workspace 0 % 1 3 -2 fdf599652

Builds 🔨

Scheme Status Duration Commit
ChaGok-Workspace 6m 28s fdf599652

@Kimyonhae Kimyonhae merged commit 04f59a8 into dev Jun 1, 2026
2 checks passed
@Kimyonhae Kimyonhae deleted the refactor/logo-icon-composer branch June 1, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀 Enhancement 기존 기능 개선, UX/옵션 추가 등

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant