[pratikbuilds]

Summary

• add the Corbit SVG avatar for agent identities and deterministic pastel initials for people without an image
• centralize avatar palette, fallback, SVG, and public types in packages/chat-ui/src/avatar.tsx
• use Tailwind utilities for avatar presentation and a color-only agent picker

Validation

• bun test packages/chat-ui/src/avatar.test.tsx packages/chat-ui/test/avatar-composition.test.tsx
• bunx tsc --noEmit -p packages/chat-ui/tsconfig.json
• bun run --cwd apps/web build
• bun run check is blocked before lint/tests by pre-existing vendored @intx type mismatches in the hub and sidecar

Follow-up CL-7478 moves the pastel palette into react-ui tokens and removes the DESIGN.md exception added here.

[TheGreatAxios]

@pratikbuilds csn you drop some screenshots here pls?

[pratikbuilds]

yes doing it now

[pratikbuilds]

[TheGreatAxios]

Looks good @pratikbuilds , rebase to main as needed. Please see what's going on re why it's not passing CI and make sure to open into full PR (non draft) once ready

[TheGreatAxios]

• Make sure it passes CI please
• Will need to think about where this avatar code should best live for Corbits but this is a great start

[TheGreatAxios]

Self-review

Verdict: implementation is sound — the Corbit avatar is an inline SVG (no binary asset, no license question), the pastel fallback color and its readable-text pairing are pure functions with deterministic-by-construction tests, and the human/agent split is consistently threaded through chat-ui, settings-ui, and apps/web. No code changes were needed.

Findings:

• CONFIRMED-OPEN: resolveAvatarFill's image branch renders via a plain <img> in packages/settings-ui/src/account-section.tsx with no onerror/reportError path if the stored avatar URL 404s or fails to load. Not introduced by this PR (pre-existing <img>, only its className line changed here), so left as-is rather than widening this PR's scope — worth its own ticket.
• DISMISSED: avatar assets are hand-authored inline SVG paths/colors in packages/chat-ui/src/avatar.tsx, not an imported binary — no license provenance concern.
• DISMISSED: avatarColorForPrincipal/generatedAvatarStyle/readableTextOn are pure (string hash → palette index → luminance check), covered by determinism/stability tests in src/avatar.test.tsx and test/avatar-composition.test.tsx across repeated calls and multiple principals.
• DISMISSED: packages/inbox/test/bulk-report-error.test.ts mock update is a rebase fix (error-sink's mock needed makeErrorEnvelope/parseErrorEnvelope after a since-merged main-branch change), not scope creep.
• DISMISSED: CorbitAvatar living in packages/chat-ui rather than corbitsdev/react-ui — it's Corbit-specific agent identity, not a generic primitive, so package-level ownership is defensible; apps/web only composes it.
• DISMISSED: no bare catch {}, no new as T on untrusted input, no exactOptionalPropertyTypes hazard in the diff.

Tests run (from packages/chat-ui):

• bun test src/avatar.test.tsx test/avatar-composition.test.tsx test/chat-workspace.test.tsx test/team-avatar-stack.test.tsx src/chat-workspace.test.ts — 88 pass, 0 fail
• bunx tsc --noEmit -p packages/chat-ui/tsconfig.json — clean
• bunx tsc --noEmit -p apps/web/tsconfig.json — clean