refactor(mobile): 앱 아키텍처를 Presentation, Domain, Data 레이어로 구조 개선 #56
Conversation
dydals3440
added
type: refactor
Summary of ChangesHello @dydals3440, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 Pull Request는 모바일 앱의 아키텍처를 Presentation, Domain, Data 레이어 패턴에 따라 대규모로 재구성하는 리팩토링을 수행했습니다. 이를 통해 앱의 유지보수성과 확장성을 향상시키고, 의존성 주입, 인증 흐름, 데이터 페칭 로직을 더욱 체계적으로 관리할 수 있도록 기반을 마련했습니다. 기존의 분산된 로직들을 중앙화된 프로바이더와 서비스 계층으로 통합하여 개발 효율성을 높이는 데 기여합니다. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이 PR은 Presentation, Domain, Data 레이어 아키텍처를 도입하여 모바일 앱의 구조를 개선하는 대규모 리팩토링을 수행합니다. 전반적으로 코드 구조가 크게 향상되었으며, 특히 의존성 주입(DI), queryOptions/mutationOptions를 사용한 데이터 페칭 로직 중앙화, 그리고 Domain/Service/Repository/Mapper의 역할 분리가 인상적입니다.
다만, 몇 가지 중요한 우려 사항이 있습니다. 첫째, 인증 토큰 갱신 로직에서 여러 요청이 동시에 401 오류를 반환할 때 발생할 수 있는 경쟁 상태(race condition)에 대한 처리가 누락되었습니다. 이는 심각한 인증 문제를 유발할 수 있습니다. 둘째, 리팩토링 과정에서 인증 기능과 관련된 많은 테스트 파일들이 삭제되었습니다. 대규모 리팩토링의 안정성을 보장하기 위해서는 테스트 코드를 제거하는 대신 새로운 아키텍처에 맞게 수정하고 유지하는 것이 매우 중요합니다. PR 설명에 '모든 테스트가 통과합니다'라고 되어 있지만, 실제로는 테스트 커버리지가 크게 감소한 것으로 보입니다. 이 부분에 대한 확인이 필요합니다.
이러한 점들을 수정하면 더욱 완성도 높은 코드가 될 것입니다. 아래에 구체적인 리뷰 의견을 남겼으니 확인 부탁드립니다.
|
기존 구조가 너무 과한 것 같아서, 아키텍처 구조를 조금 더 직관적으로 정리해봤습니다!
mapper의 위치를 어디에 두는 게 좋을지 고민을 많이 했는데, repository는 서버에서 데이터를 가져오는 책임에 집중하고, 이렇게 하면 repository는 순수한 데이터 소스 역할에 집중할 수 있고 service가 실제로 “어떤 데이터가 화면/비즈니스에 쓰일지”를 명확히 책임지는 구조가 되지 않을까 싶네요! 공부를 하다보니, repository 영역의 핵심인 서버 응답 검증을 기존에 하지 않고 있다고 생각이 들어서 리팩토링을 해보았습니다! |
- auth-client.ts: 토큰 리프레시 시 경쟁 상태(race condition) 방지 로직 추가 - auth.repository.impl.ts: 로그아웃 시 API 호출 후 토큰 삭제하도록 순서 수정 - auth-provider.tsx: 디버그용 console.log 제거
개요
Mobile 앱의 유지보수성 및 확장성 향상을 위해 기존 아키텍처를 Presentation, Domain, Data 레이어 패턴에 따라 재구성하는 대규모 리팩토링을 진행했습니다.
변경 유형
영향 범위
apps/mobile)변경 내용
presentations,services,repositories,models등으로 명확히 분리core/di를bootstrap/providers로 재구성하여 의존성 주입 개선useQuery의queryOptions및mutationOptions를 도입하여 데이터 페칭 로직 중앙화테스트
pnpm --filter mobile test체크리스트
pnpm test)pnpm typecheck)pnpm lint)관련 이슈
Closes #55