fix potential issue

Author: StellaHuang95

src/extension.ts

@@ -98,6 +98,7 @@ import { registerSystemPythonFeatures } from './managers/builtin/main';
 import { SysPythonManager } from './managers/builtin/sysPythonManager';
 import {
     createNativePythonFinder,
+    isNativeEnvInfo,
     NativePythonEnvironmentKind,
     NativePythonFinder,
 } from './managers/common/nativePythonFinder';
@@ -137,6 +138,18 @@ async function checkPetManagerMismatch(
         ['poetry', 'ms-python.python:poetry'],
     ]);
 
+    // Use a single cached refresh to avoid triggering per-kind hard refreshes on cache miss
+    let allEnvs;
+    try {
+        allEnvs = await nativeFinder.refresh(false);
+    } catch {
+        // PET query failed — don't block post-init
+        return;
+    }
+
+    // Filter to only environment entries (not manager entries)
+    const envInfos = allEnvs.filter(isNativeEnvInfo);
+
     // Group PET kinds by manager name to avoid double-counting (e.g. pyenv + pyenvVirtualEnv)
     const kindsByManager = new Map<string, NativePythonEnvironmentKind[]>();
     for (const [kind, managerName] of PET_KIND_TO_MANAGER) {
@@ -157,16 +170,8 @@ async function checkPetManagerMismatch(
             continue;
         }
 
-        // Manager not registered — check if PET found environments of any related kind
-        let totalPetEnvs = 0;
-        for (const kind of kinds) {
-            try {
-                const petEnvs = await nativeFinder.refresh(false, kind);
-                totalPetEnvs += petEnvs.length;
-            } catch {
-                // PET query failed — don't block post-init; skip this kind
-            }
-        }
+        // Manager not registered — count PET environments of any related kind
+        const totalPetEnvs = envInfos.filter((e) => e.kind !== undefined && kinds.includes(e.kind)).length;
 
         if (totalPetEnvs > 0) {
             sendTelemetryEvent(EventNames.MANAGER_DISCOVERY_MISMATCH, undefined, {

src/managers/conda/main.ts

@@ -25,13 +25,18 @@ export async function registerCondaFeatures(
         // get Conda will return only ONE conda manager, that correlates to a single conda install
         condaPath = await getConda(nativeFinder);
     } catch (ex) {
-        traceInfo('Conda not found, turning off conda features.', ex);
-        sendTelemetryEvent(EventNames.MANAGER_REGISTRATION_SKIPPED, undefined, {
-            managerName: 'conda',
-            reason: 'tool_not_found',
-        });
-        await notifyMissingManagerIfDefault('ms-python.python:conda', projectManager, api);
-        return;
+        // Only treat the known "Conda not found" error as a skip;
+        // other errors (e.g. PET timeout/crash) should propagate to safeRegister.
+        if (ex instanceof Error && ex.message === 'Conda not found') {
+            traceInfo('Conda not found, turning off conda features.', ex);
+            sendTelemetryEvent(EventNames.MANAGER_REGISTRATION_SKIPPED, undefined, {
+                managerName: 'conda',
+                reason: 'tool_not_found',
+            });
+            await notifyMissingManagerIfDefault('ms-python.python:conda', projectManager, api);
+            return;
+        }
+        throw ex;
     }
 
     // Conda was found — errors below are real registration failures (let safeRegister handle them)

src/managers/pipenv/main.ts

@@ -2,6 +2,7 @@ import { Disposable } from 'vscode';
 import { PythonEnvironmentApi } from '../../api';
 import { traceInfo } from '../../common/logging';
 import { EventNames } from '../../common/telemetry/constants';
+import { classifyError } from '../../common/telemetry/errorClassifier';
 import { sendTelemetryEvent } from '../../common/telemetry/sender';
 import { getPythonApi } from '../../features/pythonApi';
 import { PythonProjectManager } from '../../internal.api';
@@ -48,6 +49,10 @@ export async function registerPipenvFeatures(
             'Pipenv not found, turning off pipenv features. If you have pipenv installed in a non-standard location, set the "python.pipenvPath" setting.',
             ex,
         );
+        sendTelemetryEvent(EventNames.MANAGER_REGISTRATION_FAILED, undefined, {
+            managerName: 'pipenv',
+            errorType: classifyError(ex),
+        });
         await notifyMissingManagerIfDefault('ms-python.python:pipenv', projectManager, api);
     }
 }

src/managers/poetry/main.ts

@@ -2,6 +2,7 @@ import { Disposable, LogOutputChannel } from 'vscode';
 import { PythonEnvironmentApi } from '../../api';
 import { traceInfo } from '../../common/logging';
 import { EventNames } from '../../common/telemetry/constants';
+import { classifyError } from '../../common/telemetry/errorClassifier';
 import { sendTelemetryEvent } from '../../common/telemetry/sender';
 import { getPythonApi } from '../../features/pythonApi';
 import { PythonProjectManager } from '../../internal.api';
@@ -46,6 +47,10 @@ export async function registerPoetryFeatures(
         }
     } catch (ex) {
         traceInfo('Poetry not found, turning off poetry features.', ex);
+        sendTelemetryEvent(EventNames.MANAGER_REGISTRATION_FAILED, undefined, {
+            managerName: 'poetry',
+            errorType: classifyError(ex),
+        });
         await notifyMissingManagerIfDefault('ms-python.python:poetry', projectManager, api);
     }
 }