feat: hooks dashboard chart improvements#2281
Conversation
🦋 Changeset detectedLatest commit: 860292a The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🚀 Preview Environment (PR #2281)Preview URL: https://pr-2281.dev.getgram.ai
Gram Preview Bot |
alx-xo
changed the title
alx-xo
force-pushed
the
alexm/age-1842-improve-hooks-dashboard
branch
from
aa9a1c3 to
7af17c5
Compare
alx-xo
force-pushed
the
alexm/age-1842-improve-hooks-dashboard
branch
from
7af17c5 to
c8746d5
Compare
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
simplesagar
left a comment
simplesagar
left a comment
There was a problem hiding this comment.
Looks great! I think we should use this opportunity to rename the dashboard from hooks to something that represents the value it provides.
As a followup i'd consider moving the hooks setup to a top level setup experience for the organisation or as an onboarding wizard within a project.
|
changeset would be a good idea for this one since we want to broadcast it to customers |
chase-crumbaugh
left a comment
chase-crumbaugh
left a comment
There was a problem hiding this comment.
Good incremental progress
alx-xo
force-pushed
the
alexm/age-1842-improve-hooks-dashboard
branch
from
c8746d5 to
7b0f1c7
Compare
Simplify SHARED_LEGEND to { display: false } so every chart using
StackedBarChart or MultiLineChart stops rendering its legend —
legends became unreadable past ~10 series and tooltips already
carry the needed detail.
Delete the three charts that were dropped from the hooks admin
view during the working session with Chase:
- Source Usage per User (UserVolumeList)
- Source Usage per MCP Server (SourceVolumeChart)
- Errors per User (UserErrorChart)
Rows are now servers stacked by user, inverting the prior "Server Usage per User" view. With legends removed, admins get more value seeing which users drive each server than the reverse — server names usually fit in the small dataset set, while user emails were cluttering row labels.
Add UserEventCountsChart — one solid horizontal bar per user summing eventCount from the breakdown. Restructure the hooks page charts into a single 2-column grid: Server Usage (time series) | Users per Server (bar) User Usage (time series) | User Event Counts (bar) Errors per Server and Tool | (empty) Each column pairs a time-series with a bar chart so admins can read trend and distribution side by side. Row 3 is intentionally half-width — errors-over-time is a follow-up (requires failureCount on HooksTimeSeriesPoint).
alx-xo
force-pushed
the
alexm/age-1842-improve-hooks-dashboard
branch
from
e2d94fb to
d3dc75d
Compare
alx-xo
force-pushed
the
alexm/age-1842-improve-hooks-dashboard
branch
from
72fde84 to
6617d6a
Compare
…suals, and smarter default filters.
5 participants
Summary:
ErrorsOverTimeChart
afterBody), while keeping a single aggregated line in the chart.Bar Chart Styling
SHARED_BAR_SCALESfor consistent right-aligned Y-axis labels and 12px font size across all bar charts.USER_SOURCE_COLORSto the Tailwind x-400 range.BRAND_RED_COLORSretained for error intensity." errors"suffix fromServerErrorRateCharttooltip labels.Axis Label Truncation
localpart@…instead of the full address. Full label still shown in tooltips.Hook Type Filter Default
Local Toolsis now opt-in — default selection isMCP ServersandSkillsonly.&(e.g.Showing Servers & Skills).Visual