Improve context overview navigation and status metrics / 优化会话概览跳转与底部状态指标#3895
Improve context overview navigation and status metrics / 优化会话概览跳转与底部状态指标#3895SivanCola wants to merge 3 commits into
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The redesign looks good and the cache-safety note holds — this is UI/status/telemetry only, and the new TotalTokens plumbing is clean and additive (nice that the test asserts both the telemetry aggregate and ContextPanel().TotalTokens). One coordination item before I approve, on Could you rebase onto the latest main-v2 and resolve it so that:
They're complementary — TotalTokens is additive and doesn't conflict with the snapshot breakdown. The locale/App.tsx files will also pick up conflicts from a few sibling desktop merges, same as the others. Ping me after the rebase and I'll do the full pass + approve. Thanks! |
0aa9f65 to
8d8dd80
Compare
|
Rebased onto the latest What changed:
Verified:
Ready for another pass when you have time. |
451a542 to
fb869f7
Compare
|
Updated the PR again on top of the latest Additional fix:
Verified:
|
Summary
Related PRs
desktop/tabs.go; this PR keeps the total-token field needed by the UI metrics.Cache safety
Verification
npm run buildgo test . -run TestWorkspaceTabAggregatesSessionUsageTelemetryfrom the desktop module