Skip to content

Commit 555ccea

Browse files
author
Viktor Shesternyak
committed
fix(amazonq): /doc fix prompt to change folder in chat
1 parent effbebd commit 555ccea

File tree

6 files changed

+85
-4
lines changed

6 files changed

+85
-4
lines changed
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
{
2+
"type" : "bugfix",
3+
"description" : "Amazon Q /doc: fix for user prompt to change folder in chat"
4+
}

plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonqDoc/controller/DocController.kt

Lines changed: 40 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.sendCodeResu
5252
import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.sendError
5353
import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.sendFolderConfirmationMessage
5454
import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.sendMonthlyLimitError
55+
import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.sendRetryChangeFolderMessage
5556
import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.sendSystemPrompt
5657
import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.sendUpdatePlaceholder
5758
import software.aws.toolkits.jetbrains.services.amazonqDoc.messages.sendUpdatePromptProgress
@@ -330,6 +331,31 @@ class DocController(
330331
}
331332
}
332333

334+
private suspend fun promptForRetryFolderSelection(tabId: String, message: String) {
335+
messenger.sendRetryChangeFolderMessage(
336+
tabId = tabId,
337+
message = message,
338+
followUps = listOf(
339+
FollowUp(
340+
icon = FollowUpIcons.Refresh,
341+
pillText = message("amazonqDoc.prompt.folder.change"),
342+
prompt = message("amazonqDoc.prompt.folder.change"),
343+
status = FollowUpStatusType.Info,
344+
type = FollowUpTypes.MODIFY_DEFAULT_SOURCE_FOLDER
345+
),
346+
FollowUp(
347+
icon = FollowUpIcons.Cancel,
348+
pillText = message("general.cancel"),
349+
prompt = message("general.cancel"),
350+
status = FollowUpStatusType.Error,
351+
type = FollowUpTypes.CANCEL_FOLDER_SELECTION
352+
),
353+
)
354+
)
355+
356+
messenger.sendChatInputEnabledMessage(tabId, false)
357+
}
358+
333359
override suspend fun processLinkClick(message: IncomingDocMessage.ClickedLink) {
334360
BrowserUtil.browse(message.link)
335361
}
@@ -894,21 +920,31 @@ class DocController(
894920
val projectRoot = session.context.projectRoot
895921

896922
withContext(EDT) {
923+
messenger.sendAnswer(
924+
tabId = tabId,
925+
messageType = DocMessageType.Answer,
926+
message = message("amazonqDoc.prompt.choose_folder_to_continue")
927+
)
928+
897929
val selectedFolder = selectFolder(context.project, currentSourceFolder)
898930
// No folder was selected
899931
if (selectedFolder == null) {
900932
logger.info { "Cancelled dialog and not selected any folder" }
933+
promptForRetryFolderSelection(
934+
tabId,
935+
message("amazonqDoc.prompt.canceled_source_folder_selection")
936+
)
937+
901938
return@withContext
902939
}
903940

904941
// The folder is not in the workspace
905942
if (!selectedFolder.path.startsWith(projectRoot.path)) {
906943
logger.info { "Selected folder not in workspace: ${selectedFolder.path}" }
907944

908-
messenger.sendAnswer(
909-
tabId = tabId,
910-
messageType = DocMessageType.Answer,
911-
message = message("amazonqFeatureDev.follow_up.incorrect_source_folder"),
945+
promptForRetryFolderSelection(
946+
tabId,
947+
message("amazonqFeatureDev.follow_up.incorrect_source_folder")
912948
)
913949
return@withContext
914950
}

plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonqDoc/messages/DocMessage.kt

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -179,6 +179,15 @@ data class FolderConfirmationMessage(
179179
type = "folderConfirmationMessage"
180180
)
181181

182+
data class RetryChangeFolderMessage(
183+
@JsonProperty("tabID") override val tabId: String,
184+
val message: String,
185+
val followUps: List<FollowUp>?,
186+
) : UiMessage(
187+
tabId = tabId,
188+
type = "retryChangeFolderMessage"
189+
)
190+
182191
// this should come from mynah?
183192
data class ChatItemButton(
184193
val id: String,

plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonqDoc/messages/DocMessagePublisherExtensions.kt

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -208,6 +208,16 @@ suspend fun MessagePublisher.sendFolderConfirmationMessage(
208208
)
209209
}
210210

211+
suspend fun MessagePublisher.sendRetryChangeFolderMessage(
212+
tabId: String,
213+
message: String,
214+
followUps: List<FollowUp>,
215+
) {
216+
this.publish(
217+
RetryChangeFolderMessage(tabId = tabId, message = message, followUps = followUps)
218+
)
219+
}
220+
211221
suspend fun MessagePublisher.sendUpdatePromptProgress(tabId: String, progressField: ProgressField?) {
212222
this.publish(
213223
PromptProgressMessage(tabId, progressField)

plugins/amazonq/mynah-ui/src/mynah-ui/ui/apps/docChatConnector.ts

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,21 @@ export class Connector {
170170
}
171171
}
172172

173+
private processRetryChangeFolderMessage = async (messageData: any): Promise<void> => {
174+
if (this.onChatAnswerReceived !== undefined) {
175+
const answer: ChatItem = {
176+
type: ChatItemType.ANSWER,
177+
body: messageData.message ?? undefined,
178+
messageId: messageData.messageID ?? messageData.triggerID ?? '',
179+
followUp: {
180+
text: '',
181+
options: messageData.followUps,
182+
},
183+
}
184+
this.onChatAnswerReceived(messageData.tabID, answer)
185+
}
186+
}
187+
173188
private processChatMessage = async (messageData: any): Promise<void> => {
174189
if (this.onChatAnswerReceived !== undefined) {
175190
const answer: ChatItem = {
@@ -263,6 +278,11 @@ export class Connector {
263278
return
264279
}
265280

281+
if (messageData.type === 'retryChangeFolderMessage') {
282+
await this.processRetryChangeFolderMessage(messageData)
283+
return
284+
}
285+
266286
if (messageData.type === 'chatMessage') {
267287
await this.processChatMessage(messageData)
268288
return

plugins/core/resources/resources/software/aws/toolkits/resources/MessagesBundle.properties

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,8 @@ amazonqDoc.progress_message.generating=Generating documentation
5858
amazonqDoc.progress_message.scanning=Scanning source files
5959
amazonqDoc.progress_message.summarizing=Summarizing source files
6060
amazonqDoc.progress_message.updating=Okay, I'm updating the README.
61+
amazonqDoc.prompt.canceled_source_folder_selection=It looks like you didn't choose a folder. Choose a folder to continue.
62+
amazonqDoc.prompt.choose_folder_to_continue=Choose a folder to continue
6163
amazonqDoc.prompt.create=Create a README
6264
amazonqDoc.prompt.create.confirmation=Create a README for this project?
6365
amazonqDoc.prompt.folder.change=Change folder

0 commit comments

Comments
 (0)