chore: Add defensive check for response status

Author: anubhav756

packages/toolbox-core/src/toolbox_core/client.py

@@ -193,6 +193,11 @@ async def load_tool(
         # request the definition of the tool from the server
         url = f"{self.__base_url}/api/tool/{name}"
         async with self.__session.get(url, headers=resolved_headers) as response:
+            if not response.ok:
+                error_text = await response.text()
+                raise RuntimeError(
+                    f"API request failed with status {response.status} ({response.reason}). Server response: {error_text}"
+                )
             json = await response.json()
         manifest: ManifestSchema = ManifestSchema(**json)
 

packages/toolbox-core/src/toolbox_core/tool.py

@@ -289,7 +289,7 @@ async def __call__(self, *args: Any, **kwargs: Any) -> str:
             headers=headers,
         ) as resp:
             body = await resp.json()
-            if resp.status < 200 or resp.status >= 300:
+            if not resp.ok:
                 err = body.get("error", f"unexpected status from server: {resp.status}")
                 raise Exception(err)
         return body.get("result", body)