Repository navigation
refactor(designsystem): material-icons-extended·coil-network-okhttp UI 레이어 의존 제거 - #28
Merged
Merged
Conversation
- APK 크기 최적화를 위해 `material-icons-extended` 의존성을 제거하고 `material-icons-core`로 교체 - 앱 전역에서 사용하는 아이콘을 통합 관리하는 `HackathonIcons` 싱글 소스 객체 도입 - core 세트에 포함되지 않은 `Visibility`, `VisibilityOff` 아이콘을 ImageVector DSL을 통해 커스텀 구현 - 모든 feature 모듈 및 UI 컴포넌트가 `Icons` 대신 `HackathonIcons`를 참조하도록 수정 - `coil-network-okhttp` 의존성을 `app` 모듈로 이동하여 불필요한 의존성 전파 차단 - `.claude/hooks/git-state-guard.sh` 수정: 브랜치 삭제(`-D`) 및 전환(`checkout`) 명령어 허용으로 정책 완화
- 해커톤 기획, 체크리스트, 디자인 프롬프트 등 불필요한 문서 파일(.md, .cursorrules) 대거 삭제 - `.claude/CLAUDE.md` 내용을 멀티모듈 아키텍처 및 최신 빌드 환경(SDK 36, AGP 9.2.1) 기준으로 전면 개편 - 이슈 기반 브랜치 관리 및 자율 커밋 금지 등 개발 작업 규약 강화 - Google의 앱 아키텍처 가이드를 준수하도록 UseCase 및 UI 레이어 설계 원칙 재정의 - `HackathonIcons.kt` 내 사소한 코드 포맷팅 수정 (불필요한 쉼표 제거)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌𝘐𝘴𝘴𝘶𝘦𝘴
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
androidx.compose.material.icons는 더 이상 유지보수·권장되지 않으며 빌드 시간을 크게 늘린다. extended 를 제거하고 실사용 8종만:core:designsystem의HackathonIcons객체로 중앙화했다.material-icons-core에 위임.Visibility/VisibilityOff는 원본(material-icons 1.7.8) path 를 publicImageVectorDSL 로 재현 → 렌더 결과 동일.ArrowBack은 deprecatedFilled.ArrowBack→AutoMirrored.Filled.ArrowBack으로 통일.HackathonIcons만 참조.:core:designsystem과 feature 컨벤션 플러그인(AndroidFeatureConventionPlugin)에서 제거하고:app진입점으로만 이동했다.coil-network-okhttp의META-INF/services/coil3.util.FetcherServiceLoaderTarget를 ServiceLoader 로 자동 등록(merged classpath 스캔)하므로,:app클래스패스에만 있으면 전 모듈의AsyncImage가 동작한다.coil-compose(컴파일 API)는 그대로 유지.📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵
UI 변경 없음 — 아이콘은 원본 path 를 그대로 복제해 렌더 결과가 동일하다. (별도 캡처 생략)
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴
./gradlew :app:assembleDebug,lintDebug,testDebugUnitTest(CI 게이트와 동일).AsyncImage) 정상 로드 — refactor(designsystem): :core:designsystem 의 coil-network-okhttp 네트워크 의존 제거 #25 런타임 페처 자동 등록 확인 (에뮬레이터).Visibility/VisibilityOff) 렌더 확인..claude/hooks/git-state-guard.sh(이번 작업과 무관한 기존 로컬 hook 수정분)가 함께 포함돼 있습니다. 의도와 무관하면 리뷰 시 제외를 검토해 주세요.