Skip to content

🧩 refactor: Shared UI Design System Tokens - #14670

Merged
danny-avila merged 2 commits into
devfrom
codex/composer-design-system
Aug 7, 2026
Merged

danny-avila merged 2 commits into
devfrom
codex/composer-design-system

Conversation

@danny-avila

@danny-avila danny-avila commented Aug 6, 2026 •

Copy link
Copy Markdown
Collaborator

I added a conflict-safe shared design-system foundation on top of the merged #13879 migration, preparing LibreChat for the #14473 composer redesign without modifying the legacy composer input files.

  • Add reusable IconButton and Chip primitives to @librechat/client.
  • Add semantic overlay, strong-status, and status-text tokens across runtime themes, Tailwind, CSS defaults, and theme documentation.
  • Refactor shared Tag, Toast, dialogs, search animation, and Button styling to semantic tokens.
  • Fix the invalid nested-button structure in Badge and add an accessibility regression test.
  • Add component tests, theme mapping tests, and a focused guardrail against direct palette utilities and hex colors in shared primitives.

Change Type

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Testing

I ran the full @librechat/client Jest suite: 18 suites and 168 tests passed. TypeScript, Prettier, git diff --check, and Tailwind semantic-color loading also passed.

The normal package build is currently blocked before bundling by the existing local Node 24 rimraf/brace-expansion dependency mismatch.

Test Configuration:

  • Node.js 24.16.0
  • Jest 30.2.0
  • TypeScript 5.9.3

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in complex areas of my code where needed
  • I have made pertinent documentation changes
  • My changes do not introduce new warnings
  • I have written tests demonstrating that my feature works
  • Local unit tests pass with my changes
  • Any changes dependent on mine have been merged and published in downstream modules.

Copilot AI lite review requested due to automatic review settings August 6, 2026 16:43
@danny-avila
danny-avila marked this pull request as ready for review August 6, 2026 16:44
@danny-avila danny-avila changed the title 🧩 feat: Harden Shared Design System Tokens 🧩 refactor: Shared UI Design System Tokens Aug 6, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@danny-avila

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ddcf77e477

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/client/src/theme/types/index.ts
Comment thread packages/client/src/components/Badge.tsx
@danny-avila

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: 3598fa5b19

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@danny-avila
danny-avila merged commit bd5c1ad into dev Aug 7, 2026
38 checks passed
@danny-avila
danny-avila deleted the codex/composer-design-system branch August 7, 2026 02:13
danny-avila added a commit that referenced this pull request Aug 7, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
berry-13 pushed a commit that referenced this pull request Aug 7, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
berry-13 pushed a commit that referenced this pull request Aug 9, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
berry-13 pushed a commit that referenced this pull request Aug 9, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
berry-13 pushed a commit that referenced this pull request Aug 11, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
berry-13 pushed a commit that referenced this pull request Aug 11, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
berry-13 pushed a commit that referenced this pull request Aug 13, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
berry-13 pushed a commit that referenced this pull request Aug 23, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
LogicalAbsurd pushed a commit to LogicalAbsurd/LibreChat that referenced this pull request Aug 27, 2026
* feat: harden shared design system tokens

* fix: address design system review findings
berry-13 pushed a commit that referenced this pull request Sep 1, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
berry-13 pushed a commit that referenced this pull request Sep 6, 2026
* 🧩 refactor: Shared UI Design System Tokens (#14670)

* feat: harden shared design system tokens

* fix: address design system review findings

* refactor: harden composer design system

* test: scan nested composer modules
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.

2 participants