Skip to content

feat: adding gemini and XAI#216

Merged
cristiam86 merged 3 commits intomainfrom
edinaldo/dxp-234-key-set-up-new-providers
Apr 25, 2025
Merged

feat: adding gemini and XAI#216
cristiam86 merged 3 commits intomainfrom
edinaldo/dxp-234-key-set-up-new-providers

Conversation

@epsjunior
Copy link
Copy Markdown
Contributor

Brief Overview

  • Added support for Gemini and XAI LLM providers
  • Expanded .env.example to include new API keys
  • Updated internal AI provider types and configuration

🔹 Problem Solved

  • Missing support for Gemini and XAI LLMs in provider configuration

🔹 Changes Made

Provider Support

  • Added "geminiai" and "xai" to AiProviders union type
  • Included "GEMINI_API_KEY" and "XAI_API_KEY" in AiProvidersEnvVars
  • Expanded AI_PROVIDERS_CONFIG with configuration for Gemini and XAI

Environment Variables

  • Extended .env.example with placeholders for:
    • GEMINI_API_KEY
    • XAI_API_KEY
  • Added usage hints to guide developers on configuration requirements

🔹 Important Decisions

  • Kept backward compatibility by preserving current provider structures

🔹 Previously Aligned Architecture Changes

  • Built upon existing AI_PROVIDERS_CONFIG structure
  • No refactor required; changes focused on data extension only

@epsjunior epsjunior requested a review from cristiam86 April 24, 2025 20:58
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (ae8b5fe) to head (0da43b7).
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #216   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           29        29           
  Lines         1505      1534   +29     
  Branches       373       381    +8     
=========================================
+ Hits          1505      1534   +29     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cristiam86 cristiam86 merged commit 01310b2 into main Apr 25, 2025
3 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