fix(intellij): stabilized getting the correct python sdk for a project

Author: d-biehl

intellij-client/src/main/kotlin/dev/robotcode/robotcode4ij/RobotCodeHelpers.kt

@@ -6,6 +6,7 @@ import com.intellij.openapi.application.ApplicationManager
 import com.intellij.openapi.application.PathManager
 import com.intellij.openapi.diagnostic.thisLogger
 import com.intellij.openapi.project.Project
+import com.intellij.openapi.project.modules
 import com.intellij.openapi.util.Key
 import com.jetbrains.python.sdk.pythonSdk
 import java.nio.file.Path
@@ -26,14 +27,21 @@ class RobotCodeHelpers {
     }
 }
 
+val Project.robotPythonSdk: com.intellij.openapi.projectRoots.Sdk?
+    get() {
+        return this.pythonSdk ?: this.projectFile?.let {
+            this.modules.firstNotNullOfOrNull { it.pythonSdk }
+        }
+    }
+
 fun Project.checkPythonAndRobotVersion(reset: Boolean = false): Boolean {
     if (!reset && this.getUserData(RobotCodeHelpers.PYTHON_AND_ROBOT_OK_KEY) == true) {
         return true
     }
 
     val result = ApplicationManager.getApplication().executeOnPooledThread<Boolean> {
 
-        val pythonInterpreter = this.pythonSdk?.homePath
+        val pythonInterpreter = this.robotPythonSdk?.homePath
 
         if (pythonInterpreter == null) {
             thisLogger().info("No Python Interpreter defined for project '${this.name}'")
@@ -93,7 +101,7 @@ fun Project.buildRobotCodeCommandLine(
         throw IllegalArgumentException("PythonSDK is not defined or robot version is not valid for project ${this.name}")
     }
 
-    val pythonInterpreter = this.pythonSdk?.homePath
+    val pythonInterpreter = this.robotPythonSdk?.homePath
     val commandLine = GeneralCommandLine(
         pythonInterpreter,
         "-u",

intellij-client/src/main/kotlin/dev/robotcode/robotcode4ij/RobotCodePostStartupActivity.kt

@@ -1,6 +1,5 @@
 package dev.robotcode.robotcode4ij
 
-import com.intellij.openapi.Disposable
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.startup.ProjectActivity
 import com.intellij.openapi.vfs.VirtualFileManager
@@ -13,12 +12,12 @@ import dev.robotcode.robotcode4ij.testing.testManger
 import kotlinx.coroutines.flow.collect
 import kotlinx.coroutines.flow.onEach
 
-class RobotCodePostStartupActivity : ProjectActivity, Disposable {
+class RobotCodePostStartupActivity : ProjectActivity {
     override suspend fun execute(project: Project) {
         project.langServerManager.start()
         project.testManger.refresh()
 
-        VirtualFileManager.getInstance().addAsyncFileListener(RobotCodeVirtualFileListener(project), this)
+        VirtualFileManager.getInstance().addAsyncFileListener(RobotCodeVirtualFileListener(project), project.testManger)
 
         project.workspaceModel.eventLog.onEach {
             val moduleChanges = it.getChanges(ModuleEntity::class.java)
@@ -29,10 +28,5 @@ class RobotCodePostStartupActivity : ProjectActivity, Disposable {
             }
         }.collect()
     }
-    
-    override fun dispose() {
-        // do nothing
-    }
-    
 }
 

intellij-client/src/main/kotlin/dev/robotcode/robotcode4ij/testing/RobotCodeTestManager.kt

@@ -115,10 +115,13 @@ import java.util.*
         }
     }
 
-    
     private val refreshScope = CoroutineScope(Dispatchers.Default)
 
     fun refreshDebounced(file: VirtualFile) {
+        if (!project.isOpen || project.isDisposed) {
+            return
+        }
+        
         val job = refreshJobs[file]
 
         if (job != null) {
@@ -137,6 +140,10 @@ import java.util.*
     }
 
     fun refreshDebounced() {
+        if (!project.isOpen || project.isDisposed) {
+            return
+        }
+        
         refreshJobs.values.forEach { it.cancel() }
         refreshJob?.cancel()
 
@@ -148,6 +155,10 @@ import java.util.*
     }
 
     fun refresh(uri: String) {
+        if (!project.isOpen || project.isDisposed) {
+            return
+        }
+        
         thisLogger().info("Refreshing test items for $uri")
         try {
             val testItem = findTestItem(uri) ?: return