chore(logging): add header to exported log with device info; filter out RPC Request/Responses

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>

Author: bmc08gt

apps/flipcash/features/onramp/src/main/kotlin/com/flipcash/app/onramp/internal/OnRampViewModel.kt

@@ -341,7 +341,7 @@ internal class OnRampViewModel @Inject constructor(
                 val token = stateFlow.value.token
                     ?: return@flatMapResult Result.failure(IllegalStateException("No token"))
                 val amount = stateFlow.value.amountEntryState.selectedAmount
-                println("attempting buy of $amount of ${token.symbol} with $txHash")
+
                 transactionController.buy(
                     owner = owner,
                     amount = amount,

apps/flipcash/shared/onramp/coinbase/src/main/kotlin/com/flipcash/app/onramp/CoinbaseOnRampWebview.kt

@@ -3,17 +3,15 @@ package com.flipcash.app.onramp
 import android.annotation.SuppressLint
 import android.webkit.JavascriptInterface
 import android.webkit.WebChromeClient
-import android.webkit.WebSettings
 import android.webkit.WebResourceError
 import android.webkit.WebResourceRequest
+import android.webkit.WebSettings
 import android.webkit.WebView
 import android.webkit.WebViewClient
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.runtime.Composable
-import androidx.compose.runtime.remember
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.draw.alpha
-import androidx.compose.ui.viewinterop.AndroidView
 import androidx.webkit.WebSettingsCompat
 import androidx.webkit.WebViewCompat
 import androidx.webkit.WebViewFeature

apps/flipcash/shared/onramp/coinbase/src/main/kotlin/com/flipcash/app/onramp/OnRampController.kt

@@ -20,7 +20,6 @@ import com.getcode.opencode.model.financial.CurrencyCode
 import com.getcode.opencode.model.financial.Fiat
 import com.getcode.opencode.model.financial.Token
 import com.getcode.opencode.model.financial.usdf
-import com.getcode.solana.keys.Mint
 import com.getcode.solana.keys.base58
 import com.getcode.utils.base64
 import kotlinx.serialization.Serializable

libs/logging/src/main/kotlin/com/getcode/utils/FileTree.kt

@@ -1,14 +1,19 @@
 package com.getcode.utils
 
 import android.content.Context
+import android.content.pm.PackageManager
+import android.os.Build
 import android.util.Log
+import com.bugsnag.android.Bugsnag
 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
+import java.util.Locale
+import java.util.TimeZone
 
 class FileTree(
     context: Context,
@@ -17,7 +22,9 @@ class FileTree(
 
     private val traceDir = File(context.filesDir, "traces").apply { mkdirs() }
     private val logFile = File(traceDir, "trace.log")
+    private val exportFile = File(traceDir, "trace_export.log")
     private val lock = Any()
+    private val appContext: Context = context.applicationContext
 
     private val formatter = DateTimeFormatter.ISO_LOCAL_DATE_TIME
         .withZone(ZoneId.systemDefault())
@@ -65,11 +72,58 @@ class FileTree(
         }
     }
 
-    fun getLogFile(): File? = if (logFile.exists() && logFile.length() > 0) logFile else null
+    fun getLogFile(): File? {
+        if (!logFile.exists() || logFile.length() == 0L) return null
+        synchronized(lock) {
+            exportFile.delete()
+            exportFile.writeText(buildDeviceHeader(appContext))
+            logFile.inputStream().use { input ->
+                FileOutputStream(exportFile, true).use { output ->
+                    input.copyTo(output)
+                }
+            }
+        }
+        return exportFile
+    }
 
     fun clearLogs() {
         synchronized(lock) {
             logFile.delete()
+            exportFile.delete()
         }
     }
 }
+
+@Suppress("DEPRECATION")
+private fun buildDeviceHeader(context: Context): String {
+    val packageInfo = try {
+        context.packageManager.getPackageInfo(context.packageName, 0)
+    } catch (_: PackageManager.NameNotFoundException) {
+        null
+    }
+
+    val versionName = packageInfo?.versionName ?: "unknown"
+    val versionCode = packageInfo?.let {
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) it.longVersionCode else it.versionCode.toLong()
+    } ?: -1
+
+    val userId = if (Bugsnag.isStarted()) Bugsnag.getUser().id else null
+
+    return buildString {
+        appendLine("=".repeat(60))
+        appendLine("DEVICE & APP INFO")
+        appendLine("=".repeat(60))
+        appendLine("App Version:    $versionName ($versionCode)")
+        appendLine("Package:        ${context.packageName}")
+        appendLine("User ID:        ${userId ?: "not set"}")
+        appendLine("Device:         ${Build.MANUFACTURER} ${Build.MODEL}")
+        appendLine("Android:        ${Build.VERSION.RELEASE} (API ${Build.VERSION.SDK_INT})")
+        appendLine("Build:          ${Build.DISPLAY}")
+        appendLine("ABI:            ${Build.SUPPORTED_ABIS.joinToString()}")
+        appendLine("Locale:         ${Locale.getDefault()}")
+        appendLine("Timezone:       ${TimeZone.getDefault().id}")
+        appendLine("Exported:       ${Instant.now()}")
+        appendLine("=".repeat(60))
+        appendLine()
+    }
+}

libs/logging/src/main/kotlin/com/getcode/utils/Logging.kt

@@ -85,6 +85,7 @@ object TraceManager {
         val tree = FileTree(context, plugins = { plugins })
         fileTree = tree
         addPlugin(PiiMaskingPlugin())
+        addPlugin(RpcBodyFilterPlugin())
         Timber.plant(tree)
     }
 

libs/logging/src/main/kotlin/com/getcode/utils/RpcBodyFilterPlugin.kt

@@ -0,0 +1,9 @@
+package com.getcode.utils
+
+class RpcBodyFilterPlugin : TraceLogPlugin {
+    override fun process(line: String): String? {
+        if ("[RpcLogging]" !in line) return line
+        if ("Request:" in line || "Response:" in line) return null
+        return line
+    }
+}

libs/logging/src/test/kotlin/com/getcode/utils/FileTreeTest.kt

@@ -82,8 +82,9 @@ class FileTreeTest {
         Timber.d("second")
         Timber.d("third")
 
-        val lines = fileTree.getLogFile()!!.readLines().filter { it.isNotBlank() }
-        assertEquals(3, lines.size)
+        val content = fileTree.getLogFile()!!.readText()
+        val logLines = content.lines().filter { it.startsWith("[") }
+        assertEquals(3, logLines.size)
     }
 
     @Test
@@ -200,8 +201,9 @@ class FileTreeTest {
         latch.await()
         executor.shutdown()
 
-        val lines = fileTree.getLogFile()!!.readLines().filter { it.isNotBlank() }
-        assertEquals(threadCount * linesPerThread, lines.size)
+        val content = fileTree.getLogFile()!!.readText()
+        val logLines = content.lines().filter { it.startsWith("[") }
+        assertEquals(threadCount * linesPerThread, logLines.size)
     }
 
     @Test
@@ -237,16 +239,28 @@ class FileTreeTest {
         Timber.d("contains SECRET data")
         Timber.d("another normal message")
 
-        val lines = tree.getLogFile()!!.readLines().filter { it.isNotBlank() }
-        assertEquals(2, lines.size)
-        assertTrue(lines.all { !it.contains("SECRET") })
+        val content = tree.getLogFile()!!.readText()
+        val logLines = content.lines().filter { it.startsWith("[") }
+        assertEquals(2, logLines.size)
+        assertTrue(logLines.all { !it.contains("SECRET") })
     }
 
     @Test
     fun `log file path is inside traces directory`() {
         Timber.d("path check")
         val file = fileTree.getLogFile()!!
-        assertEquals("trace.log", file.name)
+        assertEquals("trace_export.log", file.name)
         assertEquals("traces", file.parentFile?.name)
     }
+
+    @Test
+    fun `exported log file includes device header`() {
+        Timber.d("header check")
+        val content = fileTree.getLogFile()!!.readText()
+        assertTrue(content.contains("DEVICE & APP INFO"))
+        assertTrue(content.contains("App Version:"))
+        assertTrue(content.contains("Device:"))
+        assertTrue(content.contains("Android:"))
+        assertTrue(content.contains("Exported:"))
+    }
 }