Repository navigation
Conversation
Amp-Thread-ID: https://ampcode.com/threads/T-4a2bb2e2-357a-4895-be05-c107eea2bdc4 Co-authored-by: Amp <[email protected]>
wolfeidau
marked this pull request as ready for review
October 6, 2026 03:48
Contributor
There was a problem hiding this comment.
Want to dig deeper?
Paste this into your agent to explore the findings from this review's Buildkite build:
Download the buildsworth logs from build 36820, then answer my questions about the findings.
Install the reading-buildsworth-logs skill to run this.
About buildsworth
Model: gpt-6.1-sol with xhigh thinking.
How to request a review: Comment @buildsworth-bk review on the PR, or request buildsworth-bk as a reviewer.
Risk labels (how buildsworth classifies risk) — buildsworth classifies risk itself from the diff. Unless repository policy already allows L2 approval, grant it by mentioning @buildsworth-bk (see approval ceiling and L2 approval grant):
- L1 — Low risk (dep bumps, docs/copy, lockfiles, small presentational fixes). buildsworth may approve by default.
- L2 — Standard risk (new UI, additive API fields, refactors). Approved only when repository policy or a verified grant allows it; otherwise comment-only.
- L3 — High risk (auth, migrations, payments, secrets, perf-critical pipeline paths). Human approval required unless repository policy explicitly allows L3 approval; per-PR grants cannot allow it.
Amp-Thread-ID: https://ampcode.com/threads/T-4a2bb2e2-357a-4895-be05-c107eea2bdc4 Co-authored-by: Amp <[email protected]>
buildsworth-bk-app
Bot
dismissed
their stale review
October 6, 2026 03:58
Superseded by a newer Buildsworth review.
wolfeidau
deleted the
pb-3588-expose-github-repository-connection-rate-limits-in-mcp-and
branch
October 6, 2026 04:16
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
When GitHub Actions workloads in Buildkite fail or slow down, one cause is the GitHub installation behind the organization's repository connection running out of API quota.
bkhad no way to see that quota, and it's easy to confuse with the Buildkite API rate limit. The repository connections REST API (already supported in go-buildkite v5.21.0) exposes it on the show endpoint.Changes
Adds
bk repository-connection(aliasrepo-connection) with two read-only subcommands. Both require organization administrator access.list(ls) lists the organization's connections: display name, type, and ID. The API doesn't paginate this list or include rate limits in it.view <connection-id>shows a connection's service account, host, and GitHub API rate limit, labelled as the GitHub installation quota rather than the Buildkite API rate limit.When there's no cached quota, or the connection type has none, the rate limit is shown as unavailable rather than as zeros. In JSON and YAML,
rate_limit,service_account, andhostare always present andnullwhen unavailable. Ifreset_athas passed, text output marks used and remaining as stale.Most of the
main.godiff is gofumpt realigning the root command struct around the longer field name.Testing
go test ./...)go fmt ./...)Disclosures / Credits
Implemented with Amp.