Add settings to disable automatic loading of extension and console

This change adds two new settings to control the automatic loading of the
PowerShell extension and integrated console:

- `powershell.startAutomatically`
- `powershell.integratedConsole.showOnStartup`

Both of which default to `true`.

Resolves #580.

Author: daviwil

package.json

@@ -305,6 +305,11 @@
       "type": "object",
       "title": "PowerShell Configuration",
       "properties": {
+        "powershell.startAutomatically": {
+          "type": "boolean",
+          "default": true,
+          "description": "If true, causes PowerShell extension features to start automatically when a PowerShell file is opened.  If false, the user must initiate startup using the 'PowerShell: Restart Current Session' command.  IntelliSense, code navigation, integrated console, code formatting, and other features will not be enabled until the extension has been started."
+        },
         "powershell.useX86Host": {
           "type": "boolean",
           "default": false,
@@ -393,6 +398,11 @@
           "type": "boolean",
           "default": true,
           "description": "Ignore blocks of code on one line. For example, if true, the braces in \"if (...) {...} else {...}\", will not be formatted."
+        },
+        "powershell.integratedConsole.showOnStartup": {
+          "type": "boolean",
+          "default": true,
+          "description": "If true, causes the integrated console to be shown automatically when the PowerShell extension is initialized."
         }
       }
     }

src/logging.ts

@@ -49,13 +49,10 @@ export class Logger {
 
     public writeAtLevel(logLevel: LogLevel, message: string, ...additionalMessages: string[]) {
         if (logLevel >= this.MinimumLogLevel) {
-            // TODO: Add timestamp
-            this.logChannel.appendLine(message);
-            fs.appendFile(this.logFilePath, message + os.EOL);
+            this.writeLine(message)
 
             additionalMessages.forEach((line) => {
-                this.logChannel.appendLine(line);
-                fs.appendFile(this.logFilePath, line + os.EOL);
+                this.writeLine(message);
             });
         }
     }
@@ -138,6 +135,14 @@ export class Logger {
                 true);
         }
     }
+
+    private writeLine(message: string) {
+        // TODO: Add timestamp
+        this.logChannel.appendLine(message);
+        if (this.logFilePath) {
+            fs.appendFile(this.logFilePath, message + os.EOL);
+        }
+    }
 }
 
 export class LanguageClientLogger implements ILogger {

src/main.ts

@@ -6,6 +6,7 @@
 
 import vscode = require('vscode');
 import utils = require('./utils');
+import Settings = require('./settings');
 import { Logger, LogLevel } from './logging';
 import { IFeature } from './feature';
 import { SessionManager } from './session';
@@ -118,7 +119,10 @@ export function activate(context: vscode.ExtensionContext): void {
             logger,
             extensionFeatures);
 
-    sessionManager.start();
+    var extensionSettings = Settings.load(utils.PowerShellLanguageId);
+    if (extensionSettings.startAutomatically) {
+        sessionManager.start();
+    }
 }
 
 export function deactivate(): void {

src/session.ts

@@ -99,13 +99,14 @@ export class SessionManager {
         this.hostVersion = this.hostVersion.split('-')[0];
 
         this.registerCommands();
-        this.createStatusBarItem();
     }
 
     public start(sessionConfig: SessionConfiguration = { type: SessionType.UseDefault }) {
         this.sessionSettings = Settings.load(utils.PowerShellLanguageId);
         this.log.startNewLog(this.sessionSettings.developer.editorServicesLogLevel);
 
+        this.createStatusBarItem();
+
         this.sessionConfiguration = this.resolveSessionConfiguration(sessionConfig);
 
         if (this.sessionConfiguration.type === SessionType.UsePath ||
@@ -317,7 +318,9 @@ export class SessionManager {
                     powerShellExePath,
                     powerShellArgs);
 
-            this.consoleTerminal.show();
+            if (this.sessionSettings.integratedConsole.showOnStartup) {
+                this.consoleTerminal.show(true);
+            }
 
             // Start the language client
             utils.waitForSessionFile(
@@ -490,7 +493,7 @@ export class SessionManager {
     }
 
     private createStatusBarItem() {
-        if (this.statusBarItem == undefined) {
+        if (this.statusBarItem === undefined) {
             // Create the status bar item and place it right next
             // to the language indicator
             this.statusBarItem =

src/settings.ts

@@ -32,11 +32,17 @@ export interface IDeveloperSettings {
 }
 
 export interface ISettings {
+    startAutomatically?: boolean;
     useX86Host?: boolean;
     enableProfileLoading?: boolean;
     scriptAnalysis?: IScriptAnalysisSettings;
     developer?: IDeveloperSettings;
     codeFormatting?: ICodeFormattingSettings;
+    integratedConsole?: IIntegratedConsoleSettings;
+}
+
+export interface IIntegratedConsoleSettings {
+    showOnStartup?: boolean;
 }
 
 export function load(myPluginId: string): ISettings {
@@ -67,11 +73,17 @@ export function load(myPluginId: string): ISettings {
         ignoreOneLineBlock: true
     };
 
+    let defaultIntegratedConsoleSettings: IIntegratedConsoleSettings = {
+        showOnStartup: true,
+    };
+
     return {
+        startAutomatically: configuration.get<boolean>("startAutomatically", true),
         useX86Host: configuration.get<boolean>("useX86Host", false),
         enableProfileLoading: configuration.get<boolean>("enableProfileLoading", false),
         scriptAnalysis: configuration.get<IScriptAnalysisSettings>("scriptAnalysis", defaultScriptAnalysisSettings),
         developer: configuration.get<IDeveloperSettings>("developer", defaultDeveloperSettings),
-        codeFormatting: configuration.get<ICodeFormattingSettings>("codeFormatting", defaultCodeFormattingSettings)
+        codeFormatting: configuration.get<ICodeFormattingSettings>("codeFormatting", defaultCodeFormattingSettings),
+        integratedConsole: configuration.get<IIntegratedConsoleSettings>("integratedConsole", defaultIntegratedConsoleSettings)
     };
 }