Changed warning when using .mirrord config folder to info. closes #29

package.json

@@ -87,6 +87,11 @@
 					"default": true,
 					"description": "Show a notification when mirrord uses active configuration."
 				},
+				"mirrord.promptUsingEnvVarConfig": {
+					"type": "boolean",
+					"default": true,
+					"description": "Show a notification when mirrord uses env var configuration."
+				},
 				"mirrord.promptUsingDefaultConfig": {
 					"type": "boolean",
 					"default": true,

src/config.ts

@@ -78,6 +78,17 @@ export function isTargetSet(verifiedConfig: VerifiedConfig): boolean {
   }
 }
 
+/**
+* Notifies the user we're running with a default config when `resolveMirrordConfig` 
+* returns `null`.
+*/
+export function notifyRunningWithDefaultConfig() {
+  new NotificationBuilder()
+    .withMessage(`Using default mirrord configuration.`)
+    .withDisableAction("promptUsingDefaultConfig")
+    .info();
+}
+
 export class MirrordConfigManager {
   private static instance?: MirrordConfigManager = undefined;
 
@@ -296,43 +307,40 @@ export class MirrordConfigManager {
     if (this.active) {
       // User has selected a config (via active config button).
       new NotificationBuilder()
-        .withMessage("using active mirrord configuration")
+        .withMessage("Using active mirrord configuration.")
         .withOpenFileAction(this.active)
         .withDisableAction("promptUsingActiveConfig")
         .info();
+
       return this.active;
     } else if (config.env?.["MIRRORD_CONFIG_FILE"]) {
       // Get the config path from the env var.
-      return vscode.Uri.parse(`file://${config.env?.["MIRRORD_CONFIG_FILE"]}`, true);
+      const configFromEnv = vscode.Uri.parse(`file://${config.env?.["MIRRORD_CONFIG_FILE"]}`, true);
+
+      new NotificationBuilder()
+        .withMessage(`Using mirrord configuration from env var "MIRRORD_CONFIG_FILE".`)
+        .withOpenFileAction(configFromEnv)
+        .withDisableAction("promptUsingEnvVarConfig")
+        .info();
+
+      return configFromEnv;
+
     } else if (folder) {
-      let predefinedConfig = await MirrordConfigManager.getDefaultConfig(folder);
-      if (predefinedConfig) {
-        new NotificationBuilder()
-          .withMessage("using a default mirrord config")
-          .withOpenFileAction(predefinedConfig)
-          .withDisableAction("promptUsingDefaultConfig")
-          .warning();
-        return predefinedConfig;
-      } else {
-        return null;
-      }
-    } else {
-      folder = vscode.workspace.workspaceFolders?.[0];
-      if (!folder) {
-        throw new Error("mirrord requires an open folder in the workspace");
-      }
+      const configFromMirrordFolder = await MirrordConfigManager.getDefaultConfig(folder);
 
-      let predefinedConfig = await MirrordConfigManager.getDefaultConfig(folder);
-      if (predefinedConfig) {
+      if (configFromMirrordFolder) {
         new NotificationBuilder()
-          .withMessage(`using a default mirrord config from folder ${folder.name} `)
-          .withOpenFileAction(predefinedConfig)
+          .withMessage(`Using mirrord configuration from ".mirrord" folder.`)
+          .withOpenFileAction(configFromMirrordFolder)
           .withDisableAction("promptUsingDefaultConfig")
-          .warning();
-        return predefinedConfig;
+          .info();
+
+        return configFromMirrordFolder;
       } else {
         return null;
       }
+    } else {
+      return null;
     }
   }
 }

src/debugger.ts

@@ -1,6 +1,6 @@
 import * as vscode from 'vscode';
 import { globalContext } from './extension';
-import { isTargetSet, MirrordConfigManager } from './config';
+import { isTargetSet, MirrordConfigManager, notifyRunningWithDefaultConfig } from './config';
 import { LAST_TARGET_KEY, MirrordAPI, mirrordFailure, MirrordExecution } from './api';
 import { updateTelemetries } from './versionCheck';
 import { getMirrordBinary } from './binaryManager';
@@ -95,8 +95,8 @@ async function main(
   let cliPath = await getMirrordBinary(false);
 
   if (!cliPath) {
-      mirrordFailure(`Couldn't download mirrord binaries or find local one in path`);
-      return null;
+    mirrordFailure(`Couldn't download mirrord binaries or find local one in path`);
+    return null;
   }
 
   let mirrordApi = new MirrordAPI(cliPath);
@@ -105,6 +105,8 @@ async function main(
   let target = null;
 
   let configPath = await MirrordConfigManager.getInstance().resolveMirrordConfig(folder, config);
+  if (configPath === null) { notifyRunningWithDefaultConfig(); }
+
   const verifiedConfig = await mirrordApi.verifyConfig(configPath, config.env);
 
   // If target wasn't specified in the config file (or there's no config file), let user choose pod from dropdown