test(platform): MVI 계약 konsist 강제 + docs/convention/mvi.md 신설 #1801

작업 유형

test — 테스트·검증·회귀 방지

주 담당 모듈

platform — Android 앱·CI·빌드·릴리스·저장소 운영

개요

MVI 계약을 konsist 로 강제하고 docs/convention/mvi.md 를 신설한다. 강제 없이 시작하면 반쪽 도입으로 파편화만 늘어난다 — 지금 onEvent 단일 진입점이 51개 중 1개인 상태가 그 증거다.

konsist 규칙

• A. MviViewModel 상속체는 MutableStateFlow·MutableSharedFlow·Channel 을 선언하지 않는다. 별도 상태 홀더를 두면 전이가 reduce 밖으로 새고 베이스를 도입한 의미가 사라진다. Channel 금지는 refactor(setting): 남은 이벤트 스트림 2곳을 UiState·MVI로 전환하고 재유입을 막는다 #1502 가 세우려는 규칙과 같은 방향이므로 두 규칙을 하나로 합친다 — 따로 두면 나중에 서로를 덮는다.
• B. feature/*/presentation 의 새 ViewModel 은 MviViewModel 을 상속한다. 전환 전 기존 49개는 allowlist 예외로 두고, 모듈 전환 이슈가 닫힐 때마다 목록에서 뺀다. 목록이 비면 규칙 B 의 예외도 함께 지운다.
• C. Intent·ReducerEvent·Effect 는 sealed interface 이고 각 마커를 구현한다.

konsist 모듈에 이미 PresentationLayerDependencyKonsistTest 가 있으므로 신규 테스트 클래스로 나란히 둔다.

docs/convention/mvi.md

• Intent 와 ReducerEvent 를 가르는 기준 — 사용자가 «하려는 것» vs 상태가 «겪은 것»
• Screen(stateful, viewModel 주입) / Content(internal stateless, state + onIntent) 2단 분리. 프리뷰·screenshotTest·Robolectric 은 Content 를 직접 렌더한다
• 일회성 이벤트 관용구 — UiState 의 nullable 신호(refactor: ViewModel 이벤트 → UI state 흡수 마이그레이션 (전수 9개 VM) #228) + Intent.ConsumeXxx 소비. onXxxConsumed() 를 public fun 으로 노출하지 않는 이유를 적는다
• composable-callback-defaults.md(refactor(platform): 컴포저블 no-op 콜백 디폴트 — 재발 가드·컨벤션 도입 + afternote 잔여 2건 #1388)와의 관계를 명시한다. MVI 화면은 콜백 파라미터 자체가 사라져 NoOpCallbackDefaultKonsistTest 의 대상이 줄어든다. 두 규칙이 충돌하는 것은 아니지만, 전환한 화면에서 규칙이 어떻게 적용되는지 문서에 적지 않으면 다음 사람이 콜백을 되살린다

완료 조건

• konsist 규칙 A·B·C 가 있고 :konsist:test 가 초록이다
• allowlist 에 전환 전 ViewModel 49개가 등재돼 있고, 각 항목이 어느 전환 이슈에 속하는지 주석으로 표시돼 있다
• docs/convention/mvi.md 가 있고 #1388 규약과의 관계가 적혀 있다

자식 이슈

No response

참고

konsist 초록이 «안 돌린» 결과였던 전례가 있다(#1657, #1614) — 규칙 추가 후 :konsist:test 가 실제로 실행됐는지 확인한다.