Skip to content

document handling for Copilot internal errors during review#22

Merged
MaceWindu merged 2 commits into
masterfrom
docs/copilot-error-skip
May 17, 2026
Merged

document handling for Copilot internal errors during review#22
MaceWindu merged 2 commits into
masterfrom
docs/copilot-error-skip

Conversation

@MaceWindu
Copy link
Copy Markdown
Owner

Summary

When Copilot returns an internal-error body (quota / service failure) instead of inline comments, re-request once and otherwise continue with the next task rather than blocking. CI gates correctness; Copilot is advisory.

Test plan

  • Markdown-only change — no code paths affected.

🤖 Generated with Claude Code

When Copilot returns an error body (quota / internal failure) instead of
inline comments, the right move is to re-request once and, if it errors
again, treat the review as unavailable and continue with the next task
rather than blocking on a check we can't influence. CI gates correctness.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 17, 2026 20:54
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

The bot does run as a GitHub Actions workflow named "Running Copilot
Code Review" — it just isn't attached to the PR through the standard
pull_requests field, so `gh pr checks` doesn't surface it. The workflow's
log carries the actual failure (errorType + statusCode), which is the
only way to tell `rate_limit` (skip and move on) apart from a transient
error (retry once). Document the lookup-by-branch query and the policy.

Today's case: both PR #21 and #22 hit `errorType: rate_limit`,
statusCode 429 ("reached your weekly rate limit, reset in ~3 hours").

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@MaceWindu MaceWindu merged commit 0d38452 into master May 17, 2026
8 checks passed
@MaceWindu MaceWindu deleted the docs/copilot-error-skip branch May 17, 2026 21:03
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.

2 participants