feat(llm): add novita api as a default provider - #829
Open
wu21-web wants to merge 7 commits into
Open
Conversation
|
|
Contributor
|
✅ OpenCodeReview: Review complete: 0 finding(s) across 1 selected item(s). |
Novita's endpoint (https://api.novita.ai/v3/openai) is OpenAI-compatible, so it registers like the other OpenAI-protocol providers (deepseek, kimi, z-ai, ...). Adds the registry entry, the matching name in the provider-order test, and the provider table in the en/zh/ja configuration docs. Model ids are taken from the live /v3/openai/models endpoint.
The registry entry, tests-adjacent docs tables (en/ja/zh configuration.md) used https://api.novita.ai/v3/openai. That path still resolves today, but Novita's current documentation (novita.ai/docs/guides/llm-api) no longer shows it; the current documented OpenAI-compatible endpoint is https://api.novita.ai/openai. Verified live 2026-08-01: both paths return identical /models and /chat/completions results, so this is a stale-citation fix, not a functional break.
The models listed for Novita were older ids that no longer reflect what the platform leads with. Point the recommendations at the three current flagships instead, each verified against api.novita.ai: moonshotai/kimi-k3 1M context, native vision zai-org/glm-5.2 1M context, long-horizon agentic work deepseek/deepseek-v4-flash-0731 1M context, cheapest of the three Context windows, output limits, input modalities and pricing were taken from the live /openai/v1/models response rather than carried over.
Co-authored-by: Tao Xin <wu2196674@icloud.com>
Rebase-equivalent update of internal/llm/providers.go, internal/llm/providers_test.go, and the en/ja/zh/ru configuration docs to match upstream/main's current content (minimax-cn, mistral, model list refreshes) while keeping the novita entry this branch adds. This is a targeted content sync of the six files that conflicted, not a full merge, so unrelated upstream changes (workflows, CI, etc.) are left untouched. Also adds the novita row to the ru docs table, which was missing.
wu21-web
force-pushed
the
feat/novita-provider
branch
from
August 10, 2026 13:55
4b1dfab to
ed51577
Compare
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
3 participants
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.
Description
This is a successor of #755 , credit: @jax-novita
Type of Change
How Has This Been Tested?
make testpasses locallyChecklist
go fmt,go vet)Related Issues
closes #673