🐛 [fix] 전 기능 영역 버그 일괄 수정 — 코드베이스 전수 조사 결과

[geeksbaek]

개요

코드베이스 전체를 전수 조사(병렬 리뷰 에이전트 4개 + 수동 코드 추적 검증)하여 발견한 버그들을 일괄 수정합니다.

Base가 codex/fix-icloud-sync-reset(#179)인 이유: SafetySnapshotService, CoreDataDailyTaskStorage 등 #179에서 변경된 파일을 함께 수정하므로 충돌 방지를 위해 #179 위에 스택했습니다. #179 머지 후 자동으로 develop으로 retarget 됩니다.

수정한 버그 (검증 완료)

#	심각도	위치	내용
1	P1	Items.reset()	npcLike relay 초기화 누락 — 데이터 초기화 후에도 NPC 좋아요가 화면에 잔존, 이후 토글 시 stale 목록 기반으로 저장
2	P1	MusicPlayerManager	block 기반 곡 종료 옵저버가 removeObserver(self)로 제거되지 않아 choice()마다 누적 — 곡 종료 시 핸들러 다중 실행(fullRepeat에서 곡 건너뜀). 또한 특정 AVPlayerItem에 바인딩되어 곡 전환 후 알림 누락 가능. 토큰 보관 + object: nil + currentItem 검증으로 수정, 등록 지점을 player 재생성 시점으로 일원화
3	P1	VillagersCellReactor / NPCCellReactor	updateLike/updateHouse가 state를 갱신하지 않아 목록 셀의 하트/집 버튼이 시각적으로 토글되지 않음 (상세 화면 reactor에는 있는 로직이 셀 reactor에 누락)
4	P1→크래시	DailyTask / CoreDataDailyTaskStorage / 매핑	CloudKit 부분 import 후 progressList가 amount보다 짧으면 UI는 amount 기준으로 셀을 만들어 toggle 시 index out of range 크래시 (메인 스레드 + 백그라운드 양쪽). toDomain() 길이 정규화 + 모델/storage toggle 보정으로 수정 (손상 데이터 자가 치유)
5	P1	SafetySnapshotService	pendingWorkItem이 CoreData 저장 알림(임의 스레드)·main·sync-reset 알림에서 동기화 없이 접근되는 data race — OSAllocatedUnfairLock 보호 + flushNow() 전용 queue 동기 실행으로 debounce 쓰기와 파일 I/O 직렬화
6	P2	NPCLikeEntity.toDomain()	genderAsia에 gender 값을 읽어 CoreData 왕복마다 genderAsia 손실
7	P2	MusicPlayerManager.setUpBackground()	아트워크 다운로드 구독이 싱글톤 disposeBag에 누적 — 느린 네트워크에서 이전 곡 이미지가 잠금화면 now playing을 덮어씀. 곡별 disposeBag으로 교체
8	P2	AppCoordinator	5개 탭 코디네이터 중 AnimalsCoordinator만 setUpParent(to:) 누락 (latent — 자식 정리 경로 무력화)
9	P2	Localizable.strings	DashboardCoordinator가 사용하는 "Unknown error" 키가 ko/en 모두 부재 — 추가

조사했지만 수정하지 않은 항목 (사유)

• TurnipPrices Monday AM 필수 검증: 전용 안내 메시지까지 구현된 의도된 제품 검증으로 판단 (predictor는 nil 허용하지만 UX 정책 영역)
• TurnipPricesReactor.calculatedPrices 상태 불일치: 해당 상태를 읽는 소비자가 없어 실사용 영향 없음 (데드 상태 — 별도 정리 대상)
• 상세 화면 reactor의 fetch 완료 전 토글 시 nil 상태 오판: 버튼 비활성화 등 제품 결정 필요
• scene 재연결 시 fresh-install 경로 재진입 등 이론적 엣지: 재현 경로 미확인

테스트

• make test — CoreDataStorageICloudResetTests 21개 통과 (progressList 손상 회귀 테스트 2개 신규)
• swiftlint — 에러 0
• make validate — 통과

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 06bb3665-38a5-4f64-898b-e95bfa4adeaa

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/fix-cross-feature-bugs

---

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