[#557] 위젯을 새로운 피쳐 모듈 기반으로 분리한다#560
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Code Review
이번 풀 리퀘스트는 앱과 위젯 간의 브릿지 및 동기화 처리를 담당하는 DevLogWidget 프레임워크를 신설하고 관련 로직을 이관하는 변경 사항을 담고 있습니다. 이를 위해 WidgetTodoSnapshotRepository를 추가하여 위젯용 데이터를 조회하도록 개선하였으며, 관련 테스트 코드도 함께 작성되었습니다. 리뷰 의견으로는 테스트 환경의 시스템 달력 설정에 영향을 받지 않고 일관된 결과를 보장할 수 있도록 WidgetSyncEventHandlerTests에서 Calendar.current 대신 Calendar(identifier: .gregorian)을 사용할 것을 제안했습니다.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
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
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.
🔗 연관된 이슈
🎯 의도
DevLogData,DevLogApp에 나뉘어 있어 모듈 책임이 흐려진 상태였음DevLogWidget모듈로 분리하고, 위젯 동기화 구현체와 핸들러를 해당 모듈에서 소유하도록 정리📝 작업 내용
📌 요약
DevLogWidget모듈 추가DevLogWidget으로 이동WidgetSyncEvent,WidgetSyncEventBusprotocol은 Data boundary에 유지DevLogWidget테스트 scheme 추가🔍 상세
WidgetSyncEventBusImpl,WidgetSyncEventHandler,WidgetSessionSyncHandler를DevLogWidget으로 이동WidgetAssembler를 추가해 위젯 동기화 bus 구현체와 handler 조립 책임을DevLogWidget으로 이동DevLogWidgettarget 의존성을DevLogCore,DevLogData로 제한WidgetTodoSnapshotRepository와 구현체를 추가해DevLogWidget이 Domain repository를 직접 참조하지 않도록 adapter 구성AppAssembler에서WidgetAssembler를DataAssembler보다 먼저 조립하도록 구성AppLayerAssembler에서 위젯 handler 등록 책임 제거.github/workflows/ci.yml에DevLogWidgetscheme 테스트 추가docs/graph.png기반 Tuist 모듈 그래프로 교체📸 영상 / 이미지 (Optional)