Skip to content

pr-sous-chef: auto-resolve outdated addressed review threads via thread-resolver sub-agent#41959

Merged
pelikhan merged 2 commits into
mainfrom
copilot/feat-pr-sous-chef-add-thread-resolver
Jun 27, 2026
Merged

pr-sous-chef: auto-resolve outdated addressed review threads via thread-resolver sub-agent#41959
pelikhan merged 2 commits into
mainfrom
copilot/feat-pr-sous-chef-add-thread-resolver

Conversation

Copilot AI commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

PR Sous Chef now auto-resolves review threads that are both unresolved and outdated, reducing stale reviewer noise without changing nudge behavior. The workflow adds a lightweight resolver path that runs independently and fails open.

  • Sub-agent: thread-resolver

    • Adds a dedicated compact sub-agent that queries PR review threads via MCP pull_requests.
    • Filters to isResolved: false && isOutdated: true.
    • Returns only thread node IDs; capped at 4 tool calls.
  • Safe output capability

    • Adds resolve-pull-request-review-thread to safe-outputs with max: 20 and target: "*".
    • Enables controlled resolution via:
      safeoutputs resolve_pull_request_review_thread --thread_id <id>
  • Main agent policy updates

    • Token-efficiency rules now require invoking thread-resolver for eligible PRs.
    • Resolver failures are silent-skip (sub_agent_error-style behavior) and do not abort PR processing.
    • Allowed safe-output list now includes resolve_pull_request_review_thread.
  • Execution contract / reporting

    • Adds a required “thread resolution” section with explicit call flow.
    • Extends final run-summary noop payload with threads_resolved=N for visibility.

Example resolution flow:

{"thread_ids":["PRRT_kwDO...","PRRT_kwDO..."]}
safeoutputs resolve_pull_request_review_thread --thread_id PRRT_kwDO...

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add thread-resolver sub-agent to PR Sous Chef pr-sous-chef: auto-resolve outdated addressed review threads via thread-resolver sub-agent Jun 27, 2026
Copilot AI requested a review from pelikhan June 27, 2026 22:37
@pelikhan pelikhan marked this pull request as ready for review June 27, 2026 22:38
Copilot AI review requested due to automatic review settings June 27, 2026 22:38
@pelikhan pelikhan merged commit 633cd56 into main Jun 27, 2026
@pelikhan pelikhan deleted the copilot/feat-pr-sous-chef-add-thread-resolver branch June 27, 2026 22:38
Copilot stopped reviewing on behalf of pelikhan due to an error June 27, 2026 22:38

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

Note

Copilot was unable to run its full agentic suite in this review.

Adds GH_AW_LABEL_COMMANDS configuration across multiple GitHub Actions “*.lock.yml” workflows, likely to enable label-driven command routing/behavior in the gh-aw scripts.

Changes:

  • Introduces GH_AW_LABEL_COMMANDS env var in multiple workflows (typically in both the “setup globals” step env and the job-level env).
  • Standardizes label-command configuration for smoke/dev/necromancer-style workflows (with some label value variations).
Show a summary per file
File Description
.github/workflows/smoke-otel-backends.lock.yml Adds GH_AW_LABEL_COMMANDS to smoke OTEL backends workflow envs.
.github/workflows/smoke-copilot.lock.yml Adds GH_AW_LABEL_COMMANDS to smoke Copilot workflow envs.
.github/workflows/smoke-copilot-sdk.lock.yml Adds GH_AW_LABEL_COMMANDS (uses smoke-sdk) to SDK smoke workflow envs.
.github/workflows/smoke-copilot-aoai-entra.lock.yml Adds GH_AW_LABEL_COMMANDS to AOAI Entra smoke workflow envs.
.github/workflows/smoke-copilot-aoai-apikey.lock.yml Adds GH_AW_LABEL_COMMANDS to AOAI API key smoke workflow envs.
.github/workflows/necromancer.lock.yml Adds GH_AW_LABEL_COMMANDS to necromancer workflow envs.
.github/workflows/dev.lock.yml Adds GH_AW_LABEL_COMMANDS to dev workflow envs.
.github/workflows/cloclo.lock.yml Adds GH_AW_LABEL_COMMANDS to cloclo workflow envs.
.github/workflows/ci-doctor.lock.yml Adds GH_AW_LABEL_COMMANDS to ci-doctor workflow envs.
.github/workflows/approach-validator.lock.yml Adds GH_AW_LABEL_COMMANDS (two labels) to approach-validator workflow envs.

Review details

Tip

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

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

Comment on lines 1131 to +1132
GH_AW_COMMANDS: "[\"smoke-otel-backends\"]"
GH_AW_LABEL_COMMANDS: "[\"smoke\"]"
Comment on lines 970 to +971
GH_AW_COMMANDS: "[\"smoke-copilot-sdk\"]"
GH_AW_LABEL_COMMANDS: "[\"smoke-sdk\"]"
Comment on lines 2000 to 2002
GH_AW_COMMANDS: "[\"smoke-copilot\"]"
GH_AW_LABEL_COMMANDS: "[\"smoke\"]"
with:
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.82.0

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.

3 participants