Skip to content

fix(desktop): retry profile model refresh - #47626

Closed
tt-a1i wants to merge 1 commit into
NousResearch:mainfrom
tt-a1i:codex/desktop-model-pill-refresh
Closed

fix(desktop): retry profile model refresh#47626
tt-a1i wants to merge 1 commit into
NousResearch:mainfrom
tt-a1i:codex/desktop-model-pill-refresh

Conversation

@tt-a1i

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

Copy link
Copy Markdown
Contributor

Summary

  • retry forced profile-default model refreshes when the desktop backend is briefly unavailable
  • ignore stale refresh results so rapid profile switches cannot clobber the current model pill
  • add regression coverage for transient failures and latest-refresh-wins ordering

Fixes #47524

Tests

  • npm --workspace apps/desktop run test:ui -- src/app/session/hooks/use-model-controls.test.tsx
  • npm --workspace apps/desktop run typecheck

@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 duplicate This issue or pull request already exists labels Jun 17, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #47537 — both retry the forced profile model refresh on transient backend unavailability and ignore stale refresh results (use-model-controls.ts) for #47524, same approach. #47537 was opened earlier. Consolidating on #47537.

@tt-a1i

tt-a1i commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Closing as duplicate of #47537 per maintainer comment. Thanks for the review.

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/) duplicate This issue or pull request already exists 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.

[Bug]: Composer model pill stuck in loading spinner after rapid profile switching

2 participants