mirror of
https://github.com/github/awesome-copilot.git
synced 2026-07-16 19:03:26 +00:00
Gordon Lam
d47a6c93b7
* Add copilot-pr-autopilot skill Skill that drives any GitHub pull request through repeated rounds of Copilot Code Review until the agent has either resolved every thread or explicitly escalated it to the human. Triggered via GraphQL (no @copilot mention needed), triages every open thread with a fix / decline / escalate rubric, replies and resolves each thread citing the pushed SHA, then re-triggers until HEAD is reviewed with zero threads awaiting the agent's reply. Includes step scripts (01 request-review, 02 check-review-status, 03 list-open-threads, 08 reply-and-resolve, 10 cleanup-outdated), shared library (_lib.ps1) with gh-CLI wrappers (Invoke-Gh, Invoke-GhGraphQL, ConvertFrom-GhJson, Assert-GhReady), reply templates, and reference docs for each step. Repo-agnostic. Requires gh CLI on PATH and repo Triage/Write for full autopilot; external PR authors get single-iteration mode with manual re-trigger via the UI re-request button or a substantive push. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Address review: split per-step references + add recap-gate circuit breaker - Fix #1: give steps 1/7/10 their own reference files (01-request-review.md, 07-commit-push.md, 10-cleanup.md); trim the inline bodies out of orchestration.md so it stays cross-cutting only. - Fix #3: add a recurring round-cap & recap gate to 09-convergence.md — default STOP every 10th round, recap all prior rounds, detect drift (out-of-scope / over-engineering / wrong-direction / belongs-in-separate-PR) with CONTINUE / REVERT-AND-SHIP / HAND-OFF verdicts. Agent reasoning, no new script. - Surface the gate from SKILL.md and orchestration.md; regenerate docs/README.skills.md. Markdown-only change; scripts unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs(copilot-pr-autopilot): surface recap gate in decision pseudo-code; clarify Copilot+human convergence and round definition - Inject the round-cap recap gate into the '## Decision: loop back or exit' pseudo-code else-branch so an agent following the code block (not just the prose) runs the STOP-every-10th-round check before looping. - Broaden the 'never terminal' paragraph: non-convergence is driven by a Copilot finding OR a human review comment (this skill handles both); the loop ends only when there are no new comments from either source AND every open thread (Copilot or human) has an agent reply/escalation. - Define a 'round' explicitly as one execution of step 1 (01-request-review), i.e. one Copilot-review trigger — the cap counts review rounds, not tool calls or fix edits. Markdown-only; no script changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * copilot-pr-autopilot: make recap-gate round count deterministic Add 09-review-round.ps1: counts Copilot Code Review submissions straight from the PR's API history (full GraphQL pagination), so the recap-gate trigger is a derived number, not a fallible agent mental tally. This removes the exact failure mode the skill exists to survive — a count drifting across a long run (the real 156-round case). The script reports Round + RecapDue (Round % RecapInterval == 0) only; it never stops the loop or picks the verdict. CONTINUE / REVERT-AND-SHIP / HAND-OFF stays agent reasoning. 09-convergence.md updated to reference the deterministic count while preserving 'no script stops the loop' and 'non-convergence = Copilot finding OR human comment'. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Regenerate docs/README.skills.md for copilot-pr-autopilot (add 09-review-round.ps1) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
60 lines
2.4 KiB
Markdown
60 lines
2.4 KiB
Markdown
# Step 5: Apply fixes
|
|
|
|
Sub-agent type: `general-purpose`, **one sub-agent per finding**,
|
|
parallel **max 5 concurrent**; budget: 5 min each. If step 4 returned
|
|
more than 5 `fix` rows, the parent runs step 5 in waves of ≤5.
|
|
|
|
## Inputs
|
|
|
|
Per sub-agent (one finding per invocation):
|
|
- `thread_id`, `file`, `line` from step 3.
|
|
- The finding `summary` and Copilot's suggested fix (if any).
|
|
- The triage `rationale` from step 4 — the agent already decided this
|
|
is a `fix`; this sub-agent only implements it.
|
|
|
|
## Return contract
|
|
|
|
```
|
|
{ thread_id, files_touched, summary, status }
|
|
```
|
|
|
|
Where `status` ∈ `complete` | `partial` | `blocked` and `summary` is
|
|
a one-line description of the change.
|
|
|
|
## Procedure
|
|
|
|
1. **Discover repo conventions for the area being edited — first.**
|
|
Before writing any code, read:
|
|
- `.github/instructions/*.md` whose `applyTo` glob matches the
|
|
file's path,
|
|
- `.github/skills/`,
|
|
- `AGENTS.md`,
|
|
- `CONTRIBUTING.md`,
|
|
- neighbor-file patterns in the same directory and recent commits
|
|
touching similar files.
|
|
2. Apply the fix in line with those conventions.
|
|
3. Return `files_touched` + a one-line `summary` + `status`.
|
|
|
|
## Gotchas
|
|
|
|
- **Max 5 concurrent fix sub-agents.** The cap prevents fix-fanout
|
|
chaos; the parent merges results and reconciles file conflicts
|
|
between waves before step 6.
|
|
- **Never invent a generic answer that contradicts repo practice.**
|
|
That's the "elephant in school" anti-pattern — a Copilot suggestion
|
|
in isolation looks right but breaks the project's lint, format,
|
|
spell-check, license-header, or framework conventions. The
|
|
discovery step is mandatory, not optional.
|
|
- **One finding per sub-agent.** Fixes that need to touch the same
|
|
file get serialized by the parent between waves — don't merge
|
|
multiple findings into one sub-agent invocation.
|
|
- **Project policy beats Copilot suggestion.** If discovery surfaces
|
|
a documented convention (spell-check allowlist mechanism, lint
|
|
suppression style, etc.) that contradicts the suggested fix, follow
|
|
the convention and reflect that in the reply body drafted in step 8
|
|
(see [04-triage.md](04-triage.md#project-specific-policy-hooks)).
|
|
- **Push back with written rationale** if implementing the fix would
|
|
over-engineer the design for a hypothetical edge case — flip the
|
|
triage to `decline` and return `status: blocked` with the rationale
|
|
so step 8 drafts the right reply.
|