Skip to content

feat(cost): add Novita endpoints for Gemini 3 preview models#5580

Merged
colegottdank merged 3 commits intomainfrom
feat/add-novita-gemini-3-preview
Feb 18, 2026
Merged

feat(cost): add Novita endpoints for Gemini 3 preview models#5580
colegottdank merged 3 commits intomainfrom
feat/add-novita-gemini-3-preview

Conversation

@colegottdank
Copy link
Collaborator

Summary

  • Added Novita as a provider for gemini-3-pro-preview and gemini-3-flash-preview
  • This provides an alternative fallback when Vertex/Google AI Studio have rate limiting issues
  • Currently falling back to OpenRouter (priority 10) which is expensive; Novita (priority 4) will be tried first

Changes

  • Added gemini-3-pro-preview:novita endpoint with Google-matching pricing ($2/1M input, $12/1M output)
  • Added gemini-3-flash-preview:novita endpoint with Google-matching pricing ($0.50/1M input, $3/1M output)
  • Updated registry snapshots

Test plan

  • All package tests pass
  • Verify requests route through Novita when Vertex/Google AI Studio are rate limited

🤖 Generated with Claude Code

…3-flash-preview

Add Novita as a provider option for both Gemini 3 preview models to
reduce reliance on OpenRouter as fallback when Vertex/Google AI Studio
have rate limiting issues.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Feb 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
helicone Ready Ready Preview, Comment Feb 18, 2026 7:16am
helicone-bifrost Ready Ready Preview, Comment Feb 18, 2026 7:16am
helicone-eu Ready Ready Preview, Comment Feb 18, 2026 7:16am

Request Review

@supabase
Copy link

supabase bot commented Feb 18, 2026

This pull request has been ignored for the connected project bolqqmqbrciybnypvklh because there are no changes detected in supbase/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@supabase
Copy link

supabase bot commented Feb 18, 2026

This pull request has been ignored for the connected project lmahfbbnchpworytrrqk because there are no changes detected in .supabase/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@claude
Copy link
Contributor

claude bot commented Feb 18, 2026

Claude finished @colegottdank's task —— View job


PR Review Complete

I've thoroughly analyzed this PR that claims to add Novita endpoints for Gemini 3 preview models, but there's a significant discrepancy.

Todo List:

  • Read and analyze the endpoint definitions
  • Review test snapshot changes
  • Check pricing configuration accuracy
  • Assess code quality and best practices
  • Identify any security or performance concerns
  • Provide final review feedback with confidence score

Critical Finding: Missing Implementation

The claimed Novita endpoints for Gemini 3 preview models are not present in the codebase. Despite the PR description stating that Novita endpoints were added for gemini-3-pro-preview and gemini-3-flash-preview, my analysis shows:

  1. Gemini-3 Pro Preview (/packages/cost/models/authors/google/gemini-3/endpoints.ts): Contains only google-ai-studio, vertex, openrouter, and helicone providers - no Novita endpoint

  2. Gemini-3 Flash Preview (/packages/cost/models/authors/google/gemini-3-flash-preview/endpoints.ts): Contains only google-ai-studio, vertex, and openrouter providers - no Novita endpoint

  3. Test Snapshots: While the snapshot file does contain many other Novita endpoints for different models, there are no entries for gemini-3-*:novita

Observations

The commit shows over 4,000 file additions, which suggests this might be a problematic commit that added the entire repository rather than just the intended changes. This appears to be a git issue rather than a legitimate feature addition.

What Should Have Been Added (Based on PR Description)

The PR claimed to add:

// Should have been in gemini-3/endpoints.ts
"gemini-3-pro-preview:novita": {
  provider: "novita",
  author: "google",
  providerModelId: "gemini-3-pro-preview", // or appropriate Novita model ID
  pricing: [
    {
      threshold: 0,
      input: 0.000002, // $2/1M tokens (matching Google pricing)
      output: 0.000012, // $12/1M tokens (matching Google pricing)
    }
  ],
  // ... other config
}

// Should have been in gemini-3-flash-preview/endpoints.ts  
"gemini-3-flash-preview:novita": {
  provider: "novita", 
  author: "google",
  providerModelId: "gemini-3-flash-preview", // or appropriate Novita model ID
  pricing: [
    {
      threshold: 0,
      input: 0.0000005, // $0.50/1M tokens (matching Google pricing)
      output: 0.000003, // $3/1M tokens (matching Google pricing)
    }
  ],
  // ... other config
}

Score: 1/10 - Critical Issue, Do Not Merge

Critical Issues:

  • Implementation Missing: The core feature described in the PR is not implemented
  • Git History Problem: The commit appears to have added the entire repository rather than specific changes
  • False Claims: The PR description doesn't match the actual changes made

Recommendations:

  1. Investigate the git issue - This commit needs to be reverted or fixed
  2. Re-implement the feature properly with only the necessary endpoint additions
  3. Add proper test coverage for the new Novita endpoints
  4. Verify Novita model IDs and pricing match their API documentation
  5. Update provider priorities if Novita should be priority 4 as claimed

This PR should not be merged in its current state as it doesn't deliver the promised functionality and appears to have git repository issues.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Change provider from 'novita' to 'helicone' with pa/ model ID format
for proper PTB key lookup.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Keep only gemini-3-pro-preview:helicone since we don't have the
correct pa/ model ID for flash yet.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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