fix: 감정 수정 다이얼로그 노출 버그 수정

[seoyoon513]

🔗 관련 이슈

• Close [BOOK-515/fix] 감정 수정 다이얼로그 노출 버그 수정 #280

📙 작업 설명

• 감정을 선택하지 않은 상태에서 바텀시트를 dismiss할 경우, 감정 재선택 시 수정 다이얼로그가 뜨는 문제 수정

🧪 테스트 내역 (선택)

• 주요 기능 정상 동작 확인
• 브라우저/기기에서 동작 확인
• 엣지 케이스 테스트 완료
• 기존 기능 영향 없음

📸 스크린샷 또는 시연 영상 (선택)

Summary by CodeRabbit

릴리스 노트

• 버그 수정
  • 감정 상세 팝업(바텀시트) 닫기 시 감정 선택 상태 처리를 개선했습니다. 커밋된 감정이 없으면 선택 상태와 맵을 초기화하고, 커밋된 감정이 있으면 해당 선택 상태로 복원합니다. 팝업 표시 플래그는 항상 해제됩니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2a8c795 and 58ae726.

📒 Files selected for processing (1)

• feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterPresenter.kt

---

Walkthrough

감정 기록 등록 프레젠터에서 감정 상세 바텀 시트 해제 이벤트 처리 로직을 변경했습니다. 바텀 시트 해제 시 committedEmotionCode가 null이면 selectedEmotionCode와 selectedEmotionMap을 초기화하고, null이 아니면 이전 선택 상태를 복원하며 항상 isEmotionDetailBottomSheetVisible를 false로 설정합니다.

Changes

Cohort / File(s)	Summary
감정 선택 상태 처리 변경 feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterPresenter.kt	OnEmotionDetailBottomSheetDismiss 핸들러에 committedEmotionCode 기반 분기 추가: null이면 selectedEmotionCode를 null로 리셋하고 selectedEmotionMap을 빈 persistent map으로 초기화, non-null이면 selectedEmotionCode/selectedEmotionMap을 committed 값으로 복원. 모든 경우에 isEmotionDetailBottomSheetVisible을 false로 설정.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 바닥 소리 멈추고 창 닫힐 때,
선택 없으면 깔끔히 비워두지요.
있던 감정은 되돌리고, 없으면 초기화—
작은 체크릿으로 버그는 사라졌네. ✨

🚥 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
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경사항의 주요 내용을 명확하게 요약하고 있습니다. '감정 수정 다이얼로그 노출 버그 수정'은 감정 선택 상태 복원 처리에 대한 핵심 변경사항을 직접적으로 설명합니다.
Linked Issues check	✅ Passed	PR 변경사항이 연결된 이슈 #280의 요구사항을 충족합니다. OnEmotionDetailBottomSheetDismiss 처리 시 committedEmotionCode 상태에 따라 selectedEmotionCode와 selectedEmotionMap을 복원 또는 초기화하여, 감정을 선택하지 않은 상태에서의 수정 다이얼로그 노출 문제를 해결합니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 RecordRegisterPresenter에서 감정 선택 상태 관리와 관련된 것으로, 이슈 #280의 범위 내에서 이루어졌습니다. 범위를 벗어난 변경사항이 없습니다.

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

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch BOOK-515-fix/#280

---

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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In
`@feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/register/RecordRegisterPresenter.kt`:
- Around line 272-278: The dismiss branch for
RecordRegisterUiEvent.OnEmotionDetailBottomSheetDismiss currently only resets
selectedEmotionCode/selectedEmotionMap when committedEmotionCode == null,
leaving a stale selection when a user opened the detail sheet for a different
emotion and dismissed without committing; update this handler so that when
committedEmotionCode != null you restore the UI selection to the committed state
(set selectedEmotionCode back to committedEmotionCode and restore
selectedEmotionMap to the map representing the committed emotion) before setting
isEmotionDetailBottomSheetVisible = false, ensuring the temporary selection
(e.g., SAD) is not preserved after dismiss.

[easyhooon]

미리 approve~