Regenerate Admin component definitions and docs - #4689
Conversation
The components.d.ts aggregate had been generated for a subset of components and then patched by hand, so it had drifted from the components as shipped: AppNav, EmptyState, Number, PressButton and ScrollBox were absent entirely, and internal generator scaffolding (AddedContext, BaseClass, ContextRequestEvent) had leaked in where the shared PolarisCustomElement base class belongs. Regenerated for the full component set, then rebuilt the 2026-10-rc docs payload. Definitions move from Polaris 2.23.0 to 2.26.0: shared.d.ts picks up ProgressProps and generalises the field documentation inherited by Checkbox, Choice, ChoiceList, Option and Switch, and Section documents an Icon in its graphic slot and Tooltip among its accessories. yarn lint and yarn type-check pass.
763912e to
e54c7cc
Compare
kyledurand
left a comment
There was a problem hiding this comment.
Looks like nothing changed except for some wording and the addition of progress component 👍
|
Heads up before this merges — the three The cause: every Admin component's In the base file these didn't exist (Avatar had 4 real members: The Technical detail for a fix (AI-friendly)Symptom
Root cause
Fix direction
How to verify the corrected payload
posted by an AI agent on Jason's behalf |
jas7457
left a comment
There was a problem hiding this comment.
Requesting changes — the regenerated generated_docs_data_v2.json payloads have an unintended regression: each Admin component's members list now includes the full native DOM surface (HTMLElement/Element/Node) as public members (~320 bogus members × 66 components), ballooning each file from ~1.48 MB to ~10.5 MB. This looks like base-class DOM-flatten guard drift during regeneration, not intended output. Details and a fix/verification path in my inline comment: #4689 (comment)
The .d.ts changes and the intended additions (Number, EmptyState, AppNav, Progress, wording, Section Icon/Tooltip) look good — just the docs JSON needs a clean regen.
posted by an AI agent on Jason's behalf
Background
The Admin
components.d.tsaggregate had been generated for a subset of components and then patched by hand, so it had drifted from the components as shipped:AppNav,EmptyState,Number,PressButtonandScrollBoxwere missing entirely, and internal generator scaffolding had leaked in.Solution
Regenerated the definitions for the full component set, with no component filter, and rebuilt the
2026-10-rcdocs payload. Definitions move from Polaris 2.23.0 to 2.26.0:shared.d.tsgainsProgressPropsand generalises the field documentation inherited byCheckbox,Choice,ChoiceList,OptionandSwitch;Sectiondocuments anIconin its graphic slot andTooltipamong its accessories.Everything here is generated output.
🎩
yarn lintandyarn type-checkpass.Checklist