more sensible err handling

Author: aviatesk

lib/runtime/environments.js

@@ -31,7 +31,7 @@ export function consumeStatusBar (statusBar) {
   const showTile = () => tileDom.style.display = ''
   const hideTile = () => tileDom.style.display = 'none'
   const updateTile = (proj) => {
-    if (!proj) return
+    if (!proj) return hideTile()
     projectName = proj.name
     dom.innerText = 'Env: ' + projectName
     projectPath = proj.path
@@ -55,20 +55,24 @@ export function consumeStatusBar (statusBar) {
       }
     } else {
       client.require('choose environment', () => {
-        const p = allProjects()
-          .then(proj => {
-            return proj.map(proj => {
+        allProjects()
+          .then(projs => {
+            if (!projs) throw '`allProject` handler unsupported'
+            if (projs.length === 0) throw 'no environment found'
+            return projs.map(proj => {
               proj.primary = proj.name
               proj.secondary = proj.path
               return proj
             })
           })
+          .then(projs => {
+            show(projs, { active: projectName }).then(proj => {
+              if (!proj) return
+              const dir = path.dirname(proj.path)
+              activateProject(dir)
+            })
+          })
           .catch(err => console.log(err))
-        show(p, { active: projectName }).then(proj => {
-          if (!proj) return
-          const dir = path.dirname(proj.path)
-          activateProject(dir)
-        })
       })
     }
   }