No floating promises

.eslintrc.json

@@ -3,7 +3,8 @@
   "parser": "@typescript-eslint/parser",
   "parserOptions": {
     "ecmaVersion": 2020,
-    "sourceType": "module"
+    "sourceType": "module",
+    "project": true
   },
   "plugins": ["@typescript-eslint"],
   "rules": {
@@ -13,6 +14,7 @@
     // TODO "@typescript-eslint/semi" rule moved to https://eslint.style
     "semi": "error",
     "no-console": "warn",
+    "@typescript-eslint/no-floating-promises": "warn",
     // Mostly fails tests, ex. expect(...).to.be.true returns a Chai.Assertion
     "@typescript-eslint/no-unused-expressions": "off",
     "@typescript-eslint/no-non-null-assertion": "off",

scripts/tsconfig.json

@@ -0,0 +1,18 @@
+{
+  "compilerOptions": {
+    "composite": true,
+
+    "rootDir": ".",
+    "outDir": "dist",
+
+    "lib": ["ES2021"],
+    "target": "ES2020",
+    "module": "commonjs",
+
+    "strict": true,
+
+    "sourceMap": true,
+
+    "types": []
+  }
+}

src/BackgroundCompilation.ts

@@ -58,7 +58,7 @@ export class BackgroundCompilation implements vscode.Disposable {
             this.workspaceFileWatcher.onDidChange(
                 debounce(
                     () => {
-                        this.runTask();
+                        void this.runTask();
                     },
                     100 /* 10 times per second */,
                     { trailing: true }

src/FolderContext.ts

@@ -154,7 +154,7 @@ export class FolderContext implements vscode.Disposable {
      * @param event event type
      */
     async fireEvent(event: FolderOperation) {
-        this.workspaceContext.fireEvent(this, event);
+        await this.workspaceContext.fireEvent(this, event);
     }
 
     /** Return edited Packages folder */

src/PackageWatcher.ts

@@ -128,7 +128,7 @@ export class PackageWatcher {
     async handleSwiftVersionFileChange() {
         const version = await this.readSwiftVersionFile();
         if (version && version.toString() !== this.currentVersion?.toString()) {
-            this.workspaceContext.fireEvent(
+            await this.workspaceContext.fireEvent(
                 this.folderContext,
                 FolderOperation.swiftVersionUpdated
             );
@@ -162,7 +162,7 @@ export class PackageWatcher {
     async handlePackageSwiftChange() {
         // Load SwiftPM Package.swift description
         await this.folderContext.reload();
-        this.workspaceContext.fireEvent(this.folderContext, FolderOperation.packageUpdated);
+        await this.workspaceContext.fireEvent(this.folderContext, FolderOperation.packageUpdated);
     }
 
     /**
@@ -175,7 +175,10 @@ export class PackageWatcher {
         await this.folderContext.reloadPackageResolved();
         // if file contents has changed then send resolve updated message
         if (this.folderContext.swiftPackage.resolved?.fileHash !== packageResolvedHash) {
-            this.workspaceContext.fireEvent(this.folderContext, FolderOperation.resolvedUpdated);
+            await this.workspaceContext.fireEvent(
+                this.folderContext,
+                FolderOperation.resolvedUpdated
+            );
         }
     }
 
@@ -186,6 +189,9 @@ export class PackageWatcher {
      */
     private async handleWorkspaceStateChange() {
         await this.folderContext.reloadWorkspaceState();
-        this.workspaceContext.fireEvent(this.folderContext, FolderOperation.workspaceStateUpdated);
+        await this.workspaceContext.fireEvent(
+            this.folderContext,
+            FolderOperation.workspaceStateUpdated
+        );
     }
 }

src/TestExplorer/TestExplorer.ts

@@ -87,9 +87,9 @@ export class TestExplorer {
                 this.testFileEdited = false;
 
                 // only run discover tests if the library has tests
-                this.folderContext.swiftPackage.getTargets(TargetType.test).then(targets => {
+                void this.folderContext.swiftPackage.getTargets(TargetType.test).then(targets => {
                     if (targets.length > 0) {
-                        this.discoverTestsInWorkspace(this.tokenSource.token);
+                        void this.discoverTestsInWorkspace(this.tokenSource.token);
                     }
                 });
             }
@@ -98,7 +98,7 @@ export class TestExplorer {
         // add file watcher to catch changes to swift test files
         const fileWatcher = this.folderContext.workspaceContext.onDidChangeSwiftFiles(({ uri }) => {
             if (this.testFileEdited === false) {
-                this.folderContext.getTestTarget(uri).then(target => {
+                void this.folderContext.getTestTarget(uri).then(target => {
                     if (target) {
                         this.testFileEdited = true;
                     }
@@ -138,7 +138,7 @@ export class TestExplorer {
                 switch (operation) {
                     case FolderOperation.add:
                         if (folder) {
-                            folder.swiftPackage.getTargets(TargetType.test).then(targets => {
+                            void folder.swiftPackage.getTargets(TargetType.test).then(targets => {
                                 if (targets.length === 0) {
                                     return;
                                 }
@@ -149,7 +149,7 @@ export class TestExplorer {
                                 if (
                                     !configuration.folder(folder.workspaceFolder).disableAutoResolve
                                 ) {
-                                    folder.testExplorer?.discoverTestsInWorkspace(
+                                    void folder.testExplorer?.discoverTestsInWorkspace(
                                         tokenSource.token
                                     );
                                 }
@@ -158,7 +158,7 @@ export class TestExplorer {
                         break;
                     case FolderOperation.packageUpdated:
                         if (folder) {
-                            folder.swiftPackage.getTargets(TargetType.test).then(targets => {
+                            void folder.swiftPackage.getTargets(TargetType.test).then(targets => {
                                 const hasTestTargets = targets.length > 0;
                                 if (hasTestTargets && !folder.hasTestExplorer()) {
                                     folder.addTestExplorer();
@@ -168,7 +168,7 @@ export class TestExplorer {
                                         !configuration.folder(folder.workspaceFolder)
                                             .disableAutoResolve
                                     ) {
-                                        folder.testExplorer?.discoverTestsInWorkspace(
+                                        void folder.testExplorer?.discoverTestsInWorkspace(
                                             tokenSource.token
                                         );
                                     }
@@ -219,7 +219,7 @@ export class TestExplorer {
         const testExplorer = folder?.testExplorer;
         if (testExplorer && symbols && uri && uri.scheme === "file") {
             if (isPathInsidePath(uri.fsPath, folder.folder.fsPath)) {
-                folder.swiftPackage.getTarget(uri.fsPath).then(target => {
+                void folder.swiftPackage.getTarget(uri.fsPath).then(target => {
                     if (target && target.type === "test") {
                         testExplorer.lspTestDiscovery
                             .getDocumentTests(folder.swiftPackage, uri)

src/TestExplorer/TestParsers/SwiftTestingOutputParser.ts

@@ -211,7 +211,7 @@ export class SwiftTestingOutputParser {
 
         rl.on("line", line => this.parse(JSON.parse(line), runState));
 
-        reader.start(readlinePipe);
+        void reader.start(readlinePipe);
     }
 
     /**

src/TestExplorer/TestRunner.ts

@@ -729,7 +729,7 @@ export class TestRunner {
                 // discarded. If the test run has already started this is a no-op so its safe to call it multiple times.
                 this.testRun.testRunStarted();
 
-                this.swiftTestOutputParser.close();
+                void this.swiftTestOutputParser.close();
             }
         } finally {
             outputStream.end();
@@ -807,7 +807,7 @@ export class TestRunner {
                 }
             });
 
-            this.folderContext.taskQueue.queueOperation(
+            void this.folderContext.taskQueue.queueOperation(
                 new TaskOperation(task),
                 this.testRun.token
             );

src/WorkspaceContext.ts

@@ -133,7 +133,7 @@ export class WorkspaceContext implements vscode.Disposable {
                     break;
                 case FolderOperation.focus:
                     this.updateContextKeys(event.folder);
-                    this.updateContextKeysForFile();
+                    void this.updateContextKeysForFile();
                     break;
                 case FolderOperation.unfocus:
                     this.updateContextKeys(event.folder);
@@ -232,7 +232,7 @@ export class WorkspaceContext implements vscode.Disposable {
             return;
         }
 
-        Promise.all([
+        void Promise.all([
             folderContext.swiftPackage.foundPackage,
             folderContext.swiftPackage.dependencies,
         ]).then(([foundPackage, dependencies]) => {
@@ -256,7 +256,7 @@ export class WorkspaceContext implements vscode.Disposable {
 
         if (this.currentFolder) {
             const languageClient = this.languageClientManager.get(this.currentFolder);
-            languageClient.useLanguageClient(async client => {
+            await languageClient.useLanguageClient(async client => {
                 const experimentalCaps = client.initializeResult?.capabilities.experimental;
                 if (!experimentalCaps) {
                     contextKeys.supportsReindexing = false;
@@ -296,7 +296,7 @@ export class WorkspaceContext implements vscode.Disposable {
                 console.log("Trying to run onDidChangeWorkspaceFolders on deleted context");
                 return;
             }
-            this.onDidChangeWorkspaceFolders(event);
+            void this.onDidChangeWorkspaceFolders(event);
         });
         // add event listener for when the active edited text document changes
         const onDidChangeActiveWindow = vscode.window.onDidChangeActiveTextEditor(async editor => {
@@ -327,7 +327,7 @@ export class WorkspaceContext implements vscode.Disposable {
                 await this.focusFolder(null);
             }
         }
-        this.initialisationComplete();
+        await this.initialisationComplete();
     }
 
     /**
@@ -450,7 +450,7 @@ export class WorkspaceContext implements vscode.Disposable {
             // if current folder is this folder send unfocus event by setting
             // current folder to undefined
             if (this.currentFolder === folder) {
-                this.focusFolder(null);
+                await this.focusFolder(null);
             }
             // run observer functions in reverse order when removing
             const observersReversed = [...this.observers];
@@ -519,10 +519,10 @@ export class WorkspaceContext implements vscode.Disposable {
         }
     }
 
-    private initialisationComplete() {
+    private async initialisationComplete() {
         this.initialisationFinished = true;
         if (this.lastFocusUri) {
-            this.focusUri(this.lastFocusUri);
+            await this.focusUri(this.lastFocusUri);
             this.lastFocusUri = undefined;
         }
     }

src/commands/createNewProject.ts

@@ -30,7 +30,7 @@ export async function createNewProject(toolchain: SwiftToolchain | undefined): P
     // run before the Swift extension is activated. Show the toolchain error notification in
     // this case.
     if (!toolchain) {
-        showToolchainError();
+        void showToolchainError();
         return;
     }
 

src/commands/dependencies/edit.ts

@@ -47,7 +47,7 @@ export async function editDependency(identifier: string, ctx: WorkspaceContext)
     );
 
     if (success) {
-        ctx.fireEvent(currentFolder, FolderOperation.resolvedUpdated);
+        await ctx.fireEvent(currentFolder, FolderOperation.resolvedUpdated);
         // add folder to workspace
         const index = vscode.workspace.workspaceFolders?.length ?? 0;
         vscode.workspace.updateWorkspaceFolders(index, 0, {

src/commands/dependencies/unedit.ts

@@ -60,7 +60,7 @@ async function uneditFolderDependency(
         );
         await folder.taskQueue.queueOperation(uneditOperation);
 
-        ctx.fireEvent(folder, FolderOperation.resolvedUpdated);
+        await ctx.fireEvent(folder, FolderOperation.resolvedUpdated);
         // find workspace folder, and check folder still exists
         const folderIndex = vscode.workspace.workspaceFolders?.findIndex(
             item => item.name === identifier

src/commands/dependencies/updateDepViewList.ts

@@ -18,6 +18,6 @@ import { FolderOperation, WorkspaceContext } from "../../WorkspaceContext";
 export function updateDependenciesViewList(ctx: WorkspaceContext, flatList: boolean) {
     if (ctx.currentFolder) {
         contextKeys.flatDependenciesList = flatList;
-        ctx.fireEvent(ctx.currentFolder, FolderOperation.packageViewUpdated);
+        void ctx.fireEvent(ctx.currentFolder, FolderOperation.packageViewUpdated);
     }
 }

src/commands/dependencies/useLocal.ts

@@ -73,7 +73,7 @@ export async function useLocalDependency(
         true
     );
     if (success) {
-        ctx.fireEvent(currentFolder, FolderOperation.resolvedUpdated);
+        await ctx.fireEvent(currentFolder, FolderOperation.resolvedUpdated);
     }
     return success;
 }

src/commands/insertFunctionComment.ts

@@ -26,5 +26,5 @@ export async function insertFunctionComment(workspaceContext: WorkspaceContext)
         return;
     }
     const line = activeEditor.selection.active.line;
-    workspaceContext.commentCompletionProvider.insert(activeEditor, line);
+    await workspaceContext.commentCompletionProvider.insert(activeEditor, line);
 }

src/commands/testMultipleTimes.ts

@@ -71,7 +71,7 @@ export async function runTestMultipleTimes(
             break;
         }
     }
-    runner.testRun.end();
+    await runner.testRun.end();
 
     return runStates;
 }

src/commands/utilities.ts

@@ -71,6 +71,6 @@ export function updateAfterError(result: boolean, folderContext: FolderContext)
     folderContext.hasResolveErrors = !result;
 
     if (triggerResolvedUpdatedEvent && !folderContext.hasResolveErrors) {
-        folderContext.fireEvent(FolderOperation.resolvedUpdated);
+        void folderContext.fireEvent(FolderOperation.resolvedUpdated);
     }
 }

src/configuration.ts

@@ -532,7 +532,7 @@ export function handleConfigurationChangeEvent(
             event.affectsConfiguration("swift.path") &&
             configuration.path !== ctx.currentFolder?.toolchain.swiftFolderPath
         ) {
-            showReloadExtensionNotification(
+            void showReloadExtensionNotification(
                 "Changing the Swift path requires Visual Studio Code be reloaded."
             );
         } else if (
@@ -542,7 +542,7 @@ export function handleConfigurationChangeEvent(
         ) {
             vscode.commands.executeCommand("swift.restartLSPServer");
         } else if (event.affectsConfiguration("swift.swiftEnvironmentVariables")) {
-            showReloadExtensionNotification(
+            void showReloadExtensionNotification(
                 "Changing environment variables requires the project be reloaded."
             );
         }

src/coverage/LcovResults.ts

@@ -88,7 +88,7 @@ export class TestCoverage {
                 this.coverageDetails.set(uri, detailedCoverage);
             }
         }
-        this.lcovTmpFiles.dispose();
+        await this.lcovTmpFiles.dispose();
     }
 
     /**

src/documentation/DocumentationPreviewEditor.ts

@@ -152,7 +152,7 @@ export class DocumentationPreviewEditor implements vscode.Disposable {
                 if (!this.activeTextEditor) {
                     break;
                 }
-                this.convertDocumentation(this.activeTextEditor);
+                void this.convertDocumentation(this.activeTextEditor);
                 break;
             case "rendered":
                 this.renderEmitter.fire();
@@ -166,7 +166,7 @@ export class DocumentationPreviewEditor implements vscode.Disposable {
         }
         this.activeTextEditor = activeTextEditor;
         this.activeTextEditorSelection = activeTextEditor.selection;
-        this.convertDocumentation(activeTextEditor);
+        void this.convertDocumentation(activeTextEditor);
     }
 
     private handleSelectionChange(event: vscode.TextEditorSelectionChangeEvent) {
@@ -177,12 +177,12 @@ export class DocumentationPreviewEditor implements vscode.Disposable {
             return;
         }
         this.activeTextEditorSelection = event.textEditor.selection;
-        this.convertDocumentation(event.textEditor);
+        void this.convertDocumentation(event.textEditor);
     }
 
     private handleDocumentChange(event: vscode.TextDocumentChangeEvent) {
         if (this.activeTextEditor?.document === event.document) {
-            this.convertDocumentation(this.activeTextEditor);
+            void this.convertDocumentation(this.activeTextEditor);
         }
     }
 

src/documentation/webview/webview.ts

@@ -11,6 +11,7 @@
 // SPDX-License-Identifier: Apache-2.0
 //
 //===----------------------------------------------------------------------===//
+/* eslint-disable @typescript-eslint/no-floating-promises */
 
 import { RenderNode, WebviewContent, WebviewMessage } from "./WebviewMessage";
 import { createCommunicationBridge } from "./CommunicationBridge";