Skip to content

action: stop reporting user-side and transient errors - #603

Merged
IanButterworth merged 1 commit into
masterfrom
ib/error-report-triage
Oct 5, 2026
Merged

IanButterworth merged 1 commit into
masterfrom
ib/error-report-triage

Conversation

@IanButterworth

Copy link
Copy Markdown
Member

Claude: written with Claude Code (Opus 5.5).


Most open reports in JuliaRegistries/TagBotErrorReports are not TagBot bugs.

Now:

Most open TagBotErrorReports issues are not TagBot bugs. A 401 Bad
credentials, a 429 rate limit, an HTML 400 page from GitHub's front end
and an invalid `ssh` secret were all reported as internal failures. Now
401s are logged but not reported, 429s and HTML 400s are treated like
5xx errors, and an undecodable key raises Abort with a hint to use the
private key.

An Abort raised outside the per-version loop was silently swallowed
since #548, so the job exited 0 with no message. It is now logged and
the job exits 1.

The registry PR cache read `pr.merged`, which is not in the list
response and cost one extra request per PR. That request also failed
with 422 on a PR with too many changed files. The cache now checks
`merged_at`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@IanButterworth
IanButterworth merged commit 5f26a87 into master Oct 5, 2026
2 checks passed
@IanButterworth
IanButterworth deleted the ib/error-report-triage branch October 5, 2026 16:35
This was referenced Oct 5, 2026
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