refactor(logging): make logger instances private and use the logger<T> method to retrieve them. #51

Author: phodal

exts/database/src/main/kotlin/cc/unitmesh/database/actions/GenerateFunctionAction.kt

@@ -19,7 +19,7 @@ class GenerateFunctionAction : ChatBaseIntention() {
 
     override fun getText(): String = AutoDevBundle.message("migration.database.plsql.generate.function")
 
-    val logger = logger<GenerateFunctionAction>()
+    private val logger = logger<GenerateFunctionAction>()
 
     override fun isAvailable(project: Project, editor: Editor?, file: PsiFile?): Boolean {
         if (editor == null || file == null) return false

exts/database/src/main/kotlin/cc/unitmesh/database/actions/GenerateUnittestAction.kt

@@ -24,7 +24,7 @@ class GenerateUnittestAction : ChatBaseIntention() {
         return file.language is OraDialect
     }
 
-    val logger = logger<GenerateUnittestAction>()
+    private val logger = logger<GenerateUnittestAction>()
 
     override fun invoke(project: Project, editor: Editor?, file: PsiFile?) {
         if (editor == null || file == null) return

java/src/main/kotlin/cc/unitmesh/idea/actions/AutoCrudAction.kt

@@ -16,6 +16,8 @@ import com.intellij.openapi.util.NlsSafe
 import com.intellij.psi.PsiFile
 
 class AutoCrudAction : ChatBaseIntention() {
+    private val logger = logger<AutoCrudAction>()
+
     override fun priority(): Int = 900
 
     override fun getText(): String = AutoDevBundle.message("intentions.crud.new.name")
@@ -75,7 +77,4 @@ class AutoCrudAction : ChatBaseIntention() {
         }
 
 
-    companion object {
-        val logger = logger<AutoCrudAction>()
-    }
 }

java/src/main/kotlin/cc/unitmesh/idea/context/JavaContextCollection.kt

@@ -11,7 +11,7 @@ import com.intellij.psi.search.searches.MethodReferencesSearch
 import com.intellij.psi.search.searches.ReferencesSearch
 
 object JavaContextCollection {
-    val logger = logger<JavaContextCollection>()
+    private val logger = logger<JavaContextCollection>()
     fun findUsages(nameIdentifierOwner: PsiNameIdentifierOwner): List<PsiReference> {
         val project = nameIdentifierOwner.project
         val searchScope = GlobalSearchScope.allScope(project) as SearchScope

java/src/main/kotlin/cc/unitmesh/idea/prompting/JavaContextPrompter.kt

@@ -21,6 +21,7 @@ import com.intellij.temporary.similar.chunks.SimilarChunksWithPaths
 import kotlinx.coroutines.runBlocking
 
 open class JavaContextPrompter : ContextPrompter() {
+    private val logger = logger<JavaContextPrompter>()
     private var additionContext: String = ""
     protected open val testDataBuilder: TestDataBuilder = JavaTestDataBuilder()
 
@@ -167,7 +168,4 @@ open class JavaContextPrompter : ContextPrompter() {
         }
     }
 
-    companion object {
-        val logger = logger<JavaContextPrompter>()
-    }
 }

src/main/kotlin/cc/unitmesh/devti/actions/console/FixThisAction.kt

@@ -12,7 +12,7 @@ import com.intellij.temporary.error.ErrorMessageProcessor
 
 class FixThisAction : ChatBaseAction() {
     override fun getActionType(): ChatActionType = ChatActionType.FIX_ISSUE
-    val logger = logger<FixThisAction>()
+    private val logger = logger<FixThisAction>()
 
     override fun actionPerformed(event: AnActionEvent) {
         val project = event.project ?: return

src/main/kotlin/cc/unitmesh/devti/actions/vcs/CommitMessageSuggestionAction.kt

@@ -32,7 +32,7 @@ class CommitMessageSuggestionAction : ChatBaseAction() {
         return ActionUpdateThread.BGT
     }
 
-    val logger = logger<CommitMessageSuggestionAction>()
+    private val logger = logger<CommitMessageSuggestionAction>()
 
     override fun getActionType(): ChatActionType = ChatActionType.GEN_COMMIT_MESSAGE
 

src/main/kotlin/cc/unitmesh/devti/actions/vcs/ReleaseNoteSuggestionAction.kt

@@ -13,10 +13,6 @@ import com.intellij.vcs.log.VcsLogDataKeys
 
 
 class ReleaseNoteSuggestionAction : AnAction() {
-    companion object {
-        val logger = logger<ReleaseNoteSuggestionAction>()
-    }
-
     override fun actionPerformed(e: AnActionEvent) {
         val project = e.project
         val vcsLog = e.getData(VcsLogDataKeys.VCS_LOG)

src/main/kotlin/cc/unitmesh/devti/custom/document/CustomLivingDocTask.kt

@@ -16,10 +16,7 @@ class CustomLivingDocTask(
     val config: CustomDocumentationConfig,
 ) :
     Task.Backgroundable(editor.project, config.title) {
-
-    companion object {
-        val logger = logger<CustomLivingDocTask>()
-    }
+    private val logger = logger<CustomLivingDocTask>()
 
     override fun run(indicator: ProgressIndicator) {
         val documentation = LivingDocumentation.forLanguage(target.language) ?: return

src/main/kotlin/cc/unitmesh/devti/custom/team/CustomActionPrompt.kt

@@ -19,7 +19,7 @@ data class CustomActionPrompt(
     var msgs: List<LlmMsg.ChatMessage> = listOf(),
 ) {
     companion object {
-        val logger = logger<CustomActionPrompt>()
+        private val logger = logger<CustomActionPrompt>()
 
         /**
          * Parses the given content and returns a TeamActionPrompt object.

src/main/kotlin/cc/unitmesh/devti/gui/chat/ChatCodingPanel.kt

@@ -8,6 +8,7 @@ import cc.unitmesh.devti.provider.ContextPrompter
 import cc.unitmesh.devti.settings.AutoDevSettingsState
 import com.intellij.ide.BrowserUtil
 import com.intellij.openapi.Disposable
+import com.intellij.openapi.diagnostic.logger
 import com.intellij.openapi.ui.DialogPanel
 import com.intellij.openapi.ui.NullableComponent
 import com.intellij.openapi.ui.SimpleToolWindowPanel
@@ -196,14 +197,17 @@ class ChatCodingPanel(private val chatCodingService: ChatCodingService, val disp
         updateUI()
     }
 
+    private val logger = logger<ChatCodingPanel>()
+
     private suspend fun updateMessageInUi(content: Flow<String>): String {
         val messageView = MessageView("", ChatRole.Assistant, "")
         myList.add(messageView)
         val startTime = System.currentTimeMillis() // 记录代码开始执行的时间
 
         var text = ""
         content.onCompletion {
-            println("onCompletion ${it?.message}")
+//            println("onCompletion ${it?.message}")
+            logger.info("onCompletion ${it?.message}")
         }.catch {
             it.printStackTrace()
         }.collect {

src/main/kotlin/cc/unitmesh/devti/gui/toolbar/NewChatAction.kt

@@ -15,7 +15,7 @@ import javax.swing.JButton
 import javax.swing.JComponent
 
 class NewChatAction : DumbAwareAction(), CustomComponentAction {
-    val logger = logger<NewChatAction>()
+    private val logger = logger<NewChatAction>()
 
     override fun actionPerformed(e: AnActionEvent) {
 

src/main/kotlin/cc/unitmesh/devti/intentions/action/CodeCompletionBaseIntention.kt

@@ -63,6 +63,6 @@ class CodeCompletionBaseIntention : ChatBaseIntention() {
     }
 
     companion object {
-        val logger = logger<CodeCompletionBaseIntention>()
+        private val logger = logger<CodeCompletionBaseIntention>()
     }
 }

src/main/kotlin/cc/unitmesh/devti/intentions/action/base/BasedDocumentationBaseIntention.kt

@@ -21,7 +21,7 @@ abstract class BasedDocumentationBaseIntention : ChatBaseIntention() {
 
     override fun startInWriteAction(): Boolean = false
 
-    val logger = logger<BasedDocumentationBaseIntention>()
+    private val logger = logger<BasedDocumentationBaseIntention>()
 
     override fun invoke(project: Project, editor: Editor?, file: PsiFile?) {
         if (editor == null || file == null) return

src/main/kotlin/cc/unitmesh/devti/intentions/action/task/BaseCompletionTask.kt

@@ -90,6 +90,6 @@ abstract class BaseCompletionTask(private val request: CodeCompletionRequest) :
     }
 
     companion object {
-        val logger = logger<CodeCompletionBaseIntention>()
+        private val logger = logger<CodeCompletionBaseIntention>()
     }
 }

src/main/kotlin/cc/unitmesh/devti/intentions/action/task/LivingDocumentationTask.kt

@@ -51,7 +51,7 @@ class LivingDocumentationTask(
     }
 
     companion object {
-        val logger = logger<LivingDocumentationTask>()
+        private val logger = logger<LivingDocumentationTask>()
     }
 }
 

src/main/kotlin/cc/unitmesh/devti/util/LLMCoroutineScope.kt

@@ -8,7 +8,7 @@ import kotlinx.coroutines.CoroutineExceptionHandler
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.SupervisorJob
 
-val logger = Logger.getInstance(LLMCoroutineScope::class.java)
+private val logger = Logger.getInstance(LLMCoroutineScope::class.java)
 
 val coroutineExceptionHandler = CoroutineExceptionHandler { _, throwable ->
     logger.error(throwable)