Move restart to separate function for clarity

Author: JustinGrote

src/extension.ts

@@ -43,17 +43,8 @@ const documentSelector: DocumentSelector = [
 
 export async function activate(context: vscode.ExtensionContext): Promise<IPowerShellExtensionClient> {
     logger = new Logger();
-    // Enables a reload-on-save for faster development
     if (context.extensionMode === vscode.ExtensionMode.Development) {
-        const watcher = vscode.workspace.createFileSystemWatcher(
-            new vscode.RelativePattern(context.extensionPath, "dist/*.js"),
-        );
-
-        context.subscriptions.push(watcher);
-        watcher.onDidChange(({ fsPath }) => {
-            vscode.window.showInformationMessage(`${fsPath.split(context.extensionPath,2)[1]} changed. Reloading Extension Host...`);
-            vscode.commands.executeCommand("workbench.action.restartExtensionHost");
-        });
+        restartOnExtensionFileChanges(context);
     }
 
     // Register a command that waits for the Externsion Terminal to be active. Can be used by .NET Attach Tasks
@@ -229,6 +220,19 @@ export async function activate(context: vscode.ExtensionContext): Promise<IPower
     };
 }
 
+/** Restarts the extension host when extension file changes are detected. Useful for development.. */
+function restartOnExtensionFileChanges(context: vscode.ExtensionContext): void {
+    const watcher = vscode.workspace.createFileSystemWatcher(
+        new vscode.RelativePattern(context.extensionPath, "dist/*.js")
+    );
+
+    context.subscriptions.push(watcher);
+    watcher.onDidChange(({ fsPath }) => {
+        vscode.window.showInformationMessage(`${fsPath.split(context.extensionPath, 2)[1]} changed. Reloading Extension Host...`);
+        vscode.commands.executeCommand("workbench.action.restartExtensionHost");
+    });
+}
+
 export async function deactivate(): Promise<void> {
     // Clean up all extension features
     for (const commandRegistration of commandRegistrations) {