Revert "Get rid of generateLogger()"

This reverts commit 826e7a3.

Author: Radiokot

CHANGELOG.md

@@ -4,7 +4,6 @@
 
 * Fixed modifying severity of the global Kermit logger
 * Added `PowerSync` tag for the logs
-* Removed `generateLogger()` global public method meant to be used internally
 
 ## 1.4.0
 

core/src/commonMain/kotlin/com/powersync/PowerSyncDatabaseFactory.kt

@@ -1,13 +1,10 @@
 package com.powersync
 
-import BuildConfig
 import co.touchlab.kermit.Logger
-import co.touchlab.kermit.Severity
-import co.touchlab.kermit.StaticConfig
-import co.touchlab.kermit.platformLogWriter
 import co.touchlab.skie.configuration.annotations.DefaultArgumentInterop
 import com.powersync.db.PowerSyncDatabaseImpl
 import com.powersync.db.schema.Schema
+import com.powersync.utils.generateLogger
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.DelicateCoroutinesApi
 import kotlinx.coroutines.GlobalScope
@@ -30,30 +27,18 @@ public fun PowerSyncDatabase(
      * This parameter is ignored for iOS.
      */
     dbDirectory: String? = null,
-): PowerSyncDatabase =
-    createPowerSyncDatabaseImpl(
+): PowerSyncDatabase {
+    val generatedLogger: Logger = generateLogger(logger)
+
+    return createPowerSyncDatabaseImpl(
         schema = schema,
         factory = factory,
         dbFilename = dbFilename,
         scope = scope,
-        logger =
-            logger
-                ?: Logger(
-                    config =
-                        StaticConfig(
-                            logWriterList =
-                                listOf(platformLogWriter()),
-                            minSeverity =
-                                if (BuildConfig.isDebug) {
-                                    Severity.Verbose
-                                } else {
-                                    Severity.Warn
-                                },
-                        ),
-                    tag = "PowerSync",
-                ),
+        logger = generatedLogger,
         dbDirectory = dbDirectory,
     )
+}
 
 internal fun createPowerSyncDatabaseImpl(
     factory: DatabaseDriverFactory,

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

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