Repository navigation
chore(app): 역할 끝난 NavigationDebugLogger 삭제 — 일회성 바텀바 디버그 프로브 - #960
Merged
Merged
Conversation
Sadturtleman
approved these changes
Aug 24, 2026
Contributor
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.
Closes #927
바텀바 표시 판정을 디버깅하려고
903bc1e0(2026-04-13) 에 붙인 일회성 프로브를 삭제한다.변경
NavigationDebugLogger.ktAppNavigation.kt:52navEntry·currentDestination·showBottomBar·currentTab은 모두 바텀바 렌더에서 계속 쓰이므로 미사용 잔재는 남지 않는다. 필요해지면903bc1e0에서 복원 가능.이슈의 "확인 필요" 항목 — 해당 바텀바 이슈 종결 여부
이슈 본문은 "아직 재발 중이면 삭제 대신
src/debug+src/releaseno-op 쌍으로 이동" 을 조건으로 걸어 뒀다. 실측 결과 발동 조건 미충족 이라 지시대로 삭제로 진행했다.903bc1e02026-04-13 18:39AppState.bottomBarRoutes) 마지막 변경ba93097f2026-05-15 — 타임레터 홈 라우트 추가(기능) 이지 버그 수정이 아님바텀바·bottom bar전수 검색)다른 PR 과의 충돌
AppNavigation.kt를 함께 건드리는 열린 PR 3건 — #917 · #919 · #830 — 은 diff 에NavigationDebugLogger등장 0회 로, 이 PR 이 지우는 줄을 건드리지 않는다. 어느 쪽이 먼저 머지돼도 충돌 없음.검증
:app:compileDebugUnitTestKotlin:app:ktlintCheck시각 검증 — skip. 삭제 대상은
BuildConfig.DEBUG분기 안에서LaunchedEffect+Log.d만 수행하는 컴포저블로, 렌더링 산출물이 없다. screenshotTest 커버 대상이 아니고 브랜치 diff 에 디자인 토큰 · 리소스 변경도 없어 baseline 재생성 불필요.에뮬 실측 — skip (재량 판단). 릴리스 빌드는
isMinifyEnabled = true라 R8 이 이미 죽은 분기와 문자열 상수까지 제거하던 코드이고, 디버그 빌드에서도 로그 출력 외 런타임 동작이 없다. 실측이 바꿀 판단이 없어 컴파일 검증으로 갈음했다.후속
#924 (Nav3 전환) 가
currentBackStackEntryAsState()·NavDestination.hierarchy·hasRoute<T>()를 걷어낼 때 이 파일도 어차피 함께 사라졌을 대상이다. 미리 지워 두면 그 작업에서 다룰 파일이 하나 줄어든다.