Skip to content

Comments

feat: add ability to provide GitHub.com token to setup-bun to avoid rate limits#901

Closed
peloyeje wants to merge 1 commit intoanthropics:mainfrom
peloyeje:fix/setup-bun-rate-limit
Closed

feat: add ability to provide GitHub.com token to setup-bun to avoid rate limits#901
peloyeje wants to merge 1 commit intoanthropics:mainfrom
peloyeje:fix/setup-bun-rate-limit

Conversation

@peloyeje
Copy link
Contributor

@peloyeje peloyeje commented Feb 4, 2026

Summary

Pass github_dotcom_token input (or fallback github.token) to the setup-bun action, which prevents 403 rate limit errors when fetching Bun release tags from GitHub API on shared runners

Follow-up of #861

Problem

The setup-bun action fetches Bun release tags from the GitHub API. Without an authenticated token, it hits rate limits (403 error) when running from shared GitHub Actions runners:

Error: API rate limit exceeded for <IP>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)

@peloyeje peloyeje force-pushed the fix/setup-bun-rate-limit branch from 60a2301 to 73fa1b1 Compare February 4, 2026 14:15
@peloyeje peloyeje force-pushed the fix/setup-bun-rate-limit branch from 73fa1b1 to a7c3766 Compare February 4, 2026 14:17
@peloyeje peloyeje marked this pull request as ready for review February 4, 2026 14:17
@ashwin-ant
Copy link
Collaborator

An alternative here might be to just pass a path to pass path_to_bun_executable instead. That way you can install Bun however you like. That might be cleaner than adding another input that's only used for one thing.

https://github.com/anthropics/claude-code-action/blob/main/action.yml#L120C3-L123

@peloyeje
Copy link
Contributor Author

peloyeje commented Feb 9, 2026

An alternative here might be to just pass a path to pass path_to_bun_executable instead. That way you can install Bun however you like. That might be cleaner than adding another input that's only used for one thing.

main/action.yml#L120C3-L123

Sounds reasonable indeed!
Just opened #913 to supersede this one

@peloyeje
Copy link
Contributor Author

peloyeje commented Feb 9, 2026

Replaced by #913

@peloyeje peloyeje closed this Feb 9, 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.

2 participants