[fix/#139] 바텀바 내비 수정

[jyvnee]

Related issue 🛠️

• closed [Fix] 바텀탭 내비 수정 #139

Work Description ✏️

• 바텀바 내비 수정

Uncompleted Tasks 😅

N/A

To Reviewers 📢

Summary by CodeRabbit

릴리스 노트

• 버그 수정
  • 앱 탐색 시 화면 상태 복원 기능을 개선했습니다.
  • 탭 간 이동 시 네비게이션 스택 동작을 최적화하여 더욱 일관된 사용자 경험을 제공합니다.
  • 백 스택 관리 로직을 개선하여 이전 화면으로 돌아갈 때 더 안정적으로 작동합니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

바텀 탭 네비게이션 로직을 개선하기 위해 MainNavigator.kt를 수정했습니다. popUpTo 대상을 동적 시작 목적지 ID에서 MainTab.HOME.route 상수로 변경하고, 상태 복원 옵션을 추가했습니다.

Changes

코호트 / 파일	변경 요약
바텀 탭 네비게이션 수정 app/src/main/java/com/poti/android/presentation/main/MainNavigator.kt	findStartDestination 임포트 제거, popUpTo 타겟을 명시적 라우트 상수(MainTab.HOME.route)로 변경, restoreState = true 옵션 추가

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3분

---

검토 의견

Modern Android Development 가이드라인 관점에서 살펴봤을 때, 몇 가지 좋은 점들이 보여요:

긍정적인 부분:

• 명시적 라우트 상수 사용으로 타입 안전성이 향상됐어요
• restoreState = true 추가로 백스택 복원 시 화면 상태를 유지하게 되어 사용자 경험이 개선돼요
• 불필요한 임포트 정리로 코드가 더 간결해졌어요

권장 사항:

• 현재 구조가 바텀 탭 네비게이션 패턴(각 탭의 자체 백스택 유지)을 따르는지 확인하세요. MAD에서는 각 탭이 독립적인 네비게이션 스택을 가지도록 권장해요
• saveState, launchSingleTop 옵션과 함께 restoreState 사용은 적절하지만, 탭 전환 시 백스택 동작을 철저히 테스트하세요

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	PR 설명이 필수 섹션을 포함하고 있으나, 작업 내용이 매우 간단히만 작성되어 구체적인 상세 설명이 부족합니다.	Work Description 섹션에서 구체적으로 어떤 네비게이션 문제를 수정했는지, 어떤 방식으로 수정했는지 더 자세히 설명해주세요.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 바텀 네비게이션 수정이라는 실제 변경 사항과 관련이 있으며, 이슈 번호도 포함되어 있습니다.
Linked Issues check	✅ Passed	PR의 코드 변경이 바텀탭 내비게이션 수정이라는 이슈 #139의 요구사항을 충족하는 것으로 보입니다.
Out of Scope Changes check	✅ Passed	MainNavigator.kt 파일의 모든 변경 사항이 바텀탭 네비게이션 수정과 관련된 범위 내의 변경입니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}