code-coordinator 0.5.55__tar.gz → 0.5.56__tar.gz

{code_coordinator-0.5.55/code_coordinator.egg-info → code_coordinator-0.5.56}/PKG-INFO

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

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

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.55",
+  "tag": "0.5.56",
   "distance": 0,
-  "node": "g20ff70836aa359817f5f585fff5a5b9a075bb022",
+  "node": "gb8053d5c79cc3a574e1df4a7543e49d6a6d09410",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-13"

{code_coordinator-0.5.55 → code_coordinator-0.5.56}/coord/smoke.py

@@ -825,6 +825,17 @@ def dispatch_smoke(
             is_worker=choice.is_worker,
         )
 
+        # #2168: pin the Test stage's model to avoid the agent falling
+        # through to the machine's ambient `claude -p` default (Opus).
+        # Mirrors the review path (coord/review.py, coord/gate_b.py):
+        # deliberately `config.models.default`, NOT `config.models.labels`
+        # — the Test stage's job (checkout, run one command, read an exit
+        # code) is a property of the repo's test command, never of the
+        # issue's tier label, so label routing (#1798) must not leak in
+        # here either.
+        smoke_model_alias = config.models.default
+        smoke_model_wire = config.models.resolve(smoke_model_alias)
+
         payload = {
             "repo_name": completed.repo_name,
             "repo_path": repo_path,
@@ -840,6 +851,7 @@ def dispatch_smoke(
             # #255: smoke checks out the worker's PR branch but the agent still
             # consults `branch` as the integration base.
             "branch": repo.default_branch or "main",
+            "model": smoke_model_wire,
         }
 
         url = f"http://{choice.machine.host}:{AGENT_PORT}/assign"
@@ -894,6 +906,7 @@ def dispatch_smoke(
         type="smoke",
         review_target=completed.branch,
         review_of_assignment_id=completed.assignment_id,
+        model=smoke_model_alias,
     )
     board.active.append(smoke_assignment)
 

{code_coordinator-0.5.55 → code_coordinator-0.5.56}/tests/test_smoke.py

@@ -576,6 +576,64 @@ def test_dispatch_smoke_sends_to_capable_different_machine(
     assert "make test" in payload["briefing"]
 
 
+def test_dispatch_smoke_pins_model_to_models_default(
+    gtk_and_server_config: Config,
+) -> None:
+    """#2168: the Test stage used to dispatch with no `model` at all, so the
+    agent fell through to the machine's ambient `claude -p` default (Opus)
+    instead of the configured `models.default`. Mirrors the review path
+    (coord/review.py, coord/gate_b.py): the resolved alias lands on both the
+    wire payload (what the agent actually runs with) and the returned
+    `Assignment.model` (what `coord usage` reports)."""
+    cfg = replace(
+        gtk_and_server_config,
+        models=replace(gtk_and_server_config.models, default="haiku"),
+    )
+    client = _FakeClient({"id": "smoke-1"})
+    result = dispatch_smoke(
+        _completed(), Board(), cfg,
+        http_client=client,
+        diff_lookup=lambda repo, branch: ["src/gtk/window.c"],
+    )
+    assert result is not None
+    assert result.model == "haiku"
+
+    assert len(client.calls) == 1
+    _, payload = client.calls[0]
+    assert payload["model"] == "haiku"
+
+
+def test_dispatch_smoke_model_pin_ignores_models_labels(
+    gtk_and_server_config: Config,
+) -> None:
+    """#2168: the Test stage's difficulty is a property of the repo's test
+    command, never the issue's tier label — mirrors #1430's review-path
+    decision. A `models.labels` entry that would route a `type="work"`
+    dispatch to `opus` must NOT leak into the Test stage's model pin; it
+    must still resolve to `models.default`. Guards against re-introducing
+    the #1798 label leak through this new path."""
+    cfg = replace(
+        gtk_and_server_config,
+        models=replace(
+            gtk_and_server_config.models,
+            default="sonnet",
+            labels={"tier:large": "opus"},
+        ),
+    )
+    client = _FakeClient({"id": "smoke-1"})
+    result = dispatch_smoke(
+        _completed(), Board(), cfg,
+        http_client=client,
+        diff_lookup=lambda repo, branch: ["src/gtk/window.c"],
+    )
+    assert result is not None
+    assert result.model == "sonnet"
+
+    assert len(client.calls) == 1
+    _, payload = client.calls[0]
+    assert payload["model"] == "sonnet"
+
+
 def test_dispatch_smoke_refuses_machine_whose_probe_contradicts_its_capability(
     gtk_and_server_config: Config,
 ) -> None: