feat(devin-lang): add support for script path configuration in run configurations #101

The commit introduces the ability to specify a script path when creating and editing run configurations for the DevInLanguage plugin. It includes changes to the DevInRunFileAction, AutoDevRunConfigurationProducer, AutoDevConfiguration, and DevInRunLineMarkersProvider classes to support this new feature. The script path is now stored in the run configuration and used to locate the script to be executed during runtime. Additionally, validation and error handling have been added to ensure that the specified script path is valid and exists.

Author: phodal

exts/devin-lang/src/main/kotlin/cc/unitmesh/devti/language/actions/DevInRunFileAction.kt

@@ -28,8 +28,7 @@ class DevInRunFileAction : DumbAwareAction() {
     }
 
     override fun actionPerformed(e: AnActionEvent) {
-        val file =
-            e.getData(CommonDataKeys.PSI_FILE) ?: return
+        val file = e.getData(CommonDataKeys.PSI_FILE) ?: return
 
         val project = file.project
         val context = ConfigurationContext.getFromContext(e.dataContext, e.place)
@@ -46,6 +45,8 @@ class DevInRunFileAction : DumbAwareAction() {
             configurationSettings
         }.configuration as AutoDevConfiguration
 
+        runConfiguration.setScriptPath(file.virtualFile.path)
+
         val builder =
             ExecutionEnvironmentBuilder.createOrNull(DefaultRunExecutor.getRunExecutorInstance(), runConfiguration)
         if (builder != null) {

exts/devin-lang/src/main/kotlin/cc/unitmesh/devti/language/run/AutoDevConfiguration.kt

@@ -1,43 +1,80 @@
 package cc.unitmesh.devti.language.run
 
+import cc.unitmesh.devti.AutoDevBundle
 import cc.unitmesh.devti.AutoDevIcons
+import cc.unitmesh.devti.runconfig.config.readString
+import cc.unitmesh.devti.runconfig.config.writeString
 import com.intellij.execution.Executor
-import com.intellij.execution.configurations.ConfigurationFactory
-import com.intellij.execution.configurations.RunConfiguration
-import com.intellij.execution.configurations.RunConfigurationBase
-import com.intellij.execution.configurations.RunProfileState
+import com.intellij.execution.configurations.*
 import com.intellij.execution.runners.ExecutionEnvironment
+import com.intellij.openapi.fileChooser.FileChooserDescriptorFactory
 import com.intellij.openapi.options.SettingsEditor
 import com.intellij.openapi.project.Project
+import com.intellij.openapi.ui.TextFieldWithBrowseButton
+import com.intellij.openapi.util.NlsSafe
+import com.intellij.openapi.util.io.FileUtil
 import com.intellij.ui.dsl.builder.panel
+import org.jdom.Element
+import org.jetbrains.annotations.NonNls
 import javax.swing.Icon
 import javax.swing.JComponent
 
 class AutoDevConfiguration(project: Project, factory: ConfigurationFactory, name: String) :
-    RunConfigurationBase<ConfigurationFactory>(project, factory, name) {
+    LocatableConfigurationBase<ConfigurationFactory>(project, factory, name) {
     override fun getIcon(): Icon = AutoDevIcons.AI_COPILOT
+    private var myScriptPath = ""
+    private val SCRIPT_PATH_TAG: @NonNls String = "SCRIPT_PATH"
 
-    override fun getState(executor: Executor, environment: ExecutionEnvironment): RunProfileState? {
+    override fun getState(executor: Executor, environment: ExecutionEnvironment): RunProfileState {
         return DevInRunConfigurationProfileState(project, this)
     }
 
-    override fun clone(): RunConfiguration {
-        return super.clone() as AutoDevConfiguration
+    override fun checkConfiguration() {
+        if (!FileUtil.exists(myScriptPath)) {
+            throw RuntimeConfigurationError(AutoDevBundle.message("devin.run.error.script.not.found"))
+        }
+    }
+
+    override fun writeExternal(element: Element) {
+        super.writeExternal(element)
+        element.writeString(SCRIPT_PATH_TAG, myScriptPath)
+    }
+
+    override fun readExternal(element: Element) {
+        super.readExternal(element)
+        myScriptPath = element.readString(SCRIPT_PATH_TAG) ?: ""
     }
 
     override fun getConfigurationEditor(): SettingsEditor<out RunConfiguration> {
         return AutoDevSettingsEditor(project)
     }
+
+    fun setScriptPath(scriptPath: String) {
+        myScriptPath = scriptPath.trim { it <= ' ' }
+    }
+
+    fun getScriptPath(): @NlsSafe String {
+        return myScriptPath
+    }
 }
 
-class AutoDevSettingsEditor(project: Project) : SettingsEditor<AutoDevConfiguration>() {
-    override fun createEditor(): JComponent = panel {
+class AutoDevSettingsEditor(val project: Project) : SettingsEditor<AutoDevConfiguration>() {
+    private val myScriptSelector: TextFieldWithBrowseButton? = null
 
+    override fun createEditor(): JComponent = panel {
+        myScriptSelector?.addBrowseFolderListener(
+            AutoDevBundle.message("devin.label.choose.file"),
+            "",
+            project,
+            FileChooserDescriptorFactory.createSingleFileDescriptor()
+        )
     }
 
     override fun resetEditorFrom(configuration: AutoDevConfiguration) {
+        myScriptSelector!!.setText(configuration.getScriptPath())
     }
 
     override fun applyEditorTo(configuration: AutoDevConfiguration) {
+        configuration.setScriptPath(myScriptSelector!!.text)
     }
 }

exts/devin-lang/src/main/kotlin/cc/unitmesh/devti/language/run/AutoDevRunConfigurationProducer.kt

@@ -20,6 +20,7 @@ class AutoDevRunConfigurationProducer : LazyRunConfigurationProducer<AutoDevConf
         val virtualFile = psiFile.virtualFile ?: return false
 
         configuration.name = virtualFile.presentableName
+        configuration.setScriptPath(virtualFile.path)
         return true
     }
 
@@ -30,8 +31,9 @@ class AutoDevRunConfigurationProducer : LazyRunConfigurationProducer<AutoDevConf
         val psiLocation = context.psiLocation ?: return false
         val psiFile = psiLocation.containingFile as? DevInFile ?: return false
         val virtualFile = psiFile.virtualFile ?: return false
+        val scriptPath = configuration.getScriptPath() ?: return false
 
-        return true
+        return virtualFile.path == scriptPath
     }
 
 }

exts/devin-lang/src/main/kotlin/cc/unitmesh/devti/language/run/DevInRunLineMarkersProvider.kt

@@ -9,9 +9,7 @@ import com.intellij.icons.AllIcons
 import com.intellij.openapi.actionSystem.ActionManager
 import com.intellij.openapi.actionSystem.AnAction
 import com.intellij.openapi.project.DumbAware
-import com.intellij.openapi.util.text.StringUtil
 import com.intellij.psi.PsiElement
-import com.intellij.util.containers.ContainerUtil
 
 class DevInRunLineMarkersProvider : RunLineMarkerContributor(), DumbAware {
     override fun getInfo(element: PsiElement): Info? {
@@ -24,13 +22,8 @@ class DevInRunLineMarkersProvider : RunLineMarkerContributor(), DumbAware {
 
         return Info(
             AllIcons.RunConfigurations.TestState.Run,
-            actions
-        ) { psiElement: PsiElement ->
-            StringUtil.join(
-                ContainerUtil.mapNotNull<AnAction, String>(actions) {
-                    AutoDevBundle.message("line.marker.run.0", psiElement.containingFile.name)
-                }, "\n"
-            )
-        }
+            { AutoDevBundle.message("line.marker.run.0", psiFile.containingFile.name) },
+            *actions
+        )
     }
 }

src/main/resources/messages/AutoDevBundle.properties

@@ -149,3 +149,5 @@ autoarkui.generate.design=Design Page
 devin.ref.loading=Loading git revision
 line.marker.run.0=Run {0}
 autodev.devti=Create property
+devin.label.choose.file=Choice DevIn File
+devin.run.error.script.not.found=Script Not Found