code-coordinator 0.5.167__tar.gz → 0.5.169__tar.gz

{code_coordinator-0.5.167 → code_coordinator-0.5.169}/PKG-INFO

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

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

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.167",
+  "tag": "0.5.169",
   "distance": 0,
-  "node": "g9016461101b4920502a7937e28cfdc70410ffc2f",
+  "node": "g1ed3da45c0f609b9b5d68fe5a65fc8e31737b7da",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-19"

{code_coordinator-0.5.167 → code_coordinator-0.5.169}/coord/drive.py

@@ -1254,6 +1254,30 @@ class MergeVerifier(Protocol):
     def branch_head_sha(self, state: IssueState) -> str | None: ...
 
 
+def _remote_matches_repo(remote_url: str, repo_github: str) -> bool:
+    """Loosely compare a ``git remote get-url origin`` URL against the
+    ``owner/repo`` GitHub identifier it should point at (#2437).
+
+    Tolerant of the URL shapes git actually produces: ``https://github.com/
+    owner/repo(.git)``, ``git@github.com:owner/repo(.git)``, and
+    ``ssh://git@github.com/owner/repo(.git)``. Anything else (a bare path, a
+    non-GitHub host) simply won't match — which is the correct, fail-closed
+    outcome: this is a sanity check that the checkout points at the expected
+    project, not a general-purpose git URL parser.
+    """
+    url = remote_url.strip()
+    if url.endswith(".git"):
+        url = url[:-4]
+    if "://" in url:
+        _scheme, _, rest = url.partition("://")
+        _host, _, path = rest.partition("/")
+        url = path
+    elif "@" in url and ":" in url:
+        _user_host, _, path = url.partition(":")
+        url = path
+    return url.strip("/").lower() == repo_github.strip("/").lower()
+
+
 @dataclass
 class GitMergeVerifier:
     """Real implementation: ``git`` for commits, ``gh`` for merge state.
@@ -1264,12 +1288,60 @@ class GitMergeVerifier:
 
     repo_path: str = ""
     warn: Callable[[str], None] = lambda msg: None
+    # #2437: memoizes which (base, reason) pairs have already been warned
+    # about, so a checkout that stays broken across many drive-loop polls
+    # gets exactly one loud warning instead of one per poll. Keyed on the
+    # reason text too, so a checkout that later fails a *different* check
+    # (e.g. it grows a valid `.git` but the wrong `origin`) still warns once
+    # for the new problem.
+    _warned: set[tuple[str, str]] = field(default_factory=set, init=False, repr=False)
 
     def _base(self, state: IssueState) -> Path | None:
         base = Path(self.repo_path).expanduser() if self.repo_path else (
             Path.home() / "src" / state.repo
         )
-        return base if (base / ".git").exists() else None
+        if not (base / ".git").exists():
+            return None
+        reason = self._unusable_reason(base, state)
+        if reason is None:
+            return base
+        key = (str(base), reason)
+        if key not in self._warned:
+            self._warned.add(key)
+            self.warn(
+                f"local checkout {base} cannot be used for merge "
+                f"verification of {state.repo} — {reason} (#2437). Merge "
+                "verification will keep returning \"could not verify\" "
+                "(never a false empty-branch) until this is fixed: replace "
+                "it with a real clone of the repo, or point `coord drive` "
+                "at one via --repo-path."
+            )
+        return None
+
+    def _unusable_reason(self, base: Path, state: IssueState) -> str | None:
+        """Why *base* can't be trusted for merge verification, or ``None`` if
+        it can.
+
+        A ``.git`` directory existing is necessary but not sufficient
+        (claude-coordinator#2437): an interrupted/stub `git init` leaves a
+        `.git` directory with no `HEAD`/`objects`/`refs`, and a checkout of
+        the wrong project entirely leaves a perfectly valid `.git` that
+        still can't answer questions about *this* repo. Both must be caught
+        here rather than surfacing as an unexplained, indefinitely-retried
+        ``None`` from :meth:`branch_has_commits`.
+        """
+        if self._git(base, "rev-parse", "--is-inside-work-tree").returncode != 0:
+            return "not a usable git working tree (`git rev-parse --is-inside-work-tree` failed)"
+        remote = self._git(base, "remote", "get-url", "origin")
+        url = (remote.stdout or "").strip()
+        if remote.returncode != 0 or not url:
+            return "has no 'origin' remote configured"
+        if state.repo_github and not _remote_matches_repo(url, state.repo_github):
+            return (
+                f"'origin' remote ({url!r}) does not match the expected "
+                f"repo {state.repo_github!r}"
+            )
+        return None
 
     @staticmethod
     def _git(base: Path, *args: str) -> subprocess.CompletedProcess[str]:

{code_coordinator-0.5.167 → code_coordinator-0.5.169}/tests/test_drive.py

@@ -53,6 +53,7 @@ from coord.drive import (
     GitMergeVerifier,
     LockBusy,
     OracleDecision,
+    _remote_matches_repo,
     coord_argv,
     decide,
     preflight,
@@ -4268,6 +4269,12 @@ def recorded_git(monkeypatch):
     ``subprocess.run`` attribute — patching it once via ``coord.drive.
     subprocess.run`` patches it for both call sites, and a single
     ``scripted`` dict drives both the git and gh sides of a scenario.
+
+    #2437's ``_base()`` checkout-validation probes (``rev-parse
+    --is-inside-work-tree`` and ``remote get-url origin``) default here to
+    "yes, a real checkout of john/claude-coordinator" — the shape almost
+    every test in this file wants — so only the handful of tests that
+    exercise the validation itself need to override them.
     """
     calls: list[list[str]] = []
     scripted: dict[tuple[str, ...], tuple[int, str]] = {}
@@ -4277,6 +4284,12 @@ def recorded_git(monkeypatch):
         for needle, (rc, out) in scripted.items():
             if all(token in argv for token in needle):
                 return subprocess.CompletedProcess(argv, rc, out, "")
+        if "rev-parse" in argv and "--is-inside-work-tree" in argv:
+            return subprocess.CompletedProcess(argv, 0, "true\n", "")
+        if "remote" in argv and "get-url" in argv:
+            return subprocess.CompletedProcess(
+                argv, 0, "https://github.com/john/claude-coordinator.git\n", ""
+            )
         return subprocess.CompletedProcess(argv, 0, "", "")
 
     monkeypatch.setattr("coord.drive.subprocess.run", fake_run)
@@ -4337,6 +4350,7 @@ def test_verify_merged_falls_back_to_cherry_when_gh_finds_no_pr(
     (tmp_path / ".git").mkdir()
     scripted[("pr", "view")] = (1, "")
     scripted[("cherry",)] = (0, "- abc\n")
+    scripted[("remote", "get-url", "origin")] = (0, "git@github.com:john/x.git\n")
     s = state(work_branch="b", repo_github="john/x")
     assert GitMergeVerifier(repo_path=str(tmp_path)).verify_merged(s) is True
     assert any("cherry" in " ".join(c) for c in calls)
@@ -4431,6 +4445,137 @@ def test_branch_has_commits_is_false_with_no_branch(tmp_path):
     assert GitMergeVerifier(repo_path=str(tmp_path)).branch_has_commits(state()) is False
 
 
+# ═══════════════════════════════════════════════════════════════════════════
+# GitMergeVerifier._base — checkout validation (#2437)
+#
+# A `.git` directory existing is necessary but not sufficient: an
+# interrupted/stub `git init` (bare `.git/info/exclude`, no `HEAD`/
+# `objects`/`refs`/`config` — the exact claude-coordinator#2286 incident)
+# satisfies `(base / ".git").exists()` while being completely unusable for
+# `git fetch`. `_base()` must actually probe the checkout and warn loudly
+# (not just silently return `None`) the first time it finds one unusable.
+# ═══════════════════════════════════════════════════════════════════════════
+
+
+def test_base_rejects_a_stub_git_dir_that_is_not_a_real_work_tree(
+    recorded_git, tmp_path
+):
+    """The exact #2437 incident: `.git` exists (so the old check passed) but
+    it's not an actual git repository — `rev-parse --is-inside-work-tree`
+    fails. Must be treated the same as no checkout at all: `None`, plus a
+    loud warning naming the reason, not a silent retry-forever."""
+    _calls, scripted = recorded_git
+    (tmp_path / ".git").mkdir()
+    scripted[("rev-parse", "--is-inside-work-tree")] = (128, "")
+    warned: list[str] = []
+    verifier = GitMergeVerifier(repo_path=str(tmp_path), warn=warned.append)
+    s = state(work_branch="b", repo_default_branch="main")
+    assert verifier.branch_has_commits(s) is None
+    assert any("not a usable git working tree" in w for w in warned), warned
+
+
+def test_base_rejects_a_checkout_with_no_origin_remote(recorded_git, tmp_path):
+    _calls, scripted = recorded_git
+    (tmp_path / ".git").mkdir()
+    scripted[("remote", "get-url", "origin")] = (2, "")
+    warned: list[str] = []
+    verifier = GitMergeVerifier(repo_path=str(tmp_path), warn=warned.append)
+    s = state(work_branch="b", repo_default_branch="main")
+    assert verifier.branch_has_commits(s) is None
+    assert any("no 'origin' remote configured" in w for w in warned), warned
+
+
+def test_base_rejects_a_checkout_of_the_wrong_repo(recorded_git, tmp_path):
+    """A perfectly valid checkout — just of the wrong project. Must not be
+    trusted to answer merge-verification questions about *this* repo."""
+    _calls, scripted = recorded_git
+    (tmp_path / ".git").mkdir()
+    scripted[("remote", "get-url", "origin")] = (
+        0,
+        "https://github.com/john/some-other-repo.git\n",
+    )
+    warned: list[str] = []
+    verifier = GitMergeVerifier(repo_path=str(tmp_path), warn=warned.append)
+    s = state(work_branch="b", repo_default_branch="main")
+    assert verifier.branch_has_commits(s) is None
+    assert any(
+        "does not match the expected repo" in w and "john/claude-coordinator" in w
+        for w in warned
+    ), warned
+
+
+def test_base_skips_the_repo_match_check_when_repo_github_is_unset(
+    recorded_git, tmp_path
+):
+    """Some callers (the ``verify_merged`` tests above) legitimately don't
+    know the GitHub identifier yet — the match check must not fire and turn
+    an otherwise-valid checkout into a false ``None``."""
+    _calls, scripted = recorded_git
+    (tmp_path / ".git").mkdir()
+    scripted[("remote", "get-url", "origin")] = (
+        0,
+        "https://github.com/someone/unrelated.git\n",
+    )
+    scripted[("rev-list",)] = (0, "2\n")
+    s = state(work_branch="b", repo_github="", repo_default_branch="main")
+    assert GitMergeVerifier(repo_path=str(tmp_path)).branch_has_commits(s) is True
+
+
+def test_base_warns_only_once_for_the_same_broken_checkout(recorded_git, tmp_path):
+    """A checkout that's been broken for days (the #2437 incident: 3 days, 6
+    drive-queue attempts) must warn once, not once per poll — the whole
+    point is a signal an operator can act on, not log spam that gets
+    filtered out."""
+    _calls, scripted = recorded_git
+    (tmp_path / ".git").mkdir()
+    scripted[("rev-parse", "--is-inside-work-tree")] = (128, "")
+    warned: list[str] = []
+    verifier = GitMergeVerifier(repo_path=str(tmp_path), warn=warned.append)
+    s = state(work_branch="b", repo_default_branch="main")
+    for _ in range(6):
+        assert verifier.branch_has_commits(s) is None
+    assert len(warned) == 1, warned
+
+
+def test_base_accepts_a_genuinely_valid_checkout(recorded_git, tmp_path):
+    """The happy path: a real work tree, an `origin` matching the expected
+    repo — no warning, and verification proceeds as normal."""
+    _calls, scripted = recorded_git
+    (tmp_path / ".git").mkdir()
+    scripted[("rev-list",)] = (0, "1\n")
+    warned: list[str] = []
+    verifier = GitMergeVerifier(repo_path=str(tmp_path), warn=warned.append)
+    s = state(work_branch="b", repo_default_branch="main")
+    assert verifier.branch_has_commits(s) is True
+    assert warned == []
+
+
+@pytest.mark.parametrize(
+    "remote_url",
+    [
+        "https://github.com/john/claude-coordinator.git",
+        "https://github.com/john/claude-coordinator",
+        "git@github.com:john/claude-coordinator.git",
+        "git@github.com:john/claude-coordinator",
+        "ssh://git@github.com/john/claude-coordinator.git",
+    ],
+)
+def test_remote_matches_repo_accepts_every_url_shape_git_produces(remote_url):
+    assert _remote_matches_repo(remote_url, "john/claude-coordinator") is True
+
+
+@pytest.mark.parametrize(
+    "remote_url",
+    [
+        "https://github.com/john/some-other-repo.git",
+        "git@github.com:someone-else/claude-coordinator.git",
+        "",
+    ],
+)
+def test_remote_matches_repo_rejects_a_mismatch(remote_url):
+    assert _remote_matches_repo(remote_url, "john/claude-coordinator") is False
+
+
 # ═══════════════════════════════════════════════════════════════════════════
 # locking
 # ═══════════════════════════════════════════════════════════════════════════

{code_coordinator-0.5.167 → code_coordinator-0.5.169}/tui/src/app/events.rs

@@ -2240,6 +2240,70 @@ impl CoordApp {
                         needs_redraw = true;
                     }
 
+                    // ── #2405: the Pipeline Completed grid's keymap ──────
+                    // Placed here, beside the Reports typing block and
+                    // ahead of the global `q`/Esc catch-all, for the same
+                    // reason: while the free-text `Window end` control has
+                    // focus every printable character (including `q`)
+                    // belongs to the value being typed. Every arm is guarded
+                    // on the Completed tab being active, so the rest of the
+                    // Pipeline keymap is untouched on all five other tabs.
+                    //
+                    // ←/→ step the focused choice control rather than
+                    // cycling tabs, matching the Reports panel; `h`/`l`
+                    // still cycle tabs, so nothing becomes unreachable.
+                    Key::Named(NamedKey::Escape)
+                        if self.active_view == SidebarView::Pipeline
+                            && self.pipeline_detail_tab == PipelineDetailTab::Completed
+                            && self.completed_grid.detail.is_some() =>
+                    {
+                        self.completed_close_detail();
+                        needs_redraw = true;
+                    }
+                    Key::Named(NamedKey::Tab)
+                        if self.active_view == SidebarView::Pipeline
+                            && self.pipeline_detail_tab == PipelineDetailTab::Completed
+                            && !self.pipeline_search.focused =>
+                    {
+                        self.completed_grid.field_sel = (self.completed_grid.field_sel + 1)
+                            % CoordApp::COMPLETED_FIELD_COUNT;
+                        needs_redraw = true;
+                    }
+                    Key::Named(NamedKey::Right)
+                        if self.active_view == SidebarView::Pipeline
+                            && self.pipeline_detail_tab == PipelineDetailTab::Completed
+                            && !self.pipeline_search.focused =>
+                    {
+                        self.completed_step_choice(true);
+                        needs_redraw = true;
+                    }
+                    Key::Named(NamedKey::Left)
+                        if self.active_view == SidebarView::Pipeline
+                            && self.pipeline_detail_tab == PipelineDetailTab::Completed
+                            && !self.pipeline_search.focused =>
+                    {
+                        self.completed_step_choice(false);
+                        needs_redraw = true;
+                    }
+                    Key::Named(NamedKey::Backspace)
+                        if self.active_view == SidebarView::Pipeline
+                            && self.pipeline_detail_tab == PipelineDetailTab::Completed
+                            && self.completed_focus_is_text()
+                            && !self.pipeline_search.focused =>
+                    {
+                        self.completed_text_backspace();
+                        needs_redraw = true;
+                    }
+                    Key::Char(ch)
+                        if self.active_view == SidebarView::Pipeline
+                            && self.pipeline_detail_tab == PipelineDetailTab::Completed
+                            && self.completed_focus_is_text()
+                            && !self.pipeline_search.focused =>
+                    {
+                        self.completed_text_insert(*ch);
+                        needs_redraw = true;
+                    }
+
                     // ── Watch overlay: control keys ─────────────────────
                     // 'b' opens the ChatController guidance overlay. When the
                     // overlay is open, ALL events are intercepted earlier in
@@ -2797,37 +2861,6 @@ impl CoordApp {
                         needs_redraw = true;
                     }
 
-                    // ── #728: Done section extend-range (→) ─────────────
-                    // `→` while the Pipeline Done section is focused cycles
-                    // the time window: H2 → H24 → D7 → All → H2.
-                    Key::Named(NamedKey::Right)
-                        if self.active_view == SidebarView::Pipeline
-                            && self.is_done_section_active()
-                            && !self.pipeline_search.focused =>
-                    {
-                        self.done_window = self.done_window.next();
-                        self.rebuild_pipeline_sidebar(None);
-                        // After rebuild the SidebarSystem starts with
-                        // active_section() == None, so the default_select block
-                        // in rebuild_pipeline_sidebar selects the FIRST section
-                        // (usually in-progress) and the restore loop only re-
-                        // focuses Done if pipeline_sel pointed at a Done row.
-                        // When the user is on the Done section HEADER (no issue
-                        // row selected) pipeline_sel is None, so the restore is
-                        // skipped and Done loses focus.  Re-assert Done focus
-                        // unconditionally here so subsequent → presses still
-                        // fire the extend-range handler.
-                        if let Some(done_idx) = self
-                            .pipeline_state_section_names
-                            .iter()
-                            .position(|&k| k == "done")
-                            .map(|i| i + 1)
-                        {
-                            self.pipeline_sidebar.set_active_section(Some(done_idx));
-                        }
-                        needs_redraw = true;
-                    }
-
                     // ── Kanban keyboard nav (#638) ───────────────────────
                     Key::Char('j') | Key::Named(NamedKey::Down)
                         if self.active_view == SidebarView::Kanban =>
@@ -3388,16 +3421,19 @@ impl CoordApp {
                     }
 
                     // ── h/l — cycle Pipeline detail tabs ─────────────────
-                    // #818 order: Overview → Issue → Log → Summary → Terminal → Overview …
+                    // #818 order, plus #2405's Completed grid last:
+                    // Overview → Issue → Log → Summary → Terminal →
+                    // Completed → Overview …
                     Key::Char('h') | Key::Named(NamedKey::Left)
                         if self.active_view == SidebarView::Pipeline =>
                     {
                         let next = match self.pipeline_detail_tab {
-                            PipelineDetailTab::Overview => PipelineDetailTab::Terminal,
+                            PipelineDetailTab::Overview => PipelineDetailTab::Completed,
                             PipelineDetailTab::Issue => PipelineDetailTab::Overview,
                             PipelineDetailTab::Log => PipelineDetailTab::Issue,
                             PipelineDetailTab::Summary => PipelineDetailTab::Log,
                             PipelineDetailTab::Terminal => PipelineDetailTab::Summary,
+                            PipelineDetailTab::Completed => PipelineDetailTab::Terminal,
                         };
                         // Release PTY focus whenever the user navigates away
                         // from the Terminal tab.
@@ -3432,7 +3468,8 @@ impl CoordApp {
                             PipelineDetailTab::Issue => PipelineDetailTab::Log,
                             PipelineDetailTab::Log => PipelineDetailTab::Summary,
                             PipelineDetailTab::Summary => PipelineDetailTab::Terminal,
-                            PipelineDetailTab::Terminal => PipelineDetailTab::Overview,
+                            PipelineDetailTab::Terminal => PipelineDetailTab::Completed,
+                            PipelineDetailTab::Completed => PipelineDetailTab::Overview,
                         };
                         // Release PTY focus whenever the user navigates away
                         // from the Terminal tab.
@@ -6053,13 +6090,15 @@ impl CoordApp {
                 if let Some(idx) =
                     hit_tab_index_from_labels(&labels, main_b.x, pos.x, tab_scroll)
                 {
-                    // #818: Overview / Issue / Log / Summary / Terminal (5 tabs).
+                    // #818: Overview / Issue / Log / Summary / Terminal,
+                    // plus #2405's Completed grid at index 5.
                     let new_tab = match idx {
                         0 => PipelineDetailTab::Overview,
                         1 => PipelineDetailTab::Issue,
                         2 => PipelineDetailTab::Log,
                         3 => PipelineDetailTab::Summary,
-                        _ => PipelineDetailTab::Terminal,
+                        4 => PipelineDetailTab::Terminal,
+                        _ => PipelineDetailTab::Completed,
                     };
                     if new_tab != self.pipeline_detail_tab {
                         if new_tab != PipelineDetailTab::Terminal {
@@ -6084,6 +6123,15 @@ impl CoordApp {
             // is rendered into the content area (main_b minus tab row), so
             // we must hit-test against that rect — not main_b directly, or
             // the y-coordinates are off by tab_h.
+            if self.pipeline_detail_tab == PipelineDetailTab::Completed {
+                let content_rect = Rect::new(
+                    main_b.x,
+                    main_b.y + tab_h,
+                    main_b.width,
+                    (main_b.height - tab_h).max(0.0),
+                );
+                return self.completed_main_click(pos, content_rect, lh);
+            }
             if self.pipeline_detail_tab == PipelineDetailTab::Overview {
                 if let Some(view) = self.build_pipeline_widget() {
                     let content_rect = Rect::new(
@@ -6774,6 +6822,30 @@ impl CoordApp {
                                 (self.pipeline_detail_scroll + 1).min(max);
                         }
                     }
+                    PipelineDetailTab::Completed => {
+                        // #2405: the grid scrolls against its own painted
+                        // viewport, not the whole main panel's row count —
+                        // the control row sits above it (the exact gap #1910
+                        // fixed for the Reports table).
+                        if self.completed_grid.detail.is_some() {
+                            let items = self
+                                .completed_detail_index()
+                                .map(|i| self.pipeline_tab_body_list_for(Some(i)).items.len())
+                                .unwrap_or(0);
+                            let max = items.saturating_sub(visible.saturating_sub(1));
+                            if delta.y > 0.0 {
+                                self.pipeline_stage_content_scroll =
+                                    self.pipeline_stage_content_scroll.saturating_sub(1);
+                            } else if delta.y < 0.0 {
+                                self.pipeline_stage_content_scroll =
+                                    (self.pipeline_stage_content_scroll + 1).min(max);
+                            }
+                        } else if delta.y > 0.0 {
+                            self.completed_scroll_by(-1);
+                        } else if delta.y < 0.0 {
+                            self.completed_scroll_by(1);
+                        }
+                    }
                     PipelineDetailTab::Overview => {
                         // #818: The body list (meta + stage content) scrolls on
                         // the Overview tab.

{code_coordinator-0.5.167 → code_coordinator-0.5.169}/tui/src/app/fixtures.rs

@@ -375,8 +375,10 @@ pub fn make_test_app(data: BoardData) -> CoordApp {
         reports_export_rx: None,
         // #217: use the default dark palette for test helpers.
         active_theme: crate::settings::Theme::Dark.to_quadraui_theme(),
-        // #728: default 2h window for tests (can be overridden per test).
-        done_window: DoneWindow::H2,
+        // #2405: completed-issues grid defaults (24h / all repos).
+        completed_grid: CompletedGrid::default(),
+        completed_form_layout: std::cell::RefCell::new(None),
+        completed_table_layout: std::cell::RefCell::new(None),
         // #816: no pending PTY-panic dialog in test helpers.
         pty_panic_dialog: None,
         // #1059: no pending Gate A dispatch-failure dialog in test helpers.

{code_coordinator-0.5.167 → code_coordinator-0.5.169}/tui/src/app/mod.rs

@@ -3330,7 +3330,7 @@ pub struct CoordApp {
     audit_no_service: bool,
     /// #1040: current time-range filter (contract §8), cycled by `t`.
     /// Persists across panel navigation (contract §11); resets to `All` on
-    /// TUI restart (in-memory only, like `done_window`).
+    /// TUI restart (in-memory only).
     audit_time_range: AuditTimeRange,
     /// #1040: current category filter (contract §9), cycled by `Tab`.
     /// Persists across panel navigation; resets to `All` on restart.
@@ -3524,10 +3524,25 @@ pub struct CoordApp {
     /// `Assignment::status_color_themed` and `stage_badge`.
     active_theme: quadraui::Theme,
 
-    // ── #728: Done-section time window ───────────────────────────────────────
-    /// Controls how far back the windowed Done list reaches.  Cycled by the
-    /// `→` key while the Done section is focused.  Resets to `H2` on restart.
-    done_window: DoneWindow,
+    // ── #2405: completed-issues grid ─────────────────────────────────────────
+    /// Controls + selection behind the Pipeline panel's `Completed` tab —
+    /// the report-style grid that replaced #728's fixed-window Done section.
+    /// In-memory only; resets to its 24h default on restart.
+    pub(crate) completed_grid: CompletedGrid,
+    /// The completed grid's last-painted control-row geometry, with the rect
+    /// it was painted into.  Same render-then-hit-test contract as
+    /// `msv::MsvLayoutCache` — and for the same reason: `FormController`'s
+    /// backend-free `handle_cached` re-derives char width as `lh * 0.6`,
+    /// which is wrong on a cell-granular TUI surface and mis-routes clicks
+    /// inside a segmented control.  `FormLayout` bounds are **form-local**
+    /// (origin `(0, 0)`), so hit-testing subtracts the stored rect.
+    pub(crate) completed_form_layout: std::cell::RefCell<Option<(Rect, FormLayout)>>,
+    /// The completed grid's last-painted `DataTable` geometry, with the rect
+    /// it was painted into.  Same render-then-hit-test pattern (and same
+    /// reason for carrying the rect) as `reports_table_layout`: the table does
+    /// not start at the main panel's origin, so a bare `pos - main_b` would
+    /// mis-hit-test by the control row's height.
+    pub(crate) completed_table_layout: std::cell::RefCell<Option<(Rect, DataTableLayout)>>,
 
     // ── #816: PTY-panic modal ─────────────────────────────────────────────────
     /// When `Some`, a dismissible modal dialog is shown explaining that a vt100
@@ -3579,54 +3594,6 @@ pub(crate) enum QueueScrollAxis {
     Horizontal,
 }
 
-/// #728: Time-window for the Done section in the Pipeline sidebar.
-///
-/// Cycled forward (H2 → H24 → D7 → All) by the `→` key while the Done
-/// section is focused.  Resets to `H2` on TUI restart (in-memory only).
-#[derive(Debug, Clone, Copy, PartialEq, Eq)]
-enum DoneWindow {
-    /// Last 2 hours (default).
-    H2,
-    /// Last 24 hours.
-    H24,
-    /// Last 7 days.
-    D7,
-    /// All done issues regardless of age.
-    All,
-}
-
-impl DoneWindow {
-    /// The maximum age in seconds for this window, or `None` for "All".
-    fn secs(self) -> Option<f64> {
-        match self {
-            DoneWindow::H2 => Some(2.0 * 3600.0),
-            DoneWindow::H24 => Some(24.0 * 3600.0),
-            DoneWindow::D7 => Some(7.0 * 86_400.0),
-            DoneWindow::All => None,
-        }
-    }
-
-    /// Cycle to the next wider window.
-    fn next(self) -> Self {
-        match self {
-            DoneWindow::H2 => DoneWindow::H24,
-            DoneWindow::H24 => DoneWindow::D7,
-            DoneWindow::D7 => DoneWindow::All,
-            DoneWindow::All => DoneWindow::H2,
-        }
-    }
-
-    /// Short label for use in the section header.
-    fn label(self) -> &'static str {
-        match self {
-            DoneWindow::H2 => "last 2h",
-            DoneWindow::H24 => "last 24h",
-            DoneWindow::D7 => "last 7d",
-            DoneWindow::All => "all",
-        }
-    }
-}
-
 impl Default for CoordApp {
     fn default() -> Self {
         Self::new()
@@ -4092,8 +4059,10 @@ impl CoordApp {
                 crate::settings::TuiSettings::load_custom_theme_file()
                     .unwrap_or_else(|| s.theme.to_quadraui_theme())
             },
-            // #728: Done section shows last 2h by default; cycled with `→`.
-            done_window: DoneWindow::H2,
+            // #2405: completed-issues grid defaults (24h / all repos).
+            completed_grid: CompletedGrid::default(),
+            completed_form_layout: std::cell::RefCell::new(None),
+            completed_table_layout: std::cell::RefCell::new(None),
             // #816: no pending PTY-panic dialog on startup.
             pty_panic_dialog: None,
             // #1059: no pending Gate A dispatch-failure dialog on startup.