code-coordinator 0.5.141__tar.gz → 0.5.142__tar.gz

{code_coordinator-0.5.141 → code_coordinator-0.5.142}/PKG-INFO

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

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

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.141",
+  "tag": "0.5.142",
   "distance": 0,
-  "node": "gef1e71db0f0c6bc0d0346e3020abaeabac680993",
+  "node": "gf633cd59c13444ede9e9471099a848d9bd3587f4",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-18"

{code_coordinator-0.5.141 → code_coordinator-0.5.142}/coord/reconcile.py

@@ -1242,6 +1242,28 @@ def _reassign(
     repo_path = machine.repo_path(failed.repo_name)
 
     retry_model = model if model is not None else failed.model
+    # #2383: `failed.model` can name a model an operator has since REMOVED
+    # from `models.escalation` (e.g. a run auto-escalated onto it before the
+    # ladder was edited) — inheriting it here forever, on every subsequent
+    # auto-reassign of the same lineage, would silently keep dispatching a
+    # model the current config no longer sanctions. Only the inherited
+    # (`model is None`) path needs this: an explicit caller-supplied `model`
+    # is a deliberate choice and passes through unclamped. And only a
+    # CLAUDE-family retry needs it — `models.escalation` is a claude-only
+    # concept (#2323's "the claude escalation ladder means nothing to
+    # another provider"), so an opencode/other-provider model like
+    # `opencode/glm-5.2` must never be walked onto the claude ladder just
+    # because it doesn't happen to appear on it.
+    if (
+        model is None
+        and resolved_provider_name == "claude"
+        and retry_model not in (*config.models.escalation, config.models.default)
+    ):
+        retry_model = (
+            config.models.escalation[-1]
+            if config.models.escalation
+            else config.models.default
+        )
     # The Assignment keeps the alias for legibility; the wire payload is
     # resolved through models.versions when an exact id is pinned.
     retry_model_wire = config.models.resolve(retry_model)

{code_coordinator-0.5.141 → code_coordinator-0.5.142}/tests/test_reconcile.py

@@ -262,6 +262,64 @@ class TestReassignThreadsProvider:
         payload = mock_post.call_args.kwargs["json"]
         assert "provider" not in payload
 
+    @patch("coord.reconcile.httpx.post")
+    def test_a_model_dropped_from_the_ladder_is_not_inherited_forever(
+        self, mock_post: MagicMock,
+    ) -> None:
+        """#2383: `failed.model` names a model an operator has since
+        removed from `models.escalation` — the auto-reassign call site
+        never passes `model=` explicitly (reconcile.py's `newly_failed`
+        loop), so without a clamp this would be inherited on EVERY future
+        retry of the same lineage, forever, regardless of what the current
+        ladder says."""
+        resp = MagicMock()
+        resp.json.return_value = {"id": "newid"}
+        mock_post.return_value = resp
+        cfg = Config(
+            repos=[Repo(name="api", github="acme/api")],
+            machines=[
+                Machine(name="laptop", host="l", repos=["api"], repo_paths={"api": "/tmp"}),
+                Machine(name="server", host="s", repos=["api"], repo_paths={"api": "/tmp"}),
+            ],
+        )
+        assert "fable" not in cfg.models.escalation  # sanity: not on the ladder
+        board = Board()
+        failed = _failed(model="fable")
+
+        result = _reassign(failed, board, cfg)
+
+        assert result is not None
+        payload = mock_post.call_args.kwargs["json"]
+        assert payload["model"] != "claude-fable-5"
+        assert payload["model"] == cfg.models.resolve(cfg.models.escalation[-1])
+
+    @patch("coord.reconcile.httpx.post")
+    def test_an_explicit_model_override_is_never_clamped(
+        self, mock_post: MagicMock,
+    ) -> None:
+        """The clamp is scoped to the INHERITED (`model=None`) path only —
+        a caller that explicitly asks for a specific model (e.g. #1291's
+        semantic-conflict escalation) must get exactly that model, even one
+        off the ordinary ladder."""
+        resp = MagicMock()
+        resp.json.return_value = {"id": "newid"}
+        mock_post.return_value = resp
+        cfg = Config(
+            repos=[Repo(name="api", github="acme/api")],
+            machines=[
+                Machine(name="laptop", host="l", repos=["api"], repo_paths={"api": "/tmp"}),
+                Machine(name="server", host="s", repos=["api"], repo_paths={"api": "/tmp"}),
+            ],
+        )
+        board = Board()
+        failed = _failed(model="sonnet")
+
+        result = _reassign(failed, board, cfg, model="opus")
+
+        assert result is not None
+        payload = mock_post.call_args.kwargs["json"]
+        assert payload["model"] == cfg.models.resolve("opus")
+
 
 # ── #1711: the provider-availability gate, applied to retry routing ────────