Cleanup JetBrains command calling to be consistent with VSC

Author: pkukielka

jetbrains/src/integrationTest/kotlin/com/sourcegraph/cody/edit/DocumentCodeTest.kt

@@ -1,7 +1,7 @@
 package com.sourcegraph.cody.edit
 
 import com.intellij.openapi.command.WriteCommandAction
-import com.sourcegraph.cody.edit.actions.DocumentCodeAction
+import com.sourcegraph.cody.chat.actions.DocumentCodeAction
 import com.sourcegraph.cody.edit.lenses.LensesService
 import com.sourcegraph.cody.edit.lenses.actions.EditAcceptAction
 import com.sourcegraph.cody.edit.lenses.actions.EditCancelAction

jetbrains/src/integrationTest/kotlin/com/sourcegraph/cody/util/EditCodeFixture.kt

@@ -9,6 +9,7 @@ import com.intellij.openapi.vfs.VirtualFile
 import com.sourcegraph.cody.agent.CodyAgentService
 import com.sourcegraph.cody.agent.protocol_generated.ClientCapabilities
 import com.sourcegraph.cody.agent.protocol_generated.ProtocolCodeLens
+import com.sourcegraph.cody.chat.actions.DocumentCodeAction
 import com.sourcegraph.cody.edit.lenses.LensListener
 import com.sourcegraph.cody.edit.lenses.LensesService
 import com.sourcegraph.cody.edit.lenses.providers.EditAcceptCodeVisionProvider
@@ -30,11 +31,14 @@ class EditCodeFixture(recordingName: String) :
   override fun checkInitialConditionsForOpenFile() {
 
     // Check the initial state of the action's presentation
-    val action = ActionManager.getInstance().getAction("cody.documentCodeAction")
+    val action = ActionManager.getInstance().getAction(DocumentCodeAction.ID)
     val event = AnActionEvent.createFromAnAction(action, null, "", createEditorContext(editor))
     action.update(event)
     val presentation = event.presentation
-    assertEquals("Action description should be empty", "", presentation.description)
+    assertEquals(
+        "Action description should be present",
+        "Documents the selected code",
+        presentation.description)
     assertTrue("Action should be enabled", presentation.isEnabled)
     assertTrue("Action should be visible", presentation.isVisible)
   }

jetbrains/src/main/kotlin/com/sourcegraph/cody/chat/actions/BaseCommandAction.kt

@@ -11,17 +11,14 @@ import com.sourcegraph.cody.agent.protocol_extensions.ProtocolTextDocumentExt
 import com.sourcegraph.cody.agent.protocol_generated.ExecuteCommandParams
 import com.sourcegraph.cody.agent.protocol_generated.Ignore_TestResult
 import com.sourcegraph.cody.auth.CodyAuthService
-import com.sourcegraph.cody.commands.CommandId
 import com.sourcegraph.cody.ignore.ActionInIgnoredFileNotification
 import com.sourcegraph.cody.ignore.IgnoreOracle
 import com.sourcegraph.common.ui.DumbAwareEDTAction
 import com.sourcegraph.config.ConfigUtil.isCodyEnabled
 import com.sourcegraph.utils.CodyEditorUtil
 import java.util.concurrent.Callable
 
-abstract class BaseCommandAction : DumbAwareEDTAction() {
-
-  abstract val myCommandId: CommandId
+abstract class BaseCommandAction(val id: String) : DumbAwareEDTAction() {
 
   override fun actionPerformed(event: AnActionEvent) {
     doAction(event.project ?: return)
@@ -48,11 +45,7 @@ abstract class BaseCommandAction : DumbAwareEDTAction() {
                 CodyAgentService.withAgent(project) { agent ->
                   agent.server.command_execute(
                       ExecuteCommandParams(
-                          command =
-                              when (myCommandId) {
-                                CommandId.Explain -> "cody.command.explain-code"
-                                CommandId.Smell -> "cody.command.smell-code"
-                              },
+                          command = id,
                           arguments = emptyList(),
                       ))
                 }

jetbrains/src/main/kotlin/com/sourcegraph/cody/chat/actions/DocumentCodeAction.kt

@@ -0,0 +1,7 @@
+package com.sourcegraph.cody.chat.actions
+
+class DocumentCodeAction : BaseCommandAction(ID) {
+  companion object {
+    const val ID = "cody.command.document-code"
+  }
+}

jetbrains/src/main/kotlin/com/sourcegraph/cody/chat/actions/EditCodeAction.kt

@@ -0,0 +1,7 @@
+package com.sourcegraph.cody.chat.actions
+
+class EditCodeAction : BaseCommandAction(ID) {
+  companion object {
+    const val ID = "cody.command.edit-code"
+  }
+}

jetbrains/src/main/kotlin/com/sourcegraph/cody/chat/actions/ExplainCodeAction.kt

@@ -0,0 +1,7 @@
+package com.sourcegraph.cody.chat.actions
+
+class ExplainCodeAction : BaseCommandAction(ID) {
+  companion object {
+    const val ID = "cody.command.explain-code"
+  }
+}

jetbrains/src/main/kotlin/com/sourcegraph/cody/chat/actions/ExplainCommand.kt

@@ -0,0 +1,7 @@
+package com.sourcegraph.cody.chat.actions
+
+class SmellCodeAction : BaseCommandAction(ID) {
+  companion object {
+    const val ID = "cody.command.smell-code"
+  }
+}

jetbrains/src/main/kotlin/com/sourcegraph/cody/chat/actions/SmellCodeAction.kt

@@ -0,0 +1,7 @@
+package com.sourcegraph.cody.chat.actions
+
+class UnitTestAction : BaseCommandAction(ID) {
+  companion object {
+    const val ID = "cody.command.unit-test"
+  }
+}