code-coordinator 0.5.298__tar.gz → 0.5.300__tar.gz

{code_coordinator-0.5.298 → code_coordinator-0.5.300}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-coordinator
-Version: 0.5.298
+Version: 0.5.300
 Summary: Coordinate Claude Code workers across multiple machines and repos.
 Author-email: John Donaghy <johnfdonaghy@gmail.com>
 License-Expression: LicenseRef-FSL-1.1-MIT

{code_coordinator-0.5.298 → code_coordinator-0.5.300}/code_coordinator.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-coordinator
-Version: 0.5.298
+Version: 0.5.300
 Summary: Coordinate Claude Code workers across multiple machines and repos.
 Author-email: John Donaghy <johnfdonaghy@gmail.com>
 License-Expression: LicenseRef-FSL-1.1-MIT

{code_coordinator-0.5.298 → code_coordinator-0.5.300}/code_coordinator.egg-info/SOURCES.txt

@@ -223,9 +223,11 @@ coord/health/registry.py
 coord/health/render.py
 coord/health/units.py
 coord/health/checks/__init__.py
+coord/health/checks/_ci_pin.py
 coord/health/checks/agent_install.py
 coord/health/checks/cargo_targets.py
 coord/health/checks/claude_binary.py
+coord/health/checks/coord_tui_ci_pin.py
 coord/health/checks/coord_web_ci_pin.py
 coord/health/checks/deploy_lane_facts.py
 coord/health/checks/disk.py
@@ -299,6 +301,7 @@ docs/ADR_COORD_TUI_CI.md
 docs/ADR_COORD_WEB_ACCEPTANCE_SUITE.md
 docs/ADR_COORD_WEB_CI.md
 docs/ADR_COORD_WEB_DIST.md
+docs/ADR_PY_RUST_MIRRORS.md
 docs/AGENT_OPERATIONS.md
 docs/ARCHITECTURE.md
 docs/ARCH_SECURITY_GATES.md
@@ -572,6 +575,7 @@ tests/test_gates.py
 tests/test_gather_fleet_tmux_sessions.py
 tests/test_gen_mock_index.py
 tests/test_generated_rust_fixture.py
+tests/test_generated_rust_requests.py
 tests/test_generated_types_fixture.py
 tests/test_github_ops.py
 tests/test_github_throttle.py
@@ -584,6 +588,7 @@ tests/test_health_cli.py
 tests/test_health_cli_graph.py
 tests/test_health_config.py
 tests/test_health_context.py
+tests/test_health_coord_tui_ci_pin.py
 tests/test_health_coord_web_ci_pin.py
 tests/test_health_deploy_lane_facts.py
 tests/test_health_fix.py
@@ -669,6 +674,7 @@ tests/test_provider_seam.py
 tests/test_providers.py
 tests/test_providers_config_labels.py
 tests/test_providers_pty.py
+tests/test_py_rust_mirror_inventory.py
 tests/test_quiet_hours.py
 tests/test_quiet_hours_store_2146.py
 tests/test_reap_merged_sessions.py

{code_coordinator-0.5.298 → code_coordinator-0.5.300}/code_coordinator.egg-info/scm_file_list.json

@@ -140,9 +140,11 @@
     "coord/health/__init__.py",
     "coord/health/aggregate.py",
     "coord/health/checks/__init__.py",
+    "coord/health/checks/_ci_pin.py",
     "coord/health/checks/agent_install.py",
     "coord/health/checks/cargo_targets.py",
     "coord/health/checks/claude_binary.py",
+    "coord/health/checks/coord_tui_ci_pin.py",
     "coord/health/checks/coord_web_ci_pin.py",
     "coord/health/checks/deploy_lane_facts.py",
     "coord/health/checks/disk.py",
@@ -291,6 +293,7 @@
     "docs/ADR_COORD_WEB_ACCEPTANCE_SUITE.md",
     "docs/ADR_COORD_WEB_CI.md",
     "docs/ADR_COORD_WEB_DIST.md",
+    "docs/ADR_PY_RUST_MIRRORS.md",
     "docs/AGENT_OPERATIONS.md",
     "docs/ARCHITECTURE.md",
     "docs/ARCH_SECURITY_GATES.md",
@@ -598,6 +601,7 @@
     "tests/test_gather_fleet_tmux_sessions.py",
     "tests/test_gen_mock_index.py",
     "tests/test_generated_rust_fixture.py",
+    "tests/test_generated_rust_requests.py",
     "tests/test_generated_types_fixture.py",
     "tests/test_github_ops.py",
     "tests/test_github_throttle.py",
@@ -610,6 +614,7 @@
     "tests/test_health_cli_graph.py",
     "tests/test_health_config.py",
     "tests/test_health_context.py",
+    "tests/test_health_coord_tui_ci_pin.py",
     "tests/test_health_coord_web_ci_pin.py",
     "tests/test_health_deploy_lane_facts.py",
     "tests/test_health_fix.py",
@@ -695,6 +700,7 @@
     "tests/test_providers.py",
     "tests/test_providers_config_labels.py",
     "tests/test_providers_pty.py",
+    "tests/test_py_rust_mirror_inventory.py",
     "tests/test_quiet_hours.py",
     "tests/test_quiet_hours_store_2146.py",
     "tests/test_reap_merged_sessions.py",

code_coordinator-0.5.300/code_coordinator.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "0.5.300",
+  "distance": 0,
+  "node": "g94e4815fd8f9649dea8eb46637198841f2c99a16",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-08-30"
+}

{code_coordinator-0.5.298 → code_coordinator-0.5.300}/coord/client.py

@@ -199,6 +199,44 @@ def post_board(svc: ServiceConfig, board: Board, *, timeout: float = _WRITE_TIME
     post_record(svc, "/board", serialize_board(board), timeout=timeout)
 
 
+def _reraise_with_detail(exc: httpx.HTTPStatusError) -> httpx.HTTPStatusError:
+    """#2907: fold the daemon's own ``{"error", "detail"}`` error body into the
+    raised exception's message.
+
+    ``serve_app.py``'s handlers answer a failure with the HTTP status code and
+    a ``{"error": "<label>", "detail": "<reason>"}`` body (see e.g.
+    ``post_assignment_session_id``) — plain ``resp.raise_for_status()``
+    discards that body, so every one of :func:`post_record`'s ~10 callers,
+    which all just catch ``httpx.HTTPError``/``Exception`` and render
+    ``str(e)``, only ever saw the generic status line ("503 Service
+    Unavailable"), never the daemon's own reason (e.g. a GitHub rate-limit
+    backoff active for another N seconds).
+
+    Mirrors ``coord.dispatch._reraise_with_body`` (#1527), the same fix for
+    the sibling ``agent_app.py`` assign-rejection body: always returns a
+    **new** ``httpx.HTTPStatusError`` carrying the *same* ``request``/
+    ``response`` as *exc*, so every existing ``except httpx.HTTPStatusError``
+    that itself inspects ``exc.response.status_code`` / ``exc.response.json()``
+    (e.g. ``coord.issue_store.post_result``, ``coord.state.close_issue``)
+    keeps working unchanged — only the message gains the detail.
+
+    ``detail`` is preferred over ``error`` when both are present, since it
+    carries the specific cause rather than the handler's own short label. A
+    non-JSON/empty body, or one with neither key, falls back to *exc*'s
+    original status-line message unchanged — degrading, never crashing,
+    while handling a failure that's already in flight.
+    """
+    reason = None
+    try:
+        body = exc.response.json()
+    except ValueError:
+        body = None
+    if isinstance(body, dict):
+        reason = body.get("detail") or body.get("error")
+    message = f"{exc} — {reason}" if reason else str(exc)
+    return httpx.HTTPStatusError(message, request=exc.request, response=exc.response)
+
+
 def post_record(
     svc: ServiceConfig,
     path: str,
@@ -213,11 +251,20 @@ def post_record(
     writing the local DB, so a thin client's result lands on the one shared DB.
     Raises ``httpx.HTTPError`` on transport/HTTP failure — the seam decides
     whether that is fatal (``post_result``) or best-effort (``post_completion``).
+
+    #2907: an HTTP error response's message carries the daemon's own
+    ``detail``/``error`` reason (see :func:`_reraise_with_detail`) rather than
+    just the generic status line — a transport failure (connection refused,
+    timeout) never reaches this and is unaffected, since there is no response
+    body to read.
     """
     resp = httpx.post(
         f"{svc.url}{path}", json=payload, headers=_headers(svc), timeout=timeout
     )
-    resp.raise_for_status()
+    try:
+        resp.raise_for_status()
+    except httpx.HTTPStatusError as exc:
+        raise _reraise_with_detail(exc) from exc
     return resp.json()
 
 

code_coordinator-0.5.300/coord/health/checks/_ci_pin.py

@@ -0,0 +1,236 @@
+"""Shared machinery for the cross-repo "which `coord` does *their* CI install?" checks.
+
+Extracted from :mod:`coord.health.checks.coord_web_ci_pin` (#2006) when #2900
+added its second consumer, :mod:`coord.health.checks.coord_tui_ci_pin`. Both
+answer the same question about a different downstream repo, so the parsing —
+walk a checkout's workflow YAML, find the ``pip install`` steps, pull the
+`coord` requirement out of each — is identical and lives here once. What is
+*not* shared is the grading: what counts as a fault depends on why that repo
+installs `coord` at all, and each check module owns its own verdicts.
+
+Nothing in here is a check. The module is underscore-prefixed on purpose:
+:func:`coord.health.registry.discover` skips ``_``-prefixed modules, so this
+cannot accidentally register anything.
+
+**Local-filesystem only.** Every function here reads a checkout already on
+this machine. No ``gh`` call, no network — cheap enough for the health poll
+tick, and absent on a machine with no such checkout, which each caller
+reports as OK ("not present on this machine") the same way every other
+checkout-derived lane reports absence.
+"""
+
+from __future__ import annotations
+
+import re
+from collections.abc import Iterator
+from dataclasses import dataclass
+from pathlib import Path
+
+import yaml
+
+#: The extra that carries uvicorn/Starlette. Since the base/`[server]` split
+#: (#1237) the bare distribution installs a client-only coord: it cannot
+#: serve anything, and it cannot even *import* ``coord.serve_app`` (Starlette
+#: is imported at module scope there). Both downstream repos need the extra —
+#: coord-web to boot ``coord web --fixture`` as its Playwright ``webServer``
+#: (#1818), coord-tui because ``scripts/codegen.py --rust`` reads
+#: ``coord.serve_app.openapi_spec()`` (#1941).
+SERVER_EXTRA = "server"
+
+#: PyPI cannot rename a project, so ``claude-coordinator`` is a permanent
+#: tombstone that will never gain another release (#2106). CI still asking
+#: for it does not get a stale coord — it gets whatever ancient version that
+#: tombstone last published, forever.
+TOMBSTONE_DIST = "claude-coordinator"
+
+#: The live distribution name.
+LIVE_DIST = "code-coordinator"
+
+_INSTALL_RE = re.compile(r"\b(?:pip3?|uv\s+pip|pipx)\s+install\b", re.IGNORECASE)
+
+# `code-coordinator[server]>=0.4.90` and friends. Deliberately tolerant of
+# quoting (`'code-coordinator[server]'`) and of the underscore spelling pip
+# normalises away, because this is reading someone else's hand-written YAML.
+_REQ_RE = re.compile(
+    r"(?P<dist>code[-_]coordinator|claude[-_]coordinator)"
+    r"(?:\s*\[(?P<extras>[^\]]*)\])?"
+    r"(?P<constraint>"
+    r"(?:\s*(?:===|[<>!~=]=|[<>])\s*[^\s'\",;\\]+)"
+    r"(?:\s*,\s*(?:===|[<>!~=]=|[<>])\s*[^\s'\",;\\]+)*"
+    r")?",
+    re.IGNORECASE,
+)
+
+
+@dataclass(frozen=True)
+class CoordPin:
+    """One ``pip install <coord>`` requirement found in a downstream repo's CI."""
+
+    workflow: str  # file name, e.g. "ci.yml"
+    job: str  # job id, e.g. "e2e"
+    spec: str  # the requirement as written, e.g. "code-coordinator[server]"
+    dist: str  # normalised distribution name
+    extras: tuple[str, ...] = ()
+    constraint: str | None = None  # ">=0.4.90", "==0.4.90", or None
+
+    @property
+    def has_server_extra(self) -> bool:
+        return SERVER_EXTRA in self.extras
+
+    @property
+    def is_tombstone(self) -> bool:
+        return self.dist == TOMBSTONE_DIST
+
+    @property
+    def is_exact_pin(self) -> bool:
+        """True for ``==``/``===``, the spec shape that rots silently.
+
+        ``~=`` is deliberately included: ``~=0.4.90`` freezes the minor
+        series just as effectively for a project on a single ``0.x`` line.
+        """
+        c = (self.constraint or "").lstrip()
+        return c.startswith(("==", "===", "~="))
+
+    @property
+    def floor(self) -> str | None:
+        """The ``>=`` version, when the spec declares one."""
+        for part in (self.constraint or "").split(","):
+            part = part.strip()
+            if part.startswith(">="):
+                return part[2:].strip()
+        return None
+
+
+def norm_dist(raw: str) -> str:
+    """``Coord_TUI`` -> ``coord-tui``; ``code_coordinator`` -> ``code-coordinator``."""
+    return raw.strip().lower().replace("_", "-")
+
+
+def _version_tuple(s: str) -> tuple[int, ...] | None:
+    parts = re.findall(r"\d+", s or "")
+    return tuple(int(p) for p in parts) if parts else None
+
+
+def release_tuple(s: str) -> tuple[int, ...] | None:
+    """A comparable tuple, but only for a version that is a real release.
+
+    ``coord.__version__`` falls back to ``0+unknown`` when it cannot resolve
+    a distribution (a source checkout with no install, a wheel-less CI box).
+    Comparing a CI floor against *that* would report every floor as "ahead of
+    this machine" — a fabricated finding, and fabricating findings is how a
+    check earns the right to be ignored. Unknown stays unknown.
+    """
+    if not s or "unknown" in s.lower():
+        return None
+    t = _version_tuple(s)
+    return t if t and len(t) >= 2 else None
+
+
+def floor_exceeds(floor: str, local: str) -> bool:
+    """True iff *floor* is strictly newer than *local*, both being releases.
+
+    Compares at equal precision — a floor of ``0.4`` is satisfied by a local
+    ``0.4.91``, so the shorter tuple is padded rather than compared as-is
+    (``(0, 4, 91) > (0, 4)`` would otherwise read as "ahead").
+    """
+    f, loc = release_tuple(floor), release_tuple(local)
+    if not f or not loc:
+        return False
+    width = max(len(f), len(loc))
+
+    def pad(t: tuple[int, ...]) -> tuple[int, ...]:
+        return t + (0,) * (width - len(t))
+
+    return pad(f) > pad(loc)
+
+
+def iter_run_steps(checkout_path: Path) -> Iterator[tuple[str, str, str]]:
+    """Yield ``(workflow_file_name, job_id, run_script)`` for every step.
+
+    A sibling of :func:`coord.health.checks.toolchain._iter_workflow_steps`,
+    but keeping the workflow/job labels: a finding here has to name *where*
+    in someone else's repo to go fix it, or it is just as much of a mystery
+    as the thing it replaced.
+    """
+    workflows_dir = checkout_path / ".github" / "workflows"
+    if not workflows_dir.is_dir():
+        return
+    try:
+        files = sorted(workflows_dir.glob("*.yml")) + sorted(workflows_dir.glob("*.yaml"))
+    except OSError:
+        return
+    for wf_file in files:
+        try:
+            doc = yaml.safe_load(wf_file.read_text(encoding="utf-8", errors="replace"))
+        except (OSError, yaml.YAMLError):
+            continue
+        if not isinstance(doc, dict):
+            continue
+        jobs = doc.get("jobs")
+        if not isinstance(jobs, dict):
+            continue
+        for job_id, job in jobs.items():
+            if not isinstance(job, dict):
+                continue
+            for step in job.get("steps") or []:
+                if not isinstance(step, dict):
+                    continue
+                run = step.get("run")
+                if isinstance(run, str) and run.strip():
+                    yield wf_file.name, str(job_id), run
+
+
+def find_coord_pins(checkout_path: Path) -> list[CoordPin]:
+    """Every ``pip install <coord>`` requirement in the checkout's workflows.
+
+    Only ``run:`` scripts that actually invoke an installer are scanned — a
+    comment or an echo naming the package is not a pin, and treating it as
+    one would grade prose.
+    """
+    found: list[CoordPin] = []
+    seen: set[tuple[str, str, str]] = set()
+    for workflow, job, run in iter_run_steps(checkout_path):
+        for line in run.splitlines():
+            if not _INSTALL_RE.search(line):
+                continue
+            for m in _REQ_RE.finditer(line):
+                extras = tuple(
+                    e.strip().lower() for e in (m.group("extras") or "").split(",") if e.strip()
+                )
+                constraint = (m.group("constraint") or "").strip() or None
+                spec = m.group(0).strip()
+                key = (workflow, job, spec)
+                if key in seen:
+                    continue
+                seen.add(key)
+                found.append(
+                    CoordPin(
+                        workflow=workflow,
+                        job=job,
+                        spec=spec,
+                        dist=norm_dist(m.group("dist")),
+                        extras=extras,
+                        constraint=constraint,
+                    )
+                )
+    return found
+
+
+def where(pin: CoordPin) -> str:
+    """``ci.yml:e2e`` — the workflow:job a finding should send you to."""
+    return f"{pin.workflow}:{pin.job}"
+
+
+def pin_values(pins: list[CoordPin]) -> list[dict]:
+    """The JSON-able ``values["pins"]`` payload both checks report."""
+    return [
+        {
+            "workflow": p.workflow,
+            "job": p.job,
+            "spec": p.spec,
+            "dist": p.dist,
+            "extras": list(p.extras),
+            "constraint": p.constraint,
+        }
+        for p in pins
+    ]