Skip to content

fix(settings): clear the image model when nothing remains selectable - #895

Merged
vastsa merged 1 commit into
mainfrom
fix/image-model-empty-selection
Sep 22, 2026
Merged

vastsa merged 1 commit into
mainfrom
fix/image-model-empty-selection

Conversation

@vastsa

@vastsa vastsa commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Hide the Models page image-model row when no marked model can be selected.
  • Clearing every image checkbox on the provider that holds the default unchecks it, instead of automatically selecting another provider.

Test plan

  • node --test apps/desktop/test/image-generation-default.test.mjs
  • node --test apps/desktop/test/model-advanced-capabilities.test.mjs
  • Settings → Models: with no selectable image model, the row is hidden
  • Uncheck every image model on the current provider and save; the settings row is unchecked

The Models page kept a checked image model after every image checkbox
on its provider was cleared, and still showed the row when no candidate
could be chosen. Hide that row when nothing is selectable, and leave
the default unchecked instead of adopting another provider.
Copilot AI lite review requested due to automatic review settings September 22, 2026 17:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vastsa
vastsa merged commit 1a6a36e into main Sep 22, 2026
4 checks passed
@vastsa
vastsa deleted the fix/image-model-empty-selection branch September 24, 2026 01:54
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