fix(amazonq): fix /transform test and remove feedback form

Author: David Hasani

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

@@ -28,7 +28,6 @@ import {
     validateCanCompileProject,
     getValidSQLConversionCandidateProjects,
     openHilPomFile,
-    getFeedbackCommentData,
 } from '../../../codewhisperer/commands/startTransformByQ'
 import { JDKVersion, TransformationCandidateProject, transformByQState } from '../../../codewhisperer/models/model'
 import {
@@ -62,7 +61,6 @@ import {
     validateSQLMetadataFile,
 } from '../../../codewhisperer/service/transformByQ/transformFileHandler'
 import { getAuthType } from '../../../auth/utils'
-import globals from '../../../shared/extensionGlobals'
 
 // These events can be interactions within the chat,
 // or elsewhere in the IDE
@@ -372,9 +370,6 @@ export class GumbyController {
             case ButtonActions.CONFIRM_SKIP_TESTS_FORM:
                 await this.handleSkipTestsSelection(message)
                 break
-            case ButtonActions.CONFIRM_FEEDBACK_FORM:
-                await this.handleFeedback(message)
-                break
             case ButtonActions.CONFIRM_SELECTIVE_TRANSFORMATION_FORM:
                 await this.handleOneOrMultipleDiffs(message)
                 break
@@ -639,17 +634,6 @@ export class GumbyController {
         }
     }
 
-    private async handleFeedback(message: any) {
-        const canRerunJob = message.formSelectedValues['TransformFeedbackRerunJob']
-        const canViewLogs = message.formSelectedValues['TransformFeedbackViewLogs']
-        const comment = `Permission to re-run job: ${canRerunJob}\nPermission to view logs: ${canViewLogs}\n${getFeedbackCommentData()}`
-        this.messenger.sendFeedbackReceivedMessage(canRerunJob, canViewLogs, message.tabID)
-        if (comment.toLowerCase().includes('yes')) {
-            // post feedback if user says yes to at least one of the questions
-            await globals.telemetry.postFeedback({ comment: comment, sentiment: 'Positive' })
-        }
-    }
-
     private resetTransformationChatFlow() {
         this.sessionStorage.getSession().conversationState = ConversationState.IDLE
     }

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

@@ -542,15 +542,6 @@ export class Messenger {
             })
         }
 
-        if (
-            transformByQState.isPartiallySucceeded() &&
-            message === CodeWhispererConstants.viewProposedChangesChatMessage
-        ) {
-            // get permission to re-run job and view logs after partially successful job is downloaded
-            // TODO: uncomment this when feature is ready
-            // this.sendFeedbackFormMessage(tabID)
-        }
-
         this.dispatcher.sendChatMessage(
             new ChatMessage(
                 {
@@ -808,64 +799,4 @@ ${codeSnippet}
             )
         )
     }
-
-    public sendFeedbackFormMessage(tabID: string) {
-        const formItems: ChatItemFormItem[] = []
-        formItems.push({
-            id: 'TransformFeedbackRerunJob',
-            type: 'radiogroup',
-            title: 'To improve our service, do we have permission to re-run your job? (you will *not* be charged)',
-            mandatory: true,
-            options: [
-                {
-                    value: 'Yes',
-                    label: 'Yes',
-                },
-                {
-                    value: 'No',
-                    label: 'No',
-                },
-            ],
-        })
-
-        formItems.push({
-            id: 'TransformFeedbackViewLogs',
-            type: 'radiogroup',
-            title: 'Do we also have permission to view the logs associated with your job?',
-            mandatory: true,
-            options: [
-                {
-                    value: 'Yes',
-                    label: 'Yes',
-                },
-                {
-                    value: 'No',
-                    label: 'No',
-                },
-            ],
-        })
-
-        this.dispatcher.sendChatPrompt(
-            new ChatPrompt(
-                {
-                    message: 'Amazon Q Permissions Form',
-                    formItems: formItems,
-                },
-                'FeedbackForm',
-                tabID,
-                false
-            )
-        )
-    }
-
-    public sendFeedbackReceivedMessage(canRerunJob: string, canViewLogs: string, tabID: string) {
-        const message = `### Response received
--------------
-| | |
-| :------------------- | -------: |
-| **Permission to re-run job**             |   ${canRerunJob}   |
-| **Permission to view logs** |  ${canViewLogs}   |
-    `
-        this.dispatcher.sendChatMessage(new ChatMessage({ message, messageType: 'prompt' }, tabID))
-    }
 }

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

@@ -18,7 +18,6 @@ export enum ButtonActions {
     CONFIRM_SQL_CONVERSION_TRANSFORMATION_FORM = 'gumbySQLConversionTransformFormConfirm',
     CANCEL_TRANSFORMATION_FORM = 'gumbyTransformFormCancel', // shared between Language Upgrade & SQL Conversion
     CONFIRM_SKIP_TESTS_FORM = 'gumbyTransformSkipTestsFormConfirm',
-    CONFIRM_FEEDBACK_FORM = 'gumbyFeedbackFormConfirm',
     CONFIRM_SELECTIVE_TRANSFORMATION_FORM = 'gumbyTransformOneOrMultipleDiffsFormConfirm',
     SELECT_SQL_CONVERSION_METADATA_FILE = 'gumbySQLConversionMetadataTransformFormConfirm',
     CONFIRM_DEPENDENCY_FORM = 'gumbyTransformDependencyFormConfirm',