@aws/lsp-codewhisperer 0.0.72 → 0.0.73

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Changelog
 
+## [0.0.73](https://github.com/aws/language-servers/compare/lsp-codewhisperer/v0.0.72...lsp-codewhisperer/v0.0.73) (2025-08-11)
+
+
+### Features
+
+* **amazonq:** read tool ui revamp ([c65428b](https://github.com/aws/language-servers/commit/c65428bab2cf5e47badf1e3a9453babcf881e60c))
+
+
+### Bug Fixes
+
+* **amazonq:** add fallback classpath generation ([#2077](https://github.com/aws/language-servers/issues/2077)) ([3a6ef14](https://github.com/aws/language-servers/commit/3a6ef14e78fa2e75b837bba6524751d65038f416))
+* **amazonq:** emit failed status for amazonq_invokeLLM ([#2071](https://github.com/aws/language-servers/issues/2071)) ([ee52a41](https://github.com/aws/language-servers/commit/ee52a41bc869b275fff708d7955b59f43b93bbd4))
+* **amazonq:** fix fallout of [#2051](https://github.com/aws/language-servers/issues/2051) ([#2057](https://github.com/aws/language-servers/issues/2057)) ([565066b](https://github.com/aws/language-servers/commit/565066bb61adda60333c9646db958d4208bcc8af))
+* **amazonq:** leverage lcs to find the chars added and removed ([#2092](https://github.com/aws/language-servers/issues/2092)) ([40379a8](https://github.com/aws/language-servers/commit/40379a887f8d42cc184239ca3175b4e673cc5286))
+* **amazonq:** skips continuous monitoring when WCS sees workspace as idle ([#2066](https://github.com/aws/language-servers/issues/2066)) ([9cb959d](https://github.com/aws/language-servers/commit/9cb959d4cc450d0907f8bf5265ba01d2aa68bcd0))
+* creating a new sesion for Edits trigger with next token ([#2094](https://github.com/aws/language-servers/issues/2094)) ([1da8730](https://github.com/aws/language-servers/commit/1da8730b6ed6ad53b6561368bf722e56d59596a4))
+* remove edit cache logic ([#2079](https://github.com/aws/language-servers/issues/2079)) ([9bc5b9c](https://github.com/aws/language-servers/commit/9bc5b9c1d77e5fee6f518f7f5016d3a0043a5a77))
+* sessionManager misused because there are 2 types of manager now ([#2090](https://github.com/aws/language-servers/issues/2090)) ([8db059a](https://github.com/aws/language-servers/commit/8db059ab83d94fd7c3ba3eb265044add31c80aea))
+* update client name to support Sagemaker AI origin for agentic chat ([#2093](https://github.com/aws/language-servers/issues/2093)) ([a746fe8](https://github.com/aws/language-servers/commit/a746fe845d5e09563b475f01ce44059dca9fd10f))
+
 ## [0.0.72](https://github.com/aws/language-servers/compare/lsp-codewhisperer/v0.0.71...lsp-codewhisperer/v0.0.72) (2025-08-06)
 
 

package/out/language-server/agenticChat/agenticChatController.js

@@ -65,6 +65,7 @@ const path_1 = require("@aws/lsp-core/out/util/path");
 const agenticChatControllerHelper_1 = require("./utils/agenticChatControllerHelper");
 const activeUserTracker_1 = require("../../shared/activeUserTracker");
 const displayFindings_1 = require("./tools/qCodeAnalysis/displayFindings");
+const IdleWorkspaceManager_1 = require("../workspaceContext/IdleWorkspaceManager");
 class AgenticChatController {
     #features;
     #chatSessionManagementService;
@@ -465,6 +466,7 @@ class AgenticChatController {
     async onChatPrompt(params, token) {
         // Phase 1: Initial Setup - This happens only once
         params.prompt.prompt = (0, utils_2.sanitizeInput)(params.prompt.prompt || '');
+        IdleWorkspaceManager_1.IdleWorkspaceManager.recordActivityTimestamp();
         const maybeDefaultResponse = !params.prompt.command && (0, utils_1.getDefaultChatResponse)(params.prompt.prompt);
         if (maybeDefaultResponse) {
             return maybeDefaultResponse;
@@ -844,6 +846,8 @@ class AgenticChatController {
                 shouldDisplayMessage = false;
                 // set the in progress tool use UI status to Error
                 await chatResultStream.updateOngoingProgressResult('Error');
+                // emit invokeLLM event with status Failed for timeout calls
+                this.#telemetryController.emitAgencticLoop_InvokeLLM(response.$metadata.requestId, conversationId, 'AgenticChat', undefined, undefined, 'Failed', this.#features.runtime.serverInfo.version ?? '', session.modelId, llmLatency, this.#toolCallLatencies, this.#timeToFirstChunk, this.#timeBetweenChunks, session.pairProgrammingMode, this.#abTestingAllocation?.experimentName, this.#abTestingAllocation?.userVariation);
                 continue;
             }
             // Add the current assistantResponse message to the history DB
@@ -881,7 +885,7 @@ class AgenticChatController {
             if (pendingToolUses.length === 0) {
                 this.recordChunk('agent_loop_done');
                 // No more tool uses, we're done
-                this.#telemetryController.emitAgencticLoop_InvokeLLM(response.$metadata.requestId, conversationId, 'AgenticChat', undefined, undefined, 'Succeeded', this.#features.runtime.serverInfo.version ?? '', session.modelId, llmLatency, this.#toolCallLatencies, this.#timeToFirstChunk, this.#timeBetweenChunks, session.pairProgrammingMode, this.#abTestingAllocation?.experimentName, this.#abTestingAllocation?.userVariation);
+                this.#telemetryController.emitAgencticLoop_InvokeLLM(response.$metadata.requestId, conversationId, 'AgenticChat', undefined, undefined, result.success ? 'Succeeded' : 'Failed', this.#features.runtime.serverInfo.version ?? '', session.modelId, llmLatency, this.#toolCallLatencies, this.#timeToFirstChunk, this.#timeBetweenChunks, session.pairProgrammingMode, this.#abTestingAllocation?.experimentName, this.#abTestingAllocation?.userVariation);
                 finalResult = result;
                 break;
             }
@@ -1112,8 +1116,7 @@ class AgenticChatController {
                 this.#toolStartTime = Date.now();
                 // remove progress UI
                 await chatResultStream.removeResultBlockAndUpdateUI(agenticChatResultStream_1.progressPrefix + toolUse.toolUseId);
-                // fsRead and listDirectory write to an existing card and could show nothing in the current position
-                if (![toolConstants_1.FS_WRITE, toolConstants_1.FS_REPLACE, toolConstants_1.FS_READ, toolConstants_1.LIST_DIRECTORY].includes(toolUse.name)) {
+                if (![toolConstants_1.FS_WRITE, toolConstants_1.FS_REPLACE].includes(toolUse.name)) {
                     await this.#showUndoAllIfRequired(chatResultStream, session);
                 }
                 // fsWrite can take a long time, so we render fsWrite  Explanatory upon partial streaming responses.
@@ -1270,10 +1273,14 @@ class AgenticChatController {
                 switch (toolUse.name) {
                     case toolConstants_1.FS_READ:
                     case toolConstants_1.LIST_DIRECTORY:
+                        const readToolResult = await this.#processReadTool(toolUse, chatResultStream);
+                        if (readToolResult) {
+                            await chatResultStream.writeResultBlock(readToolResult);
+                        }
+                        break;
                     case toolConstants_1.FILE_SEARCH:
-                        const initialListDirResult = this.#processReadOrListOrSearch(toolUse, chatResultStream);
-                        if (initialListDirResult) {
-                            await chatResultStream.writeResultBlock(initialListDirResult);
+                        if ((0, fileSearch_1.isFileSearchParams)(toolUse.input)) {
+                            await this.#processFileSearchTool(toolUse.input, toolUse.toolUseId, result, chatResultStream);
                         }
                         break;
                     // no need to write tool result for listDir,fsRead,fileSearch into chat stream
@@ -1617,7 +1624,6 @@ class AgenticChatController {
             return;
         }
         const toolMsgId = toolUse.toolUseId;
-        const chatMsgId = chatResultStream.getResult().messageId;
         let headerEmitted = false;
         const initialHeader = {
             body: 'shell',
@@ -1649,12 +1655,6 @@ class AgenticChatController {
                     body: '```',
                     header: completedHeader,
                 });
-                await chatResultStream.writeResultBlock({
-                    type: 'answer',
-                    messageId: chatMsgId,
-                    body: '',
-                    header: undefined,
-                });
                 this.#stoppedToolUses.add(toolMsgId);
             },
         });
@@ -2085,72 +2085,123 @@ class AgenticChatController {
             },
         };
     }
-    #processReadOrListOrSearch(toolUse, chatResultStream) {
-        let messageIdToUpdate = toolUse.toolUseId;
-        const currentId = chatResultStream.getMessageIdToUpdateForTool(toolUse.name);
-        if (currentId) {
-            messageIdToUpdate = currentId;
-        }
-        else {
-            chatResultStream.setMessageIdToUpdateForTool(toolUse.name, messageIdToUpdate);
-        }
+    async #processFileSearchTool(toolInput, toolUseId, result, chatResultStream) {
+        if (typeof result.output.content !== 'string')
+            return;
+        const { queryName, path: inputPath } = toolInput;
+        const resultCount = result.output.content
+            .split('\n')
+            .filter(line => line.trim().startsWith('[F]') || line.trim().startsWith('[D]')).length;
+        const chatMessage = {
+            type: 'tool',
+            messageId: toolUseId,
+            header: {
+                body: `Searched for "${queryName}" in `,
+                icon: 'search',
+                status: {
+                    text: `${resultCount} result${resultCount !== 1 ? 's' : ''} found`,
+                },
+                fileList: {
+                    filePaths: [inputPath],
+                    details: {
+                        [inputPath]: {
+                            description: inputPath,
+                            visibleName: path.basename(inputPath),
+                            clickable: false,
+                        },
+                    },
+                },
+            },
+        };
+        await chatResultStream.writeResultBlock(chatMessage);
+    }
+    async #processReadTool(toolUse, chatResultStream) {
         let currentPaths = [];
         if (toolUse.name === toolConstants_1.FS_READ) {
-            currentPaths = toolUse.input?.paths;
+            currentPaths = toolUse.input?.paths || [];
         }
-        else {
-            currentPaths.push(toolUse.input?.path);
+        else if (toolUse.name === toolConstants_1.LIST_DIRECTORY) {
+            const singlePath = toolUse.input?.path;
+            if (singlePath) {
+                currentPaths = [singlePath];
+            }
         }
-        if (!currentPaths)
-            return;
-        for (const currentPath of currentPaths) {
-            const existingPaths = chatResultStream.getMessageOperation(messageIdToUpdate)?.filePaths || [];
-            // Check if path already exists in the list
-            const isPathAlreadyProcessed = existingPaths.some(path => path.relativeFilePath === currentPath);
-            if (!isPathAlreadyProcessed) {
-                const currentFileDetail = {
-                    relativeFilePath: currentPath,
-                    lineRanges: [{ first: -1, second: -1 }],
-                };
-                chatResultStream.addMessageOperation(messageIdToUpdate, toolUse.name, [
-                    ...existingPaths,
-                    currentFileDetail,
-                ]);
+        else if (toolUse.name === toolConstants_1.FILE_SEARCH) {
+            const queryName = toolUse.input?.queryName;
+            if (queryName) {
+                currentPaths = [queryName];
             }
         }
+        else {
+            return;
+        }
+        if (currentPaths.length === 0)
+            return;
+        // Check if the last message is the same tool type
+        const lastMessage = chatResultStream.getLastMessage();
+        const isSameToolType = lastMessage?.type === 'tool' && lastMessage.header?.icon === this.#toolToIcon(toolUse.name);
+        let allPaths = currentPaths;
+        if (isSameToolType && lastMessage.messageId) {
+            // Combine with existing paths and overwrite the last message
+            const existingPaths = lastMessage.header?.fileList?.filePaths || [];
+            allPaths = [...existingPaths, ...currentPaths];
+            const blockId = chatResultStream.getMessageBlockId(lastMessage.messageId);
+            if (blockId !== undefined) {
+                // Create the updated message with combined paths
+                const updatedMessage = this.#createFileListToolMessage(toolUse, allPaths, lastMessage.messageId);
+                // Overwrite the existing block
+                await chatResultStream.overwriteResultBlock(updatedMessage, blockId);
+                return undefined; // Don't return a message since we already wrote it
+            }
+        }
+        // Create new message with current paths
+        return this.#createFileListToolMessage(toolUse, allPaths, toolUse.toolUseId);
+    }
+    #createFileListToolMessage(toolUse, filePaths, messageId) {
+        const itemCount = filePaths.length;
         let title;
-        const itemCount = chatResultStream.getMessageOperation(messageIdToUpdate)?.filePaths.length;
-        const filePathsPushed = chatResultStream.getMessageOperation(messageIdToUpdate)?.filePaths ?? [];
-        if (!itemCount) {
+        if (itemCount === 0) {
             title = 'Gathering context';
         }
         else {
             title =
                 toolUse.name === toolConstants_1.FS_READ
                     ? `${itemCount} file${itemCount > 1 ? 's' : ''} read`
-                    : toolUse.name === toolConstants_1.FILE_SEARCH
-                        ? `${itemCount} ${itemCount === 1 ? 'directory' : 'directories'} searched`
-                        : `${itemCount} ${itemCount === 1 ? 'directory' : 'directories'} listed`;
+                    : toolUse.name === toolConstants_1.LIST_DIRECTORY
+                        ? `${itemCount} ${itemCount === 1 ? 'directory' : 'directories'} listed`
+                        : '';
         }
         const details = {};
-        for (const item of filePathsPushed) {
-            details[item.relativeFilePath] = {
-                lineRanges: item.lineRanges,
-                description: item.relativeFilePath,
+        for (const filePath of filePaths) {
+            details[filePath] = {
+                description: filePath,
+                visibleName: path.basename(filePath),
+                clickable: toolUse.name === toolConstants_1.FS_READ,
             };
         }
-        const fileList = {
-            rootFolderTitle: title,
-            filePaths: filePathsPushed.map(item => item.relativeFilePath),
-            details,
-        };
         return {
             type: 'tool',
-            fileList,
-            messageId: messageIdToUpdate,
-            body: '',
+            header: {
+                body: title,
+                icon: this.#toolToIcon(toolUse.name),
+                fileList: {
+                    filePaths,
+                    details,
+                },
+            },
+            messageId,
         };
     }
+    #toolToIcon(toolName) {
+        switch (toolName) {
+            case toolConstants_1.FS_READ:
+                return 'eye';
+            case toolConstants_1.LIST_DIRECTORY:
+                return 'check-list';
+            default:
+                return undefined;
+        }
+    }
     /**
      * Process grep search results and format them for display in the chat UI
      */
@@ -2158,14 +2209,7 @@ class AgenticChatController {
         if (toolUse.name !== toolConstants_1.GREP_SEARCH) {
             return undefined;
         }
-        let messageIdToUpdate = toolUse.toolUseId;
-        const currentId = chatResultStream.getMessageIdToUpdateForTool(toolUse.name);
-        if (currentId) {
-            messageIdToUpdate = currentId;
-        }
-        else {
-            chatResultStream.setMessageIdToUpdateForTool(toolUse.name, messageIdToUpdate);
-        }
+        const messageIdToUpdate = toolUse.toolUseId;
         // Extract search results from the tool output
         const output = result.output.content;
         if (!output || !output.fileMatches || !Array.isArray(output.fileMatches)) {
@@ -2414,6 +2458,7 @@ class AgenticChatController {
         const metric = new metric_1.Metric({
             cwsprChatConversationType: 'Chat',
         });
+        IdleWorkspaceManager_1.IdleWorkspaceManager.recordActivityTimestamp();
         const triggerContext = await this.#getInlineChatTriggerContext(params);
         let response;
         let requestInput;