refactor: override getActionUpdateThread in all actions

Author: lppedd

src/main/kotlin/com/github/lppedd/cc/completion/menu/FilterAction.kt

@@ -8,6 +8,7 @@ import com.github.lppedd.cc.lookupElement.CommitTokenLookupElement
 import com.github.lppedd.cc.updateIcons
 import com.intellij.codeInsight.completion.PlainPrefixMatcher
 import com.intellij.codeInsight.lookup.impl.LookupImpl
+import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnAction
 import com.intellij.openapi.actionSystem.AnActionEvent
 import java.util.*
@@ -69,6 +70,9 @@ internal class FilterAction(
     backupItems = emptyList()
   }
 
+  override fun getActionUpdateThread(): ActionUpdateThread =
+    ActionUpdateThread.EDT
+
   override fun actionPerformed(event: AnActionEvent) {
     if (enhancer.filterSelected(this)) {
       filterItems(!isFiltered)

src/main/kotlin/com/github/lppedd/cc/completion/menu/SettingsActions.kt

@@ -10,10 +10,7 @@ import com.github.lppedd.cc.configuration.CCConfigService.ProviderFilterType.KEE
 import com.github.lppedd.cc.updateIcons
 import com.intellij.codeInsight.lookup.impl.LookupImpl
 import com.intellij.icons.AllIcons
-import com.intellij.openapi.actionSystem.ActionGroup
-import com.intellij.openapi.actionSystem.AnAction
-import com.intellij.openapi.actionSystem.AnActionEvent
-import com.intellij.openapi.actionSystem.Separator
+import com.intellij.openapi.actionSystem.*
 import com.intellij.openapi.components.service
 import com.intellij.openapi.project.DumbAware
 import com.intellij.util.ui.UIUtil
@@ -47,10 +44,17 @@ internal class SettingsActions(
     Objects.hashCode(lookup)
 
   private inner class CompletionModeChangeAction : AnAction() {
+    override fun getActionUpdateThread(): ActionUpdateThread =
+      ActionUpdateThread.EDT
+
     override fun actionPerformed(e: AnActionEvent) {
       config.completionType =
-        if (config.completionType == TEMPLATE) POPUP
-        else TEMPLATE
+        if (config.completionType == TEMPLATE) {
+          POPUP
+        } else {
+          TEMPLATE
+        }
+
       enhancer.settingChanged()
     }
 
@@ -67,10 +71,17 @@ internal class SettingsActions(
   }
 
   private inner class FilterModeChangeAction : AnAction() {
+    override fun getActionUpdateThread(): ActionUpdateThread =
+      ActionUpdateThread.EDT
+
     override fun actionPerformed(e: AnActionEvent) {
       config.providerFilterType =
-        if (config.providerFilterType == HIDE_SELECTED) KEEP_SELECTED
-        else HIDE_SELECTED
+        if (config.providerFilterType == HIDE_SELECTED) {
+          KEEP_SELECTED
+        } else {
+          HIDE_SELECTED
+        }
+
       enhancer.settingChanged()
     }
 

src/main/kotlin/com/github/lppedd/cc/editor/CommitTabAction.kt

@@ -72,7 +72,7 @@ internal class CommitTabAction : TabAction() {
         }
       }
 
-      @Suppress("deprecation")
+      @Suppress("DEPRECATION")
       return super.isEnabled(editor, dataContext)
     }
 

src/main/kotlin/com/github/lppedd/cc/ui/DialogGotoErrorAction.kt

@@ -1,6 +1,7 @@
 package com.github.lppedd.cc.ui
 
 import com.github.lppedd.cc.ui.CCDialogWrapper.ValidationNavigable
+import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.project.DumbAwareAction
 import com.intellij.ui.EditorTextField
@@ -14,6 +15,9 @@ internal abstract class DialogGotoErrorAction : DumbAwareAction() {
     isEnabledInModalContext = true
   }
 
+  override fun getActionUpdateThread(): ActionUpdateThread =
+    ActionUpdateThread.EDT
+
   override fun update(event: AnActionEvent) {
     event.presentation.isEnabledAndVisible = event.getData(ValidationNavigable.DIALOG) != null
   }

src/main/kotlin/com/github/lppedd/cc/vcs/ClearLocalMessageHistoryAction.kt

@@ -1,6 +1,7 @@
 package com.github.lppedd.cc.vcs
 
 import com.github.lppedd.cc.CCIcons
+import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.components.service
 import com.intellij.openapi.project.DumbAwareAction
@@ -16,6 +17,9 @@ internal class ClearLocalMessageHistoryAction : DumbAwareAction() {
     templatePresentation.icon = CCIcons.General.ClearMessageHistory
   }
 
+  override fun getActionUpdateThread(): ActionUpdateThread =
+    ActionUpdateThread.EDT
+
   override fun update(event: AnActionEvent) {
     event.presentation.isEnabledAndVisible = false
     val project = event.project

src/main/kotlin/com/github/lppedd/cc/vcs/commitbuilder/CommitBuilderAction.kt

@@ -4,6 +4,7 @@ import com.github.lppedd.cc.CCBundle
 import com.github.lppedd.cc.CCIcons
 import com.github.lppedd.cc.moveCaretToOffset
 import com.github.lppedd.cc.removeSelection
+import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.command.CommandProcessor
 import com.intellij.openapi.project.DumbAwareAction
@@ -24,6 +25,9 @@ internal class CommitBuilderAction : DumbAwareAction() {
     templatePresentation.icon = CCIcons.Logo
   }
 
+  override fun getActionUpdateThread(): ActionUpdateThread =
+    ActionUpdateThread.EDT
+
   override fun update(event: AnActionEvent) {
     val isVisible = event.project != null && getCommitMessagePanel(event) != null
     event.presentation.isEnabledAndVisible = isVisible

src/main/kotlin/com/github/lppedd/cc/whatsnew/WhatsNewAction.kt

@@ -1,6 +1,7 @@
 package com.github.lppedd.cc.whatsnew
 
 import com.github.lppedd.cc.CCIcons
+import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.project.DumbAwareAction
 
@@ -15,6 +16,9 @@ internal class WhatsNewAction : DumbAwareAction() {
     templatePresentation.icon = CCIcons.Logo
   }
 
+  override fun getActionUpdateThread(): ActionUpdateThread =
+    ActionUpdateThread.EDT
+
   override fun actionPerformed(event: AnActionEvent) {
     WhatsNewDialog.showForProject(event.project ?: return)
   }