Repository navigation
refactor: 위임만 하던 UseCase 4개 제거 (Phase F 완료, 21 → 11) - #371
Merged
Merged
Conversation
GetCurrentLocation / ReverseGeocode / SearchPlaces / RecentSearches 를 지우고 호출처가 Domain 포트(LocationService·PlaceRepository· RecentSearchRepository)를 직접 주입받는다. 넷 다 단일 Repository 메서드를 한 줄 위임하던 계층이라, 같은 프로토콜에 이름만 하나 더 입히고 있었다. Presentation → Domain 경계는 그대로다 — 피처가 받는 건 여전히 Domain 프로토콜이고 구체 Data 타입은 AppDIContainer만 본다. 부수 정리: - 위임 여부만 검증하던 Domain 테스트 5개 삭제 (Observe/ObserveChange/ RequestAlarmSync/GetLastRouteDetail/ReverseGeocode). 앞 커밋에서 UseCase를 지울 때 테스트 파일이 남아 DomainTests가 컴파일되지 않고 있었다. - Example/Tests 스텁이 좁은 UseCase 대신 PlaceRepository 전체를 채택하게 되어 쓰지 않는 메서드에 기본 구현을 달았다(의도된 Tests/Example 중복 규약). - RecentSearchesUseCase.fetch() → RecentSearchRepository.recentSearches()로 호출명이 바뀐다. 검증: Debug/Stage/Release 3구성 빌드 + Domain 112 · AtchaData 53 · AtchaV2 39 · HomeFeature 66 · SearchFeature 26 · SettingsFeature 18 전부 통과. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
무엇을
Phase F의 마지막 묶음.
GetCurrentLocation/ReverseGeocode/SearchPlaces/RecentSearches4개를 지우고 호출처가 Domain 포트를 직접 주입받게 했다. UseCase 21 → 11로, 리팩토링 계획의 목표치에 도달했다.넷 다 단일 Repository 메서드를 한 줄 위임하던 계층이다 — 같은 프로토콜에 이름만 하나 더 입히고 있었다. 판단 기준은 결정표 3번(다중 포트 조합 / 비즈니스 규칙 / 팀 규모)이고, 남긴 11개는 전부 ① 또는 ②를 만족한다.
경계는 그대로
피처가 주입받는 건 여전히 Domain 프로토콜(
LocationService,PlaceRepository,RecentSearchRepository)이고, 구체 Data 타입을 보는 곳은AppDIContainer하나뿐이다.Presentation → Domain ← Data는 유지된다.부수 정리
DomainTests타겟이 컴파일되지 않고 있었다 — 빌드만 도는 CI에는 잡히지 않던 구멍이다.PlaceRepository전체를 채택하게 되어, 해당 화면이 쓰지 않는 메서드에 기본 구현을 달았다.RecentSearchesUseCase.fetch()→RecentSearchRepository.recentSearches()로 호출명이 바뀐다.검증
🤖 Generated with Claude Code