fix(amazonq): ensure summary.md + icons saved, populate job history sooner, fix minor styling issues (#4928)

* fix(amazonq): add jobId to history tab and ensure summary/icons get saved

* update changelog

* fix failing test

* fix lint issue

* fix circular dep

* address comments

* remove unneeded await

* small fix to error notifications

---------

Co-authored-by: David Hasani <[email protected]>

Author: dhasani23

packages/amazonq/.changes/next-release/Bug Fix-58e09665-68a7-477e-b4a6-708cc77d7c54.json

@@ -0,0 +1,4 @@
+{
+	"type": "Bug Fix",
+	"description": "Amazon Q Code Transformation: make jobId visible in job history tab at start of job and allow summary.md + icons to be saved when accepting changes"
+}

packages/amazonq/package.json

@@ -257,25 +257,20 @@
                 }
             ],
             "view/title": [
-                {
-                    "command": "aws.amazonq.startTransformationInHub",
-                    "when": "view == aws.amazonq.transformationHub",
-                    "group": "navigation@1"
-                },
                 {
                     "command": "aws.amazonq.stopTransformationInHub",
                     "when": "view == aws.amazonq.transformationHub",
-                    "group": "navigation@2"
+                    "group": "navigation@1"
                 },
                 {
                     "command": "aws.amazonq.showPlanProgressInHub",
                     "when": "view == aws.amazonq.transformationHub",
-                    "group": "navigation@3"
+                    "group": "navigation@2"
                 },
                 {
                     "command": "aws.amazonq.showHistoryInHub",
                     "when": "view == aws.amazonq.transformationHub",
-                    "group": "navigation@4"
+                    "group": "navigation@3"
                 },
                 {
                     "command": "aws.amazonq.transformationHub.summary.reveal",
@@ -601,12 +596,6 @@
                 "category": "Amazon Q (Developer)",
                 "enablement": "aws.isDevMode"
             },
-            {
-                "command": "aws.amazonq.startTransformationInHub",
-                "title": "Start Transformation",
-                "icon": "$(play)",
-                "enablement": "gumby.isTransformAvailable"
-            },
             {
                 "command": "aws.amazonq.stopTransformationInHub",
                 "title": "Stop Transformation",

packages/core/resources/css/amazonqTransformationHub.css

@@ -0,0 +1,105 @@
+@keyframes spin {
+    0% {
+        transform: rotate(0deg);
+    }
+    100% {
+        transform: rotate(360deg);
+    }
+}
+body {
+    margin: 0;
+    padding: 0 1em;
+    height: 100vh;
+}
+
+.wrapper {
+    height: 100%;
+    display: flex;
+}
+
+.spinner {
+    display: inline-block;
+    animation: spin 1s infinite;
+}
+
+.column--container,
+.substep-container {
+    display: flex;
+    flex-direction: row;
+}
+
+.column {
+    flex-grow: 1;
+}
+
+.substep-container p {
+    margin: 0.5em 0;
+}
+
+.step,
+.simple-step {
+    padding: 0.5em 0 0.5em 0;
+    margin: 0;
+}
+
+.step {
+    padding-left: 20px;
+}
+
+.step:hover,
+.step.active {
+    color: var(--vscode-editor-selectionForeground, inherit);
+    background-color: var(--vscode-editor-selectionBackground, aliceblue);
+}
+
+#stepdetails {
+    width: 40%;
+    padding: 0 20px;
+    border-left: solid rgba(230, 230, 230, 0.5);
+    min-height: 100vh;
+    display: block;
+}
+
+#progress {
+    width: 60%;
+}
+
+.status-PENDING {
+    color: grey;
+}
+
+.status-COMPLETED {
+    color: green;
+}
+
+.status-FAILED {
+    color: red;
+}
+
+.substep {
+    display: none;
+}
+
+.substep-icon {
+    padding: 0 1em;
+}
+
+.visible {
+    display: block;
+}
+
+.center {
+    position: absolute;
+    top: 50%;
+    transform: translate(0, -50%);
+}
+
+.center-flex {
+    display: flex;
+    align-items: center;
+    justify-content: center;
+}
+
+#step-duration {
+    color: grey;
+}

packages/core/src/amazonq/webview/ui/tabs/generator.ts

@@ -28,7 +28,7 @@ export class TabDataGenerator {
         ['unknown', 'Ask a question or enter "/" for quick actions'],
         ['cwc', 'Ask a question or enter "/" for quick actions'],
         ['featuredev', 'Briefly describe a task or issue'],
-        ['gumby', 'Chat is disabled during Code Transformation.'],
+        ['gumby', 'Open a new tab to chat with Q'],
     ])
 
     private tabWelcomeMessage: Map<TabType, string> = new Map([

packages/core/src/amazonqGumby/activation.ts

@@ -55,11 +55,11 @@ export async function activate(context: ExtContext) {
             }),
 
             Commands.register('aws.amazonq.showHistoryInHub', async () => {
-                transformationHubViewProvider.updateContent('job history', 0) // 0 is dummy value for startTime - not used
+                await transformationHubViewProvider.updateContent('job history')
             }),
 
             Commands.register('aws.amazonq.showPlanProgressInHub', async (startTime: number) => {
-                transformationHubViewProvider.updateContent('plan progress', startTime)
+                await transformationHubViewProvider.updateContent('plan progress', startTime)
             }),
 
             Commands.register('aws.amazonq.showTransformationPlanInHub', async () => {

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

@@ -319,7 +319,7 @@ export class GumbyController {
     private async processHumanChatMessage(data: { message: string; tabID: string }) {
         this.messenger.sendUserPrompt(data.message, data.tabID)
         this.messenger.sendChatInputEnabled(data.tabID, false)
-        this.messenger.sendUpdatePlaceholder(data.tabID, 'Chat is disabled during Code Transformation.')
+        this.messenger.sendUpdatePlaceholder(data.tabID, 'Open a new tab to chat with Q')
 
         const session = this.sessionStorage.getSession()
         switch (session.conversationState) {

packages/core/src/amazonqGumby/telemetry/codeTransformTelemetry.ts

@@ -89,4 +89,4 @@ export const javapOutputToTelemetryValue = (javapCommandLineOutput: string) => {
     }
 }
 
-export const calculateTotalLatency = (startTime: number, endTime: number = Date.now()): number => endTime - startTime
+export const calculateTotalLatency = (startTime: number): number => Date.now() - startTime

packages/core/src/amazonqGumby/telemetry/codeTransformTelemetryState.ts

@@ -8,7 +8,6 @@ import { randomUUID } from '../../common/crypto'
 interface ICodeTransformTelemetryState {
     sessionId: string
     sessionStartTime: number
-    resultStatus: string
 }
 
 export class CodeTransformTelemetryState {
@@ -18,23 +17,18 @@ export class CodeTransformTelemetryState {
         this.mainState = {
             sessionId: randomUUID(),
             sessionStartTime: Date.now(),
-            resultStatus: '',
         }
     }
 
     public getSessionId = () => this.mainState.sessionId
     public getStartTime = () => this.mainState.sessionStartTime
-    public getResultStatus = () => this.mainState.resultStatus
 
     public setSessionId = () => {
         this.mainState.sessionId = randomUUID()
     }
     public setStartTime = () => {
         this.mainState.sessionStartTime = Date.now()
     }
-    public setResultStatus = (newValue: string) => {
-        this.mainState.resultStatus = newValue
-    }
 
     static #instance: CodeTransformTelemetryState