Repository navigation
refactor(onboarding): MVI 전환이 만든 Content seam 5건의 최소 visibility 를 정한다 (#1802 후속) #1829
Description
작업 유형
refactor — 동작 변경 없는 구조·책임·경계 개선
주 담당 모듈
onboarding — 온보딩
개요
#1802의 MVI Screen/Content 분리에서 생긴 다섯 렌더 선언의 최소 공개 범위를 확정한다. PR #1825에서 MVI 전환과 함께 처리하여 예외 baseline을 만든 뒤 별도 PR에서 같은 seam을 다시 바꾸는 순서를 없앤다.
구현 결정
Screen은 ViewModel 상태 수집과 일회성 신호 소비를 맡고, 별도 Content 파일은 상태와 콜백으로 렌더한다. 다음 다섯 Content는 같은 모듈의 다른 프로덕션 Screen 파일이 실제 호출하므로 internal이다. Content 안에서만 쓰는 렌더 helper는 private다. 테스트 접근은 공개 범위의 근거로 삼지 않는다.
| 프로덕션 소비처 | 렌더 계약 |
|---|---|
findaccount/FindIdScreen.kt |
findaccount/FindIdContent.kt |
signup/SignUpScreen.kt |
signup/SignUpContent.kt |
signup/SignUpResidentNumberScreen.kt |
signup/SignUpResidentNumberContent.kt |
signup/SignUpPasswordScreen.kt |
signup/SignUpPasswordContent.kt |
terms/OnboardingTermsScreen.kt |
terms/OnboardingTermsContent.kt |
프리뷰·screenshotTest·Robolectric은 같은 렌더 계약을 소비한다. 기존 픽셀을 바꾸지 않으므로 스크린샷 reference PNG는 유지한다. TermsType과 TermsState도 모듈 내부에서만 사용해 internal로 정하고, 다른 파일 소비가 생겨 불필요해진 TermsType의 intentional baseline도 제거한다. 프로필 이미지 picker helper는 기존 private 범위를 보존한다.
완료 조건
- 다섯 Content의 실제 프로덕션 소비 파일과 최소 visibility를 코드/KDoc에 반영한다.
-
ISSUE_1675_ONBOARDING의 다섯 예외와 stale TermsType intentional 예외를 제거한다. -
docs/convention/mvi.md에서 테스트 접근을 공개 범위의 근거로 삼던 문구를 교정한다. - onboarding 테스트·스크린샷 소스 컴파일·
:konsist:test·ktlint가 통과하고 reference PNG 변경이 없다. - PR #1825에서 #1802와 이 이슈를 함께 닫아 후속 seam 재작성을 남기지 않는다.
자식 이슈
No response
참고
실제 순서: #1863(PR #1963, 단일 실패 사유 모델) → PR #1825(#1802 + #1829 동시 구현).
기존 #1802 blocked_by는 같은 PR의 완료 범위로 합쳐져 제거하고, 공통 선행 #1863으로 교체한다.
같은 성격의 형제: #1819(receiver) · #1817(afternote) · #1818(mindrecord).
가드: #1678. 규약: docs/convention/mvi.md.
Activity
test(platform): Content seam baseline 이 판정 이슈 #1829 를 가리키게 한다 (#1802)
OnlyReadOneBook commentedon Sep 13, 2026
이 이슈를 맡은 PR 은 #1825 다 (base
refactor/1863-onboarding-failure). 본문에 "#1829 공개 범위 확정" 절이 따로 있다. FindId·SignUp·SignUpResidentNumber·SignUpPassword·OnboardingTerms 의 Content 다섯을 파일로 분리해internal로, 파일 안에서만 쓰는 렌더·프로필 picker helper 는private로 두고 TermsType·TermsState 도 모듈 내부로 좁혔다.pr-open라벨을 손으로 붙였다. #1825 는 제목이(#1802)로 끝나고 본문 마지막에서 #1802 와 이 이슈를 함께 닫는데, 라벨러가 제목의 대표 이슈만 읽어 이 이슈는 라벨도 링크 코멘트도 받지 못했다(#2082).base 가 기본 브랜치가 아니라 Development 칸도 비어 있다. #1802 쪽은 GitHub 이 참조를 잡아 뒀지만 이 이슈에는 아무 신호도 없었다. 담당은 이 이슈가 Sadturtleman 이고 #1825 작성자는 1hyok 이라, 그 PR 은
issue-assignee-exempt로 담당자 대조만 면제받은 상태다.라벨러 마커를 함께 넣어 두어 #2082 수정이 머지된 뒤 중복 코멘트가 생기지 않는다. 라벨은 그 전까지 배포본 리컨사일러가 떼어 간다.