[fix] #224 아카이브 결과 반영 누락 수정

[ikseong00]

🔗 관련 이슈

• Close [fix] 아카이브 결과 반영 누락 수정 #224

📙 작업 설명

• SelectAlbum 업로드 성공 시 PhotoUploadedResult를 전파해 ArchiveMain 갱신을 보장했습니다.
• SelectAlbum에서 앨범 생성 결과를 SelectAlbumCreatedResult로 분리했습니다.
• 앨범 생성 후 업로드/복사/이동까지 완료되는 경우에는 최종 작업 result만 전파되도록 생성 result를 back 시점에만 flush합니다.
• SelectAlbum 시스템 back도 TopBar back과 동일한 handleBack() 경로를 타도록 추가했습니다.
• ArchiveMain 최근 사진 즐겨찾기 토글 성공 후 favorite summary를 재조회합니다.

🧪 테스트 내역 (선택)

• 주요 기능 정상 동작 확인
• 브라우저/기기에서 동작 확인
• 엣지 케이스 테스트 완료
• 기존 기능 영향 없음
• git diff --cached --check
• ./gradlew :feature:select-album:api:compileDebugKotlin :feature:select-album:impl:compileDebugKotlin :feature:archive:impl:compileDebugKotlin
• 실기기 회귀 확인: 전체 사진 즐겨찾기, 전체 앨범 생성/삭제, 앨범 상세 이름 변경/사진 제거/사진 복제 결과 반영

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

기능	미리보기	기능	미리보기
-	-	-	-

💬 추가 설명 or 리뷰 포인트 (선택)

• SelectAlbum에서 앨범 생성 후 업로드/복사/이동을 완료하는 경우 중복 refresh를 피하기 위해 생성 result는 별도로 보내지 않고 최종 작업 result만 보냅니다.
• 앨범만 생성하고 SelectAlbum을 나가는 경우에는 SelectAlbumCreatedResult를 전파합니다.

Summary by CodeRabbit

• New Features
  • 앨범 생성 후 뒤로가기 시 생성 결과가 전달되며, 관련 아카이브 화면이 새로고침됩니다.
• Bug Fixes
  • 즐겨찾기 토글 성공 시 즐겨찾기 요약 정보가 즉시 반영되도록 수정되었습니다.
  • 뒤로가기 동작이 시스템 뒤로가기와 일관되게 처리되도록 개선되었습니다.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: a5a59bc2-74f7-4629-8340-c49e9d2cc7a6

📥 Commits

Reviewing files that changed from the base of the PR and between 71d5b18 and 0d89a9e.

📒 Files selected for processing (5)

• feature/archive/impl/src/main/kotlin/com/neki/android/feature/archive/impl/album/AllAlbumScreen.kt
• feature/archive/impl/src/main/kotlin/com/neki/android/feature/archive/impl/album_detail/AlbumDetailScreen.kt
• feature/archive/impl/src/main/kotlin/com/neki/android/feature/archive/impl/photo/AllPhotoScreen.kt
• feature/select-album/impl/src/main/java/com/neki/android/feature/select_album/impl/SelectAlbumScreen.kt
• feature/select-album/impl/src/main/java/com/neki/android/feature/select_album/impl/SelectAlbumViewModel.kt

---

📝 Walkthrough

Walkthrough

SelectAlbum에 앨범 생성 결과와 뒤로가기 처리 흐름이 추가되었다. 결과는 ResultEventBus를 통해 ArchiveMain으로 전달되어 화면을 갱신하며, 즐겨찾기 토글 성공 후 요약 데이터도 재조회된다.

Changes

아카이브 결과 반영 및 요약 재조회

Layer / File(s)	Summary
SelectAlbumResult 타입 및 계약 추가 feature/select-album/api/.../SelectAlbumResult.kt, feature/select-album/impl/.../SelectAlbumContract.kt	앨범 생성 결과 타입과 생성 상태, 뒤로가기 인텐트, 결과 전달 사이드이펙트가 추가됨.
SelectAlbum 뒤로가기·결과 전송 흐름 feature/select-album/impl/.../SelectAlbumViewModel.kt, feature/select-album/impl/.../SelectAlbumScreen.kt, feature/select-album/impl/build.gradle.kts	앨범 생성 후 뒤로가기 시 결과를 발행하고, 업로드 및 생성 결과를 ResultEventBus로 전송하도록 변경됨. BackHandler 의존성과 처리가 추가됨.
ArchiveMain 결과 수신 및 상태 갱신 feature/archive/impl/.../ArchiveEntryProvider.kt, feature/archive/impl/.../ArchiveMainViewModel.kt, feature/archive/impl/.../*Screen.kt	앨범 생성 결과 수신 시 ArchiveMain을 새로고침하고, 즐겨찾기 토글 성공 시 요약을 재조회함. 아카이브 하위 화면의 BackHandler 호출도 조정됨.

Estimated code review effort: 2 (Simple) | ~15 minutes

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant SelectAlbumScreen
  participant SelectAlbumViewModel
  participant LocalResultEventBus
  participant ArchiveEntryProvider
  participant ArchiveMainViewModel
  User->>SelectAlbumScreen: 뒤로가기
  SelectAlbumScreen->>SelectAlbumViewModel: OnBackPressed
  SelectAlbumViewModel-->>SelectAlbumScreen: SendAlbumCreatedResult
  SelectAlbumScreen->>LocalResultEventBus: SelectAlbumCreatedResult
  LocalResultEventBus->>ArchiveEntryProvider: 결과 전달
  ArchiveEntryProvider->>ArchiveMainViewModel: RefreshArchiveMain
  ArchiveMainViewModel->>ArchiveMainViewModel: fetchFavoriteSummary

Loading

Possibly related PRs

• Team-Neki/Team-Neki-Android#137: 즐겨찾기 토글 및 요약 갱신 흐름과 관련됨.
• Team-Neki/Team-Neki-Android#155: 결과 이벤트 수신 후 아카이브 갱신 흐름과 관련됨.
• Team-Neki/Team-Neki-Android#166: RefreshArchiveMain 기반 갱신 경로와 관련됨.

Suggested reviewers: ojongseok

Poem

토끼가 결과를 안고 폴짝폴짝 뛰어요 🐰
버스 타고 아카이브에 소식 전해요
즐겨찾기 요약도 다시 반짝
뒤로가도 결과는 꼭 찰칵
버그야 안녕, 당근 먹자! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 #224의 아카이브 결과 반영 누락 수정이라는 핵심 변경을 정확히 요약합니다.
Description check	✅ Passed	필수 섹션인 관련 이슈, 작업 설명, 테스트 내역, 스크린샷, 추가 설명이 모두 포함되어 템플릿을 충족합니다.
Linked Issues check	✅ Passed	SelectAlbum 업로드 결과 전파, 앨범 생성 결과 처리, 즐겨찾기 요약 재조회가 #224의 요구사항과 일치합니다.
Out of Scope Changes check	✅ Passed	변경 내용은 SelectAlbum과 ArchiveMain의 결과 전달 및 요약 재조회에 집중되어 눈에 띄는 범위 이탈이 없습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/#224-archive-result-refresh

---

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

[Ojongseok]

고생하셨습니다~ 문제 없어 보여 머지해주셔도 될 것 같아요.

BackHandler(enabled = true) {
    ...
}

근데 BackHandler의 enabled 파라미터의 기본값이 true라 BackHandler {} 이런식으로 작성해도 될 것 같습니다!?