Skip to content

fix(desktop): show default profile in switcher#48596

Open
tt-a1i wants to merge 1 commit into
NousResearch:mainfrom
tt-a1i:codex/desktop-default-profile-toggle
Open

fix(desktop): show default profile in switcher#48596
tt-a1i wants to merge 1 commit into
NousResearch:mainfrom
tt-a1i:codex/desktop-default-profile-toggle

Conversation

@tt-a1i

@tt-a1i tt-a1i commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Fixes #48583

Summary

  • split the desktop profile rail's All profiles aggregate from the default/root profile entry
  • show the default profile name as a compact visible pill while preserving the existing named-profile squares
  • add regression coverage for default visibility and mutually exclusive All/default active states

Test plan

  • npm run test:ui --workspace apps/desktop -- src/app/chat/sidebar/profile-switcher.test.tsx
  • npm run typecheck --workspace apps/desktop
  • cd apps/desktop && npx eslint src/app/chat/sidebar/profile-switcher.tsx src/app/chat/sidebar/profile-switcher.test.tsx
  • npx prettier --check apps/desktop/src/app/chat/sidebar/profile-switcher.tsx apps/desktop/src/app/chat/sidebar/profile-switcher.test.tsx
  • git diff --check

Note: full desktop lint/test were also sampled, but currently fail on existing unrelated files/environment setup (for example missing jsdom localStorage, node:test .cjs suites under Vitest, and unrelated lint errors).

@alt-glitch alt-glitch added type/bug Something isn't working comp/tui Terminal UI (ui-tui/ + tui_gateway/) P3 Low — cosmetic, nice to have labels Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tui Terminal UI (ui-tui/ + tui_gateway/) P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

desktop: default profile is visually hidden behind all-profiles toggle

2 participants