Skip to content

docs(afternote): Nav3 이관 뒤 코드와 어긋난 KDoc·주석을 고친다 #2170

Closed

Description

@OnlyReadOneBook

작업 유형

documentation — 문서

주 담당 모듈

afternote — 애프터노트

개요

Navigation 3 이관(#1698, PR #1824) 뒤에도 KDoc·주석 여러 곳이 Nav2 구조를 현재형으로 설명한다. #2168 에서 SenderDetailViewModel 이 Nav3 entry 안에서 savedStateHandle.toRoute 로 route 를 읽다 크래시 난 원인이 이런 설명과 같은 착각이었다. lifecycle-viewmodel-navigation3 2.11.0 의 entry SavedStateHandle 에는 NavKey 필드가 실리지 않고, entry 안 ViewModel 은 route 를 assisted 로 받는다. 주석이 옛 경로를 가리키면 다음 수정자가 같은 함정을 밟는다.

origin/develop 0073482 기준으로 전수 수색하고 반증 검증을 거쳐 확정한 자리는 아래와 같다. 코드 동작은 바꾸지 않는다.

afternote

  • detail/AfternoteDetailViewModel.kt:33, receiver/detail/ReceivedAfternoteDetailViewModel.kt:28: 상세 ID 를 SavedStateHandle 에서 읽는다고 적지만 실제는 assisted route
  • editor/AfternoteEditorRoute.kt:21-22: 수정 진입 ID 출처가 SavedStateHandle 의 itemId 라고 적지만 실제는 route.itemId
  • editor/AfternoteEditorViewModel.kt:342-343: 없어진 AfternoteNavGraphEditor 를 호출 자리로 가리킨다
  • editor/receiver/SelectReceiverRoute.kt:12-14, navigation/model/AfternoteRoute.kt:27: 선택 결과를 이전 엔트리 SavedStateHandle 로 돌려준다는 Nav2 설명과 없어진 tryApplyReceiverSelectionFromSavedState 링크
  • navigation/AfternoteNavActions.kt, receiver/navigation/ReceivedAfternoteNavActions.kt: 구현이 앱 모듈·NavGraph 에 있다고 적지만 실제는 로컬 스택 호스트
  • AfternoteHostViewModel.kt, editor/memorial/MemorialPlaylistEntry.kt, 스크린샷 테스트 FingerprintLoginScreenScreenshotTest.kt: 서브그래프·NavGraph destination·startDestination 용어

receiver

  • deliveryverification/MasterKeyViewModel.kt:36, IdentityVerificationViewModel.kt:33, IdentityVerificationEmailScreen.kt:45: 흐름 VM 을 parent backStackEntry 에서 받는다는 설명
  • IdentityVerificationIntroScreen.kt:23, DeliveryVerificationFlowViewModel.kt:26, navigation/ReceiverNavActions.kt:26, navigation/model/ReceiverRoute.kt:7-10·50-54: NavGraph·nested graph·startDestination 용어
  • 테스트 DeliveryVerificationFlowViewModelTest.kt:28: Robolectric 사유가 SavedStateHandle.toRoute 복원인데 VM 은 이미 assisted 다. 러너가 여전히 필요한지 실행으로 확인해 사유를 고치거나 러너를 뺀다

app

  • navigation/AppNavigationActions.kt:413, navigation/AppState.kt:74, 계측 테스트 ReceiverSelectionResultAndroidTest.kt:52·296, proguard-rules.pro:63: 없어진 서브그래프·이전 엔트리 SavedStateHandle·라우트 대기, nav 인자 enum 이 AfternoteType 이라는 설명

같은 수색에서 이관과 무관하게 원래부터 낡은 주석도 나왔다. 같은 파일이 대부분이라 함께 고친다: core/ui Route.Setting KDoc, EditorFormMutations.kt:26, ReceiverRoute.kt:95 완료 라우트, ReceiverSelectionResultAndroidTest.kt:71 의 없는 awaitReceiverLoad 링크, AfternoteNavActions 의 Toast 표현.

완료 조건

  • 위 자리의 주석이 현재 코드와 맞는다
  • ktlintCheck, afternote·receiver·app 관련 단위 테스트가 통과한다
  • 프로덕션 코드 동작 변경이 없다

자식 이슈

No response

참고

Activity

  1. added
    pr-open이 이슈를 구현하는 PR 이 열려 있다 — base 가 기본 브랜치가 아니어도 드러낸다
    on Sep 26, 2026
  2. removed
    pr-open이 이슈를 구현하는 PR 이 열려 있다 — base 가 기본 브랜치가 아니어도 드러낸다
    on Sep 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

area:afternote애프터노트 기능 영역documentationImprovements or additions to documentation

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions