Skip to content

Conversation

bnmajor
Copy link
Collaborator

@bnmajor bnmajor commented Oct 9, 2025

This branch depends on #12

This PR updates the project to use newer models and restructures provider dependencies for better optional package management.

  • Updated README examples and provider table to reflect new model defaults
  • Enables optional installation of provider-specific packages
  • Core package no longer requires all provider dependencies
  • Refreshed supported providers table with current model defaults

Breaking Changes

  • Existing users may need to update their configurations if they relied on specific model defaults
  • New users using Anthropic provider will need to install with pip install vtk-prompt[providers-anthropic] or pip install vtk-prompt[providers-all]

@bnmajor bnmajor force-pushed the improve-model-provider-support branch 2 times, most recently from d09baae to cd80ecc Compare October 10, 2025 13:38
@bnmajor bnmajor mentioned this pull request Oct 13, 2025
Copy link
Member

@vicentebolea vicentebolea left a 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)

@vicentebolea vicentebolea added this to the 0.1.0 milestone Oct 15, 2025
- 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
@bnmajor bnmajor force-pushed the improve-model-provider-support branch from cd80ecc to 961dae1 Compare October 17, 2025 13:41
@bnmajor bnmajor marked this pull request as ready for review October 17, 2025 13:51
@bnmajor
Copy link
Collaborator Author

bnmajor commented Oct 17, 2025

To be approve pending:

Rebase on main once #12 is merge

Done 👍

  • factor out the multiple models commit into antoher PR (or perhaps keeps this PR and open another one with the remaining changes)

See #19

@bnmajor bnmajor requested a review from vicentebolea October 17, 2025 13:54
@bnmajor bnmajor mentioned this pull request Oct 17, 2025
Copy link
Member

@vicentebolea vicentebolea left a 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!

@bnmajor bnmajor requested a review from vicentebolea October 17, 2025 18:23
Copy link
Member

@vicentebolea vicentebolea left a 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

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.

2 participants