Clarify pr-sous-chef criteria for dismissing github-actions[bot] reviews by Copilot · Pull Request #43366 · github/gh-aw

Copilot · 2026-07-04T10:39:39Z

pr-sous-chef should only dismiss github-actions[bot] CHANGES_REQUESTED reviews after the associated feedback is actually resolved. This update tightens the dismissal contract so unresolved bot reviews are preserved instead of being dismissed opportunistically.

Behavioral contract update
- Reworded the dismissal rule to require that all related feedback for a review is addressed before dismissal.
- Made the condition explicit that this applies per review, not broadly per PR.
Selection semantics
- Added guidance to build a dismissal set from only fully-resolved bot reviews.
- Added an explicit guardrail to leave unresolved github-actions[bot] reviews untouched.
Compiled workflow parity
- Regenerated the corresponding lockfile to reflect the source workflow instruction change.

safeoutputs dismiss_github_actions_reviews \
  --pull_request_number 12345 \
  --review_ids "4605056464,4605057733"

Generated by 👨‍🍳 PR Sous Chef · 8.26 AIC · ⌖ 3.14 AIC · ⊞ 3.4K · ◷
_{Comment /souschef to run again}

Generated by 👨‍🍳 PR Sous Chef · 15.7 AIC · ⌖ 5.01 AIC · ⊞ 3.4K · ◷
_{Comment /souschef to run again}

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

pelikhan · 2026-07-04T10:52:18Z

@copilot use safe out to dismiss review, instead of MCP script

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot · 2026-07-04T11:10:15Z

@copilot use safe out to dismiss review, instead of MCP script

Updated in 3adffe7. pr-sous-chef now uses the native safe-output path (dismiss-pull-request-review / safeoutputs dismiss_pull_request_review) and no longer uses the custom MCP script for review dismissal.

Copilot

Pull request overview

Updates the pr-sous-chef workflow contract/documentation around dismissing github-actions[bot] blocking reviews, and regenerates the compiled workflow lockfile to reflect the safe-outputs/tooling changes.

Changes:

Replaces the custom dismiss-github-actions-reviews safe-output script with the native dismiss-pull-request-review safe-output configuration and CLI usage.
Tightens the workflow guidance to only dismiss bot CHANGES_REQUESTED reviews when all related feedback for that review is resolved.
Regenerates .github/workflows/pr-sous-chef.lock.yml to match the updated workflow source.

Show a summary per file

File	Description
pkg/cli/pr_sous_chef_workflow_contract_test.go	Updates workflow contract assertions for the new dismissal safe-output/tooling text.
.github/workflows/pr-sous-chef.md	Rewords the dismissal contract and switches guidance to the native dismiss-review safe-output tool.
.github/workflows/pr-sous-chef.lock.yml	Regenerated compiled workflow reflecting the updated safe-output toolset and config.

Review details

Tip

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

Files reviewed: 3/3 changed files
Comments generated: 2
Review effort level: Low

+   - For each review ID in that dismissal list, call the native safe-output tool:
    ```bash
-    safeoutputs missing_tool --tool "dismiss_pull_request_review" --reason "Need a safe-output to dismiss stale github-actions[bot] CHANGES_REQUESTED reviews after all related comments are addressed."
+    safeoutputs dismiss_pull_request_review --pull_request_number 12345 --review_id 4605056464 --justification "Dismissing stale github-actions review because all related review comments appear to be addressed."
    ```
-    Then continue processing the PR normally (do not fail the run solely because dismissal was unavailable).
+   - If dismissing one review fails, record the failure and continue with the remaining review IDs; do not fail the entire run solely because one dismissal attempt failed.


+	assert.Contains(t, text, "dismiss-pull-request-review", "Workflow should configure the native safe-output path for dismissing stale github-actions reviews")
+	assert.Contains(t, text, "safeoutputs dismiss_pull_request_review", "Workflow should call the native dismiss_pull_request_review safe-output tool")


gh-aw-bot · 2026-07-04T12:22:10Z

@copilot please run the pr-finisher skill, address the outstanding review feedback, refresh the branch if needed, and rerun checks once updated.

Unresolved review feedback to address:

Copilot: Clarify pr-sous-chef criteria for dismissing github-actions[bot] reviews #43366 (comment)
Copilot: Clarify pr-sous-chef criteria for dismissing github-actions[bot] reviews #43366 (comment)

Generated by 👨🍳 PR Sous Chef

Generated by 👨‍🍳 PR Sous Chef · 8.26 AIC · ⌖ 3.14 AIC · ⊞ 3.4K · ◷
_{Comment /souschef to run again}

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

Copilot · 2026-07-04T12:31:20Z

@copilot please run the pr-finisher skill, address the outstanding review feedback, refresh the branch if needed, and rerun checks once updated.

Unresolved review feedback to address:...

Addressed both review threads in the latest commit. Added an explicit Actor guard to step 3 of the workflow that documents dismiss_pull_request_review is actor-bound and requires human-triggered runs (workflow_dispatch / /souschef) to skip dismissal entirely, leaving github-actions[bot] reviews untouched. Updated the contract test to assert both the actor-bound constraint and the human-triggered skip behavior are present in the workflow source.

github-actions · 2026-07-04T12:44:07Z

🤖 PR Triage — Run §28706384347

Field	Value
Category	`refactor`
Risk	🟡 Medium
Score	72 / 100
Action	✅ `fast_track`

Score breakdown: impact 38 + urgency 20 + quality 14

Tightens pr-sous-chef dismissal guard so github-actions[bot] CHANGES_REQUESTED reviews are only dismissed when associated feedback is resolved (+59/-164, 3 files). Approved by @pelikhan. Verify rebase state and merge when ready.

Generated by 🔧 PR Triage Agent · 56.5 AIC · ⌖ 8.62 AIC · ⊞ 5.5K · ◷

gh-aw-bot · 2026-07-04T13:26:40Z

@copilot please run the pr-finisher skill, confirm the remaining review feedback is resolved, refresh the branch if needed, and rerun checks once updated.

Unresolved review feedback to address:

Copilot: Clarify pr-sous-chef criteria for dismissing github-actions[bot] reviews #43366 (comment)
Copilot: Clarify pr-sous-chef criteria for dismissing github-actions[bot] reviews #43366 (comment)

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

Generated by 👨‍🍳 PR Sous Chef · 6.03 AIC · ⌖ 5.73 AIC · ⊞ 3.4K · ◷
_{Comment /souschef to run again}

Copilot · 2026-07-04T13:32:26Z

@copilot please run the pr-finisher skill, confirm the remaining review feedback is resolved, refresh the branch if needed, and rerun checks once updated.

Unresolved review feedback to address:...

Both review threads were addressed in commit 138d329 — the actor-bound guardrail is documented in the workflow and the contract test asserts it. No additional changes are needed.

gh-aw-bot · 2026-07-04T14:07:37Z

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

Generated by 👨‍🍳 PR Sous Chef · 15.7 AIC · ⌖ 5.01 AIC · ⊞ 3.4K · ◷
_{Comment /souschef to run again}

…s-chef-dismiss-review-bot Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

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

Copilot · 2026-07-04T14:19:04Z

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

...

Merged latest main (MCP Gateway v0.3.33 and other updates) and recompiled the lockfile. All local validation passes. The review threads were already addressed in the previous commit — CI re-trigger needed for final green checks.

gh-aw-bot · 2026-07-04T15:23:10Z

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

Generated by 👨🍳 PR Sous Chef

Generated by 👨‍🍳 PR Sous Chef · 5.38 AIC · ⌖ 4.28 AIC · ⊞ 3.4K · ◷
_{Comment /souschef to run again}

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

Copilot · 2026-07-04T15:29:54Z

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

Committed remaining generated file changes (MCP Gateway v0.3.33 sync in lockfile). All review feedback was addressed in earlier commits. Branch is up to date. Local validation passes — CI re-trigger needed for final green checks.

github-actions · 2026-07-07T10:08:28Z

🎉 This pull request is included in a new release.

Release: v0.82.3

Clarify pr-sous-chef review dismissal criteria

df27388

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI assigned Copilot and pelikhan Jul 4, 2026

Copilot AI changed the title ~~Update pr-sous-chef github-actions review dismissal criteria~~ Clarify pr-sous-chef criteria for dismissing github-actions[bot] reviews Jul 4, 2026

Copilot created this pull request from a session on behalf of pelikhan July 4, 2026 10:40 View session

Copilot finished work on behalf of pelikhan July 4, 2026 10:40

Copilot AI requested a review from pelikhan July 4, 2026 10:40

Copilot started work on behalf of pelikhan July 4, 2026 10:52 View session

Use native dismiss_pull_request_review safe output

3adffe7

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot finished work on behalf of pelikhan July 4, 2026 11:10

github-actions Bot mentioned this pull request Jul 4, 2026

[aw] No-Op Runs #39849

Closed

pelikhan approved these changes Jul 4, 2026

View reviewed changes

pelikhan marked this pull request as ready for review July 4, 2026 12:11

Copilot AI review requested due to automatic review settings July 4, 2026 12:11

Copilot started reviewing on behalf of pelikhan July 4, 2026 12:11 View session

Copilot AI reviewed Jul 4, 2026

View reviewed changes

Copilot started work on behalf of gh-aw-bot July 4, 2026 12:22 View session

Document actor-bound guardrail for dismiss_pull_request_review in hum…

138d329

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

Copilot finished work on behalf of gh-aw-bot July 4, 2026 12:31

Copilot AI requested a review from gh-aw-bot July 4, 2026 12:31

github-actions Bot added pr-action:fast_track pr-agent:copilot-swe-agent pr-priority:high pr-risk:medium pr-type:refactor labels Jul 4, 2026

github-actions Bot mentioned this pull request Jul 4, 2026

[PR Triage Report] 🤖 PR Triage Report — 2026-07-04 [Run §28706384347] #43385

Closed

Copilot started work on behalf of gh-aw-bot July 4, 2026 13:27 View session

Copilot finished work on behalf of gh-aw-bot July 4, 2026 13:32

Copilot started work on behalf of gh-aw-bot July 4, 2026 14:07 View session

Copilot AI and others added 2 commits July 4, 2026 14:15

Merge remote-tracking branch 'origin/main' into copilot/update-pr-sou…

1081bdf

…s-chef-dismiss-review-bot Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>

Merge main and recompile pr-sous-chef lockfile

e2885dd

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

Copilot finished work on behalf of gh-aw-bot July 4, 2026 14:19

Copilot started work on behalf of gh-aw-bot July 4, 2026 15:23 View session

Sync MCP Gateway to v0.3.33 in pr-sous-chef lockfile

b1acd2c

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

Copilot finished work on behalf of gh-aw-bot July 4, 2026 15:30

pelikhan merged commit 953e4f9 into main Jul 4, 2026
20 checks passed

pelikhan deleted the copilot/update-pr-sous-chef-dismiss-review-bot branch July 4, 2026 15:43

		assert.Contains(t, text, "dismiss-pull-request-review", "Workflow should configure the native safe-output path for dismissing stale github-actions reviews")
		assert.Contains(t, text, "safeoutputs dismiss_pull_request_review", "Workflow should call the native dismiss_pull_request_review safe-output tool")

Uh oh!

Conversation

Copilot AI commented Jul 4, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pelikhan commented Jul 4, 2026

Uh oh!

Copilot AI commented Jul 4, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Review details

Uh oh!

gh-aw-bot commented Jul 4, 2026

Uh oh!

Copilot AI commented Jul 4, 2026

Uh oh!

github-actions Bot commented Jul 4, 2026

🤖 PR Triage — Run §28706384347

Uh oh!

gh-aw-bot commented Jul 4, 2026

Uh oh!

Copilot AI commented Jul 4, 2026

Uh oh!

gh-aw-bot commented Jul 4, 2026

Uh oh!

Copilot AI commented Jul 4, 2026

Uh oh!

gh-aw-bot commented Jul 4, 2026

Uh oh!

Copilot AI commented Jul 4, 2026

Uh oh!

Uh oh!

github-actions Bot commented Jul 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Copilot AI commented Jul 4, 2026 •

edited by github-actions Bot

Loading