Uh oh!
There was an error while loading. Please reload this page.
feat(spec)!: prune dead ReportColumnSchema/ReportGroupingSchema exports + unread report chart groupBy (#3463) - #3488
Merged
Conversation
…ts + unread report chart groupBy (#3463) Deep-cleanup close-out of the report-chart disposition (follow-up to #3441, umbrella #1878/#1890). After the ADR-0021 single-form cutover a dataset-bound report expresses columns/grouping as dataset measure/dimension name arrays (`values`/`rows`/`columns` = `z.array(z.string())`), so ReportColumnSchema / ReportGroupingSchema were referenced by no schema body — they survived only as public type exports and were marked @deprecated in #3441. - Remove ReportColumnSchema / ReportGroupingSchema and their type exports (ReportColumn / ReportGrouping / ReportColumnInput / ReportGroupingInput) from @objectstack/spec/ui; drop the now-orphan ResponsiveConfigSchema import. - Remove ReportChart.groupBy — the [EXPERIMENTAL — not enforced] series-split field from #3441. The dataset-bound DatasetReportRenderer plots a single xAxis×yAxis series and never read it; only the retired legacy ReportViewer fallback consumed a top-level groupBy. ReportChartSchema is non-strict, so residual chart.groupBy in stored metadata is silently stripped on parse. - Drop manifest ratchet keys ui/ReportColumn, ui/ReportGrouping; regenerate content/docs/references/ui/report.mdx and the spec API-surface snapshot. No first-party or example report authored these objects or chart.groupBy. Ships as minor per the launch-window breaking-as-minor policy; changeset carries the FROM→TO migration. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
📓 Docs Drift CheckThis PR changes 1 package(s): 104 hand-written doc(s) reference the affected code and may need an implementation-accuracy re-verification:
|
Uh oh!
There was an error while loading. Please reload this page.
This was referenced Jul 25, 2026
os-zhuang added a commit
that referenced
this pull request
Jul 27, 2026
…drop re-exports — ADR-0076 Step 2 PR-C (#2462) (#3591) - assembleMetadataProtocol() in metadata-protocol is now the ONE assembly (objects + protocol shim + D12 degraded analytics); createMetadataProtocolPlugin (cloud/delegated) and ObjectQLPlugin's registerProtocol convenience mode both mount it (~112 inline lines deleted from the engine plugin; the shim return feeds the engine's mutation-rebind subscription synchronously). - objectql's six protocol re-exports removed (breaking-as-minor per launch-window, #3486/#3488 precedent). Five known importers repointed (rest x3 tests, runtime seed-loader re-export + its integration test); runtime gains the direct metadata-protocol dep. - Recipe correction (recorded on #2462): the objectql→metadata-protocol DEPENDENCY stays — ./core was already protocol-free, and forcing ~20 framework boot sites to mount two plugins is a pure boilerplate tax. Step 2's goal lands as "zero assembly ownership, single source". Verified: full build; objectql 1086, rest 380, runtime 653, dogfood 60/61 in parallel (semantic-roles 10s-hook flake passes solo). Co-authored-by: Jack Zhuang <277994282+os-zhuang@users.noreply.github.com> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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
1 participant
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.
Closes#3463 (item 1 — the spec-side prune).
Context
Deep-cleanup close-out of the report-chart disposition (follow-up to #3441, umbrella #1878/#1890). After the ADR-0021 single-form cutover a dataset-bound report expresses columns/grouping as dataset measure/dimension name arrays (
values/rows/columns=z.array(z.string())), soReportColumnSchema/ReportGroupingSchemawere referenced by no schema body — they survived only as public type exports and were marked@deprecatedin #3441.What changed
ReportColumnSchema/ReportGroupingSchemaand their type exports (ReportColumn/ReportGrouping/ReportColumnInput/ReportGroupingInput) from@objectstack/spec/ui; dropped the now-orphanResponsiveConfigSchemaimport.ReportChart.groupBy— the[EXPERIMENTAL — not enforced]series-split field flagged in docs(spec): correct ReportChart xAxis/yAxis semantics; mark dead report surface (#1890) #3441. The dataset-boundDatasetReportRendererplots a singlexAxis×yAxisseries and never read it; only the retired legacyReportViewerfallback consumed a top-levelgroupBy.ReportChartSchemais non-strict, so residualchart.groupByin stored metadata is silently stripped on parse — no tombstone needed.ui/ReportColumn/ui/ReportGrouping; regeneratedcontent/docs/references/ui/report.mdxand the spec API-surface snapshot.Breaking → ships as
minorper the launch-window breaking-as-minor policy (precedent:remove-enable-trash-mru). The changeset carries the FROM→TO migration.Migration
Nothing an author writes changes — no first-party or example report authored
ReportColumn/ReportGroupingobjects orchart.groupBy. TypeScript consumers importing the removed types from@objectstack/spec/uihave no replacement: model report columns as the dataset's measure names and grouping as its dimension names. The objectuiSpecReportColumn*/SpecReportGrouping*re-exports are removed in the companion PR (objectstack-ai/objectui#chore/3463-retire-reportviewer-fallback).Verification
@objectstack/specbuild + 6857 tests pass;tsc --noEmitclean.check:api-surface(6 exports removed, snapshot regenerated) ✓check:docs(report.mdx regenerated, in sync) ✓check:skill-refs/check:react-blocksin sync ✓tsc --noEmitclean (the spec-break guard) ✓turbo buildacross the cli dependency closure clean.🤖 Generated with Claude Code