update from comments

Author: eleanorjboyd

package.json

@@ -132,7 +132,7 @@
                     "type": "boolean",
                     "description": "%python-envs.alwaysUseUv.description%",
                     "default": true,
-                    "scope": "resource"
+                    "scope": "machine"
                 }
             }
         },

src/managers/builtin/helpers.ts

@@ -16,9 +16,7 @@ export function resetUvInstallationCache(): void {
 }
 
 export async function isUvInstalled(log?: LogOutputChannel): Promise<boolean> {
-    console.log('into isUvInstalled function');
     if (available.completed) {
-        console.log('UV installation status already determined');
         return available.promise;
     }
     log?.info(`Running: uv --version`);
@@ -46,9 +44,6 @@ export async function shouldUseUv(log?: LogOutputChannel, envPath?: string): Pro
     if (envPath) {
         // always use uv if the given environment is stored as a uv env
         const uvEnvs = await getUvEnvironments();
-        console.log(`UV Environments: ${uvEnvs.join(', ')}`);
-        console.log(`Checking if envPath ${envPath} is in UV environments`);
-        console.log(`isUVinstalled function: ${isUvInstalled(log)}`);
         if (uvEnvs.includes(envPath)) {
             return await isUvInstalled(log);
         }
@@ -58,12 +53,7 @@ export async function shouldUseUv(log?: LogOutputChannel, envPath?: string): Pro
     const config = getConfiguration('python-envs');
     const alwaysUseUv = config.get<boolean>('alwaysUseUv', true);
 
-    console.log(`alwaysUseUv setting is ${alwaysUseUv}`);
-    console.log(config.inspect<boolean>('alwaysUseUv'));
-    console.log(config.inspect<boolean>('alwaysUseUv')?.globalValue);
-
     if (alwaysUseUv) {
-        console.log(`alwaysUseUv is true, checking if UV is installed`);
         return await isUvInstalled(log);
     }
     return false;

src/managers/builtin/pipManager.ts

@@ -1,5 +1,6 @@
 import {
     CancellationError,
+    Disposable,
     Event,
     EventEmitter,
     LogOutputChannel,
@@ -8,7 +9,6 @@ import {
     ThemeIcon,
     window,
 } from 'vscode';
-import { Disposable } from 'vscode-jsonrpc';
 import {
     DidChangePackagesEventArgs,
     IconPath,

src/managers/builtin/utils.ts

@@ -149,7 +149,7 @@ async function refreshPipPackagesRaw(environment: PythonEnvironment, log?: LogOu
     } catch (ex) {
         log?.error('Error running pip list', ex);
         log?.info(
-            'Installation attempted using pip, action can be done with uv if installed and setting `alwaysUseUv` is enabled.',
+            'Package list retrieval attempted using pip, action can be done with uv if installed and setting `alwaysUseUv` is enabled.',
         );
         throw ex;
     }

src/managers/builtin/venvUtils.ts

@@ -476,7 +476,6 @@ export async function resolveVenvPythonEnvironmentPath(
 
     if (resolved.kind === NativePythonEnvironmentKind.venv || resolved.kind === NativePythonEnvironmentKind.venvUv) {
         const envInfo = await getPythonInfo(resolved);
-        // can't decide on if I need anything here....
         return api.createPythonEnvironmentItem(envInfo, manager);
     }