Show flame beside ahead-of-pace status#8
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the CodexBar limit detail row so the “ahead-of-pace” status is visually aligned with the menu bar flame indicator (and ClaudeBar), displaying 🔥 Ahead of pace in the menu UI.
Changes:
- Prefix the ahead-of-pace status row with a flame emoji in
LimitRowView.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Code Review
This pull request adds a fire emoji to the "Ahead of pace" text in LimitRowView. The review feedback suggests extracting this hardcoded string into a static constant to improve maintainability and support future localization.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
@codex review |
|
Codex Review: Didn't find any major issues. 🚀 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Why
The menu bar icon already uses a flame for an ahead-of-pace limit, but the matching detail row showed plain text. This makes the two presentations consistent and restores parity with ClaudeBar.
User impact
Ahead-of-pace limits now display as
🔥 Ahead of pacein the CodexBar menu.Validation
git diff --checkswift buildswift test— 22 tests passedmake installand relaunch of~/Applications/CodexBar.app