Skip to content

Scope PR review workflows with path-based trigger filters to reduce redundant API load#46900

Merged
pelikhan merged 5 commits into
mainfrom
copilot/deep-report-scope-pr-review-workflows
Jul 21, 2026
Merged

Scope PR review workflows with path-based trigger filters to reduce redundant API load#46900
pelikhan merged 5 commits into
mainfrom
copilot/deep-report-scope-pr-review-workflows

Conversation

Copilot AI commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

PR-triggered reviewer workflows were over-firing on non-implementation changes and driving disproportionate REST API usage. This narrows trigger scope so reviewers run when code/design-relevant files change, while preserving explicit/manual invocation paths.

  • Trigger scoping for PR reviewers

    • Added pull_request.paths-ignore to:
      • mattpocock-skills-reviewer.md
      • impeccable-skills-reviewer.md
      • pr-code-quality-reviewer.md
    • Ignored doc/metadata-only surfaces (*.md, docs/**, .changeset/**, socials/**, scratchpad/**) to avoid no-op reviews.
  • Design Decision Gate scoping

    • Added pull_request.paths to design-decision-gate.md so enforcement runs on implementation-relevant areas:
      • actions/**, cmd/**, internal/**, pkg/**, docs/adr/**, .design-gate.yml
  • Compiled workflow parity

    • Recompiled corresponding .lock.yml files so runtime behavior matches source workflow definitions.
on:
  pull_request:
    types: [ready_for_review]
    paths-ignore:
      - '*.md'
      - 'docs/**'
      - '.changeset/**'
      - 'socials/**'
      - 'scratchpad/**'

Run: https://github.com/github/gh-aw/actions/runs/29781538011

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 4.79 AIC · ⌖ 5.1 AIC · ⊞ 6.9K ·
Comment /souschef to run again


Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 9.44 AIC · ⌖ 7.25 AIC · ⊞ 6.9K ·
Comment /souschef to run again


branch update requested from https://github.com/github/gh-aw/actions/runs/29788028039

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 9.37 AIC · ⌖ 7.68 AIC · ⊞ 6.9K ·
Comment /souschef to run again


Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 9.99 AIC · ⌖ 10.1 AIC · ⊞ 6.9K ·
Comment /souschef to run again

Copilot AI and others added 2 commits July 20, 2026 18:42
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add path filters to PR-triggered review workflows Scope PR review workflows with path-based trigger filters to reduce redundant API load Jul 20, 2026
Copilot AI requested a review from pelikhan July 20, 2026 18:53
@github-actions

Copy link
Copy Markdown
Contributor

🤖 PR Triage

Field Value
Category chore
Risk 🟢 low
Priority Score 22/100
Recommended Action defer

Score breakdown: impact 8 + urgency 6 + quality 8

WIP PR adding path filters to PR-triggered review workflows. Draft, no CI. Low impact optimization — defer until WIP is complete.

Generated by 🔧 PR Triage Agent · 34.3 AIC · ⌖ 7.71 AIC · ⊞ 5.6K ·

@pelikhan
pelikhan marked this pull request as ready for review July 20, 2026 20:03
Copilot AI review requested due to automatic review settings July 20, 2026 20:03

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

Narrows automated PR-review triggers to reduce unnecessary API usage while preserving manual invocation.

Changes:

  • Ignores documentation and metadata-only changes for three reviewers.
  • Restricts the ADR gate to selected implementation paths.
  • Recompiles generated workflows.
Show a summary per file
File Description
.github/workflows/pr-code-quality-reviewer.md Adds ignored paths.
.github/workflows/pr-code-quality-reviewer.lock.yml Updates compiled trigger.
.github/workflows/mattpocock-skills-reviewer.md Adds ignored paths.
.github/workflows/mattpocock-skills-reviewer.lock.yml Updates compiled trigger.
.github/workflows/impeccable-skills-reviewer.md Adds ignored paths.
.github/workflows/impeccable-skills-reviewer.lock.yml Updates compiled trigger.
.github/workflows/design-decision-gate.md Adds implementation-path filtering.
.github/workflows/design-decision-gate.lock.yml Updates compiled trigger.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 8/8 changed files
  • Comments generated: 1
  • Review effort level: Medium

Comment thread .github/workflows/design-decision-gate.md
@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address any unresolved review feedback, and rerun checks once the branch is up to date.

Run: https://github.com/github/gh-aw/actions/runs/29777731070

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 7.66 AIC · ⌖ 7.84 AIC · ⊞ 6.9K ·
Comment /souschef to run again

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Copilot AI commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please run the pr-finisher skill, address any unresolved review feedback, and rerun checks once the branch is up to date.

Run: https://github.com/github/gh-aw/actions/runs/29777731070...

Expanded the design-decision-gate.md paths filter in 571bac0 to include all implementation surfaces that were missing: eslint-factory/**, scripts/**, tools.go, and .github/workflows/**. The lock file has been recompiled and make fmt passes.

Copilot AI requested a review from gh-aw-bot July 20, 2026 21:08
@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 4.79 AIC · ⌖ 5.1 AIC · ⊞ 6.9K ·
Comment /souschef to run again

Copilot AI commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.

...

Review thread addressed (replied to the copilot-pull-request-reviewer thread on design-decision-gate.md). make fmt passes locally. Branch is already up to date with main (merge commit 40a3826). CI needs a human re-trigger after the agent's pushes.

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.

Unresolved review threads to close out:

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 9.44 AIC · ⌖ 7.25 AIC · ⊞ 6.9K ·
Comment /souschef to run again

Copilot AI commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.

Unresolved review threads to close out:...

The review thread at #discussion_r3617231271 has been replied to with the code fix (571bac0), but the agent cannot call the resolveReviewThread GraphQL mutation from this environment. A maintainer needs to click "Resolve conversation" on that thread in the GitHub UI to close it out. Everything else is ready: make fmt passes, branch is up to date with main (40a3826).

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.

Unresolved review threads to close out:

Run: https://github.com/github/gh-aw/actions/runs/29788028039

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 9.37 AIC · ⌖ 7.68 AIC · ⊞ 6.9K ·
Comment /souschef to run again

Copilot AI commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.

Unresolved review threads to close out:...

The unresolved thread at #discussion_r3617231271 is now resolved — the GitHub API confirms is_resolved: true. The code fix was already shipped in 571bac0 (expanded design-decision-gate.md paths to cover all implementation surfaces). Local make fmt and make lint pass with 0 errors. Branch is up to date with main (merge commit 40a3826). CI must be re-triggered by a maintainer (close/reopen or workflow_dispatch) before merge.

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address any remaining review feedback, and rerun checks once the branch is up to date.

Run: https://github.com/github/gh-aw/actions/runs/29789600762

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 13.2 AIC · ⌖ 8.86 AIC · ⊞ 6.9K ·
Comment /souschef to run again

Copilot AI commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please run the pr-finisher skill, address any remaining review feedback, and rerun checks once the branch is up to date.

Run: https://github.com/github/gh-aw/actions/runs/29789600762...

Nothing remaining to action. All review threads are resolved, prior CI checks completed with success, and the branch is up to date with main (merge commit 40a3826). The PR is ready for a maintainer to re-trigger CI (close/reopen or workflow_dispatch) and merge.

@github-actions

Copy link
Copy Markdown
Contributor

🔍 PR Triage — Run §29791496271

Field Value
Category chore
Risk 🟢 low
Score 30/100 (impact 12 + urgency 8 + quality 10)
Action defer

Rationale: Path-based trigger scoping for PR reviewer workflows. Non-urgent optimization. Low risk, review in a batch with similar chore PRs.

Generated by 🔧 PR Triage Agent · 40.1 AIC · ⌖ 6.65 AIC · ⊞ 5.6K ·

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address any unresolved review feedback, confirm the branch is current, and summarize the next concrete maintainer action once checks are ready.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • pi.dev

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "pi.dev"

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · 9.99 AIC · ⌖ 10.1 AIC · ⊞ 6.9K ·
Comment /souschef to run again

Copilot AI commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please run the pr-finisher skill, address any unresolved review feedback, confirm the branch is current, and summarize the next concrete maintainer action once checks are ready.

...

PR is ready to merge. Here's the state:

  • Reviews — Only review thread (#discussion_r3617231271) is resolved; fix shipped in 571bac0.
  • Branch — Up to date with main (merge commit 40a3826), clean working tree.
  • Local checksmake fmt and make lint pass with 0 errors.
  • CI — Stale after agent pushes (agent pushes don't re-trigger workflows). Prior runs all completed success.

Next maintainer action: Re-trigger CI by closing and reopening the PR (or via workflow_dispatch), then merge once checks are green.

@pelikhan
pelikhan merged commit b828a19 into main Jul 21, 2026
5 checks passed
@pelikhan
pelikhan deleted the copilot/deep-report-scope-pr-review-workflows branch July 21, 2026 03:30
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.82.15

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[deep-report] Scope PR-review workflows with path filters to cut redundant API consumption

4 participants