docs(pr-review): orchestrator split — ADR, PRD, and issues#27
Merged
Conversation
…split Records the decision to refactor review-pr.md into a thin orchestrator plus three focused agents (ado-fetcher, re-review-coordinator, ado-writer), and defines the three operating modes and orchestration agent terms in CONTEXT.md. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds the needs-triage PRD for refactoring review-pr.md into a thin orchestrator plus ADO Fetcher, Re-review Coordinator, and ADO Writer agents, covering the pre-PR mode, compact sub-agent output, and deferred toolkit normalisation decisions. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds agent brief and updates status to ready-for-agent. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Breaks the orchestrator split PRD into 7 independently-grabbable vertical slices: ADO Fetcher, ADO Writer, and Re-review Coordinator agents (parallelizable), thin orchestrator refactor, pre-PR mode, compact sub-agent output guidance, and version bump. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…-agent Adds agent briefs and updates status on all seven issues. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Documents the decision and work breakdown for splitting the pr-review command into a thin orchestrator plus three focused agents, and introduces the three operating modes (Pre-PR / First-review / Re-review) in the plugin’s shared vocabulary.
Changes:
- Adds ADR 0013 capturing the rationale and approach for the orchestrator split.
- Updates
apps/claude-code/pr-review/CONTEXT.mdwith new operating modes and orchestration agent definitions. - Publishes the feature PRD plus 7
ready-for-agentimplementation issues for the split (including pre-PR mode and compact finding schema).
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/issues/pr-review-orchestrator-split/PRD.md | New PRD defining the orchestrator split feature and its requirements/test strategy. |
| docs/issues/pr-review-orchestrator-split/01-create-ado-fetcher-agent.md | Implementation issue for the read-only ADO Fetcher agent. |
| docs/issues/pr-review-orchestrator-split/02-create-ado-writer-agent.md | Implementation issue for the ADO Writer (write-back) agent. |
| docs/issues/pr-review-orchestrator-split/03-create-re-review-coordinator-agent.md | Implementation issue for the Re-review Coordinator agent owning the state machine. |
| docs/issues/pr-review-orchestrator-split/04-refactor-orchestrator.md | Implementation issue for refactoring review-pr.md into a thin orchestrator. |
| docs/issues/pr-review-orchestrator-split/05-add-pre-pr-mode.md | Implementation issue for adding local-only Pre-PR mode. |
| docs/issues/pr-review-orchestrator-split/06-compact-subagent-output.md | Implementation issue for enforcing compact structured findings from aspect agents. |
| docs/issues/pr-review-orchestrator-split/07-version-bump-and-release.md | Implementation issue for version bump + changelog entry after feature delivery. |
| apps/claude-code/pr-review/docs/adr/0013-orchestrator-split-for-review-pr.md | New ADR recording the architectural decision and consequences. |
| apps/claude-code/pr-review/CONTEXT.md | Adds shared terminology for modes and the three orchestration agents. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
… issues - Make prerequisite validation mode-aware in issue 04: Azure CLI and azure-devops extension are only required when a PR URL is provided; Pre-PR mode must not need ADO credentials - Align field count in issue 06: schema has six fields (severity, filePath, startLine, endLine, title, body) but acceptance criteria incorrectly said "five required fields" — updated to "six" Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… docs - CONTEXT.md: Re-review mode trigger now says "prior Bot Signature is found" instead of "prior Review Threads are detected" (the Bot Signature is the precise discriminator) - CONTEXT.md + ADR 0013 + PRD: remove "delta summary" from Re-review Coordinator responsibilities — delta reply posting belongs to the ADO Writer, not the Re-review Coordinator - ADR 0013: fix compact finding schema field name file → filePath to match all other documents - ADR 0013: fix See also reference from docs/plans/ (empty) to the actual docs/issues/pr-review-orchestrator-split/PRD.md - PRD: remove spurious ## Comments heading, place triage blockquote directly under ## Agent Brief per peer-file convention - Issue 04: clarify ADO Fetcher bootstrapping sequence — orchestrator makes a lightweight thread-list call first for mode detection and prior-commit-SHA extraction, then calls ADO Fetcher with that SHA Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
Summary
review-pr.mdinto a thin orchestrator plus three focused agents (ADO Fetcher, Re-review Coordinator, ADO Writer)CONTEXT.mdwith the three operating modes (Pre-PR, First-review, Re-review) and orchestration agent termsWhy
review-pr.mdhas grown to ~1000 lines, pushing average PR reviews past 100 K parent-context tokens. An emerging pre-PR mode request would worsen this. The split separates orchestration, ADO integration, and re-review state management into focused agents that only load when their mode is active.Test plan
ready-for-agentwith agent briefspnpm formatpasses (Markdown only)🤖 Generated with Claude Code