feat(model-config): completion_type array + CRUD endpoints#890
feat(model-config): completion_type array + CRUD endpoints#890vprashrex wants to merge 3 commits into
Conversation
…update related endpoints - Added Alembic migration to change completion_type from scalar to array in model_config table. - Updated API documentation for creating, updating, and deleting model configurations to reflect changes in completion_type. - Implemented bulk create and update functionality for model configurations. - Enhanced model configuration routes to support new array-based completion_type and added validation for input modalities. - Added tests for creating, updating, and deleting model configurations with new completion_type structure.
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…ing and update provider type in list models
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Target issue is #891
Summary
What changed
064—completion_typescalar "stt" → array ['text', 'stt']. Old index dropped, GIN index added./api/v1/models:POST— create one or many.PATCH /{provider}/{model_name}— partial update.PATCH— bulk update, atomic.DELETE /{provider}/{model_name}— hard delete.is_model_supported/list_supported_modelsuse array.contains()queries.input_modalities,output_modalities) validated against enum.app/api/docs/model_config/.Checklist
fastapi run --reload app/main.pyordocker compose upin the repository root and test.