Skip to content

feat(chat-ui): resolve avatar identity through theme tokens - #715

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-7478-avatar-palette-tokens
Sep 15, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-7478-avatar-palette-tokens

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Moves the avatar pastel palette from hardcoded hex into --avatar-1..4 theme tokens (defined once in chat-ui's stylesheet :root as the proposed upstream react-ui contract) and drops the DESIGN.md color-exception section.

Consumers (class map, CorbitAvatar field, create-agent swatches) reference tokens; no hex remains in the identity path. UI-only; no chat/mailbox/provisioner coupling.

Verification:

  • New token-contract tests fail on the old implementation (4 fail, e.g. field rendered literal fill="--avatar-3") and pass with it (avatar.test.tsx 15/15; chat-ui suite 909 pass, 0 fail)
  • chat-ui + apps/web typecheck clean; eslint + prettier clean on touched files
  • Tailwind v4 compile check: bg-(--avatar-N) emits var(--avatar-N)
  • Pre-push hook trips on pre-existing network-dependent failures in @corbits/seeding (createGitWorkflowPusher, 5s timeouts) and @workbench/hub (worktree/provider-env boot) — untouched by this diff, pushed --no-verify per lane brief

Fixes CL-7478

@linear-code

linear-code Bot commented Sep 15, 2026

Copy link
Copy Markdown

CL-7478

@TheGreatAxios
TheGreatAxios force-pushed the cl-7478-avatar-palette-tokens branch 3 times, most recently from c1749cb to 5aa1864 Compare September 15, 2026 18:37
Avatar pastels move from hardcoded hex to --avatar-1..4 tokens defined
once in chat-ui's stylesheet :root — the proposed upstream contract for
react-ui's theme. Class map, CorbitAvatar field, and the create-agent
swatches all consume the tokens by reference; DESIGN.md's color-exception
section now describes the token rule. No hex remains in the identity path.

Fixes CL-7478
@TheGreatAxios
TheGreatAxios force-pushed the cl-7478-avatar-palette-tokens branch from 5aa1864 to 37e9eb4 Compare September 15, 2026 19:05
@TheGreatAxios
TheGreatAxios merged commit ae4580f into main Sep 15, 2026
24 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-7478-avatar-palette-tokens branch September 15, 2026 19:48
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