Revert "Aggiungere LLM Providers" #51

2025-10-26T16:55:36+01:00

Berack96 commented

2025-10-26 16:55:36 +01:00

(Migrated from github.com)

Reverts Berack96/upo-appAI#48

copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed 2025-10-26 16:55:48 +01:00

copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment

Pull Request Overview

This PR reverts the addition of LLM providers (OpenAI, Mistral, DeepSeek, and xAI), removing their integrations and returning the codebase to support only Gemini and Ollama models.

Removes dependencies and imports for OpenAI, Mistral, DeepSeek, and xAI providers
Reverts model validation logic from ModelsConfig back to AppConfig
Restores the original variable naming (style instead of strategy) in the chat interface

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/app/interface/chat.py	Reverts variable name from `strategy` to `style` and removes default value assignments
src/app/configs.py	Removes multi-provider support, keeping only Gemini and Ollama; moves validation methods back to AppConfig
pyproject.toml	Removes package dependencies for openai, mistralai, deepseek, and xai
configs.yaml	Removes configuration entries for gpt, deepseek, xai, and mistral models
.env.example	Removes API key placeholders for OpenAI, Mistral, DeepSeek, and xAI

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

## Pull Request Overview This PR reverts the addition of LLM providers (OpenAI, Mistral, DeepSeek, and xAI), removing their integrations and returning the codebase to support only Gemini and Ollama models. - Removes dependencies and imports for OpenAI, Mistral, DeepSeek, and xAI providers - Reverts model validation logic from `ModelsConfig` back to `AppConfig` - Restores the original variable naming (`style` instead of `strategy`) in the chat interface ### Reviewed Changes Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments. <details> <summary>Show a summary per file</summary> | File | Description | | ---- | ----------- | | src/app/interface/chat.py | Reverts variable name from `strategy` to `style` and removes default value assignments | | src/app/configs.py | Removes multi-provider support, keeping only Gemini and Ollama; moves validation methods back to AppConfig | | pyproject.toml | Removes package dependencies for openai, mistralai, deepseek, and xai | | configs.yaml | Removes configuration entries for gpt, deepseek, xai, and mistral models | | .env.example | Removes API key placeholders for OpenAI, Mistral, DeepSeek, and xAI | </details> --- 💡 <a href="/Berack96/upo-appAI/new/main/.github?filename=*.instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add Copilot custom instructions</a> for smarter, more guided reviews. <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn how to get started</a>.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Berack/upo-app-agents#51