Update models

[omarluq]

No description provided.

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

• New Features
  • Added support for Claude Opus 4-6 across Anthropic, Azure, Bedrock and Vertex AI providers, making the new model available throughout the platform.
  • Added GLM-5 to Z.AI model options while retaining GLM-4.7 and GLM-4.5-Air for compatibility.
  • Updated routing and provider mappings to include newer Claude Sonnet and Opus variants so requests can target the latest models.

Walkthrough

Added support for Claude Opus 4-6 and GLM-5 across provider defaults and mappings; updated provider default model slices and an Azure test; adjusted example model mappings and routing identifiers.

Changes

Cohort / File(s)	Summary
Example / Mappings example.yaml	Added claude-opus-4-6 and GLM-5 entries; updated provider model_mapping targets for Anthropic/Bedrock/Azure/Vertex and routing mapping (GLM prefix adjustment).
Anthropic defaults internal/providers/anthropic.go	Added "claude-opus-4-6" to DefaultAnthropicModels.
Azure defaults & test internal/providers/azure.go, internal/providers/azure_test.go	Added "claude-opus-4-6" to DefaultAzureModels; test updated to assert presence.
Bedrock defaults internal/providers/bedrock.go	Added "anthropic.claude-opus-4-6-v1:0" to DefaultBedrockModels.
Vertex defaults internal/providers/vertex.go	Added "claude-opus-4-6" to DefaultVertexModels.
Z.AI defaults internal/providers/zai.go	Replaced "GLM-4-Plus" with "GLM-5" in DefaultZAIModels (now includes GLM-5, GLM-4.7, GLM-4.5-Air).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Hopping through slices and mapping arrays bright,

Opus and GLM join the model delight.
Tests give a wiggle, configs take flight,
I nibble a carrot and ship it tonight. 🥕

🚥 Pre-merge checks | ✅ 1 | ❌ 3

❌ Failed checks (1 warning, 2 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Update models' is too vague and generic; it does not clearly convey which models were updated or what specific changes were made.	Use a more descriptive title that specifies the key models added or changed, such as 'Add Claude Opus 4-6 and GLM-5 model support across providers' to better reflect the actual changes.
Description check	❓ Inconclusive	No pull request description was provided by the author, making it impossible to assess whether it relates to the changeset.	Add a pull request description explaining the model updates, affected providers, and the rationale for these changes.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into main

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch update-models

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

example.yaml (1)

166-169: Minor: Bedrock maps claude-haiku-3-5-20241022 while only some providers map claude-haiku-4-5 variants.

Bedrock maps claude-haiku-3-5-20241022 (Line 169) whereas Z.AI and Ollama map both claude-haiku-4-5-20251001 and claude-haiku-4-5. Vertex maps only the dated variant claude-haiku-4-5-20251001. If Bedrock now supports Haiku 4.5, consider adding that mapping for broader parity. If it doesn't, this is fine as-is.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

Updates default model inventories and example configuration to reflect newer Claude/Z.AI model identifiers across providers.

Changes:

• Add claude-opus-4-6 to default model lists for Anthropic, Azure, Bedrock, and Vertex; add GLM-5 to Z.AI defaults and remove GLM-4-Plus.
• Update example.yaml to include the new models and refresh model/provider mapping examples.
• Adjust go.sum entries consistent with dependency tidying.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
internal/providers/zai.go	Updates default Z.AI model list (adds GLM-5, removes GLM-4-Plus).
internal/providers/vertex.go	Adds a new default Vertex model entry.
internal/providers/bedrock.go	Adds a new default Bedrock model entry.
internal/providers/azure.go	Adds a new default Azure model entry.
internal/providers/anthropic.go	Adds a new default Anthropic model entry.
internal/providers/azure_test.go	Extends assertions to cover the newly added Azure default model.
example.yaml	Updates example routing/provider model lists and model mappings to include new models.
go.sum	Removes unused/older checksum entries (likely go mod tidy).

Comments suppressed due to low confidence (1)

internal/providers/vertex.go:36

• DefaultVertexModels is documented as using the Vertex format model-name@version, but the newly added entry claude-opus-4-6 has no @... version. If Vertex requires a versioned model ID in the URL path, this will cause requests to fail; otherwise the comment is now inaccurate. Please either update the model ID to the correct @version form or adjust the comment (and any related config examples) to reflect that unversioned IDs are valid here.

// DefaultVertexModels are the default models available from Vertex AI.
// Model IDs use Vertex format: model-name@version.
var DefaultVertexModels = []string{
	"claude-opus-4-6",
	"claude-sonnet-4-5@20250514",
	"claude-opus-4-5@20250514",
	"claude-haiku-3-5@20241022",
}

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The routing.model_mapping example uses a lowercase glm prefix, but model-based routing is case-sensitive (uses strings.HasPrefix) and the Z.AI models shown later in this file are GLM-... uppercase. As written, routing won't match requests for GLM-5/GLM-4.7. Update the example prefix to match the actual model IDs (or document that model matching is case-sensitive).

[Copilot]

The Vertex provider is documented as requiring model-name@version IDs, but the example Vertex model_mapping maps claude-opus-4-6 to an unversioned claude-opus-4-6. If Vertex expects a version suffix, this example will produce invalid request URLs. Please update the mapping to the correct Vertex model ID format (or update the surrounding documentation if unversioned IDs are intended/supported).

Suggested change

	"claude-opus-4-6": "claude-opus-4-6"
	"claude-opus-4-6": "claude-opus-4-6@20250219"

[Copilot]

This Bedrock model_mapping section still references the claude-sonnet-4-5-20250929 / ...-20250929-v1:0 IDs, but the code defaults in this PR were updated to ...-20250514.... Keeping a future-dated mapping here is inconsistent with the updated defaults and likely to confuse users copying the example. Consider updating these mappings (or adding additional entries) to align with the 20250514 model IDs used elsewhere in the repo.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.18%. Comparing base (ea3fa4d) to head (aa52220).

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #91      +/-   ##
==========================================
+ Coverage   78.15%   78.18%   +0.03%     
==========================================
  Files          94       94              
  Lines        5800     5800              
==========================================
+ Hits         4533     4535       +2     
+ Misses        980      979       -1     
+ Partials      287      286       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud