Skip to content
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

Introduce new LLM client architecture #570

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Introduce new LLM client architecture #570

wants to merge 1 commit into from

Conversation

lukehinds
Copy link
Contributor

Begin migration away from LiteLLM with a modular design:

  • Add new llmclient package with provider interface
  • Create bridge implementation to gradually migrate from LiteLLM
  • Simplify LLMClient class to use new provider system
  • Add type definitions for requests/responses

This change sets up the foundation for removing the LiteLLM dependency while maintaining backward compatibility. Providers will be migrated incrementally in follow-up changes. I am trying to do this without stirring up the regression shit pot too much. Happy for others to recommend other approaches

Begin migration away from LiteLLM with a modular design:

- Add new llmclient package with provider interface
- Create bridge implementation to gradually migrate from LiteLLM
- Simplify LLMClient class to use new provider system
- Add type definitions for requests/responses

This change sets up the foundation for removing the LiteLLM dependency
while maintaining backward compatibility. Providers will be migrated
incrementally in follow-up changes. I am trying to do this without
stirring up the regression shit pot too much. Happy for others to
recommend other approaches
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