Merge master into feature/cwltail

Author: aws-toolkit-automation

packages/amazonq/.changes/next-release/Bug Fix-53ead65f-187e-47d9-9432-a48200b604d6.json

@@ -0,0 +1,4 @@
+{
+	"type": "Bug Fix",
+	"description": "Code Review: Cleaned up output logs when running /review"
+}

packages/core/src/codewhisperer/util/gitUtil.ts

@@ -3,8 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { showOutputMessage } from '../../shared/utilities/messages'
-import { getLogger, globals, removeAnsi } from '../../shared'
+import { getLogger, removeAnsi } from '../../shared'
 import { ChildProcess, ChildProcessOptions } from '../../shared/utilities/processUtils'
 import { Uri } from 'vscode'
 
@@ -17,10 +16,10 @@ export async function isGitRepo(folder: Uri): Promise<boolean> {
         rejectOnErrorCode: true,
         onStdout: (text) => {
             output += text
-            showOutputMessage(removeAnsi(text), globals.outputChannel)
+            getLogger().verbose(removeAnsi(text))
         },
         onStderr: (text) => {
-            showOutputMessage(removeAnsi(text), globals.outputChannel)
+            getLogger().error(removeAnsi(text))
         },
         spawnOptions: {
             cwd: folder.fsPath,

packages/core/src/codewhisperer/util/zipUtil.ts

@@ -22,8 +22,7 @@ import {
 } from '../models/errors'
 import { ZipUseCase } from '../models/constants'
 import { ChildProcess, ChildProcessOptions } from '../../shared/utilities/processUtils'
-import { showOutputMessage } from '../../shared/utilities/messages'
-import { globals, removeAnsi } from '../../shared'
+import { removeAnsi } from '../../shared'
 
 export interface ZipMetadata {
     rootDir: string
@@ -303,10 +302,10 @@ export class ZipUtil {
             rejectOnErrorCode: false,
             onStdout: (text) => {
                 diffContent += text
-                showOutputMessage(removeAnsi(text), globals.outputChannel)
+                getLogger().verbose(removeAnsi(text))
             },
             onStderr: (text) => {
-                showOutputMessage(removeAnsi(text), globals.outputChannel)
+                getLogger().error(removeAnsi(text))
             },
             spawnOptions: {
                 cwd: projectPath,
@@ -340,10 +339,10 @@ export class ZipUtil {
             rejectOnErrorCode: true,
             onStdout: (text) => {
                 diffContent += text
-                showOutputMessage(removeAnsi(text), globals.outputChannel)
+                getLogger().verbose(removeAnsi(text))
             },
             onStderr: (text) => {
-                showOutputMessage(removeAnsi(text), globals.outputChannel)
+                getLogger().error(removeAnsi(text))
             },
             spawnOptions: {
                 cwd: projectPath,