[release] src/goLanguageServer.ts: fix broken event forwarding in middleware

Middleware must forward the intercepted event to the next handler.
Also, fixes the type error occurring when intercepting the codelens
response - it can be undefined or null.

Change-Id: I59aca7b63503cdb23da11670cfd5fa33ec06014c
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/245540
Reviewed-by: Rebecca Stambler <[email protected]>
(cherry picked from commit bd41c5e)
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/245599

Author: hyangah

src/goLanguageServer.ts

@@ -260,6 +260,9 @@ function buildLanguageClient(config: LanguageServerConfig): LanguageClient {
 			middleware: {
 				provideCodeLenses: async (doc, token, next): Promise<vscode.CodeLens[]> => {
 					const codeLens = await next(doc, token);
+					if (!codeLens || codeLens.length === 0) {
+						return codeLens;
+					}
 					return codeLens.map((lens: vscode.CodeLens) => {
 						switch (lens.command.title) {
 							case 'run test': {
@@ -365,17 +368,21 @@ function buildLanguageClient(config: LanguageServerConfig): LanguageClient {
 				},
 				// Keep track of the last file change in order to not prompt
 				// user if they are actively working.
-				didOpen: () => {
+				didOpen: (e, next) => {
 					lastUserAction = new Date();
+					next(e);
 				},
-				didChange: () => {
+				didChange: (e, next) => {
 					lastUserAction = new Date();
+					next(e);
 				},
-				didClose: () => {
+				didClose: (e, next) => {
 					lastUserAction = new Date();
+					next(e);
 				},
-				didSave: () => {
+				didSave: (e, next) => {
 					lastUserAction = new Date();
+					next(e);
 				},
 			}
 		}