refactor: update logging and dependency injection configuration

- Move logging writers to core dependency injection module
- Remove unused logging module and environment parameters from initialization
- Integrate log writer registration into crash reporting lifecycle
- Update exception handling and stack trace formatting in log writers
- Simplify application entry point by removing environment-specific configuration

Author: Coding-Meet

composeApp/src/jvmMain/kotlin/com/meet/dev/analyzer/di/CoreModule.kt

@@ -5,6 +5,8 @@ import androidx.datastore.preferences.core.PreferenceDataStoreFactory
 import androidx.datastore.preferences.core.Preferences
 import com.meet.dev.analyzer.data.datastore.AppPreferenceManager
 import com.meet.dev.analyzer.data.datastore.PathPreferenceManger
+import com.meet.dev.analyzer.utility.crash_report.FileLogWriter
+import com.meet.dev.analyzer.utility.crash_report.SentryLogWriter
 import org.koin.core.module.dsl.singleOf
 import org.koin.dsl.module
 import java.io.File
@@ -22,4 +24,7 @@ val coreModule = module {
     singleOf(::AppPreferenceManager)
     singleOf(::PathPreferenceManger)
 
+    // logging
+    single<SentryLogWriter> { SentryLogWriter() }
+    single<FileLogWriter> { FileLogWriter() }
 }

composeApp/src/jvmMain/kotlin/com/meet/dev/analyzer/di/LoggingModule.kt

@@ -10,6 +10,7 @@ import com.meet.dev.analyzer.utility.platform.AppEnvironment
 import org.koin.core.module.Module
 import org.koin.dsl.module
 
+// this method not use. it not need it SentryLogWriter and FileLogWriter move core module, and logger object is not need it to koin
 fun getLoggingModule(
     appEnvironment: AppEnvironment,
 ): Module =

composeApp/src/jvmMain/kotlin/com/meet/dev/analyzer/di/initKoin.kt

@@ -1,20 +1,15 @@
 package com.meet.dev.analyzer.di
 
-import com.meet.dev.analyzer.utility.platform.AppEnvironment
 import org.koin.core.context.startKoin
 import org.koin.dsl.KoinAppDeclaration
 
 fun initKoin(
-    appEnvironment: AppEnvironment,
     appDeclaration: KoinAppDeclaration = {}
 ) =
     startKoin {
         appDeclaration()
         modules(
             coreModule,
-            getLoggingModule(
-                appEnvironment = appEnvironment,
-            ),
             repositoryModule,
             viewModule
         )

composeApp/src/jvmMain/kotlin/com/meet/dev/analyzer/main.kt

@@ -35,9 +35,7 @@ import java.awt.Toolkit
 fun main() {
     val properties = CustomProperties.loadProperties()
     val appConfig = CustomProperties.createAppConfig(properties)
-    initKoin(
-        appEnvironment = appConfig.appEnvironment,
-    ) {}
+    initKoin()
     FileKit.init(appId = "DevAnalyzer")
     System.setProperty("apple.awt.application.appearance", "system")
     application {

composeApp/src/jvmMain/kotlin/com/meet/dev/analyzer/utility/crash_report/FileLogWriter.kt

@@ -51,7 +51,7 @@ class FileLogWriter(
 
     override fun log(severity: Severity, message: String, tag: String, throwable: Throwable?) {
         if (severity < minSeverity) return
-        val stack = throwable?.stackTraceToString() ?: return
+        val stack = throwable?.stackTraceToString() ?: ""
 
         val file = getOrCreateLogFile()
 

composeApp/src/jvmMain/kotlin/com/meet/dev/analyzer/utility/crash_report/Sentry.kt

@@ -1,12 +1,18 @@
 package com.meet.dev.analyzer.utility.crash_report
 
+import co.touchlab.kermit.Logger
 import io.sentry.Sentry
 import io.sentry.SentryLevel
+import org.koin.mp.KoinPlatform.getKoin
 
 fun initSentry(
     dns: String,
     version: String,
 ) {
+    val sentryLogWriter = getKoin().get<SentryLogWriter>()
+    if (Logger.config.logWriterList.none { it is FileLogWriter }) {
+        Logger.addLogWriter(sentryLogWriter)
+    }
     Sentry.init { options ->
         options.dsn = dns
 
@@ -20,6 +26,10 @@ fun initSentry(
 
 fun disableSentry() {
     Sentry.close()
+    val sentryLogWriter = getKoin().get<SentryLogWriter>()
+    Logger.setLogWriters(
+        Logger.config.logWriterList.filterNot { w -> w == sentryLogWriter }
+    )
 }
 
 

composeApp/src/jvmMain/kotlin/com/meet/dev/analyzer/utility/crash_report/SentryLogWriter.kt

@@ -48,7 +48,7 @@ class SentryLogWriter(
 
         if (shouldCaptureException) {
             // Avoid duplicate Sentry events for throwable logs.
-            throwable?.let { captureException(it) }
+            captureException(throwable)
             return
         }