dispatchbox 0.1.1__tar.gz → 0.1.2__tar.gz

{dispatchbox-0.1.1/dispatchbox.egg-info → dispatchbox-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dispatchbox
-Version: 0.1.1
+Version: 0.1.2
 Summary: Route events to Claude Code and Codex running on your own machines, and ask you only when it matters.
 Author: Xixian Yang
 License-Expression: MIT

{dispatchbox-0.1.1 → dispatchbox-0.1.2}/dispatchbox/__init__.py

@@ -1,3 +1,3 @@
 """dispatchbox: a thin event router that hands work to coding agents on your own machines."""
 
-__version__ = "0.1.1"
+__version__ = "0.1.2"

{dispatchbox-0.1.1 → dispatchbox-0.1.2}/dispatchbox/server.py

@@ -151,7 +151,7 @@ class Handler(BaseHTTPRequestHandler):
         try:
             self._route(method, parts, query)
         except HubError as e:
-            if parts[:1] in (["api"], ["events"], ["claim"], ["hooks"]) or self.headers.get("Accept", "").startswith("application/json"):
+            if self._wants_json(parts):
                 self._json(e.status, {"error": str(e)})
             else:
                 self._html(e.status, page("Error", f"<p>{html.escape(str(e))}</p><p><a href='/'>Back</a></p>"))
@@ -161,6 +161,13 @@ class Handler(BaseHTTPRequestHandler):
             log.exception("request failed")
             self._json(500, {"error": f"internal error: {e.__class__.__name__}"})
 
+    def _wants_json(self, parts: list[str]) -> bool:
+        """API clients get JSON errors; only a browser using the dashboard gets an HTML page."""
+        return (parts[:1] in (["api"], ["events"], ["claim"], ["hooks"])
+                or self.headers.get("Accept", "").startswith("application/json")
+                or self.headers.get("Content-Type", "").startswith("application/json")
+                or self.headers.get("Authorization", "").startswith("Bearer "))
+
     def _route(self, method: str, parts: list[str], q: dict[str, str]) -> None:
         if parts == ["healthz"]:
             return self._json(200, {"ok": True})

{dispatchbox-0.1.1 → dispatchbox-0.1.2/dispatchbox.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dispatchbox
-Version: 0.1.1
+Version: 0.1.2
 Summary: Route events to Claude Code and Codex running on your own machines, and ask you only when it matters.
 Author: Xixian Yang
 License-Expression: MIT

{dispatchbox-0.1.1 → dispatchbox-0.1.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "dispatchbox"
-version = "0.1.1"
+version = "0.1.2"
 description = "Route events to Claude Code and Codex running on your own machines, and ask you only when it matters."
 readme = "README.md"
 requires-python = ">=3.10"

{dispatchbox-0.1.1 → dispatchbox-0.1.2}/tests/test_e2e.py

@@ -130,6 +130,19 @@ class E2ETest(unittest.TestCase):
         self.assertEqual(http("GET", self.base + "/healthz", token=None)[0], 200)
         self.assertEqual(http("POST", self.base + "/events", {"source": "a"})[0], 400)
 
+    def test_api_errors_are_json_without_an_accept_header(self):
+        no_accept = {"Accept": ""}
+        s, body = http("POST", self.base + "/runs/999/decision", {"choice": "x"}, headers=no_accept)
+        self.assertEqual(s, 404)
+        self.assertEqual(json.loads(body), {"error": "run 999 not found"})
+        s, body = http("POST", self.base + "/runs/999/decision", {"choice": "x"}, token=None, headers=no_accept)
+        self.assertEqual(s, 401)
+        self.assertIn("error", json.loads(body))
+        # A browser posting the dashboard form still gets a page.
+        s, body = http("POST", self.base + "/runs/999/decision", token=None, form="choice=x",
+                       headers={"Accept": "text/html"})
+        self.assertTrue(body.lstrip().startswith("<!doctype html>"), body[:80])
+
     def test_full_loop_with_decision_and_resume(self):
         self.start_worker()
         s, body = http("POST", self.base + "/events", {"source": "github", "type": "pr.opened", "subject": "r#1"})