fix(cli/env): Don't crash if an environment directory is missing spec…

….json (#108)

* Don't crash if an environment directory is missing spec.json

* Better handling of missing spec.json file

cmd/tk/env.go

@@ -189,7 +189,12 @@ func envListCmd() *cobra.Command {
 		}
 		for _, dir := range dirs {
 			viper.Reset()
-			envs = append(envs, *setupConfiguration(dir))
+			env := setupConfiguration(dir)
+			if env == nil {
+				log.Printf("Could not setup configuration from %q", dir)
+				continue
+			}
+			envs = append(envs, *env)
 		}
 
 		if useJSON {

cmd/tk/other.go

@@ -20,9 +20,14 @@ func findBaseDirs() (dirs []string) {
 	}
 
 	if err := filepath.Walk(".", func(path string, info os.FileInfo, err error) error {
-		if _, err := os.Stat(filepath.Join(path, "main.jsonnet")); err == nil {
-			dirs = append(dirs, path)
+		requiredFiles := []string{"main.jsonnet", "spec.json"}
+		for _, name := range requiredFiles {
+			if _, err := os.Stat(filepath.Join(path, name)); err != nil {
+				// missing file, not a valid environment directory
+				return nil
+			}
 		}
+		dirs = append(dirs, path)
 		return nil
 	}); err != nil {
 		log.Fatalln(err)