Repository navigation
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
name dropping me like that, smh |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 2b537c0. Configure here.
ApprovabilityVerdict: Needs human review This PR introduces a new user-facing feature (account limit meter) spanning backend adapters, contracts, shared utilities, and frontend components across 10 files. Additionally, an unresolved review comment identifies a medium-severity bug where the meter may display misleading information when the secondary rate limit window is more constrained than the primary. You can customize Macroscope's approvability policy. Learn more. |
|
This is an older account-usage meter implementation that overlaps active #1732 and the current rate-limit event plumbing. Consolidating on one implementation will avoid duplicate UI and protocol work. This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen. |

What Changed
Added a small UI element using the same UI as the context remaining display (just using different color). With a hover tooltip for details
As it is now, this is codex-only. I didn't look into implementing into other harnesses because I don't have an active sub on those.
Why
This is a useful way to see the account limits at a glance (instead of having to open a terminal, run codex, type /status). Courtesy of @iPixelGalaxy for bullying me into using my codex subscription to make this
UI Changes
Checklist
Note
Medium Risk
Adds a new cross-layer runtime event and normalization schema for account rate limits, plus new UI driven off activity ingestion; mistakes could cause missing/incorrect limits or extra event noise but does not touch auth or persistence.
Overview
Adds end-to-end support for displaying account rate limits in the chat composer.
On the server, Codex now proactively fetches
account/rateLimits/readon session start and afterturn/completed/account/updated, emittingaccount/rateLimits/updated; provider adapters (Codex + Claude) attach a newnormalizedsnapshot toaccount.rate-limits.updated, and orchestration ingestion converts that into anaccount-rate-limits.updatedthread activity.On the web, introduces
AccountLimitMeterand wires it intoChatComposer, deriving the latest limit snapshot from activities and hiding these activities from the work log.Reviewed by Cursor Bugbot for commit 2b537c0. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Add an account limit meter to the chat composer
normalizeAccountRateLimitsutility (packages/shared/src/accountLimits.ts) that parses provider-specific rate-limit payloads into a consistentAccountRateLimitsSnapshot, selecting the most constrained window.account/updatednotifications via a newrefreshRateLimitsmethod in CodexAppServerManager.ts; failures are non-fatal and logged at debug level.account.rate-limits.updatedruntime events, which the ingestion layer converts toaccount-rate-limits.updatedactivities.account-rate-limits.updatedactivities out of work log entries in session-logic.ts.Macroscope summarized 2b537c0.