code-coordinator 0.5.205__tar.gz → 0.5.206__tar.gz

{code_coordinator-0.5.205 → code_coordinator-0.5.206}/PKG-INFO

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

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

@@ -316,6 +316,7 @@ scripts/codegen.py
 scripts/coord-test-runner.sh
 scripts/drive-batch.sh
 scripts/gen_board_fixture.py
+scripts/gen_mock_index.py
 scripts/next_release_tag.py
 scripts/run_tests_in_populated_home.sh
 scripts/run_tests_without_gh.sh
@@ -511,6 +512,7 @@ tests/test_gate_b.py
 tests/test_gate_snapshot_staleness.py
 tests/test_gates.py
 tests/test_gather_fleet_tmux_sessions.py
+tests/test_gen_mock_index.py
 tests/test_generated_types_fixture.py
 tests/test_github_ops.py
 tests/test_goal.py

{code_coordinator-0.5.205 → code_coordinator-0.5.206}/code_coordinator.egg-info/scm_file_list.json

@@ -320,6 +320,7 @@
     "scripts/coord-test-runner.sh",
     "scripts/drive-batch.sh",
     "scripts/gen_board_fixture.py",
+    "scripts/gen_mock_index.py",
     "scripts/next_release_tag.py",
     "scripts/run_tests_in_populated_home.sh",
     "scripts/run_tests_without_gh.sh",
@@ -573,6 +574,7 @@
     "tests/test_gate_snapshot_staleness.py",
     "tests/test_gates.py",
     "tests/test_gather_fleet_tmux_sessions.py",
+    "tests/test_gen_mock_index.py",
     "tests/test_generated_types_fixture.py",
     "tests/test_github_ops.py",
     "tests/test_goal.py",

{code_coordinator-0.5.205 → code_coordinator-0.5.206}/code_coordinator.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.205",
+  "tag": "0.5.206",
   "distance": 0,
-  "node": "g464b6f72ce8e60436c28ceba3d8453892108cb88",
+  "node": "g3c2392941a4c874655cc8ccc15a0ef5112a0d822",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-21"

{code_coordinator-0.5.205 → code_coordinator-0.5.206}/coord/mock_author.py

@@ -49,6 +49,32 @@ from coord.milestone_dispatch import pick_machine
 from coord.models import Machine, Proposal
 
 
+def _wants_mock_index(driver_mock_glob: str) -> bool:
+    """#2512: the navigation-index post-render step only makes sense for
+    HTML mocks — an `index.html` full of `<a href>`s to `.screen` text-grid
+    dumps (tui-tuidriver) or whatever cli-pytest renders isn't navigable in
+    a browser anyway. Gate on the mock glob rather than the driver name so
+    this stays correct if a future driver also renders `.html`."""
+    return driver_mock_glob.strip().lower().endswith(".html")
+
+
+def _mock_index_instruction(ms_dir: str) -> str:
+    """#2512: the instruction text both briefing builders below hand the
+    mock-author worker, verbatim — a *provided script* the worker runs as
+    its last step before committing, not something it free-hands per
+    milestone (see `scripts/gen_mock_index.py`'s docstring for why: every
+    Gate-A mock set should look the same, and nothing should drift by
+    taste)."""
+    return (
+        f"As your LAST step before committing, run `python "
+        f"scripts/gen_mock_index.py {ms_dir}/mocks` to (re)generate "
+        f"`{ms_dir}/mocks/index.html` — a plain navigation page linking "
+        "every mock by its own `<title>` tag (#2512). Do not hand-write "
+        "this file yourself; the script is the single source of truth so "
+        "every milestone's mock set gets the same glue page."
+    )
+
+
 def build_mock_author_briefing(
     *,
     repo_slug: str,
@@ -94,6 +120,8 @@ def build_mock_author_briefing(
         "not authoring it from scratch — read it first and edit in place. "
         "Commit and push both to this branch when done."
     )
+    if _wants_mock_index(driver_mock_glob):
+        parts.append(_mock_index_instruction(ms_dir))
     return "\n".join(parts)
 
 
@@ -104,6 +132,7 @@ def build_mock_author_amend_briefing(
     milestone_number: int,
     tracking_issue_number: int,
     amend_text: str,
+    driver_mock_glob: str,
 ) -> str:
     """#1315: seed briefing for a *targeted* Gate-A contract amendment.
 
@@ -120,6 +149,15 @@ def build_mock_author_amend_briefing(
     the ``tests/acceptance/**`` sealing guard — see ``coord/dispatch.py``
     and #1315's ``_sealed_write_guard_tools`` in ``coord/agent.py``), so a
     contract correction never needs to fall back to ``type="work"`` again.
+
+    *driver_mock_glob* (#2512) is the repo's resolved acceptance-driver mock
+    glob (e.g. ``"*.html"``, ``"*.screen"``) — already resolved by the
+    caller (:func:`dispatch_acceptance_mock` always resolves ``driver_cfg``
+    before either briefing builder runs). Threaded through purely to gate
+    whether the mock-navigation-index instruction below is worth including;
+    an amendment that only touches ``contract.md`` still gets the
+    instruction, phrased conditionally, since this function has no way to
+    know in advance whether the correction will touch ``mocks/``.
     """
     ms_dir = f"tests/acceptance/{ms_dirname(milestone_number)}"
 
@@ -137,6 +175,16 @@ def build_mock_author_amend_briefing(
         f"do not touch any file outside `{ms_dir}/`. Commit and push both "
         "to this branch when done."
     )
+    if _wants_mock_index(driver_mock_glob):
+        parts.append(
+            f"If this correction touches any file under `{ms_dir}/mocks/`, "
+            f"regenerate `{ms_dir}/mocks/index.html` before committing by "
+            f"running `python scripts/gen_mock_index.py {ms_dir}/mocks` "
+            "(#2512) — a plain navigation page linking every mock by its "
+            "own `<title>` tag. Do not hand-write this file yourself; the "
+            "script is the single source of truth so every milestone's "
+            "mock set gets the same glue page."
+        )
     parts.append("")
     parts.append("--- Requested correction ---")
     parts.append(amend_text.strip())
@@ -260,6 +308,7 @@ def dispatch_acceptance_mock(
             milestone_number=milestone_number,
             tracking_issue_number=tracking_issue_number,
             amend_text=amend_briefing,
+            driver_mock_glob=driver_cfg.mock,
         )
         proposal = Proposal(
             id=0,

{code_coordinator-0.5.205 → code_coordinator-0.5.206}/docs/ORACLE_LOOP.md

@@ -141,11 +141,40 @@ One command, framework-agnostic above the driver:
 tests/acceptance/ms-NN/
   contract.md          # the black-box surface: CLI names, key screen text, API field shapes
   mocks/               # the viewable mocks == the assertion fixtures (*.screen / *.html / …)
+    index.html          # HTML drivers only — navigation glue, see below
   <suite files>        # red at Gate A, extended JIT per issue; SEALED to the worker
   manifest.(yml|json)  # test-id → issue-slice mapping (drives --issue N)
                         # + optional expected_red: {issue: [test-id, ...]} — see below
 ```
 
+**`mocks/index.html` — navigation glue between the flat mock files (#2512).**
+A rendered mock set for an HTML driver (`web-playwright` and any future
+HTML-based driver — gated on the mock glob ending `.html`, not the driver
+name) is a pile of disconnected files: `reports-grid.html`,
+`reports-chart.html`, and so on, each self-contained and each only
+reachable if you already know its name. This is deliberately **not** the
+"clickable mocks" question (deferred — `docs/CUSTOMER_PORTAL.md:152-157`
+still holds: mocks stay self-contained static HTML, no build step, no
+framework, no live data). It is pure navigation: a plain `<ul>` of
+`<a href>` links, one per mock file, labelled from that mock's own
+`<title>` tag.
+
+This index is generated by a **deterministic script**
+(`scripts/gen_mock_index.py`), not hand-authored by the mock-author LLM —
+the same "run a provided script, don't freehand it" posture this repo
+already uses for sealed-suite tooling, so every milestone's mock set gets
+the identical glue page and nothing drifts by taste between renders.
+`coord/mock_author.py`'s seed briefings (`build_mock_author_briefing` for a
+fresh render, `build_mock_author_amend_briefing` for `--amend`) tell the
+mock-author worker to run it as the last step before committing, whenever
+the resolved driver's mock glob is HTML. Because it is a plain `.html` file
+under `mocks/`, it needs no special-casing anywhere downstream:
+`collect_mock_bundle_files` (PDR-3, #2508) already picks up every
+`mocks/*.html` file when reading a merged Gate-A branch back into a design
+round, so `index.html` rides along into the portal bundle for free. It is
+also what the local-view TUI action (#2501) should open instead of a single
+mock file, once that lands.
+
 ## A sealed slice is red by design — the `expected_red` registry (#2164)
 
 A sealed acceptance slice is **red by design** the moment it's authored — its fix doesn't exist yet.

code_coordinator-0.5.206/scripts/gen_mock_index.py

@@ -0,0 +1,107 @@
+"""Deterministic post-render step for Gate-A HTML mock sets (#2512).
+
+A Gate-A render produces one flat, disconnected `.html` file per
+screen-state under `tests/acceptance/ms-NN/mocks/` (`coord/mock_author.py`)
+— there was no way to get from one to another except knowing the filename
+and opening it directly. This script closes that gap with pure navigation
+glue: it scans `mocks/*.html` and writes `mocks/index.html`, a plain list
+of links, one per mock, labelled from each mock's own `<title>` tag (every
+mock already carries a descriptive one — see the `MOCK_AUTHOR_SYSTEM_PROMPT`
+rule in `coord/agent.py`).
+
+This is deliberately a *script the mock-author worker runs*, not something
+the mock-author LLM free-hands per milestone — the same "run a provided
+script, don't hand-roll it" posture this repo already uses for sealed-suite
+tooling (`scripts/coord-test-runner.sh`). Every Gate-A mock set gets the
+exact same glue page, generated the exact same way, so nothing drifts by
+taste between milestones or repos.
+
+Zero JS, no framework, no live data — `mocks/index.html` is still
+"self-contained static HTML" per docs/CUSTOMER_PORTAL.md's mocks-stay-static
+rule; it just links between already-static siblings.
+
+Usage:
+    python scripts/gen_mock_index.py tests/acceptance/ms-NN/mocks
+
+`coord/mock_author.py`'s seed briefings (`build_mock_author_briefing` /
+`build_mock_author_amend_briefing`) tell the mock-author worker to run
+exactly this, as the last step before committing.
+"""
+from __future__ import annotations
+
+import html
+import re
+import sys
+from pathlib import Path
+
+INDEX_NAME = "index.html"
+
+_TITLE_RE = re.compile(r"<title[^>]*>(.*?)</title>", re.IGNORECASE | re.DOTALL)
+
+
+def _extract_title(page_html: str, *, fallback: str) -> str:
+    """Pull the `<title>` text out of a rendered mock, collapsing internal
+    whitespace/newlines. Falls back to the filename when a mock is missing
+    a `<title>` tag entirely (should not happen per the mock-author system
+    prompt, but a missing label beats a crashed script)."""
+    match = _TITLE_RE.search(page_html)
+    if not match:
+        return fallback
+    title = re.sub(r"\s+", " ", match.group(1)).strip()
+    return title or fallback
+
+
+def index_mock_files(mocks_dir: Path) -> list[Path]:
+    """Every `*.html` mock in *mocks_dir* other than the index itself,
+    sorted by filename for a stable, reproducible ordering across runs."""
+    return sorted(
+        p for p in mocks_dir.glob("*.html") if p.name != INDEX_NAME
+    )
+
+
+def generate_index(mocks_dir: Path) -> str:
+    """Build the `index.html` content: an unstyled `<ul>` of links, one per
+    mock file, labelled from that mock's own `<title>`."""
+    items = []
+    for path in index_mock_files(mocks_dir):
+        page_html = path.read_text(encoding="utf-8")
+        title = _extract_title(page_html, fallback=path.name)
+        items.append(
+            f'    <li><a href="{html.escape(path.name)}">'
+            f"{html.escape(title)}</a></li>"
+        )
+    body = "\n".join(items) if items else "    <li>(no mocks found)</li>"
+    return (
+        "<!DOCTYPE html>\n"
+        "<html>\n"
+        "<head>\n"
+        '<meta charset="utf-8">\n'
+        "<title>Gate-A mock set</title>\n"
+        "</head>\n"
+        "<body>\n"
+        "<h1>Gate-A mock set</h1>\n"
+        "<ul>\n"
+        f"{body}\n"
+        "</ul>\n"
+        "</body>\n"
+        "</html>\n"
+    )
+
+
+def main(argv: list[str]) -> int:
+    if len(argv) != 2:
+        print("usage: gen_mock_index.py <mocks-dir>", file=sys.stderr)
+        return 2
+    mocks_dir = Path(argv[1])
+    if not mocks_dir.is_dir():
+        print(f"error: not a directory: {mocks_dir}", file=sys.stderr)
+        return 1
+    content = generate_index(mocks_dir)
+    out_path = mocks_dir / INDEX_NAME
+    out_path.write_text(content, encoding="utf-8")
+    print(f"wrote {out_path}")
+    return 0
+
+
+if __name__ == "__main__":
+    raise SystemExit(main(sys.argv))

code_coordinator-0.5.206/tests/test_gen_mock_index.py

@@ -0,0 +1,124 @@
+"""Tests for `scripts/gen_mock_index.py` (#2512): the deterministic
+post-render step that turns a flat `mocks/*.html` directory into a
+navigable set by writing `mocks/index.html`.
+
+See `coord/mock_author.py`'s `_wants_mock_index` / `_mock_index_instruction`
+for where the mock-author worker is told to run this script — these tests
+cover the script's own generation logic in isolation, without a worker or
+a live milestone dir.
+"""
+from __future__ import annotations
+
+from pathlib import Path
+
+from scripts.gen_mock_index import generate_index, index_mock_files, main
+
+
+def _write(path: Path, title: str | None, body: str = "<p>hi</p>") -> None:
+    head = f"<head><title>{title}</title></head>" if title is not None else "<head></head>"
+    path.write_text(f"<html>{head}<body>{body}</body></html>", encoding="utf-8")
+
+
+def test_index_mock_files_excludes_index_itself_and_sorts(tmp_path: Path):
+    mocks = tmp_path / "mocks"
+    mocks.mkdir()
+    _write(mocks / "b.html", "B")
+    _write(mocks / "a.html", "A")
+    _write(mocks / "index.html", "should be excluded")
+
+    files = index_mock_files(mocks)
+
+    assert [p.name for p in files] == ["a.html", "b.html"]
+
+
+def test_generate_index_links_each_mock_by_its_title(tmp_path: Path):
+    mocks = tmp_path / "mocks"
+    mocks.mkdir()
+    _write(mocks / "reports-grid.html", "coord web — Reports panel (ms-2 Gate-A mock)")
+    _write(mocks / "reports-chart.html", "coord web — Reports chart (ms-2 Gate-A mock)")
+
+    out = generate_index(mocks)
+
+    assert '<a href="reports-grid.html">coord web — Reports panel (ms-2 Gate-A mock)</a>' in out
+    assert '<a href="reports-chart.html">coord web — Reports chart (ms-2 Gate-A mock)</a>' in out
+
+
+def test_generate_index_falls_back_to_filename_when_title_missing(tmp_path: Path):
+    mocks = tmp_path / "mocks"
+    mocks.mkdir()
+    _write(mocks / "untitled.html", None)
+
+    out = generate_index(mocks)
+
+    assert '<a href="untitled.html">untitled.html</a>' in out
+
+
+def test_generate_index_escapes_title_html(tmp_path: Path):
+    mocks = tmp_path / "mocks"
+    mocks.mkdir()
+    _write(mocks / "weird.html", "Reports <beta> & friends")
+
+    out = generate_index(mocks)
+
+    assert "Reports &lt;beta&gt; &amp; friends" in out
+    assert "<beta>" not in out
+
+
+def test_generate_index_reports_no_mocks_found_when_dir_empty(tmp_path: Path):
+    mocks = tmp_path / "mocks"
+    mocks.mkdir()
+
+    out = generate_index(mocks)
+
+    assert "(no mocks found)" in out
+
+
+def test_generate_index_is_deterministic_across_runs(tmp_path: Path):
+    mocks = tmp_path / "mocks"
+    mocks.mkdir()
+    _write(mocks / "b.html", "B")
+    _write(mocks / "a.html", "A")
+
+    assert generate_index(mocks) == generate_index(mocks)
+
+
+def test_main_writes_index_html_file(tmp_path: Path):
+    mocks = tmp_path / "mocks"
+    mocks.mkdir()
+    _write(mocks / "only.html", "Only mock")
+
+    rc = main(["gen_mock_index.py", str(mocks)])
+
+    assert rc == 0
+    index_path = mocks / "index.html"
+    assert index_path.exists()
+    assert "Only mock" in index_path.read_text(encoding="utf-8")
+
+
+def test_main_reruns_are_idempotent(tmp_path: Path):
+    """Re-rendering an unchanged mock set must produce a byte-identical
+    index — no drift between successive `--amend` passes."""
+    mocks = tmp_path / "mocks"
+    mocks.mkdir()
+    _write(mocks / "only.html", "Only mock")
+
+    main(["gen_mock_index.py", str(mocks)])
+    first = (mocks / "index.html").read_text(encoding="utf-8")
+    main(["gen_mock_index.py", str(mocks)])
+    second = (mocks / "index.html").read_text(encoding="utf-8")
+
+    assert first == second
+
+
+def test_main_rejects_missing_directory(tmp_path: Path, capsys):
+    rc = main(["gen_mock_index.py", str(tmp_path / "nope")])
+
+    assert rc == 1
+    assert "not a directory" in capsys.readouterr().err
+
+
+def test_main_requires_exactly_one_argument(capsys):
+    rc = main(["gen_mock_index.py"])
+
+    assert rc == 2
+    assert "usage:" in capsys.readouterr().err

{code_coordinator-0.5.205 → code_coordinator-0.5.206}/tests/test_mock_author.py

@@ -120,6 +120,38 @@ def test_briefing_threads_the_html_mock_glob_for_a_web_playwright_driver():
     assert "tests/acceptance/ms-9/mocks/" in out
 
 
+# ── #2512: deterministic mocks/index.html post-render step ─────────────────
+
+
+def test_briefing_instructs_index_script_for_html_driver():
+    out = mock_author.build_mock_author_briefing(
+        repo_slug="acme/webapp",
+        milestone_title="M",
+        milestone_number=2,
+        tracking_issue_number=100,
+        tracking_issue_body="",
+        issues=[],
+        driver_kind="web-playwright",
+        driver_mock_glob="*.html",
+    )
+    assert "scripts/gen_mock_index.py tests/acceptance/ms-2/mocks" in out
+    assert "index.html" in out
+
+
+def test_briefing_omits_index_script_for_non_html_driver():
+    out = mock_author.build_mock_author_briefing(
+        repo_slug="acme/tui",
+        milestone_title="M",
+        milestone_number=2,
+        tracking_issue_number=100,
+        tracking_issue_body="",
+        issues=[],
+        driver_kind="tui-tuidriver",
+        driver_mock_glob="*.screen",
+    )
+    assert "gen_mock_index.py" not in out
+
+
 def test_mock_author_system_prompt_pins_the_locked_html_mock_shape():
     """docs/ORACLE_LOOP.md's locked (2026-07-28) decision: a `.html` mock
     must be self-contained, open in a browser, and LOOK like the screen —
@@ -330,6 +362,7 @@ def test_amend_briefing_includes_correction_text_verbatim():
         milestone_number=9,
         tracking_issue_number=100,
         amend_text="the CLI flag is --for-path, not --path",
+        driver_mock_glob="*.screen",
     )
     assert "the CLI flag is --for-path, not --path" in out
     assert "acme/api" in out
@@ -343,12 +376,37 @@ def test_amend_briefing_names_already_merged_contract_and_scopes_edits():
         milestone_number=9,
         tracking_issue_number=100,
         amend_text="fix a typo",
+        driver_mock_glob="*.screen",
     )
     assert "tests/acceptance/ms-9/contract.md" in out
     assert "ALREADY-MERGED" in out
     assert "from-scratch" in out
 
 
+def test_amend_briefing_instructs_index_script_for_html_driver():
+    out = mock_author.build_mock_author_amend_briefing(
+        repo_slug="acme/webapp",
+        milestone_title="M",
+        milestone_number=2,
+        tracking_issue_number=100,
+        amend_text="fix a typo",
+        driver_mock_glob="*.html",
+    )
+    assert "scripts/gen_mock_index.py tests/acceptance/ms-2/mocks" in out
+
+
+def test_amend_briefing_omits_index_script_for_non_html_driver():
+    out = mock_author.build_mock_author_amend_briefing(
+        repo_slug="acme/api",
+        milestone_title="M",
+        milestone_number=9,
+        tracking_issue_number=100,
+        amend_text="fix a typo",
+        driver_mock_glob="*.screen",
+    )
+    assert "gen_mock_index.py" not in out
+
+
 def test_dispatch_amend_skips_open_issues_fetch_and_uses_amend_briefing(tmp_path):
     """#1315: --amend must not re-fetch every open milestone issue (that's
     the "full fresh render" cost the issue closes the gap on) and must