feat: address copilot comments

Author: SoulPancake

openfga_sdk/api_client.py

@@ -321,9 +321,11 @@ async def __call_api(
                     and TelemetryAttributes.fga_client_request_method
                     in _telemetry_attributes
                 ):
-                    e.operation_name = _telemetry_attributes[
+                    operation_name = _telemetry_attributes.get(
                         TelemetryAttributes.fga_client_request_method
-                    ].lower()
+                    )
+                    if isinstance(operation_name, str):
+                        e.operation_name = operation_name.lower()
                 raise e
             except ApiException as e:
                 e.body = e.body.decode("utf-8")
@@ -361,9 +363,11 @@ async def __call_api(
                     and TelemetryAttributes.fga_client_request_method
                     in _telemetry_attributes
                 ):
-                    e.operation_name = _telemetry_attributes[
+                    operation_name = _telemetry_attributes.get(
                         TelemetryAttributes.fga_client_request_method
-                    ].lower()
+                    )
+                    if isinstance(operation_name, str):
+                        e.operation_name = operation_name.lower()
                 raise
 
             self.last_response = response_data

openfga_sdk/exceptions.py

@@ -116,7 +116,9 @@ def __init__(self, msg, path_to_item=None):
 
 
 class ApiException(OpenApiException):
-    def __init__(self, status=None, reason=None, http_resp=None, operation_name=None):
+    def __init__(
+        self, status=None, reason=None, http_resp=None, *, operation_name=None
+    ):
         if http_resp:
             try:
                 headers = http_resp.headers.items()
@@ -297,38 +299,52 @@ def is_server_error(self):
 
 
 class NotFoundException(ApiException):
-    def __init__(self, status=None, reason=None, http_resp=None, operation_name=None):
-        super().__init__(status, reason, http_resp, operation_name)
+    def __init__(
+        self, status=None, reason=None, http_resp=None, *, operation_name=None
+    ):
+        super().__init__(status, reason, http_resp, operation_name=operation_name)
 
 
 class UnauthorizedException(ApiException):
-    def __init__(self, status=None, reason=None, http_resp=None, operation_name=None):
-        super().__init__(status, reason, http_resp, operation_name)
+    def __init__(
+        self, status=None, reason=None, http_resp=None, *, operation_name=None
+    ):
+        super().__init__(status, reason, http_resp, operation_name=operation_name)
 
 
 class ForbiddenException(ApiException):
-    def __init__(self, status=None, reason=None, http_resp=None, operation_name=None):
-        super().__init__(status, reason, http_resp, operation_name)
+    def __init__(
+        self, status=None, reason=None, http_resp=None, *, operation_name=None
+    ):
+        super().__init__(status, reason, http_resp, operation_name=operation_name)
 
 
 class ServiceException(ApiException):
-    def __init__(self, status=None, reason=None, http_resp=None, operation_name=None):
-        super().__init__(status, reason, http_resp, operation_name)
+    def __init__(
+        self, status=None, reason=None, http_resp=None, *, operation_name=None
+    ):
+        super().__init__(status, reason, http_resp, operation_name=operation_name)
 
 
 class ValidationException(ApiException):
-    def __init__(self, status=None, reason=None, http_resp=None, operation_name=None):
-        super().__init__(status, reason, http_resp, operation_name)
+    def __init__(
+        self, status=None, reason=None, http_resp=None, *, operation_name=None
+    ):
+        super().__init__(status, reason, http_resp, operation_name=operation_name)
 
 
 class AuthenticationError(ApiException):
-    def __init__(self, status=None, reason=None, http_resp=None, operation_name=None):
-        super().__init__(status, reason, http_resp, operation_name)
+    def __init__(
+        self, status=None, reason=None, http_resp=None, *, operation_name=None
+    ):
+        super().__init__(status, reason, http_resp, operation_name=operation_name)
 
 
 class RateLimitExceededError(ApiException):
-    def __init__(self, status=None, reason=None, http_resp=None, operation_name=None):
-        super().__init__(status, reason, http_resp, operation_name)
+    def __init__(
+        self, status=None, reason=None, http_resp=None, *, operation_name=None
+    ):
+        super().__init__(status, reason, http_resp, operation_name=operation_name)
 
 
 def render_path(path_to_item):

openfga_sdk/sync/api_client.py

@@ -320,11 +320,12 @@ def __call_api(
                     and TelemetryAttributes.fga_client_request_method
                     in _telemetry_attributes
                 ):
-                    # Convert operation name to lowercase for consistency
-                    e.operation_name = _telemetry_attributes[
+                    operation_name = _telemetry_attributes.get(
                         TelemetryAttributes.fga_client_request_method
-                    ].lower()
-                raise e
+                    )
+                    if isinstance(operation_name, str):
+                        e.operation_name = operation_name.lower()
+                raise
             except ApiException as e:
                 e.body = e.body.decode("utf-8")
                 response_type = response_types_map.get(e.status, None)
@@ -362,9 +363,11 @@ def __call_api(
                     and TelemetryAttributes.fga_client_request_method
                     in _telemetry_attributes
                 ):
-                    e.operation_name = _telemetry_attributes[
+                    operation_name = _telemetry_attributes.get(
                         TelemetryAttributes.fga_client_request_method
-                    ].lower()
+                    )
+                    if isinstance(operation_name, str):
+                        e.operation_name = operation_name.lower()
                 raise
 
             self.last_response = response_data

test/error_integration_test.py

@@ -92,6 +92,7 @@ async def fga_client(self):
         try:
             await client.delete_store()
         except Exception:
+            # Ignore exceptions during cleanup (e.g., store may already be deleted or server unavailable)
             pass  # Ignore cleanup errors
 
     async def test_write_validation_error_invalid_type(self, fga_client):