Merge pull request #5 from DevelopersBreach/shreyas/kermit-logger

Add Kermit logging library and replace println statements

Author: yesshreyes

composeApp/build.gradle.kts

@@ -98,6 +98,7 @@ kotlin {
             api(libs.koin.core)
             implementation(libs.koin.compose)
             implementation(libs.koin.compose.viewmodel)
+            implementation(libs.kermit)
         }
         desktopMain.dependencies {
             implementation(compose.desktop.currentOs)

composeApp/src/commonMain/kotlin/com/developersbreach/kotlindictionarymultiplatform/Log.kt

@@ -0,0 +1,34 @@
+package com.developersbreach.kotlindictionarymultiplatform
+
+import co.touchlab.kermit.Logger
+
+object Log {
+    fun d(
+        tag: String,
+        message: String,
+    ) {
+        Logger.d(message, tag = tag)
+    }
+
+    fun i(
+        tag: String,
+        message: String,
+    ) {
+        Logger.i(message, tag = tag)
+    }
+
+    fun w(
+        tag: String,
+        message: String,
+    ) {
+        Logger.w(message, tag = tag)
+    }
+
+    fun e(
+        tag: String,
+        message: String,
+        throwable: Throwable? = null,
+    ) {
+        Logger.e(message, throwable, tag = tag)
+    }
+}

composeApp/src/commonMain/kotlin/com/developersbreach/kotlindictionarymultiplatform/core/KtorHttpClient.kt

@@ -1,5 +1,6 @@
 package com.developersbreach.kotlindictionarymultiplatform.core
 
+import com.developersbreach.kotlindictionarymultiplatform.Log
 import com.developersbreach.kotlindictionarymultiplatform.data.detail.ChatCompletionRequest
 import com.developersbreach.kotlindictionarymultiplatform.data.detail.ChatCompletionResponse
 import com.developersbreach.kotlindictionarymultiplatform.data.detail.ChatMessage
@@ -115,11 +116,11 @@ object KtorHttpClient {
         }
 
         val text = response.bodyAsText()
-        println("RAW RESPONSE:\n$text")
+        Log.i("RawResponse", "RAW RESPONSE:\n$text")
 
         // Parse response
         val chatResp = json.decodeFromString(ChatCompletionResponse.serializer(), text)
-        println(chatResp)
+        Log.i("ChatResponse", "$chatResp")
         val funcCall = chatResp.choices?.first()?.message?.functionCall ?: error("No function call in response")
 
         // The arguments field is a JSON string: parse and decode into our DTO

composeApp/src/commonMain/kotlin/com/developersbreach/kotlindictionarymultiplatform/ui/screens/TopicListScreen.kt

@@ -20,10 +20,8 @@ import androidx.compose.foundation.lazy.items
 @Composable
 fun DetailScreen(viewModel: DetailViewModel) {
     val topicState by viewModel.state.collectAsState()
-    println("check topicState: $topicState")
 
     topicState?.let { topic ->
-        println("Topic data fetched: $topic")
         LazyColumn(modifier = Modifier.padding(16.dp)) {
             item {
                 Text(text = topic.topicName, style = MaterialTheme.typography.h5)

composeApp/src/commonMain/kotlin/com/developersbreach/kotlindictionarymultiplatform/ui/screens/detail/DetailScreen.kt

@@ -4,6 +4,7 @@ import androidx.lifecycle.SavedStateHandle
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import androidx.navigation.toRoute
+import com.developersbreach.kotlindictionarymultiplatform.Log
 import com.developersbreach.kotlindictionarymultiplatform.data.detail.KotlinTopicDetails
 import com.developersbreach.kotlindictionarymultiplatform.core.KtorHttpClient
 import kotlinx.coroutines.flow.MutableStateFlow
@@ -29,11 +30,11 @@ class DetailViewModel(
         viewModelScope.launch {
             try {
                 val topic = KtorHttpClient.generateTopicDetails(topicId, API_KEY)
-                println("Fetched details: $topic")
+                Log.i("DetailViewModel", "Fetched details: $topic")
                 _state.value = topic
-                println("State updated with topic: $topic")
+                Log.i("DetailViewModel", "State updated with topic: $topic")
             } catch (e: Exception) {
-                println("Error fetching topic: ${e.message}")
+                Log.e("DetailViewModel", "Error fetching topic: ${e.message}", e)
             }
         }
     }

composeApp/src/commonMain/kotlin/com/developersbreach/kotlindictionarymultiplatform/ui/screens/detail/DetailViewModel.kt

@@ -14,6 +14,7 @@ import androidx.compose.runtime.collectAsState
 import androidx.compose.runtime.getValue
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.unit.dp
+import com.developersbreach.kotlindictionarymultiplatform.Log
 import com.developersbreach.kotlindictionarymultiplatform.data.topic.model.Topic
 
 @Composable
@@ -30,7 +31,7 @@ fun TopicScreen(
     ) {
         items(topics) { topic ->
             TopicItem(topic = topic, onClick = {
-                println("Topic clicked: ${topic.name}")
+                Log.i("TopicScreen", "Topic clicked: ${topic.name}")
                 onTopicClick(topic.name)
             })
         }
@@ -42,7 +43,7 @@ fun TopicItem(
     topic: Topic,
     onClick: () -> Unit,
 ) {
-    println("Rendering topic item: ${topic.name}")
+    Log.i("TopicItem", "Rendering topic item: ${topic.name}")
 
     Card(
         elevation = 4.dp,

composeApp/src/commonMain/kotlin/com/developersbreach/kotlindictionarymultiplatform/ui/screens/topic/TopicScreen.kt

@@ -2,6 +2,7 @@ package com.developersbreach.kotlindictionarymultiplatform.ui.screens.topic
 
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
+import com.developersbreach.kotlindictionarymultiplatform.Log
 import com.developersbreach.kotlindictionarymultiplatform.data.topic.model.Topic
 import com.developersbreach.kotlindictionarymultiplatform.data.topic.repository.TopicRepository
 import kotlinx.coroutines.flow.MutableStateFlow
@@ -21,11 +22,10 @@ class TopicViewModel : ViewModel() {
         viewModelScope.launch {
             try {
                 val topics = TopicRepository.getTopics()
-                println("Successfully fetched topics: $topics")
+                Log.i("TopicFetch", "Successfully fetched topics: $topics")
                 _topics.value = topics
             } catch (e: Exception) {
-                println("Error fetching topics: ${e.message}")
-                e.printStackTrace()
+                Log.e("TopicFetch", "Error fetching topics: ${e.message}", e)
             }
         }
     }

composeApp/src/commonMain/kotlin/com/developersbreach/kotlindictionarymultiplatform/ui/screens/topic/TopicViewModel.kt

@@ -13,6 +13,7 @@ androidx-material = "1.12.0"
 androidx-test-junit = "1.2.1"
 compose-multiplatform = "1.7.3"
 junit = "4.13.2"
+kermit = "2.0.4"
 kotlin = "2.1.10"
 kotlinx-coroutines = "1.10.1"
 kotlinx-serialization-json = "1.7.3"
@@ -23,6 +24,7 @@ ktlint = "12.2.0"
 
 [libraries]
 androidx-navigation-compose = { module = "androidx.navigation:navigation-compose", version.ref = "navigation-compose" }
+kermit = { module = "co.touchlab:kermit", version.ref = "kermit" }
 kotlin-test = { module = "org.jetbrains.kotlin:kotlin-test", version.ref = "kotlin" }
 kotlin-test-junit = { module = "org.jetbrains.kotlin:kotlin-test-junit", version.ref = "kotlin" }
 junit = { group = "junit", name = "junit", version.ref = "junit" }