Merge pull request #7731 from singhAws/code-review-tool

BlakeLazarine · web-flow · commit 61d328d4a0f9 · 2025-07-22T16:33:18.000-07:00
fix(amazonq): fix finding suppression, change icon, change tool name
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -41,7 +41,7 @@
         "skippedTestReport": "ts-node ./scripts/skippedTestReport.ts ./packages/amazonq/test/e2e/"
     },
     "devDependencies": {
-        "@aws-toolkits/telemetry": "^1.0.328",
+        "@aws-toolkits/telemetry": "^1.0.329",
         "@playwright/browser-chromium": "^1.43.1",
         "@stylistic/eslint-plugin": "^2.11.0",
         "@types/he": "^1.2.3",
diff --git a/packages/amazonq/src/lsp/chat/messages.ts b/packages/amazonq/src/lsp/chat/messages.ts
@@ -710,7 +710,7 @@ async function handlePartialResult<T extends ChatResult>(
 ) {
     const decryptedMessage = await decryptResponse<T>(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
@@ -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
@@ -907,4 +907,4 @@ export const predictionTrackerDefaultConfig = {
     maxSupplementalContext: 15,
 }
 
-export const findingsSuffix = '_qCodeReviewFindings'
+export const findingsSuffix = '_codeReviewFindings'

Original file line number	Diff line number	Diff line change
`@@ -907,4 +907,4 @@ export const predictionTrackerDefaultConfig = {`
`907`	`907`	`maxSupplementalContext: 15,`
`908`	`908`	`}`
`909`	`909`
`910`		`-export const findingsSuffix = '_qCodeReviewFindings'`
	`910`	`+export const findingsSuffix = '_codeReviewFindings'`