Add comments

Author: Kartik Raj

src/client/interpreter/interpreterService.ts

@@ -265,6 +265,7 @@ export class InterpreterService implements Disposable, IInterpreterService {
                 .then(async () => {
                     // Fetch interpreter details so the cache is updated to include the newly installed Python.
                     await this.getInterpreterDetails(pythonPath);
+                    // Fire an event as the executable for the environment has changed.
                     this.didChangeInterpreterEmitter.fire(workspaceFolder?.uri);
                     reportActiveInterpreterChanged({
                         path: pythonPath,

src/client/pythonEnvironments/base/locators/composite/envsCollectionCache.ts

@@ -150,12 +150,9 @@ export class PythonEnvInfoCache extends PythonEnvsWatcher<PythonEnvCollectionCha
         if (!found) {
             this.envs.push(env);
             this.fire({ new: env });
-        } else if (hasLatestInfo) {
-            const isValidated = this.validatedEnvs.has(env.id!);
-            if (!isValidated) {
-                // Update cache only if we have latest info and the env is not already validated.
-                this.updateEnv(found, env, true);
-            }
+        } else if (hasLatestInfo && this.validatedEnvs.has(env.id!)) {
+            // Update cache if we have latest info and the env is not already validated.
+            this.updateEnv(found, env, true);
         }
         if (hasLatestInfo) {
             traceVerbose(`Flushing env to cache ${env.id}`);