Hide "Add Python Project" context menu for already selected projects & Rename button (#432)

edited by @eleanorjboyd for clarity

## Problem
Currently, when right-clicking on a folder or file that is already added
as a Python project, the "Add Python Project" context menu item is still
shown. This can be confusing to users as they might think they need to
add the project again, or that the project wasn't properly added the
first time.

## To Test
- When user right-clicks on a file/folder not yet added as a Python
project: "Add as Python Project" is shown
- When user right-clicks on a file/folder already added as a Python
project: The menu item is hidden

This change improves the user experience by reducing confusion and
providing clearer contextual actions.

Fixes #272 and #327

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: eleanorjboyd <26030610+eleanorjboyd@users.noreply.github.com>

Author: Copilot

package.json

@@ -457,12 +457,12 @@
                 {
                     "command": "python-envs.addPythonProject",
                     "group": "inline",
-                    "when": "explorerViewletVisible && explorerResourceIsFolder"
+                    "when": "explorerViewletVisible && explorerResourceIsFolder && !python-envs:isExistingProject"
                 },
                 {
                     "command": "python-envs.addPythonProject",
                     "group": "inline",
-                    "when": "explorerViewletVisible && resourceExtname == .py"
+                    "when": "explorerViewletVisible && resourceExtname == .py && !python-envs:isExistingProject"
                 }
             ],
             "editor/title/run": [

package.nls.json

@@ -13,7 +13,7 @@
     "python-envs.terminal.revertStartupScriptChanges.title": "Revert Shell Startup Script Changes",
     "python-envs.setEnvManager.title": "Set Environment Manager",
     "python-envs.setPkgManager.title": "Set Package Manager",
-    "python-envs.addPythonProject.title": "Add Python Project",
+    "python-envs.addPythonProject.title": "Add as Python Project",
     "python-envs.removePythonProject.title": "Remove Python Project",
     "python-envs.copyEnvPath.title": "Copy Environment Path",
     "python-envs.copyProjectPath.title": "Copy Project Path",
@@ -37,4 +37,4 @@
     "python-envs.uninstallPackage.title": "Uninstall Package",
     "python.languageModelTools.python_environment.userDescription": "Get Python environment info for a file or path, including version, packages, and the command to run it.",
     "python.languageModelTools.python_install_package.userDescription": "Installs Python packages in the given workspace."
-}
+}

src/extension.ts

@@ -1,4 +1,4 @@
-import { commands, ExtensionContext, LogOutputChannel, Terminal, Uri } from 'vscode';
+import { commands, ExtensionContext, LogOutputChannel, Terminal, Uri, window } from 'vscode';
 import { PythonEnvironment, PythonEnvironmentApi } from './api';
 import { ensureCorrectVersion } from './common/extVersion';
 import { registerTools } from './common/lm.apis';
@@ -13,7 +13,6 @@ import {
     activeTerminal,
     createLogOutputChannel,
     onDidChangeActiveTerminal,
-    onDidChangeActiveTextEditor,
     onDidChangeTerminalShellIntegration,
 } from './common/window.apis';
 import { createManagerReady } from './features/common/managerReady';
@@ -88,6 +87,14 @@ export async function activate(context: ExtensionContext): Promise<PythonEnviron
     const projectManager: PythonProjectManager = new PythonProjectManagerImpl();
     context.subscriptions.push(projectManager);
 
+    // Helper function to check if a resource is an existing Python project
+    const isExistingProject = (uri: Uri | undefined): boolean => {
+        if (!uri) {
+            return false;
+        }
+        return projectManager.get(uri) !== undefined;
+    };
+
     const envVarManager: EnvVarManager = new PythonEnvVariableManager(projectManager);
     context.subscriptions.push(envVarManager);
 
@@ -194,6 +201,10 @@ export async function activate(context: ExtensionContext): Promise<PythonEnviron
             await setPackageManagerCommand(envManagers, projectManager);
         }),
         commands.registerCommand('python-envs.addPythonProject', async (resource) => {
+            // Set context to show/hide menu item depending on whether the resource is already a Python project
+            if (resource instanceof Uri) {
+                commands.executeCommand('setContext', 'python-envs:isExistingProject', isExistingProject(resource));
+            }
             await addPythonProjectCommand(resource, projectManager, envManagers, projectCreators);
         }),
         commands.registerCommand('python-envs.removePythonProject', async (item) => {
@@ -254,7 +265,7 @@ export async function activate(context: ExtensionContext): Promise<PythonEnviron
                 }
             }
         }),
-        onDidChangeActiveTextEditor(async () => {
+        window.onDidChangeActiveTextEditor(async () => {
             updateViewsAndStatus(statusBar, workspaceView, managerView, api);
         }),
         envManagers.onDidChangeEnvironment(async () => {