Merge #4312 fix(gumby): remove unneeded dep

Author: justinmk3

package-lock.json

@@ -2415,7 +2415,7 @@
             {
                 "command": "aws.amazonq.transformationHub.summary.reveal",
                 "title": "%AWS.command.q.transform.showChangeSummary%",
-                "enablement": "gumby.reviewState == InReview"
+                "enablement": "gumby.isSummaryAvailable"
             },
             {
                 "command": "aws.amazonq.transformationHub.reviewChanges.reveal",
@@ -3805,7 +3805,6 @@
             {
                 "command": "aws.amazonq.showTransformationPlanInHub",
                 "title": "Show Transformation Plan",
-                "icon": "$(book)",
                 "enablement": "gumby.isPlanAvailable"
             }
         ],
@@ -4382,7 +4381,6 @@
         "markdown-it": "^13.0.2",
         "mime-types": "^2.1.32",
         "node-fetch": "^2.7.0",
-        "parse-diff": "0.11.1",
         "portfinder": "^1.0.32",
         "sanitize-html": "^2.3.3",
         "semver": "^7.5.4",

package.json

@@ -317,6 +317,7 @@ export async function setTransformationToRunningState(userInputState: UserInputS
     await vscode.commands.executeCommand('setContext', 'gumby.isStopButtonAvailable', true)
     await vscode.commands.executeCommand('setContext', 'gumby.isTransformAvailable', false)
     await vscode.commands.executeCommand('setContext', 'gumby.isPlanAvailable', false)
+    await vscode.commands.executeCommand('setContext', 'gumby.isSummaryAvailable', false)
     await resetReviewInProgress()
 
     await vscode.commands.executeCommand('aws.amazonq.refresh')

src/codewhisperer/commands/startTransformByQ.ts

@@ -140,6 +140,9 @@ export class DiffModel {
      */
     public parseDiff(pathToDiff: string, pathToWorkspace: string): ProposedChangeNode[] {
         const diffContents = fs.readFileSync(pathToDiff, 'utf8')
+        if (!diffContents) {
+            throw new ToolkitError('diff.patch file is empty', { code: 'EmptyDiffPatch' })
+        }
         const changedFiles = parsePatch(diffContents)
         // path to the directory containing copy of the changed files in the transformed project
         const pathToTmpSrcDir = this.copyProject(pathToWorkspace, changedFiles)
@@ -361,6 +364,19 @@ export class ProposedTransformationExplorer {
                 transformByQState.setSummaryFilePath(
                     path.join(pathContainingArchive, ExportResultArchiveStructure.PathToSummary)
                 )
+                transformByQState.setResultArchiveFilePath(pathContainingArchive)
+                await vscode.commands.executeCommand('setContext', 'gumby.isSummaryAvailable', true)
+
+                // This metric is only emitted when placed before showInformationMessage
+                telemetry.codeTransform_vcsDiffViewerVisible.emit({
+                    codeTransformSessionId: codeTransformTelemetryState.getSessionId(),
+                    codeTransformJobId: transformByQState.getJobId(),
+                    result: MetadataResult.Pass,
+                })
+
+                // Do not await this so that the summary reveals without user needing to close this notification
+                void vscode.window.showInformationMessage(CodeWhispererConstants.viewProposedChangesMessage)
+                await vscode.commands.executeCommand('aws.amazonq.transformationHub.summary.reveal')
             } catch (e: any) {
                 deserializeErrorMessage =
                     (e as Error).message ??
@@ -378,19 +394,6 @@ export class ProposedTransformationExplorer {
                     reason: deserializeErrorMessage ? 'DeserializationFailed' : undefined,
                 })
             }
-
-            transformByQState.setResultArchiveFilePath(pathContainingArchive)
-
-            // This metric is only emitted when placed before showInformationMessage
-            telemetry.codeTransform_vcsDiffViewerVisible.emit({
-                codeTransformSessionId: codeTransformTelemetryState.getSessionId(),
-                codeTransformJobId: transformByQState.getJobId(),
-                result: MetadataResult.Pass,
-            })
-
-            // Do not await this so that the summary reveals without user needing to close this notification
-            void vscode.window.showInformationMessage(CodeWhispererConstants.viewProposedChangesMessage)
-            await vscode.commands.executeCommand('aws.amazonq.transformationHub.summary.reveal')
         })
 
         vscode.commands.registerCommand('aws.amazonq.transformationHub.reviewChanges.acceptChanges', async () => {