Skip to content

Fix PAT pool validation with Auto routing - #5683

Merged
mthalman merged 1 commit into
dotnet:mainfrom
jeffhandley:jeffhand-microsoft/source-build-pat-validation
Nov 9, 2026
Merged

mthalman merged 1 commit into
dotnet:mainfrom
jeffhandley:jeffhand-microsoft/source-build-pat-validation

Conversation

@jeffhandley

Copy link
Copy Markdown
Member

Summary

Align validate-pat-pool.yml with the validator in dotnet/core (814392b4):

  • Use the reviewed gh-aw setup v0.93.2 default Copilot CLI instead of independently pinning CLI 1.0.21.
  • Probe with Auto routing, disable CLI auto-update, and allow 90 seconds instead of 30 seconds; remove the obsolete --effort=low argument.
  • Report CLI setup/install failures explicitly rather than presenting untested PATs as pool failures.

Preserves Empty/Valid/Invalid/Unknown classification, the PAT secret wiring, schedule, environment, and permissions. Only the validator workflow changes; no agentic workflow/compiler or documentation updates are included.

Validation

  • Validator file exactly matches the dotnet/core reference blob.
  • YAML parsing and Bash syntax checks passed.
  • Mocked checks passed for all four PAT statuses and six summary outcomes, including failed/skipped CLI installation.
  • git diff --check passed.

Live PATs were not tested, and no workflow was dispatched.

Match core's validator with the reviewed CLI default, a longer Auto probe, and explicit setup failure reporting.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@jeffhandley
jeffhandley requested a review from a team as a code owner November 9, 2026 09:09
@mthalman
mthalman merged commit 0014009 into dotnet:main Nov 9, 2026
2 checks passed
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