code-coordinator 0.5.529__tar.gz → 0.5.530__tar.gz

{code_coordinator-0.5.529 → code_coordinator-0.5.530}/.gitignore

@@ -20,6 +20,7 @@ coordinator.yml
 .pytest.out
 .pytest.rerun.out
 .pytest.baseline.out
+.pytest.baseline-confirm.out
 .pytest-acceptance.out
 .pytest-populated-home.out
 .cargo.out

{code_coordinator-0.5.529 → code_coordinator-0.5.530}/PKG-INFO

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

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

@@ -902,6 +902,7 @@ tests/fixtures/test_report/cargo_compile_error.txt
 tests/fixtures/test_report/cargo_multi_target_failures.txt
 tests/fixtures/test_report/cargo_no_parseable_failures.txt
 tests/fixtures/test_report/pytest_collection_error.txt
+tests/fixtures/test_report/pytest_error_log_record.txt
 tests/fixtures/test_report/pytest_error_string_in_body.txt
 tests/fixtures/test_report/pytest_internal_error.txt
 tests/fixtures/test_report/pytest_mixed_failures.txt

{code_coordinator-0.5.529 → code_coordinator-0.5.530}/code_coordinator.egg-info/scm_file_list.json

@@ -438,6 +438,7 @@
     "tests/fixtures/test_report/cargo_multi_target_failures.txt",
     "tests/fixtures/test_report/cargo_no_parseable_failures.txt",
     "tests/fixtures/test_report/pytest_collection_error.txt",
+    "tests/fixtures/test_report/pytest_error_log_record.txt",
     "tests/fixtures/test_report/pytest_error_string_in_body.txt",
     "tests/fixtures/test_report/pytest_internal_error.txt",
     "tests/fixtures/test_report/pytest_mixed_failures.txt",

{code_coordinator-0.5.529 → code_coordinator-0.5.530}/code_coordinator.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.529",
+  "tag": "0.5.530",
   "distance": 0,
-  "node": "g3f472c582029adafcec0ee12c3abf4df1ccd9b88",
+  "node": "g8cda204e6985ec5ea0eca258916e347e9f330342",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-09-28"

{code_coordinator-0.5.529 → code_coordinator-0.5.530}/coord/db.py

@@ -1065,7 +1065,29 @@ def retry_on_locked(
 # can measure a row's own spend from ITS baseline rather than every
 # predecessor row's lifetime total. See `coord.drive_queue.QueueEntry.
 # legs_at_enqueue`.
-_DB_SCHEMA_VERSION = 20
+#
+# #3470: bumped 20 -> 21 for the two new indexes appended to `_SCHEMA_SQL`
+# below — not a column, same shape as #3113/#3333's table-only bumps (
+# `_MIGRATE_ADD_COLUMNS`'s length is unchanged): `idx_assignments_repo_issue`
+# on `(repo_name, issue_number)` and `idx_assignments_dispatched_at` on
+# `dispatched_at`, for the `issue-cost` report's full-history scan of
+# `assignments` + `assignments_archive` (`coord.reports.
+# _default_issue_cost_source`). `CREATE INDEX IF NOT EXISTS` in `_SCHEMA_SQL`
+# is what actually creates them on an existing database, and that only runs
+# when `_ensure_schema` runs, which is gated on this version bump.
+#
+# `assignments_archive`'s own two matching indexes are NOT here:
+# `_ensure_schema`/`_SCHEMA_SQL` only ever runs against tables that exist at
+# schema-init time, and `assignments_archive` is created lazily by `coord
+# housekeeping` (`coord.housekeeping._ensure_archive_mirror`) — a database
+# that reaches this version before its first housekeeping sweep has no such
+# table yet, so a schema-version-gated `CREATE INDEX ... ON
+# assignments_archive(...)` here could never reliably retrofit onto every
+# database. They're created instead at the one chokepoint that actually
+# knows the table exists: `_ensure_archive_mirror` itself, unconditionally
+# and idempotently (`CREATE INDEX IF NOT EXISTS`) on every housekeeping
+# sweep — see that function's own comment in coord/housekeeping.py.
+_DB_SCHEMA_VERSION = 21
 
 
 def _read_schema_version(conn: sqlite3.Connection) -> int:
@@ -1961,6 +1983,18 @@ _SCHEMA_SQL = """
 
         CREATE INDEX IF NOT EXISTS idx_assignments_status ON assignments(status);
         CREATE INDEX IF NOT EXISTS idx_assignments_machine ON assignments(machine_name);
+        -- #3470: the `issue-cost` report's full-history scan groups every
+        -- leg by (repo_name, issue_number) and orders by dispatched_at —
+        -- see coord.reports._default_issue_cost_source /
+        -- coord.usage_rollup.rollup's IssueKey grouping. `assignments_archive`
+        -- gets the same two indexes via `_migrate_archive_indexes` below,
+        -- since that table is created dynamically by `coord housekeeping`
+        -- (coord.housekeeping._ensure_archive_mirror) rather than by this
+        -- script, and may not exist yet on a fresh database.
+        CREATE INDEX IF NOT EXISTS idx_assignments_repo_issue
+            ON assignments(repo_name, issue_number);
+        CREATE INDEX IF NOT EXISTS idx_assignments_dispatched_at
+            ON assignments(dispatched_at);
         CREATE INDEX IF NOT EXISTS idx_merge_queue_state ON merge_queue(state);
         CREATE INDEX IF NOT EXISTS idx_issue_context_issue
             ON issue_context(repo_name, issue_number);

{code_coordinator-0.5.529 → code_coordinator-0.5.530}/coord/housekeeping.py

@@ -130,10 +130,37 @@ def _columns(conn: sqlite3.Connection, table: str) -> list[tuple[str, str]]:
     return sql.table_columns(conn, table)
 
 
+# #3470: index(es) to create once a given archive table's mirror exists,
+# keyed by DESTINATION table name so `_ensure_archive_mirror`'s generic
+# column-mirroring below stays generic across every archive table this
+# module knows about — only `assignments_archive` needs these, for the
+# `issue-cost` report's full-history scan
+# (`coord.reports._default_issue_cost_source`, which groups every leg by
+# `(repo_name, issue_number)` and orders by `dispatched_at`). Same two
+# columns `coord/db.py`'s `_SCHEMA_SQL` already indexes on the LIVE
+# `assignments` table — see that module's `_DB_SCHEMA_VERSION` bump comment
+# for why the archive side can't just live there too: this table is created
+# lazily, by this module, not at `_ensure_schema` time.
+_ARCHIVE_INDEXES: dict[str, tuple[str, ...]] = {
+    _ASSIGNMENTS_ARCHIVE: (
+        "CREATE INDEX IF NOT EXISTS idx_assignments_archive_repo_issue "
+        "ON assignments_archive(repo_name, issue_number)",
+        "CREATE INDEX IF NOT EXISTS idx_assignments_archive_dispatched_at "
+        "ON assignments_archive(dispatched_at)",
+    ),
+}
+
+
 def _ensure_archive_mirror(conn: sqlite3.Connection, src: str, dst: str) -> list[str]:
     """Create/extend *dst* so it has every column of *src* (no constraints — the
     archive is dumb storage).  Robust to future ``ALTER TABLE`` on *src*.
 
+    Also applies :data:`_ARCHIVE_INDEXES` for *dst*, if any — idempotent
+    (``CREATE INDEX IF NOT EXISTS``) and cheap, so this runs on every sweep
+    that touches *dst*, not just the one that creates it: an archive table
+    that already existed before #3470 shipped still converges on the next
+    sweep, and a fresh one is indexed from its very first row.
+
     Returns the shared column-name list to use for the copy.
     """
     src_cols = _columns(conn, src)
@@ -145,6 +172,8 @@ def _ensure_archive_mirror(conn: sqlite3.Connection, src: str, dst: str) -> list
         for name, ctype in src_cols:
             if name not in dst_existing:
                 sql.execute(conn, f'ALTER TABLE {dst} ADD COLUMN "{name}" {ctype}')  # noqa: S608
+    for ddl in _ARCHIVE_INDEXES.get(dst, ()):
+        sql.execute(conn, ddl)
     return [name for name, _ in src_cols]