code-coordinator 0.5.273__tar.gz → 0.5.275__tar.gz

{code_coordinator-0.5.273 → code_coordinator-0.5.275}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-coordinator
-Version: 0.5.273
+Version: 0.5.275
 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.273 → code_coordinator-0.5.275}/code_coordinator.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-coordinator
-Version: 0.5.273
+Version: 0.5.275
 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.273 → code_coordinator-0.5.275}/code_coordinator.egg-info/SOURCES.txt

@@ -130,6 +130,7 @@ coord/release_window.py
 coord/repo_edit.py
 coord/repo_onboard.py
 coord/reports.py
+coord/rest_schema.py
 coord/revalidate.py
 coord/review.py
 coord/scorecard.py
@@ -141,6 +142,7 @@ coord/split_work.py
 coord/sql.py
 coord/stage_projection.py
 coord/state.py
+coord/store_migrate.py
 coord/test_author.py
 coord/test_chat.py
 coord/test_orchestrator.py
@@ -181,6 +183,7 @@ coord/commands/scorecard.py
 coord/commands/sessions.py
 coord/commands/setup.py
 coord/commands/status.py
+coord/commands/store_migrate.py
 coord/commands/terminal.py
 coord/commands/test_gate.py
 coord/commands/tui.py
@@ -705,6 +708,7 @@ tests/test_scorecard.py
 tests/test_self_health.py
 tests/test_serve.py
 tests/test_serve_app_board_trim.py
+tests/test_serve_rest_routes.py
 tests/test_session.py
 tests/test_slice_issue_attribution.py
 tests/test_smoke.py
@@ -724,6 +728,7 @@ tests/test_state_review_findings_daemon.py
 tests/test_status_fleet_footer.py
 tests/test_stop_1567.py
 tests/test_store_contract.py
+tests/test_store_migrate.py
 tests/test_stuck.py
 tests/test_stuck_test_state_watchdog_2803.py
 tests/test_terminal_command.py
@@ -754,6 +759,8 @@ tests/test_worker_safety.py
 tests/test_worker_test_scoping.py
 tests/test_worktree.py
 tests/test_worktree_rmtree_chokepoint.py
+tests/test_write_parity.py
+tests/write_parity.py
 tests/acceptance/ms-33/audit_1039.rs
 tests/acceptance/ms-33/contract.md
 tests/acceptance/ms-33/manifest.yml

{code_coordinator-0.5.273 → code_coordinator-0.5.275}/code_coordinator.egg-info/scm_file_list.json

@@ -77,6 +77,7 @@
     "coord/commands/sessions.py",
     "coord/commands/setup.py",
     "coord/commands/status.py",
+    "coord/commands/store_migrate.py",
     "coord/commands/terminal.py",
     "coord/commands/test_gate.py",
     "coord/commands/tui.py",
@@ -230,6 +231,7 @@
     "coord/repo_edit.py",
     "coord/repo_onboard.py",
     "coord/reports.py",
+    "coord/rest_schema.py",
     "coord/revalidate.py",
     "coord/review.py",
     "coord/scorecard.py",
@@ -251,6 +253,7 @@
     "coord/sql.py",
     "coord/stage_projection.py",
     "coord/state.py",
+    "coord/store_migrate.py",
     "coord/test_author.py",
     "coord/test_chat.py",
     "coord/test_orchestrator.py",
@@ -777,6 +780,7 @@
     "tests/test_self_health.py",
     "tests/test_serve.py",
     "tests/test_serve_app_board_trim.py",
+    "tests/test_serve_rest_routes.py",
     "tests/test_session.py",
     "tests/test_slice_issue_attribution.py",
     "tests/test_smoke.py",
@@ -796,6 +800,7 @@
     "tests/test_status_fleet_footer.py",
     "tests/test_stop_1567.py",
     "tests/test_store_contract.py",
+    "tests/test_store_migrate.py",
     "tests/test_stuck.py",
     "tests/test_stuck_test_state_watchdog_2803.py",
     "tests/test_terminal_command.py",
@@ -826,6 +831,8 @@
     "tests/test_worker_test_scoping.py",
     "tests/test_worktree.py",
     "tests/test_worktree_rmtree_chokepoint.py",
+    "tests/test_write_parity.py",
+    "tests/write_parity.py",
     "tui/Cargo.lock",
     "tui/Cargo.toml",
     "tui/cargo-config-local-quadraui.toml.example",

{code_coordinator-0.5.273 → code_coordinator-0.5.275}/code_coordinator.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.273",
+  "tag": "0.5.275",
   "distance": 0,
-  "node": "gd5980777425030eea6f688b62a154d8312a34199",
+  "node": "gecadae381e27fba9b8107a833cbce6a8ba67bc3c",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-28"

{code_coordinator-0.5.273 → code_coordinator-0.5.275}/coord/cli.py

@@ -147,6 +147,7 @@ from coord.commands.release import release_group, release_preflight
 # #2220: `coord repo add` / `coord repo doctor` — onboarding a repo, and
 # verifying it actually happened across all five layers.
 from coord.commands.repo import repo_group
+from coord.commands.store_migrate import migrate_to_postgres
 from coord.commands.plan_followup import (
     _dispatch_followup,  # noqa: F401 — re-exported for tests
     approve_plan,
@@ -491,6 +492,7 @@ main.add_command(release_preflight)
 main.add_command(release_group)
 # #2220: repo onboarding + its verifier.
 main.add_command(repo_group)
+main.add_command(migrate_to_postgres)
 main.add_command(tui_group)
 
 

code_coordinator-0.5.275/coord/commands/store_migrate.py

@@ -0,0 +1,105 @@
+"""``coord migrate-to-postgres`` -- one-shot import of an existing SQLite
+``coord.db`` into the Postgres store #827's ``store:`` config seam names
+(#828 second half). See ``coord.store_migrate`` for the actual audit/copy
+logic this command is a thin CLI wrapper around.
+"""
+
+from __future__ import annotations
+
+from pathlib import Path
+
+import click
+
+from coord import db as coord_db
+from coord import sql
+from coord.store_migrate import ImportAborted, run_import
+
+
+@click.command(
+    "migrate-to-postgres",
+    help=(
+        "Import an existing SQLite coord.db into the Postgres store configured "
+        "by coordinator.yml's store: block (#828). Audits referential integrity "
+        "and type-affinity drift before writing anything."
+    ),
+)
+@click.option(
+    "--source",
+    "source_path",
+    type=click.Path(path_type=Path),
+    default=None,
+    help="Path to the source coord.db (default: this install's DB_PATH).",
+)
+@click.option(
+    "--dsn",
+    default=None,
+    help="Target Postgres DSN (default: coordinator.yml's store.dsn).",
+)
+@click.option(
+    "--force",
+    is_flag=True,
+    help="Wipe and re-import any target tables that already have rows (default: refuse).",
+)
+@click.option(
+    "--dry-run",
+    is_flag=True,
+    help="Audit the source and report row counts without opening or writing the target.",
+)
+def migrate_to_postgres(
+    source_path: Path | None, dsn: str | None, force: bool, dry_run: bool
+) -> None:
+    source_path = source_path or coord_db.DB_PATH
+    if dsn is None and not dry_run:
+        target = coord_db._resolve_store_target()
+        if not target.dsn:
+            raise click.ClickException(
+                "no --dsn given and coordinator.yml has no store.dsn configured -- "
+                "pass --dsn explicitly, or set store.backend: postgres / store.dsn "
+                "in coordinator.yml first (#827)."
+            )
+        dsn = target.dsn
+
+    click.echo(f"Source: {source_path}")
+    # Never echo the raw DSN -- it routinely embeds a password
+    # (`store.dsn`'s plain-string shape, coord/config.py), and this command's
+    # stdout is exactly the kind of text that can end up in a log or a
+    # GitHub comment (coord.db._migrate_if_needed's docstring states the
+    # policy this follows). sql.redact_dsn() prints host/dbname only.
+    target_line = sql.redact_dsn(dsn) if dsn else "(dry-run -- target is never opened)"
+    click.echo(f"Target: {target_line}")
+
+    try:
+        report = run_import(sqlite_path=source_path, dsn=dsn or "", force=force, dry_run=dry_run)
+    except ImportAborted as exc:
+        raise click.ClickException(str(exc)) from exc
+
+    click.echo()
+    if report.dry_run:
+        # The target is never opened for a dry run, so TableReport.target_rows
+        # is a placeholder 0 -- printing a source/target/status table here
+        # would read every table as a false "MISMATCH". Report source counts
+        # only; the audits (referential integrity, type affinity) already ran
+        # and would have raised ImportAborted above if either found anything.
+        click.echo(f"{'table':<30} {'source rows':>12}")
+        for t in report.tables:
+            click.echo(f"{t.table:<30} {t.source_rows:>12}")
+        click.echo(
+            "\nAudits passed (referential integrity, type affinity). "
+            "Dry run -- no target opened, no rows written."
+        )
+        return
+
+    click.echo(f"{'table':<30} {'source':>10} {'target':>10}  status")
+    all_ok = True
+    for t in report.tables:
+        status = "OK" if t.ok else "MISMATCH"
+        if not t.ok:
+            all_ok = False
+        click.echo(f"{t.table:<30} {t.source_rows:>10} {t.target_rows:>10}  {status}")
+
+    if not all_ok:
+        raise click.ClickException(
+            "row-count parity mismatch after import -- see table above; the target "
+            "was written but does not match the source row-for-row"
+        )
+    click.echo("\nImport complete: row-count parity confirmed for every table.")

{code_coordinator-0.5.273 → code_coordinator-0.5.275}/coord/db.py

@@ -361,6 +361,20 @@ def _open_postgres(dsn: str) -> Any:
     return conn
 
 
+def open_postgres_connection(dsn: str) -> Any:
+    """Public entry point onto :func:`_open_postgres` for tooling that needs
+    a fully-migrated Postgres connection outside the ``get_connection()``
+    singleton -- today, only ``coord.store_migrate``'s importer (#828), which
+    opens a target that may not (yet) be the DSN this process's own
+    ``coordinator.yml`` names, so it calls this directly with whatever DSN
+    the operator gave it rather than going through :func:`get_connection`'s
+    config-resolution path. Same guarantees as every other caller of
+    :func:`_open_postgres`: refuses under pytest, and runs the same schema
+    creation/migration path SQLite gets (#827 item 3).
+    """
+    return _open_postgres(dsn)
+
+
 # #2752: a clean, tagged release is always exactly "X.Y.Z" -- setuptools_scm
 # only emits anything else (a ".devN+g<sha>" local/dev suffix, a bare
 # "0+unknown" when no distribution resolves at all, or git-describe's

code_coordinator-0.5.275/coord/rest_schema.py

@@ -0,0 +1,227 @@
+"""Explicit wire DTOs for the daemon's resource-shaped routes (#1944).
+
+Phase B of the Store Service milestone (#60) adds resource-shaped routes
+*alongside* the ~50 RPC ones:
+
+===============================================  ==================================
+resource route                                   RPC routes it will eventually replace
+===============================================  ==================================
+``PATCH /issue/{repo_name}/{number}``            ``/issue-edit``, ``/issue-label``,
+                                                 ``/issue-labels``, ``/issue-milestone``,
+                                                 ``/issue-milestone-remove``,
+                                                 ``/issue-close``, ``/issue-reopen``
+``POST  /issue/{repo_name}/{number}/comments``   ``/issue-comment``, ``/issue-comments``
+``GET   /issue/{repo_name}/{number}/comments``   ``GET /issue-comments``
+``PATCH /assignment/{assignment_id}``            ``/assignment-usage``,
+                                                 ``/assignment-session-id``,
+                                                 ``/assignment-failure-reason``
+===============================================  ==================================
+
+Two read-only RPC routes are *already* covered by the resource reads that
+predate this issue and so need nothing new: ``POST /assignment-test-plan`` is
+``GET /assignment/{id}``'s ``test_plan`` field, and ``POST /issue-test-mode``
+is derivable from ``GET /issue/{repo}/{n}``'s ``labels`` via
+:func:`coord.models.test_mode_from_labels`.  Both are marked deprecated in the
+spec with that pointer rather than being re-added in a PATCH shape they do not
+have.
+
+**Nothing is removed here.** Every RPC route keeps working byte-identically;
+retirement is gated on the zero-usage telemetry #1945 adds.
+
+Why dataclasses rather than hand-written JSON Schema dicts: the same reason
+``coord/board_schema.py`` exists (#1849) — the declared Python type is the
+contract, ``coord.openapi.dataclass_schema`` renders it into
+``components/schemas``, and ``scripts/codegen.py`` can generate clients from
+it.  The RPC bodies these replace are hand-assembled dicts documented by
+hand-written schema literals in ``serve_app.openapi_spec()``; that is exactly
+the drift this milestone is closing.
+
+**Absent is not null.**  A PATCH is a *partial* update, so the handlers parse
+the raw request dict and test membership (``"milestone" in body``) rather than
+round-tripping through these dataclasses — that is the only way to tell
+"leave the milestone alone" (absent) from "clear the milestone" (``null``).
+The dataclasses below therefore describe the wire, and
+:func:`unknown_fields` is what enforces that a client cannot smuggle a typo'd
+field name past a handler that would silently ignore it.
+"""
+
+from __future__ import annotations
+
+import dataclasses
+from dataclasses import dataclass, field
+
+# ── PATCH /issue/{repo_name}/{number} ────────────────────────────────────────
+
+
+@dataclass(kw_only=True)
+class IssuePatch:
+    """Partial update of one issue.  Every field is optional; omit to leave alone.
+
+    Mutations are applied in a **fixed order** regardless of key order in the
+    request body — content, then labels, then milestone, then state — so that
+    a single PATCH that both relabels and closes an issue behaves the same way
+    every time.  ``applied`` in the response reports what actually ran, in
+    that order.
+    """
+
+    #: New title (``/issue-edit``). ``None``/absent leaves it unchanged.
+    title: str | None = None
+    #: New body (``/issue-edit``). ``None``/absent leaves it unchanged.
+    body: str | None = None
+    #: Labels to add on the tracker (``/issue-label`` ``add``).
+    add_labels: list[str] | None = None
+    #: Labels to remove on the tracker (``/issue-label`` ``remove``).
+    remove_labels: list[str] | None = None
+    #: Replace the *cached* label set outright (``/issue-labels``).  This is a
+    #: local-mirror write, not a tracker write — it is what ``coord sync``
+    #: uses.  Mutually exclusive with ``add_labels``/``remove_labels``.
+    labels: list[str] | None = None
+    #: Milestone number to assign (``/issue-milestone``).  An explicit JSON
+    #: ``null`` *clears* the milestone (``/issue-milestone-remove``); omitting
+    #: the key leaves it alone.
+    milestone: int | None = None
+    #: Optional title for the assigned milestone (``/issue-milestone``).
+    milestone_title: str | None = None
+    #: ``"closed"`` (``/issue-close``) or ``"open"`` (``/issue-reopen``).
+    state: str | None = None
+    #: Comment to post alongside a ``state`` change.  Only meaningful with
+    #: ``state``; to comment without changing state use
+    #: ``POST /issue/{repo}/{n}/comments``.
+    comment: str | None = None
+    #: ``/issue-close``'s ``force`` — bypass the open-children guard.
+    force: bool | None = None
+    #: ``owner/repo`` override for the tracker call, as on every RPC route.
+    repo_github: str | None = None
+
+
+@dataclass(kw_only=True)
+class IssuePatchResult:
+    """Response to ``PATCH /issue/{repo_name}/{number}``."""
+
+    #: True when at least one mutation ran.
+    updated: bool
+    #: The mutations that ran, in application order — e.g.
+    #: ``["content", "add_labels", "state"]``.
+    applied: list[str] = field(default_factory=list)
+    #: The resulting label set, when labels were touched (``/issue-label``'s
+    #: ``labels``); ``null`` otherwise.
+    labels: list[str] | None = None
+    #: Whether the label mutation changed anything (``/issue-label``'s
+    #: ``changed``); ``null`` when labels were not touched.
+    labels_changed: bool | None = None
+
+
+# ── POST /issue/{repo_name}/{number}/comments ────────────────────────────────
+
+
+@dataclass(kw_only=True)
+class IssueCommentCreate:
+    """Create (or mirror) one comment on an issue.
+
+    ``action`` selects which of the two RPC comment paths this stands in for:
+
+    * ``"post"`` (the default) — write the comment through the tracker seam,
+      the ``/issue-comment`` path.
+    * ``"capture"`` — record a comment the caller already posted into the
+      durable ``issue_comments`` mirror (``/issue-comments`` ``capture``).
+    * ``"sync"`` — backfill the mirror from the tracker
+      (``/issue-comments`` ``sync``); ``body`` is not read.
+    """
+
+    #: The comment text.  Required for ``post`` and ``capture``.
+    body: str | None = None
+    #: ``"post"`` | ``"capture"`` | ``"sync"``.  Defaults to ``"post"``.
+    action: str | None = None
+    #: ``capture`` only: the tracker's own comment id, when known.
+    gh_comment_id: int | None = None
+    #: ``capture`` only: comment author.
+    author: str | None = None
+    #: ``capture`` only: epoch-seconds creation timestamp.
+    created_at: float | None = None
+    #: ``owner/repo`` override for the tracker call.
+    repo_github: str | None = None
+
+
+@dataclass(kw_only=True)
+class IssueCommentResult:
+    """Response to ``POST /issue/{repo_name}/{number}/comments``."""
+
+    ok: bool
+    #: Which action ran — echoed so a client that omitted it sees the default.
+    action: str
+    #: ``sync`` only: how many comments were pulled into the mirror.
+    synced: int | None = None
+
+
+@dataclass(kw_only=True)
+class IssueCommentList:
+    """Response to ``GET /issue/{repo_name}/{number}/comments``."""
+
+    #: Oldest-first, exactly as ``GET /issue-comments`` returns them.
+    comments: list[dict] = field(default_factory=list)
+
+
+# ── PATCH /assignment/{assignment_id} ────────────────────────────────────────
+
+
+@dataclass(kw_only=True)
+class AssignmentPatch:
+    """Partial update of one assignment row.  Every field is optional.
+
+    Covers the three RPC field-setters — ``/assignment-usage`` (the first ten
+    fields), ``/assignment-session-id`` and ``/assignment-failure-reason``.
+    """
+
+    cost_usd: float | None = None
+    input_tokens: int | None = None
+    output_tokens: int | None = None
+    cache_creation_tokens: int | None = None
+    cache_read_tokens: int | None = None
+    num_turns: int | None = None
+    #: 0/1 flag on the wire, never a JSON bool downstream (#1849) — but the
+    #: request side accepts a JSON boolean, matching ``/assignment-usage``.
+    is_interactive: bool | None = None
+    smoke_tests: list[str] | None = None
+    completion_summary: str | None = None
+    stop_reason: str | None = None
+    #: ``/assignment-session-id``.
+    claude_session_id: str | None = None
+    #: ``/assignment-failure-reason`` — also flips the row to ``failed``.
+    failure_reason: str | None = None
+
+
+@dataclass(kw_only=True)
+class AssignmentPatchResult:
+    """Response to ``PATCH /assignment/{assignment_id}``."""
+
+    updated: bool
+    #: The mutations that ran, in application order.
+    applied: list[str] = field(default_factory=list)
+
+
+# ── validation helper ────────────────────────────────────────────────────────
+
+#: The token-ish ``/assignment-usage`` fields that are written as one group.
+USAGE_TOKEN_FIELDS = (
+    "input_tokens",
+    "output_tokens",
+    "cache_creation_tokens",
+    "cache_read_tokens",
+    "num_turns",
+)
+
+
+def declared_fields(cls: type) -> frozenset[str]:
+    """The set of field names *cls* declares on the wire."""
+    return frozenset(f.name for f in dataclasses.fields(cls))
+
+
+def unknown_fields(cls: type, body: dict) -> list[str]:
+    """Keys in *body* that *cls* does not declare, sorted.
+
+    A PATCH that silently ignores a misspelled key is worse than one that
+    refuses it: the caller gets a 200 and believes the write landed.  The
+    RPC routes tolerate unknown keys (they read the ones they want out of a
+    hand-assembled dict); the resource routes do not.
+    """
+    return sorted(set(body) - declared_fields(cls))