fix(cli): auto-propagate LLM_API_KEY to provider-specific env vars via config-driven detection#59
fix(cli): auto-propagate LLM_API_KEY to provider-specific env vars via config-driven detection#59wutongyuonce wants to merge 4 commits into
Conversation
|
Thanks @wutongyuonce . Since you're expanding the
All three are supported by LiteLLM and widely used; the env var names follow the same |
Enhance LLM Provider Support and Streamline API Key Configuration SummaryThis PR significantly expands OpenKB's support for major LLM providers and optimizes the logic for environment variable loading and distribution. By introducing a parallel "Unified Key" and "Provider-Specific Key" model, it greatly reduces configuration overhead for users while enhancing system robustness in complex multi-model scenarios. 1、Expanded Model Provider Support
2、Optimized API Key Propagation Logic
3、Refactored .env.example Template
|
| click.echo(" OpenAI: gpt-5.4-mini, gpt-5.4") | ||
| click.echo(" Anthropic: anthropic/claude-sonnet-4-6, anthropic/claude-opus-4-6") | ||
| click.echo(" Gemini: gemini/gemini-3.1-pro-preview, gemini/gemini-3-flash-preview") | ||
| click.echo(" DeepSeek: deepseek/deepseek-v4-flash, deepseek/deepseek-v4-0417") |
There was a problem hiding this comment.
why not deepseek-v4-pro?
| _KNOWN_PROVIDER_KEYS = ( | ||
| "OPENAI_API_KEY", "ANTHROPIC_API_KEY", "GEMINI_API_KEY", | ||
| "DEEPSEEK_API_KEY", "COHERE_API_KEY", "MISTRAL_API_KEY", | ||
| "TOGETHER_API_KEY", "GROQ_API_KEY", "MOONSHOT_API_KEY", |
There was a problem hiding this comment.
Please remove Together, GROQ and Cohere. I believe they are distributors, not model providers. Is that correct?
KIMI or MOONSHOT? ZHIPU or GLM, DASHSCOPE or QWEN ? Please help to confirm this confusion, thanks.
| # OpenAI: LLM_API_KEY=sk-... | ||
| # Anthropic: LLM_API_KEY=sk-ant-... | ||
| # Gemini: LLM_API_KEY=AIza... | ||
| # ============================================================================== |
There was a problem hiding this comment.
I don’t think we need to modify this env example file. Adding DeepSeek should be sufficient.
There was a problem hiding this comment.
should remove this file, we have it in repo.
|
I have made the modifications as you said.
|
No description provided.