Skip to content

Allow all Azure CLI commands in Android PR Reviewer - #12462

Open
jonathanpeppers with Copilot wants to merge 3 commits into
mainfrom
copilot/aw-fix-missing-tools-in-reviewer
Open

Allow all Azure CLI commands in Android PR Reviewer#12462
jonathanpeppers with Copilot wants to merge 3 commits into
mainfrom
copilot/aw-fix-missing-tools-in-reviewer

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

The Android PR Reviewer needs unrestricted Azure CLI access to investigate Azure DevOps build failures. The existing command policy allowed only selected az devops, az pipelines, and az rest commands, which blocked other Azure CLI commands required during investigation.

  • Replace the restricted Azure CLI subcommand allowlist with az *.
  • Regenerate android-reviewer.lock.yml so the compiled workflow permits shell(az).

Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix missing Azure CLI access in Android PR Reviewer Allow Android PR Reviewer Azure CLI version check Aug 20, 2026
Copilot AI requested a review from jonathanpeppers August 20, 2026 16:22
Comment thread .github/workflows/android-reviewer.md Outdated
Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
@jonathanpeppers jonathanpeppers changed the title Allow Android PR Reviewer Azure CLI version check Allow all Azure CLI commands in Android PR Reviewer Aug 20, 2026
@jonathanpeppers
jonathanpeppers marked this pull request as ready for review August 20, 2026 17:55
Copilot AI lite review requested due to automatic review settings August 20, 2026 17:55
Copilot AI requested a review from jonathanpeppers August 20, 2026 17:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Android PR Reviewer agentic workflow configuration to remove Azure CLI subcommand restrictions, enabling the reviewer to run arbitrary az commands while investigating Azure DevOps build failures.

Changes:

  • Replace the previous Azure CLI allowlist (az devops:*, az pipelines:*, az rest:*) with a single wildcard entry (az *) in the workflow frontmatter.
  • Regenerate android-reviewer.lock.yml so the compiled workflow permits --allow-tool shell(az).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/android-reviewer.md Broadens the workflow’s allowed az CLI usage from specific subcommands to az *.
.github/workflows/android-reviewer.lock.yml Regenerated lockfile reflecting the new shell(az) tool permission in the compiled execution step.

Copilot AI linked an issue Aug 20, 2026 that may be closed by this pull request
@jonathanpeppers jonathanpeppers added the ready-to-review This PR is ready to review/merge, I think any CI failures are just flaky (ignorable). label Aug 20, 2026
@jonathanpeppers

Copy link
Copy Markdown
Member

@dalexsoto review

@simonrozsival
simonrozsival disabled auto-merge August 21, 2026 08:22
@simonrozsival
simonrozsival enabled auto-merge (squash) August 21, 2026 08:22
@jonathanpeppers

Copy link
Copy Markdown
Member

@dalexsoto review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-review This PR is ready to review/merge, I think any CI failures are just flaky (ignorable).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aw] Android PR Reviewer is missing required tool

4 participants