Skip to content

feat: add Novita AI as LLM provider#385

Open
Alex-wuhu wants to merge 2 commits intousestrix:mainfrom
Alex-wuhu:novita-integration
Open

feat: add Novita AI as LLM provider#385
Alex-wuhu wants to merge 2 commits intousestrix:mainfrom
Alex-wuhu:novita-integration

Conversation

@Alex-wuhu
Copy link

Summary

  • Adds docs/llm-providers/novita.mdx with setup instructions for Novita AI
  • Registers the new page in docs/docs.json navigation
  • Adds Novita AI card and table entry to docs/llm-providers/overview.mdx

Novita AI provides an OpenAI-compatible inference API, so no Python code changes are needed — users configure it via the existing LLM_API_BASE mechanism that Strix already supports through LiteLLM.

Usage:
```bash
export STRIX_LLM="openai/moonshotai/kimi-k2.5"
export LLM_API_KEY="your-novita-api-key"
export LLM_API_BASE="https://api.novita.ai/openai"
```

Test plan

  • Verify docs/llm-providers/novita.mdx renders correctly in Mintlify
  • Confirm the Novita card appears in the Provider Guides section of the overview page
  • Confirm the Novita row appears in the "Bring Your Own Key" table
  • Manually test with a valid Novita API key and moonshotai/kimi-k2.5

@Alex-wuhu Alex-wuhu marked this pull request as ready for review March 22, 2026 11:29
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 22, 2026

Greptile Summary

This PR adds documentation support for Novita AI as an LLM provider. It introduces a new docs/llm-providers/novita.mdx guide, registers it in the docs/docs.json navigation, and adds a card and table entry to the provider overview page. No Python or runtime code is changed — Novita's OpenAI-compatible API is accessed through the existing LLM_API_BASE mechanism.

  • The new novita.mdx page is well-structured and correctly shows all three required environment variables (STRIX_LLM, LLM_API_KEY, LLM_API_BASE), following the same pattern as existing guides like openrouter.mdx.
  • The docs.json navigation entry is placed consistently with the card ordering in overview.mdx.
  • The "Bring Your Own Key" table in overview.mdx adds Novita alongside providers that only need two env vars, while the shared code example omits LLM_API_BASE — a user copying that snippet for Novita would route requests to OpenAI's endpoint instead of Novita's, resulting in a confusing auth failure.

Confidence Score: 4/5

  • Safe to merge; docs-only change with one minor clarity gap in the overview that could confuse Novita users.
  • All three files are correctly structured, consistent with existing provider guides, and no runtime code is touched. The one concrete issue — the overview's shared BYOK code example omitting LLM_API_BASE for Novita — is a non-blocking docs clarity problem, not a functional bug. Addressing it before merge would prevent user confusion, hence 4 rather than 5.
  • docs/llm-providers/overview.mdx — the "Bring Your Own Key" code example should acknowledge that Novita also requires LLM_API_BASE.

Important Files Changed

Filename Overview
docs/llm-providers/novita.mdx New provider guide for Novita AI; correctly includes all three required env vars (STRIX_LLM, LLM_API_KEY, LLM_API_BASE), model table, and API key instructions following the same pattern as existing provider guides.
docs/llm-providers/overview.mdx Adds Novita AI to the "Bring Your Own Key" table and Provider Guides card group; the BYOK code example only shows two env vars which works for all other listed providers but not for Novita, which also requires LLM_API_BASE.
docs/docs.json Navigation entry for llm-providers/novita correctly inserted between openrouter and vertex, consistent with the card ordering in overview.mdx.

Comments Outside Diff (1)

  1. docs/llm-providers/overview.mdx, line 28-33 (link)

    P2 Novita requires a third env var not shown in the shared example

    The code block immediately after the table only demonstrates STRIX_LLM and LLM_API_KEY. Every other provider in the table (OpenAI, Anthropic, Vertex) works with just those two variables, but Novita also requires LLM_API_BASE set to https://api.novita.ai/openai.

    A user who spots Novita in the table and copies the example below it will have requests routed to OpenAI's actual API rather than Novita's endpoint, producing an auth error that may be hard to diagnose.

    Consider adding a note after the table pointing Novita users to the dedicated guide, for example:

    > **Note:** Novita AI also requires `LLM_API_BASE`. See the [Novita AI guide](/llm-providers/novita) for the full setup.
    
    Prompt To Fix With AI
    This is a comment left during a code review.
    Path: docs/llm-providers/overview.mdx
    Line: 28-33
    
    Comment:
    **Novita requires a third env var not shown in the shared example**
    
    The code block immediately after the table only demonstrates `STRIX_LLM` and `LLM_API_KEY`. Every other provider in the table (OpenAI, Anthropic, Vertex) works with just those two variables, but Novita also requires `LLM_API_BASE` set to `https://api.novita.ai/openai`.
    
    A user who spots Novita in the table and copies the example below it will have requests routed to OpenAI's actual API rather than Novita's endpoint, producing an auth error that may be hard to diagnose.
    
    Consider adding a note after the table pointing Novita users to the dedicated guide, for example:
    
    ```
    > **Note:** Novita AI also requires `LLM_API_BASE`. See the [Novita AI guide](/llm-providers/novita) for the full setup.
    ```
    
    How can I resolve this? If you propose a fix, please make it concise.
Prompt To Fix All With AI
This is a comment left during a code review.
Path: docs/llm-providers/overview.mdx
Line: 28-33

Comment:
**Novita requires a third env var not shown in the shared example**

The code block immediately after the table only demonstrates `STRIX_LLM` and `LLM_API_KEY`. Every other provider in the table (OpenAI, Anthropic, Vertex) works with just those two variables, but Novita also requires `LLM_API_BASE` set to `https://api.novita.ai/openai`.

A user who spots Novita in the table and copies the example below it will have requests routed to OpenAI's actual API rather than Novita's endpoint, producing an auth error that may be hard to diagnose.

Consider adding a note after the table pointing Novita users to the dedicated guide, for example:

```
> **Note:** Novita AI also requires `LLM_API_BASE`. See the [Novita AI guide](/llm-providers/novita) for the full setup.
```

How can I resolve this? If you propose a fix, please make it concise.

Reviews (1): Last reviewed commit: "feat: add Novita AI as LLM provider" | Re-trigger Greptile

The overview table lists Novita alongside providers that only need
STRIX_LLM + LLM_API_KEY, but Novita also requires LLM_API_BASE to
point at api.novita.ai/openai. Without this, users copying the generic
code block would hit connection errors against api.openai.com.

Add a Note callout directing users to the Novita guide for full setup,
and mention Novita in the README supported-providers reference line.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant