feat(amazonq): auto-download results #6519
Conversation
|
|
|
||
| // attempt download for user | ||
| if (transformByQState.isSucceeded() || transformByQState.isPartiallySucceeded()) { | ||
| await vscode.commands.executeCommand('aws.amazonq.transformationHub.reviewChanges.startReview') |
There was a problem hiding this comment.
Just as a random aside, ideally this would have used our internal Commands wrapper core/src/shared/vscode/commands2.ts. This wrapper basically allows you to define the command as a const that you can execute later which is much more robust then having to pass around command ids!
In order to do that you would need to register aws.amazonq.transformationHub.reviewChanges.startReview like:
export const startReview = Commands.declare('aws.amazonq.transformationHub.reviewChanges.startReview',
() => async () => {
// implement me here
}
)
and then here you could just do startReview.tryExecute() that way you don't have to care about underlying id's!
See: https://github.com/aws/aws-toolkit-vscode/blob/master/docs/arch_develop.md#commands for more info
There was a problem hiding this comment.
Noted, https://github.com/aws/aws-toolkit-vscode/blob/master/packages/core/src/codewhisperer/service/transformByQ/transformationResultsViewProvider.ts is full of registered commands that could be refactored this way, so I think it's better if I do all of that in a separate PR.
There was a problem hiding this comment.
I'm okay with merging this as is as long as you commit/agree to making that change 😄. It should be like 15 minutes worth of work tops
There was a problem hiding this comment.
Added a TODO comment and made a note of it so I don't forget
Problem
Transformation results/artifacts expire after 24h, and sometimes users forget to manually click the "Download Proposed Changes" button within that time period.
Solution
Auto-download the results from S3 when ready.
feature/xbranches will not be squash-merged at release time.