Skip to content

[Fix] Provider settings contact unselected model services - #1425

Draft
zoomote[bot] wants to merge 1 commit into
mainfrom
fix/scope-provider-model-requests-3somxbbf7yubt
Draft

[Fix] Provider settings contact unselected model services#1425
zoomote[bot] wants to merge 1 commit into
mainfrom
fix/scope-provider-model-requests-3somxbbf7yubt

Conversation

@zoomote

@zoomote zoomote Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

​Created by Roomote. Follow up by mentioning @roomote, in the web UI, or in Discord.

Related GitHub Issue

Closes: #1424

Description

Provider settings now scope dynamic model discovery to the provider the user selected. Automatic and manual refresh paths for LiteLLM, Poe, and Moonshot include explicit provider filters, while Ollama and LM Studio no longer fall through to aggregate router discovery.

The regression coverage was written first and observed failing against the previous aggregate behavior. It covers static providers, selected Kenari discovery, local providers, and the automatic and manual refresh paths that previously omitted a provider filter.

Test Procedure

  1. Open provider settings with a static provider selected and verify no router catalog request is emitted.
  2. Select Kenari and verify the router request is filtered to kenari.
  3. Exercise automatic and manual model refresh for LiteLLM, Poe, and Moonshot and verify each message includes its own provider identifier.
  4. Run the repository test suite and webview type checks.

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Visual Snapshot (UI changes only): No rendered UI change; behavior is covered by Vitest.
  • Documentation Impact: Documentation is updated in the paired Zoo-Code-Docs PR.
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Visual Snapshots

Not applicable. This changes request routing without changing rendered UI.

Videos (interaction / animation only)

Not applicable.

Documentation Updates

Related PRs

Additional Notes

The aggregate extension-host handler remains available for explicit aggregate callers, but every reachable provider-settings path is now provider-scoped.

Get in Touch

Mention @roomote on this PR or use the links in the attribution block above.

@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

[BUG] Provider settings contact unselected third-party model endpoints

1 participant