Uh oh!
There was an error while loading. Please reload this page.
chore(plugin-report): remove dead useReportData aggregation pipeline - #1654
Merged
Conversation
ADR-0021 moved report aggregation into the semantic layer (queryDataset), so the pre-9.0 CLIENT-SIDE pipeline — the useReportData hook plus buildAggregateQuery / groupAndAggregate / pivotRows / aggregateRows / collectFields / columnKey / bucketDate / groupingValue and their result types — had no remaining consumers (verified zero imports across objectui and all five product repos). Delete the 840-line hook + its test. The one symbol still in use, mergeFilters (the dataset renderer's scope-filter combinator), moves to its own ./mergeFilters module and stays exported from the package root. DatasetReportRenderer's import is repointed; the barrel drops the dead exports. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The latest updates on your projects. Learn more about Vercel for GitHub. |
Contributor
✅ Console Performance Budget
📦 Bundle Size Report
Size Limits
|
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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.
Summary
ADR-0021 moved report aggregation into the semantic layer (
queryDataset), so the pre-9.0 client-side aggregation pipeline is dead code. Scan (task 3) confirmed zero consumers of these exports across objectui and all five product repos (cloud / mobile / templates / hotcrm / objectos):Removed:
useReportDatahook (840-line file) + its testbuildAggregateQuery,groupAndAggregate,pivotRows,aggregateRows,collectFields,columnKey,bucketDate,groupingValueReportRow,PivotMatrix,PivotHeader,UseReportDataResult,UseReportDataOptionsKept:
mergeFilters— the only symbol still used (byDatasetReportRendereras its scope-filter combinator) — moved to its own./mergeFiltersmodule and still exported from the package root. The renderer's import is repointed; the barrel drops the dead exports.This closes the ADR-0021 report migration: dataset-bound rendering, native editor, matrix cross-tab + drill, and now removal of the transitional client-aggregation path.
Test plan
pnpm vitest run— 3518 passed, 24 skipped, 0 failed (−40 from the removed hook's tests)pnpm type-check— 58/58pnpm build— 41/41🤖 Generated with Claude Code