drukbox-python-sdk 0.5.0__tar.gz → 0.5.1__tar.gz

{drukbox_python_sdk-0.5.0 → drukbox_python_sdk-0.5.1}/AGENTS.md

@@ -57,6 +57,7 @@ run.
   consumers unless the user asks for them.
 - Preserve user changes in the worktree; do not clean or rewrite unrelated
   files.
-- A version bump rides alone: one commit (`chore: bump to X`) that touches
-  only `pyproject.toml`, with no other change in it. Never fold the bump into
-  a feature or fix commit. The release tag is cut from the bump commit.
+- A version bump rides alone: its own commit (`chore: bump to X`) that
+  touches only `pyproject.toml`, in its own PR with no other change in it.
+  Never fold the bump into a feature or fix commit or PR. The release tag is
+  cut from the bump commit.

{drukbox_python_sdk-0.5.0 → drukbox_python_sdk-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: drukbox-python-sdk
-Version: 0.5.0
+Version: 0.5.1
 Summary: Async Python client for Drukbox.
 Project-URL: Homepage, https://github.com/clawhaven/drukbox-python-sdk
 Project-URL: Repository, https://github.com/clawhaven/drukbox-python-sdk

{drukbox_python_sdk-0.5.0 → drukbox_python_sdk-0.5.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "drukbox-python-sdk"
-version = "0.5.0"
+version = "0.5.1"
 description = "Async Python client for Drukbox."
 readme = "README.md"
 license = "MIT"

{drukbox_python_sdk-0.5.0 → drukbox_python_sdk-0.5.1}/src/drukbox_sdk/api.py

@@ -626,11 +626,11 @@ class SandboxAPI:
             raise SandboxUnavailableError(json_response.get("detail", "service unavailable"))
 
         if response.status_code == 422:
-            # FastAPI's 422 detail is a list of error dicts; join the messages
-            # so it reads as a line, not a raw list repr.
-            errors = json_response.get("detail") or []
-            detail = "; ".join(error.get("msg", "") for error in errors) or "validation error"
-            raise SandboxValidationError(detail)
+            # The detail is FastAPI's list of error dicts or the service's own sentence.
+            detail = json_response.get("detail")
+            if isinstance(detail, list):
+                detail = "; ".join(error.get("msg", "") for error in detail)
+            raise SandboxValidationError(detail or "validation error")
 
         if response.status_code >= 400:
             raise SandboxResponseError(json_response.get("detail", "error"))

{drukbox_python_sdk-0.5.0 → drukbox_python_sdk-0.5.1}/tests/test_api.py

@@ -744,6 +744,20 @@ async def test_422_raises_sandbox_validation_error(api: SandboxAPI):
         await api.create_host(env={"FOO BAR": "x"})
 
 
+@respx.mock
+async def test_422_string_detail_raises_sandbox_validation_error(api: SandboxAPI):
+    # Drukbox answers a refused secret with the issuer's reason as a plain string.
+    respx.post(f"{BASE_URL}/hosts").mock(
+        return_value=httpx.Response(
+            422,
+            json={"detail": "GitHub App not installed for acme", "error_code": "SECRET_REFUSED"},
+        ),
+    )
+
+    with pytest.raises(SandboxValidationError, match=r"^GitHub App not installed for acme$"):
+        await api.create_host()
+
+
 @respx.mock
 async def test_500_raises_sandbox_response_error(api: SandboxAPI):
     respx.get(f"{BASE_URL}/hosts").mock(