Skip to content

fix(amazonq): Enable stop button of chat #5671

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 13 commits into from
May 6, 2025
Merged
Show file tree
Hide file tree
Changes from 9 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@
import kotlinx.coroutines.flow.launchIn
import kotlinx.coroutines.flow.merge
import kotlinx.coroutines.flow.onEach
import kotlinx.coroutines.future.await
import kotlinx.coroutines.launch
import org.cef.browser.CefBrowser
import org.eclipse.lsp4j.Position
import org.eclipse.lsp4j.Range
import org.json.JSONObject
import software.aws.toolkits.core.utils.getLogger
import software.aws.toolkits.core.utils.warn
import software.aws.toolkits.jetbrains.services.amazonq.apps.AppConnection
Expand All @@ -32,6 +32,7 @@
import software.aws.toolkits.jetbrains.services.amazonq.lsp.encryption.JwtEncryptionManager
import software.aws.toolkits.jetbrains.services.amazonq.lsp.flareChat.AwsServerCapabilitiesProvider
import software.aws.toolkits.jetbrains.services.amazonq.lsp.flareChat.ChatCommunicationManager
import software.aws.toolkits.jetbrains.services.amazonq.lsp.flareChat.ChatCommunicationManager.Companion.convertToJsonToSendToChat
import software.aws.toolkits.jetbrains.services.amazonq.lsp.flareChat.getTextDocumentIdentifier
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.ButtonClickNotification
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.ButtonClickParams
Expand Down Expand Up @@ -90,9 +91,11 @@
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.PromptInputOptionChangeParams
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.QuickChatActionRequest
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.SEND_CHAT_COMMAND_PROMPT
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.STOP_CHAT_RESPONSE
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.SendChatPromptRequest
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.SourceLinkClickNotification
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.SourceLinkClickParams
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.StopResponseMessage
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.TabBarActionParams
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.TabBarActionRequest
import software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat.TabEventParams
Expand All @@ -104,6 +107,7 @@
import software.aws.toolkits.jetbrains.settings.MeetQSettings
import software.aws.toolkits.telemetry.MetricResult
import software.aws.toolkits.telemetry.Telemetry
import java.util.UUID
import java.util.concurrent.CompletableFuture
import java.util.function.Function

Expand Down Expand Up @@ -245,6 +249,10 @@
encryptionManager = this.encryptionManager
encryptionManager?.encrypt(chatParams)?.let { EncryptedChatParams(it, partialResultToken) }?.let { server.sendChatPrompt(it) }
} ?: (CompletableFuture.failedFuture(IllegalStateException("LSP Server not running")))

// We assume there is only one outgoing request per tab because the input is
// blocked when there is an outgoing request
chatCommunicationManager.setInflightRequestForTab(tabId, result)
showResult(result, partialResultToken, tabId, encryptionManager, browser)
}
CHAT_QUICK_ACTION -> {
Expand All @@ -262,6 +270,10 @@
}
} ?: (CompletableFuture.failedFuture(IllegalStateException("LSP Server not running")))

// We assume there is only one outgoing request per tab because the input is
// blocked when there is an outgoing request
chatCommunicationManager.setInflightRequestForTab(tabId, result)

showResult(result, partialResultToken, tabId, encryptionManager, browser)
}
CHAT_LIST_CONVERSATIONS -> {
Expand Down Expand Up @@ -318,6 +330,7 @@
CHAT_TAB_REMOVE -> {
handleChatNotification<TabEventRequest, TabEventParams>(node) { server, params ->
chatCommunicationManager.removePartialChatMessage(params.tabId)
cancelInflightRequests(params.tabId)
server.tabRemove(params)
}
}
Expand Down Expand Up @@ -414,6 +427,31 @@
server.createPrompt(params)
}
}
STOP_CHAT_RESPONSE -> {
val stopResponseRequest = serializer.deserializeChatMessages<StopResponseMessage>(node)
if (!chatCommunicationManager.hasInflightRequest(stopResponseRequest.params.tabId)) {
return
}
cancelInflightRequests(stopResponseRequest.params.tabId)
chatCommunicationManager.removePartialChatMessage(stopResponseRequest.params.tabId)

val paramsJson = JSONObject().apply {
put("title", "You stopped your current work, please provide additional examples or ask another question.")
put("additionalMessages", arrayOf<String>())
put("messageId", UUID.randomUUID().toString())
put("buttons", arrayOf<String>())
put("codeReference", arrayOf<String>())
put("body", "")
}

val uiMessage = convertToJsonToSendToChat(
command = SEND_CHAT_COMMAND_PROMPT,
tabId = stopResponseRequest.params.tabId,
params = paramsJson.toString(),
isPartialResult = false
)
browser.postChat(uiMessage)
}
}
}

Expand All @@ -426,13 +464,21 @@
) {
result.whenComplete { value, error ->
chatCommunicationManager.removePartialChatMessage(partialResultToken)
val messageToChat = ChatCommunicationManager.convertToJsonToSendToChat(

Check warning on line 467 in plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/webview/BrowserConnector.kt

View workflow job for this annotation

GitHub Actions / qodana

Redundant qualifier name

Redundant qualifier name
SEND_CHAT_COMMAND_PROMPT,
tabId,
encryptionManager?.decrypt(value).orEmpty(),
isPartialResult = false
)
browser.postChat(messageToChat)
chatCommunicationManager.removeInflightRequestForTab(tabId)
}
}

private fun cancelInflightRequests(tabId: String) {
chatCommunicationManager.getInflightRequestForTab(tabId)?.let { request ->
request.cancel(true)
chatCommunicationManager.removeInflightRequestForTab(tabId)
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,21 @@ import java.util.concurrent.ConcurrentHashMap
@Service(Service.Level.PROJECT)
class ChatCommunicationManager {
private val chatPartialResultMap = ConcurrentHashMap<String, String>()
private fun getPartialChatMessage(partialResultToken: String): String =
chatPartialResultMap.getValue(partialResultToken)
private fun getPartialChatMessage(partialResultToken: String): String? =
chatPartialResultMap.getOrDefault(partialResultToken, null)

private val inflightRequestByTabId = ConcurrentHashMap<String, CompletableFuture<String>>()

fun setInflightRequestForTab(tabId: String, result: CompletableFuture<String>) {
inflightRequestByTabId[tabId] = result
}
fun removeInflightRequestForTab(tabId: String) {
inflightRequestByTabId.remove(tabId)
}

fun getInflightRequestForTab(tabId: String): CompletableFuture<String>? = inflightRequestByTabId[tabId]

fun hasInflightRequest(tabId: String): Boolean = inflightRequestByTabId.containsKey(tabId)

fun addPartialChatMessage(tabId: String): String {
val partialResultToken: String = UUID.randomUUID().toString()
Expand All @@ -35,7 +48,7 @@ class ChatCommunicationManager {
fun handlePartialResultProgressNotification(project: Project, params: ProgressParams) {
val token = ProgressNotificationUtils.getToken(params)
val tabId = getPartialChatMessage(token)
if (tabId == null || tabId.isEmpty()) {
if (tabId.isNullOrEmpty()) {
return
}
if (params.value.isLeft || params.value.right == null) {
Expand All @@ -47,7 +60,6 @@ class ChatCommunicationManager {
val encryptedPartialChatResult = getObject(params, String::class.java)
if (encryptedPartialChatResult != null) {
val partialChatResult = AmazonQLspService.getInstance(project).encryptionManager.decrypt(encryptedPartialChatResult)

val uiMessage = convertToJsonToSendToChat(
command = SEND_CHAT_COMMAND_PROMPT,
tabId = tabId,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,3 +31,4 @@ const val CHAT_SEND_UPDATE = "aws/chat/sendChatUpdate"
const val CHAT_CREATE_PROMPT = "aws/chat/createPrompt"
const val SHOW_SAVE_FILE_DIALOG_REQUEST_METHOD = "aws/showSaveFileDialog"
const val GET_SERIALIZED_CHAT_REQUEST_METHOD = "aws/chat/getSerializedChat"
const val STOP_CHAT_RESPONSE = "stopChatResponse"
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// Copyright 2025 Amazon.com, Inc. or its affiliates. All Rights Reserved.
// SPDX-License-Identifier: Apache-2.0

package software.aws.toolkits.jetbrains.services.amazonq.lsp.model.aws.chat

data class StopResponseMessage(
val params: StopResponseParams,
)

data class StopResponseParams(
val tabId: String,
)
Loading