Skip to content

anthroppci price fix#5641

Merged
chitalian merged 1 commit intomainfrom
justin/anthropic-fix-price-2
Mar 19, 2026
Merged

anthroppci price fix#5641
chitalian merged 1 commit intomainfrom
justin/anthropic-fix-price-2

Conversation

@chitalian
Copy link
Contributor

Ticket

Link to the ticket(s) this pull request addresses.

Component/Service

What part of Helicone does this affect?

  • Web (Frontend)
  • Jawn (Backend)
  • Worker (Proxy)
  • Bifrost (Marketing)
  • AI Gateway
  • Packages
  • Infrastructure/Docker
  • Documentation

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Performance improvement
  • Refactoring

Deployment Notes

  • No special deployment steps required
  • Database migrations need to run
  • Environment variable changes required
  • Coordination with other teams needed

Screenshots / Demos

Before After

Extra Notes

Any additional context, considerations, or notes for reviewers.

Context

Why are you making this change?

Screenshots / Demos

@vercel
Copy link

vercel bot commented Mar 19, 2026

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

Project Deployment Actions Updated (UTC)
helicone Error Error Mar 19, 2026 7:28pm
helicone-bifrost Error Error Mar 19, 2026 7:28pm
helicone-eu Error Error Mar 19, 2026 7:28pm

Request Review

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.

@supabase
Copy link

supabase bot commented Mar 19, 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 Mar 19, 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 Mar 19, 2026

Claude finished @chitalian's task —— View job


PR Review Complete ✅

Todo List:

  • Read the modified pricing endpoint file
  • Read the test snapshot changes
  • Analyze what pricing data was removed
  • Check for potential issues or breaking changes
  • Provide comprehensive review feedback with confidence score

Changes Summary:

This PR removes the higher pricing tier (200,000+ token threshold) from Claude 4.6 Sonnet across all providers (anthropic, vertex, bedrock, helicone). The removed tier had doubled input costs ($6/MTok vs $3/MTok) and 1.5x output costs ($22.50/MTok vs $15/MTok).

Analysis:

✅ Positive Aspects:

  • Clean, consistent removal across all 4 provider endpoints
  • Test snapshots updated appropriately
  • No syntax errors or structural issues
  • Consistent with other recent Anthropic models that only have single-tier pricing

⚠️ Considerations:

  1. Breaking Change Potential: Applications relying on the 200k+ token tier pricing calculations will now use the base tier pricing, resulting in lower costs but potentially incorrect billing calculations
  2. Documentation Gap: No indication if this reflects actual Anthropic API changes or internal business decisions
  3. Pattern Inconsistency: Other Anthropic models (claude-sonnet-4-5-20250929, claude-sonnet-4, claude-4.5-sonnet) still retain the 200k threshold pricing tier

🔍 Code Quality:

  • File structure and formatting maintained correctly
  • TypeScript types remain valid
  • No security concerns identified
  • Performance impact is negligible (simplified pricing calculation)

📋 Recommendations:

  1. Add context: Consider adding a commit message or PR description explaining why this tier was removed
  2. Verify consistency: Confirm if other Claude 4.x models should also have this tier removed for consistency
  3. Consider migration: If this affects existing customers, ensure billing systems handle the transition gracefully

Score: 7/10 - Safe to merge with minor considerations

Suggestions Summary:

  • ✅ Code changes are technically correct and safe
  • ⚠️ Consider documenting the business rationale for removing the higher-usage pricing tier
  • 💡 Review other Claude models for pricing consistency if this represents a broader pricing strategy change

@chitalian chitalian merged commit a45ec6d into main Mar 19, 2026
9 of 13 checks passed
@chitalian chitalian deleted the justin/anthropic-fix-price-2 branch March 19, 2026 19:28
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