Skip to content

feat: add provider-wide model disable flow#577

Open
NaNomicon wants to merge 7 commits intodecolua:masterfrom
NaNomicon:agent/sisyphus/provider-disable-upstream-pr
Open

feat: add provider-wide model disable flow#577
NaNomicon wants to merge 7 commits intodecolua:masterfrom
NaNomicon:agent/sisyphus/provider-disable-upstream-pr

Conversation

@NaNomicon
Copy link
Copy Markdown

@NaNomicon NaNomicon commented Apr 13, 2026

Summary

  • add provider-wide disabled model storage and mutation flow without changing the flat alias schema
  • hide disabled models from /v1/models, reject direct and alias-resolved requests for disabled models, and keep combo execution aligned with absent-model semantics
  • add disable/enable controls on the provider page, preserve disabled state across /models import, and exclude disabled models from combo search results

Why

Temporarily stopping use of a provider model currently requires destructive remove/re-add behavior, and imports from /models can reintroduce models in inconsistent ways. This change makes disable state explicit, non-destructive, and enforced consistently across discovery, runtime, UI, import, and combo selection.

Notes

  • disabled state is provider-wide rather than account-specific
  • disabledModels stores bare model IDs and takes precedence over enabledModels
  • the provider-wide disabled list is mirrored across all connections for the provider so newly reactivated connections inherit the same state
  • branch includes the combo-search follow-up so disabled models are treated consistently in the combo modal too

NaNomicon and others added 7 commits April 13, 2026 13:53
Always call fetchConnections() after PATCH regardless of res.ok,
so optimistic state is replaced by authoritative backend state.
Also update disabledModels across all connections (not just active ones)
so newly-activated connections inherit the correct disabled list.
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@NaNomicon
Copy link
Copy Markdown
Author

Verification note: I validated the provider-disable behavior on the fork branches before splitting this PR. When re-verifying the cherry-picked upstream branch locally, the existing upstream build script (next build --webpack) failed immediately because the installed Next CLI in that environment rejected the --webpack flag. That failure appears to come from the current upstream/base environment rather than this provider-disable diff itself, but I wanted to make the caveat explicit for reviewers.

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.

1 participant