code-coordinator 0.5.232__tar.gz → 0.5.235__tar.gz

{code_coordinator-0.5.232 → code_coordinator-0.5.235}/PKG-INFO

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

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

@@ -621,6 +621,7 @@ tests/test_release_cordon_daemon_leads_2176.py
 tests/test_release_cordon_deadlock_2240.py
 tests/test_release_cordon_stuck_2490.py
 tests/test_release_propagate.py
+tests/test_release_roll_window_handoff.py
 tests/test_release_unified_1242.py
 tests/test_release_verify.py
 tests/test_release_window.py
@@ -741,6 +742,8 @@ tests/acceptance/ms-65/mocks/pipeline-tabs-independent.screen
 tests/acceptance/ms-67/approved_items_2532.rs
 tests/acceptance/ms-67/contract.md
 tests/acceptance/ms-67/manifest.yml
+tests/acceptance/ms-67/pull_decomposition_2533.rs
+tests/acceptance/ms-67/manifest.d/2533.yml
 tests/acceptance/ms-67/mocks/approved-items-chat-opened.screen
 tests/acceptance/ms-67/mocks/approved-items-context-menu.screen
 tests/acceptance/ms-67/mocks/approved-items-detail.screen

{code_coordinator-0.5.232 → code_coordinator-0.5.235}/code_coordinator.egg-info/scm_file_list.json

@@ -380,12 +380,14 @@
     "tests/acceptance/ms-65/tabs_persistence_2286.rs",
     "tests/acceptance/ms-67/approved_items_2532.rs",
     "tests/acceptance/ms-67/contract.md",
+    "tests/acceptance/ms-67/manifest.d/2533.yml",
     "tests/acceptance/ms-67/manifest.yml",
     "tests/acceptance/ms-67/mocks/approved-items-chat-opened.screen",
     "tests/acceptance/ms-67/mocks/approved-items-context-menu.screen",
     "tests/acceptance/ms-67/mocks/approved-items-detail.screen",
     "tests/acceptance/ms-67/mocks/approved-items-empty.screen",
     "tests/acceptance/ms-67/mocks/approved-items-populated.screen",
+    "tests/acceptance/ms-67/pull_decomposition_2533.rs",
     "tests/acceptance/ms-example/mocks/home-active.html",
     "tests/conftest.py",
     "tests/fixtures/board-pipeline-basic.json",
@@ -691,6 +693,7 @@
     "tests/test_release_cordon_deadlock_2240.py",
     "tests/test_release_cordon_stuck_2490.py",
     "tests/test_release_propagate.py",
+    "tests/test_release_roll_window_handoff.py",
     "tests/test_release_unified_1242.py",
     "tests/test_release_verify.py",
     "tests/test_release_window.py",

{code_coordinator-0.5.232 → code_coordinator-0.5.235}/code_coordinator.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.232",
+  "tag": "0.5.235",
   "distance": 0,
-  "node": "geaf0ae75fe41add5924a913f93d8c3a955bb2c8c",
+  "node": "g5e1480a48c81b0a8cba96e6b1a3f8efc68470379",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-22"

{code_coordinator-0.5.232 → code_coordinator-0.5.235}/coord/agent.py

@@ -1110,14 +1110,36 @@ def _pushed_commits_ahead(repo_path: Path, base: str, branch: str) -> int | None
 _ISSUE_REF_RE = re.compile(r"#(\d+)")
 
 
-def _foreign_issue_refs(subject: str, issue_number: int) -> frozenset[int]:
+def _foreign_issue_refs(
+    subject: str,
+    issue_number: int,
+    extra_allowed: frozenset[int] = frozenset(),
+) -> frozenset[int]:
     """Return the set of *foreign* issue numbers referenced in *subject* —
-    those that are not *issue_number*.  Empty if the subject doesn't reference
-    any foreign issue."""
+    those that are neither *issue_number* nor in *extra_allowed*.  Empty if
+    the subject doesn't reference any foreign issue.
+
+    *extra_allowed* (#2545): for a ``type="test-author"``/``"mock-author"``
+    merge entry, *issue_number* is the milestone's TRACKING issue (every JIT
+    slice for one milestone shares a single branch/PR — see
+    ``Assignment.for_issue_number``'s docstring), but the slice's own commit
+    correctly and necessarily cites its OWN child issue
+    (``test(ms-4): ... slice #132``). Without this, every such commit was
+    flagged FOREIGN on every single verify, regardless of whether the rebase
+    was actually clean. Callers resolve the slice's own issue number (e.g.
+    via ``coord.models.effective_issue_number``) and pass it here so a
+    reference to *either* the tracking issue or the resolved slice issue
+    counts as home, not foreign. Defaults to empty, preserving the original
+    all-blocking-except-``issue_number`` behaviour for ordinary ``work``
+    assignments.
+    """
     refs = {int(m) for m in _ISSUE_REF_RE.findall(subject)}
-    if not refs or issue_number in refs:
+    if not refs:
         return frozenset()
-    return frozenset(refs - {issue_number})
+    home = {issue_number} | extra_allowed
+    if refs & home:
+        return frozenset()
+    return frozenset(refs - home)
 
 
 def _resolve_base_ref(wt_path: Path, base: str) -> str | None:
@@ -1218,6 +1240,7 @@ def verify_merge_branch(
     *,
     base: str,
     issue_number: int,
+    extra_allowed_issue_numbers: frozenset[int] = frozenset(),
     closed_issue_numbers: frozenset[int] = frozenset(),
 ) -> MergeVerify:
     """Verify a ``--merge-of`` branch before its terminal ``done`` is recorded.
@@ -1233,10 +1256,17 @@ def verify_merge_branch(
     ``claude``, no PTY — so the gate runs as a fast local-only test fixture.
 
     Args:
+        extra_allowed_issue_numbers: Additional issue numbers, besides
+            *issue_number*, that a commit subject may legitimately reference
+            without being flagged foreign (#2545) — see
+            :func:`_foreign_issue_refs`'s ``extra_allowed`` for why a
+            ``test-author``/``mock-author`` merge entry needs this (its own
+            slice issue, not just the milestone's tracking issue).
         closed_issue_numbers: Issue numbers that are known to be closed/merged
             in the upstream repo.  When a commit's subject references *only*
-            issues in this set (and not ``issue_number``), it is downgraded
-            from a **blocking** foreign finding to an **advisory** one (#1279).
+            issues in this set (and not ``issue_number`` or
+            ``extra_allowed_issue_numbers``), it is downgraded from a
+            **blocking** foreign finding to an **advisory** one (#1279).
             The rationale: a commit referencing a *closed* issue cannot be
             live rebase-pollution — it is almost certainly a worker typo in the
             commit subject.  Pass the empty frozenset (default) to preserve the
@@ -1271,7 +1301,9 @@ def verify_merge_branch(
     foreign: list[tuple[str, str]] = []
     advisory_foreign: list[tuple[str, str]] = []
     for sha, subj in added:
-        foreign_refs = _foreign_issue_refs(subj, issue_number)
+        foreign_refs = _foreign_issue_refs(
+            subj, issue_number, extra_allowed=extra_allowed_issue_numbers
+        )
         if not foreign_refs:
             continue  # not flagged — own work or bare message
         # #1279: downgrade to advisory when ALL referenced foreign issues are
@@ -1312,6 +1344,17 @@ def resolve_closed_issue_numbers(
     Returns an empty frozenset immediately when *repo_github* is falsy or
     *foreign_commits* is empty, so callers can skip the second verify pass
     entirely (``if closed: ...``) without an extra branch.
+
+    Deliberately does **not** accept an ``extra_allowed_issue_numbers``
+    parameter to thread through to :func:`_foreign_issue_refs` (#2545
+    review): *foreign_commits* here is always the ``.foreign`` list a prior,
+    ``extra_allowed``-aware :func:`verify_merge_branch` pass already
+    produced, and ``_foreign_issue_refs`` treats any overlap with the "home"
+    set (``{issue_number} | extra_allowed``) as voiding a commit's foreign
+    status entirely — so no subject surviving into *foreign_commits* can
+    reference an ``extra_allowed`` issue in the first place. If a future
+    caller ever feeds this something other than a prior pass's ``.foreign``,
+    re-check this assumption before relying on it.
     """
     foreign_commits = list(foreign_commits)
     if not repo_github or not foreign_commits:

{code_coordinator-0.5.232 → code_coordinator-0.5.235}/coord/commands/dispatch_workers.py

@@ -3355,6 +3355,26 @@ def _dispatch_merge_of(
         )
         sys.exit(2)
 
+    # #2545: a `type="test-author"`/`"mock-author"` work row's `issue_number`
+    # (== `issue` above) is always the milestone's TRACKING issue, but its
+    # commit correctly cites the slice's OWN child issue — resolved via the
+    # canonical `coord.models.effective_issue_number` helper (built for
+    # exactly this tracking-vs-slice distinction, #1553) rather than reading
+    # `for_issue_number` directly, so this stays in sync with the other
+    # callers of that helper (e.g. `merge_queue._test_author_effective_
+    # issue_number`) instead of drifting as an independent copy. Without
+    # allowing the resolved issue too, the #604 FOREIGN check below flags
+    # every JIT slice's own, correctly-labeled commit on every single verify.
+    # Ordinary `work` assignments have no `for_issue_number`, so this
+    # resolves to `issue` itself — already in the "home" set, a harmless
+    # no-op.
+    from coord.models import effective_issue_number as _mg_effective_issue_number  # noqa: PLC0415
+
+    _mg_for_issue = _mg_effective_issue_number(work)
+    _mg_extra_allowed = (
+        frozenset({_mg_for_issue}) if _mg_for_issue and _mg_for_issue != issue else frozenset()
+    )
+
     resolved_model = model if model else cfg.models.default
     assignment_id = _uuid.uuid4().hex[:12]
     if _is_local:
@@ -3607,6 +3627,9 @@ def _dispatch_merge_of(
                 # #604: git truth overrides the agent's self-report on the
                 # merge path — a botched rebase records `blocked`, not `done`.
                 verify_merge=True,
+                # #2545: allow the JIT slice's own child issue, not just the
+                # tracking issue, in the FOREIGN check.
+                extra_allowed_issue_numbers=_mg_extra_allowed,
                 branch=work.branch,
             )
             _echo_merge_finalize(finalize_result, merge_target_branch)
@@ -3869,6 +3892,9 @@ def _dispatch_merge_of(
             exit_code=exit_code,
             started_at=started_at,
             verify_merge=True,
+            # #2545: allow the JIT slice's own child issue, not just the
+            # tracking issue, in the FOREIGN check.
+            extra_allowed_issue_numbers=_mg_extra_allowed,
         )
         _echo_merge_finalize(finalize_result, merge_target_branch)
         _mv = finalize_result.merge_verify