@aws/lsp-codewhisperer 0.0.82 → 0.0.83

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # Changelog
 
+## [0.0.83](https://github.com/aws/language-servers/compare/lsp-codewhisperer/v0.0.82...lsp-codewhisperer/v0.0.83) (2025-10-01)
+
+
+### Bug Fixes
+
+* **amazonq:** escaping user input to mitigate xss issue ([#2360](https://github.com/aws/language-servers/issues/2360)) ([45b86be](https://github.com/aws/language-servers/commit/45b86bef1a93cf9ced6fbf0c222cf5410de04c81))
+* **amazonq:** fix to add opt-out header to streaming client ([#2365](https://github.com/aws/language-servers/issues/2365)) ([692e77b](https://github.com/aws/language-servers/commit/692e77bc99770ac7d676928e95e3dc43bb91e7f0))
+* **amazonq:** handle IAM credentials expiration field to be aws sdk versions compatible and add refresh logic to codewhisperer IAM client ([#2349](https://github.com/aws/language-servers/issues/2349)) ([5eb3768](https://github.com/aws/language-servers/commit/5eb3768bf020d61d0ade767d62e13839048146e4))
+* **amazonq:** send full finding details to plugin, partial to agent ([#2356](https://github.com/aws/language-servers/issues/2356)) ([961e6ca](https://github.com/aws/language-servers/commit/961e6ca11b122481685f9f65b3da14c6a2497cc4))
+* improve history management ([#2312](https://github.com/aws/language-servers/issues/2312)) ([#2357](https://github.com/aws/language-servers/issues/2357)) ([e7aa2a6](https://github.com/aws/language-servers/commit/e7aa2a6545bcb1a8238abfde69a05432be0b6615))
+* optimize memory bank token usage and add new tab support ([#2366](https://github.com/aws/language-servers/issues/2366)) ([3057d56](https://github.com/aws/language-servers/commit/3057d56e4a3047d1715d6e3560e9f934d1de469c))
+* private package mapping during artifact generation ([#2348](https://github.com/aws/language-servers/issues/2348)) ([d56bfa1](https://github.com/aws/language-servers/commit/d56bfa191954fac8068e2bf390c2d0b88ef8b168))
+* trim new line when emitting error message ([#2359](https://github.com/aws/language-servers/issues/2359)) ([d8733a7](https://github.com/aws/language-servers/commit/d8733a75487f74815302b838802eccbf3ffec55e))
+
+
+### Reverts
+
+* fix to add opt-out header to streaming client ([#2365](https://github.com/aws/language-servers/issues/2365)) ([#2370](https://github.com/aws/language-servers/issues/2370)) ([b29478f](https://github.com/aws/language-servers/commit/b29478fa1ecc58e331ff330ff79f46b0d8c38d9e))
+
+
+### Dependencies
+
+* The following workspace dependencies were updated
+  * dependencies
+    * @aws/lsp-core bumped from ^0.0.15 to ^0.0.16
+
 ## [0.0.82](https://github.com/aws/language-servers/compare/lsp-codewhisperer/v0.0.81...lsp-codewhisperer/v0.0.82) (2025-09-24)
 
 

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

@@ -66,7 +66,6 @@ const activeUserTracker_1 = require("../../shared/activeUserTracker");
 const displayFindings_1 = require("./tools/qCodeAnalysis/displayFindings");
 const constants_3 = require("../../shared/constants");
 const IdleWorkspaceManager_1 = require("../workspaceContext/IdleWorkspaceManager");
-const escapeHTML = require("escape-html");
 const semanticSearch_1 = require("./tools/workspaceContext/semanticSearch");
 const memoryBankController_1 = require("./context/memorybank/memoryBankController");
 class AgenticChatController {
@@ -565,11 +564,17 @@ class AgenticChatController {
     }
     async onChatPrompt(params, token) {
         // Phase 1: Initial Setup - This happens only once
-        params.prompt.prompt = (0, utils_2.sanitizeInput)(params.prompt.prompt || '');
+        params.prompt.prompt = (0, utils_2.sanitizeInput)(params.prompt.prompt || '', true);
         IdleWorkspaceManager_1.IdleWorkspaceManager.recordActivityTimestamp();
+        const sessionResult = this.#chatSessionManagementService.getSession(params.tabId);
+        const { data: session, success } = sessionResult;
+        if (!success) {
+            return new server_interface_1.ResponseError(protocol_1.ErrorCodes.InternalError, sessionResult.error);
+        }
         // Memory Bank Creation Flow - Delegate to MemoryBankController
         if (this.#memoryBankController.isMemoryBankCreationRequest(params.prompt.prompt)) {
             this.#features.logging.info(`Memory Bank creation request detected for tabId: ${params.tabId}`);
+            session.isMemoryBankGeneration = true;
             // Store original prompt to prevent data loss on failure
             const originalPrompt = params.prompt.prompt;
             try {
@@ -639,17 +644,14 @@ class AgenticChatController {
                 this.#features.logging.error(`Memory Bank preparation failed: ${error}`);
                 // Restore original prompt to ensure no data loss
                 params.prompt.prompt = originalPrompt;
+                // Reset memory bank flag since preparation failed
+                session.isMemoryBankGeneration = false;
             }
         }
         const maybeDefaultResponse = !params.prompt.command && (0, utils_1.getDefaultChatResponse)(params.prompt.prompt);
         if (maybeDefaultResponse) {
             return maybeDefaultResponse;
         }
-        const sessionResult = this.#chatSessionManagementService.getSession(params.tabId);
-        const { data: session, success } = sessionResult;
-        if (!success) {
-            return new server_interface_1.ResponseError(protocol_1.ErrorCodes.InternalError, sessionResult.error);
-        }
         const compactIds = session.getAllDeferredCompactMessageIds();
         await this.#invalidateCompactCommand(params.tabId, compactIds);
         session.rejectAllDeferredToolExecutions(new toolShared_1.ToolApprovalException('Command ignored: new prompt', false));
@@ -791,8 +793,7 @@ class AgenticChatController {
      * Runs the compaction, making requests and processing tool uses until completion
      */
     #shouldCompact(currentRequestCount) {
-        // 80% of 570K limit
-        if (currentRequestCount > 456_000) {
+        if (currentRequestCount > constants_2.COMPACTION_CHARACTER_THRESHOLD) {
             this.#debug(`Current request total character count is: ${currentRequestCount}, prompting user to compact`);
             return true;
         }
@@ -821,7 +822,7 @@ class AgenticChatController {
         if (currentMessage) {
             //  Get and process the messages from history DB to maintain invariants for service requests
             try {
-                const { history: historyMessages, historyCount: historyCharCount } = this.#chatHistoryDb.fixAndGetHistory(tabId, conversationIdentifier ?? '', currentMessage, []);
+                const { history: historyMessages, historyCount: historyCharCount } = this.#chatHistoryDb.fixAndGetHistory(tabId, currentMessage, []);
                 messages = historyMessages;
                 characterCount = historyCharCount;
             }
@@ -939,7 +940,7 @@ class AgenticChatController {
             if (currentMessage) {
                 //  Get and process the messages from history DB to maintain invariants for service requests
                 try {
-                    const { history: historyMessages, historyCount: historyCharacterCount, currentCount: currentInputCount, } = this.#chatHistoryDb.fixAndGetHistory(tabId, conversationId, currentMessage, pinnedContextMessages);
+                    const { history: historyMessages, historyCount: historyCharacterCount, currentCount: currentInputCount, } = this.#chatHistoryDb.fixAndGetHistory(tabId, currentMessage, pinnedContextMessages);
                     messages = historyMessages;
                     currentRequestCount = currentInputCount + historyCharacterCount;
                     this.#debug(`Request total character count: ${currentRequestCount}`);
@@ -980,7 +981,7 @@ class AgenticChatController {
                 }
                 else {
                     this.#chatHistoryDb.addMessage(tabId, 'cwc', conversationIdentifier, {
-                        body: escapeHTML(currentMessage.userInputMessage?.content ?? ''),
+                        body: currentMessage.userInputMessage?.content ?? '',
                         type: 'prompt',
                         userIntent: currentMessage.userInputMessage?.userIntent,
                         origin: currentMessage.userInputMessage?.origin,
@@ -1322,7 +1323,9 @@ class AgenticChatController {
                             [toolConstants_1.FILE_SEARCH]: { Tool: fileSearch_1.FileSearch },
                         };
                         const { Tool } = toolMap[toolUse.name];
-                        const tool = new Tool(this.#features);
+                        const tool = toolUse.name === toolConstants_1.FS_READ && session.isMemoryBankGeneration
+                            ? new Tool(this.#features, constants_2.FSREAD_MEMORY_BANK_MAX_PER_FILE, constants_2.FSREAD_MEMORY_BANK_MAX_TOTAL)
+                            : new Tool(this.#features);
                         // For MCP tools, get the permission from McpManager
                         // const permission = McpManager.instance.getToolPerm('Built-in', toolUse.name)
                         // If permission is 'alwaysAllow', we don't need to ask for acceptance
@@ -1428,6 +1431,25 @@ class AgenticChatController {
                 const ws = this.#getWritableStream(chatResultStream, toolUse);
                 const result = await this.#features.agent.runTool(toolUse.name, toolUse.input, token, ws);
                 let toolResultContent;
+                if (toolUse.name === codeReview_1.CodeReview.toolName) {
+                    // no need to write tool result for code review, this is handled by model via chat
+                    // Push result in message so that it is picked by IDE plugin to show in issues panel
+                    const codeReviewResult = result;
+                    if (codeReviewResult?.output?.kind === 'json' &&
+                        codeReviewResult.output.success &&
+                        codeReviewResult.output.content?.findingsByFile) {
+                        await chatResultStream.writeResultBlock({
+                            type: 'tool',
+                            messageId: toolUse.toolUseId + codeReviewConstants_1.CODE_REVIEW_FINDINGS_MESSAGE_SUFFIX,
+                            body: codeReviewResult.output.content.findingsByFile,
+                        });
+                        codeReviewResult.output.content = {
+                            codeReviewId: codeReviewResult.output.content.codeReviewId,
+                            message: codeReviewResult.output.content.message,
+                            findingsByFileSimplified: codeReviewResult.output.content.findingsByFileSimplified,
+                        };
+                    }
+                }
                 if (typeof result === 'string') {
                     toolResultContent = { text: result };
                 }
@@ -1499,18 +1521,6 @@ class AgenticChatController {
                         await chatResultStream.writeResultBlock(chatResult);
                         break;
                     case codeReview_1.CodeReview.toolName:
-                        // no need to write tool result for code review, this is handled by model via chat
-                        // Push result in message so that it is picked by IDE plugin to show in issues panel
-                        const codeReviewResult = result;
-                        if (codeReviewResult?.output?.kind === 'json' &&
-                            codeReviewResult.output.success &&
-                            codeReviewResult.output.content?.findingsByFile) {
-                            await chatResultStream.writeResultBlock({
-                                type: 'tool',
-                                messageId: toolUse.toolUseId + codeReviewConstants_1.CODE_REVIEW_FINDINGS_MESSAGE_SUFFIX,
-                                body: codeReviewResult.output.content.findingsByFile,
-                            });
-                        }
                         break;
                     case displayFindings_1.DisplayFindings.toolName:
                         // no need to write tool result for code review, this is handled by model via chat
@@ -2017,7 +2027,7 @@ class AgenticChatController {
             body: constants_2.COMPACTION_HEADER_BODY,
             buttons,
         };
-        const body = (0, constants_2.COMPACTION_BODY)(Math.round((characterCount / chatDb_1.MaxOverallCharacters) * 100));
+        const body = (0, constants_2.COMPACTION_BODY)(Math.round((characterCount / constants_2.MAX_OVERALL_CHARACTERS) * 100));
         return {
             type: 'tool',
             messageId,
@@ -2568,13 +2578,15 @@ class AgenticChatController {
                     .filter(prompt => prompt.length > 0)),
             },
         });
+        // Reset memory bank flag after completion
+        session.isMemoryBankGeneration = false;
         return chatResultStream.getResult();
     }
     /**
      * Handles errors that occur during the request
      */
     async #handleRequestError(conversationId, err, errorMessageId, tabId, metric, agenticCodingMode) {
-        const errorMessage = (0, utils_2.getErrorMsg)(err) ?? constants_2.GENERIC_ERROR_MS;
+        const errorMessage = ((0, utils_2.getErrorMsg)(err) ?? constants_2.GENERIC_ERROR_MS).replace(/[\r\n]+/g, ' '); // replace new lines with empty space
         const requestID = (0, utils_2.getRequestID)(err) ?? '';
         metric.setDimension('cwsprChatResponseCode', (0, utils_2.getHttpStatusCode)(err) ?? 0);
         metric.setDimension('languageServerVersion', this.#features.runtime.serverInfo.version);
@@ -2585,6 +2597,11 @@ class AgenticChatController {
         metric.metric.cwsprChatConversationId = conversationId;
         const errorCode = err.code ?? '';
         await this.#telemetryController.emitAddMessageMetric(tabId, metric.metric, 'Failed', errorMessage, errorCode);
+        // Reset memory bank flag on request error
+        const sessionResult = this.#chatSessionManagementService.getSession(tabId);
+        if (sessionResult.success) {
+            sessionResult.data.isMemoryBankGeneration = false;
+        }
         if ((0, utils_2.isUsageLimitError)(err)) {
             if (this.#paidTierMode !== 'paidtier') {
                 this.setPaidTierMode(tabId, 'freetier-limit');