Skip to content

Commit 0efb633

Browse files
committed
use serverInstance as messageBus disposable
1 parent a5250b5 commit 0efb633

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

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

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -237,9 +237,11 @@ private class AmazonQServerInstance(private val project: Project, private val cs
237237
languageServer.initialized(InitializedParams())
238238
}
239239

240-
//may need to register listeners differently so their messageBus' don't get garbage collected
241-
val workspaceServiceHandler = WorkspaceServiceHandler(project, languageServer)
242-
workspaceServiceHandler.startWorkspaceServiceListeners()
240+
WorkspaceServiceHandler(
241+
project,
242+
languageServer,
243+
this
244+
).startWorkspaceServiceListeners()
243245
}
244246

245247
override fun dispose() {

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/workspace/WorkspaceServiceHandler.kt

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@ import software.aws.toolkits.jetbrains.utils.pluginAwareExecuteOnPooledThread
2020
class WorkspaceServiceHandler(
2121
private val project: Project,
2222
private val languageServer: AmazonQLanguageServer,
23-
) : Disposable {
23+
private val serverInstance: Disposable
24+
){
2425

2526
fun startWorkspaceServiceListeners() {
2627
startFileLifecycleListener()
@@ -55,7 +56,7 @@ class WorkspaceServiceHandler(
5556
}
5657

5758
private fun startFileLifecycleListener() {
58-
project.messageBus.connect(this).subscribe(
59+
project.messageBus.connect(serverInstance).subscribe(
5960
VirtualFileManager.VFS_CHANGES,
6061
object : BulkFileListener {
6162
override fun after(events: List<VFileEvent>) {
@@ -73,7 +74,4 @@ class WorkspaceServiceHandler(
7374
//private fun didChangeWorkspaceFolders() {
7475
// languageServer.workspaceService.didChangeWorkspaceFolders()
7576
//}
76-
77-
override fun dispose() {
78-
}
7977
}

0 commit comments

Comments
 (0)