Refactor error handling in HTTPClient and AsyncHTTPClient to provide clearer rate limit messages and maintain exception context

Author: devasheeshG

recallrai/utils/async_http_client.py

@@ -173,7 +173,7 @@ async def request(
                     http_status=response.status_code
                 )
             elif response.status_code == 429:
-                detail = response.json().get("detail", "Please try again in a few moments.")
+                detail = "429 Too Many Requests. Please try again in a few moments."
                 raise RateLimitError(
                     message=detail,
                     http_status=response.status_code
@@ -187,12 +187,12 @@ async def request(
             raise TimeoutError(
                 message=f"Request timed out: {e}",
                 http_status=0  # No HTTP status for timeout
-            )
+            ) from e
         except (ConnectError, JSONDecodeError) as e:
             raise ConnectionError(
                 message=f"Failed to connect to the API: {e}",
                 http_status=0  # No HTTP status for connection error
-            )
+            ) from e
         except Exception as e:
             # Handle other exceptions as needed
             raise e

recallrai/utils/http_client.py

@@ -138,7 +138,7 @@ def request(
                     http_status=response.status_code
                 )
             elif response.status_code == 429:
-                detail = response.json().get("detail", "Please try again in a few moments.")
+                detail = "429 Too Many Requests. Please try again in a few moments."
                 raise RateLimitError(
                     message=detail,
                     http_status=response.status_code
@@ -152,12 +152,12 @@ def request(
             raise TimeoutError(
                 message=f"Request timed out: {e}",
                 http_status=0  # No HTTP status for timeout
-            )
+            ) from e
         except (ConnectError, JSONDecodeError) as e:
             raise ConnectionError(
                 message=f"Failed to connect to the API: {e}",
                 http_status=0  # No HTTP status for connection error
-            )
+            ) from e
         except Exception as e:
             # Handle other exceptions as needed
             raise e