fix(codewhisperer): "TypeError: Cannot read properties of undefined" #…

…4210

Problem:
CI test logs are showing many of these unhandled rejections:

    rejected promise not handled within 1 second: TypeError: Cannot read properties of undefined (reading 'range')
    stack trace: TypeError: Cannot read properties of undefined (reading 'range')
        at SecurityIssueHoverProvider.handleDocumentChange (/codebuild/output/src3775465363/src/github.com/aws/aws-toolkit-vscode/src/codewhisperer/service/securityIssueProvider.ts:20:54)
        at f.value (/codebuild/output/src3775465363/src/github.com/aws/aws-toolkit-vscode/src/codewhisperer/activation.ts:331:53)

Solution:
Skip `handleDocumentChange` if the event looks invalid. The event may be
triggered by edits from our own "code generation" logic, which presumably is
causing some weird behavior.

Fix #4203

src/codewhisperer/service/securityIssueProvider.ts

@@ -5,7 +5,6 @@
 
 import * as vscode from 'vscode'
 import { AggregatedCodeScanIssue } from '../models/model'
-
 export abstract class SecurityIssueProvider {
     private _issues: AggregatedCodeScanIssue[] = []
     public get issues() {
@@ -17,6 +16,10 @@ export abstract class SecurityIssueProvider {
     }
 
     public handleDocumentChange(event: vscode.TextDocumentChangeEvent) {
+        // handleDocumentChange function may be triggered while testing by our own code generation.
+        if (!event.contentChanges || event.contentChanges.length === 0) {
+            return
+        }
         const changedRange = event.contentChanges[0].range
         const changedText = event.contentChanges[0].text
         const lineOffset = this._getLineOffset(changedRange, changedText)

src/codewhisperer/util/dependencyGraph/cloudformationDependencyGraph.ts

@@ -100,4 +100,4 @@ export class cloudformationDependencyGraph extends DependencyGraph {
     }
 }
 
-export class CsharpDependencyGraphError extends Error {}
+export class cloudformationDependencyGraphError extends Error {}