♻️ [Refactor] UMCFoundation 정비: 하드닝·공용 유틸 이식·FlexibleDecoding 통일·xcconfig 시크릿 주입#938
Merged
Conversation
- git-workflow.md: `이슈 생성 규칙` 섹션 추가 (템플릿·라벨·이슈 Type 매핑표) - Type은 생성 직후 REST PATCH로 설정 (gh 2.83.1 `--type` 미지원) - Priority/Projects는 project 스코프 필요 — iOS 보드 #3, 한글 `우선순위` 필드 안내 - CLAUDE.md: 레퍼런스 인덱스의 Git Workflow 항목에 이슈 규칙 반영
- 이슈 생성 = 제목/라벨/Type + 보드#3·우선순위 + 네이티브 Priority·Effort 기본 채움 - 보드 `우선순위`(한글) ↔ 네이티브 `Priority`(영어)를 같은 우선순위 레벨로 일치 규칙 추가 - 네이티브 이슈 Fields(베타) 설정 방식(setIssueFieldValue) 및 스킬 4-4·4-5 참조 반영
레거시 AppProduct 766개 Swift 파일을 UMCApp Tuist 모듈로 옮길 때의 파일별 목적지를 정리. 확정 필요 항목은 개발 에이전트 분석으로 목적지 확정 후 반영.
- Activity 내부 ext + Notice 5개 private ext 삭제 → UMCFoundation 정규 헬퍼로 통일
- 숫자→String 은 정수 절삭('1'), optional-string 은 bad-type swallow 로 정규화
- CLAUDE.md 규칙 #3 canonical 네이밍(decodeIntFlexible*) 정합
- Date+HourMinutes 중복 제거 → Foundation toHourMinutes() 사용
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.
✨ PR 유형
UMCFoundation정비 — 하드닝(버그/성능/견고성) + 공용 유틸 이식 + FlexibleDecoding 단일화 + xcconfig 기반 시크릿/환경 주입. (Refactor 중심 + Feature 일부)📷 스크린샷 or 영상(UI 변경 시)
UI 변경 없음 (Core/Foundation·빌드 설정·DTO 디코딩 레이어).
🛠️ 작업내용
레거시(AppProduct) 대비
UMCFoundation에 빠져 있던 것 이식 + 기존 코드 개선을 5개 커밋으로 정리했습니다.① 하드닝 (Refactor)
ko_KR_POSIX→en_US_POSIX로케일 버그 교정 (ServerDateTimeConverter,Date+Formatting)ServerDateTimeConverterhot-path ISO8601/KST 포매터static캐싱 (매 호출 생성 제거)ErrorHandler의FoundationModels를#if canImport가드 (복원력)DateFormatter.swift→Date+Formatting.swift리네임(내용은Date확장),MemberStatusstale 헤더 정리② 공용 유틸 이식 (Feat) + 테스트
Date+KST(KST 경계:kstStartOfDay/EndOfDay,…UTCISO8601,isAllDayInKST)Date+Calendar(startOfMonth/endOfMonth,datesInMonth(),isSameDay,weekDaySymbols)Error+Cancellation(Error.isCancellation),String+App(isValidHTTPURL,forceCharWrapping)③ xcconfig 기반 시크릿/환경 주입 (Feat)
Secrets/Shared.xcconfig(환경별BASE_URLDebug/Release +#include?로컬 오버라이드) +.template+READMEProject.swift앱 타겟에settings.configurations+ Info.plist$(BASE_URL)/$(KAKAO_KEY)/$(TMAP_SECRET_KEY)·Kakao URL Scheme 매핑fatalError위험) → 신규 구축.Secrets.xcconfig(실키)는 gitignore로 제외④ Notification 정의 통합 (Refactor)
Notification+Error.swift+NotificationName+App.swift→Notification+Names.swift단일 파일 (public 심볼 동일, 동작 변화 없음)⑤ FlexibleDecoding 단일 소스 통일 (Refactor)
UMCFoundation정규 헬퍼로 통일 (~90 호출부 마이그레이션)"1"), optional-string 은 bad-type swallowdecodeIntFlexible*) 정합,decodeFlexibleStringOrNil/OrEmpty/Array·decodeFirstNonEmptyString·decodeBoolFlexibleIfPresent추가Date+HourMinutes완전 중복 제거 → FoundationtoHourMinutes()사용✅ 검증 (로컬 빌드/테스트 통과)
dev.api…, Release→api…📋 추후 진행 상황
Secrets.xcconfig자동 생성 스크립트 추가 필요(README에 안내)AppState + Environment전환 시 NotificationCenter 기반 흐름 정리(기존 TODO)📌 리뷰 포인트
UMCApp/Core/Foundation/Sources/Extensions/KeyedDecodingContainer+FlexibleNumber.swift— 정규 헬퍼 세트/절삭·swallow 시맨틱UMCApp/Features/Notice/Data/Sources/DTOs/**— private ext 제거 후 정규 헬퍼로 치환된 디코딩 동작(엣지: Double→String 절삭)UMCApp/Project.swift+UMCApp/Secrets/Shared.xcconfig— 환경별 BASE_URL 주입 배선UMCApp/Core/Foundation/Sources/Extensions/ServerDateTimeConverter.swift— 포매터 캐싱/로케일✅ Checklist