diff --git a/conf/openrouter_models.json b/conf/openrouter_models.json index 51457f3..1c733f6 100644 --- a/conf/openrouter_models.json +++ b/conf/openrouter_models.json @@ -110,7 +110,7 @@ "model_name": "google/gemini-2.5-pro", "aliases": [ "gemini-2.5", - "pro-2.5-openrouter", + "pro-2.5-openrouter" ], "context_window": 1048576, "max_output_tokens": 65536, diff --git a/tests/test_model_resolution_bug.py b/tests/test_model_resolution_bug.py index 1c51d4b..6c3cf03 100644 --- a/tests/test_model_resolution_bug.py +++ b/tests/test_model_resolution_bug.py @@ -22,19 +22,21 @@ class TestModelResolutionBug: self.consensus_tool = ConsensusTool() def test_openrouter_registry_resolves_gemini_alias(self): - """Test that OpenRouter registry properly resolves 'gemini' to 'google/gemini-2.5-pro'.""" + """Test that OpenRouter registry properly resolves 'gemini' to 'google/gemini-3-pro-preview'.""" # Test the registry directly provider = OpenRouterProvider("test_key") # Test alias resolution resolved_model_name = provider._resolve_model_name("gemini") assert ( - resolved_model_name == "google/gemini-2.5-pro" - ), f"Expected 'google/gemini-2.5-pro', got '{resolved_model_name}'" + resolved_model_name == "google/gemini-3-pro-preview" + ), f"Expected 'google/gemini-3-pro-preview', got '{resolved_model_name}'" # Test that it also works with 'pro' alias resolved_pro = provider._resolve_model_name("pro") - assert resolved_pro == "google/gemini-2.5-pro", f"Expected 'google/gemini-2.5-pro', got '{resolved_pro}'" + assert ( + resolved_pro == "google/gemini-3-pro-preview" + ), f"Expected 'google/gemini-3-pro-preview', got '{resolved_pro}'" # DELETED: test_provider_registry_returns_openrouter_for_gemini # This test had a flawed mock setup - it mocked get_provider() but called get_provider_for_model().