Append stack trace to every language server error

src/LanguageServer.ts

@@ -177,6 +177,7 @@ export class LanguageServer {
      * Called when the client starts initialization
      * @param params
      */
+    @AddStackToErrorMessage
     public onInitialize(params: InitializeParams) {
         let clientCapabilities = params.capabilities;
 
@@ -222,6 +223,7 @@ export class LanguageServer {
      * Called when the client has finished initializing
      * @param params
      */
+    @AddStackToErrorMessage
     private async onInitialized() {
         let workspaceCreatedDeferred = new Deferred();
         this.initialWorkspacesCreated = workspaceCreatedDeferred.promise;
@@ -514,6 +516,7 @@ export class LanguageServer {
      * Provide a list of completion items based on the current cursor position
      * @param textDocumentPosition
      */
+    @AddStackToErrorMessage
     private async onCompletion(uri: string, position: Position) {
         //ensure programs are initialized
         await this.waitAllProgramFirstRuns();
@@ -538,6 +541,7 @@ export class LanguageServer {
      * Provide a full completion item from the selection
      * @param item
      */
+    @AddStackToErrorMessage
     private onCompletionResolve(item: CompletionItem): CompletionItem {
         if (item.data === 1) {
             item.detail = 'TypeScript details';
@@ -549,6 +553,7 @@ export class LanguageServer {
         return item;
     }
 
+    @AddStackToErrorMessage
     private async onCodeAction(params: CodeActionParams) {
         //ensure programs are initialized
         await this.waitAllProgramFirstRuns();
@@ -670,6 +675,8 @@ export class LanguageServer {
             await workspace.firstRunPromise;
         }
     }
+
+    @AddStackToErrorMessage
     private async onDidChangeConfiguration() {
         if (this.hasConfigurationCapability) {
             await this.reloadWorkspaces();
@@ -688,6 +695,7 @@ export class LanguageServer {
      * file types are watched (.brs,.bs,.xml,manifest, and any json/text/image files)
      * @param params
      */
+    @AddStackToErrorMessage
     private async onDidChangeWatchedFiles(params: DidChangeWatchedFilesParams) {
         //ensure programs are initialized
         await this.waitAllProgramFirstRuns();
@@ -859,6 +867,7 @@ export class LanguageServer {
         }
     }
 
+    @AddStackToErrorMessage
     private async onHover(params: TextDocumentPositionParams) {
         //ensure programs are initialized
         await this.waitAllProgramFirstRuns();
@@ -886,6 +895,7 @@ export class LanguageServer {
         return hover;
     }
 
+    @AddStackToErrorMessage
     private async onDocumentClose(textDocument: TextDocument): Promise<void> {
         let filePath = URI.parse(textDocument.uri).fsPath;
         let standaloneFileWorkspace = this.standaloneFileWorkspaces[filePath];
@@ -898,6 +908,7 @@ export class LanguageServer {
         }
     }
 
+    @AddStackToErrorMessage
     private async validateTextDocument(textDocument: TextDocument): Promise<void> {
         //ensure programs are initialized
         await this.waitAllProgramFirstRuns();
@@ -928,7 +939,6 @@ export class LanguageServer {
             // validate all workspaces
             await this.validateAllThrottled();
         } catch (e) {
-            this.connection.tracer.log(e);
             this.sendCriticalFailure(`Critical error parsing/ validating ${filePath}: ${e.message}`);
         }
     }
@@ -954,6 +964,7 @@ export class LanguageServer {
         this.connection.sendNotification('build-status', 'success');
     }
 
+    @AddStackToErrorMessage
     public async onWorkspaceSymbol(params: WorkspaceSymbolParams) {
         await this.waitAllProgramFirstRuns();
 
@@ -973,6 +984,7 @@ export class LanguageServer {
         return allSymbols as SymbolInformation[];
     }
 
+    @AddStackToErrorMessage
     public async onDocumentSymbol(params: DocumentSymbolParams) {
         await this.waitAllProgramFirstRuns();
 
@@ -987,6 +999,7 @@ export class LanguageServer {
         }
     }
 
+    @AddStackToErrorMessage
     private async onDefinition(params: TextDocumentPositionParams) {
         await this.waitAllProgramFirstRuns();
 
@@ -1001,6 +1014,7 @@ export class LanguageServer {
         return results;
     }
 
+    @AddStackToErrorMessage
     private async onSignatureHelp(params: SignatureHelpParams) {
         await this.waitAllProgramFirstRuns();
 
@@ -1025,7 +1039,7 @@ export class LanguageServer {
             };
             return results;
         } catch (e) {
-            this.connection.console.error(`error in onSignatureHelp: ${e.message}${e.stack ?? ''}`);
+            this.connection.console.error(`error in onSignatureHelp: ${e.stack ?? e.message ?? e}`);
             return {
                 signatures: [],
                 activeSignature: 0,
@@ -1034,6 +1048,7 @@ export class LanguageServer {
         }
     }
 
+    @AddStackToErrorMessage
     private async onReferences(params: ReferenceParams) {
         await this.waitAllProgramFirstRuns();
 
@@ -1065,6 +1080,7 @@ export class LanguageServer {
         }
     }
 
+    @AddStackToErrorMessage
     public async onExecuteCommand(params: ExecuteCommandParams) {
         await this.waitAllProgramFirstRuns();
         if (params.command === CustomCommands.TranspileFile) {
@@ -1103,3 +1119,34 @@ export interface Workspace {
 export enum CustomCommands {
     TranspileFile = 'TranspileFile'
 }
+
+/**
+ * Wraps a method. If there's an error (either sync or via a promise),
+ * this appends the error's stack trace at the end of the error message so that the connection will
+ */
+function AddStackToErrorMessage(target: any, propertyKey: string, descriptor: PropertyDescriptor) {
+    let originalMethod = descriptor.value;
+
+    //wrapping the original method
+    descriptor.value = function value(...args: any[]) {
+        try {
+            let result = originalMethod.apply(this, args);
+            //if the result looks like a promise, log if there's a rejection
+            if (result?.then) {
+                return Promise.resolve(result).catch((e: Error) => {
+                    if (e?.stack) {
+                        e.message = e.stack;
+                    }
+                    return Promise.reject(e);
+                });
+            } else {
+                return result;
+            }
+        } catch (e) {
+            if (e?.stack) {
+                e.message = e.stack;
+            }
+            throw e;
+        }
+    };
+}

tsconfig.json

@@ -10,6 +10,7 @@
         "strict": true,
         "strictNullChecks": false,
         "forceConsistentCasingInFileNames": true,
+        "experimentalDecorators": true,
         "preserveConstEnums": true,
         "downlevelIteration": true,
         "noUnusedLocals": true,