Merge pull request #200051 from microsoft/aiday/checkingEffectiveLineCount

Checking that the effectiveLineNumber is valid before accessing the line content

Author: aiday-mar

src/vs/editor/contrib/codeAction/browser/lightBulbWidget.ts

@@ -203,20 +203,22 @@ export class LightBulbWidget extends Disposable implements IContentWidget {
 		};
 
 		let effectiveLineNumber = lineNumber;
+		let effectiveColumnNumber = 1;
 		if (!lineHasSpace) {
 			if (lineNumber > 1 && !isFolded(lineNumber - 1)) {
 				effectiveLineNumber -= 1;
-			} else if (!isFolded(lineNumber + 1)) {
+			} else if ((lineNumber < model.getLineCount()) && !isFolded(lineNumber + 1)) {
 				effectiveLineNumber += 1;
 			} else if (column * fontInfo.spaceWidth < 22) {
 				// cannot show lightbulb above/below and showing
 				// it inline would overlay the cursor...
 				return this.hide();
 			}
+			effectiveColumnNumber = !!model.getLineContent(effectiveLineNumber).match(/^\S\s*$/) ? 2 : 1;
 		}
 
 		this.state = new LightBulbState.Showing(actions, trigger, atPosition, {
-			position: { lineNumber: effectiveLineNumber, column: !!model.getLineContent(effectiveLineNumber).match(/^\S\s*$/) ? 2 : 1 },
+			position: { lineNumber: effectiveLineNumber, column: effectiveColumnNumber },
 			preference: LightBulbWidget._posPref
 		});
 		this._editor.layoutContentWidget(this);