code-coordinator 0.5.308__tar.gz → 0.5.309__tar.gz

{code_coordinator-0.5.308 → code_coordinator-0.5.309}/PKG-INFO

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-coordinator
-Version: 0.5.308
+Version: 0.5.309
 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.308 → code_coordinator-0.5.309}/code_coordinator.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.308",
+  "tag": "0.5.309",
   "distance": 0,
-  "node": "gde30a174fcc1e82cb10b2f750bbe379e349de5e4",
+  "node": "g64b7ec99fe4514edb4d20e26f811aca2f270a47b",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-30"

{code_coordinator-0.5.308 → code_coordinator-0.5.309}/coord/milestone_chat.py

@@ -57,15 +57,28 @@ def pick_milestone_chat_machine(cfg: Config, repo: str) -> Machine | None:
     return pick_refinement_machine(cfg, repo)
 
 
-def _fetch_milestone_issues(slug: str, milestone_number: int) -> list[dict]:
+def _fetch_milestone_issues(
+    slug: str,
+    milestone_number: int,
+    *,
+    max_body_chars: int | None = MAX_ISSUE_BODY_CHARS,
+) -> list[dict]:
     """Best-effort fetch of open issues under *milestone_number*.
 
-    Returns a list of ``{"number", "title", "body"}`` dicts (bodies
-    truncated to ``MAX_ISSUE_BODY_CHARS`` so cohort/dependency inference has
-    signal without blowing the prompt budget), capped at ``MAX_ISSUES``.
-    Returns an empty list on any failure so the seed briefing still works
-    without issue context — the operator can still chat, just with less to
-    go on.
+    Returns a list of ``{"number", "title", "body"}`` dicts, capped at
+    ``MAX_ISSUES``. Returns an empty list on any failure so the seed
+    briefing still works without issue context — the operator can still
+    chat, just with less to go on.
+
+    *max_body_chars* defaults to ``MAX_ISSUE_BODY_CHARS`` (bodies truncated
+    so cohort/dependency inference has signal without blowing the prompt
+    budget) — that trade is right for *this* module's job, a chat session
+    inferring dependencies from a digest. It is wrong for a caller whose
+    entire output is derived from the body text itself: #2969 found Gate A's
+    mock-author (`coord/mock_author.py`) reusing this same cap, silently
+    handing it 36-82% of each issue's spec and leaving it to notice and flag
+    the gap in its own contract. Pass ``max_body_chars=None`` for callers
+    like that, where a body must arrive in full or not at all.
     """
     try:
         issues = github_ops.get_open_issues(slug)
@@ -78,8 +91,8 @@ def _fetch_milestone_issues(slug: str, milestone_number: int) -> list[dict]:
     out: list[dict] = []
     for issue in under_milestone[:MAX_ISSUES]:
         body = (issue.get("body") or "").strip()
-        if len(body) > MAX_ISSUE_BODY_CHARS:
-            body = body[:MAX_ISSUE_BODY_CHARS] + "\n...(truncated)"
+        if max_body_chars is not None and len(body) > max_body_chars:
+            body = body[:max_body_chars] + "\n...(truncated)"
         out.append({
             "number": issue.get("number"),
             "title": issue.get("title") or "",

{code_coordinator-0.5.308 → code_coordinator-0.5.309}/coord/mock_author.py

@@ -323,7 +323,13 @@ def dispatch_acceptance_mock(
             required_gates=list(config.pipeline.default_gates),
         )
     else:
-        issues = _fetch_milestone_issues(repo_cfg.github, milestone_number)
+        # #2969: Gate A's contract IS the spec — anything truncated here
+        # becomes silent guesswork in `contract.md`. Unlike milestone-chat's
+        # own cohort/dependency-inference use of this fetcher, hand the
+        # mock-author every body in full (no `max_body_chars` cap).
+        issues = _fetch_milestone_issues(
+            repo_cfg.github, milestone_number, max_body_chars=None
+        )
 
         briefing = build_mock_author_briefing(
             repo_slug=repo_cfg.github,

{code_coordinator-0.5.308 → code_coordinator-0.5.309}/tests/test_milestone_chat.py

@@ -223,6 +223,18 @@ def test_fetch_milestone_issues_returns_empty_on_failure():
     assert out == []
 
 
+def test_fetch_milestone_issues_max_body_chars_none_disables_truncation():
+    """#2969: Gate A's mock-author passes `max_body_chars=None` so it sees
+    every issue body in full — the cap below is this module's own default,
+    not a hard-coded floor other callers are stuck with."""
+    long_body = "x" * (milestone_chat.MAX_ISSUE_BODY_CHARS + 500)
+    issues = [{"number": 1, "title": "T", "body": long_body, "milestone": {"number": 9}}]
+    with patch("coord.github_ops.get_open_issues", return_value=issues):
+        out = milestone_chat._fetch_milestone_issues("acme/api", 9, max_body_chars=None)
+    assert out[0]["body"] == long_body
+    assert "(truncated)" not in out[0]["body"]
+
+
 # ── pick_milestone_chat_machine ──────────────────────────────────────────────
 
 

{code_coordinator-0.5.308 → code_coordinator-0.5.309}/tests/test_mock_author.py

@@ -352,6 +352,39 @@ def test_dispatch_success_records_assignment(tmp_path):
     mock_record.assert_called_once()
 
 
+def test_dispatch_briefs_full_issue_bodies_uncapped(tmp_path):
+    """#2969: Gate A's contract is authored straight from these bodies —
+    reusing milestone_chat's 1500-char cohort-inference cap here silently
+    fed the mock-author 36-82% of each issue's spec. The dispatched
+    briefing must carry every body in full, past that cap."""
+    from coord.models import Board
+
+    long_body = "y" * (2000) + " END-OF-BODY-MARKER"
+    cfg = _cfg_with_driver(tmp_path)
+    issue_data = {
+        "number": 100, "title": "Milestone tracker", "body": "",
+        "milestone": {"number": 9, "title": "Q3"},
+    }
+    open_issues = [
+        {
+            "number": 5, "title": "Child", "body": long_body,
+            "milestone": {"number": 9},
+        },
+    ]
+    with patch("coord.github_ops.get_issue", return_value=issue_data), \
+         patch("coord.github_ops.get_open_issues", return_value=open_issues), \
+         patch("coord.board_service.read_board", return_value=Board()), \
+         patch("coord.dispatch.dispatch_with_retry", return_value={"id": "asg-xyz"}) as mock_dispatch, \
+         patch("coord.dispatch.post_briefing"), \
+         patch("coord.state.record_dispatched"):
+        mock_author.dispatch_acceptance_mock("api", 100, cfg)
+
+    proposal = mock_dispatch.call_args[0][0]
+    assert len(long_body) > 1500  # sanity: this reproduces only past the old cap
+    assert long_body in proposal.briefing
+    assert "(truncated)" not in proposal.briefing
+
+
 # ── build_mock_author_amend_briefing / --amend dispatch (#1315) ─────────────