fix(sdk): make the capability catalog lookup case-insensitive on provider

Author: mmabrouk

sdks/python/agenta/sdk/agents/model_catalog.py

@@ -145,18 +145,29 @@ def claude_model_catalog() -> ModelCatalog:
     return _CLAUDE_CATALOG
 
 
+def _catalog_id(provider: Optional[str], model_id: str) -> str:
+    """Build the ``provider/model`` join key.
+
+    Catalog ids carry a lowercase provider, and the rest of the system (environment resolver,
+    connection matching) treats provider names case-insensitively, so a caller-supplied
+    ``"OpenAI"`` must still join.
+    """
+    head, separator, tail = model_id.partition("/")
+    if provider is None:
+        return f"{head.lower()}/{tail}" if separator else model_id
+    if separator and head.lower() == provider.lower():
+        return f"{provider.lower()}/{tail}"
+    return f"{provider.lower()}/{model_id}"
+
+
 def model_input_modalities(
     harness: Optional[str], model_id: str, *, provider: Optional[str] = None
 ) -> Optional[List[str]]:
     """Look up input modalities using the model id form accepted by ``harness``."""
     entry: Optional[ModelCatalogEntry]
     if harness in ("pi_core", "pi_agenta"):
         catalog = pi_model_catalog()
-        catalog_id = (
-            model_id
-            if provider is None or model_id.startswith(f"{provider}/")
-            else f"{provider}/{model_id}"
-        )
+        catalog_id = _catalog_id(provider, model_id)
     elif harness == "claude":
         catalog = claude_model_catalog()
         catalog_id = model_id
@@ -166,9 +177,7 @@ def model_input_modalities(
     entry = next((item for item in catalog.models if item.id == catalog_id), None)
     if harness == "claude" and entry is None:
         # Reuse the same sourced Anthropic fact from Pi's generated catalog; do not guess.
-        pi_catalog_id = (
-            model_id if model_id.startswith("anthropic/") else f"anthropic/{model_id}"
-        )
+        pi_catalog_id = _catalog_id("anthropic", model_id)
         entry = next(
             (item for item in pi_model_catalog().models if item.id == pi_catalog_id),
             None,

sdks/python/oss/tests/pytest/unit/agents/connections/test_model_catalog.py

@@ -164,6 +164,21 @@ def test_pi_input_modalities_lookup_joins_resolved_provider_and_model(harness):
     ]
 
 
+def test_input_modalities_lookup_is_case_insensitive_on_provider():
+    # Provider names are matched case-insensitively everywhere else (environment resolver,
+    # connection matching); a mixed-case provider must not silently drop the modality fact.
+    assert model_input_modalities(
+        "pi_core", "gpt-5.5", provider="OpenAI"
+    ) == model_input_modalities("pi_core", "gpt-5.5", provider="openai")
+    assert model_input_modalities("pi_core", "OpenAI/gpt-5.5", provider="OpenAI") == [
+        "text",
+        "image",
+    ]
+    assert model_input_modalities(
+        "claude", "claude-sonnet-4-6", provider="Anthropic"
+    ) == ["text", "image"]
+
+
 def test_claude_input_modalities_lookup_uses_bare_alias():
     assert model_input_modalities("claude", "sonnet", provider="anthropic") == [
         "text",