refactor: use kermit-io for file logging (WPB-26806)

app/build.gradle.kts

@@ -196,6 +196,8 @@ if (!project.hasProperty("skip.aboutlibraries")) {
 }
 
 dependencies {
+    implementation(libs.kermit.io)
+
     implementation("com.wire.kalium:kalium-logic")
     implementation("com.wire.kalium:kalium-util")
     implementation("com.wire.kalium:kalium-cells")

app/src/main/kotlin/com/wire/android/WireApplication.kt

@@ -28,6 +28,7 @@
 import androidx.lifecycle.ProcessLifecycleOwner
 import androidx.work.Configuration
 import androidx.work.WorkManager
+import co.touchlab.kermit.LogWriter
 import co.touchlab.kermit.platformLogWriter
 import com.wire.android.analytics.ObserveCurrentSessionAnalyticsUseCase
 import com.wire.android.datastore.GlobalDataStore
@@ -351,7 +352,7 @@
         ExternalLoggerManager.initDatadogLogger(applicationContext)
 
         val isLoggingEnabled = globalDataStore.value.isLoggingEnabled().firstOrNull() == true
-        val config = fullLoggerConfig(isLoggingEnabled)
+        val config = fullLoggerConfig(isLoggingEnabled, logFileWriter.value.logWriter)
 
         AppLogger.init(config)
         CoreLogger.init(config)
@@ -467,13 +468,16 @@
             listOf(platformLogWriter())
         )
 
-        fun fullLoggerConfig(isLoggingEnabled: Boolean) = if (isLoggingEnabled) {
+        fun fullLoggerConfig(isLoggingEnabled: Boolean, fileLogWriter: LogWriter? = null) = if (isLoggingEnabled) {
             KaliumLogger.Config(
                 KaliumLogLevel.VERBOSE,
-                listOf(DataDogLogger, platformLogWriter())
+                listOfNotNull(DataDogLogger, platformLogWriter(), fileLogWriter)
             )
         } else {
-            minimalLoggerConfig()
+            KaliumLogger.Config(
+                KaliumLogLevel.WARN,
+                listOfNotNull(platformLogWriter(), fileLogWriter)
+            )
         }
 
         enum class MemoryLevel(val level: Int) {

app/src/main/kotlin/com/wire/android/ui/debug/LogManagementViewModel.kt

@@ -22,6 +22,7 @@
 import androidx.compose.runtime.setValue
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
+import com.wire.android.AppLogger
 import com.wire.android.datastore.GlobalDataStore
 import com.wire.android.util.logging.LogFileWriter
 import dev.zacsweers.metro.Inject
@@ -54,16 +55,18 @@
             globalDataStore.setLoggingEnabled(isEnabled)
             if (isEnabled) {
                 logFileWriter.start()
+                AppLogger.setLogLevel(level = KaliumLogLevel.VERBOSE)
                 CoreLogger.setLoggingLevel(level = KaliumLogLevel.VERBOSE)
             } else {
                 logFileWriter.stop()
+                AppLogger.setLogLevel(level = KaliumLogLevel.WARN)
                 CoreLogger.setLoggingLevel(level = KaliumLogLevel.DISABLED)
             }
         }
     }
 
     fun deleteLogs() {
-        logFileWriter.deleteAllLogFiles()
+        viewModelScope.launch { logFileWriter.deleteAllLogFiles() }
     }
 
     fun flushLogs(): Deferred<Unit> {

app/src/main/kotlin/com/wire/android/ui/debug/UserDebugViewModel.kt

@@ -23,6 +23,7 @@
 import androidx.compose.runtime.setValue
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
+import com.wire.android.AppLogger
 import com.wire.android.datastore.GlobalDataStore
 import com.wire.android.di.CurrentAccount
 import com.wire.android.util.EMPTY
@@ -83,7 +84,7 @@
     }
 
     fun deleteLogs() {
-        logFileWriter.deleteAllLogFiles()
+        viewModelScope.launch { logFileWriter.deleteAllLogFiles() }
     }
 
     fun flushLogs(): Deferred<Unit> {
@@ -97,9 +98,11 @@
             globalDataStore.setLoggingEnabled(isEnabled)
             if (isEnabled) {
                 logFileWriter.start()
+                AppLogger.setLogLevel(level = KaliumLogLevel.VERBOSE)
                 CoreLogger.setLoggingLevel(level = KaliumLogLevel.VERBOSE)
             } else {
                 logFileWriter.stop()
+                AppLogger.setLogLevel(level = KaliumLogLevel.WARN)
                 CoreLogger.setLoggingLevel(level = KaliumLogLevel.DISABLED)
             }
         }

app/src/main/kotlin/com/wire/android/util/logging/LogFileWriter.kt

@@ -19,6 +19,7 @@
 package com.wire.android.util.logging
 
 import android.content.Context
+import co.touchlab.kermit.LogWriter
 import java.io.File
 
 /**
@@ -27,6 +28,9 @@ import java.io.File
  */
 interface LogFileWriter {
 
+    /** The Kermit sink that persists enabled diagnostics directly to files. */
+    val logWriter: LogWriter
+
     /**
      * The active logging file where logs are currently being written
      */
@@ -51,7 +55,7 @@ interface LogFileWriter {
      * Deletes all log files including active and compressed files
      *
      */
-    fun deleteAllLogFiles()
+    suspend fun deleteAllLogFiles()
 
     companion object {
         fun logsDirectory(context: Context) = File(context.cacheDir, "logs")

app/src/main/kotlin/com/wire/android/util/logging/LogFileWriterConfig.kt

@@ -19,20 +19,16 @@
 package com.wire.android.util.logging
 
 data class LogFileWriterConfig(
-    val flushIntervalMs: Long = DEFAULT_FLUSH_INTERVAL_MS,
-    val maxBufferSize: Int = DEFAULT_MAX_BUFFER_SIZE,
-    val bufferSizeBytes: Int = DEFAULT_BUFFER_SIZE_BYTES,
-    val maxFileSize: Long = DEFAULT_MAX_FILE_SIZE_BYTES,
+    val rollOnSizeBytes: Long = DEFAULT_ROLL_ON_SIZE_BYTES,
+    val maxLogFiles: Int = DEFAULT_MAX_LOG_FILES,
     val flushTimeoutMs: Long = DEFAULT_FLUSH_TIMEOUT_MS,
-    val bufferLockTimeoutMs: Long = DEFAULT_BUFFER_LOCK_TIMEOUT_MS
 ) {
     companion object {
-        private const val DEFAULT_FLUSH_INTERVAL_MS = 5000L
-        private const val DEFAULT_MAX_BUFFER_SIZE = 100
-        private const val DEFAULT_BUFFER_SIZE_BYTES = 64 * 1024
-        private const val DEFAULT_MAX_FILE_SIZE_BYTES = 25 * 1024 * 1024L // 25MB
+        private const val DEFAULT_ROLL_ON_SIZE_BYTES = 25 * 1024 * 1024L // 25 MiB
+
+        // RollingFileLogWriter counts the active file, so this retains ten rolled files.
+        private const val DEFAULT_MAX_LOG_FILES = 11
         private const val DEFAULT_FLUSH_TIMEOUT_MS = 5000L // 5 seconds
-        private const val DEFAULT_BUFFER_LOCK_TIMEOUT_MS = 3000L // 3 seconds
 
         fun default() = LogFileWriterConfig()
     }