Skip to content

fix(workflows): scope reviewer auto-triggers by changed-file subsets - #1787

Merged
jamesadevine merged 2 commits into
mainfrom
copilot/reviewer-fan-out-trigger
Aug 3, 2026
Merged

fix(workflows): scope reviewer auto-triggers by changed-file subsets#1787
jamesadevine merged 2 commits into
mainfrom
copilot/reviewer-fan-out-trigger

Conversation

Copilot AI commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

/review fan-out was over-triggering on automated docs/tests PRs. This change narrows PR-event auto-triggers so docs-only and test-only changes invoke only the relevant reviewer subset, while keeping on-demand /review fan-out unchanged.

  • Security reviewer trigger scoping
    • Added pull_request.paths to review-security so it only auto-runs for security-relevant code/workflow surfaces (src/**, ado-aw-derive/**, scripts/ado-script/src/**, selected package manifests, workflow files, Cargo.toml).
  • Compiler-contract reviewer trigger scoping
    • Tightened review-compiler-contract pull_request.paths to compiler/runtime/workflow-source changes; removed docs/test-only paths from auto-trigger scope.
  • Docs alignment
    • Updated AGENTS.md reviewer table language to reflect scoped auto-trigger behavior rather than unconditional ready_for_review fan-out.
  • Generated workflow sync
    • Regenerated lock files for both modified reviewers to keep source and compiled workflow definitions aligned.
on:
  pull_request:
    types: [ready_for_review]
    paths:
      - "src/**"
      - "ado-aw-derive/**"
      - "scripts/ado-script/src/**"
      - ".github/workflows/**"

Test plan

  • gh aw compile --validate review-compiler-contract review-security

Copilot AI and others added 2 commits August 3, 2026 09:15
Co-authored-by: jamesadevine <4742697+jamesadevine@users.noreply.github.com>
Co-authored-by: jamesadevine <4742697+jamesadevine@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@jamesadevine
jamesadevine marked this pull request as ready for review August 3, 2026 09:34
@jamesadevine
jamesadevine merged commit 08be5ec into main Aug 3, 2026
2 checks passed
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@jamesadevine
jamesadevine deleted the copilot/reviewer-fan-out-trigger branch August 3, 2026 09:34
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