code-coordinator 0.5.158__tar.gz → 0.5.163__tar.gz

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/PKG-INFO

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

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

@@ -806,6 +806,7 @@ tests/acceptance/ms-65/contract.md
 tests/acceptance/ms-65/detail_substate_2285.rs
 tests/acceptance/ms-65/manifest.yml
 tests/acceptance/ms-65/pipeline_tabs_2284.rs
+tests/acceptance/ms-65/tabs_discoverability_2287.rs
 tests/acceptance/ms-65/mocks/board-baseline-no-tabs.screen
 tests/acceptance/ms-65/mocks/board-pinned-3-tabs.screen
 tests/acceptance/ms-65/mocks/board-preview-tab.screen

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/code_coordinator.egg-info/scm_file_list.json

@@ -454,6 +454,7 @@
     "tests/acceptance/ms-65/mocks/board-tabs-overflow.screen",
     "tests/acceptance/ms-65/mocks/pipeline-tabs-independent.screen",
     "tests/acceptance/ms-65/pipeline_tabs_2284.rs",
+    "tests/acceptance/ms-65/tabs_discoverability_2287.rs",
     "tests/acceptance/ms-example/mocks/home-active.html",
     "tests/conftest.py",
     "tests/fixtures/board-pipeline-basic.json",

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/code_coordinator.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.158",
+  "tag": "0.5.163",
   "distance": 0,
-  "node": "g4e440181f25b4e02cfc8fc64b76c584c7e63a197",
+  "node": "g6fa07c5850b078c8d3d43576ceee2b342698e255",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-19"

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/coord/_board_mapping.py

@@ -161,6 +161,12 @@ def row_to_assignment(row: object) -> Assignment:
         # #2316: raw terminal stop_reason; None for rows predating this
         # column or a worker whose log carried no such field.
         stop_reason=d.get("stop_reason"),
+        # #2417: the calling worker's own assignment id, when this row was
+        # dispatched from INSIDE another worker's turn (e.g. `coord
+        # acceptance author`/`coord fix` shelled out to from a `type="work"`
+        # session's own bash tool). None for a hand or coordinator/brain
+        # dispatch, and for rows predating this column.
+        dispatched_by_assignment_id=d.get("dispatched_by_assignment_id"),
     )
 
 

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/coord/dashboard/webapp/src/api/generated.ts

@@ -174,4 +174,5 @@ export interface Assignment {
   for_issue_number: number | null
   driven_by: string | null
   stop_reason: string | null
+  dispatched_by_assignment_id: string | null
 }

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/coord/db.py

@@ -926,6 +926,14 @@ def _migrate_add_columns(conn: sqlite3.Connection) -> None:
         # submission with no preview queued yet, same as `open_question`'s
         # own default.
         "ALTER TABLE portal_submissions ADD COLUMN preview_url TEXT NOT NULL DEFAULT ''",
+        # #2417: the CALLING assignment's id when this row was dispatched by
+        # a `coord` subcommand run from INSIDE another worker's own turn
+        # (e.g. `coord acceptance author`/`coord fix` shelled out to from a
+        # `type="work"` session's own bash tool) rather than typed by a
+        # human — see `coord.models.Assignment.dispatched_by_assignment_id`.
+        # NULL for a hand dispatch, a coordinator/brain-proposed dispatch,
+        # and every row predating this column.
+        "ALTER TABLE assignments ADD COLUMN dispatched_by_assignment_id TEXT",
     ]
     for sql in migrations:
         try:

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/coord/drive.py

@@ -396,6 +396,22 @@ class DriveCounters:
     # overwhelming majority of drives — non-oracle ones — never allocate it,
     # and `--dry-run`'s counter snapshot stays unchanged for them.
     acceptance: "DriveCounters | None" = None
+    # #2416: bounded retry for an ADVISORY work row whose branch carries zero
+    # commits — `_decide_advisory`'s dead-end signature. Before this, that
+    # branch went straight to `_die()` on every observation, so the only
+    # thing that could ever supersede the terminal row was a human running
+    # `coord retry <aid>` by hand (`coord diagnose --stage work --reset` is
+    # NOT the escape hatch here — it never sets `needs_reset` for this
+    # shape). `drive-queue`'s own automatic retry does not take that path
+    # either: it just relaunches a brand new `coord drive` process, which
+    # re-observes the SAME terminal row and dies again in seconds, burning a
+    # queue attempt+backoff cycle for nothing (coord-portal#119, drive-queue
+    # entry 432). Bounded the same way `work_retries` is (`opts.
+    # max_work_retries`) so a genuinely unactionable row (the issue's real
+    # resolution is a sibling task, not another attempt) still reaches a
+    # terminal `_die()` — never an unbounded loop — once the budget is
+    # spent.
+    advisory_retries: int = 0
 
     def slice_budget(self) -> "DriveCounters":
         """This run's slice-landing budget, created on first use (#2079)."""
@@ -1603,8 +1619,15 @@ def decide(
     if state.work_status == "done":
         pass
     elif state.work_status == "advisory":
-        advisory = _decide_advisory(state, opts, machine, verifier)
-        if advisory.is_exit:
+        advisory = _decide_advisory(state, opts, counters, machine, verifier)
+        # #2416: `_decide_advisory` now returns a RUN action (a bounded
+        # `coord retry` dispatch) for the zero-commit case, not only an EXIT
+        # — `is_exit` alone would let a RUN action fall through to the
+        # Test/Review/Merge logic below as though the row were validated
+        # work. Only its "proceeding under --accept-advisory" branch returns
+        # WAIT (with warnings to fold in); every other branch (EXIT or RUN)
+        # is this poll's whole answer.
+        if advisory.kind != WAIT:
             return advisory
         warnings = advisory.warnings
     elif state.work_status == "refused_policy":
@@ -1825,6 +1848,7 @@ def _dispatch_work_stage(
 def _decide_advisory(
     state: IssueState,
     opts: DriveOptions,
+    counters: DriveCounters,
     machine: str,
     verifier: MergeVerifier,
 ) -> Action:
@@ -1835,13 +1859,47 @@ def _decide_advisory(
     ``tui/target/debug/coord-tui``, which a Python diff never produces, so
     #1323's stash-miss check downgrades a perfectly good DONE.  Ask git which
     case this actually is rather than trusting the status.
+
+    #2416: a GENUINE zero-commit advisory (no branch, or a branch verified to
+    carry no commits) used to be an immediate, unconditional `_die()` — the
+    ONE thing that can actually change this row, `coord retry <aid>`'s
+    zero-commit-advisory path (#1606: `ahead == 0` → reassign a fresh
+    worker), was never invoked automatically. That made every automatic
+    retry — `drive-queue`'s own included — a no-op: a brand new `coord
+    drive` process just re-observes the identical terminal row and dies
+    again in seconds, spending a queue attempt+backoff cycle for nothing.
+    Mirrors the `work_status == "failed"` branch above: a bounded number of
+    `coord retry` dispatches (`opts.max_work_retries`, via `counters.
+    advisory_retries`) before finally giving up, so a row whose correct
+    resolution really is "dispatch something else" still reaches a terminal
+    `_die()` naming `coord retry` for a human — never an unbounded loop.
     """
     if not state.work_branch or not verifier.branch_has_commits(state):
-        return _die(
-            f"work {state.work_aid} exited ADVISORY with no commits on its branch —\n"
-            "   nothing was pushed, so there is nothing to test, review, or merge.\n"
-            f"   inspect: coord log {state.work_aid} --machine "
-            f"{state.work_machine or machine}"
+        budget = opts.max_work_retries
+        if counters.advisory_retries >= budget:
+            return _die(
+                f"work {state.work_aid} exited ADVISORY with no commits on its "
+                f"branch {counters.advisory_retries} time(s) in a row (budget "
+                f"{budget}, #2416) — nothing was pushed, so there is nothing to "
+                "test, review, or merge, and retrying has not produced a "
+                "different outcome.\n"
+                f"   inspect: coord log {state.work_aid} --machine "
+                f"{state.work_machine or machine}\n"
+                f"   this needs an operator decision: `coord retry "
+                f"{state.work_aid}` by hand once the underlying blocker is "
+                "understood, or dispatch an independent follow-up issue if "
+                "another attempt at this one is not the right fix."
+            )
+        counters.advisory_retries += 1
+        return Action(
+            kind=RUN,
+            label=(
+                f"WORK: {state.work_aid} exited ADVISORY with no commits → "
+                f"coord retry {state.work_aid} (attempt "
+                f"{counters.advisory_retries}/{budget}, #2416)"
+            ),
+            command=("retry", state.work_aid),
+            error_message=f"coord retry failed for {state.work_aid}",
         )
     if not opts.accept_advisory:
         return _die(

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/coord/models.py

@@ -723,6 +723,29 @@ class Assignment:
     # `advisory` — the same bucket as a worker that looked and correctly
     # found nothing to do — so nobody re-drove it.
     stop_reason: str | None = None
+    # #2417: the CALLING assignment's id when this row was dispatched by a
+    # `coord` subcommand run from INSIDE another worker's own turn (e.g. a
+    # `type="work"` session shelling out to `coord acceptance author repo ms
+    # --issue N` or `coord fix <other-id>`), as opposed to a human typing the
+    # same command in their own shell. Populated centrally in
+    # `coord.state.record_dispatched`/`record_dispatched_assignment` from the
+    # `COORD_ASSIGNMENT_ID` env var (see `coord.agent._build_worker_env`'s
+    # #2217 note — every headless worker's subprocess environment carries its
+    # own assignment id, so a `coord` CLI the worker shells out to inherits
+    # it automatically; a human's own shell never has it set). `None` for a
+    # hand dispatch, a coordinator/brain-proposed dispatch, and rows
+    # predating this column.
+    #
+    # This is the missing link #2417 reported: before this field, the only
+    # way to discover that a work row had dispatched an independent sibling
+    # assignment (and whether that sibling then succeeded) was grepping the
+    # worker's raw `claude -p` transcript for the printed "Dispatched ... to
+    # ..." line and manually cross-referencing a second `coord log` by hand.
+    # With this field, the ORIGIN row can be found by any consumer (the
+    # board, `coord audit`, the TUI) via a reverse lookup: "which assignment
+    # has `dispatched_by_assignment_id == this row's id`?" — see
+    # `coord.state.find_dispatched_children`.
+    dispatched_by_assignment_id: str | None = None
 
 
 def effective_issue_number(assignment: "Assignment | dict") -> int:
@@ -859,6 +882,15 @@ class Proposal:
     # through) carries durable provenance. `None` for every other caller
     # (brain-proposed, milestone dispatch, ...).
     driven_by: str | None = None
+    # #2417: mirrors `Assignment.dispatched_by_assignment_id` — see that
+    # field's docstring. Set here (rather than left for the caller to
+    # populate) by `coord.state.record_dispatched` reading
+    # `COORD_ASSIGNMENT_ID` from the environment at record time, so every
+    # `Proposal`-based dispatch path (`coord fix`/`coord pr`/`coord review`
+    # follow-ups via `_dispatch_followup`, `coord assign`,
+    # `coord/dispatch_workers.py`, ...) gets this for free without each call
+    # site threading it through by hand.
+    dispatched_by_assignment_id: str | None = None
 
 
 @dataclass

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/coord/state.py

@@ -17,7 +17,7 @@ import os
 import sqlite3
 import time
 import warnings
-from dataclasses import asdict
+from dataclasses import asdict, replace
 from datetime import datetime, timezone
 from pathlib import Path
 from typing import TYPE_CHECKING
@@ -561,6 +561,10 @@ def _dispatched_dict_from_payload(a: dict) -> dict:
         "provider_name": a.get("provider_name"),
         # #1499: durable drive provenance; None for a hand `coord assign`.
         "driven_by": a.get("driven_by"),
+        # #2417: the calling worker's own assignment id, when this row was
+        # dispatched from INSIDE another worker's turn. None for a hand or
+        # coordinator/brain dispatch.
+        "dispatched_by_assignment_id": a.get("dispatched_by_assignment_id"),
     }
 
 
@@ -605,6 +609,10 @@ def _row_to_dispatched_dict(row: object) -> dict:
         "provider_name": d.get("provider_name"),
         # #1499: durable drive provenance; None for a hand `coord assign`.
         "driven_by": d.get("driven_by"),
+        # #2417: the calling worker's own assignment id, when this row was
+        # dispatched from INSIDE another worker's turn. None for a hand or
+        # coordinator/brain dispatch.
+        "dispatched_by_assignment_id": d.get("dispatched_by_assignment_id"),
     }
 
 
@@ -799,6 +807,21 @@ def _thin_client_local_board_guard(fn_name: str) -> None:
     _log.warning(msg)
 
 
+def _dispatched_by_from_env() -> str | None:
+    """The calling worker's own assignment id, if this process is running
+    INSIDE a headless worker's turn (#2417).
+
+    `COORD_ASSIGNMENT_ID` is set on a worker subprocess's own environment by
+    `coord.agent._build_worker_env` (#2217) — so a `coord` CLI invocation
+    the worker shells out to (e.g. `coord acceptance author` dispatching an
+    independent test-author sibling, `coord fix <other-id>` escalating an
+    unrelated assignment) inherits it automatically. A human typing the same
+    command in their own shell never has it set, so this correctly returns
+    `None` for a hand/coordinator dispatch.
+    """
+    return os.environ.get("COORD_ASSIGNMENT_ID") or None
+
+
 def record_dispatched(
     *,
     assignment_id: str,
@@ -807,6 +830,15 @@ def record_dispatched(
     provider_name: str | None = None,
 ) -> None:
     """Record a newly dispatched assignment — routes to the daemon when set."""
+    # #2417: stamp the calling worker's own assignment id (if any) BEFORE
+    # this crosses the HTTP boundary to a possibly-remote daemon — the env
+    # var only exists on the machine/process that is actually dispatching,
+    # never on the daemon host. Caller-supplied values (none of today's
+    # callers set one explicitly) are left alone.
+    if proposal.dispatched_by_assignment_id is None:
+        proposal = replace(
+            proposal, dispatched_by_assignment_id=_dispatched_by_from_env()
+        )
     svc = _board_service()
     resp = _route_write(
         svc,
@@ -874,8 +906,8 @@ def _record_dispatched_local(
             assignment_id, machine_name, repo_name, repo_github,
             issue_number, issue_title, status, type, briefing,
             files_allowed, model, dispatched_at, required_gates,
-            provider_name, branch, driven_by
-        ) VALUES (?, ?, ?, ?, ?, ?, 'running', ?, ?, ?, ?, ?, ?, ?, ?, ?)
+            provider_name, branch, driven_by, dispatched_by_assignment_id
+        ) VALUES (?, ?, ?, ?, ?, ?, 'running', ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
         ON CONFLICT(assignment_id) DO NOTHING""",
         (
             assignment_id,
@@ -893,6 +925,7 @@ def _record_dispatched_local(
             provider_name,
             branch,
             proposal.driven_by,
+            proposal.dispatched_by_assignment_id,
         ),
     )
     conn.commit()
@@ -915,7 +948,16 @@ def _record_dispatched_local(
             # the audit log alone, after the driver process has exited.
             actor="drive" if proposal.driven_by else "coordinator",
             summary=f"Dispatched {proposal.type} to {proposal.machine_name}: "
-            f"{proposal.repo_name}#{proposal.issue_number}",
+            f"{proposal.repo_name}#{proposal.issue_number}"
+            # #2417: make the sibling-dispatch link visible in the default
+            # (non-`--json`) `coord audit` table, not just in `details` —
+            # this is what let coord-portal#119's dispatch of a test-author
+            # sibling go unnoticed without grepping the raw worker transcript.
+            + (
+                f" (dispatched by assignment {proposal.dispatched_by_assignment_id})"
+                if proposal.dispatched_by_assignment_id
+                else ""
+            ),
             repo=proposal.repo_name,
             issue=proposal.issue_number,
             assignment_id=assignment_id,
@@ -924,6 +966,11 @@ def _record_dispatched_local(
                 "type": proposal.type,
                 "branch": branch,
                 "driven_by": proposal.driven_by,
+                # #2417: surfaces "this was dispatched BY a worker's own
+                # turn" directly in `coord audit`, without cross-referencing
+                # the raw `claude -p` transcript for the printed "Dispatched
+                # ... to ..." line.
+                "dispatched_by_assignment_id": proposal.dispatched_by_assignment_id,
             },
         )
 
@@ -934,6 +981,12 @@ def record_dispatched_assignment(
     repo_github: str,
 ) -> None:
     """Record a dispatched assignment — routes to the daemon when set."""
+    # #2417: see record_dispatched's matching comment — must be stamped here,
+    # on the dispatching process's own env, before the daemon HTTP hop.
+    if assignment.dispatched_by_assignment_id is None:
+        assignment = replace(
+            assignment, dispatched_by_assignment_id=_dispatched_by_from_env()
+        )
     svc = _board_service()
     resp = _route_write(
         svc, "/dispatched", {"assignment": asdict(assignment), "repo_github": repo_github}
@@ -971,7 +1024,8 @@ def _record_dispatched_assignment_local(
             issue_number, issue_title, status, type, briefing,
             files_allowed, model, dispatched_at, review_of_assignment_id,
             review_target, required_gates, review_iteration,
-            provider_name, branch, for_issue_number, driven_by
+            provider_name, branch, for_issue_number, driven_by,
+            dispatched_by_assignment_id
         ) VALUES (?, ?, ?, ?, ?, ?, 'running', ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,
             -- #1553: a follow-up dispatched off another assignment (review,
             -- smoke, [fix-N], retry, pr-helper) inherits that parent's
@@ -989,7 +1043,7 @@ def _record_dispatched_assignment_local(
                 SELECT p.for_issue_number FROM assignments p
                 WHERE p.assignment_id = ?
             )),
-            ?)
+            ?, ?)
         ON CONFLICT(assignment_id) DO UPDATE SET
             status = 'running',
             machine_name = excluded.machine_name,
@@ -1013,7 +1067,13 @@ def _record_dispatched_assignment_local(
             for_issue_number = COALESCE(excluded.for_issue_number, for_issue_number),
             -- #1499: COALESCE so a re-dispatch/reload doesn't clear the
             -- drive provenance already recorded for this assignment.
-            driven_by = COALESCE(excluded.driven_by, driven_by)""",
+            driven_by = COALESCE(excluded.driven_by, driven_by),
+            -- #2417: COALESCE so a re-dispatch/reload doesn't clear the
+            -- calling-worker provenance already recorded for this
+            -- assignment.
+            dispatched_by_assignment_id = COALESCE(
+                excluded.dispatched_by_assignment_id, dispatched_by_assignment_id
+            )""",
         (
             assignment.assignment_id or "",
             assignment.machine_name,
@@ -1039,6 +1099,7 @@ def _record_dispatched_assignment_local(
             # positional).
             assignment.review_of_assignment_id,
             assignment.driven_by,
+            assignment.dispatched_by_assignment_id,
         ),
     )
     conn.commit()
@@ -1051,7 +1112,13 @@ def _record_dispatched_assignment_local(
         # to a human `coord assign` (the exact gap #1499 reported).
         actor="drive" if assignment.driven_by else "coordinator",
         summary=f"Dispatched {assignment.type} to {assignment.machine_name}: "
-        f"{assignment.repo_name}#{assignment.issue_number}",
+        f"{assignment.repo_name}#{assignment.issue_number}"
+        # #2417: see the matching Proposal-path comment above.
+        + (
+            f" (dispatched by assignment {assignment.dispatched_by_assignment_id})"
+            if assignment.dispatched_by_assignment_id
+            else ""
+        ),
         repo=assignment.repo_name,
         issue=assignment.issue_number,
         assignment_id=assignment.assignment_id,
@@ -1062,6 +1129,8 @@ def _record_dispatched_assignment_local(
             "review_target": assignment.review_target,
             "review_iteration": assignment.review_iteration,
             "driven_by": assignment.driven_by,
+            # #2417: see record_dispatched's matching comment.
+            "dispatched_by_assignment_id": assignment.dispatched_by_assignment_id,
         },
     )
 

{code_coordinator-0.5.158 → code_coordinator-0.5.163}/tests/acceptance/ms-65/manifest.yml

@@ -143,6 +143,50 @@ tests:
   # THIS IS A GREEN RATCHET — see the `expected_red` block below.
   detail_substate_2285::a_single_open_tab_keeps_todays_sub_tab_and_scroll_behaviour: 2285
 
+  # ── #2287 (T-5) — discoverability: status bar / `?` help / right-click ────
+  # Slice: tabs_discoverability_2287.rs · mocks: board-tab-context-menu.screen,
+  # board-tabs-help-overlay.screen (plus board-preview-tab.screen and
+  # board-baseline-no-tabs.screen — §8a's own hint-present / hint-absent
+  # reference pair)
+  #
+  # §8a — with >=1 tab open the status bar advertises the tab gestures
+  tabs_discoverability_2287::status_bar_shows_the_tab_hints_while_a_tab_is_open: 2287
+  # §8a — …as the contract's pinned, verbatim substring (two spaces between
+  # segments). Split from the clause above so a spacing/ordering deviation
+  # reds exactly one id.
+  tabs_discoverability_2287::status_bar_tab_hint_is_the_contracts_verbatim_string: 2287
+  # §8a + #2287 AC — the dynamic half: the hint goes away again when the last
+  # tab closes (a latched hint passes the static control but fails this)
+  tabs_discoverability_2287::status_bar_tab_hints_disappear_when_the_last_tab_closes: 2287
+  # §8a — CONTROL: zero tabs => no tab hint, and the Board's own pre-ms-65
+  # hints survive. THIS IS THE GREEN CONTROL — see `expected_red` below.
+  tabs_discoverability_2287::zero_tabs_status_bar_keeps_the_boards_own_hints_and_shows_no_tab_hints: 2287
+  # §8b — `?` opens an overlay titled "Board — Help"
+  tabs_discoverability_2287::help_overlay_opens_on_question_mark_with_the_board_title: 2287
+  # §8b — …listing the four pinned key-binding phrases ("open preview tab",
+  # "pin tab", "close active tab", "cycle tabs")
+  tabs_discoverability_2287::help_overlay_lists_the_document_tab_key_bindings: 2287
+  # §8b — …and the two mouse gestures the status bar cannot advertise,
+  # under the "Document tabs" heading
+  tabs_discoverability_2287::help_overlay_lists_the_mouse_tab_gestures_under_a_document_tabs_heading: 2287
+  # §8b — ` Esc=close ` while open, and `Esc` closes it
+  tabs_discoverability_2287::help_overlay_advertises_and_honours_escape_to_close: 2287
+  # §8c — right-clicking a tab opens the four-item menu (Close checked as a
+  # standalone item, not as a prefix of its two longer siblings)
+  tabs_discoverability_2287::right_clicking_a_tab_opens_a_menu_with_close_close_others_close_all_and_pin_tab: 2287
+  # §8c — "Close others" leaves exactly the right-clicked (middle) tab
+  tabs_discoverability_2287::close_others_leaves_exactly_the_right_clicked_tab: 2287
+  # §8c — "Close all" leaves none and restores the zero-tab state
+  tabs_discoverability_2287::close_all_leaves_no_tabs_and_restores_the_zero_tab_state: 2287
+  # §8c — "Close" closes the right-clicked tab and no other (see finding 16)
+  tabs_discoverability_2287::close_menu_item_closes_the_right_clicked_tab_and_no_other: 2287
+  # §8c + #2287 AC — "Pin tab" promotes a preview tab (drops `∘ `) without
+  # changing count or order
+  tabs_discoverability_2287::pin_tab_promotes_a_preview_tab_without_changing_count_or_order: 2287
+  # §8c — "hidden or inert" on an already-pinned tab, asserted through the
+  # outcome both readings share (see finding 17)
+  tabs_discoverability_2287::pin_tab_is_hidden_or_inert_on_an_already_pinned_tab: 2287
+
 # ── expected_red (#2164) ───────────────────────────────────────────────────
 # Hand-derived from a REAL run of the sealed suite against origin/main at the
 # time of authoring (`cd tui && cargo test --test acceptance --features
@@ -249,6 +293,40 @@ expected_red:
     - detail_substate_2285::closing_a_tab_discards_its_sub_state_and_reopening_starts_from_the_defaults
     - detail_substate_2285::terminal_sub_tab_selection_follows_its_own_tab
 
+  # #2287 — OBSERVED, not intended. Taken from two consecutive real runs of
+  # the sealed suite (`cd tui && RUSTC_BOOTSTRAP=1 cargo test --test acceptance
+  # --features test-support -- -Z unstable-options --format json`, the driver
+  # command in coordinator.yml) at authoring time: of this slice's 14 ids, the
+  # same 13 FAILED and the same 1 passed on both runs. Every one of the 13 died
+  # on a §8 clause, never on a §2/§4 precondition — verified by reading the
+  # panic line of each failure (the §8c ones all land in `assert_menu_item` /
+  # `click_menu_item`, i.e. "no such menu"; the §8a/§8b ones on the missing
+  # hint string and the missing overlay). The 1 that passed is deliberately
+  # ABSENT below:
+  #
+  #   * `zero_tabs_status_bar_keeps_the_boards_own_hints_and_shows_no_tab_hints`
+  #     — the CONTROL. Green today (nothing renders "click=preview", and the
+  #     Board status bar already reads `… n=notify  m=merge  R=retry  P=purge
+  #     q=quit`) and it must STAY green after #2287 lands: an implementation
+  #     that appends the tab hints unconditionally, or that replaces the
+  #     Board's own hint set with them outright, passes every §8a test above
+  #     and breaks exactly this one. Listing it would make its inevitable pass
+  #     a HARD FAILURE.
+  2287:
+    - tabs_discoverability_2287::status_bar_shows_the_tab_hints_while_a_tab_is_open
+    - tabs_discoverability_2287::status_bar_tab_hint_is_the_contracts_verbatim_string
+    - tabs_discoverability_2287::status_bar_tab_hints_disappear_when_the_last_tab_closes
+    - tabs_discoverability_2287::help_overlay_opens_on_question_mark_with_the_board_title
+    - tabs_discoverability_2287::help_overlay_lists_the_document_tab_key_bindings
+    - tabs_discoverability_2287::help_overlay_lists_the_mouse_tab_gestures_under_a_document_tabs_heading
+    - tabs_discoverability_2287::help_overlay_advertises_and_honours_escape_to_close
+    - tabs_discoverability_2287::right_clicking_a_tab_opens_a_menu_with_close_close_others_close_all_and_pin_tab
+    - tabs_discoverability_2287::close_others_leaves_exactly_the_right_clicked_tab
+    - tabs_discoverability_2287::close_all_leaves_no_tabs_and_restores_the_zero_tab_state
+    - tabs_discoverability_2287::close_menu_item_closes_the_right_clicked_tab_and_no_other
+    - tabs_discoverability_2287::pin_tab_promotes_a_preview_tab_without_changing_count_or_order
+    - tabs_discoverability_2287::pin_tab_is_hidden_or_inert_on_an_already_pinned_tab
+
 # ═══════════════════════════════════════════════════════════════════════════
 # FOR THE COORDINATOR — three findings from authoring this slice. None of them
 # is a worker's to fix, and none was resolved by editing the contract (a
@@ -506,3 +584,88 @@ expected_red:
 # And this slice deliberately does NOT assert the sub-tab-switch scroll reset
 # the app performs today (`detail_scroll = 0` on a sub-tab click): no clause
 # pins it either way, so pinning it could force a wrong implementation.
+
+# ═══════════════════════════════════════════════════════════════════════════
+# FOR THE COORDINATOR — findings from authoring the #2287 slice (contract §8).
+# Independent of the #2282 / #2283 / #2284 / #2285 notes above; again, none is
+# a worker's to fix by editing this suite, and none was resolved by editing
+# the contract (a test-author may not amend one).
+# ═══════════════════════════════════════════════════════════════════════════
+#
+# 14. §8b's `?` overlay has TWO sections and the second one is #2288's, not
+#     #2287's. §8b pins "Document tabs" AND "Split (§9's four keys)", and
+#     `mocks/board-tabs-help-overlay.screen` renders the four split rows
+#     (`Ctrl-W v  split right`, `Ctrl-W s  split down`, `Ctrl-W w  next pane`,
+#     `Ctrl-W x  close pane`). This slice asserts the Document-tabs section
+#     only. #2287's own acceptance criteria say just "`?` lists the tab key
+#     bindings"; the split keys belong to §9 (#2288), which the milestone's
+#     work order puts AFTER this issue, and §9 note 2 says `Ctrl-W s` is a
+#     reserved placeholder that does nothing in ms-65. Asserting those rows
+#     here would gate #2287's acceptance run on a later issue's surface — and,
+#     for `Ctrl-W s`, on a binding the contract says must NOT work yet. If the
+#     Split section is meant to be gated by acceptance, it wants an id in
+#     #2288's slice (the overlay is #2287's to build; the rows it lists for
+#     split are #2288's to justify). Flagged as a `TODO(test-author)` in the
+#     slice too.
+#
+# 15. §8a's hint is pinned "whenever the active panel owns a tab set", but the
+#     Pipeline half is not asserted. Pipeline owns its own set (§3), so the
+#     hint presumably applies there too, but the contract names no
+#     Pipeline hint-present mock and finding 7 above (the #2284 author's) is a
+#     live blocker: entering the Pipeline panel from this external test crate
+#     can wholesale-replace the seeded fixture, because `start_data_load()`'s
+#     deterministic short-circuit is gated `#[cfg(test)]` only. Rather than
+#     add a fourteenth id with a known race, this slice asserts §8a on the
+#     Board only — where both of §8a's own reference mocks live. If finding
+#     7's one-line cfg widening lands, a Pipeline hint id is cheap to add.
+#
+# 16. §8c does not say what the menu's **"Close"** item does. The section
+#     spells out the effect of "Close others", "Close all" and "Pin tab" and
+#     is silent on the first item. `close_menu_item_closes_the_right_clicked_
+#     tab_and_no_other` takes the only reading the label admits, and the one
+#     §4 already pins for the `×` route: the RIGHT-CLICKED tab closes, the
+#     others survive. The fixture deliberately right-clicks a tab that is NOT
+#     the active one (#102 while #103 is active) so the two readings are
+#     distinguishable — if "Close" was meant to close the *active* tab, that
+#     needs a contract amendment, not a silent implementation choice.
+#     Flagged as a `TODO(test-author)` in the slice.
+#
+# 17. §8c's "hidden **or** inert" is permissive, so neither the presence nor
+#     the absence of "Pin tab" on an already-pinned tab is assertable on its
+#     own. `pin_tab_is_hidden_or_inert_on_an_already_pinned_tab` asserts the
+#     outcome BOTH readings share instead — the tab set is unchanged (count,
+#     order, and still no `∘ ` marker anywhere) — taking the item's branch
+#     when it renders and dismissing the menu when it does not. It is not
+#     vacuous: it first requires the menu to open on a pinned tab at all
+#     (§8c gates only the one item, not the menu), which is why it was RED at
+#     authoring time.
+#
+# 18. §8b's overlay is driven with ZERO tabs open, though
+#     `mocks/board-tabs-help-overlay.screen` depicts it over a 3-tab board.
+#     §8b's testable clause carries no tab precondition ("`driver.press('?')`
+#     while the Board panel is active"), and #2287's own rationale — the
+#     gestures are "undiscoverable by anyone who did not read the issue" — is
+#     at its strongest before the operator has opened anything. If the tab
+#     section of the overlay is meant to be conditional on an open tab set,
+#     that is a contract amendment; as written, four ids here would be red
+#     against such an implementation.
+#
+# Also worth knowing, not a contract problem: (a) the shipped Board status bar
+# is a full row of segments — `coord-tui  Board  [Sidebar]  ↻ 0s  QUEUE: empty
+# FLEET: OK  (coord health  n=notify  m=merge  R=retry  P=purge  q=quit` — and
+# is already truncated at 120 columns, whereas §8a's own reference mock renders
+# only ` click=preview  dbl-click=pin  ctrl-w=close  ctrl-tab=next  q=quit`.
+# The control asserts the four Board hint segments (`n=notify`, `R=retry`,
+# `P=purge`, `q=quit`) survive in the ZERO-tab state only, which is the only
+# state §8a calls "unchanged"; it deliberately pins nothing about what the row
+# looks like once the tab hints are in it, since the mock implies the left-hand
+# segments give way. (b) `"Close"` is a prefix of both `"Close others"` and
+# `"Close all"`, so a bare `screen_contains("Close")` would pass on a menu
+# missing the item entirely; the slice's `menu_item_row()` rejects a match
+# whose label is immediately followed by another item's continuation. It does
+# NOT require the row to end after the label — §8c pins no accelerator column
+# for this menu, but ms-38's menus do render one (`Refresh … r`), so an item
+# carrying an accelerator must still count. (c) §8c pins no gesture for
+# activating an open menu item; a left click on the item's own label is the
+# only activation path any context menu in this app already documents, so that
+# is what the slice sends.