Skip to content

fix: make homebrew tap update script retry-safe - #66

Merged
designcode merged 2 commits into
mainfrom
feat/homebrew-tap
Apr 6, 2026
Merged

designcode merged 2 commits into
mainfrom
feat/homebrew-tap

Conversation

@designcode

@designcode designcode commented Apr 6, 2026 •

Copy link
Copy Markdown
Collaborator

Handle re-runs when the branch was pushed but PR creation failed (e.g. due to missing label). Checks for existing open PRs and remote branches before attempting to push or create a new PR.


Note

Low Risk
Low risk: changes are limited to the Homebrew tap automation script, but could affect release/update CI behavior if the new PR/branch detection or gh auth assumptions are wrong.

Overview
Makes scripts/update-homebrew.sh idempotent on CI reruns by checking for an existing open PR for the version branch and exiting early.

If the remote branch already exists but no PR is open, the script now skips re-downloading artifacts/pushing and directly creates the PR. It also exports GH_TOKEN once for gh CLI usage and drops the previously-required PR label flag.

Reviewed by Cursor Bugbot for commit 0ecef5c. Bugbot is set up for automated code reviews on this repo. Configure here.

Comment thread scripts/update-homebrew.sh Resolved
designcode and others added 2 commits April 6, 2026 16:57
Handle re-runs when the branch was pushed but PR creation failed (e.g.
due to missing label). Checks for existing open PRs and remote branches
before attempting to push or create a new PR.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Existing-PR and remote-branch checks now run before downloading
archives and computing SHA256 hashes, avoiding unnecessary work on
retries.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 0ecef5c. Configure here.

Comment thread scripts/update-homebrew.sh Resolved
@designcode
designcode merged commit d673984 into main Apr 6, 2026
3 checks passed
@designcode
designcode deleted the feat/homebrew-tap branch April 6, 2026 15:45
@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.16.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants