fix(codewhisperer): skip activation when disabled #2736

Problem:
For troubleshooting issues potentially related to codewhisperer, it is
useful for users to be able to fully disable codewhisperer to rule it
out as a possible cause. But codewhisperer activate() is called even
when it is disabled in settings, and this creates various handlers for
numerous vscode events, which could be a potential source of performance
issues or other bugs.

Solution:
Skip codewhisperer activation entirely if it is disabled in
settings.

Author: justinmk3

.changes/next-release/Bug Fix-b99c68f5-b3fe-417d-ac9a-cd5b63697396.json

@@ -0,0 +1,4 @@
+{
+	"type": "Bug Fix",
+	"description": "CodeWhisperer features were not fully disabled if the experiment was disabled"
+}

src/codewhisperer/activation.ts

@@ -48,6 +48,11 @@ import { Commands } from '../shared/vscode/commands2'
 const performance = globalThis.performance ?? require('perf_hooks').performance
 
 export async function activate(context: ExtContext): Promise<void> {
+    const codewhispererSettings = CodeWhispererSettings.instance
+    if (!codewhispererSettings.isEnabled()) {
+        return
+    }
+
     /**
      * Enable essential intellisense default settings for AWS C9 IDE
      */
@@ -64,7 +69,6 @@ export async function activate(context: ExtContext): Promise<void> {
     /**
      * Service control
      */
-    const codewhispererSettings = CodeWhispererSettings.instance
     const client = new codewhispererClient.DefaultCodeWhispererClient()
 
     const referenceHoverProvider = new ReferenceHoverProvider()