feat(ui): BarList distribution primitive for the B2 batch (U5 B2)#441
Conversation
The Merge Analytics mockup's "distribution" panels (PR size, reviewer turnaround) and Model Usage's per-model breakdown are horizontal bar lists, not line charts — the missing primitive for the batch. Adds: - BarList: one labeled row per category with a proportional, tone- colored bar and a trailing value. Ideal for Record<string, number> breakdowns (PR sizes, per-model tokens/cost, error patterns); the bar is decorative, the label + value are real text. - charts.ts: barPercent(value, scale) — value's clamped 0–100 share, 0 for non-positive value/scale; exported and unit-tested from the app suite (9 chart tests total). No BarChart (vertical/categorical bars over invented data) is added — only this real, high-reuse distribution list. Verified in Storybook (3 stories) light + dark. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 12 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
1 participant
Summary
A small primitive slice that fills the last gap for the B2 batch. The Merge Analytics mockup's distribution panels (PR size, reviewer turnaround) and Model Usage's per-model breakdown are horizontal bar lists, not line charts — so this adds the missing primitive.
New primitive (libs/ui)
BarList— one labeled row per category with a proportional, tone-colored bar and a trailing value. Ideal forRecord<string, number>breakdowns (PR sizes, per-model tokens/cost, error patterns). The bar is decorative (aria-hidden); the label and value are real text, so no extra ARIA is needed.charts.ts:barPercent(value, scale)— a value's clamped 0–100 share, returning 0 for non-positive value/scale — exported and unit-tested from the app suite (9 chart tests total).What this deliberately does not add
No vertical/categorical
BarChartover invented data — only this real, high-reuse distribution list. It unlocks Merge Analytics (its main content), Model Usage's model breakdown, and the deferred Analytics/Productivity drilldowns (error_patterns,common_issues,agent_stats).Verification
barPercent);libs/ui+ Electron tsc clean; Storybook rebuilt (3 stories), BarList verified against the mockup in light + dark (proportional bars, value labels, single-row).Next in B2
Merge Analytics composes
AnalyticsDashboard+BarList; then Model Usage.🤖 Generated with Claude Code