feat: optimized IO file logger

app/src/androidTest/kotlin/com/wire/android/ui/debug/DebugScreenComposeTest.kt

@@ -20,6 +20,7 @@ package com.wire.android.ui.debug
 import androidx.compose.ui.test.junit4.createComposeRule
 import com.wire.android.extensions.waitUntilExists
 import com.wire.android.ui.WireTestTheme
+import kotlinx.coroutines.CompletableDeferred
 import kotlinx.coroutines.test.runTest
 import org.junit.Rule
 import org.junit.Test
@@ -40,6 +41,7 @@ class DebugScreenComposeTest {
                         onDeleteLogs = {},
                         onDatabaseLoggerEnabledChanged = {},
                         onShowFeatureFlags = {},
+                        onFlushLogs = { CompletableDeferred(Unit) },
                     )
             }
         }

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

@@ -40,7 +40,7 @@
 import com.wire.android.util.AppNameUtil
 import com.wire.android.util.CurrentScreenManager
 import com.wire.android.util.DataDogLogger
-import com.wire.android.util.LogFileWriter
+import com.wire.android.util.logging.LogFileWriter
 import com.wire.android.util.getGitBuildId
 import com.wire.android.util.lifecycle.SyncLifecycleManager
 import com.wire.android.workmanager.WireWorkerFactory
@@ -59,8 +59,10 @@
 import kotlinx.coroutines.flow.flatMapLatest
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
+import kotlinx.coroutines.withTimeout
 import javax.inject.Inject
 
+@Suppress("TooManyFunctions")
 @HiltAndroidApp
 class WireApplication : BaseApp() {
 
@@ -107,6 +109,8 @@
 
         enableStrictMode()
 
+        setupGlobalExceptionHandler()
+
         startActivityLifecycleCallback()
 
         globalAppScope.launch {
@@ -160,6 +164,82 @@
         }
     }
 
+    private fun setupGlobalExceptionHandler() {
+        setupUncaughtExceptionHandler()
+        setupHistoricalExitMonitoring()
+    }
+
+    @Suppress("TooGenericExceptionCaught")
+    private fun setupUncaughtExceptionHandler() {
+        val defaultHandler = Thread.getDefaultUncaughtExceptionHandler()
+        Thread.setDefaultUncaughtExceptionHandler { thread, exception ->
+            flushLogsBeforeCrash()
+            defaultHandler?.uncaughtException(thread, exception)
+        }
+    }
+
+    @Suppress("TooGenericExceptionCaught")
+    private fun flushLogsBeforeCrash() {
+        // Use fire-and-forget approach to avoid blocking the crash handler
+        // which could lead to ANRs. We attempt a quick flush but don't wait for it.
+        try {
+            globalAppScope.launch(Dispatchers.IO) {
+                try {
+                    // Use a very short timeout to avoid delaying the crash
+                    withTimeout(CRASH_FLUSH_TIMEOUT_MS) {
+                        logFileWriter.get().forceFlush()
+                    }
+                    appLogger.i("Logs flushed before crash")
+                } catch (e: Exception) {
+                    // Log errors but don't block the crash handler
+                    appLogger.e("Failed to flush logs before crash", e)
+                }
+            }
+        } catch (e: Exception) {
+            // Ignore any launch failures - we don't want to interfere with crash handling
+        }
+    }
+
+    @Suppress("TooGenericExceptionCaught")
+    private fun setupHistoricalExitMonitoring() {
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R) {
+            try {
+                val activityManager = getSystemService(ACTIVITY_SERVICE) as android.app.ActivityManager
+                activityManager.setProcessStateSummary(ByteArray(0))
+
+                // This will be called after the app exits, so we can't flush here,
+                // but we log it for diagnostics
+                globalAppScope.launch {
+                    activityManager.getHistoricalProcessExitReasons(packageName, 0, MAX_HISTORICAL_EXIT_REASONS)
+                        .forEach { info ->
+                            logPreviousExitReason(info)
+                        }
+                }
+            } catch (e: Exception) {
+                appLogger.e("Failed to setup app exit monitoring", e)
+            }
+        }
+    }
+
+    private fun logPreviousExitReason(info: android.app.ApplicationExitInfo) {
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R) {
+            when (info.reason) {
+                android.app.ApplicationExitInfo.REASON_ANR -> {
+                    appLogger.w("Previous app exit was due to ANR at ${info.timestamp}")
+                }
+                android.app.ApplicationExitInfo.REASON_CRASH -> {
+                    appLogger.w("Previous app exit was due to crash at ${info.timestamp}")
+                }
+                android.app.ApplicationExitInfo.REASON_LOW_MEMORY -> {
+                    appLogger.w("Previous app exit was due to low memory at ${info.timestamp}")
+                }
+                else -> {
+                    appLogger.i("Previous app exit reason: ${info.reason} at ${info.timestamp}")
+                }
+            }
+        }
+    }
+
     @Suppress("EmptyFunctionBlock")
     private fun startActivityLifecycleCallback() {
         registerActivityLifecycleCallbacks(object : ActivityLifecycleCallbacks {
@@ -290,7 +370,9 @@
     override fun onLowMemory() {
         super.onLowMemory()
         appLogger.w("onLowMemory called - Stopping logging, buckling the seatbelt and hoping for the best!")
-        logFileWriter.get().stop()
+        globalAppScope.launch {
+            logFileWriter.get().stop()
+        }
     }
 
     private companion object {
@@ -313,5 +395,7 @@
         }
 
         private const val TAG = "WireApplication"
+        private const val CRASH_FLUSH_TIMEOUT_MS = 1000L
+        private const val MAX_HISTORICAL_EXIT_REASONS = 5
     }
 }

app/src/main/kotlin/com/wire/android/di/LogWriterModule.kt

@@ -19,7 +19,10 @@
 package com.wire.android.di
 
 import android.content.Context
-import com.wire.android.util.LogFileWriter
+import com.wire.android.BuildConfig
+import com.wire.android.util.logging.LogFileWriterV1Impl
+   import com.wire.android.util.logging.LogFileWriter
+import com.wire.android.util.logging.LogFileWriterV2Impl
 import dagger.Module
 import dagger.Provides
 import dagger.hilt.InstallIn
@@ -34,7 +37,12 @@ class LogWriterModule {
     @Singleton
     @Provides
     fun provideKaliumFileWriter(@ApplicationContext context: Context): LogFileWriter {
-        val logsDirectory = LogFileWriter.logsDirectory(context)
-        return LogFileWriter(logsDirectory)
+        if (BuildConfig.USE_ASYNC_FLUSH_LOGGING) {
+            val logsDirectory = LogFileWriter.logsDirectory(context)
+            return LogFileWriterV2Impl(logsDirectory)
+        } else {
+            val logsDirectory = LogFileWriter.logsDirectory(context)
+            return LogFileWriterV1Impl(logsDirectory)
+        }
     }
 }

app/src/main/kotlin/com/wire/android/navigation/OtherDestinations.kt

@@ -26,10 +26,10 @@ import com.ramcosta.composedestinations.spec.Direction
 import com.wire.android.BuildConfig
 import com.wire.android.R
 import com.wire.android.util.EmailComposer
-import com.wire.android.util.LogFileWriter
 import com.wire.android.util.getDeviceIdString
 import com.wire.android.util.getGitBuildId
 import com.wire.android.util.getUrisOfFilesInDirectory
+import com.wire.android.util.logging.LogFileWriter
 import com.wire.android.util.multipleFileSharingIntent
 import com.wire.android.util.sha256
 

app/src/main/kotlin/com/wire/android/notification/WireNotificationManager.kt

@@ -28,7 +28,7 @@ import com.wire.android.util.CurrentScreen
 import com.wire.android.util.CurrentScreenManager
 import com.wire.android.util.dispatchers.DispatcherProvider
 import com.wire.android.util.lifecycle.SyncLifecycleManager
-import com.wire.android.util.logIfEmptyUserName
+import com.wire.android.util.logging.logIfEmptyUserName
 import com.wire.kalium.logger.obfuscateId
 import com.wire.kalium.logic.CoreLogic
 import com.wire.kalium.logic.data.id.ConversationId

app/src/main/kotlin/com/wire/android/services/CallServiceManager.kt

@@ -21,7 +21,7 @@ import com.wire.android.appLogger
 import com.wire.android.di.KaliumCoreLogic
 import com.wire.android.notification.CallNotificationData
 import com.wire.android.services.CallService.Action
-import com.wire.android.util.logIfEmptyUserName
+import com.wire.android.util.logging.logIfEmptyUserName
 import com.wire.kalium.common.functional.Either
 import com.wire.kalium.common.functional.fold
 import com.wire.kalium.common.functional.left

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

@@ -31,6 +31,7 @@ import androidx.compose.foundation.rememberScrollState
 import androidx.compose.foundation.verticalScroll
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.remember
+import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.platform.ClipboardManager
@@ -61,6 +62,10 @@ import com.wire.android.util.AppNameUtil
 import com.wire.android.util.getMimeType
 import com.wire.android.util.getUrisOfFilesInDirectory
 import com.wire.android.util.multipleFileSharingIntent
+import kotlinx.coroutines.CompletableDeferred
+import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.Deferred
+import kotlinx.coroutines.launch
 import java.io.File
 
 @WireDestination
@@ -74,6 +79,7 @@ fun DebugScreen(
         state = userDebugViewModel.state,
         onLoggingEnabledChange = userDebugViewModel::setLoggingEnabledState,
         onDeleteLogs = userDebugViewModel::deleteLogs,
+        onFlushLogs = userDebugViewModel::flushLogs,
         onDatabaseLoggerEnabledChanged = userDebugViewModel::setDatabaseLoggerEnabledState,
         onShowFeatureFlags = {
             navigator.navigate(NavigationCommand(DebugFeatureFlagsScreenDestination))
@@ -88,6 +94,7 @@ internal fun UserDebugContent(
     onLoggingEnabledChange: (Boolean) -> Unit,
     onDatabaseLoggerEnabledChanged: (Boolean) -> Unit,
     onDeleteLogs: () -> Unit,
+    onFlushLogs: () -> Deferred<Unit>,
     onShowFeatureFlags: () -> Unit,
 ) {
     val debugContentState: DebugContentState = rememberDebugContentState(state.logPath)
@@ -113,7 +120,7 @@ internal fun UserDebugContent(
                     isLoggingEnabled = isLoggingEnabled,
                     onLoggingEnabledChange = onLoggingEnabledChange,
                     onDeleteLogs = onDeleteLogs,
-                    onShareLogs = debugContentState::shareLogs,
+                    onShareLogs = { debugContentState.shareLogs(onFlushLogs) },
                     isDBLoggerEnabled = state.isDBLoggingEnabled,
                     onDBLoggerEnabledChange = onDatabaseLoggerEnabledChanged,
                     isPrivateBuild = BuildConfig.PRIVATE_BUILD,
@@ -181,13 +188,15 @@ fun rememberDebugContentState(logPath: String): DebugContentState {
     val context = LocalContext.current
     val clipboardManager = LocalClipboardManager.current
     val scrollState = rememberScrollState()
+    val coroutineScope = rememberCoroutineScope()
 
     return remember {
         DebugContentState(
             context,
             clipboardManager,
             logPath,
-            scrollState
+            scrollState,
+            coroutineScope
         )
     }
 }
@@ -196,7 +205,8 @@ data class DebugContentState(
     val context: Context,
     val clipboardManager: ClipboardManager,
     val logPath: String,
-    val scrollState: ScrollState
+    val scrollState: ScrollState,
+    val coroutineScope: CoroutineScope
 ) {
     fun copyToClipboard(text: String) {
         clipboardManager.setText(AnnotatedString(text))
@@ -207,18 +217,22 @@ data class DebugContentState(
         ).show()
     }
 
-    fun shareLogs() {
-        val dir = File(logPath).parentFile
-        val fileUris =
-            if (dir != null && dir.exists()) context.getUrisOfFilesInDirectory(dir) else arrayListOf()
-        val intent = context.multipleFileSharingIntent(fileUris)
-        // The first log file is simply text, not compressed. Get its mime type separately
-        // and set it as the mime type for the intent.
-        intent.type = fileUris.firstOrNull()?.getMimeType(context) ?: "text/plain"
-        // Get all other mime types and add them
-        val mimeTypes = fileUris.drop(1).mapNotNull { it.getMimeType(context) }
-        intent.putExtra(Intent.EXTRA_MIME_TYPES, mimeTypes.toSet().toTypedArray())
-        context.startActivity(intent)
+    fun shareLogs(onFlushLogs: () -> Deferred<Unit>) {
+        coroutineScope.launch {
+            // Flush any buffered logs before sharing to ensure completeness
+            onFlushLogs().await()
+            val dir = File(logPath).parentFile
+            val fileUris =
+                if (dir != null && dir.exists()) context.getUrisOfFilesInDirectory(dir) else arrayListOf()
+            val intent = context.multipleFileSharingIntent(fileUris)
+            // The first log file is simply text, not compressed. Get its mime type separately
+            // and set it as the mime type for the intent.
+            intent.type = fileUris.firstOrNull()?.getMimeType(context) ?: "text/plain"
+            // Get all other mime types and add them
+            val mimeTypes = fileUris.drop(1).mapNotNull { it.getMimeType(context) }
+            intent.putExtra(Intent.EXTRA_MIME_TYPES, mimeTypes.toSet().toTypedArray())
+            context.startActivity(intent)
+        }
     }
 }
 
@@ -233,6 +247,7 @@ internal fun PreviewUserDebugContent() = WireTheme {
         onNavigationPressed = {},
         onLoggingEnabledChange = {},
         onDeleteLogs = {},
+        onFlushLogs = { CompletableDeferred(Unit) },
         onDatabaseLoggerEnabledChanged = {},
         onShowFeatureFlags = {},
     )

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

@@ -26,14 +26,16 @@
 import com.wire.android.datastore.GlobalDataStore
 import com.wire.android.di.CurrentAccount
 import com.wire.android.util.EMPTY
-import com.wire.android.util.LogFileWriter
+import com.wire.android.util.logging.LogFileWriter
 import com.wire.kalium.common.logger.CoreLogger
 import com.wire.kalium.logger.KaliumLogLevel
 import com.wire.kalium.logic.data.user.UserId
 import com.wire.kalium.logic.feature.client.ObserveCurrentClientIdUseCase
 import com.wire.kalium.logic.feature.debug.ChangeProfilingUseCase
 import com.wire.kalium.logic.feature.debug.ObserveDatabaseLoggerStateUseCase
 import dagger.hilt.android.lifecycle.HiltViewModel
+import kotlinx.coroutines.Deferred
+import kotlinx.coroutines.async
 import kotlinx.coroutines.launch
 import javax.inject.Inject
 
@@ -87,6 +89,12 @@
         logFileWriter.deleteAllLogFiles()
     }
 
+    fun flushLogs(): Deferred<Unit> {
+        return viewModelScope.async {
+            logFileWriter.forceFlush()
+        }
+    }
+
     fun setLoggingEnabledState(isEnabled: Boolean) {
         viewModelScope.launch {
             globalDataStore.setLoggingEnabled(isEnabled)