edits

Author: eleanorjboyd

src/common/pickers/environments.ts

@@ -157,12 +157,12 @@ export async function pickEnvironment(
     ];
 
     if (options?.recommended) {
-        // Include interpreter path in description for recommended environment too
         const pathDescription = options.recommended.displayPath;
-        const description = options.recommended.description && options.recommended.description.trim()
-            ? `${options.recommended.description} (${pathDescription})`
-            : pathDescription;
-            
+        const description =
+            options.recommended.description && options.recommended.description.trim()
+                ? `${options.recommended.description} (${pathDescription})`
+                : pathDescription;
+
         items.push(
             {
                 label: Common.recommended,
@@ -185,12 +185,10 @@ export async function pickEnvironment(
         const envs = await manager.getEnvironments('all');
         items.push(
             ...envs.map((e) => {
-                // Include interpreter path in description. If original description exists and is not empty, append path to it.
                 const pathDescription = e.displayPath;
-                const description = e.description && e.description.trim()
-                    ? `${e.description} (${pathDescription})`
-                    : pathDescription;
-                
+                const description =
+                    e.description && e.description.trim() ? `${e.description} (${pathDescription})` : pathDescription;
+
                 return {
                     label: e.displayName ?? e.name,
                     description: description,
@@ -207,12 +205,10 @@ export async function pickEnvironment(
 
 export async function pickEnvironmentFrom(environments: PythonEnvironment[]): Promise<PythonEnvironment | undefined> {
     const items = environments.map((e) => {
-        // Include interpreter path in description. If original description exists and is not empty, append path to it.
         const pathDescription = e.displayPath;
-        const description = e.description && e.description.trim()
-            ? `${e.description} (${pathDescription})`
-            : pathDescription;
-        
+        const description =
+            e.description && e.description.trim() ? `${e.description} (${pathDescription})` : pathDescription;
+
         return {
             label: e.displayName ?? e.name,
             description: description,

src/test/common/environmentPicker.unit.test.ts

@@ -28,65 +28,60 @@ suite('Environment Picker Description Logic', () => {
     suite('Description formatting with interpreter path', () => {
         test('should use displayPath as description when no original description exists', () => {
             const env = createMockEnvironment('/usr/local/bin/python');
-            
+
             // This is the logic from our updated picker
             const pathDescription = env.displayPath;
-            const description = env.description && env.description.trim()
-                ? `${env.description} (${pathDescription})`
-                : pathDescription;
-            
+            const description =
+                env.description && env.description.trim() ? `${env.description} (${pathDescription})` : pathDescription;
+
             assert.strictEqual(description, '/usr/local/bin/python');
         });
 
         test('should append displayPath to existing description in parentheses', () => {
             const env = createMockEnvironment('/home/user/.venv/bin/python', 'Virtual Environment');
-            
+
             // This is the logic from our updated picker
             const pathDescription = env.displayPath;
-            const description = env.description && env.description.trim()
-                ? `${env.description} (${pathDescription})`
-                : pathDescription;
-            
+            const description =
+                env.description && env.description.trim() ? `${env.description} (${pathDescription})` : pathDescription;
+
             assert.strictEqual(description, 'Virtual Environment (/home/user/.venv/bin/python)');
         });
 
         test('should handle complex paths correctly', () => {
             const complexPath = '/usr/local/anaconda3/envs/my-project-env/bin/python';
             const env = createMockEnvironment(complexPath, 'Conda Environment');
-            
+
             // This is the logic from our updated picker
             const pathDescription = env.displayPath;
-            const description = env.description && env.description.trim()
-                ? `${env.description} (${pathDescription})`
-                : pathDescription;
-            
+            const description =
+                env.description && env.description.trim() ? `${env.description} (${pathDescription})` : pathDescription;
+
             assert.strictEqual(description, `Conda Environment (${complexPath})`);
         });
 
         test('should handle empty description correctly', () => {
             const env = createMockEnvironment('/opt/python/bin/python', '');
-            
-            // This is the logic from our updated picker  
+
+            // This is the logic from our updated picker
             const pathDescription = env.displayPath;
-            const description = env.description && env.description.trim()
-                ? `${env.description} (${pathDescription})`
-                : pathDescription;
-            
+            const description =
+                env.description && env.description.trim() ? `${env.description} (${pathDescription})` : pathDescription;
+
             // Empty string should be treated like no description, so just use path
             assert.strictEqual(description, '/opt/python/bin/python');
         });
 
         test('should handle Windows paths correctly', () => {
             const windowsPath = 'C:\\Python39\\python.exe';
             const env = createMockEnvironment(windowsPath, 'System Python');
-            
+
             // This is the logic from our updated picker
             const pathDescription = env.displayPath;
-            const description = env.description && env.description.trim()
-                ? `${env.description} (${pathDescription})`
-                : pathDescription;
-            
+            const description =
+                env.description && env.description.trim() ? `${env.description} (${pathDescription})` : pathDescription;
+
             assert.strictEqual(description, 'System Python (C:\\Python39\\python.exe)');
         });
     });
-});
+});