fix: Recognize Copilot CLI#8249
Merged
Merged
Conversation
Fixes #8234. `azd` attempts to determine if it is being invoked by an AI agent (e.g. Claude, Gemini, Copilot, etc.) and alter its behavior to be more "AI friendly". This includes setting the --`no-prompt` parameter by default, and reducing the console output from spinners, progress bars, and the like. The most reliable way of doing this is by checking for certain environment variables set by the AI agent process. However, `azd` does not check for the variable currently used by Copilot CLI and VS Code: `COPILOT_CLI`. Here we add a check for this specific environment variable and update the tests accordingly.
tmeschter
requested review from
JeffreyCA,
hemarina,
jongio,
rajeshkamal5050,
tg-msft,
vhvb1989,
wbreza and
weikanglim
as code owners
May 19, 2026 20:15
📋 Prioritization NoteThanks for the contribution! The linked issue isn't in the current milestone yet. |
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR updates azd's AI-agent detection to recognize Copilot CLI/VS Code via the COPILOT_CLI environment variable, ensuring agent-friendly defaults (e.g., --no-prompt, reduced TTY UX) apply consistently.
Changes:
- Add
COPILOT_CLIto the known environment variable patterns for GitHub Copilot CLI detection. - Extend unit/integration tests to validate agent detection behavior when
COPILOT_CLIis present. - Ensure test helpers clear
COPILOT_CLIto avoid cross-test contamination.
Show a summary per file
| File | Description |
|---|---|
| cli/azd/internal/terminal/terminal_test.go | Adds TTY-disable test case for COPILOT_CLI and clears it in env cleanup helper. |
| cli/azd/internal/runcontext/agentdetect/detect_test.go | Adds detection test case for COPILOT_CLI and clears it in env cleanup helper. |
| cli/azd/internal/runcontext/agentdetect/detect_env.go | Adds COPILOT_CLI to Copilot env var detection patterns. |
| cli/azd/cmd/auto_install_test.go | Adds global flag parsing test to ensure --no-prompt auto-enables via COPILOT_CLI. |
| cli/azd/cmd/auto_install_integration_test.go | Adds integration test for COPILOT_CLI and clears it in env cleanup helper. |
Copilot's findings
- Files reviewed: 5/5 changed files
- Comments generated: 1
wbreza
approved these changes
May 19, 2026
wbreza
left a comment
wbreza
left a comment
Contributor
There was a problem hiding this comment.
Clean, surgical fix. Adds COPILOT_CLI to the agent-detection patterns with thorough parallel test coverage across detection, auto-install (unit + integration), and terminal TTY tests. Notably, all four env-var cleanup lists (clearAgentEnvVars, clearTestEnvVars, clearAgentEnvVarsForTest) are updated consistently — a detail that's commonly missed and would cause cross-test bleed. PR description aligns with the implementation and references the originating issue.
LGTM.
tmeschter
enabled auto-merge (squash)
May 19, 2026 21:03
Collaborator
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash: pwsh: WindowsPowerShell install MSI install Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
Copilot AI
pushed a commit
that referenced
this pull request
May 21, 2026
Fixes #8234. `azd` attempts to determine if it is being invoked by an AI agent (e.g. Claude, Gemini, Copilot, etc.) and alter its behavior to be more "AI friendly". This includes setting the --`no-prompt` parameter by default, and reducing the console output from spinners, progress bars, and the like. The most reliable way of doing this is by checking for certain environment variables set by the AI agent process. However, `azd` does not check for the variable currently used by Copilot CLI and VS Code: `COPILOT_CLI`. Here we add a check for this specific environment variable and update the tests accordingly. Co-authored-by: therealjohn <1501196+therealjohn@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #8234.
azdattempts to determine if it is being invoked by an AI agent (e.g. Claude, Gemini, Copilot, etc.) and alter its behavior to be more "AI friendly". This includes setting the --no-promptparameter by default, and reducing the console output from spinners, progress bars, and the like.The most reliable way of doing this is by checking for certain environment variables set by the AI agent process. However,
azddoes not check for the variable currently used by Copilot CLI and VS Code:COPILOT_CLI.Here we add a check for this specific environment variable and update the tests accordingly.