Replace Stetho with Flipper, add LeakCanary

app/src/debug/java/gq/kirmanak/mealient/App.kt

@@ -0,0 +1,33 @@
+package gq.kirmanak.mealient
+
+import android.app.Application
+import com.facebook.flipper.android.AndroidFlipperClient
+import com.facebook.flipper.android.utils.FlipperUtils
+import com.facebook.flipper.core.FlipperPlugin
+import com.facebook.soloader.SoLoader
+import dagger.hilt.android.HiltAndroidApp
+import timber.log.Timber
+import javax.inject.Inject
+
+@HiltAndroidApp
+class App : Application() {
+  // Use @JvmSuppressWildcards because otherwise dagger can't inject it (https://stackoverflow.com/a/43149382)
+  @Inject
+  lateinit var flipperPlugins: Set<@JvmSuppressWildcards FlipperPlugin>
+
+  override fun onCreate() {
+    super.onCreate()
+    Timber.plant(Timber.DebugTree())
+    Timber.v("onCreate() called")
+    setupFlipper()
+  }
+
+  private fun setupFlipper() {
+    if (FlipperUtils.shouldEnableFlipper(this)) {
+      SoLoader.init(this, false)
+      val flipperClient = AndroidFlipperClient.getInstance(this)
+      for (flipperPlugin in flipperPlugins) flipperClient.addPlugin(flipperPlugin)
+      flipperClient.start()
+    }
+  }
+}

app/src/debug/java/gq/kirmanak/mealient/di/DebugModule.kt

@@ -0,0 +1,78 @@
+package gq.kirmanak.mealient.di
+
+import android.content.Context
+import com.facebook.flipper.core.FlipperPlugin
+import com.facebook.flipper.plugins.databases.DatabasesFlipperPlugin
+import com.facebook.flipper.plugins.inspector.DescriptorMapping
+import com.facebook.flipper.plugins.inspector.InspectorFlipperPlugin
+import com.facebook.flipper.plugins.leakcanary2.FlipperLeakListener
+import com.facebook.flipper.plugins.leakcanary2.LeakCanary2FlipperPlugin
+import com.facebook.flipper.plugins.network.FlipperOkhttpInterceptor
+import com.facebook.flipper.plugins.network.NetworkFlipperPlugin
+import com.facebook.flipper.plugins.sharedpreferences.SharedPreferencesFlipperPlugin
+import dagger.Module
+import dagger.Provides
+import dagger.hilt.InstallIn
+import dagger.hilt.android.qualifiers.ApplicationContext
+import dagger.hilt.components.SingletonComponent
+import dagger.multibindings.IntoSet
+import leakcanary.LeakCanary
+import okhttp3.Interceptor
+import okhttp3.logging.HttpLoggingInterceptor
+import timber.log.Timber
+import javax.inject.Singleton
+
+@Module
+@InstallIn(SingletonComponent::class)
+object DebugModule {
+  @Provides
+  @Singleton
+  @IntoSet
+  fun provideLoggingInterceptor(): Interceptor {
+    val interceptor = HttpLoggingInterceptor { message -> Timber.tag("OkHttp").v(message) }
+    interceptor.level = HttpLoggingInterceptor.Level.BODY
+    return interceptor
+  }
+
+  @Provides
+  @Singleton
+  @IntoSet
+  fun provideFlipperInterceptor(networkFlipperPlugin: NetworkFlipperPlugin): Interceptor {
+    return FlipperOkhttpInterceptor(networkFlipperPlugin)
+  }
+
+  @Provides
+  @Singleton
+  fun networkFlipperPlugin() = NetworkFlipperPlugin()
+
+  @Provides
+  @Singleton
+  @IntoSet
+  fun bindNetworkFlipperPlugin(plugin: NetworkFlipperPlugin): FlipperPlugin = plugin
+
+  @Provides
+  @Singleton
+  @IntoSet
+  fun sharedPreferencesPlugin(@ApplicationContext context: Context): FlipperPlugin =
+    SharedPreferencesFlipperPlugin(context)
+
+  @Provides
+  @Singleton
+  @IntoSet
+  fun leakCanaryPlugin(): FlipperPlugin {
+    LeakCanary.config = LeakCanary.config.copy(onHeapAnalyzedListener = FlipperLeakListener())
+    return LeakCanary2FlipperPlugin()
+  }
+
+  @Provides
+  @Singleton
+  @IntoSet
+  fun databasesPlugin(@ApplicationContext context: Context): FlipperPlugin =
+    DatabasesFlipperPlugin(context)
+
+  @Provides
+  @Singleton
+  @IntoSet
+  fun inspectorPlugin(@ApplicationContext context: Context): FlipperPlugin =
+    InspectorFlipperPlugin(context, DescriptorMapping.withDefaults())
+}

app/src/main/java/gq/kirmanak/mealient/App.kt

@@ -1,14 +0,0 @@
-package gq.kirmanak.mealient
-
-import android.app.Application
-import dagger.hilt.android.HiltAndroidApp
-import timber.log.Timber
-
-@HiltAndroidApp
-class App : Application() {
-    override fun onCreate() {
-        super.onCreate()
-        if (BuildConfig.DEBUG) Timber.plant(Timber.DebugTree())
-        Timber.v("onCreate() called")
-    }
-}

app/src/main/java/gq/kirmanak/mealient/data/impl/OkHttpBuilder.kt

@@ -1,27 +1,21 @@
 package gq.kirmanak.mealient.data.impl
 
-import gq.kirmanak.mealient.BuildConfig
-import gq.kirmanak.mealient.data.auth.impl.AuthOkHttpInterceptor
 import okhttp3.Interceptor
 import okhttp3.OkHttpClient
-import okhttp3.logging.HttpLoggingInterceptor
 import timber.log.Timber
 import javax.inject.Inject
 
-class OkHttpBuilder @Inject constructor(
-    private val authOkHttpInterceptor: AuthOkHttpInterceptor
+class OkHttpBuilder
+@Inject
+constructor(
+    // Use @JvmSuppressWildcards because otherwise dagger can't inject it (https://stackoverflow.com/a/43149382)
+    private val interceptors: Set<@JvmSuppressWildcards Interceptor>
 ) {
+
     fun buildOkHttp(): OkHttpClient {
         Timber.v("buildOkHttp() called")
-        val builder = OkHttpClient.Builder()
-        if (BuildConfig.DEBUG) builder.addNetworkInterceptor(buildLoggingInterceptor())
-        builder.addNetworkInterceptor(authOkHttpInterceptor)
-        return builder.build()
+        return OkHttpClient.Builder()
+            .apply { for (interceptor in interceptors) addNetworkInterceptor(interceptor) }
+            .build()
     }
-
-    private fun buildLoggingInterceptor(): Interceptor {
-        val interceptor = HttpLoggingInterceptor { message -> Timber.tag("OkHttp").v(message) }
-        interceptor.level = HttpLoggingInterceptor.Level.BODY
-        return interceptor
-    }
-}
+}

app/src/main/java/gq/kirmanak/mealient/di/AuthModule.kt

@@ -4,26 +4,33 @@ import dagger.Binds
 import dagger.Module
 import dagger.hilt.InstallIn
 import dagger.hilt.components.SingletonComponent
+import dagger.multibindings.IntoSet
 import gq.kirmanak.mealient.data.auth.AuthDataSource
 import gq.kirmanak.mealient.data.auth.AuthRepo
 import gq.kirmanak.mealient.data.auth.AuthStorage
 import gq.kirmanak.mealient.data.auth.impl.AuthDataSourceImpl
+import gq.kirmanak.mealient.data.auth.impl.AuthOkHttpInterceptor
 import gq.kirmanak.mealient.data.auth.impl.AuthRepoImpl
 import gq.kirmanak.mealient.data.auth.impl.AuthStorageImpl
 import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.serialization.ExperimentalSerializationApi
+import okhttp3.Interceptor
 
 @ExperimentalCoroutinesApi
 @ExperimentalSerializationApi
 @Module
 @InstallIn(SingletonComponent::class)
 interface AuthModule {
-    @Binds
-    fun bindAuthDataSource(authDataSourceImpl: AuthDataSourceImpl): AuthDataSource
+  @Binds
+  fun bindAuthDataSource(authDataSourceImpl: AuthDataSourceImpl): AuthDataSource
 
-    @Binds
-    fun bindAuthStorage(authStorageImpl: AuthStorageImpl): AuthStorage
+  @Binds
+  fun bindAuthStorage(authStorageImpl: AuthStorageImpl): AuthStorage
 
-    @Binds
-    fun bindAuthRepo(authRepo: AuthRepoImpl): AuthRepo
-}
+  @Binds
+  fun bindAuthRepo(authRepo: AuthRepoImpl): AuthRepo
+
+  @Binds
+  @IntoSet
+  fun bindAuthInterceptor(authOkHttpInterceptor: AuthOkHttpInterceptor): Interceptor
+}

app/src/release/java/gq/kirmanak/mealient/App.kt

@@ -0,0 +1,7 @@
+package gq.kirmanak.mealient
+
+import android.app.Application
+import dagger.hilt.android.HiltAndroidApp
+
+@HiltAndroidApp
+class App : Application()