fix(codecatalyst): don't display onboard node when using existing con…

…nection (#4988)
aws · May 15, 2024 · 974b188 · 974b188
1 parent 0537ebc
commit 974b188
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/packages/core/src/codecatalyst/auth.ts b/packages/core/src/codecatalyst/auth.ts
@@ -84,13 +84,17 @@ export class CodeCatalystAuthenticationProvider {
         this.onDidChangeActiveConnection(async () => {
             if (this.activeConnection) {
                 await this.setScopeExpired(this.activeConnection, false)
+                await this.isConnectionOnboarded(this.activeConnection, true)
             }
             await setCodeCatalystConnectedContext(this.isConnectionValid())
             this.onDidChangeEmitter.fire()
         })
 
         this.onAccessDeniedException(async (showReauthPrompt: boolean) => {
             await this.accessDeniedExceptionHandler(showReauthPrompt)
+            if (this.activeConnection) {
+                await this.isConnectionOnboarded(this.activeConnection, true)
+            }
             this.onDidChangeEmitter.fire()
         })
 

diff --git a/packages/toolkit/.changes/next-release/Bug Fix-b9005bc6-20e0-48f3-a67a-ab23ee2cf62c.json b/packages/toolkit/.changes/next-release/Bug Fix-b9005bc6-20e0-48f3-a67a-ab23ee2cf62c.json
@@ -0,0 +1,4 @@
+{
+	"type": "Bug Fix",
+	"description": "CodeCatalyst: No longer display a prompt to 'Onboard' if re-using an existing connection from Amazon Q and it is already onboarded."
+}