feat: migrate from production console logging to full exportable app logs

bmc08gt · bmc08gt · commit 5bb47ba9f1f1 · 2026-04-07T15:13:21.000-04:00
Signed-off-by: Brandon McAnsh &lt;git@bmcreations.dev&gt;
diff --git a/apps/flipcash/app/src/main/AndroidManifest.xml b/apps/flipcash/app/src/main/AndroidManifest.xml
@@ -245,6 +245,16 @@
             <meta-data android:name="photopicker_activity:0:required" android:value="" />
         </service>
 
+        <provider
+            android:name="androidx.core.content.FileProvider"
+            android:authorities="${applicationId}.fileprovider"
+            android:exported="false"
+            android:grantUriPermissions="true">
+            <meta-data
+                android:name="android.support.FILE_PROVIDER_PATHS"
+                android:resource="@xml/file_paths" />
+        </provider>
+
         <provider
             android:name="androidx.startup.InitializationProvider"
             android:authorities="${applicationId}.androidx-startup"
diff --git a/apps/flipcash/app/src/main/kotlin/com/flipcash/app/internal/startup/TraceInitializer.kt b/apps/flipcash/app/src/main/kotlin/com/flipcash/app/internal/startup/TraceInitializer.kt
@@ -7,13 +7,16 @@ import com.bugsnag.android.Configuration
 import com.flipcash.app.android.BuildConfig
 import com.flipcash.app.internal.debug.FlipcashDebugTree
 import com.flipcash.app.internal.debug.FlipcashErrorCallback
+import com.getcode.utils.TraceManager
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
 import timber.log.Timber
 
 class TraceInitializer: Initializer<Unit> {
     override fun create(context: Context) {
+        TraceManager.initialize(context)
+
         if (BuildConfig.DEBUG) {
             Timber.plant(FlipcashDebugTree)
         } else {
diff --git a/apps/flipcash/app/src/main/res/xml/file_paths.xml b/apps/flipcash/app/src/main/res/xml/file_paths.xml
@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="utf-8"?>
+<paths>
+    <files-path name="traces" path="traces/" />
+</paths>
diff --git a/apps/flipcash/core/src/main/kotlin/com/flipcash/app/core/android/IntentUtils.kt b/apps/flipcash/core/src/main/kotlin/com/flipcash/app/core/android/IntentUtils.kt
@@ -7,8 +7,10 @@ import android.content.pm.PackageManager
 import android.net.Uri
 import android.provider.MediaStore
 import android.provider.Settings
+import androidx.core.content.FileProvider
 import androidx.core.net.toUri
 import com.flipcash.app.core.util.Linkify
+import java.io.File
 
 
 object IntentUtils {
@@ -56,4 +58,20 @@ object IntentUtils {
         setData(Uri.parse("https://play.google.com/store/apps/details?id=$packageName"))
         flags = Intent.FLAG_ACTIVITY_NEW_TASK
     }
+
+    fun shareFile(context: Context, file: File, mimeType: String): Intent {
+        val uri = FileProvider.getUriForFile(
+            context,
+            "${context.packageName}.fileprovider",
+            file
+        )
+        val sendIntent = Intent(Intent.ACTION_SEND).apply {
+            putExtra(Intent.EXTRA_STREAM, uri)
+            type = mimeType
+            addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION)
+        }
+        return Intent.createChooser(sendIntent, null).apply {
+            flags = Intent.FLAG_ACTIVITY_NEW_TASK
+        }
+    }
 }
diff --git a/apps/flipcash/core/src/main/res/values/strings.xml b/apps/flipcash/core/src/main/res/values/strings.xml
@@ -364,6 +364,7 @@
     <string name="description_noTransactionHistory">Your recent activity will appear here when you send or receive money</string>
 
     <string name="title_billCustomizer">Bill Creator</string>
+    <string name="title_exportLogs">Export Logs</string>
 
     <string name="error_title_CashReturnedToWallet">Something Went Wrong</string>
     <string name="error_description_CashReturnedToWallet">The cash was returned to your wallet</string>
diff --git a/apps/flipcash/features/advanced/src/main/kotlin/com/flipcash/app/advanced/AdvancedFeaturesScreen.kt b/apps/flipcash/features/advanced/src/main/kotlin/com/flipcash/app/advanced/AdvancedFeaturesScreen.kt
@@ -6,16 +6,19 @@ import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
+import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.res.stringResource
 import androidx.hilt.navigation.compose.hiltViewModel
 import com.flipcash.app.advanced.internal.AdvancedFeaturesScreen
 import com.flipcash.app.advanced.internal.AdvancedFeaturesScreenViewModel
 import com.flipcash.app.bill.customization.LocalBillPlaygroundController
+import com.flipcash.app.core.android.IntentUtils
 import com.flipcash.core.R
 import com.getcode.navigation.core.LocalCodeNavigator
 import com.getcode.opencode.model.financial.Token
 import com.getcode.opencode.model.financial.usdf
 import com.getcode.ui.components.AppBarWithTitle
+import com.getcode.utils.TraceManager
 import kotlinx.coroutines.flow.filterIsInstance
 import kotlinx.coroutines.flow.launchIn
 import kotlinx.coroutines.flow.onEach
@@ -25,6 +28,7 @@ fun AdvancedFeaturesScreen() {
     val navigator = LocalCodeNavigator.current
     val billPlayground = LocalBillPlaygroundController.current
     val viewModel = hiltViewModel<AdvancedFeaturesScreenViewModel>()
+    val context = LocalContext.current
 
     Column(
         modifier = Modifier.fillMaxSize(),
@@ -57,4 +61,17 @@ fun AdvancedFeaturesScreen() {
             }
             .launchIn(this)
     }
+
+    LaunchedEffect(viewModel) {
+        viewModel.eventFlow
+            .filterIsInstance<AdvancedFeaturesScreenViewModel.Event.ExportLogs>()
+            .onEach {
+                val logFile = TraceManager.getLogFile()
+                if (logFile != null) {
+                    val intent = IntentUtils.shareFile(context, logFile, "text/plain")
+                    context.startActivity(intent)
+                }
+            }
+            .launchIn(this)
+    }
 }
diff --git a/apps/flipcash/features/advanced/src/main/kotlin/com/flipcash/app/advanced/internal/AdvancedFeatureMenuItems.kt b/apps/flipcash/features/advanced/src/main/kotlin/com/flipcash/app/advanced/internal/AdvancedFeatureMenuItems.kt
@@ -2,6 +2,7 @@ package com.flipcash.app.advanced.internal
 
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.outlined.Palette
+import androidx.compose.material.icons.outlined.Share
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.graphics.painter.Painter
 import androidx.compose.ui.graphics.vector.rememberVectorPainter
@@ -28,4 +29,12 @@ internal data object Deposit : FullMenuItem<AdvancedFeaturesScreenViewModel.Even
     override val action: AdvancedFeaturesScreenViewModel.Event = AdvancedFeaturesScreenViewModel.Event.OpenScreen(
         AppRoute.Sheets.TokenSelection(purpose = TokenPurpose.Deposit)
     )
+}
+
+internal data object ExportLogs : FullMenuItem<AdvancedFeaturesScreenViewModel.Event>() {
+    override val icon: Painter
+        @Composable get() = rememberVectorPainter(Icons.Outlined.Share)
+    override val name: String
+        @Composable get() = stringResource(R.string.title_exportLogs)
+    override val action: AdvancedFeaturesScreenViewModel.Event = AdvancedFeaturesScreenViewModel.Event.ExportLogs
 }
diff --git a/apps/flipcash/features/advanced/src/main/kotlin/com/flipcash/app/advanced/internal/AdvancedFeaturesScreenViewModel.kt b/apps/flipcash/features/advanced/src/main/kotlin/com/flipcash/app/advanced/internal/AdvancedFeaturesScreenViewModel.kt
@@ -20,6 +20,7 @@ import javax.inject.Inject
 private val FullMenuList = buildList {
     add(BillCustomizer)
     add(Deposit)
+    add(ExportLogs)
 }
 
 @HiltViewModel
@@ -44,6 +45,7 @@ internal class AdvancedFeaturesScreenViewModel @Inject constructor(
         data class OpenScreen(val screen: AppRoute) : Event
 
         data object OpenBillPlayground: Event
+        data object ExportLogs: Event
     }
 
     init {
@@ -83,6 +85,7 @@ internal class AdvancedFeaturesScreenViewModel @Inject constructor(
 
                 is Event.OpenScreen -> { state -> state }
                 is Event.OpenBillPlayground -> { state -> state }
+                is Event.ExportLogs -> { state -> state }
             }
         }
     }
diff --git a/apps/flipcash/shared/featureflags/src/main/kotlin/com/flipcash/app/featureflags/FeatureFlag.kt b/apps/flipcash/shared/featureflags/src/main/kotlin/com/flipcash/app/featureflags/FeatureFlag.kt
@@ -1,7 +1,6 @@
 package com.flipcash.app.featureflags
 
 import com.flipcash.app.ksp.annotations.FeatureFlagMarker
-import com.flipcash.shared.flags.BuildConfig
 
 sealed interface FeatureFlag {
     val key: String
@@ -73,15 +72,6 @@ sealed interface FeatureFlag {
         override val persistLogOut: Boolean = false
     }
 
-    @FeatureFlagMarker
-    data object ProductionLogging: FeatureFlag {
-        override val key: String = "production_logging_enabled"
-        override val default: Boolean = false
-        override val launched: Boolean = false
-        override val visible: Boolean = !BuildConfig.DEBUG
-        override val persistLogOut: Boolean = true
-    }
-
     @FeatureFlagMarker
     data object CashReservesEnabled: FeatureFlag {
         override val key: String = "cash_reserves_enabled"
@@ -156,7 +146,6 @@ val FeatureFlag.title: String
         FeatureFlag.Pools -> "Betting Pools"
         FeatureFlag.OnRamp -> "Onramp"
         FeatureFlag.BillCustomizer -> "Bill Customizer"
-        FeatureFlag.ProductionLogging -> "Production Logging"
         FeatureFlag.CashReservesEnabled -> "Cash Reserves"
         FeatureFlag.MarketCapChart -> "Market Cap Chart"
         FeatureFlag.CoinbaseOnRamp -> "Coinbase Onramp"
@@ -174,7 +163,6 @@ val FeatureFlag.message: String
         FeatureFlag.Pools -> "When enabled, you'll be able to participate in and create betting pools with other users for a chance to win a share of the prize"
         FeatureFlag.OnRamp -> "When enabled, you'll gain the ability to fund your wallet from external sources via providers using a debit card or via another wallet (like Phantom)"
         FeatureFlag.BillCustomizer -> "When enabled, you'll gain access to the bill customization playground"
-        FeatureFlag.ProductionLogging -> "When enabled, traces will print to log output"
         FeatureFlag.CashReservesEnabled -> "When enabled, USDC will be brandished as Cash Reserves throughout the app"
         FeatureFlag.MarketCapChart -> "When enabled, you'll gain access to the market cap chart in token info"
         FeatureFlag.CoinbaseOnRamp -> "When enabled, you'll gain access to the Coinbase onramp for token buys"
diff --git a/apps/flipcash/shared/featureflags/src/main/kotlin/com/flipcash/app/featureflags/internal/InternalFeatureFlagController.kt b/apps/flipcash/shared/featureflags/src/main/kotlin/com/flipcash/app/featureflags/internal/InternalFeatureFlagController.kt
@@ -10,15 +10,13 @@ import androidx.datastore.preferences.preferencesDataStoreFile
 import com.flipcash.app.featureflags.BetaFeature
 import com.flipcash.app.featureflags.FeatureFlag
 import com.flipcash.app.featureflags.FeatureFlagController
-import com.getcode.utils.TraceManager
 import dagger.hilt.android.qualifiers.ApplicationContext
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.SupervisorJob
 import kotlinx.coroutines.flow.SharingStarted
 import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.firstOrNull
-import kotlinx.coroutines.flow.launchIn
 import kotlinx.coroutines.flow.map
 import kotlinx.coroutines.flow.onEach
 import kotlinx.coroutines.flow.stateIn
@@ -51,10 +49,6 @@ internal class InternalFeatureFlagController @Inject constructor(
         FeatureFlag.entries
             .filter { it.launched }
             .onEach { reset(it) }
-
-        observe(FeatureFlag.ProductionLogging)
-            .onEach { TraceManager.enableProductionTraces(it) }
-            .launchIn(dataScope)
     }
 
     override fun enableBetaFeatures() {
diff --git a/libs/logging/build.gradle.kts b/libs/logging/build.gradle.kts
@@ -24,6 +24,9 @@ android {
 }
 
 dependencies {
+    testImplementation(kotlin("test"))
+    testImplementation(libs.robolectric)
+
     api(libs.timber)
     implementation(libs.bugsnag)
     implementation(libs.rxjava)
diff --git a/libs/logging/src/main/kotlin/com/getcode/utils/FileTree.kt b/libs/logging/src/main/kotlin/com/getcode/utils/FileTree.kt
@@ -0,0 +1,75 @@
+package com.getcode.utils
+
+import android.content.Context
+import android.util.Log
+import timber.log.Timber
+import java.io.File
+import java.io.FileOutputStream
+import java.io.OutputStreamWriter
+import java.time.Instant
+import java.time.ZoneId
+import java.time.format.DateTimeFormatter
+
+class FileTree(
+    context: Context,
+    private val plugins: () -> List<TraceLogPlugin>,
+) : Timber.Tree() {
+
+    private val traceDir = File(context.filesDir, "traces").apply { mkdirs() }
+    private val logFile = File(traceDir, "trace.log")
+    private val lock = Any()
+
+    private val formatter = DateTimeFormatter.ISO_LOCAL_DATE_TIME
+        .withZone(ZoneId.systemDefault())
+
+    init {
+        // Clear previous session logs on cold launch
+        traceDir.listFiles()?.forEach { it.delete() }
+    }
+
+    override fun log(priority: Int, tag: String?, message: String, t: Throwable?) {
+        val priorityLetter = when (priority) {
+            Log.VERBOSE -> "V"
+            Log.DEBUG -> "D"
+            Log.INFO -> "I"
+            Log.WARN -> "W"
+            Log.ERROR -> "E"
+            Log.ASSERT -> "A"
+            else -> "?"
+        }
+
+        val timestamp = formatter.format(Instant.now())
+        val tagPart = if (tag != null) " [$tag]" else ""
+
+        val baseLine = "[$timestamp] [$priorityLetter]$tagPart $message"
+        val stackTrace = t?.stackTraceToString()?.let { "\n$it" }.orEmpty()
+        val fullLine = "$baseLine$stackTrace"
+
+        // Run through plugin pipeline
+        var processed: String? = fullLine
+        for (plugin in plugins()) {
+            processed = plugin.process(processed ?: return)
+            if (processed == null) return
+        }
+
+        synchronized(lock) {
+            try {
+                FileOutputStream(logFile, true).use { fos ->
+                    OutputStreamWriter(fos, Charsets.UTF_8).use { writer ->
+                        writer.appendLine(processed)
+                    }
+                }
+            } catch (_: Exception) {
+                // Silently ignore file write failures to avoid infinite logging loops
+            }
+        }
+    }
+
+    fun getLogFile(): File? = if (logFile.exists() && logFile.length() > 0) logFile else null
+
+    fun clearLogs() {
+        synchronized(lock) {
+            logFile.delete()
+        }
+    }
+}
diff --git a/libs/logging/src/main/kotlin/com/getcode/utils/Logging.kt b/libs/logging/src/main/kotlin/com/getcode/utils/Logging.kt
@@ -1,12 +1,13 @@
 package com.getcode.utils
 
 import android.annotation.SuppressLint
+import android.content.Context
 import com.bugsnag.android.BreadcrumbType
 import com.bugsnag.android.Bugsnag
-import com.getcode.libs.logging.BuildConfig
 import com.google.firebase.crashlytics.CustomKeysAndValues
 import com.google.firebase.crashlytics.FirebaseCrashlytics
 import timber.log.Timber
+import java.io.File
 import kotlin.time.Duration
 import kotlin.time.TimeSource
 import kotlin.time.measureTime
@@ -68,30 +69,36 @@ private fun TraceType.toBugsnagBreadcrumbType(): BreadcrumbType? {
 }
 
 object TraceManager {
-    private val productionTraceTree = object: Timber.Tree() {
-        override fun log(
-            priority: Int,
-            tag: String?,
-            message: String,
-            t: Throwable?
-        ) {
-            println("[TRACE] $tag $message")
-            t?.printStackTrace()
-        }
+    private var fileTree: FileTree? = null
+    val plugins: MutableList<TraceLogPlugin> = mutableListOf()
 
+    fun addPlugin(plugin: TraceLogPlugin) {
+        plugins.add(plugin)
     }
-    fun enableProductionTraces(enable: Boolean) {
-        if (!BuildConfig.DEBUG) {
-            if (enable) {
-                if (!Timber.forest().contains(productionTraceTree)) {
-                    Timber.plant(productionTraceTree)
-                }
-            } else {
-                if (Timber.forest().contains(productionTraceTree)) {
-                    Timber.uproot(productionTraceTree)
-                }
-            }
-        }
+
+    fun removePlugin(plugin: TraceLogPlugin) {
+        plugins.remove(plugin)
+    }
+
+    fun initialize(context: Context) {
+        if (fileTree != null) return
+        val tree = FileTree(context, plugins = { plugins })
+        fileTree = tree
+        addPlugin(PiiMaskingPlugin())
+        Timber.plant(tree)
+    }
+
+    fun getLogFile(): File? = fileTree?.getLogFile()
+
+    fun clearLogs() {
+        fileTree?.clearLogs()
+    }
+
+    @androidx.annotation.VisibleForTesting
+    internal fun reset() {
+        fileTree?.let { Timber.uproot(it) }
+        fileTree = null
+        plugins.clear()
     }
 }
 
diff --git a/libs/logging/src/main/kotlin/com/getcode/utils/PiiMaskingPlugin.kt b/libs/logging/src/main/kotlin/com/getcode/utils/PiiMaskingPlugin.kt
diff --git a/libs/logging/src/main/kotlin/com/getcode/utils/TraceLogPlugin.kt b/libs/logging/src/main/kotlin/com/getcode/utils/TraceLogPlugin.kt
diff --git a/libs/logging/src/test/kotlin/com/getcode/utils/FileTreeTest.kt b/libs/logging/src/test/kotlin/com/getcode/utils/FileTreeTest.kt
diff --git a/libs/logging/src/test/kotlin/com/getcode/utils/PiiMaskingPluginTest.kt b/libs/logging/src/test/kotlin/com/getcode/utils/PiiMaskingPluginTest.kt
diff --git a/libs/logging/src/test/kotlin/com/getcode/utils/TraceManagerTest.kt b/libs/logging/src/test/kotlin/com/getcode/utils/TraceManagerTest.kt

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@ import javax.inject.Inject`
`20`	`20`	`private val FullMenuList = buildList {`
`21`	`21`	`add(BillCustomizer)`
`22`	`22`	`add(Deposit)`
	`23`	`+ add(ExportLogs)`
`23`	`24`	`}`
`24`	`25`
`25`	`26`	`@HiltViewModel`
`@@ -44,6 +45,7 @@ internal class AdvancedFeaturesScreenViewModel @Inject constructor(`
`44`	`45`	`data class OpenScreen(val screen: AppRoute) : Event`
`45`	`46`
`46`	`47`	`data object OpenBillPlayground: Event`
	`48`	`+ data object ExportLogs: Event`
`47`	`49`	`}`
`48`	`50`
`49`	`51`	`init {`
`@@ -83,6 +85,7 @@ internal class AdvancedFeaturesScreenViewModel @Inject constructor(`
`83`	`85`
`84`	`86`	`is Event.OpenScreen -> { state -> state }`
`85`	`87`	`is Event.OpenBillPlayground -> { state -> state }`
	`88`	`+ is Event.ExportLogs -> { state -> state }`
`86`	`89`	`}`
`87`	`90`	`}`
`88`	`91`	`}`