Merge pull request #200 from usetrmnl/fix/issue-199-enable-r8-minification

feat: Enable R8 minification and resource shrinking (#199)

Author: hossain-khan

app/build.gradle.kts

@@ -81,10 +81,12 @@ android {
             buildConfigField("Boolean", "USE_FAKE_API", "false")
 
             // Enables code shrinking, obfuscation, and optimization
-            isMinifyEnabled = false
+            // See https://github.com/usetrmnl/trmnl-android/issues/199
+            isMinifyEnabled = true
 
             // Enables resource shrinking, which is performed by the Android Gradle plugin.
-            isShrinkResources = false
+            // See https://github.com/usetrmnl/trmnl-android/issues/199
+            isShrinkResources = true
 
             proguardFiles(
                 getDefaultProguardFile("proguard-android-optimize.txt"),

app/src/main/java/ink/trmnl/android/di/NetworkModule.kt

@@ -10,6 +10,7 @@ import com.squareup.moshi.Moshi
 import com.squareup.moshi.kotlin.reflect.KotlinJsonAdapterFactory
 import dagger.Module
 import dagger.Provides
+import ink.trmnl.android.BuildConfig
 import ink.trmnl.android.network.TrmnlApiService
 import okhttp3.Cache
 import okhttp3.OkHttpClient
@@ -30,11 +31,6 @@ object NetworkModule {
     fun provideOkHttpClient(
         @ApplicationContext context: Context,
     ): OkHttpClient {
-        val loggingInterceptor =
-            HttpLoggingInterceptor().apply {
-                level = HttpLoggingInterceptor.Level.BODY
-            }
-
         // Create cache directory
         val cacheDir = File(context.cacheDir, "http_cache")
         val cache = Cache(cacheDir, CACHE_SIZE)
@@ -59,8 +55,16 @@ object NetworkModule {
                         .header("User-Agent", userAgent)
                         .build()
                 chain.proceed(request)
-            }.addInterceptor(loggingInterceptor)
-            .cache(cache)
+            }.apply {
+                // Only add logging interceptor in debug builds
+                if (BuildConfig.DEBUG) {
+                    addInterceptor(
+                        HttpLoggingInterceptor().apply {
+                            level = HttpLoggingInterceptor.Level.BODY
+                        },
+                    )
+                }
+            }.cache(cache)
             .connectTimeout(30, TimeUnit.SECONDS)
             .readTimeout(30, TimeUnit.SECONDS)
             .writeTimeout(30, TimeUnit.SECONDS)