cleanup

Author: Will-ShaoHua

plugins/amazonq/chat/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/QLoginWebview.kt

@@ -182,16 +182,16 @@ class QWebviewBrowser(val project: Project, private val parentDisposable: Dispos
                     ToolkitConnectionManager.getInstance(project)
                         .activeConnectionForFeature(QConnection.getInstance()) as? AwsBearerTokenConnection
                     )?.let { connection ->
-                        runInEdt {
-                            SsoLogoutAction(connection).actionPerformed(
-                                AnActionEvent.createFromDataContext(
-                                    "qBrowser",
-                                    null,
-                                    DataContext.EMPTY_CONTEXT
-                                )
+                    runInEdt {
+                        SsoLogoutAction(connection).actionPerformed(
+                            AnActionEvent.createFromDataContext(
+                                "qBrowser",
+                                null,
+                                DataContext.EMPTY_CONTEXT
                             )
-                        }
+                        )
                     }
+                }
             }
 
             is BrowserMessage.Reauth -> {
@@ -302,9 +302,8 @@ class QWebviewBrowser(val project: Project, private val parentDisposable: Dispos
                             profiles: ${writeValueAsString(profiles ?: "")},
                             errorMessage: '$errorMessage'
                         }
-                    """.trimIndent()
+                        """.trimIndent()
 
-                        println(jsonData)
                         executeJS("window.ideClient.prepareUi($jsonData)")
                     }
                 }

plugins/core/jetbrains-community/src/software/aws/toolkits/jetbrains/core/webview/BrowserMessage.kt

@@ -29,8 +29,8 @@ import com.fasterxml.jackson.annotation.JsonTypeInfo
     JsonSubTypes.Type(value = BrowserMessage.SwitchProfile::class, name = "switchProfile"),
     JsonSubTypes.Type(value = BrowserMessage.PublishWebviewTelemetry::class, name = "webviewTelemetry")
 )
-// Webview to IDE
 sealed interface BrowserMessage {
+
     // FIX_WHEN_MIN_IS_233: data objects are not stable until Kotlin 1.9
     // https://kotlinlang.org/docs/whatsnew19.html#stable-data-objects-for-symmetry-with-data-classes
     object PrepareUi : BrowserMessage

plugins/core/webview/src/ideClient.ts

@@ -49,7 +49,6 @@ export class IdeClient {
                 result = new ListProfilePendingResult()
                 break
         }
-        console.log(result)
         this.store.commit('setProfilesResult', result)
     }
 
@@ -88,6 +87,6 @@ export class IdeClient {
     cancelLogin(): void {
         // this.reset()
         this.store.commit('setStage', 'START')
-        window.ideApi.postMessage({command: 'cancelLogin'})
+        window.ideApi.postMessage({ command: 'cancelLogin' })
     }
 }

plugins/core/webview/src/model.ts

@@ -71,23 +71,17 @@ export interface ListProfileResult {
 
 export class ListProfileSuccessResult implements ListProfileResult {
     status: 'succeeded' = 'succeeded'
-
-    constructor(readonly profiles: Profile[]) {
-    }
+    constructor(readonly profiles: Profile[]) {}
 }
 
 export class ListProfileFailureResult implements ListProfileResult {
     status: 'failed' = 'failed'
-
-    constructor(readonly errorMessage: string) {
-    }
+    constructor(readonly errorMessage: string) {}
 }
 
 export class ListProfilePendingResult implements ListProfileResult {
     status: 'pending' = 'pending'
-
-    constructor() {
-    }
+    constructor() {}
 }
 
 export enum LoginIdentifier {
@@ -146,8 +140,7 @@ export class BuilderId implements LoginOption {
 export class ExistConnection implements LoginOption {
     id: LoginIdentifier = LoginIdentifier.EXISTING_LOGINS
 
-    constructor(readonly pluginConnectionId: string) {
-    }
+    constructor(readonly pluginConnectionId: string) {}
 
     // this case only happens for bearer connection for now
     requiresBrowser(): boolean {

plugins/core/webview/src/q-ui/components/profileSelection.vue

@@ -4,7 +4,7 @@
     <div @keydown.enter="handleContinueClick">
         <div class="font-amazon">
             <template v-if="isWaitingResponse">
-                <div class="header bottomMargin">Fetching Q Developer profiles...this may take a minute.</div>
+                <div class="title bottom-small-gap">Fetching Q Developer profiles...this may take a minute.</div>
             </template>
 
             <template v-else>
@@ -94,25 +94,12 @@ export default defineComponent({
             if (profileResult instanceof ListProfileSuccessResult) {
                 this.availableProfiles = profileResult.profiles
             } else {
-                this.errorMessage = (profileResult as ListProfileFailureResult).errorMessage
+                this.errorMessage = GENERIC_PROFILE_LOAD_ERROR
             }
 
             return false
         }
     },
-    mounted() {
-        // console.debug("Vuex raw profiles:", this.$store.state.profiles);
-        // this.availableProfiles = this.$store.state.profiles;
-        // this.errorMessage = this.$store.state.errorMessage ? GENERIC_PROFILE_LOAD_ERROR : undefined;
-        // this.isRefreshing = false;
-        // watch(() => this.$store.state.profiles, (newVal, oldVal) => {
-        //     this.availableProfiles = newVal
-        //     this.isRefreshing = false
-        // });
-        // watch(() => this.$store.state.errorMessage, (newVal) => {
-        //     this.errorMessage = newVal ? GENERIC_PROFILE_LOAD_ERROR : undefined;
-        // })
-    },
     methods: {
         toggleItemSelection(profile: Profile) {
             this.selectedProfile = profile;