Skip to content

fix: keep provider-returned models visible#8324

Closed
DhyeyMavani2003 wants to merge 2 commits into
aaif-goose:mainfrom
DhyeyMavani2003:codex/provider-model-list-trust-provider
Closed

fix: keep provider-returned models visible#8324
DhyeyMavani2003 wants to merge 2 commits into
aaif-goose:mainfrom
DhyeyMavani2003:codex/provider-model-list-trust-provider

Conversation

@DhyeyMavani2003
Copy link
Copy Markdown

@DhyeyMavani2003 DhyeyMavani2003 commented Apr 5, 2026

Addressing #8321

Summary

  • stop treating the canonical registry as an allowlist in fetch_recommended_models
  • keep every provider-returned model visible while still ranking canonically-known text/tool-capable models first
  • add regression coverage for unknown models and custom proxy model lists

Testing

  • cargo fmt

Context

This addresses the provider model list behavior where Goose silently hid models that were returned by the provider but were missing from the bundled canonical registry. The registry is now advisory for ranking instead of gating visibility.

Signed-off-by: Dhyey Mavani <82772894+DhyeyMavani2003@users.noreply.github.com>
Signed-off-by: Dhyey Mavani <82772894+DhyeyMavani2003@users.noreply.github.com>
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