Merge pull request #6039 from mr-lee/master

fix(amazonq): fixed bug with high CPU use event loop for security scan findings
aws · Nov 19, 2024 · 10a9458 · 10a9458
2 parents f81d9a1 + 2e4c369
commit 10a9458
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/packages/core/src/codewhisperer/views/securityIssue/securityIssueWebview.ts b/packages/core/src/codewhisperer/views/securityIssue/securityIssueWebview.ts
@@ -74,7 +74,8 @@ const Panel = VueWebview.compilePanel(SecurityIssueWebview)
 let activePanel: InstanceType<typeof Panel> | undefined
 
 export async function showSecurityIssueWebview(ctx: vscode.ExtensionContext, issue: CodeScanIssue, filePath: string) {
-    activePanel ??= new Panel(ctx)
+    // always create a new panel per finding
+    activePanel = new Panel(ctx)
     activePanel.server.setIssue(issue)
     activePanel.server.setFilePath(filePath)
 

diff --git a/packages/core/src/codewhisperer/views/securityIssue/vue/root.vue b/packages/core/src/codewhisperer/views/securityIssue/vue/root.vue
@@ -129,9 +129,6 @@ export default defineComponent({
     created() {
         this.getData()
     },
-    updated() {
-        this.getData()
-    },
     methods: {
         async getData() {
             const issue = await client.getIssue()