Skip to content

Commit 5855def

Browse files
committed
fix: address code scanning comments
1 parent 6d2eb92 commit 5855def

File tree

3 files changed

+7
-11
lines changed

3 files changed

+7
-11
lines changed

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

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -235,11 +235,6 @@ object AmazonQChatServer : JsonRpcMethodProvider {
235235
CreatePromptParams::class.java
236236
)
237237

238-
val sendPinnedContext = JsonRpcNotification(
239-
CHAT_SEND_PINNED_CONTEXT,
240-
LSPAny::class.java
241-
)
242-
243238
val pinnedContextAdd = JsonRpcNotification(
244239
CHAT_PINNED_CONTEXT_ADD,
245240
LSPAny::class.java

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
@@ -71,6 +71,7 @@ import software.aws.toolkits.jetbrains.isDeveloperMode
7171
import software.aws.toolkits.jetbrains.services.amazonq.lsp.artifacts.ArtifactManager
7272
import software.aws.toolkits.jetbrains.services.amazonq.lsp.auth.DefaultAuthCredentialsService
7373
import software.aws.toolkits.jetbrains.services.amazonq.lsp.dependencies.DefaultModuleDependenciesService
74+
import software.aws.toolkits.jetbrains.services.amazonq.lsp.editor.ActiveEditorChangeListener
7475
import software.aws.toolkits.jetbrains.services.amazonq.lsp.encryption.JwtEncryptionManager
7576
import software.aws.toolkits.jetbrains.services.amazonq.lsp.flareChat.AmazonQLspTypeAdapterFactory
7677
import software.aws.toolkits.jetbrains.services.amazonq.lsp.flareChat.AwsExtendedInitializeResult
@@ -561,11 +562,12 @@ private class AmazonQServerInstance(private val project: Project, private val cs
561562
thread
562563
}
563564

564-
// Register active editor change listener
565-
software.aws.toolkits.jetbrains.services.amazonq.lsp.editor.ActiveEditorChangeListener.register(
565+
ActiveEditorChangeListener.register(
566566
project,
567567
executor
568-
)
568+
).also {
569+
Disposer.register(this, it)
570+
}
569571
LOG.info { "Registered active editor change listener" }
570572
}
571573
}

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class ActiveEditorChangeListener(
2828
private val executor: ScheduledExecutorService,
2929
) : Disposable {
3030
private var debounceTask: ScheduledFuture<*>? = null
31-
private val DEBOUNCE_DELAY_MS = 100L
31+
private val debounceDelayMs = 100L
3232

3333
init {
3434
val connection = project.messageBus.connect(this)
@@ -63,7 +63,7 @@ class ActiveEditorChangeListener(
6363
} catch (e: Exception) {
6464
LOG.warn(e) { "Failed to send active editor changed notification" }
6565
}
66-
}, DEBOUNCE_DELAY_MS, TimeUnit.MILLISECONDS)
66+
}, debounceDelayMs, TimeUnit.MILLISECONDS)
6767
}
6868

6969
override fun dispose() {
@@ -75,7 +75,6 @@ class ActiveEditorChangeListener(
7575

7676
fun register(project: Project, executor: ScheduledExecutorService): ActiveEditorChangeListener {
7777
val listener = ActiveEditorChangeListener(project, executor)
78-
Disposer.register(project, listener)
7978
return listener
8079
}
8180
}

0 commit comments

Comments
 (0)