feat: display Claude Fable weekly quota - #407
Conversation
Model: gpt-6
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 09e9fa9296
ℹ️ 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".
| label={formatAgentRateLimitWindowLabel( | ||
| window, | ||
| formatRateLimitWindowShortLabel(window.windowDurationSeconds), | ||
| t | ||
| )} |
There was a problem hiding this comment.
When the provider card is below the @sm container breakpoint, this changed block is hidden by hidden ... @sm:flex, while the @sm:hidden fallback later in the same component still uses only formatRateLimitWindowShortLabel. Consequently, Claude's shared weekly and Fable weekly meters both appear as 7d on mobile and narrow settings panels, making the new quota indistinguishable; apply formatAgentRateLimitWindowLabel in that fallback as well.
Useful? React with 👍 / 👎.
Model: gpt-6
2 participants
Claude's Fable weekly sub-cap was dropped because the adapter only read the shared 5-hour and 7-day fields, and Lody rendered window names from duration alone. Preserve the additive Core window label through ACP parsing and show it beside the duration in the usage popover, Provider row, and machine quota panel. Shared and Fable weekly constraints remain separate even when their duration, utilization, or reset time matches.
Pins the merged companion implementations: Core #3 and Claude adapter #25. Core is pinned to
7bc6332d3f007876895b4a3a827be0060f4d5318and Claude to414718e5238a7ed5da0ff23bec31bff4450ffa5f, the confirmed merge commits on their main branches. Both builds and all 8 Claude usage tests passed again with these revisions. Older unlabeled windows remain compatible; snapshots replace the window list, so removed scoped quotas do not linger. Also repairs the existing Codex lockfile specifier mismatch without changing its resolved version, allowing frozen installs.Validation: 34 targeted tests passed across adapter parsing, ACP reception, display, and legacy keying/migration. Core and Claude builds, Core typecheck, Claude lint/format checks, repository-wide typecheck and lint, i18n, public-boundary checks, and formatting of changed files passed. Browser checks passed for the real usage-popover component at 1024px and 390px and the compact Provider row. Full
NODE_ENV=test pnpm checkis still running its test phase. OAuth fixtures are synthetic; no live account credentials were used.