Skip to content

Fix au.anthropic.claude opus 4 6 v1#20731

Merged
krrishdholakia merged 3 commits intoBerriAI:mainfrom
anttttti:fix-au.anthropic.claude-opus-4-6-v1
Feb 16, 2026
Merged

Fix au.anthropic.claude opus 4 6 v1#20731
krrishdholakia merged 3 commits intoBerriAI:mainfrom
anttttti:fix-au.anthropic.claude-opus-4-6-v1

Conversation

@anttttti
Copy link
Contributor

@anttttti anttttti commented Feb 9, 2026

Relevant issues

Fixes missing au.anthropic.claude-opus-4-6-v1. Removed duplicate entries for apac.anthropic.claude-opus-4-6-v1, which doesn't exist

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test

Changes

@vercel
Copy link

vercel bot commented Feb 9, 2026

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

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview, Comment Feb 10, 2026 5:55am

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 9, 2026

Greptile Overview

Greptile Summary

This PR correctly fixes the regional prefix for the Claude Opus 4.6 model in the Australia region, changing from the incorrect apac.anthropic.claude-opus-4-6-v1 to the correct au.anthropic.claude-opus-4-6-v1.

Key Changes:

  • Removed duplicate apac.anthropic.claude-opus-4-6-v1 entries (30 lines each) from both JSON configuration files
  • These duplicate entries were causing JSON parsing issues
  • Corrected the remaining entry to use au. prefix for Australia region (ap-southeast-2)
  • Added comprehensive test coverage verifying the correct prefix usage and bedrock_converse_models registration
  • Test documentation clarifies AWS Bedrock regional prefix conventions: us. (United States), eu. (Europe), au. (Australia/ap-southeast-2), apac. (Singapore/ap-southeast-1)

Important Note:
The code in litellm/llms/bedrock/chat/converse_transformation.py:305 currently only strips us., eu., and apac. prefixes but doesn't handle the newly-added au. prefix. This should be addressed to ensure proper handling of Australia region models.

Confidence Score: 4/5

  • This PR is safe to merge but needs a follow-up fix for the regional prefix handling
  • The PR correctly fixes the model configuration by removing duplicate entries and using the proper au. prefix for Australia. Tests are comprehensive and validate both the JSON structure and runtime registration. However, there's a logic issue in converse_transformation.py that doesn't handle the au. prefix, which needs to be addressed in a follow-up PR to ensure complete functionality.
  • The litellm/llms/bedrock/chat/converse_transformation.py file needs attention - it must be updated to handle the au. prefix in the regional prefix stripping logic

Important Files Changed

Filename Overview
model_prices_and_context_window.json Correctly removes duplicate apac.anthropic.claude-opus-4-6-v1 entries and adds the proper au.anthropic.claude-opus-4-6-v1 model configuration for Australia region
litellm/model_prices_and_context_window_backup.json Backup file correctly mirrors the changes made to the main configuration file
tests/test_litellm/test_claude_opus_4_6_config.py Adds comprehensive test coverage for au. prefix and verifies the incorrect apac. prefix is not used, with detailed documentation of AWS regional prefix conventions

Sequence Diagram

sequenceDiagram
    participant User
    participant LiteLLM
    participant ModelConfig as model_prices_and_context_window.json
    participant BedrockConverse as bedrock_converse_models
    participant Bedrock as AWS Bedrock API

    Note over ModelConfig: Before PR: Had duplicate apac. entries
    Note over ModelConfig: After PR: Single au. entry (correct)
    
    User->>LiteLLM: Request with au.anthropic.claude-opus-4-6-v1
    LiteLLM->>ModelConfig: Load model configuration
    ModelConfig-->>LiteLLM: Return au. model config
    LiteLLM->>BedrockConverse: Register au. model
    BedrockConverse-->>LiteLLM: Model registered
    LiteLLM->>Bedrock: Call Australia region (ap-southeast-2)
    Bedrock-->>LiteLLM: Response
    LiteLLM-->>User: Return result
    
    Note over ModelConfig,BedrockConverse: PR correctly maps Australia to au. prefix
    Note over ModelConfig,BedrockConverse: apac. is for Singapore (ap-southeast-1)
Loading

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.

3 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 9, 2026

Additional Comments (1)

litellm/llms/bedrock/chat/converse_transformation.py
Missing au. prefix in regional prefix list. This code strips regional prefixes but only handles us., eu., and apac. - it doesn't handle the au. prefix introduced in this PR. This may cause issues when processing Australia region models.

        for prefix in ["us.", "eu.", "apac.", "au."]:

@krrishdholakia krrishdholakia changed the base branch from main to litellm_oss_staging_02_09_2026 February 10, 2026 04:06
@krrishdholakia krrishdholakia changed the base branch from litellm_oss_staging_02_09_2026 to main February 10, 2026 04:06
@krrishdholakia
Copy link
Member

@anttttti can you fix the merge conflicts, so we can merge this in?

cc: @shin-bot-litellm for monitoring this pr

…aude-opus-4-6-v1

# Conflicts:
#	litellm/model_prices_and_context_window_backup.json
#	model_prices_and_context_window.json
@anttttti
Copy link
Contributor Author

@krrishdholakia Merged. Easy conflicts. Updated changes from both side.

@anttttti
Copy link
Contributor Author

@anttttti can you fix the merge conflicts, so we can merge this in?

cc: @shin-bot-litellm for monitoring this pr

Can this be merged?

@anttttti
Copy link
Contributor Author

@krrishdholakia bump on this. Merge?

@krrishdholakia krrishdholakia merged commit 9e8d1a2 into BerriAI:main Feb 16, 2026
7 of 8 checks passed
@krrishdholakia
Copy link
Member

done, thanks for the bump

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.

2 participants