Update Homebrew cask install command#1189
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates installation instructions to use the standard Homebrew cask name instead of a custom tap, simplifying setup for users.
Changes:
- Replace
brew install --cask steipete/tap/codexbarwithbrew install --cask codexbar
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ### Homebrew | ||
| ```bash | ||
| brew install --cask steipete/tap/codexbar | ||
| brew install --cask codexbar |
|
Codex review: needs real behavior proof before merge. Reviewed May 27, 2026, 7:24 AM ET / 11:24 UTC. Summary Reproducibility: yes. The compatibility problem is source-verifiable from the PR diff plus Homebrew cask metadata showing the default Review metrics: 1 noteworthy metric.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Copy recommended automerge instructionNext step before merge Security Review findings
Review detailsBest possible solution: Make the README install guidance architecture-aware: either keep the tap command as primary/fallback, or label Do we have a high-confidence way to reproduce the issue? Yes. The compatibility problem is source-verifiable from the PR diff plus Homebrew cask metadata showing the default Is this the best way to solve the issue? No. Replacing the command alone is not the best fix; the maintainable docs path is to preserve the tap/fallback command or make the official cask's Apple Silicon-only scope explicit. Full review comments:
Overall correctness: patch is incorrect AGENTS.md: found and applied where relevant. Codex review notes: model gpt-5.5, reasoning high; reviewed against 83ed8e405541. Label changesLabel changes:
Label justifications:
Evidence reviewedAcceptance criteria:
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 08a7ef4a9a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| ### Homebrew | ||
| ```bash | ||
| brew install --cask steipete/tap/codexbar | ||
| brew install --cask codexbar |
There was a problem hiding this comment.
Preserve Intel Homebrew install path
For Intel users on macOS 14+, this new command resolves to the Homebrew/homebrew-cask codexbar cask, whose current metadata has depends_on arch: :arm64; Homebrew will reject the install before downloading the universal release. The previous tap cask does not declare that arm64-only constraint, so this README change removes the only Homebrew install path that works for Intel Macs despite the Requirements section still saying only macOS 14+ is needed.
Useful? React with 👍 / 👎.
|
@copilot I opened a Homebrew/homebrew-cask PR to fix that here: I also verified this on an Intel Mac:
Both |
|
To use Codex here, create a Codex account and connect to github. |
| ### Homebrew | ||
| ```bash | ||
| brew install --cask steipete/tap/codexbar | ||
| brew install --cask codexbar |
Summary
Context
Verification
Docs-only change; build/tests not run.