Add CLI cards view#1925
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Codex review: needs maintainer review before merge. Reviewed July 5, 2026, 8:20 PM ET / 00:20 UTC. Summary Reproducibility: not applicable. as a bug reproduction: this is a feature PR. The contributor supplied redacted current-head terminal output for both card modes, and source/tests cover the prior renderer issue. Review metrics: 2 noteworthy metrics.
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:
Next step before merge
Maintainer decision needed
Security Review detailsBest possible solution: Land this PR if a maintainer accepts the new CLI surface and the pending CI jobs finish green; otherwise narrow the card view into the maintainer-preferred CLI shape. Do we have a high-confidence way to reproduce the issue? Not applicable as a bug reproduction: this is a feature PR. The contributor supplied redacted current-head terminal output for both card modes, and source/tests cover the prior renderer issue. Is this the best way to solve the issue? Yes for the technical path: the branch reuses the existing usage fetch pipeline and adds focused renderers, tests, and docs. Product acceptance of the separate command and env override remains maintainer-owned. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against b43a0ef645ab. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat 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
Review history (4 earlier review cycles)
|
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Addressed the ClawSweeper findings in
Validation:
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
Co-authored-by: Cursor <cursoragent@cursor.com>
Let standard brief, card grid, and enhanced remaining bars render empty at 0% used or 0% remaining instead of forcing one filled cell. Add coverage for standard brief and exhausted card grid paths. Document that cards stdout is always rendered text and --json-output affects stderr only.
|
@clawsweeper re-review — pushed 7566af4 with zero-bar fixes, expanded tests, and redacted terminal proof in the PR body. |
|
🦞🧹 I asked ClawSweeper to review this item again. |
Summary
codexbar cards, a one-shot terminal usage snapshot with a responsive provider card grid.codexbar cards --brief, a compact Provider / Usage / Reset table with source/plan detail and enhanced terminal colors.codexbar usagefetch pipeline, summarize provider failures in a footer, and document the new command.Test plan
swift test --filter 'CLICards|CLITerminalCapabilities'(19 tests)swift build --product CodexBarCLI -c releasegit diff --checkmake checkreaches SwiftFormat cleanly, then fails on this Linux host when SwiftLint/SourceKit cannot loadlibsourcekitdInProc.so.Terminal proof (redacted)
codexbar cards --no-coloron Linux (head7566af4):codexbar cards --brief --no-coloron Linux (head7566af4):Notes
Package.swiftandPackage.resolvedare intentionally unchanged in the final PR surface.CODEXBAR_CARDS_ENHANCED=1as an explicit override.7566af4fixes zero-usage/zero-remaining bar rendering in standard brief and card grid paths.