Skip to content

show active AI provider at the top of the providers list#486

Merged
atomantic merged 2 commits into
mainfrom
cos/task-mplqocrs/agent-6d7933d5
May 25, 2026
Merged

show active AI provider at the top of the providers list#486
atomantic merged 2 commits into
mainfrom
cos/task-mplqocrs/agent-6d7933d5

Conversation

@atomantic
Copy link
Copy Markdown
Owner

Summary

  • On the AI Providers settings page, the provider marked DEFAULT (the active provider) now renders first in the list. Other providers keep their existing order beneath it.
  • Implementation is a partition + spread in the render path — no state or sort key changes.

Test plan

  • Visit Settings → AI Providers with multiple providers configured.
  • Confirm the provider tagged DEFAULT appears at the top of the list.
  • Click "Set Default" on a different provider; confirm the list re-orders so the newly-active one moves to the top without a refresh.
  • With no active provider set (activeProviderId null), confirm the list renders in the original order without crashing.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Reorders the AI Providers settings list so the currently active provider (tagged DEFAULT) appears first, while preserving the existing relative order of all other providers.

Changes:

  • Adjusted the provider list render path to move the active provider to the top of the displayed list.
  • Kept ordering logic local to rendering (no persisted sort key/state changes).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread client/src/pages/AIProviders.jsx
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@atomantic atomantic merged commit e551a40 into main May 25, 2026
3 checks passed
@atomantic atomantic deleted the cos/task-mplqocrs/agent-6d7933d5 branch May 25, 2026 22:18
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