PR fixes: refactoring, syntax changes

Author: tgodara-aws

packages/core/src/amazonqGumby/activation.ts

@@ -72,6 +72,13 @@ export async function activate(context: ExtContext) {
             )
         }),
 
+        Commands.register(
+            'aws.amazonq.transformationHub.updateContent',
+            async (button, startTime, historyFileUpdated) => {
+                await transformationHubViewProvider.updateContent(button, startTime, historyFileUpdated)
+            }
+        ),
+
         workspaceChangeEvent
     )
 }

packages/core/src/amazonqGumby/chat/controller/controller.ts

@@ -191,13 +191,8 @@ export class GumbyController {
 
     private async transformInitiated(message: any) {
         // check if any jobs potentially still in progress on backend
-        const history = readHistoryFile()
-        let numInProgress = 0
-        history.forEach((job) => {
-            if (job.status === 'FAILED') {
-                numInProgress += 1
-            }
-        })
+        const history = await readHistoryFile()
+        const numInProgress = history.filter((job) => job.status === 'FAILED').length
         this.messenger.sendViewHistoryMessage(message.tabID, numInProgress)
         if (transformByQState.isRefreshInProgress()) {
             this.messenger.sendMessage(CodeWhispererConstants.refreshInProgressChatMessage, message.tabID, 'ai-prompt')

packages/core/src/codewhisperer/commands/startTransformByQ.ts

@@ -749,15 +749,14 @@ export async function postTransformationJob() {
             })
     }
 
+    // delete original upload ZIP at very end of transformation
+    fs.rmSync(transformByQState.getPayloadFilePath(), { force: true })
+
     if (
         transformByQState.isSucceeded() ||
         transformByQState.isPartiallySucceeded() ||
         transformByQState.isCancelled()
     ) {
-        if (transformByQState.getPayloadFilePath()) {
-            // delete original upload ZIP at very end of transformation
-            fs.rmSync(transformByQState.getPayloadFilePath(), { force: true })
-        }
         // delete the copy of the upload ZIP
         fs.rmSync(path.join(transformByQState.getJobHistoryPath(), 'zipped-code.zip'), { force: true })
         // delete transformation job metadata file (no longer needed)
@@ -801,7 +800,7 @@ export async function postTransformationJob() {
         ]
 
         const jobDetails = fields.join('\t') + '\n'
-        fs.writeFileSync(historyLogFilePath, jobDetails, { flag: 'a' })
+        fs.writeFileSync(historyLogFilePath, jobDetails, { flag: 'a' }) // 'a' flag used to append to file
         await vscode.commands.executeCommand(
             'aws.amazonq.transformationHub.updateContent',
             'job history',

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

@@ -812,7 +812,8 @@ export const viewHistoryMessage = (numInProgress: number) =>
         : 'View previous transformations run from the IDE'
 
 export const transformationHistoryTableDescription =
-    'This table lists the most recent jobs that you have run in the past 30 days. To open the diff patch and summary files, click the provided links. Jobs with a status of FAILED may still be in progress. Resume them within 12 hours of starting the job to get an updated job status and artifacts. Click the refresh icon to do so. The diff patch and summary will appear once they are available.'
+    'This table lists the most recent jobs that you have run in the past 30 days. To open the diff patch and summary files, click the provided links. To get an updated job status, click the refresh icon. The diff patch and summary will appear once they are available.<br><br>' +
+    'Jobs with a status of FAILED may still be in progress. Resume these jobs within 12 hours of starting the job to get an updated job status and artifacts.'
 
 export const refreshErrorChatMessage =
     "Sorry, I couldn't refresh the job. Please try again or start a new transformation."
@@ -939,3 +940,14 @@ export const codeReviewFindingsSuffix = '_codeReviewFindings'
 export const displayFindingsSuffix = '_displayFindings'
 
 export const displayFindingsDetectorName = 'DisplayFindings'
+export const findingsSuffix = '_codeReviewFindings'
+
+export interface HistoryObject {
+    startTime: string
+    projectName: string
+    status: string
+    duration: string
+    diffPath: string
+    summaryPath: string
+    jobId: string
+}

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

@@ -1157,6 +1157,7 @@ export class TransformByQState {
         this.buildLog = ''
         this.customBuildCommand = ''
         this.intervalId = undefined
+        this.jobHistoryPath = ''
     }
 }
 

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

@@ -451,10 +451,16 @@ export class ProposedTransformationExplorer {
                     transformByQState.getSummaryFilePath(),
                     path.join(transformByQState.getJobHistoryPath(), 'summary', 'summary.md')
                 )
-                fs.copyFileSync(
-                    path.join(path.dirname(transformByQState.getSummaryFilePath()), 'buildCommandOutput.log'),
-                    path.join(transformByQState.getJobHistoryPath(), 'summary', 'buildCommandOutput.log')
-                )
+                if (
+                    fs.existsSync(
+                        path.join(path.dirname(transformByQState.getSummaryFilePath()), 'buildCommandOutput.log')
+                    )
+                ) {
+                    fs.copyFileSync(
+                        path.join(path.dirname(transformByQState.getSummaryFilePath()), 'buildCommandOutput.log'),
+                        path.join(transformByQState.getJobHistoryPath(), 'summary', 'buildCommandOutput.log')
+                    )
+                }
 
                 transformByQState.setResultArchiveFilePath(pathContainingArchive)
                 await setContext('gumby.isSummaryAvailable', true)