Skip to content

chat: avoid redundant environment tool confirmation - #26072

Open
eleanorjboyd wants to merge 1 commit into
microsoft:mainfrom
eleanorjboyd:agents/allow-all-tool-cal-issue
Open

chat: avoid redundant environment tool confirmation#26072
eleanorjboyd wants to merge 1 commit into
microsoft:mainfrom
eleanorjboyd:agents/allow-all-tool-cal-issue

Conversation

@eleanorjboyd

Copy link
Copy Markdown
Member

Summary

  • remove the extra confirmation shown before the Python environment picker
  • preserve the picker's explicit user selection before creating or selecting an environment
  • add regression coverage ensuring the tool does not request confirmation during preparation

Why

The contributed configurePythonEnvironment tool always returned confirmationMessages, which caused Copilot CLI sessions to stop for approval even when Allow all was enabled. The environment picker already requires the user to explicitly choose an environment action, so the preceding confirmation was redundant.

The environment picker already requires an explicit user choice, so an additional tool confirmation blocks Allow All sessions unnecessarily.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants