show only valid esp idf setups

src/setup/existingIdfSetups.ts

@@ -20,22 +20,32 @@ import { ESP } from "../config";
 import { getEspIdfFromCMake } from "../utils";
 import { IdfSetup } from "../views/setup/types";
 import { getIdfMd5sum, loadEspIdfJson } from "./espIdfJson";
+import { checkIdfSetup } from "./setupValidation/espIdfSetup";
 
-export function getPreviousIdfSetups() {
+export async function getPreviousIdfSetups() {
   const setupKeys = ESP.GlobalConfiguration.store.getIdfSetupKeys();
   const idfSetups: IdfSetup[] = [];
   for (let idfSetupKey of setupKeys) {
     let idfSetup = ESP.GlobalConfiguration.store.get<IdfSetup>(
       idfSetupKey,
       undefined
     );
+    idfSetup.isValid = await checkIdfSetup(idfSetup);
+    idfSetup.version = await getEspIdfFromCMake(idfSetup.idfPath);
     if (idfSetup) {
       idfSetups.push(idfSetup);
     }
   }
   return idfSetups;
 }
 
+export async function clearPreviousIdfSetups() {
+  const setupKeys = ESP.GlobalConfiguration.store.getIdfSetupKeys();
+  for (let idfSetupKey of setupKeys) {
+    ESP.GlobalConfiguration.store.clear(idfSetupKey);
+  }
+}
+
 export async function createIdfSetup(
   idfPath: string,
   toolsPath: string,
@@ -51,7 +61,9 @@ export async function createIdfSetup(
     toolsPath,
     python: pythonPath,
     version: idfVersion,
+    isValid: false
   };
+  newIdfSetup.isValid = await checkIdfSetup(newIdfSetup);
   addIdfSetup(newIdfSetup);
 }
 
@@ -73,14 +85,17 @@ export async function loadIdfSetupsFromEspIdfJson(toolsPath: string) {
   ) {
     let idfSetups: IdfSetup[] = [];
     for (let idfInstalledKey of Object.keys(espIdfJson.idfInstalled)) {
-      idfSetups.push({
+      let setupConf: IdfSetup = {
         id: idfInstalledKey,
         idfPath: espIdfJson.idfInstalled[idfInstalledKey].path,
         gitPath: espIdfJson.gitPath,
         python: espIdfJson.idfInstalled[idfInstalledKey].python,
         version: espIdfJson.idfInstalled[idfInstalledKey].version,
         toolsPath: toolsPath,
-      });
+        isValid: false
+      } as IdfSetup;
+      setupConf.isValid = await checkIdfSetup(setupConf);
+      idfSetups.push(setupConf);
     }
     return idfSetups;
   }

src/setup/setupInit.ts

@@ -131,7 +131,7 @@ export async function getSetupInitialValues(
   const espIdfTagsList = await getEspIdfTags();
   progress.report({ increment: 10, message: "Getting Python versions..." });
   const pythonVersions = await getPythonList(extensionPath);
-  const idfSetups = getPreviousIdfSetups();
+  const idfSetups = await getPreviousIdfSetups();
   const setupInitArgs = {
     espIdfVersionsList,
     espIdfTagsList,

src/views/setup/ExistingSetup.vue

@@ -1,35 +1,51 @@
 <template>
-  <div class="centerize notification">
-    <div
-      class="notification install-choice"
-      v-for="(prevSetup, i) in idfSetups"
-      :key="prevSetup.id"
-      :data-config-id="prevSetup.idfPath"
-      @click="useIdfSetup(i)"
-    >
-      <label :for="prevSetup.id" class="subtitle">
-        {{ prevSetup.idfPath }}</label
+  <div class="notification">
+    <div class="notification install-choice" @click="goTo('/autoinstall', setupMode.express)">
+      <label class="subtitle"> Search ESP-IDF in system...</label>
+    </div>
+    <div v-for="(prevSetup, i) in idfSetups" :key="prevSetup.id" class="my-1">
+      <div
+        class="notification install-choice"
+        v-show="prevSetup.isValid"
+        :data-config-id="prevSetup.idfPath"
+        @click="useIdfSetup(i)"
       >
-      <p>IDF Version {{ prevSetup.version }}</p>
-      <p>Python: {{ prevSetup.python }}</p>
-      <p>IDF Tools Path: {{ prevSetup.toolsPath }}</p>
-      <p>Git path: {{ prevSetup.gitPath }}</p>
+        <label :for="prevSetup.id" class="subtitle">
+          {{ prevSetup.idfPath }}</label
+        >
+        <p>IDF Version {{ prevSetup.version }}</p>
+        <p>Python: {{ prevSetup.python }}</p>
+        <p>IDF Tools Path: {{ prevSetup.toolsPath }}</p>
+        <p>Git path: {{ prevSetup.gitPath }}</p>
+        <p>Is valid {{ prevSetup.isValid }}</p>
+      </div>
     </div>
   </div>
 </template>
 
 <script lang="ts">
 import { Component, Vue } from "vue-property-decorator";
-import { Action, State } from "vuex-class";
-import { IdfSetup } from "./types";
+import { Action, Mutation, State } from "vuex-class";
+import { IdfSetup, SetupMode } from "./types";
+import { router } from "./main";
 
 @Component
 export default class existingSetup extends Vue {
   @State("idfSetups") private storeIdfSetups: IdfSetup[];
   @Action useIdfSetup: (id: number) => void;
+  @Mutation setSetupMode: (mode: SetupMode) => void;
 
   get idfSetups() {
     return this.storeIdfSetups;
   }
+
+  get setupMode() {
+    return SetupMode;
+  }
+
+  goTo(route: string, setupMode: SetupMode) {
+    router.push(route);
+    this.setSetupMode(setupMode);
+  }
 }
 </script>

src/views/setup/Home.vue

@@ -72,7 +72,7 @@
         <label for="existing" class="subtitle" data-config-id="existing-setup"
           >USE EXISTING SETUP</label
         >
-        <p>Select existing ESP-IDF setup saved in the extension.</p>
+        <p>Select existing ESP-IDF setup saved in the extension or find ESP-IDF in your system.</p>
       </div>
     </div>
   </div>

src/views/setup/types.ts

@@ -26,6 +26,7 @@ export interface IdfSetup {
   toolsPath: string;
   idfPath: string;
   gitPath: string;
+  isValid: boolean;
 }
 
 export enum IdfMirror {