Repository navigation
Replies: 1 comment
|
Closing: current main already covers this as of f845895271. If you still see it on a build that includes that commit, reply and we'll reopen. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Problem
Hiding a model is a per-client preference today:
providerModelPreferences[instanceId].hiddenModelslives in client settings and is applied inapps/web/src/modelSelection.ts. The server still sends every model to every client, and the mobile app does not read hidden models at all. So there is no way to hide a model for an environment as a whole. #9350 asked for this on mobile and was closed with a pointer here.This matters when a provider instance can only actually use some models. Our team routes Claude and Codex through a shared gateway (via the instance's
ANTHROPIC_BASE_URL/ Codex home), and the gateway refuses certain models for certain people. T3 still lists those models on every client. Someone picks one and the turn fails with a gateway error. On desktop we work around it by having a script writeclient-settings.json. That only takes effect after a reload, and nothing can reach the phone or a browser client.Proposal (small, opt-in)
Add an optional
hiddenModels: string[]to a provider instance's server settings (settings.json→providerInstances.<id>). The server applies it when it builds that instance's model list (aroundproviderModelsFromSettingsinapps/server/src/provider/providerSnapshot.ts), so every client — desktop, browser and mobile — receives the filtered list.settings.jsonalready hot-reloads, so an external tool can update the list without a restart.Questions for maintainers
settings.jsononly, or also a control in the provider settings UI?If the direction and scope are acceptable, I'm happy to send a focused PR with tests covering the server snapshot and all three client types.
Environment: observed on T3 Code 0.0.42 (macOS and Linux desktop, iOS app); code references checked against
mainat 54084ae.All reactions