Merge pull request #72 from lfnovo/fix/config-dict-api-key-issue-68

fix: resolve config dict api_key not being unpacked in OpenAI-based providers

Author: lfnovo

CHANGELOG.md

@@ -5,6 +5,23 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.17.1] - 2026-01-24
+
+### Fixed
+
+- **Config Dict API Key Not Unpacked** - Fixed providers ignoring `api_key` passed via config dict (#68)
+  - Affected providers: OpenRouter, DeepSeek, xAI (LLM), Groq (STT)
+  - These providers inherit from OpenAI-compatible parent classes and were checking for `api_key` before the config dict was unpacked
+  - Now correctly extracts `api_key` and `base_url` from config dict before setting provider defaults
+  - Example that now works:
+    ```python
+    model = AIFactory.create_language(
+        "openrouter",
+        "anthropic/claude-3.5-sonnet",
+        config={"api_key": "sk-or-v1-xxxxx"}
+    )
+    ```
+
 ## [2.17.0] - 2026-01-23
 
 ### Added

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "esperanto"
-version = "2.17.0"
+version = "2.17.1"
 description = "A light-weight, production-ready, unified interface for various AI model providers"
 authors = [
     { name = "LUIS NOVO", email = "lfnovo@gmail.com" }

src/esperanto/providers/llm/deepseek.py

@@ -25,6 +25,13 @@ def provider(self) -> str:
         return "deepseek"
 
     def __post_init__(self):
+        # Extract api_key and base_url from config dict first (before parent sets OpenAI defaults)
+        if hasattr(self, "config") and self.config:
+            if "api_key" in self.config:
+                self.api_key = self.config["api_key"]
+            if "base_url" in self.config:
+                self.base_url = self.config["base_url"]
+
         # Initialize DeepSeek-specific configuration
         self.base_url = self.base_url or os.getenv(
             "DEEPSEEK_BASE_URL", "https://api.deepseek.com/v1"
@@ -37,7 +44,7 @@ def __post_init__(self):
                 "DeepSeek API key not found. Set the DEEPSEEK_API_KEY environment variable."
             )
 
-        # Call parent's post_init to set up normalized response handling
+        # Call parent's post_init (won't overwrite since values are already set)
         super().__post_init__()
 
         # DeepSeek supports JSON mode like OpenAI (handled by parent)

src/esperanto/providers/llm/openrouter.py

@@ -37,6 +37,13 @@ class OpenRouterLanguageModel(OpenAILanguageModel):
     api_key: Optional[str] = None  # Changed type hint
 
     def __post_init__(self):
+        # Extract api_key and base_url from config dict first (before parent sets OpenAI defaults)
+        if hasattr(self, "config") and self.config:
+            if "api_key" in self.config:
+                self.api_key = self.config["api_key"]
+            if "base_url" in self.config:
+                self.base_url = self.config["base_url"]
+
         # Initialize OpenRouter-specific configuration
         self.base_url = self.base_url or os.getenv(
             "OPENROUTER_BASE_URL", "https://openrouter.ai/api/v1"
@@ -48,7 +55,7 @@ def __post_init__(self):
                 "OpenRouter API key not found. Set the OPENROUTER_API_KEY environment variable."
             )
 
-        # Call parent's post_init to set up HTTP clients
+        # Call parent's post_init (won't overwrite since values are already set)
         super().__post_init__()
 
     def _get_headers(self) -> Dict[str, str]:

src/esperanto/providers/llm/xai.py

@@ -20,6 +20,13 @@ class XAILanguageModel(OpenAILanguageModel):
     api_key: Optional[str] = None  # Changed type hint
 
     def __post_init__(self):
+        # Extract api_key and base_url from config dict first (before parent sets OpenAI defaults)
+        if hasattr(self, "config") and self.config:
+            if "api_key" in self.config:
+                self.api_key = self.config["api_key"]
+            if "base_url" in self.config:
+                self.base_url = self.config["base_url"]
+
         # Initialize XAI-specific configuration
         self.base_url = self.base_url or os.getenv(
             "XAI_BASE_URL", "https://api.x.ai/v1"
@@ -31,7 +38,7 @@ def __post_init__(self):
                 "XAI API key not found. Set the XAI_API_KEY environment variable."
             )
 
-        # Call parent's post_init to set up HTTP clients
+        # Call parent's post_init (won't overwrite since values are already set)
         super().__post_init__()
 
 

src/esperanto/providers/stt/groq.py

@@ -14,15 +14,22 @@ class GroqSpeechToTextModel(OpenAISpeechToTextModel):
 
     def __post_init__(self):
         """Initialize HTTP clients with Groq configuration."""
-        # Set Groq-specific API key and base URL before calling parent
+        # Extract api_key and base_url from config dict first (before parent sets OpenAI defaults)
+        if hasattr(self, "config") and self.config:
+            if "api_key" in self.config:
+                self.api_key = self.config["api_key"]
+            if "base_url" in self.config:
+                self.base_url = self.config["base_url"]
+
+        # Set Groq-specific API key and base URL
         self.api_key = self.api_key or os.getenv("GROQ_API_KEY")
         if not self.api_key:
             raise ValueError("Groq API key not found")
-        
+
         # Set Groq's OpenAI-compatible base URL
         self.base_url = self.base_url or "https://api.groq.com/openai/v1"
-        
-        # Call parent's post_init which will initialize HTTP clients
+
+        # Call parent's post_init (won't overwrite since values are already set)
         super().__post_init__()
 
     def _get_default_model(self) -> str:

tests/providers/llm/test_deepseek_provider.py

@@ -52,6 +52,34 @@ def test_to_langchain():
     assert lc is not None
 
 
+def test_initialization_with_api_key_in_config():
+    """Test that api_key can be passed via config dict (GitHub issue #68)."""
+    model = DeepSeekLanguageModel(config={"api_key": "config-test-key"})
+    assert model.api_key == "config-test-key"
+    assert model.base_url == "https://api.deepseek.com/v1"
+
+
+def test_initialization_with_base_url_in_config():
+    """Test that base_url can be passed via config dict."""
+    model = DeepSeekLanguageModel(
+        api_key="test-key",
+        config={"base_url": "https://custom.deepseek.com/v1"}
+    )
+    assert model.base_url == "https://custom.deepseek.com/v1"
+
+
+def test_initialization_with_api_key_and_base_url_in_config():
+    """Test that both api_key and base_url can be passed via config dict."""
+    model = DeepSeekLanguageModel(
+        config={
+            "api_key": "config-test-key",
+            "base_url": "https://custom.deepseek.com/v1"
+        }
+    )
+    assert model.api_key == "config-test-key"
+    assert model.base_url == "https://custom.deepseek.com/v1"
+
+
 # =============================================================================
 # Tool Calling Tests
 # =============================================================================

tests/providers/llm/test_openrouter_provider.py

@@ -39,6 +39,34 @@ def test_custom_base_url():
     assert model.base_url == "https://custom.openrouter.ai/v1"
 
 
+def test_initialization_with_api_key_in_config():
+    """Test that api_key can be passed via config dict (GitHub issue #68)."""
+    model = OpenRouterLanguageModel(config={"api_key": "config-test-key"})
+    assert model.api_key == "config-test-key"
+    assert model.base_url == "https://openrouter.ai/api/v1"
+
+
+def test_initialization_with_base_url_in_config():
+    """Test that base_url can be passed via config dict."""
+    model = OpenRouterLanguageModel(
+        api_key="test-key",
+        config={"base_url": "https://custom.openrouter.ai/v1"}
+    )
+    assert model.base_url == "https://custom.openrouter.ai/v1"
+
+
+def test_initialization_with_api_key_and_base_url_in_config():
+    """Test that both api_key and base_url can be passed via config dict."""
+    model = OpenRouterLanguageModel(
+        config={
+            "api_key": "config-test-key",
+            "base_url": "https://custom.openrouter.ai/v1"
+        }
+    )
+    assert model.api_key == "config-test-key"
+    assert model.base_url == "https://custom.openrouter.ai/v1"
+
+
 # =============================================================================
 # Tool Calling Tests
 # =============================================================================

tests/providers/llm/test_xai_provider.py

@@ -38,6 +38,34 @@ def test_custom_base_url():
     assert model.base_url == "https://custom.x.ai/v1"
 
 
+def test_initialization_with_api_key_in_config():
+    """Test that api_key can be passed via config dict (GitHub issue #68)."""
+    model = XAILanguageModel(config={"api_key": "config-test-key"})
+    assert model.api_key == "config-test-key"
+    assert model.base_url == "https://api.x.ai/v1"
+
+
+def test_initialization_with_base_url_in_config():
+    """Test that base_url can be passed via config dict."""
+    model = XAILanguageModel(
+        api_key="test-key",
+        config={"base_url": "https://custom.x.ai/v1"}
+    )
+    assert model.base_url == "https://custom.x.ai/v1"
+
+
+def test_initialization_with_api_key_and_base_url_in_config():
+    """Test that both api_key and base_url can be passed via config dict."""
+    model = XAILanguageModel(
+        config={
+            "api_key": "config-test-key",
+            "base_url": "https://custom.x.ai/v1"
+        }
+    )
+    assert model.api_key == "config-test-key"
+    assert model.base_url == "https://custom.x.ai/v1"
+
+
 # =============================================================================
 # Tool Calling Tests
 # =============================================================================

tests/providers/stt/test_groq.py

@@ -23,13 +23,48 @@ def test_factory_creates_groq_stt():
     """Test that AIFactory creates Groq STT model."""
     from unittest.mock import patch
     import os
-    
+
     # Mock the environment variable to provide an API key
     with patch.dict(os.environ, {'GROQ_API_KEY': 'test-key'}):
         model = AIFactory.create_speech_to_text("groq")
         assert isinstance(model, GroqSpeechToTextModel)
 
 
+def test_initialization_with_api_key():
+    """Test initialization with api_key parameter."""
+    model = GroqSpeechToTextModel(api_key="test-key")
+    assert model.api_key == "test-key"
+    assert model.base_url == "https://api.groq.com/openai/v1"
+
+
+def test_initialization_with_api_key_in_config():
+    """Test that api_key can be passed via config dict (GitHub issue #68)."""
+    model = GroqSpeechToTextModel(config={"api_key": "config-test-key"})
+    assert model.api_key == "config-test-key"
+    assert model.base_url == "https://api.groq.com/openai/v1"
+
+
+def test_initialization_with_base_url_in_config():
+    """Test that base_url can be passed via config dict."""
+    model = GroqSpeechToTextModel(
+        api_key="test-key",
+        config={"base_url": "https://custom.groq.com/v1"}
+    )
+    assert model.base_url == "https://custom.groq.com/v1"
+
+
+def test_initialization_with_api_key_and_base_url_in_config():
+    """Test that both api_key and base_url can be passed via config dict."""
+    model = GroqSpeechToTextModel(
+        config={
+            "api_key": "config-test-key",
+            "base_url": "https://custom.groq.com/v1"
+        }
+    )
+    assert model.api_key == "config-test-key"
+    assert model.base_url == "https://custom.groq.com/v1"
+
+
 def test_groq_transcribe(audio_file):
     """Test Groq transcribe method with httpx mocking."""
     from unittest.mock import Mock