Skip to content

fix(model): rearchitect how model and providedr information is stored#2877

Open
tusharmath wants to merge 3 commits intomainfrom
provider-model-atomic
Open

fix(model): rearchitect how model and providedr information is stored#2877
tusharmath wants to merge 3 commits intomainfrom
provider-model-atomic

Conversation

@tusharmath
Copy link
Copy Markdown
Collaborator

@tusharmath tusharmath changed the title provider model atomic fix(model): rearchitect how model and providedr information is stored Apr 7, 2026
@github-actions github-actions bot added the type: fix Iterations on existing features or infrastructure. label Apr 7, 2026
async fn get_default_provider(&self) -> anyhow::Result<Provider<Url>>;

/// Sets the default provider for all the agents
async fn set_default_provider(&self, provider_id: ProviderId) -> anyhow::Result<()>;
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drop the setters and getters for default model also from api. Update the implementation to use only the model+provider selector to select them atomically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: fix Iterations on existing features or infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant