Skip to content

Commit 6879024

Browse files
Document permission requirement: full loop needs repo Triage/Write
Add an explicit 'Permissions: who can run the full loop' section to SKILL.md so users know up-front which mode applies to them. Repo collaborators with Triage/Write get hands-off multi-round autopilot. External PR authors get single-iteration mode plus a documented manual re-trigger (UI 🔄 or push a substantive commit). The limitation is GitHub's, not ours — verified that requestReviewsByLogin is the only public API for bot reviewers and it requires write permission. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 3a4684d commit 6879024

1 file changed

Lines changed: 11 additions & 0 deletions

File tree

skills/copilot-pr-autopilot/SKILL.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,17 @@ installed and authenticated (see Prerequisites).
4545
fails (the script can't reliably tell "Copilot disabled" from
4646
"Copilot enabled but not yet triggered" from API state alone).
4747

48+
### Permissions: who can run the full loop
49+
50+
The full multi-round autopilot (steps 1 → 9 → 1) needs **Triage or Write** permission on the target repo, because GitHub's only public API for adding the Copilot bot as a reviewer (`requestReviewsByLogin`) is gated on that permission. Verified against the public REST + GraphQL surface in this PR's commit history — there is no public-API path for bot reviewers without write permission.
51+
52+
| You are… | What works |
53+
|---|---|
54+
| **Repo collaborator with Triage / Write** | Full loop: `01` triggers Copilot, `02` waits, `04``08` triage / fix / reply, loop back to `01`. Hands-off. |
55+
| **External PR author (no write permission)** | `01` will throw a clear actionable error. Use `-SingleIteration` mode: address all current findings in one pass, then either click the UI 🔄 next to Copilot, **or** push a substantive commit (the `synchronize` event auto-triggers Copilot on most repos). Then re-run `02` to verify. |
56+
57+
In single-iteration mode the loop's convergence boolean is `Converged: true` iff `OpenThreadsAwaitingReply == 0` (the agent's side is done). The maintainer-side re-trigger then drives any additional rounds.
58+
4859
Every script dot-sources [scripts/_lib.ps1](scripts/_lib.ps1) which
4960
runs `Assert-GhReady` on load: if `gh` is missing OR `gh auth status`
5061
fails, the script halts **before any work** with a single actionable

0 commit comments

Comments
 (0)