Skip to content

Conversation

@ammar-agent
Copy link
Collaborator

Generated with mux

Description

Changes the default model behavior from implicit (LRU-based) to explicit user selection.

Changes

  • Explicit Default: Users can now star a model in the selector to make it the default.
  • Persistence: Default model is stored separately from the LRU list.
  • Safety:
    • Default model cannot be null (falls back to system default).
    • Default model cannot be deleted from the list.
    • Default status cannot be unset (only replaced by selecting a new default).
  • UI: Added star icon button to model selector rows.

Testing

  • Verified model selector UI interactions (star/unstar/delete).
  • Checked persistence across reloads.
  • Verified fallback behavior when no explicit default is set (initially).

@ammar-agent ammar-agent force-pushed the explicit-default-model-selector branch 6 times, most recently from 9fc1d86 to 8d22f5b Compare November 24, 2025 17:06
- Replace implicit LRU default behavior with explicit user selection
- Add star button to model selector to set default model
- Ensure default model cannot be deleted or unset (only replaced)
- Persist default model separately from LRU list
@ammar-agent ammar-agent force-pushed the explicit-default-model-selector branch from 8d22f5b to 62ce0ab Compare November 24, 2025 17:54
@ammario ammario merged commit 0d6339d into main Nov 24, 2025
18 of 19 checks passed
@ammario ammario deleted the explicit-default-model-selector branch November 24, 2025 18:20
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