Skip to content

Commit 6a60792

Browse files
committed
fix(amazonq): do not call didChangeDependencyPaths on EDT
1 parent 67bd41c commit 6a60792

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/dependencies/DefaultModuleDependenciesService.kt

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import com.intellij.openapi.roots.ModuleRootListener
1111
import software.aws.toolkits.jetbrains.services.amazonq.lsp.AmazonQLspService
1212
import software.aws.toolkits.jetbrains.services.amazonq.lsp.dependencies.ModuleDependencyProvider.Companion.EP_NAME
1313
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.dependencies.DidChangeDependencyPathsParams
14-
import java.util.concurrent.CompletableFuture
14+
import software.aws.toolkits.jetbrains.utils.pluginAwareExecuteOnPooledThread
1515

1616
class DefaultModuleDependenciesService(
1717
private val project: Project,
@@ -34,10 +34,13 @@ class DefaultModuleDependenciesService(
3434
syncAllModules()
3535
}
3636

37-
override fun didChangeDependencyPaths(params: DidChangeDependencyPathsParams): CompletableFuture<Unit> =
37+
override fun didChangeDependencyPaths(params: DidChangeDependencyPathsParams) {
3838
AmazonQLspService.executeIfRunning(project) { languageServer ->
39-
languageServer.didChangeDependencyPaths(params)
40-
}?.toCompletableFuture() ?: CompletableFuture.failedFuture(IllegalStateException("LSP Server not running"))
39+
pluginAwareExecuteOnPooledThread {
40+
languageServer.didChangeDependencyPaths(params)
41+
}
42+
}
43+
}
4144

4245
private fun syncAllModules() {
4346
ModuleManager.getInstance(project).modules.forEach { module ->

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/dependencies/ModuleDependenciesService.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,5 @@ import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.dependenci
77
import java.util.concurrent.CompletableFuture
88

99
interface ModuleDependenciesService {
10-
fun didChangeDependencyPaths(params: DidChangeDependencyPathsParams): CompletableFuture<Unit>
10+
fun didChangeDependencyPaths(params: DidChangeDependencyPathsParams)
1111
}

0 commit comments

Comments
 (0)