✨ remove extra env tool confirmation prompt

Author: Yashraj-Jangra

src/client/chat/selectEnvTool.ts

@@ -23,7 +23,6 @@ import { TerminalCodeExecutionProvider } from '../terminals/codeExecution/termin
 import {
     doesWorkspaceHaveVenvOrCondaEnv,
     getEnvDetailsForResponse,
-    getToolResponseIfNotebook,
     IResourceReference,
     raceCancellationError,
 } from './utils';
@@ -120,39 +119,11 @@ export class SelectPythonEnvTool extends BaseTool<ISelectPythonEnvToolArguments>
     }
 
     async prepareInvocationImpl(
-        options: LanguageModelToolInvocationPrepareOptions<ISelectPythonEnvToolArguments>,
-        resource: Uri | undefined,
+        _options: LanguageModelToolInvocationPrepareOptions<ISelectPythonEnvToolArguments>,
+        _resource: Uri | undefined,
         _token: CancellationToken,
     ): Promise<PreparedToolInvocation> {
-        if (getToolResponseIfNotebook(resource)) {
-            return {};
-        }
-        const hasVenvOrCondaEnvInWorkspaceFolder = doesWorkspaceHaveVenvOrCondaEnv(resource, this.api);
-
-        if (
-            hasVenvOrCondaEnvInWorkspaceFolder ||
-            !workspace.workspaceFolders?.length ||
-            options.input.reason === 'cancelled'
-        ) {
-            return {
-                confirmationMessages: {
-                    title: l10n.t('Select a Python Environment?'),
-                    message: '',
-                },
-            };
-        }
-
-        return {
-            confirmationMessages: {
-                title: l10n.t('Configure a Python Environment?'),
-                message: l10n.t(
-                    [
-                        'The recommended option is to create a new Python Environment, providing the benefit of isolating packages from other environments.  ',
-                        'Optionally you could select an existing Python Environment.',
-                    ].join('\n'),
-                ),
-            },
-        };
+        return {};
     }
 }
 

src/test/chat/selectEnvTool.unit.test.ts

@@ -0,0 +1,26 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
+'use strict';
+
+import { expect } from 'chai';
+import { CancellationTokenSource } from 'vscode';
+import { SelectPythonEnvTool } from '../../client/chat/selectEnvTool';
+
+suite('Select Python Environment Tool', () => {
+    test('Does not request confirmation before showing the environment picker', async () => {
+        const tool = Object.create(SelectPythonEnvTool.prototype) as SelectPythonEnvTool;
+        const tokenSource = new CancellationTokenSource();
+
+        try {
+            const result = await tool.prepareInvocation(
+                { input: { resourcePath: '/workspace' } },
+                tokenSource.token,
+            );
+
+            expect(result.confirmationMessages).to.be.undefined;
+        } finally {
+            tokenSource.dispose();
+        }
+    });
+});

todo.md

@@ -0,0 +1,14 @@
+# Project Tracker & Continuity Log
+
+## Progress
+- [x] Forked `microsoft/vscode-python` to `Yashraj-Jangra/vscode-python` and cloned locally.
+- [x] Created feature branch `chat-avoid-redundant-env-tool-confirmation`.
+- [x] Updated `src/client/chat/selectEnvTool.ts` to return `{}` from `prepareInvocationImpl`, removing the redundant pre-invocation confirmation modal.
+- [x] Added unit test in `src/test/chat/selectEnvTool.unit.test.ts` to verify confirmation is not requested.
+- [x] Verified compilation (`npx tsc -p ./`), unit test execution (`mocha`), and linting (`npm run lint`).
+- [ ] Commit changes with clean human commit formatting.
+- [ ] Push feature branch to `Yashraj-Jangra/vscode-python` and submit PR to `microsoft/vscode-python`.
+
+## Next Steps
+- Commit and push to GitHub fork.
+- Open PR on `microsoft/vscode-python`.