Repository navigation
feat(webkit): [ENG-48075] drop templates exports; Templates stories become compositions - #977
Merged
Merged
Conversation
…ecome compositions
isaque-bock-azion
force-pushed
the
feat/remove-templates-category
branch
from
September 17, 2026 18:46
013db0d to
2ee91d3
Compare
isaque-bock-azion
changed the title
lucas-langeloh-azion
approved these changes
Sep 17, 2026
guilherme-santana-azion
approved these changes
Sep 17, 2026
rafael-garbinatto-azion
approved these changes
Sep 17, 2026
Merged
gabriel-lisboa-azion
added a commit
that referenced
this pull request
Sep 21, 2026
demo/vue-sample had drifted 540 ahead / 31 behind main while most of its webkit work was reviewed and landed upstream (#941 command-menu, #942 GlobalHeader placements, #943 NavigationMenu, #952 Menu, #960 sidebar collapse, #973 floating-panel placement, #975 paginator, #977 templates exports, #921 amount ladder), after which main swept comments (#951) and reworked the CLI onto a derived bundle (#968/#970). Main is the source for everything the two share. Of the 28 conflicted source files, 17 differed from main only in comments — their code was byte-identical — and 6 more were genuinely newer on main, so all 23 take main verbatim, including the use-placement TypeScript rewrite that replaces the old index.js. Three carve-outs keep demo work main does not have yet: - sidebar: main's CSS-driven collapse is the base; the `side` axis is re-applied on top of it as data-[side=end] variants rather than the JS edge map it used to be, and the drag sign moves into the rail composable. The one test that asserted the removed inline transform now asserts the panel's published edge, since the direction is CSS the unstyled test env cannot see. - theme: texts.data.js keeps demo's 13 extra text-wrap and 10 extra tracking declarations (PR #982, in flight). Both sides carry the same 34 entries, so an entry count would not have caught the loss. - doc-prose keeps its chrome-padding fix (PR #981, in flight). catalog.json, authoring-baseline.json, the theme dist and the lockfile are regenerated rather than hand-resolved; the 13 conflicted visual baselines take main's and are regenerated from CI next. Verified: every file main touched is byte-identical to main except the 22 accounted for above; 2364 browser tests, 179 toolkit tests, lint, vue-tsc, 99.98% type coverage, both ratchets, the Storybook build and the webkit-sample build all pass.
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
4 participants
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
templatesexports from@aziontech/webkit(deploy-success,onboarding-form,plan-success,platform-shell,sign-up-card) and regeneratescatalog.json; the sources undersrc/components/templates/and their specs/tests stay in the repo.Templates/*stories are rewritten as compositions of published components (GlobalHeader,CardBox,Item,LogView,Tag,Link,TabView,Button, …) with a runnable "Show code" SFC; the inheritedTag :valuebug becomeslabel.packages/webkit/AGENTS.mdandapps/storybook/README.mddescribe Templates as pure compositions.How to test
node .claude/hooks/validate-story-source.mjs --all→OK: 111 story files comply.pnpm --filter @aziontech/webkit catalog:check→ no diff;grep -c "templates/" packages/webkit/catalog.json→0.pnpm storybook:dev→ openTemplates/DeploySuccess,PlanSuccess,OnboardingForm,PlatformShell,SignUpCard: each renders and its Docs "Show code" panel imports only@aziontech/webkit/*subpaths that exist inpackage.json#exports.pnpm --filter @aziontech/webkit test:gate,pnpm webkit:lint,pnpm webkit:type-checkpass.Notes
@aziontech/webkit/{deploy-success,onboarding-form,plan-success,platform-shell,sign-up-card}. Consumers copy the markup from the matchingTemplates/*story. Shipped as a minor by the author's decision.