[front] feat: wire Sources tab of the usage filter panel to real, period-scoped data - #30177
Open
avervaet wants to merge 1 commit into
Conversation
Contributor
Author
This was referenced Aug 7, 2026
Open
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-skills-tab
branch
from
August 7, 2026 11:03
29bb1bd to
0d816bc
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-sources-tab
branch
from
August 7, 2026 11:03
d48a735 to
b5849e2
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-skills-tab
branch
from
August 7, 2026 13:56
0d816bc to
dbe1a6d
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-sources-tab
branch
from
August 7, 2026 13:56
b5849e2 to
04d9774
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-skills-tab
branch
from
August 7, 2026 15:22
dbe1a6d to
859e89f
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-sources-tab
branch
from
August 7, 2026 15:22
04d9774 to
2e50141
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-skills-tab
branch
from
August 7, 2026 15:39
859e89f to
e0e3e5f
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-sources-tab
branch
from
August 7, 2026 15:39
2e50141 to
b415e36
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-skills-tab
branch
from
August 7, 2026 16:02
e0e3e5f to
3c9f7ce
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-sources-tab
branch
from
August 7, 2026 16:02
b415e36 to
6c5f164
Compare
…iod-scoped data Sources now come from the same period-ranked consumption data the Attribution table uses (useConsumptionTop), instead of the mock connector list. The "source" dimension is the message's origin channel (Slack, API, Web, CLI, ...), not a data-source connector, so the connector-logo concept from the mock is dropped along with it — the source entity icon now falls back to none, like tool and skill. Wires the page's UsageFilter state into the actual consumption queries for the "source" dimension, which it never did before. This was the last mock category, so usageFilterMockData.ts and the categoryEntities prop are removed entirely.
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-skills-tab
branch
from
August 7, 2026 16:44
3c9f7ce to
caa798d
Compare
avervaet
force-pushed
the
avervaet/wiring-usage-filter-component-sources-tab
branch
from
August 7, 2026 16:44
6c5f164 to
ba4e8c8
Compare
| [topSourceRows] | ||
| ); | ||
|
|
||
| const resolvedCategoryOptions = useMemo<{ |
Contributor
There was a problem hiding this comment.
React Doctor · react-doctor/no-usememo-simple-expression (warning)
This useMemo rebuilds a tiny literal whose reference is never relied on, so remove the useMemo and build the value inline
Fix → Remove the useMemo. Property reads, math, and ternaries are already fast, so wrapping them doesn't help
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
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.
Sources now come from the same period-ranked consumption data the
Attribution table uses (useConsumptionTop), instead of the mock
connector list. The "source" dimension is the message's origin
channel (Slack, API, Web, CLI, ...), not a data-source connector, so
the connector-logo concept from the mock is dropped along with it —
the source entity icon now falls back to none, like tool and skill.
Wires the page's UsageFilter state into the actual consumption
queries for the "source" dimension, which it never did before. This
was the last mock category, so usageFilterMockData.ts and the
categoryEntities prop are removed entirely.
Description
Tests
Risk
Deploy Plan