On Usage, select Past 24h, choose Hour under Breakdown, and switch from Cost to Tokens. The summary and chart change, but the provider columns and Total still show dollar costs. The same hardcoded cells affect the daily breakdown.
Use the selected metric for provider and total values in the time breakdown. Show Cost in the trailing column in Tokens mode, and Tokens in Cost mode. Keep the selected grouping and row order.
This is a small, focused fix for an obvious bug under the CONTRIBUTING.md exception. It connects the existing metric selector to the existing hourly and daily table. It adds no new capability, setting, or default. Web and desktop share this table; no server, contract, or mobile behavior changes.
vp run check:changed passed format, lint, knip, typecheck for two packages, and the focused tests. usageBreakdown.test.ts passed all 4 tests, including hourly and daily values, zero-cost usage with tokens, missing providers, and the secondary column.$1.65 to 5.12M, Claude Code from $2.86 to 9.01M, and Total from $4.51 to 14.1M. The final column changed from Tokens to Cost. Hour stayed selected. Switching back restored the cost values.Before screenshots are from the original report. After screenshots are from the local preview with a later snapshot of usage data, so the overall totals differ. The Browser panel captures have soft text; the observed table values are recorded above.
Tokens mode before still shows provider costs
Tokens mode after shows provider tokens and trailing cost
Implemented with GPT-6.1 Sol through the Codex harness in T3 Code.
Review completed
Added 1 commit
Verdict: Approved at 6a08907
Macroscope's review found this PR approvable — This is a small, self-contained web bug fix that makes the existing Cost/Tokens selector control provider and total values in daily and hourly breakdowns. It adds focused coverage without changing APIs, defaults, data processing, or deployment behavior.
You can add or adjust custom eligibility rules. Learn more.
No actionable comments were generated in the recent review. 🎉
aa16c6a1-cc69-4f54-afee-a63d51924b24Reviewing files that changed from the base of the PR and between ef92441313da327139df75daa8f723c0f42348f6 and 6a08907243e1247d109439195bf031af6cd4131a.
apps/web/src/components/usage/UsagePage.tsxapps/web/src/components/usage/usageBreakdown.test.tsapps/web/src/components/usage/usageBreakdown.tsIncluded review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.
The usage time-breakdown table now displays cost or token values according to the selected metric. Its secondary column shows the other metric, and missing provider totals format as zero.
Changes: Usage time breakdown
| Layer / File(s) | Summary |
|---|---|
Metric configuration and tests apps/web/src/components/usage/usageBreakdown.ts, apps/web/src/components/usage/usageBreakdown.test.ts | timeBreakdownMetric selects primary and secondary values and their formatters for cost or tokens. Tests cover daily and hourly totals, including missing provider totals. |
Usage table integration apps/web/src/components/usage/UsagePage.tsx | The time-breakdown table uses the selected metric configuration for its secondary column label, provider values, and period totals. |
Priority: ➖ Normal
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Bug fix
Suggested reviewers: t3dotgg, 0pilatos0
6a089The table appears ready to merge after normal checks; no actionable risk was identified.
| Check name | Status | Explanation |
|---|---|---|
| Title check | Passed | The title clearly and concisely describes the main fix: making the usage time breakdown respect the selected metric. |
| Description check | Passed | The description includes all required sections, explains the bug and fix, documents the scope exception, and provides focused automated and manual verification with before-and-after screenshots. |
| Linked Issues check | Passed | Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | Passed | Check skipped because no linked issues were found for this pull request. |
Comment @coderabbitai help to get the list of available commands.