refactor(devti): improve file handling and code structure

- Rename and restructure Dockerfile creation logic
- Enhance JSON file processing in devcontainer setup
- Add support for making shell scripts executable
- Simplify and clarify code in AutoDevContainer and AutoDevRunAction classes

Author: phodal

core/src/main/kotlin/cc/unitmesh/devti/gui/snippet/AutoDevRunAction.kt

@@ -32,20 +32,20 @@ class AutoDevRunAction : AnAction(AutoDevBundle.message("autodev.run.action")) {
         val document = editor.document
         val file = FileDocumentManager.getInstance().getFile(document)
 
-        if (file != null) {
-            val lightFile = file as? LightVirtualFile
-            if (lightFile?.language == JsonLanguage.INSTANCE) {
-                val virtualFile = AutoDevContainer.updateForDevContainer(project, file, document.text)
-                    ?: lightFile
-                e.presentation.isEnabled = RunService.provider(project, virtualFile) != null
-                return
-            }
+        if (file == null) {
+            e.presentation.isEnabled = false
+            return
+        }
 
-            e.presentation.isEnabled = RunService.provider(project, file) != null
+        val lightFile = file as? LightVirtualFile
+        if (lightFile?.language == JsonLanguage.INSTANCE) {
+            val virtualFile = AutoDevContainer.updateForDevContainer(project, file, document.text)
+                ?: lightFile
+            e.presentation.isEnabled = RunService.provider(project, virtualFile) != null
             return
         }
 
-        e.presentation.isEnabled = false
+        e.presentation.isEnabled = RunService.provider(project, file) != null
     }
 
     override fun actionPerformed(e: AnActionEvent) {
@@ -60,23 +60,28 @@ class AutoDevRunAction : AnAction(AutoDevBundle.message("autodev.run.action")) {
         var scratchFile: VirtualFile? = ScratchRootType.getInstance()
             .createScratchFile(project, file.name, originPsiFile.language, document.text)
 
-        if (scratchFile?.extension == "Dockerfile") {
-            scratchFile = createDockerFile(project, document.text) ?: scratchFile
-        } else if (scratchFile?.extension?.lowercase() == "json") {
-            scratchFile = AutoDevContainer.updateForDevContainer(project, file as LightVirtualFile, document.text) ?: scratchFile
+        val extension = scratchFile?.extension
+        when {
+            extension == "Dockerfile" -> {
+                scratchFile = createDockerFile(project, document.text) ?: scratchFile
+            }
+
+            extension?.lowercase() == "json" -> {
+                scratchFile = AutoDevContainer.updateForDevContainer(project, file as LightVirtualFile, document.text)
+                    ?: scratchFile
+            }
+
+            scratchFile?.extension == "sh" -> {
+                File(scratchFile.path).setExecutable(true)
+            }
         }
 
         if (scratchFile == null) {
             AutoDevNotifications.warn(project, "Cannot create scratch file")
             return
         }
 
-        if (scratchFile.extension == "sh") {
-            File(scratchFile.path).setExecutable(true)
-        }
-
-        var psiFile = PsiManager.getInstance(project).findFile(scratchFile)
-            ?: return
+        var psiFile = PsiManager.getInstance(project).findFile(scratchFile) ?: return
 
         try {
             RunService.provider(project, scratchFile)
@@ -88,25 +93,18 @@ class AutoDevRunAction : AnAction(AutoDevBundle.message("autodev.run.action")) {
         }
     }
 
-    private fun createDockerFile(project: Project, text: @NlsSafe String): VirtualFile? {
-        val projectDir = project.guessProjectDir()
-        if (projectDir == null) {
-            return null
-        }
-
-        return runWriteAction {
-            try {
-                // 在项目根目录创建名为 dockerfile 的文件
-                var dockerfile = projectDir.findChild("Dockerfile")
-                if (dockerfile == null) {
-                    dockerfile = projectDir.createChildData(null, "Dockerfile")
-                }
-
-                dockerfile.setBinaryContent(text.toByteArray())
-                return@runWriteAction dockerfile
-            } catch (e: IOException) {
-                return@runWriteAction null
+    private fun createDockerFile(project: Project, text: String): VirtualFile? = runWriteAction {
+        val projectDir = project.guessProjectDir()!!
+        try {
+            var dockerfile = projectDir.findChild("Dockerfile")
+            if (dockerfile == null) {
+                dockerfile = projectDir.createChildData(null, "Dockerfile")
             }
+
+            dockerfile.setBinaryContent(text.toByteArray())
+            return@runWriteAction dockerfile
+        } catch (e: IOException) {
+            return@runWriteAction null
         }
     }
 }

core/src/main/kotlin/cc/unitmesh/devti/gui/snippet/container/AutoDevContainer.kt

@@ -11,16 +11,12 @@ object AutoDevContainer {
     private val DEV_CONTAINER_PROPS =
         setOf("image", "dockerFile", "containerEnv", "remoteUser", "customizations", "features")
 
-    fun updateForDevContainer(
-        project: Project,
-        lightVirtualFile: LightVirtualFile,
-        content: String
-    ): LightVirtualFile? {
-        val fileName = lightVirtualFile.name.lowercase()
+    fun updateForDevContainer(project: Project, vfile: LightVirtualFile, content: String): LightVirtualFile? {
+        val fileName = vfile.name.lowercase()
         if ((!content.startsWith("{") && !content.endsWith("}"))) return null
 
         if (fileName == "devcontainer.json" || fileName.contains("devcontainer")) {
-            return lightVirtualFile
+            return vfile
         }
 
         val objectMapper = ObjectMapper().registerModule(KotlinModule.Builder().build())
@@ -52,8 +48,6 @@ object AutoDevContainer {
         }
 
         if (!isDevContainer) return null
-        val newFile = LightVirtualFile("devcontainer.json", JsonLanguage.INSTANCE, content)
-
-        return newFile
+        return LightVirtualFile("devcontainer.json", JsonLanguage.INSTANCE, content)
     }
 }