feature: local-usage-stats (4/4) - #1134
Open
myk1yt wants to merge 116 commits into
Open
Conversation
myk1yt
requested review from
JamesRobert20,
edelauna,
hannesrudolph,
navedmerchant and
taltas
as code owners
August 4, 2026 11:47
Contributor
|
Important Review skippedToo many files! This PR contains 186 files, which is 36 over the limit of 150. To get a review, reduce the PR to 150 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (186)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
github-actions
Bot
added
the
awaiting-review
myk1yt
force-pushed
the
pr/b16-stats-ui-v2
branch
2 times, most recently
from
August 4, 2026 20:41
f398395 to
2fe7eb2
Compare
github-actions
Bot
added
has-conflicts
myk1yt
force-pushed
the
pr/b16-stats-ui-v2
branch
2 times, most recently
from
August 5, 2026 09:24
307a751 to
6ba1f75
Compare
github-actions
Bot
removed
the
awaiting-review
myk1yt
force-pushed
the
pr/b16-stats-ui-v2
branch
2 times, most recently
from
August 6, 2026 07:28
66c6878 to
79d9135
Compare
github-actions
Bot
added
has-conflicts
…cit-any Add new test file to eslint-suppressions.json with count of 26 no-explicit-any suppressions. These are standard test patterns (mock objects, private property access via 'as any') consistent with other test files in the suppressions list. Fixes CI lint failure in PR #25 compile (lint) job.
github-actions
Bot
added
awaiting-review
github-actions
Bot
removed
the
awaiting-review
added 3 commits
August 7, 2026 16:21
… coverage - Add coverage tests for UsageStatsService, UsageStatsStreamCoordinator, UsageStatsMigration, UsageStatsProjection, DashboardTaskProjection, UsageAggregator, costRecalculation, and UsageRecorder. - Add new safeWriteJson spec to cover rollback and failure paths. - Add dashboard webview tests for TaskList, AnimatedNumber, SessionDetail, dashboardStreamReducer, and useDashboardStatsStream edge cases. - Fix vscode mock so RelativePattern is constructible for file watcher tests.
github-actions
Bot
added
the
awaiting-review
github-actions
Bot
removed
the
awaiting-review
added 2 commits
August 8, 2026 11:55
- playwright-ct.config.ts: alias '@/i18n/TranslationContext' to the CT mock. DashboardSummary and UsageHeatmap import useAppTranslation via '@/i18n/...' which bypassed the existing '@src/i18n/...' alias, pulling the real TranslationContext -> ExtensionStateContext -> @roo-code/types barrel (zod) into the CT bundle and throwing 'ReferenceError: z is not defined' at mount. Fixes all 3 webview visual snapshot failures (bundle error, not pixel mismatch). - useDashboardStatsStream.spec.tsx: add coverage for the 10s loading timeout ERROR dispatch and the requestTaskPage no-cursor guard to close the patch-coverage shortfall.
github-actions
Bot
added
the
awaiting-review
added 2 commits
August 8, 2026 13:03
…compatibility Root cause: DashboardSummary and UsageHeatmap components use StandardTooltip which requires a Radix TooltipProvider in the component tree. Playwright CT mounts components without the app's provider hierarchy, causing silent render failures (elements not found). Additionally, the ProviderBreakdownFixture was defined inline in the test file, which Playwright CT cannot mount (requires test story or external fixture). Fix: Extract all fixtures into StatsPanel.visual.fixture.tsx with proper TranslationContext.Provider and TooltipProvider wrappers. Run: https://github.com/Zoo-Code-Org/Zoo-Code/actions/runs/31237627413
…ov coverage gap - Move data-testid in ProviderBreakdownFixture from outer div to table (Playwright CT cannot find data-testid on root-level mount element) - Add CI-generated baseline screenshots for StatsPanel visual tests - Add 5 coverage tests for handleRebuildUsageStats and handleGetDashboardTaskDetail error branches (79.86% -> 80%+)
github-actions
Bot
added
has-conflicts
github-actions
Bot
added
awaiting-review
- mimo.spec.ts: replace manual 'any[]' stream-collection loop with the typed collectStream() helper. The merge in b4c997b introduced an extra 'any[]' that pushed the file to 19 no-explicit-any violations, exceeding the eslint-suppressions budget of 18 and failing the compile job. - UsageStatsMigration.spec.ts: reduce checkpoint-resume test from 2500 to 1200 events (still crosses the 1000-event batch boundary). The 2500-event run exceeded the 60s test timeout on windows-latest under parallel load; the per-event db.prepare() hot path makes SQLite writes the dominant cost.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stack Position
feature/local-usage-statsDescription
https://www.youtube.com/shorts/UHnnOCM1_f0
Full Feature Description
feature/local-usage-statsusage-stats.ts,src/services/stats, the provider/task capture pathsTask.ts, the stats IPCusageStatsMessageHandler.ts, and the UIDashboardView.tsxanduseDashboardStatsStream.ts.Why Split Into 17 PRs
Instead of submitting this feature as a single unified PR, it was split into individual PRs because as code size grows, safely reviewing a PR becomes very difficult. The feature was broken into mutually exclusive individual PRs so that each can be reviewed independently.
What This PR Specifically Changes
Adds SQLite projection, transactional/idempotent migration, local-day rollup, rebuild/query/stream IPC, epoch guard, dashboard summary/session/heatmap/loading/retry UI, and visual coverage. Removes other feature files.
Included Files
src/services/stats/UsageStatsDatabase.tssrc/services/stats/UsageStatsMigration.tssrc/services/stats/UsageStatsProjection.tssrc/services/stats/UsageStatsStreamCoordinator.tssrc/core/webview/usageStatsMessageHandler.tswebview-ui/src/components/dashboard/DashboardView.tsxwebview-ui/src/components/dashboard/useDashboardStatsStream.tswebview-ui/src/components/dashboard/DashboardView.visual.tsxExclusion Scope