Skip to content

revert(web): restore sparse hourly usage breakdown - #7718

Merged
maria-rcks merged 1 commit into
mainfrom
fix/restore-usage-numbers
Aug 20, 2026
Merged

revert(web): restore sparse hourly usage breakdown#7718
maria-rcks merged 1 commit into
mainfrom
fix/restore-usage-numbers

Conversation

@t3-code

@t3-code t3-code Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

What Changed

Why

The zero-filled chronological table put the oldest hours first, so recent non-zero usage values could be buried below a screen of empty rows and appear to have disappeared. The chart remains unchanged and still shows the complete 24-hour window.

UI Changes

Browser screenshots were skipped at the requester's direction. Before this fix, Hour rendered 24 rows from oldest to newest. After this fix, it renders only reported activity from newest to oldest.

Verification

  • regression test failed on main with 24 rows instead of 2, then passed after the revert
  • 17 focused usage tests pass
  • web typecheck passes
  • changed-file lint and formatting pass
  • git diff --check passes

Built with GPT-5.6-sol in the Codex harness.

request provenance


Note

Low Risk
UI-only table ordering/filtering for usage breakdown; no auth, data, or API changes. The chart still uses the full hourly window.

Overview
The 24h usage Hour breakdown no longer pads empty hours or lists oldest-first. It now shows only reported hourly rows, newest first, so recent spend is not buried under a screen of zeros. The chart still covers the full window.

Adds a UsagePage regression test that the table has two activity rows with the newer $11.00 above $13.00.

Reviewed by Cursor Bugbot for commit dcc98ac. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Restore sparse hourly usage breakdown in UsagePage

Stops synthesizing zero-filled empty hours for 1-day windows in UsagePage.tsx. The breakdown now shows only hours present in merged.hourly, ordered newest-first, and the zeroHour helper is removed. Adds a test suite in UsagePage.test.tsx that verifies sparse rows and newest-first ordering.

Macroscope summarized dcc98ac.

Co-authored-by: maria <254055478+maria-rcks@users.noreply.github.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Aug 20, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at dcc98ac

Macroscope's review found this PR approvable — This revert simplifies hourly usage table display by showing only hours with actual data instead of zero-filling every hour. The change removes complexity, adds a test, and restores previous behavior with clear intent.

You can add or adjust custom eligibility rules. Learn more.

@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.4 KiB 13.5 KiB +33 B (+0.2%) 15.1 KiB
Codex Thread snapshot wire 6.9 KiB 6.9 KiB +13 B (+0.2%) 7.3 KiB
Codex Live turn WebSocket wire 6.5 KiB 6.6 KiB +20 B (+0.3%) 7.8 KiB
Codex Live turn WebSocket decoded 55.0 KiB 55.0 KiB 0 B (0.0%) 66.4 KiB
Codex Live turn messages 16 16 0 (0.0%) 21
Claude Total thread wire 13.5 KiB 13.4 KiB −29 B (−0.2%) 15.1 KiB
Claude Thread snapshot wire 6.9 KiB 6.9 KiB +1 B (+0.0%) 7.3 KiB
Claude Live turn WebSocket wire 6.6 KiB 6.5 KiB −30 B (−0.4%) 7.8 KiB
Claude Live turn WebSocket decoded 55.8 KiB 55.8 KiB 0 B (0.0%) 66.4 KiB
Claude Live turn messages 16 16 0 (0.0%) 21

Baseline: 21e80a0 · PR result: dcc98ac · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 109.4 KiB
  • Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@maria-rcks
maria-rcks merged commit aa17ec6 into main Aug 20, 2026
17 checks passed
@maria-rcks
maria-rcks deleted the fix/restore-usage-numbers branch August 20, 2026 20:18
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 20, 2026
## What's Changed
* fix(web): retry failed thread bootstraps with a fresh id by @SunkenInTime in pingdotgg/t3code#7664
* fix(web): copy terminal selection instead of a blank clipboard by @sethwebster in pingdotgg/t3code#7678
* revert(web): restore sparse hourly usage breakdown by @t3-code[bot] in pingdotgg/t3code#7718
* fix(server): reconcile orphaned provider sessions by @t3-code[bot] in pingdotgg/t3code#7719
* fix(web): fix subagent row left border being cut off by @sameerr03 in pingdotgg/t3code#7207
* fix(web): unify composer control rounding by @kototok903 in pingdotgg/t3code#5957

## New Contributors
* @sethwebster made their first contribution in pingdotgg/t3code#7678
* @sameerr03 made their first contribution in pingdotgg/t3code#7207
* @kototok903 made their first contribution in pingdotgg/t3code#5957

**Full Changelog**: pingdotgg/t3code@v0.0.34-nightly.20260820.1142...v0.0.34-nightly.20260820.1146

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.34-nightly.20260820.1146
frankdavidcorona pushed a commit to frankdavidcorona/t3code that referenced this pull request Aug 21, 2026
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
Co-authored-by: maria <254055478+maria-rcks@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant