-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat: add support for custom provider configuration in the user config #537
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
@kshern Could you take a look at the CI failures and then I will have a proper read through the implementation before commenting further. Thank you! |
We are planning to have a unified config, so could you update this PR so that the changes apply to |
@kshern Will merge once it's a single config and CI issue have been fixed. |
Feat/add provider config
Feat/migrate provider to config
Thanks for your patience! I've now completed the refactor to integrate the custom provider logic into StoredConfig as discussed.
and CI issue have been fixed The PR should now be ready for another look. Please let me know if there's anything else needed. Thanks again for your guidance! |
Fixed merge conflicts and adjusted code structure and comments a bit. This is now going in, thank you for making your first contribution to the project, keep them coming! |
What
Why
This change enables users to flexibly customize and extend provider endpoints and API keys without modifying the codebase, making the CLI more adaptable for various LLM backends and enterprise use cases.
How
loadProvidersFromFile
andgetMergedProviders
in config logic.Checklist