sensible shell defaults for Windows (fixes #431)

terminus-terminal/src/shells/winDefault.ts

@@ -0,0 +1,50 @@
+import { exec } from 'mz/child_process'
+import { Injectable } from '@angular/core'
+import { HostAppService, Platform } from 'terminus-core'
+
+import { ShellProvider, IShell } from '../api'
+
+import { WSLShellProvider } from './wsl'
+import { PowerShellCoreShellProvider } from './powershellCore'
+import { WindowsStockShellsProvider } from './windowsStock'
+
+
+@Injectable()
+export class WindowsDefaultShellProvider extends ShellProvider {
+    private providers: ShellProvider[]
+
+    constructor (
+        psc: PowerShellCoreShellProvider,
+        wsl: WSLShellProvider,
+        stock: WindowsStockShellsProvider,
+        private hostApp: HostAppService,
+    ) {
+        super()
+        this.providers = [
+            psc
+            wsl,
+            stock,
+        ]
+    }
+
+    async provide (): Promise<IShell[]> {
+        if (this.hostApp.platform !== Platform.Windows) {
+            return []
+        }
+        // Figure out a sensible default
+        let shellLists = await Promise.all(this.providers.map(x => x.provide()))
+        for (let list of shellLists) {
+            if (list.length) {
+                let shell = list[list.length - 1]
+
+                return [{
+                    ...shell,
+                    id: 'default',
+                    name: 'User default',
+                }]
+            }
+        }
+
+        return []
+    }
+}

terminus-terminal/src/shells/windowsStock.ts

@@ -36,7 +36,7 @@ export class WindowsStockShellsProvider extends ShellProvider {
             { id: 'cmd', name: 'CMD (stock)', command: 'cmd.exe' },
             {
                 id: 'powershell',
-                name: 'Windows PowerShell',
+                name: 'PowerShell',
                 command: 'powershell.exe',
                 env: {
                     TERM: 'cygwin',