fix: make summary button enabled after accept/reject changes

David Hasani · David Hasani · commit 30ecea9581dc · 2024-01-19T12:16:22.000-08:00
diff --git a/package.json b/package.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"
             }
         ],
diff --git a/src/codewhisperer/commands/startTransformByQ.ts b/src/codewhisperer/commands/startTransformByQ.ts
@@ -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')
diff --git a/src/codewhisperer/service/transformationResultsViewProvider.ts b/src/codewhisperer/service/transformationResultsViewProvider.ts
@@ -365,6 +365,7 @@ export class ProposedTransformationExplorer {
                     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({

Original file line number	Diff line number	Diff line change
`@@ -2415,7 +2415,7 @@`
`2415`	`2415`	`{`
`2416`	`2416`	`"command": "aws.amazonq.transformationHub.summary.reveal",`
`2417`	`2417`	`"title": "%AWS.command.q.transform.showChangeSummary%",`
`2418`		`- "enablement": "gumby.reviewState == InReview"`
	`2418`	`+ "enablement": "gumby.isSummaryAvailable"`
`2419`	`2419`	`},`
`2420`	`2420`	`{`
`2421`	`2421`	`"command": "aws.amazonq.transformationHub.reviewChanges.reveal",`
`@@ -3805,7 +3805,6 @@`
`3805`	`3805`	`{`
`3806`	`3806`	`"command": "aws.amazonq.showTransformationPlanInHub",`
`3807`	`3807`	`"title": "Show Transformation Plan",`
`3808`		`- "icon": "$(book)",`
`3809`	`3808`	`"enablement": "gumby.isPlanAvailable"`
`3810`	`3809`	`}`
`3811`	`3810`	`],`
Original file line number	Diff line number	Diff line change
`@@ -365,6 +365,7 @@ export class ProposedTransformationExplorer {`
`365`	`365`	`path.join(pathContainingArchive, ExportResultArchiveStructure.PathToSummary)`
`366`	`366`	`)`
`367`	`367`	`transformByQState.setResultArchiveFilePath(pathContainingArchive)`
	`368`	`+ await vscode.commands.executeCommand('setContext', 'gumby.isSummaryAvailable', true)`
`368`	`369`
`369`	`370`	`// This metric is only emitted when placed before showInformationMessage`
`370`	`371`	`telemetry.codeTransform_vcsDiffViewerVisible.emit({`