Skip to content

chore(app): 역할 끝난 NavigationDebugLogger 삭제 — 일회성 바텀바 디버그 프로브 - #960

Merged
OnlyReadOneBook merged 2 commits into
developfrom
chore/927
Aug 24, 2026
Merged

OnlyReadOneBook merged 2 commits into
developfrom
chore/927

Conversation

@OnlyReadOneBook

Copy link
Copy Markdown
Collaborator

Closes #927

바텀바 표시 판정을 디버깅하려고 903bc1e0 (2026-04-13) 에 붙인 일회성 프로브를 삭제한다.

변경

파일 내용
NavigationDebugLogger.kt 삭제 (42줄)
AppNavigation.kt:52 호출 1줄 삭제 — 유일 호출부

navEntry · currentDestination · showBottomBar · currentTab 은 모두 바텀바 렌더에서 계속 쓰이므로 미사용 잔재는 남지 않는다. 필요해지면 903bc1e0 에서 복원 가능.

이슈의 "확인 필요" 항목 — 해당 바텀바 이슈 종결 여부

이슈 본문은 "아직 재발 중이면 삭제 대신 src/debug + src/release no-op 쌍으로 이동" 을 조건으로 걸어 뒀다. 실측 결과 발동 조건 미충족 이라 지시대로 삭제로 진행했다.

확인 항목 실측
프로브 추가 903bc1e0 2026-04-13 18:39
프로브 자체 수정 이력 0건 — 추가 커밋 하나가 전부
바텀바 판정 로직(AppState.bottomBarRoutes) 마지막 변경 ba93097f 2026-05-15 — 타임레터 홈 라우트 추가(기능) 이지 버그 수정이 아님
그 이후 3개월 이상 무변경
바텀바 관련 open 이슈 0건 (바텀바 · bottom bar 전수 검색)

다른 PR 과의 충돌

AppNavigation.kt 를 함께 건드리는 열린 PR 3건 — #917 · #919 · #830 — 은 diff 에 NavigationDebugLogger 등장 0회 로, 이 PR 이 지우는 줄을 건드리지 않는다. 어느 쪽이 먼저 머지돼도 충돌 없음.

검증

항목 결과
:app:compileDebugUnitTestKotlin 통과
:app:ktlintCheck 통과
pre-commit ktlint (staged snapshot) 통과

시각 검증 — skip. 삭제 대상은 BuildConfig.DEBUG 분기 안에서 LaunchedEffect + Log.d 만 수행하는 컴포저블로, 렌더링 산출물이 없다. screenshotTest 커버 대상이 아니고 브랜치 diff 에 디자인 토큰 · 리소스 변경도 없어 baseline 재생성 불필요.

에뮬 실측 — skip (재량 판단). 릴리스 빌드는 isMinifyEnabled = true 라 R8 이 이미 죽은 분기와 문자열 상수까지 제거하던 코드이고, 디버그 빌드에서도 로그 출력 외 런타임 동작이 없다. 실측이 바꿀 판단이 없어 컴파일 검증으로 갈음했다.

후속

#924 (Nav3 전환) 가 currentBackStackEntryAsState() · NavDestination.hierarchy · hasRoute<T>() 를 걷어낼 때 이 파일도 어차피 함께 사라졌을 대상이다. 미리 지워 두면 그 작업에서 다룰 파일이 하나 줄어든다.

바텀바 표시 판정 디버깅용으로 903bc1e 에 추가한 일회성 프로브다.
추가 이후 4개월간 수정 이력이 없고, 판정 로직(AppState.bottomBarRoutes)은
2026-05-15 타임레터 라우트 추가 이후 무변경 — 프로브가 잡던 문제는 종결됐다.

- NavigationDebugLogger.kt 삭제
- AppNavigation.kt 호출 1줄 삭제 (유일 호출부)

필요해지면 903bc1e 에서 복원 가능.
@OnlyReadOneBook
OnlyReadOneBook merged commit 28283c2 into develop Aug 24, 2026
13 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the chore/927 branch August 24, 2026 09:17
@github-actions

Copy link
Copy Markdown
Contributor

🚀 QA 배포 권장

QA 포인트

이 판단은 배포를 자동 실행하지 않습니다. deploy일 때 Release Distribution을 수동 실행하세요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NavigationDebugLogger 삭제 — 역할 끝난 일회성 디버그 프로브

2 participants