Respond to PR review:

- Use printError instead of println
- Revert formatting of Status.kt
- Change RefreshType to NONE

Author: spipau

src/main/kotlin/com/jetpackduba/gitnuro/extensions/FileExtensions.kt

@@ -1,9 +1,13 @@
 package com.jetpackduba.gitnuro.extensions
 
+import com.jetpackduba.gitnuro.logging.printError
+import com.jetpackduba.gitnuro.system.OS
+import com.jetpackduba.gitnuro.system.currentOs
 import kotlinx.io.IOException
 import java.awt.Desktop
 import java.io.File
-import java.util.*
+
+private const val TAG = "FileExtensions"
 
 fun File.openDirectory(dirName: String): File {
     val newDir = File(this, dirName)
@@ -18,7 +22,7 @@ fun File.openDirectory(dirName: String): File {
 fun File.openFileInFolder() {
 
     if (!exists() || !isDirectory) {
-        println("Folder with path $path does not exist or is not a folder")
+        printError(TAG, "Folder with path $path does not exist or is not a folder")
         return
     }
 
@@ -31,25 +35,25 @@ fun File.openFileInFolder() {
             }
         }
     } catch (e: Exception) {
-        println("Desktop API failed: ${e.message}")
+        printError(TAG, "Desktop API failed: ${e.message}")
     }
 
     // Fallback
-    val os = System.getProperty("os.name").lowercase(Locale.getDefault())
-    val command = when {
-        os.contains("linux") -> listOf("xdg-open", absolutePath)
-        os.contains("mac") -> listOf("open", absolutePath)
-        os.contains("windows") -> listOf("explorer", absolutePath)
+    val os = currentOs
+    val command = when (os) {
+        OS.LINUX -> listOf("xdg-open", absolutePath)
+        OS.MAC -> listOf("open", absolutePath)
+        OS.WINDOWS -> listOf("explorer", absolutePath)
         else -> null
     }
 
     if (command != null) {
         try {
             ProcessBuilder(command).start()
         } catch (ex: IOException) {
-            println("Failed to open file explorer: ${ex.message}")
+            printError(TAG, "Failed to open file explorer: ${ex.message}")
         }
     } else {
-        println("Unsupported OS: $os")
+        printError(TAG, "Unsupported OS: $os")
     }
 }

src/main/kotlin/com/jetpackduba/gitnuro/git/workspace/Status.kt

@@ -6,10 +6,7 @@ import androidx.compose.ui.graphics.vector.ImageVector
 import com.jetpackduba.gitnuro.extensions.icon
 import com.jetpackduba.gitnuro.extensions.iconColor
 
-data class StatusEntry(
-    val filePath: String,
-    val statusType: StatusType
-) {
+data class StatusEntry(val filePath: String, val statusType: StatusType) {
     val icon: ImageVector
         get() = statusType.icon
 

src/main/kotlin/com/jetpackduba/gitnuro/viewmodels/CommitChangesViewModel.kt

@@ -158,7 +158,7 @@ class CommitChangesViewModel @Inject constructor(
     }
 
     fun openFileInFolder(folderPath: String?) = tabState.runOperation(
-        refreshType = RefreshType.UNCOMMITTED_CHANGES,
+        refreshType = RefreshType.NONE,
     ) {
         folderPath?.let { File(it).openFileInFolder() }
     }

src/main/kotlin/com/jetpackduba/gitnuro/viewmodels/StatusViewModel.kt

@@ -513,7 +513,7 @@ class StatusViewModel @Inject constructor(
     }
 
     fun openFileInFolder(folderPath: String?) = tabState.runOperation(
-        refreshType = RefreshType.UNCOMMITTED_CHANGES,
+        refreshType = RefreshType.NONE,
     ) {
         folderPath?.let { File(it).openFileInFolder() }
     }