fix(ai): Update LLM endpoints

jira: GDAI-339
risk: low

Author: malmiron

gooddata-sdk/gooddata_sdk/catalog/organization/entity_model/llm_endpoint.py

@@ -48,7 +48,6 @@ def init(
         id: str,
         title: str,
         token: str,
-        description: Optional[str] = None,
         provider: Optional[str] = None,
         base_url: Optional[str] = None,
         llm_organization: Optional[str] = None,
@@ -59,7 +58,6 @@ def init(
             attributes=CatalogLlmEndpointAttributes(
                 title=title,
                 token=token,
-                description=description,
                 provider=provider,
                 base_url=base_url,
                 llm_organization=llm_organization,
@@ -73,7 +71,6 @@ def from_api(cls, entity: dict[str, Any]) -> CatalogLlmEndpoint:
         attr = CatalogLlmEndpointAttributes(
             title=safeget(ea, ["title"]),
             token="",  # Token is not returned for security reasons
-            description=safeget(ea, ["description"]),
             provider=safeget(ea, ["provider"]),
             base_url=safeget(ea, ["baseUrl"]),
             llm_organization=safeget(ea, ["llmOrganization"]),
@@ -100,7 +97,6 @@ def init(
         id: str,
         title: Optional[str] = None,
         token: Optional[str] = None,
-        description: Optional[str] = None,
         provider: Optional[str] = None,
         base_url: Optional[str] = None,
         llm_organization: Optional[str] = None,
@@ -111,7 +107,6 @@ def init(
             attributes=CatalogLlmEndpointPatchAttributes(
                 title=title,
                 token=token,
-                description=description,
                 provider=provider,
                 base_url=base_url,
                 llm_organization=llm_organization,
@@ -124,7 +119,6 @@ def init(
 class CatalogLlmEndpointAttributes(Base):
     title: str
     token: str
-    description: Optional[str] = None
     provider: Optional[str] = None
     base_url: Optional[str] = None
     llm_organization: Optional[str] = None
@@ -139,7 +133,6 @@ def client_class() -> type[JsonApiLlmEndpointInAttributes]:
 class CatalogLlmEndpointPatchAttributes(Base):
     title: Optional[str] = None
     token: Optional[str] = None
-    description: Optional[str] = None
     provider: Optional[str] = None
     base_url: Optional[str] = None
     llm_organization: Optional[str] = None

gooddata-sdk/gooddata_sdk/catalog/organization/service.py

@@ -574,7 +574,6 @@ def create_llm_endpoint(
         id: str,
         title: str,
         token: str,
-        description: Optional[str] = None,
         provider: Optional[str] = None,
         base_url: Optional[str] = None,
         llm_organization: Optional[str] = None,
@@ -587,7 +586,6 @@ def create_llm_endpoint(
             id: Identifier of the LLM endpoint
             title: User-facing title of the LLM Provider
             token: The token to use to connect to the LLM provider
-            description: Optional user-facing description of the LLM endpoint
             provider: Optional LLM provider name (e.g., "openai")
             base_url: Optional base URL for custom LLM endpoint
             llm_organization: Optional LLM organization identifier
@@ -600,7 +598,6 @@ def create_llm_endpoint(
             id=id,
             title=title,
             token=token,
-            description=description,
             provider=provider,
             base_url=base_url,
             llm_organization=llm_organization,
@@ -617,7 +614,6 @@ def update_llm_endpoint(
         id: str,
         title: Optional[str] = None,
         token: Optional[str] = None,
-        description: Optional[str] = None,
         provider: Optional[str] = None,
         base_url: Optional[str] = None,
         llm_organization: Optional[str] = None,
@@ -630,7 +626,6 @@ def update_llm_endpoint(
             id: Identifier of the LLM endpoint
             title: User-facing title of the LLM Provider
             token: The token to use to connect to the LLM provider. If not provided, the existing token will be preserved.
-            description: User-facing description of the LLM endpoint
             provider: LLM provider name (e.g., "openai")
             base_url: Base URL for custom LLM endpoint
             llm_organization: LLM organization identifier
@@ -643,7 +638,6 @@ def update_llm_endpoint(
             id=id,
             title=title,
             token=token,
-            description=description,
             provider=provider,
             base_url=base_url,
             llm_organization=llm_organization,

gooddata-sdk/tests/catalog/fixtures/organization/create_llm_endpoint.yaml

@@ -75,7 +75,6 @@ interactions:
           attributes:
             title: Test Endpoint 2
             token: secret-token-2
-            description: Test Description
             provider: OPENAI
             baseUrl: https://api.example.com
             llmOrganization: org1

gooddata-sdk/tests/catalog/fixtures/organization/delete_llm_endpoint.yaml

@@ -161,7 +161,7 @@ interactions:
             to access it.
           status: 404
           title: Not Found
-          traceId: af6c2c610c999d92ae5b97820b9370cb
+          traceId: 85a65755a01301cfc27020eabbbdf90a
   - request:
       method: DELETE
       uri: http://localhost:3000/api/v1/entities/llmEndpoints/endpoint1

gooddata-sdk/tests/catalog/fixtures/organization/update_llm_endpoint.yaml

@@ -139,7 +139,6 @@ interactions:
           attributes:
             title: Updated Title 2
             token: new-token
-            description: Updated Description
             provider: OPENAI
             baseUrl: https://api.updated.com
             llmOrganization: org2

gooddata-sdk/tests/catalog/test_catalog_organization.py

@@ -464,7 +464,6 @@ def test_create_llm_endpoint(test_config):
         full_id = "endpoint2"
         full_title = "Test Endpoint 2"
         full_token = "secret-token-2"
-        full_description = "Test Description"
         full_provider = "OPENAI"
         full_base_url = "https://api.example.com"
         full_llm_org = "org1"
@@ -474,7 +473,6 @@ def test_create_llm_endpoint(test_config):
             id=full_id,
             title=full_title,
             token=full_token,
-            description=full_description,
             provider=full_provider,
             base_url=full_base_url,
             llm_organization=full_llm_org,
@@ -485,8 +483,6 @@ def test_create_llm_endpoint(test_config):
         assert llm_endpoint_2.attributes.title == full_title
         # Token is not returned in the API response for security reasons
         assert not llm_endpoint_2.attributes.token
-        # Description is not returned in the API response. TODO: Check if this is correct.
-        # assert llm_endpoint_2.attributes.description == full_description
         assert llm_endpoint_2.attributes.provider == full_provider
         assert llm_endpoint_2.attributes.base_url == full_base_url
         assert llm_endpoint_2.attributes.llm_organization == full_llm_org
@@ -523,7 +519,6 @@ def test_update_llm_endpoint(test_config):
         # Test with all optional parameters
         full_title = "Updated Title 2"
         full_token = "new-token"
-        full_description = "Updated Description"
         full_provider = "OPENAI"
         full_base_url = "https://api.updated.com"
         full_llm_org = "org2"
@@ -533,7 +528,6 @@ def test_update_llm_endpoint(test_config):
             id=initial_id,
             title=full_title,
             token=full_token,
-            description=full_description,
             provider=full_provider,
             base_url=full_base_url,
             llm_organization=full_llm_org,
@@ -545,8 +539,6 @@ def test_update_llm_endpoint(test_config):
         assert llm_endpoint_2.attributes.title == full_title
         # Token is not returned in the API response for security reasons
         assert not llm_endpoint_2.attributes.token
-        # Description is not returned in the API response. TODO: Check if this is correct.
-        # assert llm_endpoint_2.attributes.description == full_description
         assert llm_endpoint_2.attributes.provider == full_provider
         assert llm_endpoint_2.attributes.base_url == full_base_url
         assert llm_endpoint_2.attributes.llm_organization == full_llm_org
@@ -562,8 +554,6 @@ def test_update_llm_endpoint(test_config):
         # Token is not returned in the API response for security reasons
         assert not llm_endpoint_3.attributes.token
         # Verify other fields are preserved
-        # Description is not returned in the API response. TODO: Check if this is correct.
-        # assert llm_endpoint_3.attributes.description == full_description
         assert llm_endpoint_3.attributes.provider == full_provider
         assert llm_endpoint_3.attributes.base_url == full_base_url
         assert llm_endpoint_3.attributes.llm_organization == full_llm_org