glaip-sdk 0.1.1__py3-none-any.whl → 0.1.2__py3-none-any.whl

glaip_sdk/client/base.py

@@ -7,6 +7,7 @@ Authors:
 
 import logging
 import os
+from collections.abc import Iterable, Mapping
 from typing import Any, NoReturn, Union
 
 import httpx
@@ -289,7 +290,7 @@ class BaseClient:
                 return parsed.get("data", parsed) if unwrap else parsed
             else:
                 error_type = parsed.get("error", "UnknownError")
-                message = parsed.get("message", DEFAULT_ERROR_MESSAGE)
+                message = self._format_error_dict({key: value for key, value in parsed.items() if key != "success"})
                 self._raise_api_error(
                     400,
                     message,
@@ -332,12 +333,58 @@ class BaseClient:
                 return validation_message
             return f"Validation error: {parsed}"
 
+        formatted_details = None
+        if "details" in parsed:
+            formatted_details = self._format_error_details(parsed["details"])
+
         message = parsed.get("message")
         if message:
+            if formatted_details:
+                return f"{message}\n{formatted_details}"
             return message
 
+        if formatted_details:
+            return formatted_details
+
         return str(parsed) if parsed else DEFAULT_ERROR_MESSAGE
 
+    def _format_error_details(self, details: Any) -> str | None:
+        """Render generic error details into a human-readable string."""
+        if details is None:
+            return None
+
+        if isinstance(details, dict):
+            return self._format_detail_mapping(details)
+
+        if isinstance(details, (list, tuple, set)):
+            return self._format_detail_iterable(details)
+
+        return f"Details: {details}"
+
+    @staticmethod
+    def _format_detail_mapping(details: Mapping[str, Any]) -> str | None:
+        """Format details provided as a mapping."""
+        entries = [f"  {key}: {value}" for key, value in details.items()]
+        if not entries:
+            return None
+        return "Details:\n" + "\n".join(entries)
+
+    @staticmethod
+    def _format_detail_iterable(details: Iterable[Any]) -> str | None:
+        """Format details provided as an iterable collection."""
+        entries: list[str] = []
+        for item in details:
+            if isinstance(item, Mapping):
+                inner = ", ".join(f"{k}={v}" for k, v in item.items())
+                entries.append(f"  - {inner if inner else '{}'}")
+            else:
+                entries.append(f"  - {item}")
+
+        if not entries:
+            return None
+
+        return "Details:\n" + "\n".join(entries)
+
     def _format_validation_errors(self, errors: list[Any]) -> str | None:
         """Render validation errors into a human-readable string."""
         entries: list[str] = []
@@ -356,6 +403,22 @@ class BaseClient:
 
         return "Validation errors:\n" + "\n".join(entries)
 
+    @staticmethod
+    def _is_no_content_response(response: httpx.Response) -> bool:
+        """Return True when the response contains no content."""
+        return response.status_code == 204
+
+    @staticmethod
+    def _is_success_status(response: httpx.Response) -> bool:
+        """Return True for successful HTTP status codes."""
+        return 200 <= response.status_code < 300
+
+    def _handle_error_response(self, response: httpx.Response) -> None:
+        """Raise an API error for non-success responses."""
+        error_message = self._get_error_message(response)
+        parsed_content = self._parse_response_content(response)
+        self._raise_api_error(response.status_code, error_message, payload=parsed_content)
+
     def _handle_response(
         self,
         response: httpx.Response,
@@ -364,22 +427,17 @@ class BaseClient:
     ) -> Any:
         """Handle HTTP response with proper error handling."""
         # Handle no-content success before general error handling
-        if response.status_code == 204:
+        if self._is_no_content_response(response):
             return None
 
-        # Handle error status codes
-        if not (200 <= response.status_code < 300):
-            error_message = self._get_error_message(response)
-            # Try to parse response content for payload
-            parsed_content = self._parse_response_content(response)
-            self._raise_api_error(response.status_code, error_message, payload=parsed_content)
-            return None  # Won't be reached but helps with type checking
-
-        parsed = self._parse_response_content(response)
-        if parsed is None:
-            return None
+        if self._is_success_status(response):
+            parsed = self._parse_response_content(response)
+            if parsed is None:
+                return None
+            return self._handle_success_response(parsed, unwrap=unwrap)
 
-        return self._handle_success_response(parsed, unwrap=unwrap)
+        self._handle_error_response(response)
+        return None
 
     def _raise_api_error(
         self,

{glaip_sdk-0.1.1.dist-info → glaip_sdk-0.1.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: glaip-sdk
-Version: 0.1.1
+Version: 0.1.2
 Summary: Python SDK for GL AIP (GDP Labs AI Agent Package) - Simplified CLI Design
 License: MIT
 Author: Raymond Christopher

{glaip_sdk-0.1.1.dist-info → glaip_sdk-0.1.2.dist-info}/RECORD

@@ -39,7 +39,7 @@ glaip_sdk/cli/validators.py,sha256=Squ2W-fMz9kfvhtTt7pCcAYnzFU28ZxxTEqH1vF9r00,5
 glaip_sdk/client/__init__.py,sha256=nYLXfBVTTWwKjP0e63iumPYO4k5FifwWaELQPaPIKIg,188
 glaip_sdk/client/_agent_payloads.py,sha256=QjaSqXZvDNX0bQVKC7Eb5nHj7KxhU0zLSfNUQiKf2J8,16269
 glaip_sdk/client/agents.py,sha256=hRkt-D2ZRkeWamPqKsqKwl74ZjScKL4UHLbbVezGCWc,37597
-glaip_sdk/client/base.py,sha256=-GC5P5fB9V5lqyCuEJoxD3KBvCT98TzlMvFFFjD6kIc,16047
+glaip_sdk/client/base.py,sha256=ikW33raz2M6rXzo3JmhttfXXuVdMv5zBRKEZkU1F-4I,18176
 glaip_sdk/client/main.py,sha256=tELAA36rzthnNKTgwZ6lLPb3Au8Wh1mF8Kz-9N-YtCg,8652
 glaip_sdk/client/mcps.py,sha256=GQ1EBTSVc-WrFigyw8iocK34DT3TMW85XtnDeOawP9E,8945
 glaip_sdk/client/run_rendering.py,sha256=T6NF3SjTWSV6IpU2TexBq8AFKU1yq98NLsTp1N4B2yk,12209
@@ -76,7 +76,7 @@ glaip_sdk/utils/resource_refs.py,sha256=OpwJxTCKq3RIqLAmKE0xspJplD4IhKqLCo0OqPaf
 glaip_sdk/utils/run_renderer.py,sha256=d_VMI6LbvHPUUeRmGqh5wK_lHqDEIAcym2iqpbtDad0,1365
 glaip_sdk/utils/serialization.py,sha256=cvlPQ73y4jhsbJeQ6Bv7vaq0NV8uZ27p2vD_R-nyiRQ,12646
 glaip_sdk/utils/validation.py,sha256=NPDexNgGUIoLkEIz6hl3K6EG7ZKSEkcNLDElqm8-Ng4,7019
-glaip_sdk-0.1.1.dist-info/METADATA,sha256=_D9k3juoxlATe8KReWJkjTTwVf6hOH8Q82ooCyj7YMY,6023
-glaip_sdk-0.1.1.dist-info/WHEEL,sha256=sP946D7jFCHeNz5Iq4fL4Lu-PrWrFsgfLXbbkciIZwg,88
-glaip_sdk-0.1.1.dist-info/entry_points.txt,sha256=EGs8NO8J1fdFMWA3CsF7sKBEvtHb_fujdCoNPhfMouE,47
-glaip_sdk-0.1.1.dist-info/RECORD,,
+glaip_sdk-0.1.2.dist-info/METADATA,sha256=VQWPGOU1y8-yuf8bUrODnK24DVrobD9GJ_wJxAfhybk,6023
+glaip_sdk-0.1.2.dist-info/WHEEL,sha256=sP946D7jFCHeNz5Iq4fL4Lu-PrWrFsgfLXbbkciIZwg,88
+glaip_sdk-0.1.2.dist-info/entry_points.txt,sha256=EGs8NO8J1fdFMWA3CsF7sKBEvtHb_fujdCoNPhfMouE,47
+glaip_sdk-0.1.2.dist-info/RECORD,,