fix: infinite indexing / intellisense blocked

Author: denbezrukov

gradle.properties

@@ -11,7 +11,7 @@ pluginUntilBuild = 243.*
 
 # IntelliJ Platform Properties -> https://plugins.jetbrains.com/docs/intellij/tools-gradle-intellij-plugin.html#configuration-intellij-extension
 platformType = WS
-platformVersion = 2024.1.1
+platformVersion = 2024.3
 
 # Plugin Dependencies -> https://plugins.jetbrains.com/docs/intellij/plugin-dependencies.html
 # Example: platformPlugins = com.jetbrains.php:203.4449.22, org.intellij.scala:2023.3.27@EAP

src/main/kotlin/com/github/biomejs/intellijbiome/lsp/BiomeLspServerSupportProvider.kt

@@ -3,20 +3,30 @@ package com.github.biomejs.intellijbiome.lsp
 import com.github.biomejs.intellijbiome.BiomeBundle
 import com.github.biomejs.intellijbiome.BiomeIcons
 import com.github.biomejs.intellijbiome.BiomePackage
-import com.github.biomejs.intellijbiome.extensions.runBiomeCLI
+import com.github.biomejs.intellijbiome.extensions.isNodeScript
 import com.github.biomejs.intellijbiome.listeners.BIOME_CONFIG_RESOLVED_TOPIC
 import com.github.biomejs.intellijbiome.services.BiomeServerService
 import com.github.biomejs.intellijbiome.settings.BiomeConfigurable
 import com.github.biomejs.intellijbiome.settings.BiomeSettings
 import com.intellij.execution.ExecutionException
 import com.intellij.execution.configurations.GeneralCommandLine
+import com.intellij.execution.process.OSProcessHandler
+import com.intellij.javascript.nodejs.execution.NodeTargetRun
+import com.intellij.javascript.nodejs.execution.NodeTargetRunOptions.Companion.of
+import com.intellij.javascript.nodejs.execution.withInvisibleProgress
+import com.intellij.javascript.nodejs.interpreter.NodeJsInterpreterManager
+import com.intellij.javascript.nodejs.interpreter.local.NodeJsLocalInterpreter
+import com.intellij.javascript.nodejs.interpreter.wsl.WslNodeInterpreter
+import com.intellij.lang.javascript.JavaScriptBundle
 import com.intellij.openapi.components.service
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.vfs.VirtualFile
 import com.intellij.platform.lsp.api.*
 import com.intellij.platform.lsp.api.customization.LspFormattingSupport
 import com.intellij.platform.lsp.api.lsWidget.LspServerWidgetItem
 import com.intellij.util.SmartList
+import java.io.File
+import kotlin.io.path.Path
 
 
 @Suppress("UnstableApiUsage")
@@ -38,29 +48,32 @@ class BiomeLspServerSupportProvider : LspServerSupportProvider {
         serverStarter.ensureServerStarted(BiomeLspServerDescriptor(project, executable, configPath))
     }
 
-    override fun createLspServerWidgetItem(lspServer: LspServer, currentFile: VirtualFile?) = LspServerWidgetItem(
+    override fun createLspServerWidgetItem(lspServer: LspServer,
+        currentFile: VirtualFile?) = LspServerWidgetItem(
         lspServer, currentFile,
         BiomeIcons.BiomeIcon, BiomeConfigurable::class.java
     )
 }
 
 @Suppress("UnstableApiUsage")
-class BiomeLspServerManagerListener(val project: Project) : LspServerManagerListener {
-    override fun serverStateChanged(lspServer: LspServer) {
-        if (lspServer.descriptor is BiomeLspServerDescriptor && lspServer.state == LspServerState.ShutdownUnexpectedly) {
-            // restart again if the server was shutdown unexpectedly.
-            // This can be caused by race condition, when we restart LSP server because of config change,
-            // but Intellij also tried to send a request to it at the same time.
-            // Unfortunate There is no way prevent IDEA send requests after LSP started.
-            project.service<BiomeServerService>().restartBiomeServer()
+private class BiomeLspServerDescriptor(project: Project,
+    val executable: String,
+    val configPath: String?) :
+    ProjectWideLspServerDescriptor(project, "Biome") {
+    private val biomePackage = BiomePackage(project)
+    private val executableFile = File(executable)
+    private val params: SmartList<String> by lazy {
+        val params = SmartList("lsp-proxy")
+        if (!configPath.isNullOrEmpty()) {
+            params.add("--config-path")
+            params.add(configPath)
         }
+        return@lazy params
     }
-}
 
-@Suppress("UnstableApiUsage")
-private class BiomeLspServerDescriptor(project: Project, val executable: String, val configPath: String?) :
-    ProjectWideLspServerDescriptor(project, "Biome") {
-    private val biomePackage = BiomePackage(project)
+    init {
+        biomePackage.versionNumber()?.let { project.messageBus.syncPublisher(BIOME_CONFIG_RESOLVED_TOPIC).resolved(it) }
+    }
 
     override fun isSupportedFile(file: VirtualFile): Boolean {
         val settings = BiomeSettings.getInstance(project)
@@ -72,25 +85,44 @@ private class BiomeLspServerDescriptor(project: Project, val executable: String,
     }
 
     override fun createCommandLine(): GeneralCommandLine {
-        val params = SmartList("lsp-proxy")
-
-        if (!configPath.isNullOrEmpty()) {
-            params.add("--config-path")
-            params.add(configPath)
+        // we're here only if we specify an executable file
+        return GeneralCommandLine().apply {
+            withExePath(executable)
+            withParentEnvironmentType(GeneralCommandLine.ParentEnvironmentType.CONSOLE)
+            project.basePath?.let {
+                withWorkingDirectory(Path(it))
+            }
+            addParameters(params)
         }
 
+    }
+
+    override fun startServerProcess(): OSProcessHandler {
         if (executable.isEmpty()) {
             throw ExecutionException(BiomeBundle.message("biome.language.server.not.found"))
         }
 
-        val version = biomePackage.versionNumber()
+        if (!(executableFile.isFile && executableFile.isNodeScript())) {
+            return super.startServerProcess()
+        }
 
-        version?.let { project.messageBus.syncPublisher(BIOME_CONFIG_RESOLVED_TOPIC).resolved(it) }
+        val interpreter = NodeJsInterpreterManager.getInstance(project).interpreter
+        if (interpreter !is NodeJsLocalInterpreter && interpreter !is WslNodeInterpreter) {
+            throw ExecutionException(JavaScriptBundle.message("lsp.interpreter.error"))
+        }
 
-        return GeneralCommandLine().runBiomeCLI(project, executable).apply {
+        val target = NodeTargetRun(interpreter, project, null, of(false))
+        target.commandLineBuilder.apply {
+            project.basePath?.let { this.setWorkingDirectory(target.path(it)) }
+            addParameter(target.path(executable))
             addParameters(params)
-            withWorkDirectory(configPath)
+            addParameter("--stdio")
+            charset = Charsets.UTF_8
         }
+
+        val process = withInvisibleProgress { target.startProcessEx() }
+
+        return process.processHandler
     }
 
     override val lspGoToDefinitionSupport = false