Skip to content

fix(provider): Add sync button and resolve bug merge models - #290

Merged
co-l merged 1 commit into
co-l:developfrom
JamesDAdams:fix/multi-mode-merge
Sep 2, 2026
Merged

fix(provider): Add sync button and resolve bug merge models#290
co-l merged 1 commit into
co-l:developfrom
JamesDAdams:fix/multi-mode-merge

Conversation

@JamesDAdams

Copy link
Copy Markdown
Contributor

Summary

  • Add sync button on add/edit provider for check if a new models is available
  • Fix duplicate models on Sync: In ProviderModal.tsx, fetchModels now claims both the merged model's id and each member's apiModelId. This prevents raw catalog models fetched from the provider backend from duplicating already configured and mode-merged models in the models list and in the Selected Models view.
  • Preserve model selection during Sync: Newly discovered models fetched from the provider catalog are added to models and displayed in Available Models (unselected) without polluting Selected Models.
  • Constrain mode suffix detection and parsing: Constrained splitModeSuffix to recognized MODE_SUFFIXES to prevent unrelated multi-hyphen model names from being grouped into pseudo-mode families.
  • Fix Ollama tag preservation: Ensured parseModelValue validates against known reasoning effort values before splitting on a colon, preventing Ollama model tags (such as :latest, :instruct, :32b) from being stripped and treated as reasoning efforts.
  • Preserve model metadata on update: Ensured updateModelSettings preserves existing model metadata (modes, requestBody, selected).

AI-Enhanced Development

Tell what models helped shape this PR:

  • AI Models: Gemini 3.7 Flash

Cache Impact

Does this PR affect anything cached — system prompts, tool definitions, skills, or other context?

  • No

@co-l
co-l force-pushed the fix/multi-mode-merge branch from cfc702f to 71521e9 Compare September 2, 2026 17:08
@co-l
co-l merged commit 58ec384 into co-l:develop Sep 2, 2026
4 of 6 checks passed
@co-l

co-l commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Merged, will be released in v2.0.136

AI-summary

  • Reviewed the sync/model-dedup changes and fixed 2 i18n lint errors that would have failed CI (Sync button label and its title now go through t() with en + fr)
  • Verified the full unit suite (5087 passed), typecheck and lint clean, and the provider-rest/model-selection e2e green
  • Rebased the PR onto latest develop and pushed the branch to the contributor's fork so it merges cleanly
  • Squash-merged

co-l added a commit that referenced this pull request Sep 2, 2026
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