feat(llm): switch default provider to DeepSeek - #65
Merged
Merged
Conversation
Code, env example, and docs all flagged MiMo as the default, but the actual project setup (and the demo recording about to be made) runs against DeepSeek. The factory's auto-detect order put MiMo first and the no-key fallback was also MiMo — both wrong relative to what the README says. Realigns the factory and docs to match reality: - factory: DEEPSEEK_API_KEY auto-detects deepseek; both keys configured picks deepseek; no key falls back to deepseek (so the resulting LLMConfigError mentions the documented default). - test_factory: the both-keys-present test now pins deepseek and is renamed to match the new contract. - .env.example: DeepSeek block moved to the top with the default notes; MiMo block kept underneath as the explicit alternative. - README: every "默认走小米 MiMo" line, env-var example, key-prefix table, dependency table and FAQ row now lead with DeepSeek. - docs/API.md: env table row order flipped, default-provider note rewritten so the new "both keys → deepseek" tiebreak is explicit. 318 backend tests still pass; the factory contract change is covered by the rewritten test. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
功能描述
把"默认 LLM provider"从小米 MiMo 切换为 DeepSeek,让 README / .env.example / factory 三处对齐——之前文档写默认 MiMo,但实际项目跑的是 DeepSeek。
实现思路
factory(代码契约):
测试:
`.env.example`:DeepSeek 块挪到最上面 + 写明"默认";MiMo 块保留在下面作为备选;`STORY2SCRIPT_LLM_PROVIDER` 注释更新说明新的优先级。
README:
`docs/API.md`:环境变量表 `DEEPSEEK_API_KEY` 移到第一行 + 默认 provider;`STORY2SCRIPT_LLM_PROVIDER` 取值表序改成 `deepseek / mimo` 并显式写明双 key 时选哪个。
测试方式
依赖与复用声明
无新增第三方依赖。改动是"默认值翻转" + 文档同步,不引入任何能力。
Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com