Fix some logging issues (#242)

* Do not modify global Kermit severity

* Add 'PowerSync' tag for the logs

* Get rid of generateLogger()

* Fix lint issues

* Apply format

* Revert "Get rid of generateLogger()"

This reverts commit 826e7a3.

* Update core/src/commonMain/kotlin/com/powersync/utils/Log.kt

---------

Co-authored-by: Simon Binder <simon@journeyapps.com>
Co-authored-by: stevensJourney <51082125+stevensJourney@users.noreply.github.com>

Author: 3 people

CHANGELOG.md

@@ -4,6 +4,8 @@
 
 * Add `PowerSyncDatabase.getCrudTransactions()`, returning a flow of transactions. This is useful
   to upload multiple transactions in a batch.
+* Fix modifying severity of the global Kermit logger
+* Add `PowerSync` tag for the logs
 
 ## 1.4.0
 

core/src/commonMain/kotlin/com/powersync/utils/Log.kt

@@ -1,24 +1,28 @@
 package com.powersync.utils
 
+import BuildConfig
 import co.touchlab.kermit.Logger
 import co.touchlab.kermit.Severity
+import co.touchlab.kermit.StaticConfig
+import co.touchlab.kermit.platformLogWriter
 
 /*
  * Generates a logger with the appropriate severity level based on the build type
  * if no Logger is provided.
 */
-public fun generateLogger(logger: Logger?): Logger {
-    if (logger != null) {
-        return logger
-    }
-
-    val defaultLogger: Logger = Logger
-
-    if (BuildConfig.isDebug) {
-        Logger.setMinSeverity(Severity.Verbose)
-    } else {
-        Logger.setMinSeverity(Severity.Warn)
-    }
-
-    return defaultLogger
-}
+public fun generateLogger(logger: Logger?): Logger =
+    logger
+        ?: Logger(
+            config =
+                StaticConfig(
+                    logWriterList =
+                        listOf(platformLogWriter()),
+                    minSeverity =
+                        if (BuildConfig.isDebug) {
+                            Severity.Verbose
+                        } else {
+                            Severity.Warn
+                        },
+                ),
+            tag = "PowerSync",
+        )