Skip to content

Name the Copilot reviewer @copilot; skip the review poll when no reviewer was requested - #154

Merged
dmccoystephenson merged 1 commit into
mainfrom
fix/copilot-reviewer-handle
Oct 4, 2026
Merged

dmccoystephenson merged 1 commit into
mainfrom
fix/copilot-reviewer-handle

Conversation

@dmccoystephenson

Copy link
Copy Markdown
Member

The template's REVIEWER guidance said to use Copilot when Copilot review is configured. Generated skills therefore run gh pr edit <n> --add-reviewer Copilot, which now fails:

GraphQL: Could not resolve user with login 'copilot'. (requestReviewsByLogin)

On Dans-Plugins/Medieval-Factions, Copilot last reviewed a dev-loop PR on 2026-08-08 (#1999–#2001), and PRs #2083 and #2085 got no review at all. With --add-reviewer @copilot (gh 2.92.0), Copilot reviewed #2088 within about 2.5 minutes on 2026-10-04.

Changes (create-dev-loop.md, two lines)

  • The REVIEWER placeholder row now says to write @copilot, and explains why.
  • The "Reviewer addition fails" edge case names that error and its fix, and says not to run the Phase 5 poll when no reviewer was requested, since no review can arrive. That poll cost about 20 minutes on #2083. This is the generic form of Dans-Plugins/medieval-factions-dev-loop#69.

Reach: 48 generated skills under /root/local-skills contain --add-reviewer Copilot. They pick this up through the normal maintain-dev-loop propagation; none are edited here.


drafted by Claude on behalf of Daniel Stephenson

🤖 Generated with Claude Code

https://claude.ai/code/session_01RVZE12t6A9wpwgX4B422DV

…reviewer was requested

The bare `Copilot` login now fails ("Could not resolve user with login 'copilot'"); `@copilot`
produced a Copilot review within minutes on Dans-Plugins/Medieval-Factions#2088 (2026-10-04).
48 generated dev loops carry the old form; see Dans-Plugins/medieval-factions-dev-loop#69.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RVZE12t6A9wpwgX4B422DV
@dmccoystephenson
dmccoystephenson force-pushed the fix/copilot-reviewer-handle branch from 330e622 to 7954c5d Compare October 4, 2026 23:49
@dmccoystephenson
dmccoystephenson merged commit b80d361 into main Oct 4, 2026
1 check passed
@dmccoystephenson
dmccoystephenson deleted the fix/copilot-reviewer-handle branch October 4, 2026 23:50
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.

1 participant