28 lines
1007 B
Markdown
28 lines
1007 B
Markdown
# LLM Clients - Consistency Improvements
|
|
|
|
## Completed
|
|
|
|
### 1. `validate_model()` warning path
|
|
- `create_llm_client()` now calls `validate_model()` and emits a warning for
|
|
unknown model names instead of failing immediately.
|
|
|
|
## Remaining Issues to Fix
|
|
|
|
### 2. Inconsistent parameter handling
|
|
| Client | API Key Param | Special Params |
|
|
|--------|---------------|----------------|
|
|
| OpenAI | `api_key` | `reasoning_effort` |
|
|
| Anthropic | `api_key` | `thinking_config` → `thinking` |
|
|
| Google | `google_api_key` | `thinking_budget` |
|
|
|
|
**Fix:** Standardize with unified `api_key` that maps to provider-specific keys
|
|
|
|
### 3. `base_url` accepted but ignored
|
|
- `AnthropicClient`: accepts `base_url` but never uses it
|
|
- `GoogleClient`: accepts `base_url` but never uses it (correct - Google doesn't support it)
|
|
|
|
**Fix:** Remove unused `base_url` from clients that don't support it
|
|
|
|
### 4. Update validators.py with models from CLI
|
|
- Sync `VALID_MODELS` dict with CLI model options after Feature 2 is complete
|