fix: hide non-user-generated messages when reloading history

Author: tsmithsz

server/aws-lsp-codewhisperer/src/language-server/agenticChat/agenticChatController.ts

@@ -524,6 +524,7 @@ export class AgenticChatController implements ChatHandlers {
         let currentRequestInput = { ...initialRequestInput }
         let finalResult: Result<AgenticChatResultWithMetadata, string> | null = null
         let iterationCount = 0
+        let shouldDisplayMessage = true
         metric.recordStart()
 
         while (iterationCount < maxAgentLoopIterations) {
@@ -574,8 +575,10 @@ export class AgenticChatController implements ChatHandlers {
                     userIntent: currentMessage.userInputMessage?.userIntent,
                     origin: currentMessage.userInputMessage?.origin,
                     userInputMessageContext: currentMessage.userInputMessage?.userInputMessageContext,
+                    shouldDisplayMessage: shouldDisplayMessage,
                 })
             }
+            shouldDisplayMessage = true
 
             // Phase 4: Response Processing
             const result = await this.#processGenerateAssistantResponseResponseWithTimeout(
@@ -620,6 +623,7 @@ export class AgenticChatController implements ChatHandlers {
                             name: result.data!.toolUses[k].name,
                             input: result.data!.toolUses[k].input,
                         })),
+                    shouldDisplayMessage: shouldDisplayMessage,
                 })
             } else {
                 this.#features.logging.warn('No ChatResult body in response, skipping adding to history')
@@ -642,6 +646,7 @@ export class AgenticChatController implements ChatHandlers {
                 if (toolResults.some(toolResult => this.#shouldSendBackErrorContent(toolResult))) {
                     content = 'There was an error processing one or more tool uses. Try again, do not apologize.'
                 }
+                shouldDisplayMessage = false
                 metric.setDimension('cwsprChatConversationType', 'AgenticChatWithToolUse')
             } else {
                 // Send an error card to UI?
@@ -653,6 +658,7 @@ export class AgenticChatController implements ChatHandlers {
                 if (result.error.startsWith('ToolUse input is invalid JSON:')) {
                     content =
                         'Your toolUse input is incomplete because it is too large. Break this task down into multiple tool uses with smaller input. Do not apologize.'
+                    shouldDisplayMessage = false
                 }
             }
             currentRequestInput = this.#updateRequestInputWithToolResults(currentRequestInput, toolResults, content)

server/aws-lsp-codewhisperer/src/language-server/agenticChat/tabBarController.ts

@@ -218,7 +218,7 @@ export class TabBarController {
     async restoreTab(selectedTab?: Tab | null) {
         if (selectedTab) {
             const messages = selectedTab.conversations.flatMap((conv: Conversation) =>
-                conv.messages.flatMap(msg => messageToChatMessage(msg))
+                conv.messages.filter(msg => msg.shouldDisplayMessage != false).flatMap(msg => messageToChatMessage(msg))
             )
 
             const { tabId } = await this.#features.chat.openTab({ newTabOptions: { data: { messages } } })

server/aws-lsp-codewhisperer/src/language-server/agenticChat/tools/chatDb/chatDb.ts

@@ -313,8 +313,9 @@ export class ChatDatabase {
 
             const tabData = historyId ? tabCollection.findOne({ historyId }) : undefined
             const tabTitle =
-                (message.type === 'prompt' && message.body.trim().length > 0 ? message.body : tabData?.title) ||
-                'Amazon Q Chat'
+                (message.type === 'prompt' && message.shouldDisplayMessage !== false && message.body.trim().length > 0
+                    ? message.body
+                    : tabData?.title) || 'Amazon Q Chat'
             message = this.formatChatHistoryMessage(message)
             if (tabData) {
                 this.#features.logging.log(`Updating existing tab with historyId=${historyId}`)