code-coordinator 0.5.653__tar.gz → 0.5.654__tar.gz

{code_coordinator-0.5.653 → code_coordinator-0.5.654}/PKG-INFO

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

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

@@ -925,7 +925,6 @@ tests/test_tmux_session.py
 tests/test_transcript_floor.py
 tests/test_tui_pty_driver.py
 tests/test_uat_checks.py
-tests/test_ultracode_workflow_3722.py
 tests/test_upstream_gaps.py
 tests/test_usage.py
 tests/test_usage_limits.py

{code_coordinator-0.5.653 → code_coordinator-0.5.654}/code_coordinator.egg-info/scm_file_list.json

@@ -952,7 +952,6 @@
     "tests/test_transcript_floor.py",
     "tests/test_tui_pty_driver.py",
     "tests/test_uat_checks.py",
-    "tests/test_ultracode_workflow_3722.py",
     "tests/test_upstream_gaps.py",
     "tests/test_usage.py",
     "tests/test_usage_limits.py",

code_coordinator-0.5.654/code_coordinator.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "0.5.654",
+  "distance": 0,
+  "node": "gfb5fe0770972418d5fdcaf022adb307f60f81c3c",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-10-19"
+}

{code_coordinator-0.5.653 → code_coordinator-0.5.654}/coord/agent.py

@@ -1050,31 +1050,6 @@ def _wait_for_proc_or_result(
 # ``issue_labels`` precedent (``provider_issue_labels``) already is.
 WORKFLOW_LEG_LABEL = "coord:workflow"
 
-# #3722: the model-selection label (`models.labels.ultracode` in
-# coordinator.yml, resolved independently by `coord.config.ModelsConfig`)
-# that operators expect to *also* mean "run this thoroughly" — i.e. imply
-# `WORKFLOW_LEG_LABEL` — rather than silently staying a single opus leg.
-ULTRACODE_LABEL = "ultracode"
-
-
-def is_workflow_leg(proposal_type: str, issue_labels: list[str] | None) -> bool:
-    """Whether a ``type="work"`` dispatch should run as a multi-agent workflow.
-
-    True when the issue carries ``WORKFLOW_LEG_LABEL`` or ``ULTRACODE_LABEL``
-    (#3722: ``ultracode`` implies the workflow opt-in, not just the opus
-    model). Every non-``"work"`` proposal type (test, review, fix, ...) is
-    never a workflow leg, regardless of labels.
-
-    Single source of truth for this question: ``coord/dispatch.py`` (the
-    wire ``AssignmentSpec.workflow`` payload) and ``coord/state.py`` (the
-    recorded assignment ``mode`` column) both call this instead of
-    re-deriving the condition, so the two surfaces cannot drift apart.
-    """
-    labels = issue_labels or []
-    return proposal_type == "work" and (
-        WORKFLOW_LEG_LABEL in labels or ULTRACODE_LABEL in labels
-    )
-
 
 @dataclass
 class AssignmentSpec:

{code_coordinator-0.5.653 → code_coordinator-0.5.654}/coord/dispatch.py

@@ -1345,21 +1345,17 @@ def dispatch(
     # kwarg.
     if proposal.issue_labels:
         payload["issue_labels"] = list(proposal.issue_labels)
-    # #3681/#3722: the `coord:workflow` opt-in label — or `ultracode`, which
-    # implies it (operators read "ultracode" as "do the most thorough thing",
-    # not just "pick opus") — only for `type="work"` dispatches, mirroring
-    # `provider_issue_labels`'s identical `type == "work"` gate above (a
-    # label meant for the eventual work leg must not leak into a
-    # review/smoke/plan proposal). Only sent when True — never False — same
-    # "don't 400 an older agent on an unknown kwarg" discipline as every
-    # other optional field in this block; an agent predating
-    # `AssignmentSpec.workflow` simply runs the leg as a normal
-    # (non-workflow) Work leg. `is_workflow_leg` is the single source of
-    # truth for this question — `coord.state`'s recorded `mode` column
-    # calls the same function.
-    from coord.agent import is_workflow_leg  # noqa: PLC0415
-
-    if is_workflow_leg(proposal.type, proposal.issue_labels):
+    # #3681: the `coord:workflow` opt-in label — only for `type="work"`
+    # dispatches, mirroring `provider_issue_labels`'s identical
+    # `type == "work"` gate above (a label meant for the eventual work leg
+    # must not leak into a review/smoke/plan proposal). Only sent when
+    # True — never False — same "don't 400 an older agent on an unknown
+    # kwarg" discipline as every other optional field in this block; an
+    # agent predating `AssignmentSpec.workflow` simply runs the leg as a
+    # normal (non-workflow) Work leg.
+    from coord.agent import WORKFLOW_LEG_LABEL  # noqa: PLC0415
+
+    if proposal.type == "work" and WORKFLOW_LEG_LABEL in (proposal.issue_labels or []):
         payload["workflow"] = True
     # #2131: per-leg spend ceiling, resolved here (CLI/daemon lane) and
     # carried on the wire so a config-free agent is covered too. Sent ONLY

{code_coordinator-0.5.653 → code_coordinator-0.5.654}/coord/state.py

@@ -1064,20 +1064,23 @@ def _record_dispatched_local(
     # is never branch=NULL.  Mirrors agent.py:1021 exactly:
     #   branch_name = existing_branch or f"issue-{issue_number}-{_slugify(issue_title)}"
     # where proposal.target_branch maps to existing_branch.
-    from coord.agent import _slugify, is_workflow_leg  # noqa: PLC0415
+    from coord.agent import WORKFLOW_LEG_LABEL, _slugify  # noqa: PLC0415
 
     branch = proposal.target_branch or (
         f"issue-{proposal.issue_number}-{_slugify(proposal.issue_title)}"
     )
 
-    # #3681/#3722: "workflow" only for a `type="work"` dispatch whose issue
-    # carries the opt-in `coord:workflow` label, or `ultracode` (which
-    # implies it) — `is_workflow_leg` is the single source of truth for this
-    # question, also called by `coord.dispatch`'s wire-payload `workflow`
-    # flag, so the two surfaces cannot drift apart. Every other proposal
-    # type (test, review, fix, ...) and every unlabeled work issue records
-    # "single". See `coord.agent.AssignmentSpec.workflow`.
-    mode = "workflow" if is_workflow_leg(proposal.type, proposal.issue_labels) else "single"
+    # #3681: "workflow" only for a `type="work"` dispatch whose issue carries
+    # the opt-in `coord:workflow` label (mirrors coord.dispatch's own
+    # `proposal.type == "work"` gating for the wire-payload `workflow` flag)
+    # — every other proposal type (test, review, fix, ...) and every
+    # unlabeled work issue records "single". See
+    # `coord.agent.AssignmentSpec.workflow`.
+    mode = (
+        "workflow"
+        if proposal.type == "work" and WORKFLOW_LEG_LABEL in (proposal.issue_labels or [])
+        else "single"
+    )
 
     conn = get_connection()
     cur = sql.execute(conn,

{code_coordinator-0.5.653 → code_coordinator-0.5.654}/tests/test_dispatch.py

@@ -400,29 +400,6 @@ class TestDispatch:
         payload = mock_post.call_args.kwargs["json"]
         assert payload["workflow"] is True
 
-    @patch("coord.dispatch.httpx.post")
-    def test_payload_carries_workflow_true_for_ultracode_work_proposal(
-        self, mock_post: MagicMock, config: Config,
-    ) -> None:
-        """#3722: `ultracode` implies the `coord:workflow` opt-in on a
-        `type="work"` proposal, even with no explicit `coord:workflow`
-        label — operators read "ultracode" as "do the most thorough thing",
-        which means running as a workflow, not just picking the opus
-        model."""
-        mock_resp = MagicMock()
-        mock_resp.json.return_value = {"ok": True}
-        mock_post.return_value = mock_resp
-
-        p = Proposal(
-            id=1, machine_name="laptop", repo_name="api",
-            issue_number=3722, issue_title="run as a workflow via ultracode",
-            rationale="multi-agent leg", type="work",
-            issue_labels=["ultracode"],
-        )
-        dispatch(p, config)
-        payload = mock_post.call_args.kwargs["json"]
-        assert payload["workflow"] is True
-
     @patch("coord.dispatch.httpx.post")
     def test_payload_omits_workflow_when_label_absent(
         self, mock_post: MagicMock, config: Config, proposal: Proposal,
@@ -459,28 +436,6 @@ class TestDispatch:
         payload = mock_post.call_args.kwargs["json"]
         assert "workflow" not in payload
 
-    @patch("coord.dispatch.httpx.post")
-    def test_payload_omits_workflow_for_non_work_proposal_with_ultracode(
-        self, mock_post: MagicMock, config: Config,
-    ) -> None:
-        """#3722: `ultracode`'s workflow implication is gated to
-        `type="work"` the same way `coord:workflow` itself is — a review/
-        smoke/plan proposal carrying `ultracode` (e.g. from the same issue's
-        labels) never gets `workflow=True`."""
-        mock_resp = MagicMock()
-        mock_resp.json.return_value = {"ok": True}
-        mock_post.return_value = mock_resp
-
-        p = Proposal(
-            id=1, machine_name="laptop", repo_name="api",
-            issue_number=3722, issue_title="review the ultracode change",
-            rationale="review", type="review",
-            issue_labels=["ultracode"],
-        )
-        dispatch(p, config)
-        payload = mock_post.call_args.kwargs["json"]
-        assert "workflow" not in payload
-
     @patch("coord.dispatch.httpx.post")
     def test_payload_carries_coordinator_only_files(
         self, mock_post: MagicMock, proposal: Proposal,

{code_coordinator-0.5.653 → code_coordinator-0.5.654}/tests/test_state.py

@@ -1661,31 +1661,6 @@ class TestRecordDispatchedMode:
         row = next(a for a in board.active if a.assignment_id == assignment_id)
         assert row.mode == "workflow"
 
-    def test_ultracode_work_leg_records_workflow(self, coord_db) -> None:
-        """#3722: `ultracode` implies the `coord:workflow` opt-in even with
-        no explicit `coord:workflow` label on the issue."""
-        from coord.state import load_assignment_mode, record_dispatched
-
-        proposal = Proposal(
-            id=14,
-            machine_name="precision",
-            repo_name="myrepo",
-            issue_number=3722,
-            issue_title="Run this as a workflow via ultracode",
-            rationale="test",
-            briefing="run it thoroughly",
-            type="work",
-            issue_labels=["ultracode"],
-        )
-        assignment_id = "aid-3722-ultracode"
-        record_dispatched(
-            assignment_id=assignment_id,
-            proposal=proposal,
-            repo_github="acme/myrepo",
-        )
-
-        assert load_assignment_mode(assignment_id) == "workflow"
-
     def test_normal_work_leg_records_single(self, coord_db) -> None:
         from coord.state import load_assignment_mode, record_dispatched
 

code_coordinator-0.5.653/code_coordinator.egg-info/scm_version.json

@@ -1,8 +0,0 @@
-{
-  "tag": "0.5.653",
-  "distance": 0,
-  "node": "g034c4ce50582442b4d52149298a699c921b2d2ec",
-  "dirty": false,
-  "branch": "HEAD",
-  "node_date": "2026-10-18"
-}

code_coordinator-0.5.653/tests/test_ultracode_workflow_3722.py

@@ -1,38 +0,0 @@
-"""#3722: `ultracode` must imply `coord.agent.WORKFLOW_LEG_LABEL`'s workflow
-opt-in for `type="work"` legs, not just select the opus model.
-
-`coord.agent.is_workflow_leg` is the single source of truth this issue
-introduces — both `coord/dispatch.py` (the wire `AssignmentSpec.workflow`
-payload, covered in `tests/test_dispatch.py`) and `coord/state.py` (the
-recorded assignment `mode` column, covered in `tests/test_state.py`) call
-it rather than re-deriving the condition. This file exercises the function
-directly.
-"""
-
-from __future__ import annotations
-
-from coord.agent import ULTRACODE_LABEL, WORKFLOW_LEG_LABEL, is_workflow_leg
-
-
-class TestIsWorkflowLeg:
-    def test_ultracode_label_implies_workflow_for_work_leg(self) -> None:
-        assert is_workflow_leg("work", ["ultracode"]) is True
-
-    def test_workflow_label_still_implies_workflow_for_work_leg(self) -> None:
-        assert is_workflow_leg("work", [WORKFLOW_LEG_LABEL]) is True
-
-    def test_no_opt_in_label_is_not_a_workflow_leg(self) -> None:
-        assert is_workflow_leg("work", ["bug"]) is False
-        assert is_workflow_leg("work", None) is False
-        assert is_workflow_leg("work", []) is False
-
-    def test_ultracode_on_non_work_leg_is_not_a_workflow_leg(self) -> None:
-        """`ultracode`'s workflow implication is gated to `type="work"`,
-        mirroring the existing `WORKFLOW_LEG_LABEL` gate — a review/test/fix
-        leg on the same issue must not opt in."""
-        for leg_type in ("review", "test", "fix", "plan"):
-            assert is_workflow_leg(leg_type, ["ultracode"]) is False
-            assert is_workflow_leg(leg_type, [WORKFLOW_LEG_LABEL]) is False
-
-    def test_ultracode_label_constant(self) -> None:
-        assert ULTRACODE_LABEL == "ultracode"