From 9b51191213e508bcb17a70073913b9124d54556f Mon Sep 17 00:00:00 2001 From: Blake Lazarine Date: Thu, 17 Jul 2025 16:42:56 -0700 Subject: [PATCH 1/3] fix(amazonq): rename QCodeReview tool to CodeReview --- packages/amazonq/src/lsp/chat/messages.ts | 2 +- packages/amazonq/src/lsp/client.ts | 2 +- packages/core/src/codewhisperer/models/constants.ts | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/amazonq/src/lsp/chat/messages.ts b/packages/amazonq/src/lsp/chat/messages.ts index f869bbe0da3..607c3d7bdc0 100644 --- a/packages/amazonq/src/lsp/chat/messages.ts +++ b/packages/amazonq/src/lsp/chat/messages.ts @@ -671,7 +671,7 @@ async function handlePartialResult( ) { const decryptedMessage = await decryptResponse(partialResult, encryptionKey) - // This is to filter out the message containing findings from qCodeReview tool to update CodeIssues panel + // This is to filter out the message containing findings from CodeReview tool to update CodeIssues panel decryptedMessage.additionalMessages = decryptedMessage.additionalMessages?.filter( (message) => !(message.messageId !== undefined && message.messageId.endsWith(CodeWhispererConstants.findingsSuffix)) diff --git a/packages/amazonq/src/lsp/client.ts b/packages/amazonq/src/lsp/client.ts index 98427d17276..5799a87f748 100644 --- a/packages/amazonq/src/lsp/client.ts +++ b/packages/amazonq/src/lsp/client.ts @@ -168,7 +168,7 @@ export async function startLanguageServer( reroute: true, modelSelection: true, workspaceFilePath: vscode.workspace.workspaceFile?.fsPath, - qCodeReviewInChat: true, + CodeReviewInChat: false, }, window: { notifications: true, diff --git a/packages/core/src/codewhisperer/models/constants.ts b/packages/core/src/codewhisperer/models/constants.ts index b0403e4fd3c..4a11d6e98b2 100644 --- a/packages/core/src/codewhisperer/models/constants.ts +++ b/packages/core/src/codewhisperer/models/constants.ts @@ -907,4 +907,4 @@ export const predictionTrackerDefaultConfig = { maxSupplementalContext: 15, } -export const findingsSuffix = '_qCodeReviewFindings' +export const findingsSuffix = '_CodeReviewFindings' From dfdf3772146f37061963983341cc3f29d4500db3 Mon Sep 17 00:00:00 2001 From: Blake Lazarine Date: Thu, 17 Jul 2025 17:24:12 -0700 Subject: [PATCH 2/3] fix(amazonq): fix issue with casing --- packages/amazonq/src/lsp/client.ts | 2 +- packages/core/src/codewhisperer/models/constants.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/amazonq/src/lsp/client.ts b/packages/amazonq/src/lsp/client.ts index 5799a87f748..ce83938d158 100644 --- a/packages/amazonq/src/lsp/client.ts +++ b/packages/amazonq/src/lsp/client.ts @@ -168,7 +168,7 @@ export async function startLanguageServer( reroute: true, modelSelection: true, workspaceFilePath: vscode.workspace.workspaceFile?.fsPath, - CodeReviewInChat: false, + codeReviewInChat: false, }, window: { notifications: true, diff --git a/packages/core/src/codewhisperer/models/constants.ts b/packages/core/src/codewhisperer/models/constants.ts index 4a11d6e98b2..9e1eb2b7f94 100644 --- a/packages/core/src/codewhisperer/models/constants.ts +++ b/packages/core/src/codewhisperer/models/constants.ts @@ -907,4 +907,4 @@ export const predictionTrackerDefaultConfig = { maxSupplementalContext: 15, } -export const findingsSuffix = '_CodeReviewFindings' +export const findingsSuffix = '_codeReviewFindings' From e62889f98f5cd43ccd58b0d94a8a0691a947139e Mon Sep 17 00:00:00 2001 From: mkovelam Date: Fri, 18 Jul 2025 15:59:56 -0700 Subject: [PATCH 3/3] feat(amazonq): enabling code review tool --- packages/amazonq/src/lsp/client.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/amazonq/src/lsp/client.ts b/packages/amazonq/src/lsp/client.ts index f577966a342..dce60a8a832 100644 --- a/packages/amazonq/src/lsp/client.ts +++ b/packages/amazonq/src/lsp/client.ts @@ -186,7 +186,7 @@ export async function startLanguageServer( reroute: true, modelSelection: true, workspaceFilePath: vscode.workspace.workspaceFile?.fsPath, - codeReviewInChat: false, + codeReviewInChat: true, }, window: { notifications: true,