Skip to content

Feat/merge UI library#137

Open
HagerDakroury wants to merge 10 commits intomainfrom
feat/merge-ui-library
Open

Feat/merge UI library#137
HagerDakroury wants to merge 10 commits intomainfrom
feat/merge-ui-library

Conversation

@HagerDakroury
Copy link
Contributor

@HagerDakroury HagerDakroury commented Feb 23, 2026

Merged https://github.com/uselagoon/ui-library.

The package currently lives under src/ui-library
Screenshot 2026-02-23 at 8 38 14 PM

Add @ui-lib/* path alias to tsconfig.json pointing to src/ui-library/*.
Replace all @/ imports within src/ui-library/ to @ui-lib/ to avoid
conflicts with the app's existing @/ alias.
Move all CSS variables, theme tokens, and base layer rules from
ui-library's index.css into the app's globals.css. Remove the
@uselagoon/ui-library CSS import and delete the now-redundant
index.css. App-specific :root overrides (--background, --foreground)
are preserved in @layer base so they take precedence.
Replace all imports from the external @uselagoon/ui-library package
with local path aliases:
- @uselagoon/ui-library/schemas -> @/ui-library/schemas
- @uselagoon/ui-library/dist/... -> @/ui-library/...
- @uselagoon/ui-library -> @/ui-library
Add all dependencies that ui-library requires (radix-ui primitives,
tanstack/react-table, class-variance-authority, shadcn utilities,
recharts, zod, etc.) to lagoon-beta-ui's package.json.
Remove the external @uselagoon/ui-library dependency since its source
is now merged locally. Remove the uilib:update script and strip the
yarn uilib:update prefix from dev and build scripts.
Remove the external Chromatic refs block that pointed to the
ui-library's separate Storybook. The stories glob already covers
src/ui-library/stories/ so all ui-library stories are now included
directly.
The initial bulk replacement only caught single-quoted imports.
This fixes the remaining double-quoted @/ imports in ChangeFeed
components and ActionButton story.
@CGoodwin90 CGoodwin90 self-requested a review February 25, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant