fix: let Python Envs extension handle missing python in conda envs (#24986)

fixes microsoft/vscode-python-environments#289

Author: karthiknadig

src/client/envExt/api.legacy.ts

@@ -1,14 +1,14 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
 
-import { Terminal, Uri, WorkspaceFolder } from 'vscode';
+import { Terminal, Uri } from 'vscode';
 import { getEnvExtApi, getEnvironment } from './api.internal';
 import { EnvironmentType, PythonEnvironment as PythonEnvironmentLegacy } from '../pythonEnvironments/info';
 import { PythonEnvironment, PythonTerminalOptions } from './types';
 import { Architecture } from '../common/utils/platform';
 import { parseVersion } from '../pythonEnvironments/base/info/pythonVersion';
 import { PythonEnvType } from '../pythonEnvironments/base/info';
-import { traceError, traceInfo } from '../logging';
+import { traceError } from '../logging';
 import { reportActiveInterpreterChanged } from '../environmentApi';
 import { getWorkspaceFolder, getWorkspaceFolders } from '../common/vscodeApis/workspaceApis';
 
@@ -120,36 +120,6 @@ export async function getActiveInterpreterLegacy(resource?: Uri): Promise<Python
     return pythonEnv ? toLegacyType(pythonEnv) : undefined;
 }
 
-export async function ensureEnvironmentContainsPythonLegacy(
-    pythonPath: string,
-    workspaceFolder: WorkspaceFolder | undefined,
-    callback: () => void,
-): Promise<void> {
-    const api = await getEnvExtApi();
-    const pythonEnv = await api.resolveEnvironment(Uri.file(pythonPath));
-    if (!pythonEnv) {
-        traceError(`EnvExt: Failed to resolve environment for ${pythonPath}`);
-        return;
-    }
-
-    const envType = toEnvironmentType(pythonEnv);
-    if (envType === EnvironmentType.Conda) {
-        const packages = await api.getPackages(pythonEnv);
-        if (packages && packages.length > 0 && packages.some((pkg) => pkg.name.toLowerCase() === 'python')) {
-            return;
-        }
-        traceInfo(`EnvExt: Python not found in ${envType} environment ${pythonPath}`);
-        traceInfo(`EnvExt: Installing Python in ${envType} environment ${pythonPath}`);
-        await api.installPackages(pythonEnv, ['python']);
-        previousEnvMap.set(workspaceFolder?.uri.fsPath || '', pythonEnv);
-        reportActiveInterpreterChanged({
-            path: pythonPath,
-            resource: workspaceFolder,
-        });
-        callback();
-    }
-}
-
 export async function setInterpreterLegacy(pythonPath: string, uri: Uri | undefined): Promise<void> {
     const api = await getEnvExtApi();
     const pythonEnv = await api.resolveEnvironment(Uri.file(pythonPath));

src/client/interpreter/interpreterService.ts

@@ -45,7 +45,7 @@ import {
 } from '../pythonEnvironments/base/locator';
 import { sleep } from '../common/utils/async';
 import { useEnvExtension } from '../envExt/api.internal';
-import { ensureEnvironmentContainsPythonLegacy, getActiveInterpreterLegacy } from '../envExt/api.legacy';
+import { getActiveInterpreterLegacy } from '../envExt/api.legacy';
 
 type StoredPythonEnvironment = PythonEnvironment & { store?: boolean };
 
@@ -290,9 +290,6 @@ export class InterpreterService implements Disposable, IInterpreterService {
     @cache(-1, true)
     private async ensureEnvironmentContainsPython(pythonPath: string, workspaceFolder: WorkspaceFolder | undefined) {
         if (useEnvExtension()) {
-            await ensureEnvironmentContainsPythonLegacy(pythonPath, workspaceFolder, () => {
-                this.didChangeInterpreterEmitter.fire(workspaceFolder?.uri);
-            });
             return;
         }