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

{drukbox_python_sdk-0.4.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.4.0 → drukbox_python_sdk-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: drukbox-python-sdk
-Version: 0.4.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.4.0 → drukbox_python_sdk-0.5.1}/pyproject.toml

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

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

@@ -198,7 +198,8 @@ class DoctorReport:
 
     ok: bool
     active_provider: str
-    tailscale_enabled: bool
+    # Whether the active provider's hosts join the tailnet.
+    tailnet: bool
     checks: list[DoctorCheck]
 
 
@@ -490,7 +491,8 @@ class SandboxAPI:
         return DoctorReport(
             ok=data["ok"],
             active_provider=data["active_provider"],
-            tailscale_enabled=data["tailscale_enabled"],
+            # drukbox before "Join the tailnet per provider" sent tailscale_enabled.
+            tailnet=data["tailnet"] if "tailnet" in data else data["tailscale_enabled"],
             checks=[DoctorCheck(**check) for check in data["checks"]],
         )
 
@@ -624,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.4.0 → drukbox_python_sdk-0.5.1}/tests/test_api.py

@@ -48,7 +48,7 @@ def _doctor_payload(**overrides: Any) -> dict[str, Any]:
     payload: dict[str, Any] = {
         "ok": True,
         "active_provider": "aws",
-        "tailscale_enabled": True,
+        "tailnet": True,
         "checks": [
             {
                 "name": "db",
@@ -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(
@@ -925,6 +939,17 @@ def test_from_env_strips_trailing_slash_via_constructor(
     assert sandbox.base_url == "https://trailing.test"
 
 
+@respx.mock
+async def test_doctor_reads_tailscale_enabled_from_an_older_drukbox(api: SandboxAPI):
+    payload = _doctor_payload()
+    payload["tailscale_enabled"] = payload.pop("tailnet")
+    respx.get(f"{BASE_URL}/doctor").mock(return_value=httpx.Response(200, json=payload))
+
+    report = await api.doctor()
+
+    assert report.tailnet is True
+
+
 @respx.mock
 async def test_doctor_parses_report_and_checks(api: SandboxAPI):
     route = respx.get(f"{BASE_URL}/doctor").mock(
@@ -937,7 +962,7 @@ async def test_doctor_parses_report_and_checks(api: SandboxAPI):
     assert route.calls.last.request.headers["Authorization"] == "Bearer t-test"
     assert report.ok is True
     assert report.active_provider == "aws"
-    assert report.tailscale_enabled is True
+    assert report.tailnet is True
     assert [check.name for check in report.checks] == ["db", "provider", "tailscale"]
     provider = next(check for check in report.checks if check.name == "provider")
     assert provider.status == "ok"