ci: cap release-please search depth to spare the shared PAT rate limit - #14
Merged
Merged
Conversation
release-please-action was called with its default release-search-depth (400) and commit-search-depth (500). It re-runs on every push to master across all forks, all authenticating as the same org PAT, so those deep searches burn the user's hourly GitHub API budget in bursts — a night of Renovate automerges or a run of manual merges trips "API rate limit already exceeded". The forks are single-package manifests with single-digit release counts and a monthly release window, so cap the depths well above real history (release 40, commit 250). This cuts per-run API cost ~10x on the release search while staying far above any realistic commits-since-last-release, so no changelog entries are missed. Co-Authored-By: Claude Opus 4.8 <[email protected]> Claude-Session: https://claude.ai/code/session_01UcmNgwVhqkNE1eViGYLnJ9
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.
What & why
release-please is failing with:
85048617is the shared org PAT user. The reusable workflow calledrelease-please-actionwith onlytoken, leavingrelease-search-depth(400) andcommit-search-depth(500) at their defaults. release-please re-runs on every push to master across all 12 forks, all authenticating as that same PAT, so each run pages back through hundreds of releases/commits and a burst of merges (a night of Renovate automerges, or a run of manual merges) exhausts the user's ~5000/hr API budget.The rate limit itself is transient and resets hourly — release-please recovers on the next push/dispatch, nothing is lost. This change is the durable fix so it stops recurring.
Change
The forks are single-package manifests with single-digit release counts and a monthly release cadence, so 40/250 sit well above real history — this cuts ~10× off the release-search paging while staying far above any realistic commits-since-last-release, so no changelog entries are missed.
Rollout
Reusable-workflow change; forks pick it up when their
@<sha>digest pin to this repo bumps (Renovate handles those digest updates). No fork edits needed.Type of change
fix/feat/perf— consumer-facingdeps— dependency updatechore/ci/build/docs/test/refactor— maintenance (ci)Checklist
commitlintpasses)Generated by Claude Code