chore(fix): update network client retry strategies and content wrappe…

…r defaults

android-core/src/main/kotlin/co/anitrend/core/android/compose/design/ContentWrapper.kt

@@ -13,6 +13,7 @@ import androidx.compose.foundation.layout.size
 import androidx.compose.material3.CircularProgressIndicator
 import androidx.compose.material3.FilledTonalButton
 import androidx.compose.material3.MaterialTheme
+import androidx.compose.material3.Surface
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
@@ -74,25 +75,28 @@ private fun LoadingContent(
     config: IStateLayoutConfig,
     modifier: Modifier = Modifier,
 ) {
-    Column(
-        modifier = modifier,
-        verticalArrangement = Arrangement.Center,
-        horizontalAlignment = Alignment.CenterHorizontally
-    ) {
-        ContentImage(drawableResource = config.loadingDrawable)
-        Spacer(modifier = Modifier.height(8.dp))
-        Row(
-            horizontalArrangement = Arrangement.spacedBy(16.dp),
-            verticalAlignment = Alignment.CenterVertically,
+    Surface {
+        Column(
+            modifier = modifier,
+            verticalArrangement = Arrangement.Center,
+            horizontalAlignment = Alignment.CenterHorizontally
         ) {
-            CircularProgressIndicator(
-                modifier = Modifier.size(24.dp).align(alignment = Alignment.CenterVertically),
-                color = MaterialTheme.colorScheme.surfaceVariant,
-                strokeWidth = 4.dp,
-                trackColor = MaterialTheme.colorScheme.secondary,
-            )
-            config.loadingMessage?.also {
-                ContentText(text = stringResource(it))
+            ContentImage(drawableResource = config.loadingDrawable)
+            Spacer(modifier = Modifier.height(8.dp))
+            Row(
+                horizontalArrangement = Arrangement.spacedBy(8.dp),
+                verticalAlignment = Alignment.CenterVertically,
+            ) {
+                CircularProgressIndicator(
+                    modifier = Modifier.size(16.dp)
+                        .align(alignment = Alignment.CenterVertically),
+                    color = MaterialTheme.colorScheme.surfaceVariant,
+                    strokeWidth = 2.dp,
+                    trackColor = MaterialTheme.colorScheme.secondary,
+                )
+                config.loadingMessage?.also {
+                    ContentText(text = stringResource(it))
+                }
             }
         }
     }
@@ -106,22 +110,24 @@ private fun ErrorContent(
     onClick: suspend () -> Unit,
 ) {
     val scope = rememberCoroutineScope()
-    Column(
-        modifier = modifier,
-        verticalArrangement = Arrangement.Center,
-        horizontalAlignment = Alignment.CenterHorizontally
-    ) {
-        ContentImage(drawableResource = config.errorDrawable)
-        Spacer(modifier = Modifier.height(16.dp))
-        state.details.message?.also {
-            ContentText(text = it)
-        }
-        config.retryAction?.also {
+    Surface {
+        Column(
+            modifier = modifier,
+            verticalArrangement = Arrangement.Center,
+            horizontalAlignment = Alignment.CenterHorizontally
+        ) {
+            ContentImage(drawableResource = config.errorDrawable)
             Spacer(modifier = Modifier.height(16.dp))
-            FilledTonalButton(
-                onClick = { scope.launch { onClick() } },
-            ) {
-                Text(text = stringResource(it))
+            state.details.message?.also {
+                ContentText(text = it)
+            }
+            config.retryAction?.also {
+                Spacer(modifier = Modifier.height(16.dp))
+                FilledTonalButton(
+                    onClick = { scope.launch { onClick() } },
+                ) {
+                    Text(text = stringResource(it))
+                }
             }
         }
     }
@@ -174,7 +180,8 @@ fun <P: IParam> ContentWrapper(
 }
 
 
-@AniTrendPreview.Mobile
+@AniTrendPreview.Light
+@AniTrendPreview.Dark
 @Composable
 private fun ContentWrapperPreview(
     @PreviewParameter(provider = ContentWrapperPreviewParameter::class) loadState: LoadState
@@ -187,7 +194,7 @@ private fun ContentWrapperPreview(
         defaultMessage = co.anitrend.core.android.R.string.app_controller_message_missing_param,
         retryAction = co.anitrend.core.android.R.string.action_share
     )
-    PreviewTheme(wrapInSurface = true) {
+    PreviewTheme {
         when (loadState) {
             is LoadState.Error -> ErrorContent(
                 config = config,

app-core/src/main/kotlin/co/anitrend/core/component/viewmodel/state/AniTrendViewModelState.kt

@@ -36,22 +36,22 @@ abstract class AniTrendViewModelState<T> : ISupportViewModelState<T>,
     protected abstract val interactor: IUseCase
 
     override val model = state.switchMap {
-        Timber.i("Performing `model` switch map using $coroutineContext on $this")
+        Timber.d("Performing `model` switch map using $coroutineContext on $this")
         it.model.asLiveData(coroutineContext)
     }
 
     override val loadState = state.switchMap {
-        Timber.i("Performing `loadState` switch map using $coroutineContext on $this")
+        Timber.d("Performing `loadState` switch map using $coroutineContext on $this")
         it.loadState.asLiveData(coroutineContext)
     }
 
     override val refreshState = state.switchMap {
-        Timber.i("Performing `refreshState` switch map using $coroutineContext on $this")
+        Timber.d("Performing `refreshState` switch map using $coroutineContext on $this")
         it.refreshState.asLiveData(coroutineContext)
     }
 
     val combinedLoadState = state.switchMap {
-        Timber.i("Performing `combinedLoadState` switch map using $coroutineContext on $this")
+        Timber.d("Performing `combinedLoadState` switch map using $coroutineContext on $this")
         val result = merge(it.loadState, it.refreshState)
         result.asLiveData(coroutineContext)
     }

app-core/src/main/kotlin/co/anitrend/core/koin/Modules.kt

@@ -35,7 +35,9 @@ import co.anitrend.core.coil.client.CoilRequestClient
 import co.anitrend.core.coil.fetch.RequestImageFetcher
 import co.anitrend.core.coil.mapper.RequestImageMapper
 import co.anitrend.data.android.koin.dataModules
+import co.anitrend.data.android.network.cache.CacheHelper
 import co.anitrend.data.android.network.model.NetworkMessage
+import co.anitrend.data.core.extensions.defaultBuilder
 import coil.ImageLoader
 import coil.decode.GifDecoder
 import coil.decode.ImageDecoderDecoder
@@ -45,6 +47,7 @@ import coil.disk.DiskCache
 import coil.memory.MemoryCache
 import io.wax911.emojify.EmojiManager
 import io.wax911.emojify.serializer.kotlinx.KotlinxDeserializer
+import okhttp3.CookieJar
 import okhttp3.OkHttpClient
 import okhttp3.logging.HttpLoggingInterceptor
 import org.koin.android.ext.koin.androidContext
@@ -113,9 +116,15 @@ private val configurationModule = module {
         val memoryLimit = if (isLowRamDevice) 0.15 else 0.35
         val storageController = get<IStorageController>()
 
-        val client = get<OkHttpClient.Builder> {
-            parametersOf(HttpLoggingInterceptor.Level.BASIC)
-        }.build()
+        val client = defaultBuilder()
+            .cookieJar(get<CookieJar>())
+            .cache(
+                CacheHelper.createCache(
+                    androidContext(),
+                    "coil-okhttp"
+                )
+            )
+            .build()
 
         val imageCache = storageController.getImageCache(
             context = androidContext()

app-data-android/src/main/kotlin/co/anitrend/data/android/controller/strategy/contract/ControllerStrategy.kt

@@ -22,6 +22,7 @@ import co.anitrend.arch.request.callback.RequestCallback
 import co.anitrend.data.android.network.model.NetworkMessage
 import retrofit2.HttpException
 import java.net.SocketTimeoutException
+import java.net.UnknownHostException
 
 /**
  * Contract for controller strategy
@@ -49,6 +50,13 @@ abstract class ControllerStrategy<D> {
                 cause
             )
         }
+        is UnknownHostException -> {
+            RequestError(
+                networkMessage.connectivityErrorTittle,
+                networkMessage.connectivityErrorMessage,
+                cause
+            )
+        }
         else -> {
             if (cause == null)
                 RequestError(

app-data-android/src/main/kotlin/co/anitrend/data/android/controller/strategy/policy/OfflineStrategy.kt

@@ -44,7 +44,7 @@ class OfflineStrategy<D> private constructor(
     ): D? {
         runCatching {
             block()
-        }.onSuccess {result ->
+        }.onSuccess { result ->
             callback.recordSuccess()
             return result
         }.onFailure { exception ->

app-data-android/src/main/kotlin/co/anitrend/data/android/network/client/DeferrableNetworkClient.kt

@@ -55,8 +55,7 @@ abstract class DeferrableNetworkClient<T> : AbstractNetworkClient<Deferred<Respo
      */
     override fun defaultShouldRetry(exception: Throwable) = when (exception) {
         is HttpException -> exception.code() == 429
-        is SocketTimeoutException,
-        is IOException -> true
+        is SocketTimeoutException -> true
         else -> false
     }
 
@@ -120,4 +119,4 @@ abstract class DeferrableNetworkClient<T> : AbstractNetworkClient<Deferred<Respo
         maxAttempts,
         shouldRetry
     ).bodyOrThrow()
-}
+}

app-data-android/src/main/kotlin/co/anitrend/data/android/network/client/OkHttpCallNetworkClient.kt

@@ -52,8 +52,7 @@ abstract class OkHttpCallNetworkClient : AbstractNetworkClient<Call, Response>()
      */
     override fun defaultShouldRetry(exception: Throwable) = when (exception) {
         is OkHttpException -> exception.code == 429
-        is SocketTimeoutException,
-        is IOException -> true
+        is SocketTimeoutException -> true
         else -> false
     }
 
@@ -118,4 +117,4 @@ abstract class OkHttpCallNetworkClient : AbstractNetworkClient<Call, Response>()
         maxAttempts,
         shouldRetry
     ).bodyOrThrow()
-}
+}

app-data-android/src/main/kotlin/co/anitrend/data/android/network/client/RetrofitCallNetworkClient.kt

@@ -46,8 +46,7 @@ internal abstract class RetrofitCallNetworkClient<T> : AbstractNetworkClient<Cal
      */
     override fun defaultShouldRetry(exception: Throwable) = when (exception) {
         is HttpException -> exception.code() == 429
-        is SocketTimeoutException,
-        is IOException -> true
+        is SocketTimeoutException -> true
         else -> false
     }
 
@@ -112,4 +111,4 @@ internal abstract class RetrofitCallNetworkClient<T> : AbstractNetworkClient<Cal
         maxAttempts,
         shouldRetry
     ).bodyOrThrow()
-}
+}

app-data-edge/.config/configuration.properties

@@ -1,2 +1 @@
-#edgeHost="https://api.anitrend.docker.localhost/graphql/"
-edgeHost="https://graphql.anitrend.co/graphql/"
+edgeHost="https://api.anitrend.co"

app-data-edge/src/main/kotlin/co/anitrend/data/edge/core/api/factory/EdgeApiFactory.kt

@@ -45,6 +45,6 @@ internal class EdgeApiFactory(
     }
 
     companion object {
-        const val BASE_ENDPOINT_PATH = "/graphql/"
+        const val BASE_ENDPOINT_PATH = "/graphql"
     }
 }

app-data/graphql.config.yml

@@ -8,12 +8,11 @@ extensions:
         user-agent: JS GraphQL
       introspect: true
     AniTrend:
-      url: https://graphql.anitrend.co/graphql/
+      url: https://api.anitrend.co/graphql
       headers:
         User-Agent: "JS GraphQL"
         Accept: "*/*"
         HOST: "graphql.anitrend.co"
         Accept-Encoding: "gzip, deflate, br"
-        Accept-Language: "en-GB,en-US;q=0.9,en;q=0.8"
         Content-Type: "application/json"
       introspect: true

app-data/src/main/kotlin/co/anitrend/data/auth/datasource/local/AuthLocalSource.kt

@@ -39,7 +39,7 @@ internal abstract class AuthLocalSource : AbstractLocalSource<AuthEntity>() {
         delete from auth
         where user_id = :userId
         """)
-    abstract fun clearByUserId(userId: Long)
+    abstract suspend fun clearByUserId(userId: Long)
 
     @Query("""
             select * from auth
@@ -51,4 +51,4 @@ internal abstract class AuthLocalSource : AbstractLocalSource<AuthEntity>() {
             select user_id from auth
         """)
     abstract suspend fun userIds(): List<Long>
-}
+}

app-data/src/test/kotlin/co/anitrend/data/model/query/QueryMappingTest.kt

@@ -66,7 +66,7 @@ class QueryMappingTest {
                     month = 5
                 ).toFuzzyDateInt()
                 season = MediaSeason.FALL
-                format_in = MediaFormat.values().asList()
+                format_in = MediaFormat.entries
                 onList = true
                 status_in = listOf(
                     MediaStatus.FINISHED,

common-media-ui/src/main/kotlin/co/anitrend/common/media/ui/compose/item/MediaCarouselItem.kt

@@ -112,7 +112,6 @@ fun MediaCarouselItem(
         items(
             count = carouselItems.size,
             key = { carouselItems[it].hashCode() },
-            contentType = { carouselItems[it].carouselType },
         ) { index ->
             val carouselItem = carouselItems[index]
             CarouselHeader(

common-media-ui/src/main/kotlin/co/anitrend/common/media/ui/compose/item/MediaCompactItem.kt

@@ -150,7 +150,7 @@ fun MediaCompactItemList(
         items(
             count = mediaItems.size,
             key = { mediaItems[it].hashCode() },
-            contentType = { mediaItems[it].category.type }
+            contentType = { mediaItems[it].category }
         ) { index ->
             MediaCompactItem(
                 media = mediaItems[index],
@@ -185,7 +185,13 @@ private fun MediaCompactItemPreview() {
             mediaPreferenceData = MediaPreferenceData(
                 ScoreFormat.POINT_10_DECIMAL,
             ),
-            mediaItemClick = {}
+            mediaItemClick = {},
+            modifier = Modifier
+                .padding(8.dp)
+                .size(
+                    height = 265.dp,
+                    width = 150.dp,
+                ),
         )
     }
 }