feat: pipeline orchestrator + review-responder thread ID fix#130
Merged
feat: pipeline orchestrator + review-responder thread ID fix#130
Conversation
Add step 6: agent queries real PRRT_ thread IDs using gh api graphql before resolving. No bash: tool config needed — the responder already has --allow-all-tools from the compiler default (no explicit bash tools = unrestricted). Adding bash: would restrict the allowlist and break CI commands (uv, ruff, pyright, pytest). GraphQL query uses proper -f/-F variables for owner, name, and PR number. Refs #114. Closes #117. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Pipeline orchestrator owns the full lifecycle (issue → PR → merge): - Dispatches implementer for eligible issues (one at a time) - Gates: no dispatch if aw PR in flight OR implementer already running - Uses actions toolset to check in-progress workflow runs - Unsticks stuck PRs: requests reviews, resolves threads via GraphQL - Uses GITHUB_REPOSITORY_OWNER for responder identity - Behind main: logs and skips (requires manual rebase) Removes dispatch-workflow from code-health and test-analysis — orchestrator is now the sole dispatcher. Deletes pr-rescue.yml. Closes #116. Closes #90. Refs #129. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR introduces a Pipeline Orchestrator — a new gh-aw agent workflow that owns the full lifecycle of agent work from issue dispatch to PR merge. It also updates the review-responder instructions to query real GraphQL thread IDs before resolving, and removes the old pr-rescue.yml bash script.
Changes:
- New
pipeline-orchestrator.mdagent workflow that dispatches the issue-implementer for eligible issues (one at a time) and unsticks stuck PRs (requests Copilot reviews, resolves addressed threads via GraphQL, logs behind-main PRs) - Removed
dispatch-workflowcapability fromcode-healthandtest-analysisworkflows (orchestrator is now the sole dispatcher) and deletedpr-rescue.yml - Updated review-responder instructions with a new step 6 for querying real
PRRT_thread IDs viagh api graphql, plus extensive documentation and changelog updates
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/pipeline-orchestrator.md |
New orchestrator agent: issue dispatch + PR unsticking logic |
.github/workflows/pipeline-orchestrator.lock.yml |
Auto-generated compiled workflow for the orchestrator |
.github/workflows/review-responder.md |
Added step 6 for GraphQL thread ID lookup before resolving |
.github/workflows/code-health.md |
Removed dispatch-workflow safe-output |
.github/workflows/code-health.lock.yml |
Recompiled: removed dispatch-workflow config and permissions |
.github/workflows/test-analysis.md |
Removed dispatch-workflow safe-output |
.github/workflows/test-analysis.lock.yml |
Recompiled: removed dispatch-workflow config and permissions |
.github/workflows/pr-rescue.yml |
Deleted: replaced by pipeline orchestrator |
docs/agentic-workflows.md |
Updated architecture diagram, added pitfalls #13-17, orchestrator docs |
docs/changelog.md |
Changelog entries for orchestrator and thread ID fix |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
CI fixer dispatch was hardwired into ci.yml as a separate job. Now the orchestrator detects CI failures on aw-labeled PRs and dispatches ci-fixer — all dispatch decisions in one place. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
microsasa
force-pushed
the
feat/pr-rescue-agent
branch
from
1521078 to
8315dff
Compare
There was a problem hiding this comment.
Pull request overview
This PR introduces a pipeline orchestrator gh-aw agent that centralizes the lifecycle management of agent-created PRs, replaces the old pr-rescue.yml bash script, and fixes the review-responder's thread ID hallucination problem.
Changes:
- New
pipeline-orchestrator.mdagent workflow that dispatches implementer for eligible issues (one at a time) and unsticks stalled PRs (requests Copilot review, resolves addressed threads via GraphQL, dispatches CI fixer) - Updated
review-responder.mdwith a new step 6 to query realPRRT_thread IDs viagh api graphqlbefore resolving, preventing hallucinated IDs - Removed
dispatch-workflowfrom code-health and test-analysis agents, removed CI fixer dispatch fromci.yml, and deletedpr-rescue.yml— all dispatch decisions now centralized in the orchestrator
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/pipeline-orchestrator.md |
New orchestrator agent: issue dispatch + PR unsticking logic |
.github/workflows/pipeline-orchestrator.lock.yml |
Compiled workflow for the orchestrator |
.github/workflows/review-responder.md |
Added step 6: GraphQL thread ID lookup before resolving |
.github/workflows/ci.yml |
Removed dispatch-ci-fixer job (moved to orchestrator) |
.github/workflows/code-health.md |
Removed dispatch-workflow config and updated instructions |
.github/workflows/code-health.lock.yml |
Compiled changes: removed dispatch-workflow, actions:write |
.github/workflows/test-analysis.md |
Removed dispatch-workflow config and updated instructions |
.github/workflows/test-analysis.lock.yml |
Compiled changes: removed dispatch-workflow, actions:write |
.github/workflows/pr-rescue.yml |
Deleted — replaced by pipeline orchestrator |
docs/agentic-workflows.md |
Updated architecture diagram, added pitfalls #13-17, history |
docs/changelog.md |
Added changelog entries for orchestrator and label gate fix |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This was referenced Mar 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pipeline Orchestrator
New gh-aw agent that owns the full lifecycle of agent work — from issue to merged PR.
Issue dispatch (one at a time):
aw-labeled PR is already in flight AND no implementer run is in progresscode-health/test-auditissues with no PRissue-implementerfor the first eligible issuePR orchestration (unstick what's in flight):
$GITHUB_REPOSITORY_OWNERfor identity)Removes
dispatch-workflowfrom code-health and test-analysis — orchestrator is the sole dispatcher. Deletespr-rescue.yml.Review-Responder Thread ID Fix
Adds step 6: query real
PRRT_thread IDs viagh api graphqlbefore resolving. Nobash:tool config added — the responder already has--allow-all-tools(addingbash:would restrict the allowlist and break CI commands).Documentation
Closes #116. Closes #117. Closes #90. Refs #129.