Merge release/1.1.3 to develop

[DongChyeon]

Related issue 🛠

closed #<issue_number>

어떤 변경사항이 있었나요?

• 🐞 BugFix Something isn't working
• 🎨 Design Markup & styling
• 📃 Docs Documentation writing and editing (README.md, etc.)
• ✨ Feature Feature
• 🔨 Refactor Code refactoring
• ⚙️ Setting Development environment setup
• ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

• PR 컨벤션에 맞게 작성했습니다. (필수)
• merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
• Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
• BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

• 작업 내용

Uncompleted Tasks 😅

• Task1

To Reviewers 📢

Summary by CodeRabbit

• New Features

  • 홈 업데이트 안내 배너 추가(닫기/다시 보지 않기)
  • 포춘 딥링크 지원(orbitapp://fortune)
  • 알람 발생 후 당일 1회 포춘 자동 생성(백그라운드 스케줄링)

• UI/UX

  • 포춘 로딩 화면 개편(새 애니메이션 및 동적 연출)
  • 내비게이션 바 스크림 및 시스템 바 패딩 반영
  • 알람 상호작용·온보딩·알람 편집 레이아웃 개선
  • 미션 선택 UI 개선 및 선택 상태 표시

• Improvements

  • “하루 첫 알람 해제” 판별 로직 개선(정확도 향상)
  • 재예약 시 비활성 알람 제외 처리
  • 이미지 로딩 성능 및 라이브러리 업데이트
  • 앱 버전 업데이트: 1.1.3

• Permissions

  • 네트워크 상태 권한 추가

• Chores

  • WorkManager/Hilt 통합 및 의존성 정비 (Work, Hilt, Coil 등)

[coderabbitai]

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	PR 제목 'Merge release/1.1.3 to develop'은 소스와 대상 브랜치를 명확히 표시하여 PR의 핵심 목적(릴리스 브랜치 병합)을 정확하게 요약합니다. 간결하고 판독성이 좋아 히스토리 스캔 시 무엇을 수행했는지 분명히 알 수 있습니다. 다만 제목이 변경된 주요 기능이나 버그 수정을 설명하지 않으므로, 빠른 맥락 파악을 위해 더 서술적인 제목을 권장합니다.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch release/1.1.3

---

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

[codecov]

Codecov Report

❌ Patch coverage is 1.16009% with 426 lines in your changes missing coverage. Please review.
✅ Project coverage is 4.06%. Comparing base (374369f) to head (700e803).
⚠️ Report is 70 commits behind head on develop.

Files with missing lines	Patch %	Lines
...e/component/bottomsheet/UpdateNoticeBottomSheet.kt	0.00%	86 Missing ⚠️
...ure/home/src/main/java/com/yapp/home/HomeScreen.kt	0.00%	56 Missing ⚠️
...larm/receivers/AlarmInteractionActivityReceiver.kt	0.00%	47 Missing ⚠️
...m/component/bottomsheet/AlarmMissionBottomSheet.kt	0.00%	45 Missing ⚠️
.../home/src/main/java/com/yapp/home/HomeViewModel.kt	0.00%	41 Missing ⚠️
.../com/yapp/home/alarm/addedit/AlarmAddEditScreen.kt	0.00%	37 Missing ⚠️
...ata/local/datasource/FortuneLocalDataSourceImpl.kt	0.00%	28 Missing ⚠️
...home/component/bottomsheet/AlarmListBottomSheet.kt	0.00%	21 Missing ⚠️
...ain/java/com/yapp/alarm/receivers/AlarmReceiver.kt	0.00%	20 Missing ⚠️
.../main/java/com/yapp/alarm/AndroidAlarmScheduler.kt	0.00%	10 Missing ⚠️
... and 6 more

❌ Your project status has failed because the head coverage (4.06%) is below the target coverage (60.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

@@             Coverage Diff              @@
##             develop    #255      +/-   ##
============================================
- Coverage       4.25%   4.06%   -0.19%     
  Complexity        53      53              
============================================
  Files             50      51       +1     
  Lines           4446    4670     +224     
  Branches         649     689      +40     
============================================
+ Hits             189     190       +1     
- Misses          4247    4470     +223     
  Partials          10      10              

Files with missing lines	Coverage Δ
.../data/local/datasource/AlarmLocalDataSourceImpl.kt	0.00% <ø> (ø)
...om/yapp/data/repositoryimpl/AlarmRepositoryImpl.kt	0.00% <ø> (ø)
...e/home/src/main/java/com/yapp/home/HomeContract.kt	0.00% <0.00%> (ø)
.../main/java/com/yapp/alarm/services/AlarmService.kt	0.00% <0.00%> (ø)
...yapp/data/repositoryimpl/UserInfoRepositoryImpl.kt	0.00% <0.00%> (ø)
...p/data/local/datasource/UserLocalDataSourceImpl.kt	0.00% <0.00%> (ø)
...om/yapp/alarm/receivers/RescheduleAlarmReceiver.kt	0.00% <0.00%> (ø)
.../yapp/data/repositoryimpl/FortuneRepositoryImpl.kt	42.30% <35.71%> (+0.64%)	⬆️
.../main/java/com/yapp/alarm/AndroidAlarmScheduler.kt	0.00% <0.00%> (ø)
...ain/java/com/yapp/alarm/receivers/AlarmReceiver.kt	0.00% <0.00%> (ø)
... and 8 more

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[coderabbitai]

Actionable comments posted: 9

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (7)

core/network/src/main/java/com/yapp/network/di/NetworkModule.kt (2)

41-50: 타임아웃 300초는 과도 — 실패 감지 지연/리소스 점유 증가

보편값으로는 과합니다. 일반 API는 더 짧게, 대용량 업/다운로드만 per-call로 늘리는 게 안전합니다. 총 호출 제한을 위해 callTimeout도 고려해 주세요.

-            .readTimeout(300, TimeUnit.SECONDS)
-            .writeTimeout(300, TimeUnit.SECONDS)
-            .connectTimeout(300, TimeUnit.SECONDS)
+            .connectTimeout(15, TimeUnit.SECONDS)
+            .readTimeout(30, TimeUnit.SECONDS)
+            .writeTimeout(30, TimeUnit.SECONDS)
+            .callTimeout(60, TimeUnit.SECONDS)

---

41-50: 긴급: 인증 Interceptor/Authenticator 누락 — 보호 API 실패(블로커), 즉시 복구 필요

core/network/src/main/java/com/yapp/network/di/NetworkModule.kt의 provideHttpClient(라인 41–49)에서 OkHttpClient에 loggingInterceptor만 등록되어 있으며, 저장소(.kt/.java) 전체 검색 결과 addInterceptor/.authenticator 호출 및 Authorization 토큰 주입 흔적이 발견되지 않음 — 보호 API는 Authorization 헤더가 없으면 401/실패 발생.
조치(권장): provideHttpClient에 인증 Interceptor 및/또는 Authenticator 의존성 주입 후 Builder에 적용(예: optional authInterceptor/authenticator 파라미터 후 addInterceptor/authenticator 호출) 및 토큰 주입·401 재발급 로직 정상 동작 확인.

feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingViewModel.kt (1)

134-147: 빈 사운드 목록 시 IndexOutOfBoundsException 위험.

sounds가 빈 리스트면 index 0 접근에서 크래시가 납니다. 안전 가드를 추가하세요.

다음처럼 기본 선택 로직을 안전하게 바꾸는 것을 제안합니다.

-        alarmUseCase.getAlarmSounds().onSuccess { sounds ->
-            val defaultSoundIndex = sounds.indexOfFirst { it.title == "Homecoming" }.takeIf { it >= 0 } ?: 0
-            val defaultSoundUri = sounds[defaultSoundIndex]
+        alarmUseCase.getAlarmSounds().onSuccess { sounds ->
+            val defaultSound = sounds.firstOrNull { it.title == "Homecoming" } ?: sounds.firstOrNull()
+            if (defaultSound == null) {
+                Log.w("OnboardingViewModel", "No alarm sounds available; skip creating alarm")
+                return@onSuccess
+            }

그리고 사용부:

-                soundUri = "${defaultSoundUri.uri}",
+                soundUri = defaultSound.uri,

feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingGenderScreen.kt (1)

73-79: 선택 이벤트 로깅이 누락되었습니다 (logEvent 미주입).

OnboardingGenderScreen의 logEvent 파라미터가 기본값 빈 람다여서 성별 선택 로그가 사라집니다. Route에서 주입하세요.

다음과 같이 주입을 권장합니다.

     OnboardingGenderScreen(
         state = state,
         bottomSheetState = bottomSheetState,
         currentStep = 5,
         totalSteps = 6,
         processAction = viewModel::processAction,
+        logEvent = analyticsHelper::logEvent,
     )

feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/AlarmInteractionActivity.kt (1)

102-105: API 33 전용 registerReceiver 오버로드 사용으로 하위 OS에서 크래시 발생

RECEIVER_EXPORTED 플래그를 받는 3-인자 오버로드는 Android 13(API 33)+ 전용입니다. 현재 코드 그대로면 API 32 이하에서 NoSuchMethodError로 크래시가 납니다. SDK 가드로 분기해 주세요.

 private fun registerAlarmInteractionActivityCloseReceiver() {
     val filter = IntentFilter(AlarmConstants.ACTION_ALARM_INTERACTION_ACTIVITY_CLOSE)
-    registerReceiver(broadcastReceiver, filter, RECEIVER_EXPORTED)
+    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
+        registerReceiver(broadcastReceiver, filter, RECEIVER_EXPORTED)
+    } else {
+        @Suppress("DEPRECATION")
+        registerReceiver(broadcastReceiver, filter)
+    }
 }

core/common/src/main/java/com/yapp/common/navigation/route/MissionRoute.kt (1)

3-3: 잘못된 import로 컴파일 실패 가능성

MissionMode는 com.yapp.domain.model 패키지로 보입니다. import 경로를 수정해 주세요.

-import com.yapp.domain.MissionMode
+import com.yapp.domain.model.MissionMode

feature/home/src/main/java/com/yapp/home/HomeScreen.kt (1)

295-299: LocalDensity 캡처 누락으로 컴파일 에러 유발.

아래에서 placeable.height.toDp()를 호출하지만 Density 스코프가 없어 컴파일에 실패합니다. LocalDensity.current를 캡처해 사용해 주세요.

     var sheetHalfExpandHeight by remember { mutableStateOf(0.dp) }
 
+    val density = LocalDensity.current

🧹 Nitpick comments (70)

feature/home/src/main/java/com/yapp/home/alarm/addedit/AlarmAddEditScreen.kt (4)

308-316: 하드코딩된 20.dp 반복 최소화

동일한 수평 패딩 값이 여러 곳에서 반복됩니다. 로컬 상수로 추출해 유지보수성을 높이면 좋겠습니다.

아래처럼 간단히 정리할 수 있습니다:

@@
-    OrbitBottomSheetLayout(sheetState = bottomSheetState) {
-        Column(
+    OrbitBottomSheetLayout(sheetState = bottomSheetState) {
+        val contentHorizontalPadding = 20.dp
+        Column(
@@
-            AlarmAddEditSelectDaysSection(
-                modifier = Modifier.padding(horizontal = 20.dp),
+            AlarmAddEditSelectDaysSection(
+                modifier = Modifier.padding(horizontal = contentHorizontalPadding),
@@
-            AlarmAddEditSettingsSection(
-                modifier = Modifier.padding(horizontal = 20.dp),
+            AlarmAddEditSettingsSection(
+                modifier = Modifier.padding(horizontal = contentHorizontalPadding),
@@
-                    .padding(
-                        start = 20.dp,
-                        end = 20.dp,
+                    .padding(
+                        start = contentHorizontalPadding,
+                        end = contentHorizontalPadding,
                         bottom = 12.dp,
                     ),

Also applies to: 325-329

---

286-286: 네비게이션 바 패딩 중복 여지

OrbitBottomSheetLayout이 내부에서 navigationBarsPadding()을 적용하므로, 저장 버튼의 bottom = 12.dp와 합쳐져 일부 기기(3‑button 내비게이션 등)에서 지나치게 큰 하단 여백이 생길 수 있습니다. UX측 확인 후 필요 시 버튼 컨테이너에는 imePadding()만 추가하거나, 하단 여백을 조건부로 줄이는 것을 고려해 주세요.

Also applies to: 321-329

---

321-324: 저장 버튼 활성화 조건 연동 제안

항상 enabled = true로 되어 있어 비정상 입력/미변경 상태에서도 저장 가능할 수 있습니다. 유효성/변경 여부 플래그가 있다면 연동을 권장합니다(예: enabled = state.isSaveEnabled).

---

288-331: 소형 화면/폰트 확대한 환경에서의 스크롤 가능성 검토

현재 루트 Column은 스크롤이 없어, 내용이 길어지는 상태(언어/폰트 스케일, 시스템 제스처 영역 합산)에서 하단 버튼이 가려질 수 있습니다. 여유가 있다면 LazyColumn 전환 혹은 섹션별 스크롤 처리 검토를 제안합니다.

feature/home/src/main/java/com/yapp/home/component/bottomsheet/AlarmListBottomSheet.kt (1)

326-331: 두 개의 DropdownMenu가 동시에 열릴 수 있는 상태 공간

이제 항상 Compose 트리에 존재하므로 menuExpanded와 sortDropDownMenuExpanded가 동시에 true가 되면 두 메뉴가 겹쳐 열릴 수 있습니다. 호출 측 불변식으로 상호 배타를 보장하지 않는다면, 단일 상태로 통합하는 편이 안전합니다.

간단 제안:

• 단일 상태로 통합: enum class Menu { None, Main, Sort }를 사용하고 expanded = (menu == Main/Sort)로 제어.
• 혹은 onClickSort에서 반드시 menuExpanded=false를 보장.

호출부에서 두 불린을 동시에 true로 만들지 않는 로직이 있는지 확인해주세요. 없으면 위 리팩터를 권장합니다.

Also applies to: 333-338

data/src/main/java/com/yapp/data/remote/di/ServiceModule.kt (1)

16-17: 메서드 네이밍 일관성(nit): provide… 형태 권장

Hilt 관례상 provideXxx로 맞추면 가독성이 좋아집니다.

-    fun providesApiService(retrofit: Retrofit): ApiService =
+    fun provideApiService(retrofit: Retrofit): ApiService =

core/network/src/main/java/com/yapp/network/di/NetworkModule.kt (3)

54-63: kotlinx.serialization 컨버터 Opt‑in 필요 가능성

프로젝트 설정에 따라 asConverterFactory 사용 시 ExperimentalSerializationApi Opt‑in이 필요할 수 있습니다. 컴파일 경고/에러가 보인다면 아래처럼 명시 Opt‑in 해 주세요.

-    fun provideRetrofit(
+    @OptIn(kotlinx.serialization.ExperimentalSerializationApi::class)
+    fun provideRetrofit(
         okHttpClient: OkHttpClient,
         buildConfigFieldProvider: BuildConfigFieldProvider,
         json: Json,
     ): Retrofit =

---

54-63: 비‑JSON/빈 응답 대비: Scalars 컨버터 선등록 권장

서버가 text/plain 또는 빈 바디(204/200 without body)를 반환하는 엔드포인트가 있다면 JSON 컨버터만으로는 실패할 수 있습니다. 필요 시 ScalarsConverterFactory를 JSON보다 먼저 추가해 주세요.

         Retrofit.Builder()
-            .addConverterFactory(json.asConverterFactory("application/json".toMediaType()))
+            .addConverterFactory(retrofit2.converter.scalars.ScalarsConverterFactory.create())
+            .addConverterFactory(json.asConverterFactory("application/json".toMediaType()))
             .baseUrl(buildConfigFieldProvider.get().baseUrl)
             .client(okHttpClient)
             .build()

---

54-63: baseUrl 트레일링 슬래시 보장

Retrofit은 baseUrl이 반드시 '/'로 끝나야 합니다. BuildConfig 값이 보장되지 않으면 런타임 예외가 납니다. 방어 코드 추가를 권장합니다.

-            .baseUrl(buildConfigFieldProvider.get().baseUrl)
+            .baseUrl(
+                buildConfigFieldProvider.get().baseUrl.let {
+                    if (it.endsWith("/")) it else "$it/"
+                }
+            )

feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingGenderScreen.kt (2)

168-168: 하드코딩된 버튼 라벨 i18n 처리 필요.

다국어 및 접근성 일관성을 위해 stringResource 사용을 권장합니다.

리소스 키는 프로젝트 상황에 맞게 교체하세요.

-            buttonLabel = "다음",
+            buttonLabel = stringResource(id = R.string.common_button_next),

---

190-206: UI 라벨을 상태 값으로 직접 저장/비교하는 패턴은 i18n/도메인 일관성에 취약합니다.

표시 문자열(남성/여성) 대신 도메인 코드(예: "MALE"/"FEMALE" 또는 enum)를 상태에 보관하고, 라벨은 stringResource로 매핑하세요.

국제화와 서버 스키마 변경에 강한 형태로 다음과 같이 개선을 제안합니다.

-                    listOf("남성", "여성").forEach { gender ->
+                    listOf(
+                        "MALE" to stringResource(R.string.gender_male),
+                        "FEMALE" to stringResource(R.string.gender_female),
+                    ).forEach { (genderCode, genderLabel) ->
                         Box(modifier = Modifier.weight(1f)) {
                             OrbitGenderToggle(
-                                label = gender,
-                                isSelected = state.selectedGender == gender,
+                                label = genderLabel,
+                                isSelected = state.selectedGender == genderCode,
                                 onToggle = {
                                     logEvent(
                                         AnalyticsEvent(
                                             type = "onboarding_gender_select",
                                             properties = mapOf(
-                                                AnalyticsEvent.OnboardingPropertiesKeys.GENDER to gender,
+                                                AnalyticsEvent.OnboardingPropertiesKeys.GENDER to genderCode,
                                             ),
                                         ),
                                     )
-                                    processAction(OnboardingContract.Action.UpdateGender(gender))
+                                    processAction(OnboardingContract.Action.UpdateGender(genderCode))
                                 },
                             )
                         }
                     }

리소스 키(R.string.gender_male/female)는 실제 프로젝트 키로 교체 바랍니다.

feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/action/AlarmActionContract.kt (1)

17-18: Boolean? → Boolean으로 단순화하고 null 게이팅 제거 제안

초기 로딩은 initialLoading로 이미 가려지므로 shouldShowMissionStart의 삼중 상태(null/true/false)는 불필요합니다. 불변(Boolean)로 좁혀 타입 안정성과 분기 단순화를 가져가면 좋겠습니다.

아래처럼 변경 시, 화면 쪽의 isFirstMission != null 분기도 제거 가능합니다.

-        val shouldShowMissionStart: Boolean? = null,
+        val shouldShowMissionStart: Boolean = false,

화면 반영(다른 파일)도 함께 필요합니다:

-    isFirstMission: Boolean?,
+    shouldShowMissionStart: Boolean,

및 버튼 렌더링에서 null 분기 제거.

feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/action/AlarmActionScreen.kt (5)

115-126: 파라미터 명/타입을 상태와 일치하도록 정리

컴포저블 파라미터 isFirstMission: Boolean?는 상태명(shouldShowMissionStart)과 의미가 다르고 nullable입니다. 동일 명칭·불변(Boolean)으로 정리하면 이해와 사용이 쉬워집니다.

-private fun AlarmActionContent(
+private fun AlarmActionContent(
     isAm: Boolean,
     hour: Int,
     minute: Int,
     todayDate: String,
     snoozeEnabled: Boolean,
     snoozeInterval: Int,
     snoozeCount: Int,
-    isFirstMission: Boolean?,
+    shouldShowMissionStart: Boolean,
     onSnoozeClick: () -> Unit,
     onDismissClick: () -> Unit,
 )

호출부도 함께:

-            isFirstMission = state.shouldShowMissionStart,
+            shouldShowMissionStart = state.shouldShowMissionStart,

---

170-189: 버튼 렌더링의 null 게이팅 제거로 UI 단순화

initialLoading 화면이 있으므로 여기서의 null 분기는 불필요합니다. 항상 버튼을 렌더링하고 라벨만 분기하세요.

-        if (isFirstMission != null) {
-            OrbitButton(
-                label = if (isFirstMission) {
+        OrbitButton(
+            label = if (shouldShowMissionStart) {
                 stringResource(id = R.string.alarm_off_mission_start_btn)
             } else {
                 stringResource(id = R.string.alarm_off_btn)
-            },
-            enabled = true,
-            modifier = Modifier
-                .padding(
-                    start = 40.dp,
-                    end = 40.dp,
-                    bottom = 48.dp,
-                )
-                .height(62.dp),
-            onClick = onDismissClick,
-            )
-        } else {
-            Spacer(modifier = Modifier.height(62.dp))
-        }
+            },
+            enabled = true,
+            modifier = Modifier
+                .padding(start = 40.dp, end = 40.dp, bottom = 48.dp)
+                .height(62.dp),
+            onClick = onDismissClick,
+        )

---

150-154: contentDescription 하드코딩 → stringResource로 i18n

접근성 문자열은 리소스로 분리하는 것이 좋습니다. 리소스 추가 후 교체를 권장합니다.

예:

-            contentDescription = "Alarm Action Character",
+            contentDescription = stringResource(id = R.string.alarm_action_character_cd),

---

208-213: "오전/오후" 하드코딩 제거

다국어 대응을 위해 문자열 리터럴 대신 stringResource 사용을 권장합니다.

예:

text = stringResource(if (isAm) R.string.am else R.string.pm)

---

299-313: 프리뷰 케이스 보강(미션 유무 2종)

미션 시작/일반 종료 버튼 라벨을 모두 검증할 수 있도록 프리뷰를 2개로 나누는 것을 권장합니다.

feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/action/AlarmActionViewModel.kt (3)

30-36: 초기 Intent 순서 OK, 다만 초기로딩 해제 타이밍만 일관화 제안

fetchShouldShowMissionStart() → initializeAlarmState() → startClock() 순서는 합리적입니다. initialLoading = false는 매초가 아니라 첫 틱에서만 내리도록 하면 불필요한 상태 갱신을 줄일 수 있습니다.

---

64-82: 무한 루프는 취소 친화적으로

취소 시그널 인지 명확성을 위해 while (true) 대신 coroutineContext.isActive 사용을 권장합니다.

+import kotlinx.coroutines.isActive
@@
-    private fun startClock() = intent {
-        while (true) {
+    private fun startClock() = intent {
+        while (coroutineContext.isActive) {
             val now = LocalTime.now()
             ...
             delay(1000L)
         }
     }

---

84-98: Snooze 다중 탭 가드

연속 탭 시 사이드이펙트 중복 내비게이션 가능성이 있습니다. 일시적 디스에이블 플래그를 상태에 추가하거나, 처리 중 가드를 권장합니다.

feature/home/src/main/AndroidManifest.xml (1)

3-3: 권한 중복 선언 정리 제안

ACCESS_NETWORK_STATE는 보통 앱 모듈 매니페스트에서만 선언합니다. 피처 모듈 선언은 병합 시 중복이 되고, 동적/온디맨드 모듈인 경우 불필요 권한 노출이 될 수 있습니다. 앱 매니페스트에만 유지하고 여기서는 제거하는 것을 권장합니다.

적용 diff:

-    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
+    <!-- moved to app/src/main/AndroidManifest.xml -->

core/alarm/src/main/java/com/yapp/alarm/scheduler/PostFortuneTaskScheduler.kt (1)

3-5: 스케줄링 가시성 향상을 위한 반환값/문서화 제안

호출 측에서 “큐잉 성공/스킵” 여부나 트래킹 ID가 필요할 수 있습니다. Boolean(성공/스킵) 반환 또는 KDoc로 동작 보장을 명시하는 개선을 제안합니다.

예시 diff:

-interface PostFortuneTaskScheduler {
-    fun enqueueOnceForToday()
-}
+/**
+ * 오늘 날짜 기준으로 1회만 큐잉합니다(중복 방지).
+ * @return true면 새로 큐잉됨, false면 이미 큐잉되어 스킵됨.
+ */
+interface PostFortuneTaskScheduler {
+    fun enqueueOnceForToday(): Boolean
+}

core/alarm/src/main/java/com/yapp/alarm/AndroidAlarmScheduler.kt (1)

20-27: 로그 오버헤드 절감(스택트레이스/릴리스 빌드 차단)

Throwable 스택트레이스는 비용이 큽니다. 디버그/로거 가드로 제한하고 스택트레이스는 제거하세요.

권장 diff:

-    private fun logSchedule(tag: String, alarm: Alarm, triggerMillis: Long, extra: String = "") {
-        Log.d("ScheduleTrace", "scheduleAlarm Called", Throwable())
-        Log.d(
-            "AlarmSchedule",
-            "[$tag] id=${alarm.id}, repeatDays=${alarm.repeatDays}, " +
-                "time=${java.time.Instant.ofEpochMilli(triggerMillis)} $extra",
-        )
-    }
+    private fun logSchedule(tag: String, alarm: Alarm, triggerMillis: Long, extra: String = "") {
+        if (!BuildConfig.DEBUG && !Log.isLoggable("AlarmSchedule", Log.DEBUG)) return
+        Log.d(
+            "AlarmSchedule",
+            "[$tag] id=${alarm.id}, repeatDays=${alarm.repeatDays}, " +
+                "time=${java.time.Instant.ofEpochMilli(triggerMillis)} $extra"
+        )
+    }

core/ui/src/main/java/com/yapp/ui/component/navigation/NavigationBarScrim.kt (1)

16-26: 재사용성 개선: 색상/모디파이어 파라미터화 제안

테마 스크림 컬러 사용이나 zIndex 조정이 필요할 수 있어 파라미터화하면 활용도가 올라갑니다.

예시 diff:

-@Composable
-fun BoxScope.NavigationBarScrim() {
-    Box(
-        modifier = Modifier
-            .align(Alignment.BottomCenter)
-            .fillMaxWidth()
-            .windowInsetsBottomHeight(WindowInsets.navigationBars)
-            .background(Color.Black)
-            .zIndex(1f),
-    )
-}
+@Composable
+fun BoxScope.NavigationBarScrim(
+    modifier: Modifier = Modifier,
+    color: Color = Color.Black,
+    zIndex: Float = 1f,
+) {
+    Box(
+        modifier = modifier
+            .align(Alignment.BottomCenter)
+            .fillMaxWidth()
+            .windowInsetsBottomHeight(WindowInsets.navigationBars)
+            .background(color)
+            .zIndex(zIndex),
+    )
+}

feature/home/src/main/res/values/strings.xml (2)

51-53: '선택됨' 라벨의 사용 맥락을 a11y/UX 관점에서 확인해 주세요.

학습 메모상(selectedMissionType/Count는 내부 상태)과 같이 '선택됨'이 가시 텍스트로 노출되면 저장/확정 상태로 오인될 수 있습니다. 스크린리더용 contentDescription로만 쓰이거나, UI에서는 체크/선택 상태가 명확히 전달되도록 확인 부탁드립니다.

---

128-129: '다시 보지 않기' 동작 타입(체크박스 vs 버튼) 명확화 및 텍스트 일관성 확인

해당 문자열이 토글(체크박스)인지, 1회성 버튼인지에 따라 접근성 레이블/상태 안내가 달라집니다. 위젯 타입과 일치하도록 보조 텍스트(예: 설명/접근성 레이블)를 추가할지 검토해 주세요.

app/src/main/java/com/yapp/orbit/di/AppVersionModule.kt (1)

11-18: @nAmed 대신 타입 세이프한 @qualifier 사용 제안

문자열 Qualifier는 충돌/오타 리스크가 있습니다. 전용 @qualifier를 도입해 타입 세이프하게 관리해 주세요.

 package com.yapp.orbit.di
@@
 import dagger.hilt.components.SingletonComponent
-import javax.inject.Named
 import javax.inject.Singleton
+import javax.inject.Qualifier
@@
-@Module
-@InstallIn(SingletonComponent::class)
-object AppVersionModule {
+@Qualifier
+@Retention(AnnotationRetention.BINARY)
+annotation class AppVersion
+
+@Module
+@InstallIn(SingletonComponent::class)
+object AppVersionModule {
     @Provides
     @Singleton
-    @Named("appVersion")
-    fun provideAppVersion(): String = BuildConfig.VERSION_NAME
+    @AppVersion
+    fun provideAppVersion(): String = BuildConfig.VERSION_NAME
 }

core/designsystem/src/main/res/raw/fortune_loading.json (1)

1-1: Lottie 원본 라이선스/크기 검증 필요

• 대용량 base64 WebP 에셋 인라인은 APK/메모리 풋프린트를 늘립니다. 가능하면 벡터화/에셋 분리/압축을 검토하세요.
• 원본 라이선스(상업적 이용/변경 허용 여부)와 귀속 표기를 확인해 주세요.

domain/src/main/java/com/yapp/domain/model/AlarmDay.kt (1)

16-23: 양방향 매핑 로직은 맞습니다만, enum 순서 의존성 제거 권장

현재 구현은 entries/ordinal에 의존합니다. enum 순서 변경에 취약하므로 명시적 매핑으로 바꾸는 것을 권장합니다.

-fun AlarmDay.toDayOfWeek(): DayOfWeek {
-    return DayOfWeek.of(((this.ordinal + 6) % 7) + 1)
-}
+fun AlarmDay.toDayOfWeek(): DayOfWeek = when (this) {
+    AlarmDay.SUN -> DayOfWeek.SUNDAY
+    AlarmDay.MON -> DayOfWeek.MONDAY
+    AlarmDay.TUE -> DayOfWeek.TUESDAY
+    AlarmDay.WED -> DayOfWeek.WEDNESDAY
+    AlarmDay.THU -> DayOfWeek.THURSDAY
+    AlarmDay.FRI -> DayOfWeek.FRIDAY
+    AlarmDay.SAT -> DayOfWeek.SATURDAY
+}
 
-fun DayOfWeek.toAlarmDay(): AlarmDay {
-    val index = (this.value % 7)
-    return AlarmDay.entries[index]
-}
+fun DayOfWeek.toAlarmDay(): AlarmDay = when (this) {
+    DayOfWeek.SUNDAY -> AlarmDay.SUN
+    DayOfWeek.MONDAY -> AlarmDay.MON
+    DayOfWeek.TUESDAY -> AlarmDay.TUE
+    DayOfWeek.WEDNESDAY -> AlarmDay.WED
+    DayOfWeek.THURSDAY -> AlarmDay.THU
+    DayOfWeek.FRIDAY -> AlarmDay.FRI
+    DayOfWeek.SATURDAY -> AlarmDay.SAT
+}

data/src/main/java/com/yapp/data/local/datasource/UserLocalDataSource.kt (1)

9-10: Epoch 단위/타임존 명세 필요

• updateNoticeLastShownDateEpochFlow의 단위(초/밀리초)와 기준(UTC/local midnight) 명시가 필요합니다. 날짜 비교 로직과 불일치 시 오동작할 수 있습니다.
• markUpdateNoticeShownToday가 어떤 기준 시각을 저장하는지 KDoc에 명확히 남겨 주세요.

Also applies to: 15-16

app/src/main/java/com/yapp/orbit/OrbitApplication.kt (1)

11-11: 빈 기본 생성자 제거 가능

static analysis 도구가 지적한 것처럼 빈 기본 생성자 ()는 제거할 수 있습니다.

-class OrbitApplication() : Application(), Configuration.Provider {
+class OrbitApplication : Application(), Configuration.Provider {

app/src/main/AndroidManifest.xml (1)

5-5: ACCESS_NETWORK_STATE 권한 중복 선언

android.permission.ACCESS_NETWORK_STATE 권한이 5번과 14번 라인에 중복 선언되어 있습니다.

14번 라인의 중복된 권한 선언을 제거하세요:

-    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>

Also applies to: 14-14

core/alarm/src/main/java/com/yapp/alarm/receivers/AlarmReceiver.kt (1)

114-119: 로컬 함수 선언 위치 개선 필요

ringsToday() 함수가 코루틴 스코프 내부에 선언되어 있습니다. 이 함수는 상태를 변경하지 않고 순수 함수이므로 클래스 레벨이나 상단에 선언하는 것이 더 적절합니다.

함수를 클래스의 private 메서드로 이동하거나 companion object에 추가하세요:

-                fun Alarm.ringsToday(): Boolean {
-                    if (repeatDays == 0) return true
-
-                    val todayAlarmDay = LocalDate.now().dayOfWeek.toAlarmDay()
-                    return (repeatDays and todayAlarmDay.bitValue) != 0
-                }

클래스 레벨에 추가:

private fun Alarm.ringsToday(): Boolean {
    if (repeatDays == 0) return true
    
    val todayAlarmDay = LocalDate.now().dayOfWeek.toAlarmDay()
    return (repeatDays and todayAlarmDay.bitValue) != 0
}

domain/src/main/java/com/yapp/domain/repository/FortuneRepository.kt (2)

20-20: 이름 불일치 가능성: markFortuneAsFailed vs markFortuneFailed

datastore(UserPreferences) 쪽 구현은 markFortuneFailed로 보입니다. 레이어 간 명명 통일을 권장합니다. 도메인 메서드를 markFortuneFailed로 맞추거나 데이터 계층을 AsFailed로 맞춰 주세요.

-    suspend fun markFortuneAsFailed()
+    suspend fun markFortuneFailed()

---

9-16: 확인 완료 — 도메인 API 변경이 전파됨; KDoc 마이그레이션 가이드 추가 권장

검증 결과: 이전 심볼(fortuneDateFlow 등)은 코드베이스에서 발견되지 않았고, 새 심볼(fortuneDateEpochFlow, hasUnseenFortuneFlow, shouldShowFortuneToolTipFlow, isFirstAlarmDismissedTodayFlow, fortuneCreateStatusFlow)이 domain/data/feature 모듈 전역에서 사용되고 있습니다.
확인 파일(예): domain/src/main/java/com/yapp/domain/repository/FortuneRepository.kt, data/src/main/java/com/yapp/data/repositoryimpl/FortuneRepositoryImpl.kt, data/src/main/java/com/yapp/data/local/datasource/FortuneLocalDataSourceImpl.kt, core/datastore/UserPreferences.kt, feature/* (Home/Mission/Fortune/Alarm).
조치: 각 Flow의 의미(하루/epoch 기준, reset 조건)와 mark* 계열 메서드의 상태 전이를 간단한 KDoc으로 문서화하세요.

core/datastore/src/main/java/com/yapp/datastore/UserPreferences.kt (2)

44-45: todayEpoch 테스트 가능성/결정성 개선 제안

로컬 시간대/자정 경계 이슈 테스트를 위해 Clock 주입(또는 provider 함수 주입)을 권장합니다.

예: 생성자에 Clock 주입 후 LocalDate.now(clock).toEpochDay() 사용.

---

108-116: FIRST_ALARM_DISMISSED_TODAY 플래그 정리(옵션)

플로우에서 날짜 비교로 안전하지만, 데이터 청결을 위해 날짜가 바뀌면 플래그를 false로 재설정하는 유지보수용 정리 로직을 고려해볼 수 있습니다.

feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/AlarmInteractionActivity.kt (1)

52-65: 네비게이션 바 패딩 + 스크림 조합 사용 시 이중 패딩 여부 확인 요청

NavHost에 navigationBarsPadding을 주고 바닥에 스크림을 깔았으니 대부분 OK입니다. 다만 내부 화면들이 별도로 navigationBarsPadding/WindowInsets를 처리하고 있다면 하단 여백이 과다해질 수 있어 실제 단말(제스처/3버튼 내비 모두)에서 확인 부탁드립니다. 또한 Box에 크기 제약이 모호한 경우를 막으려면 fillMaxSize를 부여하는 것도 고려해 주세요.

core/common/src/main/java/com/yapp/common/navigation/route/MissionRoute.kt (1)

8-11: 문자열 대신 타입 세이프 enum 사용 고려

missionMode: String 대신 MissionMode 자체를 사용하면 오타/스펙 불일치를 컴파일 타임에 차단할 수 있습니다(필요 시 @serializable enum). 네비게이션 인코딩/디코딩 영향 범위를 감안해 후속 리팩터로 검토해 주세요.

feature/fortune/src/main/java/com/yapp/fortune/page/FortunePager.kt (1)

43-61: 페이지 인덱스 매직넘버 정리 제안

in 1..4, 5 -> 등 하드코딩된 인덱스는 유지보수 시 오류를 유발합니다. 상수/파생 값으로 중앙집중화하거나 state 기반으로 계산하도록 정리해 주세요.

feature/fortune/src/main/java/com/yapp/fortune/FortuneNavGraph.kt (1)

23-27: 딥링크 URI 패턴 상수화

"orbitapp://fortune" 문자열을 상수로 분리해 재사용하면 오탈자/중복 관리가 수월합니다.

feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/snooze/AlarmSnoozeTimerViewModel.kt (1)

47-49: Clock 주입으로 테스트 용이성/일관성 향상 제안

LocalDate.now() 대신 Clock을 DI 받아 LocalDate.now(clock)을 쓰면 오프라인/타임존/테스트 시나리오에서 안정적입니다.

// 예시
class AlarmSnoozeTimerViewModel @Inject constructor(
    private val clock: Clock,
    ...
) : ViewModel(), ... {
    ...
    val todayDate = LocalDate.now(clock).toEpochDay()
}

feature/fortune/src/main/java/com/yapp/fortune/scheduler/WorkManagerPostFortuneTaskScheduler.kt (1)

21-31: 일일 고유 워크 이름 전략 OK — 태그/모니터링 추가 권장

유니크 네이밍+KEEP 정책은 의도에 부합합니다. 운영 모니터링을 위해 태그를 추가하고, 실패/재시도 추적이 필요하면 로그/분석 이벤트를 Worker에서 발행하세요.

 val req = OneTimeWorkRequestBuilder<PostFortuneWorker>()
     .setConstraints(constraints)
-    .setBackoffCriteria(BackoffPolicy.EXPONENTIAL, 15, TimeUnit.SECONDS)
+    .setBackoffCriteria(BackoffPolicy.EXPONENTIAL, 15, TimeUnit.SECONDS)
+    .addTag(POST_FORTUNE_TAG)
     .build()

// 파일 하단 등
private const val POST_FORTUNE_TAG = "post_fortune"

또한, PostFortuneWorker가 @HiltWorker로 선언되고 HiltWorkerFactory가 Application에 연결되어 있는지 확인해 주세요.

domain/src/main/java/com/yapp/domain/repository/UserInfoRepository.kt (1)

11-12: 이름 명확성: Epoch → EpochDay 권장

updateNoticeLastShownDateEpochFlow는 밀리초 epoch로 오해될 수 있습니다. 일 단위라면 ...EpochDayFlow와 같이 명확히 하는 것을 고려해 주세요(도메인 전반 동시 정리 전제).

feature/fortune/src/main/java/com/yapp/fortune/FortuneScreen.kt (4)

220-224: LottieAnimation의 width가 무시됩니다 (내부에서 fillMaxWidth 고정).

현재 LottieAnimation 컴포저블이 내부적으로 modifier.fillMaxWidth()를 강제하므로, 호출부의 .width(375.dp)는 효과가 없습니다. 의도대로 가로/세로 비율을 맞추려면 aspectRatio로 제어하거나 width를 제거하세요.

아래처럼 수정 권장:

-            LottieAnimation(
-                modifier = Modifier
-                    .width(375.dp)
-                    .height(267.dp),
-                resId = core.designsystem.R.raw.fortune_loading,
-            )
+            LottieAnimation(
+                modifier = Modifier
+                    .fillMaxWidth()
+                    .aspectRatio(375f / 267f),
+                resId = core.designsystem.R.raw.fortune_loading,
+            )

추가 import:

+import androidx.compose.foundation.layout.aspectRatio

---

174-181: 슬라이딩 인디케이터 count 하드코딩(6) → 실제 페이지 수와 동기화 필요.

rememberPagerState(pageCount = { state.fortunePages.size + 2 })와 불일치 시 지표 개수 오류가 납니다. 동일 계산식 또는 pagerState.pageCount로 치환하세요.

-                SlidingIndicator(
-                    currentIndex = pagerState.currentPage,
-                    count = 6,
+                SlidingIndicator(
+                    currentIndex = pagerState.currentPage,
+                    count = pagerState.pageCount, // 대안: state.fortunePages.size + 2

PagerState.pageCount 가용 여부 확인 부탁드립니다. 불가하면 동일 람다 계산식을 사용하세요.

---

83-91: 빈 이벤트 타입("") 로깅 방지.

else -> ""로 설정된 경우에도 로그를 쌓습니다. 공백인 경우 로깅을 스킵하세요.

-        analyticsHelper.logEvent(
-            AnalyticsEvent(
-                type = eventType,
-                properties = mapOf(
-                    AnalyticsEvent.FortunePropertiesKeys.FORTUNE_PAGE_NUMBER to pagerState.currentPage + 1,
-                ),
-            ),
-        )
+        if (eventType.isNotBlank()) {
+            analyticsHelper.logEvent(
+                AnalyticsEvent(
+                    type = eventType,
+                    properties = mapOf(
+                        AnalyticsEvent.FortunePropertiesKeys.FORTUNE_PAGE_NUMBER to pagerState.currentPage + 1,
+                    ),
+                ),
+            )
+        }

---

191-199: 무한 루프 LaunchedEffect 단순화 가능.

while(true) + delay 대신 rememberInfiniteTransition 등 애니메이션 API로 토글하면 가독성과 취소 처리 모두 개선됩니다. 기능은 동일하므로 선택사항입니다.

val transition = rememberInfiniteTransition(label = "deliverToggle")
val phase by transition.animateFloat(
    initialValue = 0f, targetValue = 1f,
    animationSpec = infiniteRepeatable(animation = tween(2000), repeatMode = RepeatMode.Reverse),
    label = "phase"
)
val isDelivering = phase > 0.5f

data/src/main/java/com/yapp/data/repositoryimpl/UserInfoRepositoryImpl.kt (1)

20-21: Epoch 단위 명시 필요(초 vs 밀리초).

updateNoticeLastShownDateEpochFlow: Flow<Long?>의 단위가 불명확합니다. 인터페이스/도메인 전반에서 epochMillis 등으로 이름을 명시하거나 KDoc에 단위를 고정해 주세요.

data/src/main/java/com/yapp/data/local/datasource/UserLocalDataSourceImpl.kt (1)

14-15: Epoch 단위 명시 필요(초 vs 밀리초).

updateNoticeLastShownDateEpochFlow의 단위를 인터페이스와 일관되게 문서화/명명해 주세요. 상위 레이어와 동일 제안 적용 권장.

feature/fortune/src/main/java/com/yapp/fortune/FortuneViewModel.kt (2)

53-72: 중복 Success 방지: 동일 상태 재방출 시 중복 fetch 가능.

collect는 동일 Success 재방출에도 fetchAndUpdateFortune를 다시 호출할 수 있습니다. distinctUntilChanged/collectLatest로 디바운스하세요.

-import kotlinx.coroutines.flow.first
+import kotlinx.coroutines.flow.first
+import kotlinx.coroutines.flow.collectLatest
+import kotlinx.coroutines.flow.distinctUntilChanged
@@
-    private fun observeFortune() = intent {
-        fortuneRepository.fortuneCreateStatusFlow.collect { status ->
+    private fun observeFortune() = intent {
+        fortuneRepository.fortuneCreateStatusFlow
+            .distinctUntilChanged()
+            .collectLatest { status ->

---

99-103: 네트워크 실패 시 UX 대응 필요.

getFortune 실패 시 로딩만 해제되고 화면이 정지 상태가 될 수 있습니다. 홈 이동 또는 재시도/알림 토스트 중 하나로 처리하세요.

         }.onFailure { error ->
             Log.e("FortuneViewModel", "운세 데이터 요청 실패: ${error.message}")
             reduce { state.copy(isLoading = false) }
+            postSideEffect(FortuneContract.SideEffect.NavigateToHome)
         }

core/alarm/src/main/java/com/yapp/alarm/receivers/AlarmInteractionActivityReceiver.kt (2)

49-53: Broadcast 시간 제한 대비: Flow first()에 타임아웃 권장.

브로드캐스트 컨텍스트에서는 지연이 길어지면 OS가 중단할 수 있습니다. withTimeout으로 안전장치 추가를 권장합니다.

-import kotlinx.coroutines.withContext
+import kotlinx.coroutines.withContext
+import kotlinx.coroutines.withTimeout
@@
-                            val (fortuneCreateStatus, hasUnseenFortune) = withContext(Dispatchers.IO) {
-                                val status = fortuneRepository.fortuneCreateStatusFlow.first()
-                                val unseen = fortuneRepository.hasUnseenFortuneFlow.first()
-                                status to unseen
-                            }
+                            val (fortuneCreateStatus, hasUnseenFortune) = withContext(Dispatchers.IO) {
+                                withTimeout(3_000) {
+                                    val status = fortuneRepository.fortuneCreateStatusFlow.first()
+                                    val unseen = fortuneRepository.hasUnseenFortuneFlow.first()
+                                    status to unseen
+                                }
+                            }

---

81-82: Failure/Idle 분기 처리 명시적임.

아무 동작 없음이 의도라면 OK입니다. 필요 시 로깅/트레이싱만 추가 검토.

feature/fortune/src/main/java/com/yapp/fortune/worker/PostFortuneWorker.kt (1)

49-55: 재시도 기준을 구분(재시도 가능/불가)하세요.

서버 4xx 등 영구 실패에서도 Result.retry()를 반환하면 백오프 재시도가 불필요하게 계속될 수 있습니다. 저장소/도메인 에러 타입을 구분해 재시도 불가인 경우 Result.failure()를 반환하는 분기를 추가해 주세요.

-                        onFailure = {
-                            fortuneRepository.markFortuneAsFailed()
-                            // WM 백오프 규칙에 따라 재시도
-                            Result.retry()
-                        },
+                        onFailure = { e ->
+                            fortuneRepository.markFortuneAsFailed()
+                            // 예: 도메인 예외/HTTP 상태에 따라 재시도 여부 결정
+                            if (e is TransientNetworkException /* or 5xx */) {
+                                Result.retry()
+                            } else {
+                                Result.failure()
+                            }
+                        },

feature/home/src/main/java/com/yapp/home/component/bottomsheet/UpdateNoticeBottomSheet.kt (3)

110-115: 액션 영역에 접근성 Role 부여.

버튼 역할을 명시해 스크린리더 탐색 품질을 높여주세요.

-                Box(
+                Box(
                     modifier = Modifier
                         .weight(1f)
-                        .clickable(onClick = onDontShowAgain)
+                        .clickable(role = androidx.compose.ui.semantics.Role.Button, onClick = onDontShowAgain)
                         .padding(vertical = 14.dp),
                     contentAlignment = Alignment.Center,
                 ) {
@@
-                Box(
+                Box(
                     modifier = Modifier
                         .weight(1f)
-                        .clickable(onClick = onClose)
+                        .clickable(role = androidx.compose.ui.semantics.Role.Button, onClick = onClose)
                         .padding(vertical = 14.dp),
                     contentAlignment = Alignment.Center,
                 ) {

Also applies to: 124-128

---

64-70: 스크림 클릭에 Ripple 제거 및 라벨 부여.

전체 스크림 클릭에 물결 효과가 불필요하며, 스크린리더 라벨을 부여하면 더 낫습니다.

-    Box(
+    Box(
         modifier = Modifier
             .fillMaxSize()
-            .background(Color(0xFF17191F).copy(alpha = 0.85f))
-            .clickable(onClick = onClose),
+            .background(Color(0xFF17191F).copy(alpha = 0.85f))
+            .clickable(
+                onClickLabel = stringResource(id = R.string.update_notice_bottom_sheet_close),
+                indication = null,
+                interactionSource = remember { MutableInteractionSource() },
+                onClick = onClose,
+            ),
         contentAlignment = Alignment.BottomCenter,
     ) {

---

93-101: 배너 이미지 로딩 crossfade 추가로 전환 품질 개선.

네트워크 로딩시 깜빡임을 줄입니다.

+import coil.request.ImageRequest
@@
-                AsyncImage(
-                    model = imageUrl,
+                AsyncImage(
+                    model = ImageRequest.Builder(context)
+                        .data(imageUrl)
+                        .crossfade(true)
+                        .build(),
                     contentDescription = null,
                     contentScale = ContentScale.Crop,
                     modifier = Modifier
                         .fillMaxWidth()
                         .aspectRatio(1f),
                 )

feature/mission/src/main/java/com/yapp/mission/MissionViewModel.kt (1)

142-149: 두 Flow를 일관된 스냅샷으로 읽어 조건 판정하세요.

서로 다른 시점의 first() 결과 조합으로 경계 타이밍에서 잘못된 판단이 가능. combine으로 스냅샷을 만든 뒤 first()를 권장합니다.

-        val fortuneCreateStatus = fortuneRepository.fortuneCreateStatusFlow.first()
-        val hasUnseenFortune = fortuneRepository.hasUnseenFortuneFlow.first()
+        val (fortuneCreateStatus, hasUnseenFortune) =
+            kotlinx.coroutines.flow.combine(
+                fortuneRepository.fortuneCreateStatusFlow,
+                fortuneRepository.hasUnseenFortuneFlow,
+            ) { a, b -> a to b }.first()

core/alarm/src/main/java/com/yapp/alarm/services/AlarmService.kt (2)

112-115: Notification ID의 Int 변환 안전화.

Long → Int 캐스팅은 overflow 위험이 있습니다. 항상 양의 32비트 범위로 매핑하세요.

-                startForeground(
-                    notificationId.toInt(),
-                    createNotification(alarm, shouldNavigateToMission(alarm.missionType)),
-                )
+                val notificationIdInt = (notificationId % Int.MAX_VALUE).toInt()
+                startForeground(
+                    notificationIdInt,
+                    createNotification(alarm, shouldNavigateToMission(alarm.missionType)),
+                )

---

199-206: 서비스 생명주기 스코프로 통일.

별도 CoroutineScope 대신 serviceScope를 사용해 취소·종료 시 일관되게 정리되도록 합니다.

-        CoroutineScope(Dispatchers.IO).launch {
+        serviceScope.launch {
             alarmUseCase.updateAlarmActive(
                 id = alarmId,
                 active = false,
             )
         }

feature/home/src/main/java/com/yapp/home/HomeViewModel.kt (1)

457-464: 네트워크 판정 완화/주입형 Clock 고려.

• VALIDATED 미충족(포털/초기 연결)에서도 표시를 막을 수 있습니다. INTERNET만 만족 시 일단 표시하도록 완화하거나, 최소 지연 후 재평가를 고려하세요.
• LocalDate.now() 기반 로직이 많으므로 테스트 용이성과 경계 시점(자정) 제어를 위해 Clock 주입도 고려해 볼 만합니다.

data/src/main/java/com/yapp/data/local/datasource/FortuneLocalDataSource.kt (2)

8-8: epoch 단위 명확히 해주세요.

fortuneDateEpochFlow가 epoch day(Long, toEpochDay) 기반이면 이름을 fortuneDateEpochDayFlow 등으로 더 구체화하거나 KDoc으로 “epoch milliseconds가 아님”을 명시해 주세요. 혼동으로 인한 버그를 줄일 수 있습니다.

---

12-12: Tooltip 네이밍 일관성 깨짐(“ToolTip” vs “Tooltip”).

shouldShowFortuneToolTipFlow와 markFortuneTooltipShown()의 표기가 다릅니다. 일관되게 Tooltip로 맞춰 주세요.

적용 예(인터페이스 기준):

-    val shouldShowFortuneToolTipFlow: Flow<Boolean>
+    val shouldShowFortuneTooltipFlow: Flow<Boolean>

Also applies to: 21-21

feature/home/src/main/java/com/yapp/home/HomeScreen.kt (4)

416-422: API 레벨 분기 대신 안전 영역(WindowInsets.safeDrawing) 사용 권장.

SDK 35 분기는 유지보수 비용이 크고 기기별/제조사별 인셋 차이를 흡수하지 못합니다. 안전하게 WindowInsets.safeDrawing 기반으로 상/하단 인셋을 합산하세요.

-                                val offset = if (Build.VERSION.SDK_INT < 35) {
-                                    0.dp
-                                } else {
-                                    statusBarHeight + navBarHeight
-                                }
-                                sheetHalfExpandHeight = screenHeight - contentHeight - offset
+                                val safeInsets = WindowInsets.safeDrawing.asPaddingValues()
+                                val systemBarsOffset = safeInsets.calculateTopPadding() + safeInsets.calculateBottomPadding()
+                                sheetHalfExpandHeight = screenHeight - contentHeight - systemBarsOffset

다양한 API(30/33/35) 및 제스처 내비게이션/3‑button 내비게이션에서 절반 확장 위치가 의도대로인지 검증 부탁드립니다.

---

515-518: 아이콘 접근성 라벨 현지화 필요.

contentDescription = "Mail" / "Setting"은 하드코딩/영문입니다. stringResource로 현지화된 라벨을 제공해 주세요.

예:

contentDescription = stringResource(id = R.string.cd_mail)
contentDescription = stringResource(id = R.string.cd_setting)

문자열 리소스가 없다면 추가 필요.

Also applies to: 556-560

---

604-607: 장식용 이미지는 contentDescription을 null로.

SkyImage는 장식용으로 보입니다. 스크린리더 중복 낭독을 방지하려면 contentDescription = null 처리하세요.

-        contentDescription = "IMG_MAIN_SKY",
+        contentDescription = null,

---

659-663: CD 일관화: 장식용은 null, 의미 있는 컨트롤은 리소스로.

• 말풍선/별/빈 상태 일러스트 등 장식용은 contentDescription = null 권장.
• AddAlarmButton 내 아이콘은 옆 텍스트가 라벨 역할을 하므로 아이콘 CD는 null이 안전합니다(중복 낭독 방지).

예:

-                contentDescription = "IMG_MAIN_SPEECH_BUBBLE",
+                contentDescription = null,

-            contentDescription = "Add Alarm",
+            contentDescription = null,

Also applies to: 672-680, 748-758, 827-831

data/src/main/java/com/yapp/data/local/datasource/FortuneLocalDataSourceImpl.kt (2)

22-35: 오늘 판정 로직의 시간 소스 단일화/테스트 용이성 개선.

LocalDate.now() 기반 todayEpoch()는 테스트/타임존 경계(자정 전후)에서 취약합니다. 공용 DateProvider/Clock를 주입하거나 UserPreferences의 동일 로직을 단일 소스로 노출해(예: todayEpochDay()), 여기서도 그 값을 사용하세요. 이렇게 하면 도메인/데이터스토어 간 불일치 가능성을 제거할 수 있습니다.

---

36-36: 중복 유틸 제거 제안.

todayEpoch()는 UserPreferences와의 중복 가능성이 있습니다. 한 곳으로 모으고 재사용해 주세요.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8852463 and f1fd6b6.

⛔ Files ignored due to path filters (8)

• core/designsystem/src/main/res/drawable-xhdpi/ic_100_buble.png is excluded by !**/*.png
• core/designsystem/src/main/res/drawable-xhdpi/ic_fortune_delivering_speech_bubble.png is excluded by !**/*.png
• core/designsystem/src/main/res/drawable-xhdpi/ic_fortune_waiting_speech_bubble.png is excluded by !**/*.png
• core/designsystem/src/main/res/drawable-xxhdpi/ic_100_buble.png is excluded by !**/*.png
• core/designsystem/src/main/res/drawable-xxhdpi/ic_fortune_delivering_speech_bubble.png is excluded by !**/*.png
• core/designsystem/src/main/res/drawable-xxhdpi/ic_fortune_waiting_speech_bubble.png is excluded by !**/*.png
• core/designsystem/src/main/res/drawable/ic_100_buble.png is excluded by !**/*.png
• project.dot.png is excluded by !**/*.png

📒 Files selected for processing (70)

• .github/workflows/android_ci.yml (0 hunks)
• app/build.gradle.kts (2 hunks)
• app/src/main/AndroidManifest.xml (3 hunks)
• app/src/main/java/com/yapp/orbit/OrbitApplication.kt (1 hunks)
• app/src/main/java/com/yapp/orbit/OrbitNavHost.kt (4 hunks)
• app/src/main/java/com/yapp/orbit/di/AppVersionModule.kt (1 hunks)
• build-logic/src/main/java/com/yapp/convention/HiltAndroid.kt (1 hunks)
• build-logic/src/main/java/orbit.android.feature.gradle.kts (0 hunks)
• core/alarm/src/main/java/com/yapp/alarm/AndroidAlarmScheduler.kt (5 hunks)
• core/alarm/src/main/java/com/yapp/alarm/receivers/AlarmInteractionActivityReceiver.kt (2 hunks)
• core/alarm/src/main/java/com/yapp/alarm/receivers/AlarmReceiver.kt (3 hunks)
• core/alarm/src/main/java/com/yapp/alarm/receivers/RescheduleAlarmReceiver.kt (2 hunks)
• core/alarm/src/main/java/com/yapp/alarm/scheduler/PostFortuneTaskScheduler.kt (1 hunks)
• core/alarm/src/main/java/com/yapp/alarm/services/AlarmService.kt (5 hunks)
• core/common/src/main/java/com/yapp/common/navigation/route/MissionRoute.kt (1 hunks)
• core/datastore/src/main/java/com/yapp/datastore/UserPreferences.kt (3 hunks)
• core/designsystem/src/main/res/raw/fortune_loading.json (1 hunks)
• core/network/src/main/java/com/yapp/network/di/NetworkModule.kt (2 hunks)
• core/network/src/main/java/com/yapp/network/di/Qualifier.kt (0 hunks)
• core/ui/src/main/java/com/yapp/ui/component/navigation/NavigationBarScrim.kt (1 hunks)
• data/src/main/java/com/yapp/data/local/datasource/AlarmLocalDataSource.kt (0 hunks)
• data/src/main/java/com/yapp/data/local/datasource/AlarmLocalDataSourceImpl.kt (0 hunks)
• data/src/main/java/com/yapp/data/local/datasource/FortuneLocalDataSource.kt (1 hunks)
• data/src/main/java/com/yapp/data/local/datasource/FortuneLocalDataSourceImpl.kt (2 hunks)
• data/src/main/java/com/yapp/data/local/datasource/UserLocalDataSource.kt (1 hunks)
• data/src/main/java/com/yapp/data/local/datasource/UserLocalDataSourceImpl.kt (2 hunks)
• data/src/main/java/com/yapp/data/remote/di/ServiceModule.kt (1 hunks)
• data/src/main/java/com/yapp/data/repositoryimpl/AlarmRepositoryImpl.kt (0 hunks)
• data/src/main/java/com/yapp/data/repositoryimpl/FortuneRepositoryImpl.kt (2 hunks)
• data/src/main/java/com/yapp/data/repositoryimpl/UserInfoRepositoryImpl.kt (1 hunks)
• domain/src/main/java/com/yapp/domain/model/AlarmDay.kt (2 hunks)
• domain/src/main/java/com/yapp/domain/model/FortuneCreateStatus.kt (1 hunks)
• domain/src/main/java/com/yapp/domain/model/MissionMode.kt (1 hunks)
• domain/src/main/java/com/yapp/domain/repository/AlarmRepository.kt (0 hunks)
• domain/src/main/java/com/yapp/domain/repository/FortuneRepository.kt (1 hunks)
• domain/src/main/java/com/yapp/domain/repository/UserInfoRepository.kt (1 hunks)
• feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/AlarmInteractionActivity.kt (2 hunks)
• feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/action/AlarmActionContract.kt (1 hunks)
• feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/action/AlarmActionScreen.kt (1 hunks)
• feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/action/AlarmActionViewModel.kt (3 hunks)
• feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/snooze/AlarmSnoozeTimerViewModel.kt (1 hunks)
• feature/fortune/build.gradle.kts (1 hunks)
• feature/fortune/src/main/java/com/yapp/fortune/FortuneNavGraph.kt (2 hunks)
• feature/fortune/src/main/java/com/yapp/fortune/FortuneScreen.kt (3 hunks)
• feature/fortune/src/main/java/com/yapp/fortune/FortuneViewModel.kt (4 hunks)
• feature/fortune/src/main/java/com/yapp/fortune/di/SchedulerModule.kt (1 hunks)
• feature/fortune/src/main/java/com/yapp/fortune/page/FortunePager.kt (1 hunks)
• feature/fortune/src/main/java/com/yapp/fortune/scheduler/WorkManagerPostFortuneTaskScheduler.kt (1 hunks)
• feature/fortune/src/main/java/com/yapp/fortune/worker/PostFortuneWorker.kt (1 hunks)
• feature/home/build.gradle.kts (1 hunks)
• feature/home/src/main/AndroidManifest.xml (1 hunks)
• feature/home/src/main/java/com/yapp/home/HomeContract.kt (2 hunks)
• feature/home/src/main/java/com/yapp/home/HomeScreen.kt (17 hunks)
• feature/home/src/main/java/com/yapp/home/HomeViewModel.kt (9 hunks)
• feature/home/src/main/java/com/yapp/home/alarm/addedit/AlarmAddEditScreen.kt (2 hunks)
• feature/home/src/main/java/com/yapp/home/alarm/component/bottomsheet/AlarmMissionBottomSheet.kt (10 hunks)
• feature/home/src/main/java/com/yapp/home/component/bottomsheet/AlarmListBottomSheet.kt (3 hunks)
• feature/home/src/main/java/com/yapp/home/component/bottomsheet/UpdateNoticeBottomSheet.kt (1 hunks)
• feature/home/src/main/res/values/strings.xml (2 hunks)
• feature/mission/src/main/java/com/yapp/mission/MissionContract.kt (0 hunks)
• feature/mission/src/main/java/com/yapp/mission/MissionNavGraph.kt (2 hunks)
• feature/mission/src/main/java/com/yapp/mission/MissionScreen.kt (0 hunks)
• feature/mission/src/main/java/com/yapp/mission/MissionViewModel.kt (2 hunks)
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingAccessScreen.kt (0 hunks)
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingBirthdayScreen.kt (0 hunks)
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingCompleteScreen2.kt (0 hunks)
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingGenderScreen.kt (3 hunks)
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingNavGraph.kt (1 hunks)
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingViewModel.kt (1 hunks)
• gradle/libs.versions.toml (4 hunks)

💤 Files with no reviewable changes (12)

• feature/mission/src/main/java/com/yapp/mission/MissionContract.kt
• domain/src/main/java/com/yapp/domain/repository/AlarmRepository.kt
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingBirthdayScreen.kt
• .github/workflows/android_ci.yml
• data/src/main/java/com/yapp/data/local/datasource/AlarmLocalDataSource.kt
• build-logic/src/main/java/orbit.android.feature.gradle.kts
• data/src/main/java/com/yapp/data/repositoryimpl/AlarmRepositoryImpl.kt
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingCompleteScreen2.kt
• feature/mission/src/main/java/com/yapp/mission/MissionScreen.kt
• feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingAccessScreen.kt
• data/src/main/java/com/yapp/data/local/datasource/AlarmLocalDataSourceImpl.kt
• core/network/src/main/java/com/yapp/network/di/Qualifier.kt

🧰 Additional context used

🧠 Learnings (4)

📚 Learning: 2025-07-27T15:20:35.256Z

Learnt from: DongChyeon
PR: YAPP-Github/Orbit-Android#238
File: feature/mission/src/main/java/com/yapp/mission/MissionScreen.kt:118-120
Timestamp: 2025-07-27T15:20:35.256Z
Learning: MissionRoute와 MissionScreen이 같은 파일(feature/mission/src/main/java/com/yapp/mission/MissionScreen.kt)에 있을 때, MissionRoute에서 BackHandler를 사용하면 파일 레벨의 import 문은 유지되어야 함.

Applied to files:

• feature/mission/src/main/java/com/yapp/mission/MissionNavGraph.kt
• core/common/src/main/java/com/yapp/common/navigation/route/MissionRoute.kt

📚 Learning: 2025-09-14T15:32:32.628Z

Learnt from: DongChyeon
PR: YAPP-Github/Orbit-Android#252
File: feature/fortune/src/main/java/com/yapp/fortune/worker/PostFortuneWorker.kt:24-56
Timestamp: 2025-09-14T15:32:32.628Z
Learning: 운세 요청 간에는 최소 1분 간격이 보장되어 있어서 PostFortuneWorker에서 동시성/원자성 문제를 고려하지 않아도 됨

Applied to files:

• feature/fortune/src/main/java/com/yapp/fortune/worker/PostFortuneWorker.kt

📚 Learning: 2025-07-23T10:29:14.146Z

Learnt from: DongChyeon
PR: YAPP-Github/Orbit-Android#234
File: feature/home/src/main/java/com/yapp/home/alarm/component/bottomsheet/AlarmMissionBottomSheet.kt:73-76
Timestamp: 2025-07-23T10:29:14.146Z
Learning: AlarmMissionBottomSheet에서 missionType/missionCount 파라미터는 현재 저장된 값을 UI에 표시하기 위해 사용되고, selectedMissionType/selectedMissionCount는 사용자가 변경 중인 내부 작업 상태를 관리하기 위해 사용된다. onDone이나 onSave 콜백을 통해 명시적으로 저장할 때만 변경사항이 반영되는 UX 패턴이다.

Applied to files:

• feature/home/src/main/res/values/strings.xml
• feature/home/src/main/java/com/yapp/home/alarm/component/bottomsheet/AlarmMissionBottomSheet.kt

📚 Learning: 2025-09-15T07:43:50.275Z

Learnt from: DongChyeon
PR: YAPP-Github/Orbit-Android#254
File: data/src/main/java/com/yapp/data/local/datasource/FortuneLocalDataSourceImpl.kt:22-34
Timestamp: 2025-09-15T07:43:50.275Z
Learning: FortuneCreateStatusFlow에서 todayEpoch()를 combine 내부에서 직접 호출하는 것이 충분한 이유: 운세 생성할 때마다 fortuneDateEpochFlow가 변경되어 combine이 재평가되므로, 그 순간의 todayEpoch() 계산으로 충분함. 운세 요청 간격 제약으로 인해 자정 롤오버 문제는 실용적으로 발생하지 않음.

Applied to files:

• feature/home/src/main/java/com/yapp/home/HomeViewModel.kt

🧬 Code graph analysis (10)

feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/AlarmInteractionActivity.kt (2)

feature/alarm-interaction/src/main/java/com/yapp/alarm/interaction/AlarmInteractionNavGraph.kt (1)

• alarmInteractionNavGraph (33-69)

core/ui/src/main/java/com/yapp/ui/component/navigation/NavigationBarScrim.kt (1)

• NavigationBarScrim (16-26)

feature/fortune/src/main/java/com/yapp/fortune/page/FortunePager.kt (1)

feature/fortune/src/main/java/com/yapp/fortune/page/FortuneCompletePage.kt (1)

• FortuneCompletePage (28-104)

feature/home/src/main/java/com/yapp/home/component/bottomsheet/UpdateNoticeBottomSheet.kt (1)

core/designsystem/src/main/java/com/yapp/designsystem/theme/Theme.kt (1)

• OrbitTheme (12-28)

feature/fortune/src/main/java/com/yapp/fortune/FortuneScreen.kt (2)

core/ui/src/main/java/com/yapp/ui/component/lottie/LottieAnimation.kt (1)

• LottieAnimation (22-85)

core/designsystem/src/main/java/com/yapp/designsystem/theme/Theme.kt (1)

• OrbitTheme (12-28)

feature/home/src/main/java/com/yapp/home/HomeScreen.kt (2)

feature/home/src/main/java/com/yapp/home/HomeViewModel.kt (1)

• processAction (55-87)

feature/home/src/main/java/com/yapp/home/component/bottomsheet/UpdateNoticeBottomSheet.kt (1)

• UpdateNoticeBottomSheet (51-139)

feature/home/src/main/java/com/yapp/home/alarm/addedit/AlarmAddEditScreen.kt (1)

core/ui/src/main/java/com/yapp/ui/component/bottomsheet/OrbitBottomSheetLayout.kt (1)

• OrbitBottomSheetLayout (31-59)

feature/home/src/main/java/com/yapp/home/component/bottomsheet/AlarmListBottomSheet.kt (2)

feature/home/src/main/java/com/yapp/home/component/AlarmListDropDownMenu.kt (1)

• AlarmListDropDownMenu (43-72)

feature/home/src/main/java/com/yapp/home/component/AlarmSortDropDownMenu.kt (1)

• AlarmSortDropDownMenu (36-67)

feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingGenderScreen.kt (4)

core/ui/src/main/java/com/yapp/ui/component/bottomsheet/OrbitBottomSheetLayout.kt (1)

• OrbitBottomSheetLayout (31-59)

feature/onboarding/src/main/java/com/yapp/onboarding/OnBoardingScreen.kt (1)

• OnboardingScreen (14-54)

feature/onboarding/src/main/java/com/yapp/onboarding/OnboardingViewModel.kt (1)

• processAction (49-67)

core/ui/src/main/java/com/yapp/ui/toggle/OrbitGenderToggle.kt (1)

• OrbitGenderToggle (30-98)

core/datastore/src/main/java/com/yapp/datastore/UserPreferences.kt (1)

data/src/main/java/com/yapp/data/local/datasource/FortuneLocalDataSourceImpl.kt (1)

• todayEpoch (36-36)

data/src/main/java/com/yapp/data/local/datasource/FortuneLocalDataSourceImpl.kt (1)

core/datastore/src/main/java/com/yapp/datastore/UserPreferences.kt (1)

• todayEpoch (44-44)

🪛 detekt (1.23.8)

app/src/main/java/com/yapp/orbit/OrbitApplication.kt

[warning] 11-11: An empty default constructor can be removed.

(detekt.empty-blocks.EmptyDefaultConstructor)

[coderabbitai]

Actionable comments posted: 1

♻️ Duplicate comments (3)

feature/home/src/main/java/com/yapp/home/component/bottomsheet/AlarmListBottomSheet.kt (2)

102-114: snapshotFlow 중복 이벤트 방지 개선 필요

현재 구현에서는 distinctUntilChanged()를 적용했지만, LaunchedEffect의 key가 여전히 Unit입니다. sheetState를 key로 사용하는 것이 더 안전합니다.

다음과 같이 수정하세요:

-    LaunchedEffect(Unit) {
+    LaunchedEffect(sheetState) {
         snapshotFlow { sheetState.currentValue }
             .distinctUntilChanged()
             .collectLatest { value ->

---

98-98: Hidden 상태 전이 방지 필요

confirmValueChange 제거로 사용자가 드래그하여 Hidden 상태로 전이할 수 있습니다. 시트가 항상 표시되어야 한다면 Hidden 상태를 명시적으로 차단해야 합니다.

다음 중 하나를 적용하세요:

옵션 1: Hidden 상태 건너뛰기 (권장)

-    val sheetState = rememberStandardBottomSheetState()
+    val sheetState = rememberStandardBottomSheetState(
+        skipHiddenState = true
+    )

옵션 2: confirmValueChange로 명시적 차단

-    val sheetState = rememberStandardBottomSheetState()
+    val sheetState = rememberStandardBottomSheetState(
+        confirmValueChange = { it != SheetValue.Hidden }
+    )

core/alarm/src/main/java/com/yapp/alarm/receivers/RescheduleAlarmReceiver.kt (1)

30-31: goAsync 적용 굿 + 앱 업데이트 후 재스케줄도 함께 처리 제안

부팅 후만 처리하면 앱 업데이트 직후 알람 재설정이 누락될 수 있습니다. ACTION_MY_PACKAGE_REPLACED도 함께 수신해 동일 로직을 태우는 걸 권장합니다. (이전 리뷰 코멘트와 동일 제안)

-        if (intent.action == Intent.ACTION_BOOT_COMPLETED) {
+        if (intent.action == Intent.ACTION_BOOT_COMPLETED || intent.action == Intent.ACTION_MY_PACKAGE_REPLACED) {
             val pending = goAsync()
             rescheduleAlarm(pending)
         }

🧹 Nitpick comments (1)

core/alarm/src/main/java/com/yapp/alarm/receivers/RescheduleAlarmReceiver.kt (1)

35-44: 예외 로깅 추가로 문제 진단성 개선 제안

현재 예외가 삼켜져 원인 파악이 어렵습니다. catch에서 로깅 후 finish 보장 유지 권장.

-        CoroutineScope(Dispatchers.IO + SupervisorJob()).launch {
-            try {
-                val alarms = alarmUseCase.getAllAlarms().first()
-                alarms
-                    .filter { it.isAlarmActive }
-                    .forEach { alarm -> androidAlarmScheduler.scheduleAlarm(alarm) }
-            } finally {
-                pendingResult.finish()
-            }
-        }
+        CoroutineScope(Dispatchers.IO + SupervisorJob()).launch {
+            try {
+                val alarms = alarmUseCase.getAllAlarms().first()
+                alarms
+                    .filter { it.isAlarmActive }
+                    .forEach { alarm -> androidAlarmScheduler.scheduleAlarm(alarm) }
+            } catch (t: Throwable) {
+                android.util.Log.e("RescheduleAlarmReceiver", "Failed to reschedule alarms", t)
+            } finally {
+                pendingResult.finish()
+            }
+        }

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f1fd6b6 and 700e803.

📒 Files selected for processing (2)

• core/alarm/src/main/java/com/yapp/alarm/receivers/RescheduleAlarmReceiver.kt (2 hunks)
• feature/home/src/main/java/com/yapp/home/component/bottomsheet/AlarmListBottomSheet.kt (4 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

feature/home/src/main/java/com/yapp/home/component/bottomsheet/AlarmListBottomSheet.kt (2)

feature/home/src/main/java/com/yapp/home/component/AlarmListDropDownMenu.kt (1)

• AlarmListDropDownMenu (43-72)

feature/home/src/main/java/com/yapp/home/component/AlarmSortDropDownMenu.kt (1)

• AlarmSortDropDownMenu (36-67)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: build
• GitHub Check: Continuous Deployment

🔇 Additional comments (4)

feature/home/src/main/java/com/yapp/home/component/bottomsheet/AlarmListBottomSheet.kt (2)

329-341: 드롭다운 메뉴 조건부 렌더링 제거 확인

AlarmListDropDownMenu와 AlarmSortDropDownMenu가 이제 항상 렌더링되며, expanded 상태만 전달됩니다. 이는 성능상 문제가 없지만, 두 메뉴가 동시에 열릴 가능성이 있습니다.

두 메뉴가 동시에 열리지 않도록 상태 관리가 적절히 되고 있는지 확인이 필요합니다. 특히 menuExpanded와 sortDropDownMenuExpanded가 상호 배타적으로 관리되는지 검증하세요.

---

93-93: onExpanded 콜백 사용 확인 — 해결됨

feature/home/src/main/java/com/yapp/home/HomeScreen.kt (lines 394–396)에서 onExpanded가 호출되어 processAction(HomeContract.Action.HideToolTip)를 실행합니다. 수정 불필요.

core/alarm/src/main/java/com/yapp/alarm/receivers/RescheduleAlarmReceiver.kt (2)

11-12: 필요 import 추가 적절

SupervisorJob, flow.first 도입 모두 상황에 맞고 👍

---

29-33: Manifest 설정 확인: RECEIVE_BOOT_COMPLETED·intent-filter·exported 확인됨

app/src/main/AndroidManifest.xml에 RECEIVE_BOOT_COMPLETED 권한(라인 11)과 RescheduleAlarmReceiver 리시버 선언(라인 72–76) — intent-filter(android.intent.action.BOOT_COMPLETED) 및 android:exported="true"(라인 73)가 존재하므로 BOOT_COMPLETED 수신 설정은 충족됩니다.