Fixes #381

Author: isc-bsaviano

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## [2.8.1] - 2025-XX-XX
+- Fix issue [#381](https://github.com/intersystems/language-server/issues/381): Fix extension crashes due to request forwarding
+
 ## [2.8.0] - 2025-07-28
 - Fix issue [#378](https://github.com/intersystems/language-server/issues/378): Remove `intersystems.language-server.completion.showInternal` setting
 - Fix issue [#380](https://github.com/intersystems/language-server/issues/380): Add the Deprecated tag to DocumentSymbols in classes

server/src/providers/completion.ts

@@ -719,7 +719,7 @@ async function globalsOrRoutines(
 }
 
 export async function onCompletion(params: CompletionParams): Promise<CompletionItem[] | null> {
-	var result: CompletionItem[] = [];
+	let result: CompletionItem[] = [];
 	const doc = documents.get(params.textDocument.uri);
 	if (doc === undefined) {return null;}
 	const parsed = await getParsedDocument(params.textDocument.uri);
@@ -729,14 +729,14 @@ export async function onCompletion(params: CompletionParams): Promise<Completion
 	const prevline = doc.getText(Range.create(Position.create(params.position.line,0),params.position));
 	const prevlineLower = prevline.toLowerCase();
 	const classregex = /^class[ ]+%?[\p{L}\d]+(\.{1}[\p{L}\d]+)* +extends[ ]+(\(([%]?[\p{L}\d]+(\.{1}[\p{L}\d]+)*,[ ]*)*)?$/iu;
-	var firsttwotokens = "";
+	let firsttwotokens = "";
 	if (parsed[params.position.line].length >= 2) {
 		firsttwotokens = doc.getText(Range.create(
 			params.position.line,parsed[params.position.line][0].p,
 			params.position.line,parsed[params.position.line][1].p+parsed[params.position.line][1].c
 		));
 	}
-	var thistoken: number = -1;
+	let thistoken: number = -1;
 	for (let i = 0; i < parsed[params.position.line].length; i++) {
 		const symbolstart: number = parsed[params.position.line][i].p;
 		const symbolend: number =  parsed[params.position.line][i].p + parsed[params.position.line][i].c;
@@ -775,7 +775,6 @@ export async function onCompletion(params: CompletionParams): Promise<Completion
 	const settings = await getLanguageServerSettings(params.textDocument.uri);
 	const asRegex = /\s+as\s+$/;
 	const parenAndCommaRegex = /[,(]\s*$/;
-	const squareAndCommaRegex = /[,[]\s*$/;
 
 	if (prevline.endsWith("$$$") && [ld.cos_langindex,ld.sql_langindex].includes(triggerlang)) {
 		// This is a macro

server/src/providers/requestForwarding.ts

@@ -14,10 +14,11 @@ interface IsolateEmbeddedLanguageParams {
  */
 export async function languageAtPosition(params: TextDocumentPositionParams): Promise<number> {
 	const doc = documents.get(params.textDocument.uri);
-	if (doc === undefined) {return -1;}
+	if (!doc) return -1;
 	const parsed = await getParsedDocument(params.textDocument.uri);
-	if (parsed === undefined) {return -1;}
-	if (params.position.line === parsed.length) {return -1;}
+	if (!parsed) return -1;
+	if (params.position.line >= parsed.length) return -1;
+	if (!parsed[params.position.line]?.length) return -1;
 
 	let thistoken: number = -1;
 	for (let i = 0; i < parsed[params.position.line].length; i++) {