fix(codewhisperer): line tracker infinite loop when focus is on OUTPUT/LOGS #4591

Problem:
`LineTracker` is subscribing `vscode.TextDocumentChangeEvent` and VSCode will also fire the above event when the plugin is adding new entry of logs. 
Therefore when user's focus is at log file, the following events will happen in order
1. VSCode `onContentChanged`
2. `LineTracker` emits events to update CWSPR UI components
3. `ActiveStateController.refresh()` & `LineAnnotationController.refresh()` 
4. `refresh()` invoke `isConnectionValid()`, which will add logs
5. back to step (1)

Author: Will-ShaoHua

.changes/next-release/Bug Fix-492bca4b-aec1-414d-a719-275c854aa1d2.json

@@ -0,0 +1,4 @@
+{
+	"type": "Bug Fix",
+	"description": "CodeWhisperer: Infinite loop of logging if users' editor focus is on OUTPUT tab"
+}

packages/core/src/codewhisperer/tracker/lineTracker.ts

@@ -115,8 +115,9 @@ export class LineTracker implements vscode.Disposable {
 
     // @VisibleForTesting
     onContentChanged(e: vscode.TextDocumentChangeEvent) {
-        if (e.document === vscode.window.activeTextEditor?.document && e.contentChanges.length > 0) {
-            this._editor = vscode.window.activeTextEditor
+        const editor = vscode.window.activeTextEditor
+        if (e.document === editor?.document && e.contentChanges.length > 0 && isTextEditor(editor)) {
+            this._editor = editor
             this._selections = toLineSelections(this._editor?.selections)
 
             this.notifyLinesChanged('content')

packages/core/src/codewhisperer/views/activeStateController.ts

@@ -114,11 +114,6 @@ export class ActiveStateController implements vscode.Disposable {
     }, 1000)
 
     private async _refresh(editor: vscode.TextEditor | undefined, shouldDisplay?: boolean) {
-        if (!this.container.auth.isConnectionValid()) {
-            this.clear(this._editor)
-            return
-        }
-
         if (!editor && !this._editor) {
             return
         }
@@ -140,6 +135,11 @@ export class ActiveStateController implements vscode.Disposable {
             return
         }
 
+        if (!this.container.auth.isConnectionValid()) {
+            this.clear(this._editor)
+            return
+        }
+
         if (shouldDisplay !== undefined) {
             await this.updateDecorations(editor, selections, shouldDisplay)
         } else {

packages/core/src/codewhisperer/views/lineAnnotationController.ts

@@ -327,11 +327,6 @@ export class LineAnnotationController implements vscode.Disposable {
             return
         }
 
-        if (!this.container.auth.isConnectionValid()) {
-            this.clear()
-            return
-        }
-
         if (this.isTutorialDone()) {
             this.clear()
             return
@@ -360,6 +355,11 @@ export class LineAnnotationController implements vscode.Disposable {
             return
         }
 
+        if (!this.container.auth.isConnectionValid()) {
+            this.clear()
+            return
+        }
+
         await this.updateDecorations(editor, selections, source, force)
     }