[Fix/#137] 뒤로가기 이슈 수정

[doyeon0307]

Related issue 🛠️

• closed [Fix] 뒤로가기 이슈 #137

Work Description ✏️

• 탑바 뒤로가기 클릭으로 뒤로갈 때, 등록뷰 > 아티스트 선택에서 밖으로 못 빠져나가는 문제

Screenshot 📸

휴

Uncompleted Tasks 😅

• [ ]

To Reviewers 📢

민누야 고마버

Summary by CodeRabbit

릴리스 노트

• 버그 수정

  • 아티스트 선택 화면에서 뒤로 가기 네비게이션 동작을 개선했습니다.

• 개선사항

  • 뒤로 가기 버튼 및 제스처에 대한 안정성을 강화했습니다.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

파티 아티스트 선택 화면에서 뒤로가기 동작을 수정했습니다. 네비게이션 팝 로직을 전용 네비게이션 함수에서 직접 popBackStack() 호출로 변경하고, BackHandler import를 추가했습니다.

Changes

Cohort / File(s)	변경 내용
뒤로가기 네비게이션 수정 app/src/main/java/com/poti/android/presentation/party/create/PartyCreateNavigation.kt, app/src/main/java/com/poti/android/presentation/party/create/PartyArtistSelectScreen.kt	ArtistSelect 라우트의 뒤로가기 핸들러를 navigateToPartyCreateFromArtistSelect() 함수에서 navController.popBackStack()로 변경. PartyArtistSelectScreen에 BackHandler import 추가 (향후 사용 예상)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 분

Possibly related PRs

• [Feat/#114] create view auto fill #118: PartyCreateNavigation.kt에서 ArtistSelect → PartyCreate 네비게이션 동작 방식 수정과 일관된 변경입니다.
• Fix/#117 UI qa #131: PartyArtistSelectScreen.kt에 대한 유사한 수정이 포함되어 있습니다.

Suggested labels

🛠️ Fix

---

Modern Android Development 관점 평가

이 변경은 Compose Navigation의 백스택 관리를 단순화한 합리적인 수정입니다. 다만 다음 사항들을 검토하시면 좋겠습니다.

• BackHandler 활용: PartyArtistSelectScreen에 import된 BackHandler가 실제로 사용되고 있는지 확인해 주세요. Compose의 BackHandler API는 뒤로가기 제스처를 더 세밀하게 제어할 수 있으므로, 추후 화면 상태 정리(state cleanup) 등이 필요하다면 활용하시면 좋습니다.
• 네비게이션 의도: popBackStack()은 단순 백스택 제거이므로, 기존 navigateToPartyCreateFromArtistSelect()가 수행하던 추가 동작(예: popUpTo, launchSingleTop)이 정말 불필요한지 확인 바랍니다.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

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.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 주요 변경사항을 명확하게 설명하고 있습니다. '[Fix/#137] 뒤로가기 이슈 수정'은 문제의 내용과 해결책을 구체적으로 나타내고 있습니다.
Description check	✅ Passed	PR 설명이 필수 템플릿 섹션을 대부분 포함하고 있습니다. 관련 이슈, 작업 내용, 스크린샷 섹션이 있으나 스크린샷이 제대로 제공되지 않았습니다.
Linked Issues check	✅ Passed	PR의 코드 변경사항이 #137 이슈의 문제 해결과 일치합니다. 아티스트 선택 화면에서의 뒤로가기 네비게이션 문제를 수정했습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 #137 이슈의 뒤로가기 네비게이션 문제 해결과 관련되어 있으며, 범위를 벗어난 변경사항은 없습니다.

_{✏️ 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.}

[github-actions]

✅ Ktlint check passed.

Run: https://github.com/team-poti/POTI-ANDROID/actions/runs/21285969341