Skip to content

[Feat] MainActivity Compose 연동 및 하위 Fragment 제거 #40

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
oungsi2000 merged 20 commits into from
Jan 28, 2026
Merged

[Feat] MainActivity Compose 연동 및 하위 Fragment 제거 #40

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
8c715fd
refactor(main): 하단 네비게이션을 Compose로 전환
oungsi2000 Jan 21, 2026
220a587
refactor(Navigation): 메인 탭 화면별 Navigation Graph 모듈화
oungsi2000 Jan 21, 2026
06ccf08
refactor(Main): `currentDestination` 로직 안정성 강화
oungsi2000 Jan 21, 2026
1faefcc
refactor(main): MainActivity를 Compose 기반으로 전환
oungsi2000 Jan 21, 2026
f66eebc
fix(PlaceMap): 메모리 누수 로직으로 Stop 상태일 때에도 Destroy가 되는 버그 해결
oungsi2000 Jan 21, 2026
824fbe9
refactor(PlaceMap): NaverMap 라이프사이클 관리 코드 추가
oungsi2000 Jan 22, 2026
72ccac5
refactor(main): 하단 탭 재선택 시 데이터 갱신 기능 추가
oungsi2000 Jan 22, 2026
6cfd1b7
refactor(PlaceMap): 지도 화면을 NavHost 외부로 분리하여 상태 유지
oungsi2000 Jan 22, 2026
d0648dd
refactor:kotlinx.LocalDateTime 적용 및, @Serializable 적용
oungsi2000 Jan 22, 2026
4aa5aac
refactor(navigation): NavGraph에서 불필요한 `padding` 파라미터 제거
oungsi2000 Jan 22, 2026
80b20ef
feat(PlaceDetail): 장소 상세 화면을 NavHost 내 컴포저블로 전환
oungsi2000 Jan 22, 2026
28507ee
refactor(main): 기존 Android 기반 MainActivity를 완전한 Compose 기반으로 전환하고 관련 …
oungsi2000 Jan 23, 2026
b9251e5
test: 테스트 코드에서 `LiveData` 테스트 방식을 `observeEvent` 헬퍼로 변경
oungsi2000 Jan 23, 2026
be5f972
refactor(PlaceMap): 지도 화면 숨김 시 터치 이벤트 비활성화
oungsi2000 Jan 27, 2026
bf9df25
refactor(setting): `successFlow`를 `success`로 이름 변경
oungsi2000 Jan 27, 2026
d352259
refactor(UI): 하단 네비게이션 바에 시스템 패딩 적용
oungsi2000 Jan 27, 2026
65c2fbe
refactor(main): 알림 권한 요청 로직을 Activity로 이동
oungsi2000 Jan 27, 2026
7ef34f0
fix(PlaceDetail): 다이얼로그 표시 중 뒤로가기 동작 버그 수정
oungsi2000 Jan 27, 2026
91abcef
refactor(PlaceMap): `remember` 키 추가로 SideEffect 핸들러 재생성 로직 개선
oungsi2000 Jan 27, 2026
4c5d379
feat(main): 하단 네비게이션 바에 그림자 효과 추가
oungsi2000 Jan 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions app/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -149,6 +149,7 @@ android {

dependencies {
ktlintRuleset(libs.ktlint)
implementation(libs.kotlinx.datetime)
implementation(libs.map.sdk)
implementation(libs.androidx.navigation.compose)
implementation(libs.play.services.location)
Expand Down
2 changes: 1 addition & 1 deletion app/src/main/java/com/daedan/festabook/data/model/response/notice/NoticeResponse.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
package com.daedan.festabook.data.model.response.notice

import com.daedan.festabook.domain.model.Notice
import kotlinx.datetime.LocalDateTime
import kotlinx.serialization.SerialName
import kotlinx.serialization.Serializable
import java.time.LocalDateTime

@Serializable
data class NoticeResponse(
Expand Down
2 changes: 1 addition & 1 deletion app/src/main/java/com/daedan/festabook/data/model/response/place/PlaceDetailResponse.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@ import com.daedan.festabook.domain.model.Notice
import com.daedan.festabook.domain.model.Place
import com.daedan.festabook.domain.model.PlaceDetail
import com.daedan.festabook.domain.model.PlaceDetailImage
import kotlinx.datetime.LocalDateTime
import kotlinx.serialization.SerialName
import kotlinx.serialization.Serializable
import java.time.LocalDateTime
import java.time.LocalTime
import java.time.format.DateTimeFormatter

Expand Down
2 changes: 1 addition & 1 deletion app/src/main/java/com/daedan/festabook/domain/model/Notice.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
package com.daedan.festabook.domain.model

import java.time.LocalDateTime
import kotlinx.datetime.LocalDateTime

data class Notice(
val id: Long,
Expand Down
6 changes: 3 additions & 3 deletions app/src/main/java/com/daedan/festabook/presentation/common/FragmentUtil.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,11 @@ import android.os.Parcelable
import android.util.TypedValue
import android.view.View
import android.view.ViewGroup
import androidx.compose.ui.platform.ComposeView
import androidx.core.view.WindowInsetsCompat
import androidx.fragment.app.Fragment
import com.daedan.festabook.R
import com.daedan.festabook.data.util.ApiResultException
import com.google.android.material.floatingactionbutton.FloatingActionButton
import com.google.android.material.snackbar.Snackbar
import java.io.Serializable

Expand Down Expand Up @@ -63,7 +63,7 @@ fun Activity.showErrorSnackBar(msg: String) {
Snackbar.LENGTH_SHORT,
)
snackBar.setAnchorView(
findViewById<FloatingActionButton>(R.id.bab_menu),
findViewById<ComposeView>(R.id.cv_main),
)
snackBar
.setAction(
Expand Down Expand Up @@ -116,7 +116,7 @@ fun Activity.showSnackBar(msg: String) {
Snackbar.LENGTH_SHORT,
)
snackBar.setAnchorView(
findViewById<FloatingActionButton>(R.id.bab_menu),
findViewById<ComposeView>(R.id.cv_main),
)
snackBar
.setAction(
Expand Down
2 changes: 1 addition & 1 deletion app/src/main/java/com/daedan/festabook/presentation/common/PermissionUtil.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ fun showNotificationDeniedSnackbar(
view,
text,
Snackbar.LENGTH_LONG,
).setAnchorView(view.rootView.findViewById(R.id.bab_menu))
).setAnchorView(view.rootView.findViewById(R.id.cv_main))
.setAction(context.getString(R.string.move_to_setting_text)) {
val intent =
Intent(Settings.ACTION_APPLICATION_DETAILS_SETTINGS).apply {
Expand Down
18 changes: 13 additions & 5 deletions app/src/main/java/com/daedan/festabook/presentation/home/navigation/HomeNavigation.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,22 +1,30 @@
package com.daedan.festabook.presentation.home.navigation

import androidx.compose.foundation.layout.PaddingValues
import androidx.compose.foundation.layout.padding
import androidx.compose.ui.Modifier
import androidx.compose.runtime.getValue
import androidx.lifecycle.compose.collectAsStateWithLifecycle
import androidx.navigation.NavGraphBuilder
import androidx.navigation.compose.composable
import com.daedan.festabook.presentation.home.HomeViewModel
import com.daedan.festabook.presentation.home.component.HomeScreen
import com.daedan.festabook.presentation.main.MainTabRoute
import com.daedan.festabook.presentation.main.MainViewModel
import com.daedan.festabook.presentation.main.component.FirstVisitDialog

fun NavGraphBuilder.homeNavGraph(
padding: PaddingValues,
viewModel: HomeViewModel,
mainViewModel: MainViewModel,
onSubscriptionConfirm: () -> Unit,
onNavigateToExplore: () -> Unit,
) {
composable<MainTabRoute.Home> {
val isFirstVisit by mainViewModel.isFirstVisit.collectAsStateWithLifecycle()
if (isFirstVisit) {
FirstVisitDialog(
onConfirm = { onSubscriptionConfirm() },
onDecline = { mainViewModel.declineAlert() },
)
}
HomeScreen(
modifier = Modifier.padding(padding),
viewModel = viewModel,
onNavigateToExplore = onNavigateToExplore,
)
Expand Down
28 changes: 13 additions & 15 deletions app/src/main/java/com/daedan/festabook/presentation/main/FestabookNavigator.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
package com.daedan.festabook.presentation.main

import androidx.compose.runtime.Composable
import androidx.compose.runtime.collectAsState
import androidx.compose.runtime.remember
import androidx.navigation.NavDestination.Companion.hasRoute
import androidx.navigation.NavGraph.Companion.findStartDestination
import androidx.navigation.NavHostController
import androidx.navigation.NavOptions
import androidx.navigation.compose.currentBackStackEntryAsState
import androidx.navigation.compose.rememberNavController
import androidx.navigation.navOptions

Expand All @@ -17,19 +17,11 @@ class FestabookNavigator(
@Composable
get() =
navController
.currentBackStackEntryAsState()
.visibleEntries
.collectAsState()
.value
.lastOrNull { it.destination.route != null }
?.destination

private val defaultNavOptions =
navOptions {
popUpTo(navController.graph.findStartDestination().id) {
saveState = true
}
launchSingleTop = true
restoreState = true
}

val currentTab
@Composable
get() =
Expand All @@ -46,10 +38,16 @@ class FestabookNavigator(

val startRoute = MainTabRoute.Home // TODO: Splash와 Explore 연동 시 변경

fun navigateToMainTab(route: FestabookRoute) {
fun navigateToMainTab(tab: FestabookMainTab) {
navController.navigate(
route,
defaultNavOptions,
tab.route,
navOptions {
popUpTo(navController.graph.findStartDestination().id) {
saveState = true
}
launchSingleTop = true
restoreState = true
},
)
}

Expand Down
8 changes: 6 additions & 2 deletions app/src/main/java/com/daedan/festabook/presentation/main/FestabookRoute.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,15 +1,19 @@
package com.daedan.festabook.presentation.main

import com.daedan.festabook.presentation.placeDetail.model.PlaceDetailUiModel
import com.daedan.festabook.presentation.placeMap.model.PlaceUiModel
import kotlinx.serialization.Serializable

@Serializable
sealed interface FestabookRoute {
@Serializable
data object Splash : FestabookRoute

// TODO: PlaceUiModel, PlaceDetailUiModel 생성자에 추가 후 UiModel에 @Serializable 어노테이션 필요
@Serializable
data object PlaceDetail : FestabookRoute
data class PlaceDetail(
val placeUiModel: PlaceUiModel? = null,
val placeDetailUiModel: PlaceDetailUiModel? = null,
) : FestabookRoute
Comment on lines +3 to +16
Copy link

@coderabbitai coderabbitai bot Jan 23, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

# Find PlaceDetailUiModel and PlaceUiModel files
fd 'PlaceDetailUiModel|PlaceUiModel' -t f --extension kt

Repository: festabook/android

Length of output: 178

🏁 Script executed:

# Try searching using web for information about Jetpack Compose navigation serialization constraints
# and common issues with navigation route payloads
curl -s "https://developer.android.com/" 2>/dev/null | head -20 || echo "Web access limited"

Repository: festabook/android

Length of output: 808

🌐 Web query:

Jetpack Compose Navigation kotlinx.serialization route payload bundle size limits constraints

💡 Result:

In Navigation Compose, “type-safe routes” that use kotlinx.serialization still end up as navigation arguments associated with a NavBackStackEntry (and commonly accessed via toRoute() / SavedStateHandle). (developer.android.com) Those arguments are ultimately stored/transported using Android’s state/Bundles (e.g., SavedStateHandle / SavedState → Bundle). (developer.android.com)

Practical size limits / constraints (what will bite you)

  • Binder transaction limit (~1MB, process-wide shared buffer): Any time these arguments/state are sent through Binder (e.g., saving/restoring state, activity/fragment lifecycle handoff, etc.), you can hit TransactionTooLargeException. Android’s docs state the Binder transaction buffer is currently 1MB and shared across transactions in the process. (developer.android.com)
  • Saved-state guidance is far smaller than 1MB: In practice you should keep saved state tiny. A commonly cited guideline is < ~50KB for saved instance state, because the system may need to retain it for a long time for back navigation/process death restore. (stackoverflow.com)
  • Type constraint (Bundle-compatible) still applies: When using SavedStateHandle with navigation, values must be types that can go in a Bundle (or you need another mechanism like a ViewModel/repository). (developer.android.com)

What this means for “serialized route payloads”

  • Serializing a whole object (especially to a string/JSON) and passing it as a nav arg is still constrained by Bundle/Binder limits, and can blow up when state is saved/restored (not necessarily at the moment you call navigate()).
  • Best practice is to pass small identifiers (IDs, keys) in the route, and load the heavy object from a repository/DB/shared ViewModel instead of bundling it.

If you share a concrete example of your route payload (types + approximate size), I can suggest a safer pattern and where it should live (route vs SavedStateHandle vs repository).

Citations:

🌐 Web query:

Android Compose navigation @serializable route performance best practices data classes

💡 Result:

Best practices (performance + reliability) for @Serializable route data classes in Navigation-Compose (type-safe routes)

  1. Keep route arguments small and primitive-ish

    • Prefer passing IDs, enums, small strings, etc., then load the real data from a repository/ViewModel.
    • Type-safe routes are turned into navigation arguments (and can also participate in deep-link URL parameter generation), so large/nested objects (big lists, JSON blobs) increase allocations, encoding/decoding work, and risk hitting size limits. (medium.com)

  2. Parse (toRoute()) once, not on every recomposition

    • If you do backStackEntry.toRoute<MyRoute>() inside a composable body, don’t let it re-run on every recomposition—cache it:
    composable<Product> { entry ->
  val route = remember(entry) { entry.toRoute<Product>() }
  ProductScreen(route.id)
}

    This follows general Compose guidance to avoid repeated “work” during recomposition. (developer.android.com)

  3. Better: decode in the ViewModel via SavedStateHandle.toRoute()

    • This makes the decode happen once per VM instance and keeps the composable “dumb”:
    class ProductViewModel(savedStateHandle: SavedStateHandle) : ViewModel() {
  val route: Product = savedStateHandle.toRoute()
}

    Note: SavedStateHandle.toRoute() currently depends on Android Bundle (relevant for tests and also hints it’s not “free”). (medium.com)

  4. Use the right route shape

    • @Serializable object for no-arg destinations, @Serializable data class for destinations with args. (developer.android.com)

  5. If you need custom/complex argument types, use the supported customization points (don’t stringify yourself)

    • Navigation added a typeMap parameter for toRoute() to support custom argument types. (developer.android.com)

  6. Baseline profile / macro-level perf

    • If navigation + serialization is on a critical path (cold start / first screen), also lean on Compose perf tooling and Baseline Profiles. (developer.android.com)

Citations:

라우트에서 전체 모델 객체 전달 방식 개선 필수.

PlaceUiModel과 PlaceDetailUiModel 전체 객체를 라우트 페이로드로 전달하는 것은 안드로이드 네비게이션 모범 사례에 위배됩니다. Navigation Compose의 @Serializable 라우트는 Bundle 인자로 처리되며, Bundle은 Binder 트랜잭션 한계(~1MB 공유 버퍼)와 백 스택 저장 시 크기 제한(권장: <50KB)에 제약을 받습니다. 상태 저장/복원 중 TransactionTooLargeException이 발생할 수 있습니다.

개선 방안: PlaceDetail 라우트에는 ID나 최소 필드만 전달하고, 실제 데이터는 Repository나 ViewModel에서 로드하세요.

🤖 Prompt for AI Agents 
In `@app/src/main/java/com/daedan/festabook/presentation/main/FestabookRoute.kt`
around lines 3 - 16, The FestabookRoute.PlaceDetail currently serializes and
passes full PlaceUiModel and PlaceDetailUiModel (which risks
TransactionTooLargeException); change the sealed class to carry only a small
identifier (e.g., placeId: String or Long or minimal primitive key) instead of
PlaceUiModel/PlaceDetailUiModel, remove those model fields from
FestabookRoute.PlaceDetail, and update navigation callers to pass the id; then
fetch the full PlaceUiModel/PlaceDetailUiModel inside the destination via the
Repository or ViewModel (e.g., use the placeId in your ViewModel to load data)
and update any code referencing FestabookRoute.PlaceDetail to use the id-based
payload.

Copy link
Contributor Author

@oungsi2000 oungsi2000 Jan 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

측정결과 PlaceDetailUiModel의 크기는 1kb가 넘지 않습니다. 추후 반영하도록 하겠습니다

Copy link

@coderabbitai coderabbitai bot Jan 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!


@Serializable
data object Explore : FestabookRoute
Expand Down
Loading