outfitter-dispatch 0.2.0__tar.gz → 0.2.1__tar.gz

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: outfitter-dispatch
-Version: 0.2.0
+Version: 0.2.1
 Summary: Local control plane for orchestrating Codex agent lanes over the Codex App Server.
 Project-URL: Homepage, https://github.com/outfitter-dev/dispatch
 Project-URL: Repository, https://github.com/outfitter-dev/dispatch

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/plugins/dispatch/skills/dispatch/SKILL.md

@@ -89,7 +89,9 @@ processes and there is no cross-process write interlock.
 
 Attach is bounded: if the app-server stalls, the underlying `thread/resume`
 times out (~15s) and `attach` fails with a clear `app_server` error, registering
-no lane. There is no half-attached state to clean up.
+no lane. There is no half-attached state to clean up. Large persisted histories
+are supported; attach should not fail only because a resumed thread has more than
+64 KiB of turns.
 
 ## Discover Sessions
 

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "outfitter-dispatch"
-version = "0.2.0"
+version = "0.2.1"
 description = "Local control plane for orchestrating Codex agent lanes over the Codex App Server."
 readme = "README.md"
 requires-python = ">=3.13"

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/src/outfitter/dispatch/client/client.py

@@ -15,7 +15,7 @@ from typing import Self
 
 from pydantic import BaseModel
 
-from .errors import ProtocolError, TransportError
+from .errors import ClientError, TransportError
 from .events import LaneEvent
 from .models import (
     ApprovalPolicy,
@@ -101,9 +101,12 @@ class AppServerClient:
                     if message is None:
                         break
                     self._router.handle(message)
-            except (TransportError, ProtocolError) as exc:
+            except ClientError as exc:
                 self._router.fail_all(exc)
                 return
+            except Exception as exc:
+                self._router.fail_all(TransportError(f"app-server read loop failed: {exc}"))
+                return
             self._router.fail_all(TransportError("app-server stream closed (stdout EOF)"))
         finally:
             self._closed_event.set()  # wake supervisors waiting on wait_closed()

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/src/outfitter/dispatch/client/transport.py

@@ -18,6 +18,8 @@ from typing import Protocol
 
 from .errors import ProtocolError, TransportError
 
+DEFAULT_STDIO_READ_LIMIT = 64 * 1024 * 1024
+
 
 class Transport(Protocol):
     """Bidirectional JSON message channel to one app-server connection."""
@@ -38,9 +40,11 @@ class StdioTransport:
         self,
         argv: Sequence[str] = ("codex", "app-server", "--listen", "stdio://"),
         env: Mapping[str, str] | None = None,
+        read_limit: int = DEFAULT_STDIO_READ_LIMIT,
     ) -> None:
         self._argv = tuple(argv)
         self._env = dict(env) if env is not None else None
+        self._read_limit = read_limit
         self._proc: asyncio.subprocess.Process | None = None
         self._stderr_tail: deque[str] = deque(maxlen=50)
         self._stderr_task: asyncio.Task[None] | None = None
@@ -53,6 +57,7 @@ class StdioTransport:
                 stdout=asyncio.subprocess.PIPE,
                 stderr=asyncio.subprocess.PIPE,
                 env=self._env,
+                limit=self._read_limit,
             )
         except OSError as exc:  # binary missing / not executable
             raise TransportError(f"failed to spawn {self._argv[0]}: {exc}") from exc
@@ -82,7 +87,13 @@ class StdioTransport:
         proc = self._require_proc()
         assert proc.stdout is not None
         while True:
-            raw = await proc.stdout.readline()
+            try:
+                raw = await proc.stdout.readline()
+            except ValueError as exc:
+                raise ProtocolError(
+                    "app-server JSONL line exceeded dispatch's stdio reader limit "
+                    f"({self._read_limit} bytes)"
+                ) from exc
             if raw == b"":  # EOF: process closed stdout (crashed or exited)
                 return None
             text = raw.decode(errors="replace").strip()

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/src/outfitter/dispatch/contracts/derive_cli.py

@@ -125,7 +125,7 @@ def _op_command(
     def command(**kwargs: object) -> None:
         json_requested = bool(kwargs.pop("json", False))
         if op.intent == "destroy":
-            typer.confirm(f"Run destroy op {op.id!r}?", abort=True)
+            typer.confirm(f"Run destroy op {op.id!r}?", abort=True, err=json_requested)
         result = invoke(op.id, dict(kwargs))
         render(op, result)
         _ignore_json(json_requested)

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/src/outfitter/dispatch/core/handlers.py

@@ -9,6 +9,7 @@ from __future__ import annotations
 
 import asyncio
 
+from outfitter.dispatch.client.errors import AppServerError as ClientAppServerError
 from outfitter.dispatch.client.errors import ClientError
 from outfitter.dispatch.client.models import SandboxPolicy, ThreadGoal, ThreadInfo, ThreadSandbox
 from outfitter.dispatch.contracts.context import Ctx
@@ -596,12 +597,21 @@ async def discover(inp: DiscoverInput, ctx: Ctx) -> Discovery:
 async def archive(inp: LaneInput, ctx: Ctx) -> LaneRef:
     lane = await _resolve(ctx, inp.lane)
     _require_writable(lane)  # archiving mutates the shared thread store (ADR-0005)
-    await ctx.client.thread_archive(lane.id)
+    try:
+        await ctx.client.thread_archive(lane.id)
+    except ClientAppServerError as exc:
+        if not _is_no_rollout_archive_error(exc):
+            raise
+        ctx.log.info("lane.archive_local_no_rollout", lane=lane.id)
     await ctx.registry.update_lane_status(lane.id, "archived")
     await ctx.registry.log_action("archive", lane=lane.id)
     return _ref(await ctx.registry.get_lane(lane.id))
 
 
+def _is_no_rollout_archive_error(exc: ClientAppServerError) -> bool:
+    return exc.code == -32600 and "no rollout found" in exc.message.lower()
+
+
 async def status(inp: StatusInput, ctx: Ctx) -> StatusOutput:
     lanes = await ctx.registry.list_lanes()
     triggers = await ctx.registry.list_triggers()

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/tests/client/test_client.py

@@ -7,7 +7,7 @@ import asyncio
 import pytest
 
 from outfitter.dispatch.client.client import AppServerClient
-from outfitter.dispatch.client.errors import AppServerError, TransportError
+from outfitter.dispatch.client.errors import AppServerError, ProtocolError, TransportError
 from outfitter.dispatch.client.events import TurnCompleted
 
 from .conftest import FakeTransport, Responder
@@ -189,6 +189,17 @@ async def test_eof_fails_pending_request_with_transport_error(
         await task
 
 
+async def test_reader_error_fails_pending_request() -> None:
+    class BrokenTransport(FakeTransport):
+        async def receive(self) -> dict[str, object] | None:
+            raise ProtocolError("line too large")
+
+    c = AppServerClient(BrokenTransport())
+    await c.start()
+    with pytest.raises(ProtocolError, match="line too large"):
+        await c.thread_read("L1")
+
+
 async def test_events_stream_yields_projected_lane_event(
     client: tuple[AppServerClient, FakeTransport],
 ) -> None:

outfitter_dispatch-0.2.1/tests/client/test_transport.py

@@ -0,0 +1,63 @@
+"""Unit tests for the stdio App Server transport."""
+
+from __future__ import annotations
+
+import asyncio
+import json
+import sys
+from collections.abc import Sequence
+from typing import Any
+
+import pytest
+
+from outfitter.dispatch.client.transport import StdioTransport
+
+
+async def test_stdio_transport_sets_reader_limit(monkeypatch: pytest.MonkeyPatch) -> None:
+    captured: dict[str, Any] = {}
+
+    class DummyProcess:
+        returncode = None
+        stdin = None
+        stdout = None
+        stderr = None
+
+    async def fake_create_subprocess_exec(*argv: str, **kwargs: Any) -> DummyProcess:
+        captured["argv"] = argv
+        captured["kwargs"] = kwargs
+        return DummyProcess()
+
+    monkeypatch.setattr(asyncio, "create_subprocess_exec", fake_create_subprocess_exec)
+
+    transport = StdioTransport(argv=("codex", "app-server"), read_limit=123_456)
+    await transport.start()
+
+    assert captured["argv"] == ("codex", "app-server")
+    assert captured["kwargs"]["limit"] == 123_456
+    assert captured["kwargs"]["stdin"] is asyncio.subprocess.PIPE
+    assert captured["kwargs"]["stdout"] is asyncio.subprocess.PIPE
+    assert captured["kwargs"]["stderr"] is asyncio.subprocess.PIPE
+
+
+def test_stdio_transport_accepts_sequence_argv() -> None:
+    argv: Sequence[str] = ("codex", "app-server", "--listen", "stdio://")
+    transport = StdioTransport(argv=argv)
+    assert transport.returncode is None
+
+
+async def test_stdio_transport_reads_large_jsonl_line() -> None:
+    payload = {"id": 1, "result": {"text": "x" * 70_000}}
+    script = (
+        "import json, sys; "
+        f"sys.stdout.write({json.dumps(json.dumps(payload))} + '\\n'); "
+        "sys.stdout.flush()"
+    )
+    transport = StdioTransport(
+        argv=(sys.executable, "-c", script),
+        read_limit=128 * 1024,
+    )
+    await transport.start()
+    try:
+        assert await transport.receive() == payload
+    finally:
+        await transport.close()

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/tests/core/test_handlers.py

@@ -9,6 +9,7 @@ from pathlib import Path
 import pytest
 import pytest_asyncio
 
+from outfitter.dispatch.client.errors import AppServerError as ClientAppServerError
 from outfitter.dispatch.client.errors import TransportError
 from outfitter.dispatch.client.models import ThreadGoal, ThreadInfo, ThreadStatus
 from outfitter.dispatch.contracts.errors import AppServerError, AuthorityError, ValidationError
@@ -543,6 +544,26 @@ async def test_roster_then_archive_flips_status(store: Registry) -> None:
     assert len(everything.lanes) == 1
 
 
+class _NoRolloutArchiveClient(FakeLaneClient):
+    async def thread_archive(self, thread_id: str) -> None:
+        self._record("thread_archive", thread_id=thread_id)
+        raise ClientAppServerError(-32600, f"no rollout found for thread id {thread_id}")
+
+
+async def test_archive_no_rollout_lane_marks_local_lane_archived(store: Registry) -> None:
+    client = _NoRolloutArchiveClient()
+    ctx = make_ctx(store, client)
+    await handlers.new_lane(NewInput(name="smoke", ephemeral=True, send=False), ctx)
+
+    archived = await handlers.archive(LaneInput(lane="lane-1"), ctx)
+
+    assert archived.status == "archived"
+    assert (await handlers.roster(RosterInput(), ctx)).lanes == []
+    everything = await handlers.roster(RosterInput(include_archived=True), ctx)
+    assert [lane.id for lane in everything.lanes] == ["lane-1"]
+    assert any(name == "thread_archive" for name, _ in client.calls)
+
+
 async def test_status_and_log_reflect_activity(store: Registry) -> None:
     ctx = make_ctx(store)
     await handlers.open_lane(OpenInput(name="one"), ctx)

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/tests/surfaces/test_derive_cli.py

@@ -2,6 +2,8 @@
 
 from __future__ import annotations
 
+import json
+
 import typer
 from typer.testing import CliRunner
 
@@ -149,6 +151,24 @@ def test_lane_archive_prompts_for_confirmation() -> None:
     assert confirmed.exit_code == 0
 
 
+def test_json_destroy_prompt_does_not_pollute_stdout() -> None:
+    def invoke(op_id: str, params: dict[str, object]) -> dict[str, object]:
+        return {"id": "L1", "handle": "@x", "source": "own", "status": "archived"}
+
+    app = derive_cli(REGISTRY, invoke)
+    result = runner.invoke(app, ["lane", "archive", "L1", "--json"], input="y\n")
+
+    assert result.exit_code == 0
+    assert "Run destroy op" not in result.stdout
+    payload = result.stdout[result.stdout.index("{") :]
+    assert json.loads(payload) == {
+        "id": "L1",
+        "handle": "@x",
+        "source": "own",
+        "status": "archived",
+    }
+
+
 def test_schema_command_prints_derived_schema_without_daemon() -> None:
     app = derive_cli(REGISTRY, lambda _op, _params: {})
 

{outfitter_dispatch-0.2.0 → outfitter_dispatch-0.2.1}/uv.lock

@@ -466,7 +466,7 @@ wheels = [
 
 [[package]]
 name = "outfitter-dispatch"
-version = "0.2.0"
+version = "0.2.1"
 source = { editable = "." }
 dependencies = [
     { name = "aiosqlite" },