removed redudnant check for one or multiple diffs when accepting changes

Author: ntarakad-aws

packages/core/src/codewhisperer/models/constants.ts

@@ -36,6 +36,7 @@ const patchDescriptions: { [key: string]: string } = {
     'Miscellaneous Processing Documentation upgrade':
         'This diff patch covers a diverse set of upgrades spanning ORMs, XML processing, API documentation, and more.',
     'Upgrade Deprecated API': '',
+    'Updated dependencies to latest version': '',
 }
 
 export const JsonConfigFileNamingConvention = new Set([

packages/core/src/codewhisperer/service/transformByQ/transformationResultsViewProvider.ts

@@ -10,7 +10,13 @@ import { parsePatch, applyPatches, ParsedDiff } from 'diff'
 import path from 'path'
 import vscode from 'vscode'
 import { ExportIntent } from '@amzn/codewhisperer-streaming'
-import { TransformByQReviewStatus, transformByQState, PatchInfo, DescriptionContent } from '../../models/model'
+import {
+    TransformByQReviewStatus,
+    transformByQState,
+    PatchInfo,
+    DescriptionContent,
+    TransformationType,
+} from '../../models/model'
 import { ExportResultArchiveStructure, downloadExportResultArchive } from '../../../shared/utilities/download'
 import { getLogger } from '../../../shared/logger'
 import { telemetry } from '../../../shared/telemetry/telemetry'
@@ -538,38 +544,30 @@ export class ProposedTransformationExplorer {
             }
 
             //We do this to ensure that the changesAppliedChatMessage is only sent to user when they accept the first diff.patch
-            if (transformByQState.getMultipleDiffs()) {
-                if (diffModel.currentPatchIndex === patchFiles.length - 1) {
-                    transformByQState.getChatControllers()?.transformationFinished.fire({
-                        message: CodeWhispererConstants.changesAppliedChatMessageMultipleDiffs(
-                            diffModel.currentPatchIndex,
-                            patchFiles.length,
-                            patchFilesDescriptions
-                                ? patchFilesDescriptions.content[diffModel.currentPatchIndex].name
-                                : undefined
-                        ),
-                        tabID: ChatSessionManager.Instance.getSession().tabID,
-                        includeStartNewTransformationButton: true,
-                    })
-                } else {
-                    transformByQState.getChatControllers()?.transformationFinished.fire({
-                        message: CodeWhispererConstants.changesAppliedChatMessageMultipleDiffs(
-                            diffModel.currentPatchIndex,
-                            patchFiles.length,
-                            patchFilesDescriptions
-                                ? patchFilesDescriptions.content[diffModel.currentPatchIndex].name
-                                : undefined
-                        ),
-                        tabID: ChatSessionManager.Instance.getSession().tabID,
-                        includeStartNewTransformationButton: false,
-                    })
-                }
-            } else {
+            if (diffModel.currentPatchIndex === patchFiles.length - 1) {
                 transformByQState.getChatControllers()?.transformationFinished.fire({
-                    message: CodeWhispererConstants.changesAppliedChatMessageOneDiff,
+                    message: CodeWhispererConstants.changesAppliedChatMessageMultipleDiffs(
+                        diffModel.currentPatchIndex,
+                        patchFiles.length,
+                        patchFilesDescriptions
+                            ? patchFilesDescriptions.content[diffModel.currentPatchIndex].name
+                            : undefined
+                    ),
                     tabID: ChatSessionManager.Instance.getSession().tabID,
                     includeStartNewTransformationButton: true,
                 })
+            } else {
+                transformByQState.getChatControllers()?.transformationFinished.fire({
+                    message: CodeWhispererConstants.changesAppliedChatMessageMultipleDiffs(
+                        diffModel.currentPatchIndex,
+                        patchFiles.length,
+                        patchFilesDescriptions
+                            ? patchFilesDescriptions.content[diffModel.currentPatchIndex].name
+                            : undefined
+                    ),
+                    tabID: ChatSessionManager.Instance.getSession().tabID,
+                    includeStartNewTransformationButton: false,
+                })
             }
 
             // Load the next patch file