-
Notifications
You must be signed in to change notification settings - Fork 0
Improve model provider support #13
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
base: master
Are you sure you want to change the base?
Conversation
d09baae
to
cd80ecc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be approve pending:
- Rebase on main once #12 is merge
- factor out the multiple models commit into antoher PR (or perhaps keeps this PR and open another one with the remaining changes)
- Extract VTKPromptClient class from prompt.py to client.py - Extract CLI interface and main() function to cli.py - Convert prompt.py to backward compatibility module with re-exports Maintains full backward compatibility for existing imports.
- Update defaults: gpt-4o -> gpt-5, claude-haiku -> claude-opus-4 - Add provider_utils.py for centralized model management - Support temperature control detection per model
cd80ecc
to
961dae1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
everything else looks good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved upon rebasing to latest main
This branch depends on #12
This PR updates the project to use newer models and restructures provider dependencies for better optional package management.
Breaking Changes
pip install vtk-prompt[providers-anthropic]
orpip install vtk-prompt[providers-all]