refactor(core): Handle rate limit exceptions and improve starred repos sync

This commit introduces centralized handling for `RateLimitException` across various features and refactors the starred repositories synchronization logic.

When a GitHub API rate limit is exceeded, the app will now fail gracefully instead of crashing. This affects app updates, repository searching, and fetching starred repositories.

Additionally, the synchronization behavior for starred repositories has been updated. The sync threshold has been extended to 24 hours to reduce unnecessary API calls. A "Sign In" button has been added to the "Starred" screen, guiding unauthenticated users to log in to view their starred repositories.

- **refactor(core)**: Propagate `RateLimitException` from data sources up to the view models in the search, apps, and starred features to prevent crashes.
- **feat(starred)**: Added a `OnSingInClick` action and navigation to the authentication screen from the "Starred" page for unauthenticated users.
- **refactor(starred)**: Extended the sync threshold for starred repositories from 6 to 24 hours.

Author: rainxchzed

composeApp/src/commonMain/kotlin/zed/rainxch/githubstore/app/navigation/AppNavigation.kt

@@ -184,6 +184,11 @@ fun AppNavigation(
                                 )
                             )
                         },
+                        onNavigateToAuthentication = {
+                            navController.navigate(
+                                GithubStoreGraph.AuthenticationScreen
+                            )
+                        },
                         onNavigateToDeveloperProfile = { username ->
                             navController.navigate(
                                 GithubStoreGraph.DeveloperProfileScreen(

core/data/src/commonMain/kotlin/zed/rainxch/core/data/repository/StarredRepositoryImpl.kt

@@ -26,7 +26,9 @@ import zed.rainxch.core.data.local.db.dao.StarredRepoDao
 import zed.rainxch.core.data.mappers.toDomain
 import zed.rainxch.core.data.mappers.toEntity
 import zed.rainxch.core.domain.model.Platform
+import zed.rainxch.core.domain.model.RateLimitException
 import zed.rainxch.core.domain.repository.StarredRepository
+import kotlin.coroutines.cancellation.CancellationException
 import kotlin.time.Clock
 import kotlin.time.ExperimentalTime
 import kotlin.time.Instant
@@ -39,7 +41,7 @@ class StarredRepositoryImpl(
 ) : StarredRepository {
 
     companion object {
-        private const val SYNC_THRESHOLD_MS = 6 * 60 * 60 * 1000L // 6 hours
+        private const val SYNC_THRESHOLD_MS = 24 * 60 * 60 * 1000L // 24 hours
     }
 
     override fun getAllStarred(): Flow<List<zed.rainxch.core.domain.model.StarredRepository>> {
@@ -99,7 +101,6 @@ class StarredRepositoryImpl(
                 val now = Clock.System.now().toEpochMilliseconds()
                 val starredRepos = mutableListOf<zed.rainxch.core.domain.model.StarredRepository>()
 
-                // Process in parallel to avoid sequential N+1 delays
                 coroutineScope {
                     val semaphore = Semaphore(25)
                     val deferredResults = allRepos.map { repo ->
@@ -145,6 +146,10 @@ class StarredRepositoryImpl(
                 starredRepoDao.replaceAllStarred(starredRepos.map { it.toEntity() })
 
                 Result.success(Unit)
+            } catch (e: RateLimitException) {
+                throw e
+            } catch (e: CancellationException) {
+                throw e
             } catch (e: Exception) {
                 Logger.e(e) { "Failed to sync starred repos" }
                 Result.failure(e)
@@ -185,6 +190,10 @@ class StarredRepositoryImpl(
             }
 
             relevantAssets.isNotEmpty()
+        } catch (e: RateLimitException) {
+            throw e
+        } catch (e: CancellationException) {
+            throw e
         } catch (e: Exception) {
             Logger.w(e) { "Failed to check valid assets for $owner/$repo" }
             false

feature/apps/data/src/commonMain/kotlin/zed/rainxch/apps/data/repository/AppsRepositoryImpl.kt

@@ -13,6 +13,7 @@ import zed.rainxch.core.data.network.executeRequest
 import zed.rainxch.core.domain.logging.GitHubStoreLogger
 import zed.rainxch.core.domain.model.GithubRelease
 import zed.rainxch.core.domain.model.InstalledApp
+import zed.rainxch.core.domain.model.RateLimitException
 import zed.rainxch.core.domain.repository.InstalledAppsRepository
 import zed.rainxch.core.domain.utils.AppLauncher
 
@@ -53,13 +54,15 @@ class AppsRepositoryImpl(
                     header(HttpHeaders.Accept, "application/vnd.github+json")
                     parameter("per_page", 10)
                 }
-            }.getOrNull() ?: return null
+            }.getOrThrow()
 
             releases
                 .asSequence()
                 .filter { (it.draft != true) && (it.prerelease != true) }
                 .maxByOrNull { it.publishedAt ?: it.createdAt ?: "" }
                 ?.toDomain()
+        } catch (e: RateLimitException) {
+            throw e
         } catch (e: Exception) {
             logger.error("Failed to fetch latest release for $owner/$repo: ${e.message}")
             null

feature/apps/presentation/src/commonMain/kotlin/zed/rainxch/apps/presentation/AppsViewModel.kt

@@ -22,6 +22,7 @@ import zed.rainxch.apps.presentation.model.UpdateAllProgress
 import zed.rainxch.apps.presentation.model.UpdateState
 import zed.rainxch.core.domain.logging.GitHubStoreLogger
 import zed.rainxch.core.domain.model.InstalledApp
+import zed.rainxch.core.domain.model.RateLimitException
 import zed.rainxch.core.domain.network.Downloader
 import zed.rainxch.core.domain.repository.InstalledAppsRepository
 import zed.rainxch.core.domain.system.Installer
@@ -289,9 +290,11 @@ class AppsViewModel(
                 cleanupUpdate(app.packageName, app.latestAssetName)
                 updateAppState(app.packageName, UpdateState.Idle)
                 throw e
+            } catch (e: RateLimitException) {
+                logger.debug("Rate limited during update for ${app.packageName}")
+                updateAppState(app.packageName, UpdateState.Idle)
             } catch (e: Exception) {
                 logger.error("Update failed for ${app.packageName}: ${e.message}")
-                e.printStackTrace()
                 cleanupUpdate(app.packageName, app.latestAssetName)
                 updateAppState(
                     app.packageName,

feature/search/data/src/commonMain/kotlin/zed/rainxch/search/data/repository/SearchRepositoryImpl.kt

@@ -26,6 +26,7 @@ import zed.rainxch.core.data.mappers.toSummary
 import zed.rainxch.core.data.network.executeRequest
 import zed.rainxch.core.domain.model.GithubRepoSummary
 import zed.rainxch.core.domain.model.PaginatedDiscoveryRepositories
+import zed.rainxch.core.domain.model.RateLimitException
 import zed.rainxch.domain.model.ProgrammingLanguage
 import zed.rainxch.domain.model.SearchPlatform
 import zed.rainxch.domain.repository.SearchRepository
@@ -169,6 +170,8 @@ class SearchRepositoryImpl(
                     )
                 }
             }
+        } catch (e: RateLimitException) {
+            throw e
         } catch (e: CancellationException) {
             throw e
         }

feature/search/presentation/src/commonMain/kotlin/zed/rainxch/search/presentation/SearchViewModel.kt

@@ -15,6 +15,7 @@ import kotlinx.coroutines.flow.update
 import kotlinx.coroutines.launch
 import org.jetbrains.compose.resources.getString
 import zed.rainxch.core.domain.logging.GitHubStoreLogger
+import zed.rainxch.core.domain.model.RateLimitException
 import zed.rainxch.core.domain.repository.FavouritesRepository
 import zed.rainxch.core.domain.repository.InstalledAppsRepository
 import zed.rainxch.core.domain.repository.StarredRepository
@@ -225,6 +226,8 @@ class SearchViewModel(
                 _state.update {
                     it.copy(isLoading = false, isLoadingMore = false)
                 }
+            } catch (e: RateLimitException) {
+                logger.debug("Rate limit exceed: ${e.message}")
             } catch (e: CancellationException) {
                 logger.debug("Search cancelled (expected): ${e.message}")
             } catch (e: Exception) {

feature/starred/presentation/src/commonMain/kotlin/zed/rainxch/starred/presentation/StarredReposAction.kt

@@ -6,6 +6,7 @@ sealed interface StarredReposAction {
     data object OnNavigateBackClick : StarredReposAction
     data object OnRefresh : StarredReposAction
     data object OnRetrySync : StarredReposAction
+    data object OnSingInClick : StarredReposAction
     data object OnDismissError : StarredReposAction
     data class OnRepositoryClick(val repository: StarredRepositoryUi) : StarredReposAction
     data class OnDeveloperProfileClick(val username: String) : StarredReposAction

feature/starred/presentation/src/commonMain/kotlin/zed/rainxch/starred/presentation/StarredReposRoot.kt

@@ -58,6 +58,7 @@ fun StarredReposRoot(
     onNavigateBack: () -> Unit,
     onNavigateToDetails: (repoId: Long) -> Unit,
     onNavigateToDeveloperProfile: (username: String) -> Unit,
+    onNavigateToAuthentication: () -> Unit,
     viewModel: StarredReposViewModel = koinViewModel()
 ) {
     val state by viewModel.state.collectAsStateWithLifecycle()
@@ -69,6 +70,7 @@ fun StarredReposRoot(
                 StarredReposAction.OnNavigateBackClick -> onNavigateBack()
                 is StarredReposAction.OnRepositoryClick -> onNavigateToDetails(action.repository.repoId)
                 is StarredReposAction.OnDeveloperProfileClick -> onNavigateToDeveloperProfile(action.username)
+                StarredReposAction.OnSingInClick -> onNavigateToAuthentication()
                 else -> viewModel.onAction(action)
             }
         }
@@ -104,6 +106,10 @@ fun StarredScreen(
                         title = stringResource(Res.string.sign_in_required),
                         message = stringResource(Res.string.sign_in_with_github_for_stars),
                         icon = Icons.Default.Star,
+                        actionText = stringResource(Res.string.sign_in_with_github),
+                        onActionClick = {
+                            onAction(StarredReposAction.OnSingInClick)
+                        },
                         modifier = Modifier.align(Alignment.Center)
                     )
                 }
@@ -121,7 +127,9 @@ fun StarredScreen(
                         icon = Icons.Default.Star,
                         actionText = if (state.errorMessage != null) stringResource(Res.string.retry) else null,
                         onActionClick = if (state.errorMessage != null) {
-                            { onAction(StarredReposAction.OnRetrySync) }
+                            {
+                                onAction(StarredReposAction.OnRetrySync)
+                            }
                         } else null,
                         modifier = Modifier.align(Alignment.Center)
                     )

feature/starred/presentation/src/commonMain/kotlin/zed/rainxch/starred/presentation/StarredReposViewModel.kt

@@ -136,6 +136,10 @@ class StarredReposViewModel (
                 // Handled in composable
             }
 
+            is StarredReposAction.OnSingInClick -> {
+                // Handled in composable
+            }
+
             StarredReposAction.OnRefresh -> {
                 syncStarredRepos(forceRefresh = true)
             }