Skip to content

[codex] Double pro quota ratio and align quota display#37

Merged
Dikevin merged 3 commits into
masterfrom
codex/pro-quota-double
May 8, 2026
Merged

[codex] Double pro quota ratio and align quota display#37
Dikevin merged 3 commits into
masterfrom
codex/pro-quota-double

Conversation

@Dikevin
Copy link
Copy Markdown
Collaborator

@Dikevin Dikevin commented May 8, 2026

Summary

This PR doubles the normalized pro quota capacity and makes codexm list and codexm tui use the same quota display budgets and styling rules. It also removes unit tests that only asserted static quota/config mappings.

What changed

  • Changed pro one-week capacity from 25/3 to 50/3 plus-equivalent units.
  • Centralized shared quota display widths, ANSI-aware padding/truncation, identity compaction, and blocked-row styling in src/cli/quota-display.ts.
  • Updated CLI and TUI list rendering to share column budgets; USED is fixed at width 6 so decimal labels like 62.5% fit.
  • Made TUI switch to stacked list/detail layout earlier when side-by-side panes would crowd the detail area.
  • Documented the remaining CLI/TUI differences that are caused by stdout vs interactive pane rendering.
  • Removed tests whose only purpose was asserting static config/profile values.

Impact

User-visible: pro accounts now contribute double the normalized quota capacity, CLI/TUI quota rows align more consistently, and medium-width TUI terminals prefer stacked layout before the detail pane becomes cramped.

No release is included in this PR: package version, git tag, and npm publish are intentionally deferred.

Validation

  • git diff --check passed.
  • pnpm verify:full passed outside the sandbox. The sandboxed run failed only because the test process could not chmod /Users/bytedance/.codex-team (EPERM).
  • Live run against built dist/cli.js with temp HOME/CODEX_HOME:
    • node dist/cli.js list and PTY width 76 list rendering showed decimal USED labels aligned.
    • node dist/cli.js tui in PTY width 120 used stacked list/detail layout.
    • node dist/cli.js tui in PTY width 136 used wide layout with 62.5% / 27.4% inside the USED columns.

@Dikevin Dikevin marked this pull request as ready for review May 8, 2026 09:19
@Dikevin Dikevin merged commit 863b220 into master May 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant