Skip to content

feat: refactor to multi-provider abstraction with registry pattern#20

Merged
dkargatzis merged 1 commit intomainfrom
feat/refactor-ai-providers
Nov 15, 2025
Merged

feat: refactor to multi-provider abstraction with registry pattern#20
dkargatzis merged 1 commit intomainfrom
feat/refactor-ai-providers

Conversation

@dkargatzis
Copy link
Member

  • Move providers from src/core/ai_providers to src/providers
  • Consolidate integration logic into provider classes
  • Add OpenAI, Bedrock, and Vertex AI provider implementations
  • Update all agents to use new get_chat_model() factory
  • Fix structured output schema for LLMEvaluationResponse
  • Update tests to mock get_chat_model instead of ChatOpenAI

- Move providers from src/core/ai_providers to src/providers
- Consolidate integration logic into provider classes
- Add OpenAI, Bedrock, and Vertex AI provider implementations
- Update all agents to use new get_chat_model() factory
- Fix structured output schema for LLMEvaluationResponse
- Update tests to mock get_chat_model instead of ChatOpenAI

Signed-off-by: Dimitris Kargatzis <dkargatzis@gmail.com>
@dkargatzis dkargatzis self-assigned this Nov 13, 2025
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 21.01167% with 203 lines in your changes missing coverage. Please review.

❌ Your patch status has failed because the patch coverage (21.0%) is below the target coverage (80.0%). You can increase the patch coverage or adjust the target coverage.
❌ Your project status has failed because the head coverage (24.2%) is below the target coverage (80.0%). You can increase the head coverage or adjust the target coverage.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##            main     #20     +/-   ##
=======================================
- Coverage   25.4%   24.2%   -1.3%     
=======================================
  Files         59      67      +8     
  Lines       4098    4632    +534     
=======================================
+ Hits        1044    1124     +80     
- Misses      3054    3508    +454     

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 766c5d7...b24a889. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dkargatzis dkargatzis merged commit def033b into main Nov 15, 2025
2 checks passed
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