refactor: improve error handling in AgentHumanAPI

- Enhanced exception handling in the AgentHumanAPI class to separately manage APIStatusError and AgentHumanException.
- Added logging for non-retryable APIStatusError to provide better insights during failures.
- Adjusted retry logic to ensure proper handling of exceptions and maintain clarity in error reporting.

Author: jcancinar

livekit-plugins/livekit-plugins-agenthuman/livekit/plugins/agenthuman/api.py

@@ -117,8 +117,15 @@ async def create_session(
                         raise AgentHumanException(
                             f"Unexpected API response structure: {session_data}"
                         ) from e
-            except (APIStatusError, AgentHumanException):
+            except AgentHumanException:
                 raise
+            except APIStatusError as e:
+                last_exc = e
+                if not e.retryable:
+                    raise
+                logger.warning(
+                    "[agenthuman] failed to call agenthuman api", extra={"error": str(e)}
+                )
             except Exception as e:
                 last_exc = e
                 if isinstance(e, APIConnectionError):
@@ -128,9 +135,11 @@ async def create_session(
                 else:
                     logger.exception("[agenthuman] failed to call agenthuman api")
 
-                if i < self._conn_options.max_retry - 1:
-                    await asyncio.sleep(self._conn_options.retry_interval)
+            if i < self._conn_options.max_retry - 1:
+                await asyncio.sleep(self._conn_options.retry_interval)
 
+        if isinstance(last_exc, APIStatusError):
+            raise last_exc
         raise APIConnectionError(
             "Failed to create AgentHuman session after all retries"
         ) from last_exc