diffler 0.16.1

//! Status screen state and handlers: neogit-style sections with inline
//! diff expansion, folding, stage/unstage/discard, and cursor preservation
//! across refreshes.

use std::collections::{BTreeSet, HashMap};
use std::path::Path;

use diffler_core::model::FileDiff;
use diffler_core::review::Review;
use diffler_core::source::ReviewSource;
use diffler_core::vcs::{BranchInfo, LogEntry, NetworkOp, Vcs, VcsError};

use super::enrich::EnrichOutcome;
use super::rowsel::RowSelect;
use super::{App, BranchAction, FileHighlights, Modal, PendingOp};
use crate::config::FileLayout;
use crate::keymap::Action;
use crate::tree::{self, TreeNode, TreeRow};

/// Heading for the trailing unpushed-commits section, shared by the renderer
/// and the search labels so a `/` match lines up with the displayed text.
pub(crate) const UNPUSHED_TITLE: &str = "Unpushed";

/// Heading for the trailing recent-commits section, shared by the renderer and
/// the search labels so a `/` match lines up with the displayed text.
pub(crate) const RECENT_TITLE: &str = "Recent commits";

/// Heading for the trailing CI-runs section (when a provider is detected).
pub(crate) const CI_TITLE: &str = "CI runs";

/// Heading for the trailing Branches section.
pub(crate) const BRANCHES_TITLE: &str = "Branches";

/// Heading for the leading Open-pull-requests section (when a forge is
/// detected).
pub(crate) const PRS_TITLE: &str = "Open pull requests";

/// Heading for the review's walkthroughs group.
pub(crate) const WALKTHROUGHS_TITLE: &str = "Walkthroughs";

/// How far the unpushed walk goes. Nothing prunes it in a repository whose
/// remote refs sit off HEAD's history, where the honest answer is the whole
/// log, so the ceiling keeps both the walk and the row list finite.
pub(crate) const UNPUSHED_LIMIT: usize = 100;

/// How many local branches the inline status section shows (the full list is
/// reachable through the branch picker).
const BRANCHES_INLINE_LIMIT: usize = 10;

/// How many of the repo's other open PRs the inline status section shows (the
/// full list is reachable through the Prs screen).
const PRS_INLINE_LIMIT: usize = 5;

/// How many runs the trailing CI section shows (the full list lives on the
/// Runs screen).
const CI_INLINE_LIMIT: usize = 5;

/// Status screen sections, in display order.
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub enum Section {
    Untracked,
    Unstaged,
    Staged,
}

impl Section {
    pub const ALL: [Self; 3] = [Self::Untracked, Self::Unstaged, Self::Staged];

    /// jj has no index, so its `Vcs::status` puts the whole working copy in
    /// `Staged`.
    pub fn title(self, has_index: bool) -> &'static str {
        match (self, has_index) {
            (Self::Staged, false) => "Working copy (@)",
            (Self::Untracked, _) => "Untracked",
            (Self::Unstaged, _) => "Unstaged changes",
            (Self::Staged, true) => "Staged changes",
        }
    }

    fn index(self) -> usize {
        match self {
            Self::Untracked => 0,
            Self::Unstaged => 1,
            Self::Staged => 2,
        }
    }
}

/// The status screen's collapsible groups below (and including) the
/// unpushed-commits list, each addressed by one slot of
/// [`StatusView::group_folded`].
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub enum Group {
    /// Commits on HEAD its upstream lacks: local and actionable, so it opens
    /// unfolded.
    Unpushed,
    /// The repo's open pull requests, minus the branch's own (already shown
    /// above the divider). Fetched lazily: nothing is requested from the
    /// forge until this group is first unfolded.
    Prs,
    Branches,
    Recent,
    /// Every run in `App::runs`, whether or not it also nests under a row
    /// above. Listing them all is what keeps a run reachable when it belongs
    /// to nothing on screen: a scheduled job, a cron run, a run on a branch
    /// that fell off the list.
    Ci,
    /// The review's walkthroughs, listed under their header.
    Walkthrough,
}

impl Group {
    /// Variant count: `group_folded`'s array length derives from this one
    /// spot, so it grows automatically as variants are added.
    pub(crate) const COUNT: usize = 6;

    pub(crate) fn index(self) -> usize {
        match self {
            Self::Unpushed => 0,
            Self::Prs => 1,
            Self::Branches => 2,
            Self::Recent => 3,
            Self::Ci => 4,
            Self::Walkthrough => 5,
        }
    }
}

/// Only Unpushed opens by default: the rest of the repo band starts folded.
fn default_group_folded() -> [bool; Group::COUNT] {
    let mut folded = [true; Group::COUNT];
    if let Some(slot) = folded.get_mut(Group::Unpushed.index()) {
        *slot = false;
    }
    folded
}

/// One cursor-addressable row of the status screen: section headers, directory
/// rows, file rows, and (when a file is expanded inline) hunk headers and
/// diff lines, plus the trailing Recent commits section. Holds an owned
/// directory path (the fold key), so it is `Clone`, not `Copy`.
#[derive(Debug, Clone, PartialEq, Eq)]
pub enum Row {
    SectionHeader {
        section: Section,
        count: usize,
    },
    /// A directory in a section's file tree; `path` is the fold key, `name` the
    /// display name (a joined `a/b/c` for a collapsed single-child chain),
    /// `depth` the indentation under the header.
    Dir {
        section: Section,
        path: String,
        name: String,
        depth: usize,
    },
    File {
        section: Section,
        index: usize,
        depth: usize,
    },
    HunkHeader {
        section: Section,
        file: usize,
        hunk: usize,
    },
    DiffLine {
        section: Section,
        file: usize,
        hunk: usize,
        line: usize,
    },
    /// Header of the Unpushed section: commits no remote has yet. `capped`
    /// marks a count the walk stopped short of finishing.
    UnpushedHeader {
        count: usize,
        capped: bool,
    },
    /// One such commit; `index` into `StatusView::unpushed`.
    Unpushed {
        index: usize,
    },
    RecentHeader {
        count: usize,
    },
    Commit {
        index: usize,
    },
    /// The branch's open pull request; Enter reviews it.
    Pr,
    /// Header of the review's walkthroughs; `count` is how many there are.
    WalkthroughHeader {
        count: usize,
    },
    /// One walkthrough of the review, named by id. Enter opens the diff in
    /// its walkthrough layout.
    Walkthrough {
        id: String,
    },
    /// Divider between the branch band (this branch) and the repo band (this
    /// repo). Furniture, not a cursor-addressable row: it never holds the
    /// cursor, has no search label, and is skipped by every movement.
    RepoDivider,
    /// Header of the leading Open-pull-requests section; `None` while the
    /// first fetch is still in flight.
    PrsHeader {
        count: Option<usize>,
    },
    /// One PR in the inline section, other than the branch's own; `index`
    /// into `App::other_prs`.
    OpenPr {
        index: usize,
    },
    /// Header of the trailing Branches section.
    BranchesHeader {
        count: usize,
    },
    /// One local branch in the inline section; `index` into
    /// `StatusView::branches`.
    Branch {
        index: usize,
    },
    /// Header of the trailing CI-runs section.
    CiHeader {
        count: usize,
    },
    /// One CI run; `index` into `App::runs`. `nested` is `true` under the
    /// commit row that triggered it (deeper indent), `false` in the flat CI
    /// section.
    CiRun {
        index: usize,
        nested: bool,
    },
}

/// Where the cursor logically sits, so it can be restored after a refresh
/// reshuffles the rows.
#[derive(Debug, Clone, PartialEq, Eq)]
pub(super) enum CursorAnchor {
    Section(Section),
    Dir {
        section: Section,
        path: String,
    },
    File {
        section: Section,
        path: String,
        hunk: Option<usize>,
        /// The diff line the cursor sat on, by the number the file gives it:
        /// a poll that re-reads the working tree rebuilds every hunk, and the
        /// reader is still on the same line.
        line: Option<LineAnchor>,
    },
    Unpushed,
    UnpushedCommit(usize),
    Recent,
    Commit(usize),
    Pr,
    WalkthroughHeader,
    Walkthrough(String),
    Prs,
    PrsRow(usize),
    Branches,
    BranchRow(String),
    Ci,
    CiRun(usize),
}

/// One diff line by the numbers it carries: an addition and a context line
/// have a new-side number, a deletion only an old-side one.
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub(super) struct LineAnchor {
    old_no: Option<u32>,
    new_no: Option<u32>,
}

/// The unpushed commits and whether the walk that found them stopped at its
/// ceiling, so a count rendered from this can say `100+` there.
pub struct Unpushed {
    pub commits: Vec<LogEntry>,
    pub capped: bool,
}

/// One walkthrough of the repo, as its own review file holds it: what the
/// status screen's row needs, without going through whatever diff is open.
pub struct WalkthroughRow {
    pub id: String,
    pub title: String,
    pub stops: usize,
    pub all_seen: bool,
    /// Whether the walkthrough has a summary: `<cr>` opens on it when so,
    /// else on the first stop.
    pub has_summary: bool,
}

/// Every walkthrough source on disk (the working review's in-memory edits
/// included), newest published first. Real disk IO, so this runs once per
/// refresh (`App::new`, `App::apply_refresh`), never from the render path.
pub(super) fn load_walkthroughs(review: &Review) -> Vec<WalkthroughRow> {
    let mut rows: Vec<(u64, WalkthroughRow)> = review
        .all_reviews()
        .unwrap_or_default()
        .into_iter()
        .filter_map(|(source, session)| {
            let ReviewSource::Walkthrough { id } = source else {
                return None;
            };
            let walkthrough = session.walkthrough.as_ref()?;
            let all_seen = !walkthrough.stops.is_empty()
                && walkthrough
                    .stops
                    .iter()
                    .all(|stop| session.is_stop_seen(stop));
            Some((
                walkthrough.at,
                WalkthroughRow {
                    id,
                    title: walkthrough.title.clone(),
                    stops: walkthrough.stops.len(),
                    all_seen,
                    has_summary: walkthrough.summary.is_some(),
                },
            ))
        })
        .collect();
    rows.sort_by_key(|(at, _)| std::cmp::Reverse(*at));
    rows.into_iter().map(|(_, row)| row).collect()
}

/// All state owned by the status screen.
pub struct StatusView {
    pub cursor: usize,
    /// Where a `V` selection started, so a run of commits can be reviewed as
    /// one range.
    pub anchor: Option<usize>,
    pub folded: [bool; 3],
    /// Commits no remote has yet: local, free to act on, so shown unfolded by
    /// default. `None` in a repository with no remote-tracking refs, where
    /// the section is absent entirely.
    pub unpushed: Option<Unpushed>,
    pub recent: Vec<LogEntry>,
    /// Local branches, newest tip first, capped to `BRANCHES_INLINE_LIMIT`.
    pub branches: Vec<BranchInfo>,
    /// Every walkthrough source on disk, newest published first. Loaded once
    /// per refresh, not per frame: real disk IO.
    pub walkthroughs: Vec<WalkthroughRow>,
    /// Fold state of the repo-band groups (and Unpushed above them), indexed
    /// by [`Group::index`]. The whole repo band starts folded, unlike the
    /// always-open branch band above it.
    pub group_folded: [bool; Group::COUNT],
    /// Commits whose CI runs are unfolded beneath them, keyed by oid rather
    /// than row index so the set survives a refresh that reorders the list.
    pub(crate) unfolded_commits: BTreeSet<String>,
    /// Body height of the last render, so half-page motions step by a screenful.
    pub(crate) viewport: u16,
    /// Per-section set of file paths whose inline diff is expanded.
    expanded: [BTreeSet<String>; 3],
    /// Per-section set of folded directory paths in that section's file tree.
    folded_dirs: [BTreeSet<String>; 3],
    /// Per-section set of file paths whose inline diff has received its
    /// background enrichment (intra-line emphasis), so a landed file isn't
    /// re-queued. Cleared when the status sections are rebuilt (refresh).
    enriched: [BTreeSet<String>; 3],
    /// Per-file syntax spans for inline diffs, keyed by path plus both-sides
    /// content hash: a partially staged file shows up in two sections with
    /// different content, and each needs its own entry to stay settled.
    /// Filled lazily, only for expanded files.
    pub(crate) highlights: HashMap<(String, String), FileHighlights>,
    /// Whether `App::prs` has been fetched at least once, so the inline
    /// Open-pull-requests group can tell "nothing fetched yet" (still shown,
    /// no count) from "the repo truly has no open PRs" (hidden).
    pub prs_loaded: bool,
    /// A PR-list fetch is out and its answer has not landed. Unfolding again
    /// while one is in flight would put a second identical request on the wire,
    /// since `prs_loaded` only turns true once the first one answers.
    pub prs_in_flight: bool,
    /// Last render's body rect, line scroll, and per-rendered-line row index
    /// (rows vary in height, so a screen row maps back to a `visible_rows`
    /// index only through this table). Drives mouse hit-testing.
    pub(crate) body: ratatui::layout::Rect,
    pub(crate) scroll: u16,
    pub(crate) line_rows: Vec<Option<usize>>,
}

impl RowSelect for StatusView {
    fn cursor(&self) -> usize {
        self.cursor
    }

    fn anchor(&self) -> Option<usize> {
        self.anchor
    }

    fn set_anchor(&mut self, anchor: Option<usize>) {
        self.anchor = anchor;
    }
}

impl StatusView {
    pub(super) fn new(
        unpushed: Option<Unpushed>,
        recent: Vec<LogEntry>,
        branches: Vec<BranchInfo>,
        walkthroughs: Vec<WalkthroughRow>,
    ) -> Self {
        Self {
            cursor: 0,
            anchor: None,
            folded: [false; 3],
            unpushed,
            recent,
            branches,
            walkthroughs,
            group_folded: default_group_folded(),
            unfolded_commits: BTreeSet::new(),
            viewport: 0,
            expanded: [const { BTreeSet::new() }; 3],
            folded_dirs: [const { BTreeSet::new() }; 3],
            enriched: [const { BTreeSet::new() }; 3],
            highlights: HashMap::new(),
            prs_loaded: false,
            prs_in_flight: false,
            body: ratatui::layout::Rect::default(),
            scroll: 0,
            line_rows: Vec::new(),
        }
    }

    /// The unpushed commits, empty where the repository has no remote to
    /// measure against.
    pub(crate) fn unpushed_commits(&self) -> &[LogEntry] {
        self.unpushed
            .as_ref()
            .map_or(&[], |unpushed| unpushed.commits.as_slice())
    }

    /// Forget which inline diffs have been enriched (after a refresh rebuilds
    /// the status sections unenriched).
    pub(super) fn clear_enriched(&mut self) {
        for set in &mut self.enriched {
            set.clear();
        }
    }
}

/// The Unpushed and Recent commit lists, as `(unpushed, recent)`, with
/// `unpushed` `None` in a repository that has no remote. Loaded together and
/// returned or discarded as a pair: recent is filtered against unpushed, so
/// applying one without the other leaves a pushed commit missing from both
/// sections. Recent is walked deep enough to still fill `limit` after the
/// unpushed commits are taken out of it.
pub(super) fn load_commit_lists(
    vcs: &dyn Vcs,
    limit: usize,
    unpushed_limit: usize,
) -> Result<(Option<Unpushed>, Vec<LogEntry>), VcsError> {
    let unpushed = vcs.unpushed(unpushed_limit)?.map(|commits| Unpushed {
        capped: commits.len() == unpushed_limit,
        commits,
    });
    let ahead = unpushed
        .as_ref()
        .map_or(&[][..], |unpushed| unpushed.commits.as_slice());
    let mut recent = vcs.log(limit + ahead.len())?;
    recent.retain(|entry| !ahead.iter().any(|local| local.oid == entry.oid));
    recent.truncate(limit);
    Ok((unpushed, recent))
}

/// Every local branch, the checked-out one first and the rest newest tip first.
/// The inline section renders only the first `BRANCHES_INLINE_LIMIT`, so the
/// header still counts the whole repo and a capped list reads as a cut rather
/// than as the total. Head leads so sitting on an old branch while ten newer
/// ones exist cannot cut your own branch out of the list.
pub(super) fn load_branches(vcs: &dyn Vcs) -> Result<Vec<BranchInfo>, VcsError> {
    let mut branches = vcs.branches()?;
    branches.sort_by_key(|branch| (!branch.is_head, std::cmp::Reverse(branch.tip_unix)));
    // a divergence costs a config read and a graph walk each, which a
    // repository carrying hundreds of branches feels on every refresh, so only
    // the rows the section renders ask for one
    for branch in branches.iter_mut().take(BRANCHES_INLINE_LIMIT) {
        branch.divergence = vcs.divergence(&branch.name).unwrap_or_default();
    }
    Ok(branches)
}

impl App {
    /// Re-read every walkthrough source on disk. Called after a refresh, and
    /// after any change to a walkthrough's own file (publish, delete, a stop
    /// removed), so the status screen's listing never trails the change that
    /// just made it.
    pub(crate) fn reload_walkthroughs(&mut self) {
        self.status.walkthroughs = load_walkthroughs(&self.review);
    }

    pub fn is_folded(&self, section: Section) -> bool {
        self.status
            .folded
            .get(section.index())
            .copied()
            .unwrap_or(false)
    }

    pub fn is_group_folded(&self, group: Group) -> bool {
        self.status
            .group_folded
            .get(group.index())
            .copied()
            .unwrap_or(false)
    }

    /// The repo's open PRs minus the current branch's own (already shown
    /// above the divider as `Row::Pr`), for the inline repo-band list.
    pub(crate) fn other_prs(&self) -> Vec<&crate::ci::PullRequest> {
        let own = self.pr.as_ref().map(|pr| pr.number);
        self.prs
            .iter()
            .filter(|pr| Some(pr.number) != own)
            .collect()
    }

    /// Indices into `App::runs` whose triggering commit is `oid` (a commit's
    /// own sha, or a PR's head sha).
    pub(crate) fn runs_for_commit(&self, oid: &str) -> Vec<usize> {
        self.runs
            .iter()
            .enumerate()
            .filter(|(_, run)| run.commit == oid)
            .map(|(index, _)| index)
            .collect()
    }

    /// Run rows to render under the commit `oid`, empty while it is folded or
    /// has no runs of its own.
    fn nested_runs(&self, oid: &str) -> Vec<Row> {
        if !self.status.unfolded_commits.contains(oid) {
            return Vec::new();
        }
        self.runs_for_commit(oid)
            .into_iter()
            .map(|index| Row::CiRun {
                index,
                nested: true,
            })
            .collect()
    }

    /// Show or hide a commit's runs. A commit with none stays as it is: the row
    /// carries no fold marker, so a keypress there must do nothing visible.
    fn toggle_commit_runs(&mut self, oid: String) {
        if self.runs_for_commit(&oid).is_empty() {
            return;
        }
        if !self.status.unfolded_commits.remove(&oid) {
            self.status.unfolded_commits.insert(oid);
        }
    }

    /// Indices into `App::runs` whose branch is `name`.
    pub(crate) fn runs_for_branch(&self, name: &str) -> Vec<usize> {
        self.runs
            .iter()
            .enumerate()
            .filter(|(_, run)| run.branch == name)
            .map(|(index, _)| index)
            .collect()
    }

    /// The worst status among `indices`' runs, or `None` when none match: the
    /// honest rendering for a row with nothing loaded for it yet. A cancelled
    /// or skipped run counts only when no other run of the same workflow
    /// reached a verdict, since a forge cancels the duplicate a concurrency
    /// group supersedes and that says nothing about the commit.
    pub(crate) fn ci_rollup(&self, indices: &[usize]) -> Option<crate::ci::JobStatus> {
        use crate::ci::JobStatus;
        let runs: Vec<&crate::ci::CiRun> = indices
            .iter()
            .filter_map(|index| self.runs.get(*index))
            .collect();
        let inconclusive =
            |status: JobStatus| matches!(status, JobStatus::Neutral | JobStatus::Skipped);
        let decided = |run: &crate::ci::CiRun| {
            runs.iter().any(|other| {
                other.name == run.name && other.remote == run.remote && !inconclusive(other.status)
            })
        };
        runs.iter()
            .filter(|run| !inconclusive(run.status) || !decided(run))
            .map(|run| run.status)
            .reduce(JobStatus::worse)
    }

    pub fn is_expanded(&self, section: Section, path: &str) -> bool {
        self.status
            .expanded
            .get(section.index())
            .is_some_and(|set| set.contains(path))
    }

    /// Whether the directory `path` is folded in `section`'s file tree.
    pub fn is_dir_folded(&self, section: Section, path: &str) -> bool {
        self.status
            .folded_dirs
            .get(section.index())
            .is_some_and(|set| set.contains(path))
    }

    fn section_folded_dirs(&self, section: Section) -> BTreeSet<String> {
        self.status
            .folded_dirs
            .get(section.index())
            .cloned()
            .unwrap_or_default()
    }

    /// Layout-aware flattened rows for a section's files. The flat list is a
    /// degenerate tree (one File node per file, depth 0, no Dir nodes), so the
    /// caller's row-building and the cursor model are identical for both
    /// layouts. The tree honors the section's folded directories.
    fn section_layout_rows(&self, section: Section, files: &[FileDiff]) -> Vec<TreeRow> {
        match self.config.ui.status_file_layout {
            // review and kinds are diff-sidebar-only (config rejects them
            // here); a stray value degrades to the flat list
            FileLayout::List | FileLayout::Review | FileLayout::Kinds | FileLayout::Walkthrough => {
                let paths: Vec<&str> = files.iter().map(|f| f.path.as_str()).collect();
                tree::flat_rows(&paths)
            }
            FileLayout::Tree => {
                let folded = self.section_folded_dirs(section);
                let paths: Vec<&str> = files.iter().map(|f| f.path.as_str()).collect();
                tree::visible_rows(&paths, &folded)
            }
        }
    }

    /// Flattened cursor-addressable rows given current fold/expansion state.
    /// Empty sections are hidden, neogit-style; blank separators are a
    /// rendering concern, so j/k skip them by construction.
    // one flat block per band, straight-line by design
    #[allow(clippy::too_many_lines, clippy::cognitive_complexity)]
    pub fn visible_rows(&self) -> Vec<Row> {
        let mut rows = Vec::new();
        // the branch's own PR leads the band: it is the thing the branch is
        // for, and it lands from an async fetch, so whoever sets `pr`
        // re-seats the cursor over the rows it displaces
        if self.pr.is_some() {
            rows.push(Row::Pr);
        }
        // the repo's walkthroughs, right where the reader decides what to
        // open; newest first, so the one just published leads
        if !self.status.walkthroughs.is_empty() {
            rows.push(Row::WalkthroughHeader {
                count: self.status.walkthroughs.len(),
            });
            if !self.is_group_folded(Group::Walkthrough) {
                rows.extend(
                    self.status
                        .walkthroughs
                        .iter()
                        .map(|w| Row::Walkthrough { id: w.id.clone() }),
                );
            }
        }
        for section in Section::ALL {
            let files = self.section_files(section);
            if files.is_empty() {
                continue;
            }
            rows.push(Row::SectionHeader {
                section,
                count: files.len(),
            });
            if self.is_folded(section) {
                continue;
            }
            // List renders a degenerate tree (one File row per file at depth 0,
            // no Dir rows), so the same cursor/navigation model serves both
            // layouts; Tree groups files under collapsible directory rows.
            for tree_row in self.section_layout_rows(section, files) {
                match tree_row.node {
                    // status sections never produce review buckets, stops,
                    // or the walkthrough sidebar's leading row
                    TreeNode::Section { .. }
                    | TreeNode::Stop { .. }
                    | TreeNode::WalkthroughSummary => {}
                    TreeNode::Dir { path, name } => rows.push(Row::Dir {
                        section,
                        path,
                        name,
                        depth: tree_row.depth,
                    }),
                    TreeNode::File { index, .. } => {
                        rows.push(Row::File {
                            section,
                            index,
                            depth: tree_row.depth,
                        });
                        let Some(file) = files.get(index) else {
                            continue;
                        };
                        if !self.is_expanded(section, &file.path) {
                            continue;
                        }
                        for (hunk_index, hunk) in file.hunks.iter().enumerate() {
                            rows.push(Row::HunkHeader {
                                section,
                                file: index,
                                hunk: hunk_index,
                            });
                            rows.extend((0..hunk.lines.len()).map(|line| Row::DiffLine {
                                section,
                                file: index,
                                hunk: hunk_index,
                                line,
                            }));
                        }
                    }
                }
            }
        }
        // this-branch band: HEAD's own history, which `Vcs::log` walks from
        // HEAD, so recent commits belong here and not under the repo divider
        if let Some(unpushed) = &self.status.unpushed {
            rows.push(Row::UnpushedHeader {
                count: unpushed.commits.len(),
                capped: unpushed.capped,
            });
            if !self.is_group_folded(Group::Unpushed) {
                for (index, entry) in unpushed.commits.iter().enumerate() {
                    rows.push(Row::Unpushed { index });
                    rows.extend(self.nested_runs(&entry.oid));
                }
            }
        }
        {
            rows.push(Row::RecentHeader {
                count: self.status.recent.len(),
            });
            if !self.is_group_folded(Group::Recent) {
                for (index, entry) in self.status.recent.iter().enumerate() {
                    rows.push(Row::Commit { index });
                    rows.extend(self.nested_runs(&entry.oid));
                }
            }
        }
        // this-repo band, folded by default: branches, open PRs, CI runs
        //
        // a group is present when the repo can have the thing at all, never
        // because it currently has one: a count of zero is an answer, and a row
        // that deletes itself on unfold hides whether it was even asked
        let has_forge = !self.ci_remotes().is_empty();
        rows.push(Row::RepoDivider);
        {
            rows.push(Row::BranchesHeader {
                count: self.status.branches.len(),
            });
            if !self.is_group_folded(Group::Branches) {
                let shown = self.status.branches.len().min(BRANCHES_INLINE_LIMIT);
                rows.extend((0..shown).map(|index| Row::Branch { index }));
            }
        }
        if has_forge {
            let others = self.other_prs();
            rows.push(Row::PrsHeader {
                count: self.status.prs_loaded.then_some(others.len()),
            });
            if !self.is_group_folded(Group::Prs) {
                let shown = others.len().min(PRS_INLINE_LIMIT);
                rows.extend((0..shown).map(|index| Row::OpenPr { index }));
            }
        }
        if has_forge {
            rows.push(Row::CiHeader {
                count: self.runs.len(),
            });
            if !self.is_group_folded(Group::Ci) {
                let shown = self.runs.len().min(CI_INLINE_LIMIT);
                rows.extend((0..shown).map(|index| Row::CiRun {
                    index,
                    nested: false,
                }));
            }
        }
        rows
    }

    /// Searchable `(row index, text)` pairs for the `/` search: section
    /// titles, directory names, file paths, and recent-commit lines. Inline
    /// diff rows are left out: the diff view is where code is searched.
    pub(crate) fn status_search_rows(&self) -> Vec<(usize, String)> {
        self.visible_rows()
            .iter()
            .enumerate()
            .filter_map(|(index, row)| self.status_row_label(row).map(|text| (index, text)))
            .collect()
    }

    fn status_row_label(&self, row: &Row) -> Option<String> {
        Some(match row {
            Row::SectionHeader { section, .. } => {
                section.title(self.review.vcs.has_index()).to_owned()
            }
            Row::UnpushedHeader { .. } => UNPUSHED_TITLE.to_owned(),
            Row::Unpushed { index } => self.status.unpushed_commits().get(*index)?.subject.clone(),
            Row::RecentHeader { .. } => RECENT_TITLE.to_owned(),
            Row::Dir { name, .. } => name.clone(),
            Row::File { section, index, .. } => self
                .status_file_name(self.section_files(*section).get(*index)?)
                .to_owned(),
            Row::Commit { index } => self.status.recent.get(*index)?.subject.clone(),
            Row::BranchesHeader { .. } => BRANCHES_TITLE.to_owned(),
            Row::Branch { index } => self.status.branches.get(*index)?.name.clone(),
            Row::Pr => {
                let pr = self.pr.as_ref()?;
                format!("PR #{} {}", pr.number, pr.title)
            }
            Row::WalkthroughHeader { .. } => WALKTHROUGHS_TITLE.to_owned(),
            Row::Walkthrough { id } => self
                .status
                .walkthroughs
                .iter()
                .find(|w| w.id == *id)?
                .title
                .clone(),
            Row::PrsHeader { .. } => PRS_TITLE.to_owned(),
            Row::OpenPr { index } => {
                let pr = self.other_prs();
                let pr = pr.get(*index)?;
                format!("PR #{} {}", pr.number, pr.title)
            }
            Row::CiHeader { .. } => CI_TITLE.to_owned(),
            Row::CiRun { index, .. } => self.runs.get(*index)?.name.clone(),
            Row::HunkHeader { .. } | Row::DiffLine { .. } | Row::RepoDivider => return None,
        })
    }

    /// The text a file row displays: the basename in the tree layout (the
    /// directory rows above carry the path), the whole repo-relative path in
    /// the flat list. The search labels and the renderer share it so a `/`
    /// match highlights exactly the displayed substring.
    pub(crate) fn status_file_name<'a>(&self, file: &'a FileDiff) -> &'a str {
        if self.config.ui.status_file_layout == FileLayout::List {
            file.path.as_str()
        } else {
            file.path.rsplit('/').next().unwrap_or(&file.path)
        }
    }

    pub fn section_files(&self, section: Section) -> &[FileDiff] {
        let model = match section {
            Section::Untracked => &self.review.status.untracked,
            Section::Unstaged => &self.review.status.unstaged,
            Section::Staged => &self.review.status.staged,
        };
        &model.files
    }

    /// Queue background enrichment (intra-line emphasis + syntax highlight)
    /// for every currently-expanded inline diff. Cheap and deduped by
    /// content, so the renderer calls it per frame; the expanded rows draw
    /// plain until the outcome lands as an `AppEvent::Enriched` event: draw
    /// only renders.
    pub(crate) fn queue_enrich_status_expanded(&mut self) {
        let semantic = self.config.ui.semantic_diff;
        let algorithm = self.config.diff.algorithm;
        for section in Section::ALL {
            let index = section.index();
            let model = match section {
                Section::Untracked => &self.review.status.untracked,
                Section::Unstaged => &self.review.status.unstaged,
                Section::Staged => &self.review.status.staged,
            };
            let (Some(expanded), Some(enriched)) = (
                self.status.expanded.get(index),
                self.status.enriched.get(index),
            ) else {
                continue;
            };
            for file in &model.files {
                if file.binary || file.hunks.is_empty() || !expanded.contains(&file.path) {
                    continue;
                }
                let ready = enriched.contains(&file.path)
                    && self
                        .status
                        .highlights
                        .contains_key(&(file.path.clone(), file.sides_hash()));
                super::enrich::queue_if_stale(
                    &mut self.enrich_inflight,
                    &mut self.pending_enrich,
                    file,
                    semantic,
                    algorithm,
                    ready,
                );
            }
        }
    }

    /// Install a finished enrichment into every status-section file it still
    /// matches (same path and content): swap in the emphasised hunks, cache
    /// the syntax highlights, and mark the file enriched so it isn't
    /// re-queued. A stale outcome (the file changed while the job ran)
    /// matches nothing and is dropped; the next frame re-queues.
    pub(super) fn install_status_enrichment(&mut self, outcome: &EnrichOutcome) {
        // entries for content no section still shows are dead; drop them so
        // an edited file doesn't accumulate one entry per past hash
        let live: Vec<String> = [
            &self.review.status.untracked,
            &self.review.status.unstaged,
            &self.review.status.staged,
        ]
        .into_iter()
        .flat_map(|model| &model.files)
        .filter(|file| file.path == outcome.path)
        .map(FileDiff::sides_hash)
        .collect();
        self.status
            .highlights
            .retain(|(path, hash), _| *path != outcome.path || live.contains(hash));
        for section in Section::ALL {
            let index = section.index();
            let model = match section {
                Section::Untracked => &mut self.review.status.untracked,
                Section::Unstaged => &mut self.review.status.unstaged,
                Section::Staged => &mut self.review.status.staged,
            };
            let Some(enriched) = self.status.enriched.get_mut(index) else {
                continue;
            };
            for file in &mut model.files {
                if file.path == outcome.path && file.sides_hash() == outcome.hash {
                    file.hunks.clone_from(&outcome.hunks);
                    self.status.highlights.insert(
                        (outcome.path.clone(), outcome.hash.clone()),
                        outcome.highlights.clone(),
                    );
                    enriched.insert(file.path.clone());
                }
            }
        }
    }

    /// Move the cursor by half a screenful, clamped to the visible rows.
    fn status_page(&mut self, down: bool, full: bool) {
        let step = super::page_step(self.status.viewport, full);
        let rows = self.visible_rows();
        let last = rows.len().saturating_sub(1);
        self.status.cursor = if down {
            nearest_selectable(&rows, (self.status.cursor + step).min(last), true)
        } else {
            nearest_selectable(&rows, self.status.cursor.saturating_sub(step), false)
        };
    }

    pub(super) fn status_mouse(&mut self, gesture: super::MouseGesture) {
        use super::MouseGesture;
        match gesture {
            MouseGesture::Scroll { down, .. } => {
                let delta = if down { 3 } else { -3 };
                let rows = self.visible_rows();
                let last = rows.len().saturating_sub(1);
                let target = self.status.cursor.saturating_add_signed(delta).min(last);
                self.status.cursor = nearest_selectable(&rows, target, down);
            }
            // single-click selects; double-click activates (open file/commit,
            // or fold the section/dir/recent header), like `<cr>`/`<tab>`
            MouseGesture::Press { col, row } => {
                self.status_select_at(col, row);
            }
            MouseGesture::DoublePress { col, row } => {
                if self.status_select_at(col, row) {
                    self.status_activate_cursor();
                }
            }
            // a right-click cancels what is in progress, which here is a run of
            // rows the reader was selecting
            MouseGesture::Cancel => self.status.set_anchor(None),
            MouseGesture::Drag { .. } => {}
        }
    }

    /// Move the cursor to the row under `(col, row)`. Returns whether a row was
    /// hit (so a double-click only activates on a real row).
    fn status_select_at(&mut self, col: u16, row: u16) -> bool {
        let Some(line) = super::hit_index(self.status.body, self.status.scroll as usize, col, row)
        else {
            return false;
        };
        let Some(Some(index)) = self.status.line_rows.get(line).copied() else {
            return false;
        };
        if index >= self.visible_rows().len() {
            return false;
        }
        // a click names where the reader wants to be, so it clears any
        // anchor and starts a fresh selection there
        self.status.set_anchor(None);
        self.status.cursor = index;
        true
    }

    fn status_activate_cursor(&mut self) {
        match self.cursor_row() {
            Some(
                Row::File { .. }
                | Row::Unpushed { .. }
                | Row::Commit { .. }
                | Row::Branch { .. }
                | Row::OpenPr { .. }
                | Row::CiRun { .. }
                | Row::Walkthrough { .. },
            ) => {
                self.open_at_cursor();
            }
            Some(
                Row::SectionHeader { .. }
                | Row::Dir { .. }
                | Row::UnpushedHeader { .. }
                | Row::PrsHeader { .. }
                | Row::BranchesHeader { .. }
                | Row::RecentHeader { .. }
                | Row::CiHeader { .. }
                | Row::WalkthroughHeader { .. },
            ) => {
                self.toggle_fold();
            }
            _ => {}
        }
    }

    pub(super) fn dispatch_status(&mut self, action: Action) {
        match action {
            Action::VisualSelect => self.status.toggle_visual(),
            Action::MoveDown => {
                let rows = self.visible_rows();
                let last = rows.len().saturating_sub(1);
                self.status.cursor =
                    nearest_selectable(&rows, (self.status.cursor + 1).min(last), true);
            }
            Action::MoveUp => {
                let rows = self.visible_rows();
                self.status.cursor =
                    nearest_selectable(&rows, self.status.cursor.saturating_sub(1), false);
            }
            Action::HalfPageDown => self.status_page(true, false),
            Action::HalfPageUp => self.status_page(false, false),
            Action::FullPageDown => self.status_page(true, true),
            Action::FullPageUp => self.status_page(false, true),
            Action::GoTop => {
                let rows = self.visible_rows();
                self.status.cursor = nearest_selectable(&rows, 0, true);
            }
            Action::GoBottom => {
                let rows = self.visible_rows();
                let last = rows.len().saturating_sub(1);
                self.status.cursor = nearest_selectable(&rows, last, false);
            }
            Action::NextHunk => self.jump(true, is_hunk_header),
            Action::PrevHunk => self.jump(false, is_hunk_header),
            Action::NextSection => self.jump(true, is_section_header),
            Action::PrevSection => self.jump(false, is_section_header),
            Action::ToggleFold => self.toggle_fold(),
            Action::Stage => self.stage_at_cursor(),
            Action::Unstage => self.unstage_at_cursor(),
            Action::StageAll => self.stage_all(),
            Action::UnstageAll => self.unstage_all(),
            Action::Discard => self.discard_at_cursor(),
            Action::DeleteComment => self.delete_walkthrough_at_cursor(),
            Action::Open => self.open_at_cursor(),
            Action::OpenReviewDiff | Action::DiffWorkingTree => self.open_working_tree_diff(None),
            Action::DiffBase => self.diff_against_base(),
            Action::DiffLastCommit => self.open_against_diff("HEAD~1"),
            Action::DiffBranch => self.diff_against_branch(),
            Action::DiffCommit => self.diff_against_commit(),
            Action::MarkViewed => self.toggle_viewed(),
            Action::LogView => self.open_log(),
            Action::CommitFlow => self.commit_flow(),
            Action::CommitExtend => self.commit_extend(),
            Action::CommitAmend => self.commit_amend(),
            Action::CommitReword => self.commit_reword(),
            Action::BranchCheckout => self.checkout_at_status_cursor(),
            Action::BranchCreateCheckout => self.branch_name_input(true),
            Action::BranchCreate => self.branch_name_input(false),
            Action::BranchDelete => self.open_branch_list(BranchAction::Delete),
            Action::Push => self.push(),
            Action::PushSetUpstream => self.push_set_upstream(),
            Action::Pull => self.pull(),
            Action::Fetch => self.request_network(NetworkOp::Fetch, "fetch"),
            Action::FetchAll => self.request_network(NetworkOp::FetchAll, "fetch --all"),
            Action::StashPush => self.stash_push(),
            Action::StashPop => self.stash_pop(),
            Action::OpenEditor => self.editor_at_status_cursor(),
            Action::CopyUrl => self.copy_at_status_cursor(),
            other => {
                self.info(format!("{} is not implemented yet", other.name()));
            }
        }
    }

    /// Review against the branch a pull request would target: the primary
    /// remote's default branch, or a local main/master.
    fn diff_against_base(&mut self) {
        let remotes = self.review.vcs.remotes().unwrap_or_default();
        let primary = remotes
            .iter()
            .find(|name| *name == "origin")
            .or_else(|| remotes.first())
            .map_or("origin", String::as_str);
        match self.review.vcs.default_branch(primary) {
            Ok(Some(branch)) => self.open_against_diff(&branch),
            Ok(None) => self.info("no base branch detected; pick one with d b"),
            Err(err) => self.error(err.to_string()),
        }
    }

    fn diff_against_branch(&mut self) {
        match self.review.vcs.all_branches() {
            Ok(branches) => {
                let entries = branches
                    .into_iter()
                    .map(|name| super::RevChoice {
                        rev: name.clone(),
                        label: name,
                    })
                    .collect();
                self.open_rev_list("Diff against branch", entries);
            }
            Err(err) => self.error(err.to_string()),
        }
    }

    fn diff_against_commit(&mut self) {
        match self.review.vcs.log(super::log::LOG_LIMIT) {
            Ok(entries) => {
                let entries = entries
                    .into_iter()
                    .map(|entry| super::RevChoice {
                        label: format!("{} {}", entry.oid7, entry.subject),
                        rev: entry.oid,
                    })
                    .collect();
                self.open_rev_list("Diff against commit", entries);
            }
            Err(err) => self.error(err.to_string()),
        }
    }

    /// A pull-request row checks that request out; anywhere else the key keeps
    /// its usual job and opens the branch picker. The branch's own PR is left
    /// out: its head is the branch already under you.
    fn checkout_at_status_cursor(&mut self) {
        let pr = match self.cursor_row() {
            Some(Row::OpenPr { index }) => self.other_prs().get(index).map(|pr| (*pr).clone()),
            _ => None,
        };
        match pr {
            Some(pr) => self.checkout_pr(&pr),
            None => self.open_branch_list(BranchAction::Checkout),
        }
    }

    /// Copy whatever the cursor is on, in the form you would paste elsewhere:
    /// a pull request as its forge URL, a commit as its full sha, a file or a
    /// folder as its repo-relative path.
    fn copy_at_status_cursor(&mut self) {
        let copied = match self.cursor_row() {
            Some(Row::Dir { path, .. }) => Some((Some(path.clone()), path)),
            Some(Row::Pr) => self
                .pr
                .as_ref()
                .map(|pr| (pr.url.clone(), format!("#{}", pr.number))),
            Some(Row::Walkthrough { id }) => self
                .status
                .walkthroughs
                .iter()
                .find(|w| w.id == id)
                .map(|w| (Some(w.title.clone()), w.title.clone())),
            Some(Row::OpenPr { index }) => self
                .other_prs()
                .get(index)
                .map(|pr| (pr.url.clone(), format!("#{}", pr.number))),
            Some(Row::Commit { index }) => self
                .status
                .recent
                .get(index)
                .map(|entry| (Some(entry.oid.clone()), entry.oid7.clone())),
            Some(Row::Unpushed { index }) => self
                .status
                .unpushed_commits()
                .get(index)
                .map(|entry| (Some(entry.oid.clone()), entry.oid7.clone())),
            // a file row, a hunk header, or a line inside an expanded diff:
            // all of them address one file, the way the editor jump reads them
            // a file row, a hunk header, or a line inside an expanded diff:
            // all of them address one file, the way the editor jump reads them
            other => other
                .as_ref()
                .and_then(|row| self.row_file(row))
                .map(|(_, file, _)| (Some(file.path.clone()), file.path.clone())),
        };
        let Some((value, label)) = copied else {
            self.info("nothing to copy here");
            return;
        };
        self.copy_or_report(value, &label);
    }

    fn editor_at_status_cursor(&mut self) {
        match self.status_cursor_file_line() {
            Some((path, line)) => self.request_editor(&path, line),
            None => self.info("no file under the cursor"),
        }
    }

    /// The file and line the status cursor addresses, shared by the editor
    /// jump and blame.
    pub(crate) fn status_cursor_file_line(&self) -> Option<(String, Option<u32>)> {
        let row = self.cursor_row()?;
        // For an expanded inline diff line, pass the line number so the
        // editor opens at the right spot, same as the dedicated diff view.
        let line_no = if let Row::DiffLine {
            section,
            file,
            hunk,
            line,
        } = row
        {
            self.section_files(section)
                .get(file)
                .and_then(|f| f.hunks.get(hunk))
                .and_then(|h| h.lines.get(line))
                .and_then(|l| l.new_no.or(l.old_no))
        } else {
            None
        };
        let path = self.row_file(&row).map(|(_, file, _)| file.path.clone())?;
        Some((path, line_no))
    }

    fn cursor_row(&self) -> Option<Row> {
        self.visible_rows().get(self.status.cursor).cloned()
    }

    /// The file a row addresses, with the hunk index for hunk-scoped rows.
    pub fn row_file(&self, row: &Row) -> Option<(Section, &FileDiff, Option<usize>)> {
        match *row {
            Row::File { section, index, .. } => self
                .section_files(section)
                .get(index)
                .map(|file| (section, file, None)),
            Row::HunkHeader {
                section,
                file,
                hunk,
            }
            | Row::DiffLine {
                section,
                file,
                hunk,
                ..
            } => self
                .section_files(section)
                .get(file)
                .map(|f| (section, f, Some(hunk))),
            Row::Dir { .. }
            | Row::SectionHeader { .. }
            | Row::UnpushedHeader { .. }
            | Row::Unpushed { .. }
            | Row::Pr
            | Row::WalkthroughHeader { .. }
            | Row::Walkthrough { .. }
            | Row::RepoDivider
            | Row::PrsHeader { .. }
            | Row::OpenPr { .. }
            | Row::BranchesHeader { .. }
            | Row::Branch { .. }
            | Row::RecentHeader { .. }
            | Row::Commit { .. }
            | Row::CiHeader { .. }
            | Row::CiRun { .. } => None,
        }
    }

    fn stage_at_cursor(&mut self) {
        let Some(row) = self.cursor_row() else {
            return;
        };
        let Some((section, file, hunk)) = self.row_file(&row) else {
            return;
        };
        // jj's whole working copy sits in Staged, so we let its backend decline
        if section == Section::Staged && self.review.vcs.has_index() {
            self.info("already staged");
            return;
        }
        let path = file.path.clone();
        match hunk {
            None => self.vcs_op(move |vcs| vcs.stage(Path::new(&path))),
            Some(hunk) => {
                let Some(id) = file.hunks.get(hunk).map(|h| h.id.clone()) else {
                    return;
                };
                self.vcs_op(move |vcs| vcs.stage_hunk(Path::new(&path), &id));
            }
        }
    }

    fn unstage_at_cursor(&mut self) {
        let Some(row) = self.cursor_row() else {
            return;
        };
        let Some((section, file, hunk)) = self.row_file(&row) else {
            return;
        };
        if section != Section::Staged {
            self.info("not staged");
            return;
        }
        let path = file.path.clone();
        match hunk {
            None => self.vcs_op(move |vcs| vcs.unstage(Path::new(&path))),
            Some(hunk) => {
                let Some(id) = file.hunks.get(hunk).map(|h| h.id.clone()) else {
                    return;
                };
                self.vcs_op(move |vcs| vcs.unstage_hunk(Path::new(&path), &id));
            }
        }
    }

    /// Stage everything, resolved by the backend: the section model is a
    /// snapshot, and a file edited on disk since the last refresh is missing
    /// from it, which is what made this take two presses.
    fn stage_all(&mut self) {
        if self.review.vcs.has_index()
            && self.section_files(Section::Untracked).is_empty()
            && self.section_files(Section::Unstaged).is_empty()
        {
            self.info("nothing to stage");
            return;
        }
        self.vcs_op(|vcs| vcs.stage_everything());
    }

    fn unstage_all(&mut self) {
        if self.section_files(Section::Staged).is_empty() {
            self.info("nothing staged");
            return;
        }
        self.vcs_op(|vcs| vcs.unstage_everything());
    }

    fn discard_at_cursor(&mut self) {
        let Some(row) = self.cursor_row() else {
            return;
        };
        let Some((_, file, _)) = self.row_file(&row) else {
            return;
        };
        let path = file.path.clone();
        self.modal = Some(Modal::Confirm {
            message: format!("Discard changes to {path}?"),
            on_confirm: PendingOp::Discard { path },
        });
    }

    /// `d` is the diff transient's key on this screen, and a walkthrough
    /// row's delete only while the cursor sits on one; the header takes no
    /// special action, like every other header.
    pub(super) fn status_cursor_on_walkthrough(&self) -> bool {
        self.screen() == super::Screen::Status
            && matches!(self.cursor_row(), Some(Row::Walkthrough { .. }))
    }

    fn delete_walkthrough_at_cursor(&mut self) {
        if let Some(Row::Walkthrough { id }) = self.cursor_row() {
            self.confirm_delete_walkthrough(&id);
        }
    }

    fn stash_push(&mut self) {
        self.message = None;
        self.vcs_op(|vcs| vcs.stash_push(None));
        if self.message.is_none() {
            self.info("stashed changes");
        }
    }

    fn stash_pop(&mut self) {
        self.message = None;
        self.vcs_op(|vcs| vcs.stash_pop());
        if self.message.is_none() {
            self.info("popped latest stash");
        }
    }

    /// The oids of the commits a `V` selection covers, newest first. Rows that
    /// are not commits sit between the sections and are simply passed over.
    fn selected_commit_oids(&self) -> Vec<String> {
        let Some((top, bottom)) = self.status.selection() else {
            return Vec::new();
        };
        let rows = self.visible_rows();
        let unpushed = self.status.unpushed_commits();
        rows.get(top..=bottom)
            .unwrap_or_default()
            .iter()
            .filter_map(|row| match *row {
                Row::Unpushed { index } => unpushed.get(index).map(|entry| entry.oid.clone()),
                Row::Commit { index } => {
                    self.status.recent.get(index).map(|entry| entry.oid.clone())
                }
                _ => None,
            })
            .collect()
    }

    /// `<cr>` with a selection: the selected commits read as one review, since
    /// a run of commits is usually one piece of work. The list is newest first,
    /// so the last of it is the range's oldest end.
    fn open_selected_commits(&mut self) -> bool {
        let oids = self.selected_commit_oids();
        let (Some(newest), Some(oldest)) = (oids.first(), oids.last()) else {
            return false;
        };
        if oids.len() < 2 {
            return false;
        }
        let (oldest, newest) = (oldest.clone(), newest.clone());
        self.status.set_anchor(None);
        self.open_range_diff(&oldest, &newest);
        true
    }

    fn open_at_cursor(&mut self) {
        if self.open_selected_commits() {
            return;
        }
        let Some(row) = self.cursor_row() else {
            return;
        };
        match &row {
            Row::Unpushed { index } => {
                let Some(oid) = self
                    .status
                    .unpushed_commits()
                    .get(*index)
                    .map(|entry| entry.oid.clone())
                else {
                    return;
                };
                self.open_commit_diff(&oid);
            }
            Row::Commit { index } => {
                let Some(oid) = self.status.recent.get(*index).map(|e| e.oid.clone()) else {
                    return;
                };
                self.open_commit_diff(&oid);
            }
            Row::Pr => self.open_pr_review(),
            Row::Walkthrough { id } => {
                let has_summary = self
                    .status
                    .walkthroughs
                    .iter()
                    .any(|w| w.id == *id && w.has_summary);
                let slide = if has_summary {
                    crate::app::diff::Slide::Summary
                } else {
                    crate::app::diff::Slide::Stop(0)
                };
                self.open_walkthrough(id, slide);
            }
            // a PR row reviews that PR directly; the header opens the full list
            Row::OpenPr { index } => {
                let Some(pr) = self.other_prs().get(*index).map(|pr| (*pr).clone()) else {
                    return;
                };
                self.open_pr_review_for(pr);
            }
            Row::PrsHeader { .. } => self.open_prs(),
            // a branch row checks it out directly; the header opens the full picker
            Row::Branch { index } => {
                let Some(name) = self.status.branches.get(*index).map(|b| b.name.clone()) else {
                    return;
                };
                self.checkout_branch(&name);
            }
            Row::BranchesHeader { .. } => self.open_branch_list(BranchAction::Checkout),
            // a CI run opens its graph directly; the header opens the full Runs list
            Row::CiRun { index, .. } => {
                self.runs_cursor = *index;
                self.open_selected_run();
            }
            Row::CiHeader { .. } => self.open_runs(),
            Row::RecentHeader { .. } => self.open_log(),
            // a section header opens the full review diff, starting the
            // walk at the section's first file (when the review covers it)
            Row::SectionHeader { section, .. } => {
                let section = *section;
                let review_model = self.review.model();
                let path = self
                    .section_files(section)
                    .iter()
                    .find(|f| review_model.files.iter().any(|m| m.path == f.path))
                    .map(|f| f.path.clone());
                self.open_working_tree_diff(path.as_deref());
            }
            // file/hunk/diff rows open the file; a dir row has no file: no-op
            row => {
                let Some(path) = self.row_file(row).map(|(_, file, _)| file.path.clone()) else {
                    return;
                };
                self.open_working_tree_file(&path);
            }
        }
    }

    fn toggle_viewed(&mut self) {
        let Some(row) = self.cursor_row() else {
            return;
        };
        let Some(path) = self.row_file(&row).map(|(_, file, _)| file.path.clone()) else {
            return;
        };
        let Some(hash) = self
            .review
            .model()
            .files
            .iter()
            .find(|f| f.path == path)
            .map(FileDiff::content_hash)
        else {
            self.info(format!("{path} is not part of the review diff"));
            return;
        };
        if self.review.session.is_viewed(&path, &hash) {
            self.review.session.unmark_viewed(&path);
        } else {
            self.review.session.mark_viewed(&path, &hash);
            // a viewed file reads as done: collapse its inline diffs
            for set in &mut self.status.expanded {
                set.remove(&path);
            }
        }
        if let Err(err) = self.review.save() {
            self.error(err.to_string());
        }
        self.clamp_cursor();
    }

    #[allow(clippy::too_many_lines)] // one match arm per row kind, straight-line by design
    fn toggle_fold(&mut self) {
        let Some(row) = self.cursor_row() else {
            return;
        };
        match row {
            Row::SectionHeader { section, .. } => {
                if let Some(folded) = self.status.folded.get_mut(section.index()) {
                    *folded ^= true;
                }
                self.cursor_to_section_header(section);
            }
            // a directory folds/unfolds in place; its row stays under the cursor
            Row::Dir { section, path, .. } => {
                if let Some(set) = self.status.folded_dirs.get_mut(section.index())
                    && !set.remove(&path)
                {
                    set.insert(path.clone());
                }
                self.seat_cursor_on(
                    |row| matches!(row, Row::Dir { section: s, path: p, .. } if *s == section && *p == path),
                );
            }
            Row::File { section, index, .. } => {
                let Some(path) = self
                    .section_files(section)
                    .get(index)
                    .map(|f| f.path.clone())
                else {
                    return;
                };
                if let Some(set) = self.status.expanded.get_mut(section.index())
                    && !set.remove(&path)
                {
                    set.insert(path);
                }
            }
            Row::HunkHeader { section, file, .. } | Row::DiffLine { section, file, .. } => {
                let Some(path) = self
                    .section_files(section)
                    .get(file)
                    .map(|f| f.path.clone())
                else {
                    return;
                };
                if let Some(set) = self.status.expanded.get_mut(section.index()) {
                    set.remove(&path);
                }
                // collapsing from inside lands the cursor on the file row
                self.seat_cursor_on(
                    |row| matches!(row, Row::File { section: s, index, .. } if *s == section && *index == file),
                );
            }
            Row::UnpushedHeader { .. } => {
                self.toggle_group(Group::Unpushed, |row| {
                    matches!(row, Row::UnpushedHeader { .. })
                });
            }
            Row::PrsHeader { .. } => {
                self.toggle_group(Group::Prs, |row| matches!(row, Row::PrsHeader { .. }));
                // the list is fetched once, the first time the group opens;
                // a re-fold and re-unfold rides the regular CI poll instead
                if !self.is_group_folded(Group::Prs)
                    && !self.status.prs_loaded
                    && !self.status.prs_in_flight
                {
                    self.status.prs_in_flight = true;
                    self.pending_ci = Some(super::CiRequest::Prs);
                }
            }
            Row::RecentHeader { .. } => {
                self.toggle_group(Group::Recent, |row| matches!(row, Row::RecentHeader { .. }));
            }
            Row::BranchesHeader { .. } => {
                self.toggle_group(Group::Branches, |row| {
                    matches!(row, Row::BranchesHeader { .. })
                });
            }
            Row::CiHeader { .. } => {
                self.toggle_group(Group::Ci, |row| matches!(row, Row::CiHeader { .. }));
            }
            Row::WalkthroughHeader { .. } => {
                self.toggle_group(Group::Walkthrough, |row| {
                    matches!(row, Row::WalkthroughHeader { .. })
                });
            }
            Row::Unpushed { index } => {
                if let Some(oid) = self
                    .status
                    .unpushed_commits()
                    .get(index)
                    .map(|entry| entry.oid.clone())
                {
                    self.toggle_commit_runs(oid);
                }
            }
            Row::Commit { index } => {
                if let Some(oid) = self.status.recent.get(index).map(|e| e.oid.clone()) {
                    self.toggle_commit_runs(oid);
                }
            }
            Row::Pr
            | Row::Walkthrough { .. }
            | Row::RepoDivider
            | Row::CiRun { .. }
            | Row::Branch { .. }
            | Row::OpenPr { .. } => {}
        }
        self.clamp_cursor();
    }

    /// Fold/unfold one repo-band group (or the always-open Unpushed group
    /// above it) and land the cursor on its header, the way every one of
    /// these toggles must.
    fn toggle_group(&mut self, group: Group, is_header: impl Fn(&Row) -> bool) {
        if let Some(folded) = self.status.group_folded.get_mut(group.index()) {
            *folded ^= true;
        }
        self.seat_cursor_on(is_header);
    }

    /// Move the cursor onto the first visible row matching `pred`, if any.
    /// Every fold toggle needs this re-seat once the row set it sits in shifts.
    fn seat_cursor_on(&mut self, pred: impl Fn(&Row) -> bool) {
        self.status.set_anchor(None);
        if let Some(position) = self.visible_rows().iter().position(pred) {
            self.status.cursor = position;
        }
    }

    fn cursor_to_section_header(&mut self, section: Section) {
        self.seat_cursor_on(
            |row| matches!(row, Row::SectionHeader { section: s, .. } if *s == section),
        );
    }

    /// Move the cursor to the next/previous row matching `target`.
    fn jump(&mut self, forward: bool, target: impl Fn(&Row) -> bool) {
        let rows = self.visible_rows();
        if let Some(position) = super::step_to(&rows, self.status.cursor, forward, target) {
            self.status.cursor = position;
        }
    }

    pub(super) fn status_cursor_anchor(&self) -> Option<CursorAnchor> {
        let row = self.cursor_row()?;
        Some(match &row {
            Row::SectionHeader { section, .. } => CursorAnchor::Section(*section),
            Row::Dir { section, path, .. } => CursorAnchor::Dir {
                section: *section,
                path: path.clone(),
            },
            Row::UnpushedHeader { .. } => CursorAnchor::Unpushed,
            Row::Unpushed { index } => CursorAnchor::UnpushedCommit(*index),
            Row::RecentHeader { .. } => CursorAnchor::Recent,
            Row::Commit { index } => CursorAnchor::Commit(*index),
            Row::Pr => CursorAnchor::Pr,
            Row::WalkthroughHeader { .. } => CursorAnchor::WalkthroughHeader,
            Row::Walkthrough { id } => CursorAnchor::Walkthrough(id.clone()),
            // furniture: the cursor never actually rests here
            Row::RepoDivider => return None,
            Row::PrsHeader { .. } => CursorAnchor::Prs,
            Row::OpenPr { index } => CursorAnchor::PrsRow(*index),
            Row::BranchesHeader { .. } => CursorAnchor::Branches,
            Row::Branch { index } => {
                CursorAnchor::BranchRow(self.status.branches.get(*index)?.name.clone())
            }
            Row::CiHeader { .. } => CursorAnchor::Ci,
            Row::CiRun { index, .. } => CursorAnchor::CiRun(*index),
            Row::File { .. } | Row::HunkHeader { .. } | Row::DiffLine { .. } => {
                let (section, file, hunk) = self.row_file(&row)?;
                let line = match row {
                    Row::DiffLine {
                        section,
                        file,
                        hunk,
                        line,
                    } => self
                        .section_files(section)
                        .get(file)?
                        .hunks
                        .get(hunk)?
                        .lines
                        .get(line)
                        .map(|line| LineAnchor {
                            old_no: line.old_no,
                            new_no: line.new_no,
                        }),
                    _ => None,
                };
                CursorAnchor::File {
                    section,
                    path: file.path.clone(),
                    hunk,
                    line,
                }
            }
        })
    }

    /// Re-seat the cursor after rows changed: exact hunk → same file in the
    /// same section → same path anywhere → the section header → clamp.
    pub(super) fn restore_status_cursor(&mut self, anchor: Option<CursorAnchor>) {
        self.status.set_anchor(None);
        let Some(anchor) = anchor else {
            self.clamp_cursor();
            return;
        };
        let rows = self.visible_rows();
        let position = match &anchor {
            CursorAnchor::Section(section) => rows
                .iter()
                .position(|r| matches!(r, Row::SectionHeader { section: s, .. } if s == section)),
            CursorAnchor::Unpushed => header_pos(&rows, |r| matches!(r, Row::UnpushedHeader { .. })),
            CursorAnchor::UnpushedCommit(index) => indexed_or_header(
                &rows,
                |r| matches!(r, Row::Unpushed { index: i } if i == index),
                |r| matches!(r, Row::UnpushedHeader { .. }),
            ),
            CursorAnchor::Recent => header_pos(&rows, |r| matches!(r, Row::RecentHeader { .. })),
            CursorAnchor::Commit(index) => indexed_or_header(
                &rows,
                |r| matches!(r, Row::Commit { index: i } if i == index),
                |r| matches!(r, Row::RecentHeader { .. }),
            ),
            CursorAnchor::Pr => header_pos(&rows, |r| matches!(r, Row::Pr)),
            CursorAnchor::WalkthroughHeader => {
                header_pos(&rows, |r| matches!(r, Row::WalkthroughHeader { .. }))
            }
            CursorAnchor::Walkthrough(id) => indexed_or_header(
                &rows,
                |r| matches!(r, Row::Walkthrough { id: i } if i == id),
                |r| matches!(r, Row::WalkthroughHeader { .. }),
            ),
            CursorAnchor::Prs => header_pos(&rows, |r| matches!(r, Row::PrsHeader { .. })),
            CursorAnchor::PrsRow(index) => indexed_or_header(
                &rows,
                |r| matches!(r, Row::OpenPr { index: i } if i == index),
                |r| matches!(r, Row::PrsHeader { .. }),
            ),
            CursorAnchor::Branches => header_pos(&rows, |r| matches!(r, Row::BranchesHeader { .. })),
            // by name: checking a branch out re-sorts the list under the cursor
            CursorAnchor::BranchRow(name) => indexed_or_header(
                &rows,
                |r| {
                    matches!(r, Row::Branch { index }
                        if self.status.branches.get(*index).is_some_and(|b| b.name == *name))
                },
                |r| matches!(r, Row::BranchesHeader { .. }),
            ),
            CursorAnchor::Ci => header_pos(&rows, |r| matches!(r, Row::CiHeader { .. })),
            CursorAnchor::CiRun(index) => indexed_or_header(
                &rows,
                |r| matches!(r, Row::CiRun { index: i, .. } if i == index),
                |r| matches!(r, Row::CiHeader { .. }),
            ),
            // a folded dir survives a refresh by its path; fall back to the
            // section header when the directory is gone
            CursorAnchor::Dir { section, path } => rows
                .iter()
                .position(
                    |r| matches!(r, Row::Dir { section: s, path: p, .. } if s == section && p == path),
                )
                .or_else(|| {
                    rows.iter().position(
                        |r| matches!(r, Row::SectionHeader { section: s, .. } if s == section),
                    )
                }),
            CursorAnchor::File {
                section,
                path,
                hunk,
                line,
            } => self.file_anchor_position(&rows, *section, path, *hunk, *line),
        };
        match position {
            Some(position) => self.status.cursor = position,
            None => self.clamp_cursor(),
        }
    }

    /// Where a file's cursor lands after the rows changed, narrowest first:
    /// the line it was reading, that line's hunk, the file in its own section,
    /// the file wherever it moved to, then the section header.
    fn file_anchor_position(
        &self,
        rows: &[Row],
        section: Section,
        path: &str,
        hunk: Option<usize>,
        line: Option<LineAnchor>,
    ) -> Option<usize> {
        let file_at = |row: &Row| -> Option<(Section, usize)> {
            match row {
                Row::File { section, index, .. } => Some((*section, *index)),
                _ => None,
            }
        };
        let path_matches = |s: Section, index: usize| {
            self.section_files(s)
                .get(index)
                .is_some_and(|f| f.path == *path)
        };
        let line_position = line.and_then(|wanted| {
            rows.iter().position(|r| {
                let Row::DiffLine {
                    section: s,
                    file,
                    hunk,
                    line,
                } = r
                else {
                    return false;
                };
                *s == section
                    && path_matches(*s, *file)
                    && self
                        .section_files(*s)
                        .get(*file)
                        .and_then(|f| f.hunks.get(*hunk)?.lines.get(*line))
                        .is_some_and(|found| {
                            found.old_no == wanted.old_no && found.new_no == wanted.new_no
                        })
            })
        });
        let hunk_position = hunk.and_then(|h| {
            rows.iter().position(|r| {
                matches!(
                    r,
                    Row::HunkHeader { section: s, file, hunk } if *s == section && *hunk == h && path_matches(*s, *file)
                )
            })
        });
        line_position
            .or(hunk_position)
            .or_else(|| {
                rows.iter().position(|r| {
                    file_at(r).is_some_and(|(s, index)| s == section && path_matches(s, index))
                })
            })
            .or_else(|| {
                rows.iter()
                    .position(|r| file_at(r).is_some_and(|(s, index)| path_matches(s, index)))
            })
            .or_else(|| {
                rows.iter().position(
                    |r| matches!(r, Row::SectionHeader { section: s, .. } if *s == section),
                )
            })
    }

    pub(super) fn clamp_cursor(&mut self) {
        // a selection names rows by position, so it cannot outlive a rebuild of
        // the row list: the rows it covered are no longer the rows it covered
        self.status.set_anchor(None);
        let rows = self.visible_rows();
        let clamped = self.status.cursor.min(rows.len().saturating_sub(1));
        self.status.cursor = nearest_selectable(&rows, clamped, true);
    }
}

/// A header row's position, for a [`CursorAnchor`] that names nothing else.
fn header_pos(rows: &[Row], header: impl Fn(&Row) -> bool) -> Option<usize> {
    rows.iter().position(header)
}

/// An indexed row's position, falling back to its group's header once the
/// item itself is gone (moved section, disappeared commit, and so on).
fn indexed_or_header(
    rows: &[Row],
    at: impl Fn(&Row) -> bool,
    header: impl Fn(&Row) -> bool,
) -> Option<usize> {
    rows.iter()
        .position(at)
        .or_else(|| header_pos(rows, header))
}

fn is_hunk_header(row: &Row) -> bool {
    matches!(row, Row::HunkHeader { .. })
}

fn is_section_header(row: &Row) -> bool {
    matches!(
        row,
        Row::SectionHeader { .. }
            | Row::UnpushedHeader { .. }
            | Row::PrsHeader { .. }
            | Row::BranchesHeader { .. }
            | Row::RecentHeader { .. }
            | Row::CiHeader { .. }
            | Row::WalkthroughHeader { .. }
    )
}

/// Furniture rows the cursor must never land on.
fn is_selectable(row: &Row) -> bool {
    !matches!(row, Row::RepoDivider)
}

fn first_selectable_from(rows: &[Row], start: usize) -> Option<usize> {
    rows.iter()
        .enumerate()
        .skip(start)
        .find(|(_, row)| is_selectable(row))
        .map(|(index, _)| index)
}

fn last_selectable_up_to(rows: &[Row], end: usize) -> Option<usize> {
    rows.iter()
        .enumerate()
        .take(end + 1)
        .rev()
        .find(|(_, row)| is_selectable(row))
        .map(|(index, _)| index)
}

/// The selectable row nearest `index`, preferring `forward`'s direction and
/// falling back to the other one, so a divider (alone, doubled up, or at
/// either end) can never trap the cursor.
fn nearest_selectable(rows: &[Row], index: usize, forward: bool) -> usize {
    if rows.is_empty() {
        return 0;
    }
    let index = index.min(rows.len() - 1);
    let found = if forward {
        first_selectable_from(rows, index).or_else(|| last_selectable_up_to(rows, index))
    } else {
        last_selectable_up_to(rows, index).or_else(|| first_selectable_from(rows, index))
    };
    found.unwrap_or(0)
}

#[cfg(test)]
mod tests {
    use diffler_core::source::ReviewSource;

    use super::super::{CiRequest, Screen};
    use super::*;
    use crate::app::App;
    use crate::config::LoadedConfig;
    use crate::event::AppEvent;
    use crate::test_support::{Fixture, ctrl_key, key, standard_fixture, two_hunk_fixture};
    use crossterm::event::{KeyCode, KeyEvent, KeyModifiers};

    fn app() -> (Fixture, App) {
        let fixture = standard_fixture();
        let app = App::new(fixture.review(), LoadedConfig::default());
        (fixture, app)
    }

    /// The status screen with `data.txt`'s diff unfolded and the cursor on a
    /// line inside it, which is where a reader sits while a poll lands.
    fn app_reading_a_hunk() -> (Fixture, App) {
        let fixture = two_hunk_fixture();
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        // onto the file row, unfold it, then down into the hunk's lines
        app.handle(key('j'));
        app.handle(key('\t'));
        for _ in 0..4 {
            app.handle(key('j'));
        }
        assert!(
            matches!(app.cursor_row(), Some(Row::DiffLine { .. })),
            "the cursor is inside the diff: {:?}",
            app.cursor_row()
        );
        (fixture, app)
    }

    /// The line under the cursor, by the numbers the file gives it.
    fn cursor_line_numbers(app: &App) -> (Option<u32>, Option<u32>) {
        let Some(Row::DiffLine {
            section,
            file,
            hunk,
            line,
        }) = app.cursor_row()
        else {
            panic!("cursor is not on a diff line: {:?}", app.cursor_row());
        };
        let line = app.section_files(section)[file].hunks[hunk].lines[line].clone();
        (line.old_no, line.new_no)
    }

    /// A CI poll or a watcher echo rebuilds the status without changing a
    /// byte, and the reader keeps their place in the hunk they were reading.
    #[test]
    fn a_refresh_keeps_the_cursor_on_the_diff_line_it_was_on() {
        let (_fixture, mut app) = app_reading_a_hunk();
        let before = app.status.cursor;
        let numbers = cursor_line_numbers(&app);

        app.queue_refresh();
        app.settle_refresh();

        assert_eq!(app.status.cursor, before, "the row did not move");
        assert_eq!(cursor_line_numbers(&app), numbers);
    }

    /// An edit below the cursor rebuilds the hunks, and the cursor follows its
    /// own line.
    #[test]
    fn an_edit_elsewhere_leaves_the_cursor_on_its_own_line() {
        let (fixture, mut app) = app_reading_a_hunk();
        let numbers = cursor_line_numbers(&app);

        // the fixture edits both ends of data.txt; touch the far end again
        let text = std::fs::read_to_string(fixture.root.join("data.txt")).expect("read");
        fixture.write("data.txt", &text.replace("line twenty", "line twenty-one"));
        app.queue_refresh();
        app.settle_refresh();

        assert!(
            matches!(app.cursor_row(), Some(Row::DiffLine { .. })),
            "still on a diff line, not bumped to the header: {:?}",
            app.cursor_row()
        );
        assert_eq!(cursor_line_numbers(&app), numbers);
    }

    /// An app whose status file layout is forced to `layout`, overriding the
    /// default.
    fn app_with_status_layout(layout: crate::config::FileLayout) -> (Fixture, App) {
        let fixture = standard_fixture();
        let mut loaded = LoadedConfig::default();
        loaded.config.ui.status_file_layout = layout;
        let app = App::new(fixture.review(), loaded);
        (fixture, app)
    }

    /// Move the cursor onto the first row matching `pred`.
    /// Three commits on the status screen, so a run of them can be selected.
    fn app_with_commits() -> (Fixture, App) {
        let fixture = standard_fixture();
        fixture.write("notes.txt", "alpha\nbeta\n");
        fixture.commit_all("add beta note");
        fixture.write(
            "src/util.rs",
            "pub fn twice(x: u32) -> u32 {\n    x * 2\n}\n",
        );
        fixture.commit_all("add util module");
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(key('\t'));
        (fixture, app)
    }

    /// `V` over a run of commits and `<cr>`: the whole run reads as one range,
    /// which is what a stack of commits usually is.
    #[test]
    fn enter_on_a_selected_run_of_commits_opens_their_combined_range() {
        let (_fixture, mut app) = app_with_commits();
        cursor_to(&mut app, |row| matches!(row, Row::Commit { index: 0 }));
        let newest = app.status.recent[0].oid.clone();
        let oldest = app.status.recent[1].oid.clone();

        app.handle(key('V'));
        app.handle(key('j'));
        app.handle(key('\n'));

        let diff = app.diff.as_ref().expect("diff view");
        let diffler_core::source::ReviewSource::Range {
            oldest: from,
            newest: to,
        } = &diff.source
        else {
            panic!("expected a range, got {:?}", diff.source);
        };
        assert_eq!(
            (from.as_str(), to.as_str()),
            (oldest.as_str(), newest.as_str())
        );
        assert!(app.status.anchor.is_none(), "opening ends the selection");
    }

    /// One commit selected is one commit: the range only earns itself with a
    /// second, so `<cr>` opens the single review it always did.
    #[test]
    fn enter_on_a_single_selected_commit_opens_that_commit() {
        let (_fixture, mut app) = app_with_commits();
        cursor_to(&mut app, |row| matches!(row, Row::Commit { index: 0 }));
        let newest = app.status.recent[0].oid.clone();

        app.handle(key('V'));
        app.handle(key('\n'));

        let diff = app.diff.as_ref().expect("diff view");
        assert_eq!(
            diff.source,
            diffler_core::source::ReviewSource::commit(&newest)
        );
    }

    /// A selection dragged up over the section header covers rows that are not
    /// commits; they carry no oid, so the range is still the commits in it.
    #[test]
    fn a_selection_reaching_past_the_header_ranges_only_the_commits() {
        let (_fixture, mut app) = app_with_commits();
        cursor_to(&mut app, |row| matches!(row, Row::Commit { index: 1 }));
        let oldest = app.status.recent[1].oid.clone();
        let newest = app.status.recent[0].oid.clone();

        app.handle(key('V'));
        app.handle(key('k'));
        app.handle(key('k'));
        assert!(
            matches!(app.cursor_row(), Some(Row::RecentHeader { .. })),
            "the selection reaches the header"
        );
        app.handle(key('\n'));

        let diff = app.diff.as_ref().expect("diff view");
        let diffler_core::source::ReviewSource::Range {
            oldest: from,
            newest: to,
        } = &diff.source
        else {
            panic!("expected a range, got {:?}", diff.source);
        };
        assert_eq!(
            (from.as_str(), to.as_str()),
            (oldest.as_str(), newest.as_str())
        );
    }

    /// Expanding a file's diff inserts rows above the selected commits, so the
    /// rows the selection named are no longer those rows. It ends there rather
    /// than reviewing whatever moved under it.
    #[test]
    fn expanding_a_file_above_the_selection_ends_it() {
        let fixture = standard_fixture();
        fixture.commit_all("second commit");
        fixture.write("notes.txt", "alpha\nbeta\ngamma\n");
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(key('\t'));
        cursor_to(&mut app, |row| matches!(row, Row::Commit { index: 0 }));
        app.handle(key('V'));
        app.handle(key('j'));
        assert!(app.status.anchor.is_some(), "a run of commits is selected");

        cursor_to(&mut app, |row| matches!(row, Row::File { .. }));
        app.handle(key('\t'));

        assert!(app.status.anchor.is_none(), "the rows it named have moved");
        app.handle(key('\n'));
        assert!(
            !matches!(
                app.diff.as_ref().map(|diff| &diff.source),
                Some(diffler_core::source::ReviewSource::Range { .. })
            ),
            "no range is built from rows nobody selected"
        );
    }

    /// Esc drops a status selection, the way it drops one anywhere else.
    #[test]
    fn escape_cancels_the_status_selection() {
        let (_fixture, mut app) = app_with_commits();
        cursor_to(&mut app, |row| matches!(row, Row::Commit { index: 0 }));
        app.handle(key('V'));
        assert!(app.status.anchor.is_some());
        app.handle(crate::test_support::esc_key());
        assert!(app.status.anchor.is_none());
    }

    fn cursor_to(app: &mut App, pred: impl Fn(&Row) -> bool) -> Row {
        let rows = app.visible_rows();
        let position = rows.iter().position(pred).expect("row present");
        app.status.cursor = position;
        rows.into_iter().nth(position).expect("row present")
    }

    fn file_row_in(section: Section) -> impl Fn(&Row) -> bool {
        move |row| matches!(row, Row::File { section: s, .. } if *s == section)
    }

    fn esc() -> AppEvent {
        AppEvent::Key(KeyEvent::new(KeyCode::Esc, KeyModifiers::NONE))
    }

    #[test]
    fn enter_on_the_pr_row_queues_the_head_fetch() {
        let (_fixture, mut app) = app();
        app.pr = Some(crate::ci::PullRequest {
            number: 7,
            title: "Add widgets".into(),
            url: None,
            head_ref: "feat/x".into(),
            author: String::new(),
            base_ref: "main".into(),
            head_oid: "0000000000000000000000000000000000000abc".into(),
        });
        cursor_to(&mut app, |row| matches!(row, Row::Pr));
        app.handle(AppEvent::Key(KeyEvent::new(
            KeyCode::Enter,
            KeyModifiers::NONE,
        )));
        // the head isn't local, so the open waits on a forge fetch
        let git = app.pending_git.take().expect("fetch queued");
        assert_eq!(git.argv[..2], ["git".to_owned(), "fetch".to_owned()]);
        assert!(
            git.argv.iter().any(|a| a == "refs/pull/7/head"),
            "{:?}",
            git.argv
        );
        assert_eq!(app.pending_pr_open.as_ref().map(|p| p.number), Some(7));
    }

    #[test]
    fn cursor_moves_and_clamps() {
        let (_fixture, mut app) = app();
        // flat default: untracked (header + todo.md) + unstaged (header +
        // lib.rs) + staged (header + ci.yml) + the repo-band divider +
        // branches header + recent commits header: 9 rows
        assert_eq!(app.visible_rows().len(), 9);
        app.handle(key('k'));
        assert_eq!(app.status.cursor, 0, "MoveUp clamps at the top");
        for _ in 0..20 {
            app.handle(key('j'));
        }
        assert_eq!(app.status.cursor, 8, "MoveDown clamps at the last row");
    }

    #[test]
    fn gg_and_shift_g_jump_to_the_edges() {
        let (_fixture, mut app) = app();
        app.handle(key('G'));
        assert_eq!(app.status.cursor, app.visible_rows().len() - 1);
        app.handle(key('g'));
        app.handle(key('g'));
        assert_eq!(app.status.cursor, 0);
    }

    #[test]
    fn half_page_motions_step_by_the_viewport_and_clamp() {
        let (_fixture, mut app) = app();
        assert_eq!(app.visible_rows().len(), 9);
        // a half-page of a 4-row body is 2 rows
        app.status.viewport = 4;
        app.handle(ctrl_key('d'));
        assert_eq!(app.status.cursor, 2);
        app.handle(ctrl_key('d'));
        assert_eq!(app.status.cursor, 4);
        app.handle(ctrl_key('u'));
        assert_eq!(app.status.cursor, 2);
        // a tall viewport clamps to the last row, never past it
        app.status.viewport = 40;
        app.handle(ctrl_key('d'));
        assert_eq!(app.status.cursor, 8);
        app.handle(ctrl_key('u'));
        assert_eq!(app.status.cursor, 0);
    }

    #[test]
    fn fold_toggles_the_section_under_the_cursor() {
        let (_fixture, mut app) = app();
        // the untracked section holds one root-level file (todo.md)
        app.handle(key('\t'));
        assert!(app.is_folded(Section::Untracked));
        assert_eq!(app.visible_rows().len(), 8);
        app.handle(key('\t'));
        assert!(!app.is_folded(Section::Untracked));
        assert_eq!(app.visible_rows().len(), 9);
    }

    #[test]
    fn the_default_layout_lists_files_flat_with_no_dir_rows() {
        let (_fixture, app) = app();
        let rows = app.visible_rows();
        // the flat magit list emits no Dir rows at all
        assert!(
            !rows.iter().any(|r| matches!(r, Row::Dir { .. })),
            "flat list has no directory rows: {rows:?}"
        );
        // every file row sits at depth 0 (no tree indentation)
        assert!(
            rows.iter()
                .all(|r| !matches!(r, Row::File { depth, .. } if *depth != 0)),
            "flat file rows live at depth 0: {rows:?}"
        );
        // the nested unstaged file is still present, just without its src dir
        let unstaged_files = rows
            .iter()
            .filter(|r| {
                matches!(
                    r,
                    Row::File {
                        section: Section::Unstaged,
                        ..
                    }
                )
            })
            .count();
        assert_eq!(unstaged_files, 1, "the src/lib.rs row is there: {rows:?}");
    }

    #[test]
    fn the_tree_layout_lists_its_files_as_a_directory_tree() {
        let (_fixture, app) = app_with_status_layout(crate::config::FileLayout::Tree);
        // unstaged holds src/lib.rs: a src dir row precedes the file row
        let rows = app.visible_rows();
        let unstaged: Vec<&Row> = rows
            .iter()
            .skip_while(|r| {
                !matches!(
                    r,
                    Row::SectionHeader {
                        section: Section::Unstaged,
                        ..
                    }
                )
            })
            .skip(1)
            .take_while(|r| {
                !matches!(
                    r,
                    Row::SectionHeader { .. }
                        | Row::RepoDivider
                        | Row::BranchesHeader { .. }
                        | Row::RecentHeader { .. }
                        | Row::CiHeader { .. }
                )
            })
            .collect();
        assert!(
            matches!(unstaged.first(), Some(Row::Dir { path, depth: 0, .. }) if path == "src"),
            "src dir row at depth 0: {unstaged:?}"
        );
        assert!(
            matches!(unstaged.get(1), Some(Row::File { depth: 1, .. })),
            "the file row nests under the dir: {unstaged:?}"
        );
    }

    #[test]
    fn tab_on_a_dir_row_folds_it_and_hides_its_files() {
        let (_fixture, mut app) = app_with_status_layout(crate::config::FileLayout::Tree);
        // cursor onto the src dir row in the unstaged section
        cursor_to(
            &mut app,
            |row| matches!(row, Row::Dir { path, .. } if path == "src"),
        );
        app.handle(key('\t'));
        assert!(app.is_dir_folded(Section::Unstaged, "src"));
        assert!(
            !app.visible_rows().iter().any(|r| matches!(
                r,
                Row::File {
                    section: Section::Unstaged,
                    ..
                }
            )),
            "folding src/ hid the file under it"
        );
        // the cursor stayed on the (still-visible) dir row
        assert!(matches!(
            app.visible_rows()[app.status.cursor],
            Row::Dir { path: ref p, .. } if p == "src"
        ));
        // tab again unfolds and the file returns
        app.handle(key('\t'));
        assert!(!app.is_dir_folded(Section::Unstaged, "src"));
        assert!(app.visible_rows().iter().any(|r| matches!(
            r,
            Row::File {
                section: Section::Unstaged,
                ..
            }
        )));
    }

    #[test]
    fn untracked_files_slot_into_their_tree_by_path() {
        let fixture = Fixture::new();
        fixture.write("src/lib.rs", "pub fn answer() -> u32 {\n    41\n}\n");
        fixture.commit_all("initial commit");
        // a new untracked file in a nested directory
        fixture.write("docs/api/intro.md", "# intro\n");
        let mut loaded = LoadedConfig::default();
        loaded.config.ui.status_file_layout = crate::config::FileLayout::Tree;
        let app = App::new(fixture.review(), loaded);
        let rows = app.visible_rows();
        let kinds: Vec<String> = rows
            .iter()
            .skip_while(|r| {
                !matches!(
                    r,
                    Row::SectionHeader {
                        section: Section::Untracked,
                        ..
                    }
                )
            })
            .skip(1)
            .take_while(|r| {
                !matches!(
                    r,
                    Row::SectionHeader { .. }
                        | Row::RepoDivider
                        | Row::BranchesHeader { .. }
                        | Row::RecentHeader { .. }
                        | Row::CiHeader { .. }
                )
            })
            .map(|r| match r {
                Row::Dir { path, depth, .. } => format!("dir:{path}@{depth}"),
                Row::File { index, depth, .. } => format!("file:{index}@{depth}"),
                other => format!("{other:?}"),
            })
            .collect();
        assert_eq!(
            kinds,
            vec![
                // docs/ and api/ are a single-child chain, collapsed to one row
                "dir:docs/api@0".to_owned(),
                "file:0@1".to_owned(),
            ],
            "the untracked file nests under the collapsed docs/api chain"
        );
    }

    #[test]
    fn tab_on_a_file_row_expands_its_inline_diff() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('\t'));
        assert!(app.is_expanded(Section::Unstaged, "src/lib.rs"));
        let rows = app.visible_rows();
        assert!(rows.iter().any(is_hunk_header), "hunk rows appear inline");
        assert!(
            rows.iter().any(|r| matches!(r, Row::DiffLine { .. })),
            "diff line rows appear inline"
        );
        app.handle(key('\t'));
        assert!(!app.is_expanded(Section::Unstaged, "src/lib.rs"));
    }

    #[test]
    fn tab_inside_an_expanded_diff_collapses_back_to_the_file_row() {
        let (_fixture, mut app) = app();
        let row = cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('\t'));
        app.handle(key('j'));
        app.handle(key('j'));
        assert!(matches!(
            app.visible_rows()[app.status.cursor],
            Row::DiffLine { .. }
        ));
        app.handle(key('\t'));
        assert!(!app.is_expanded(Section::Unstaged, "src/lib.rs"));
        assert_eq!(app.visible_rows()[app.status.cursor], row);
    }

    #[test]
    fn expansion_survives_refresh() {
        let (fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('\t'));
        fixture.write("another.md", "more\n");
        app.handle(ctrl_key('r'));
        assert!(app.is_expanded(Section::Unstaged, "src/lib.rs"));
        assert!(app.visible_rows().iter().any(is_hunk_header));
    }

    #[test]
    fn hunk_jumps_move_between_hunk_headers() {
        let fixture = two_hunk_fixture();
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('\t'));
        // the brackets step groups on this screen, leaving the hunk walk bound
        // to nothing by default and reachable only through a config remap
        app.dispatch_status(Action::NextHunk);
        let first = app.status.cursor;
        assert!(is_hunk_header(&app.visible_rows()[first]));
        app.dispatch_status(Action::NextHunk);
        let second = app.status.cursor;
        assert!(second > first, "second hunk header is further down");
        assert!(is_hunk_header(&app.visible_rows()[second]));
        app.dispatch_status(Action::PrevHunk);
        assert_eq!(app.status.cursor, first);
    }

    #[test]
    fn section_jumps_move_between_headers() {
        let (_fixture, mut app) = app();
        app.handle(ctrl_key('n'));
        assert!(matches!(
            app.visible_rows()[app.status.cursor],
            Row::SectionHeader {
                section: Section::Unstaged,
                ..
            }
        ));
        app.handle(ctrl_key('n'));
        app.handle(ctrl_key('n'));
        // recent commits close the branch band, so they come before the divider
        assert!(matches!(
            app.visible_rows()[app.status.cursor],
            Row::RecentHeader { .. }
        ));
        app.handle(ctrl_key('p'));
        assert!(matches!(
            app.visible_rows()[app.status.cursor],
            Row::SectionHeader {
                section: Section::Staged,
                ..
            }
        ));
    }

    #[test]
    fn stage_on_a_file_row_moves_it_to_staged() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('s'));
        app.settle_refresh();
        assert_eq!(app.section_files(Section::Unstaged).len(), 0);
        let staged: Vec<_> = app
            .section_files(Section::Staged)
            .iter()
            .map(|f| f.path.as_str())
            .collect();
        assert!(staged.contains(&"src/lib.rs"));
    }

    #[test]
    fn stage_on_an_untracked_row_moves_it_to_staged() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Untracked));
        app.handle(key('s'));
        app.settle_refresh();
        assert_eq!(app.section_files(Section::Untracked).len(), 0);
        assert!(
            app.section_files(Section::Staged)
                .iter()
                .any(|f| f.path == "todo.md")
        );
    }

    #[test]
    fn stage_in_the_staged_section_hints_already_staged() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Staged));
        app.handle(key('s'));
        let message = app.message.clone().expect("message");
        assert_eq!(message.severity, super::super::Severity::Info);
        assert!(message.text.contains("already staged"));
        assert_eq!(app.section_files(Section::Staged).len(), 1);
    }

    #[test]
    fn unstage_outside_the_staged_section_hints() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('u'));
        let message = app.message.expect("message");
        assert!(message.text.contains("not staged"));
    }

    #[test]
    fn unstage_moves_a_staged_file_back() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Staged));
        app.handle(key('u'));
        app.settle_refresh();
        assert_eq!(app.section_files(Section::Staged).len(), 0);
        // ci.yml was a staged new file: unstaging makes it untracked again
        assert!(
            app.section_files(Section::Untracked)
                .iter()
                .any(|f| f.path == "ci.yml")
        );
    }

    #[test]
    fn stage_one_hunk_splits_the_file_across_sections() {
        let fixture = two_hunk_fixture();
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('\t'));
        app.dispatch_status(Action::NextHunk);
        assert!(is_hunk_header(&app.visible_rows()[app.status.cursor]));
        app.handle(key('s'));
        app.settle_refresh();
        let in_section = |section: Section| {
            app.section_files(section)
                .iter()
                .any(|f| f.path == "data.txt")
        };
        assert!(in_section(Section::Staged), "staged hunk lands in staged");
        assert!(in_section(Section::Unstaged), "other hunk stays unstaged");
    }

    /// The section model is a snapshot. A file edited on disk since the last
    /// refresh is absent from it, and staging from that list quietly skipped
    /// the edit, so the key had to be pressed twice.
    #[test]
    fn stage_all_catches_a_file_edited_since_the_last_refresh() {
        let (fixture, mut app) = app();
        fixture.write("ci.yml", "on: push\nedited: externally\n");

        app.handle(key('S'));
        app.settle_refresh();
        assert_eq!(app.section_files(Section::Untracked).len(), 0);
        assert_eq!(
            app.section_files(Section::Unstaged).len(),
            0,
            "one press stages the external edit too"
        );
        assert_eq!(app.section_files(Section::Staged).len(), 3);
    }

    /// Unstaging resolves against the repository for the same reason staging
    /// does, so it clears an entry that landed after the last refresh.
    #[test]
    fn unstage_all_clears_a_file_staged_since_the_last_refresh() {
        let (fixture, mut app) = app();
        app.handle(key('S'));
        app.settle_refresh();
        // something stages another file behind diffler's back
        fixture.write("late.txt", "late\n");
        fixture.stage("late.txt");

        app.handle(key('U'));
        app.settle_refresh();
        assert_eq!(
            app.section_files(Section::Staged).len(),
            0,
            "one press unstages what landed since the last refresh"
        );
    }

    #[test]
    fn stage_all_and_unstage_all_move_everything() {
        let (_fixture, mut app) = app();
        app.handle(key('S'));
        app.settle_refresh();
        assert_eq!(app.section_files(Section::Untracked).len(), 0);
        assert_eq!(app.section_files(Section::Unstaged).len(), 0);
        assert_eq!(app.section_files(Section::Staged).len(), 3);
        app.handle(key('U'));
        app.settle_refresh();
        assert_eq!(app.section_files(Section::Staged).len(), 0);
    }

    #[test]
    fn discard_asks_for_confirmation_and_cancels_on_n() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('x'));
        let Some(Modal::Confirm { message, .. }) = &app.modal else {
            panic!("expected a confirm modal");
        };
        assert!(message.contains("src/lib.rs"));
        // while the modal is up, normal keys are swallowed
        let cursor = app.status.cursor;
        app.handle(key('j'));
        assert_eq!(app.status.cursor, cursor);
        app.handle(key('n'));
        assert_eq!(app.modal, None);
        assert_eq!(app.section_files(Section::Unstaged).len(), 1);
    }

    #[test]
    fn discard_confirmed_with_y_drops_the_change() {
        let (fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('x'));
        app.handle(key('y'));
        app.settle_refresh();
        assert_eq!(app.modal, None);
        assert_eq!(app.section_files(Section::Unstaged).len(), 0);
        let content = std::fs::read_to_string(fixture.root.join("src/lib.rs")).unwrap();
        assert!(content.contains("41"), "worktree restored to HEAD");
    }

    #[test]
    fn discard_an_untracked_file_deletes_it() {
        let (fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Untracked));
        app.handle(key('x'));
        app.handle(key('y'));
        app.settle_refresh();
        assert!(!fixture.root.join("todo.md").exists());
        assert_eq!(app.section_files(Section::Untracked).len(), 0);
    }

    #[test]
    fn escape_cancels_the_confirm_modal() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Untracked));
        app.handle(key('x'));
        app.handle(esc());
        assert_eq!(app.modal, None);
        assert_eq!(app.section_files(Section::Untracked).len(), 1);
    }

    #[test]
    fn open_pushes_a_diff_screen_scoped_to_the_cursor_file() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('\n'));
        assert_eq!(app.screen(), Screen::Diff);
        let diff = app.diff.as_ref().expect("diff view");
        assert_eq!(diff.source, ReviewSource::WorkingTree);
        assert_eq!(
            diff.focus,
            super::super::Pane::Diff,
            "a file row focuses the diff"
        );
        let path = app.diff_cursor_path().expect("cursor on the scoped file");
        assert_eq!(path, "src/lib.rs");
    }

    #[test]
    fn open_on_a_section_header_starts_the_review_walk_at_its_first_file() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| {
            matches!(
                row,
                Row::SectionHeader {
                    section: Section::Staged,
                    ..
                }
            )
        });
        app.handle(key('\n'));
        assert_eq!(app.screen(), Screen::Diff);
        let diff = app.diff.as_ref().expect("diff view");
        assert_eq!(diff.source, ReviewSource::WorkingTree);
        assert_eq!(
            diff.focus,
            super::super::Pane::List,
            "a section header focuses the sidebar"
        );
        assert_eq!(
            app.diff_cursor_path().as_deref(),
            Some("ci.yml"),
            "selection on the staged section's first review file"
        );
        // unscoped: the whole review diff is in the view
        let model = app.diff.as_ref().unwrap().model(&app.review);
        assert!(model.files.iter().any(|f| f.path == "src/lib.rs"));
    }

    #[test]
    fn open_on_a_header_whose_files_left_the_review_lands_at_the_top() {
        let (_fixture, mut app) = app();
        // simulate the staged file leaving the review diff (e.g. a stage
        // reverted in the worktree between refreshes)
        app.review.model_mut().files.retain(|f| f.path != "ci.yml");
        cursor_to(&mut app, |row| {
            matches!(
                row,
                Row::SectionHeader {
                    section: Section::Staged,
                    ..
                }
            )
        });
        app.handle(key('\n'));
        assert_eq!(app.screen(), Screen::Diff);
        assert_eq!(
            app.diff.as_ref().expect("diff view").cursor,
            0,
            "no section file in the review diff: open at the top"
        );
    }

    #[test]
    fn shift_d_opens_the_full_review_diff_at_the_top() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Staged));
        app.handle(key('D'));
        assert_eq!(app.screen(), Screen::Diff);
        let diff = app.diff.as_ref().expect("diff view");
        assert_eq!(diff.source, ReviewSource::WorkingTree);
        assert_eq!(diff.cursor, 0, "unscoped open starts at the top");
    }

    #[test]
    fn open_on_a_commit_row_pushes_a_commit_diff() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(key('\t'));
        app.handle(key('j'));
        app.handle(key('\n'));
        assert_eq!(app.screen(), Screen::Diff);
        let diff = app.diff.as_ref().expect("diff view");
        let ReviewSource::Commit { oid } = &diff.source else {
            panic!("expected a commit source, got {:?}", diff.source);
        };
        assert_eq!(oid.len(), 40);
    }

    #[test]
    fn viewed_toggle_persists_to_disk_and_collapses_the_file() {
        let (fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('\t'));
        assert!(app.is_expanded(Section::Unstaged, "src/lib.rs"));
        app.handle(key('m'));
        assert!(app.is_path_viewed("src/lib.rs"));
        assert!(
            !app.is_expanded(Section::Unstaged, "src/lib.rs"),
            "marking viewed collapses the inline diff"
        );
        let reloaded = diffler_core::store::load(&fixture.root).unwrap();
        assert!(reloaded.viewed.contains_key("src/lib.rs"));

        app.handle(key('m'));
        assert!(!app.is_path_viewed("src/lib.rs"));
        let reloaded = diffler_core::store::load(&fixture.root).unwrap();
        assert!(!reloaded.viewed.contains_key("src/lib.rs"));
    }

    #[test]
    fn viewed_counts_feed_the_status_bar() {
        let (_fixture, mut app) = app();
        assert_eq!(app.viewed_counts(), (3, 0));
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('m'));
        assert_eq!(app.viewed_counts(), (3, 1));
    }

    #[test]
    fn viewed_counts_follow_the_open_commit_diff() {
        let (_fixture, mut app) = app();
        let oid = app.status.recent[0].oid.clone();
        app.open_commit_diff(&oid);
        let total = app
            .diff
            .as_ref()
            .and_then(|d| d.commit_model.as_ref())
            .expect("commit model")
            .files
            .len();
        assert_eq!(app.viewed_counts(), (total, 0));
        app.handle(key('m'));
        assert_eq!(
            app.viewed_counts(),
            (total, 1),
            "counts read the commit source, not the working tree"
        );
    }

    #[test]
    fn e_requests_the_editor_on_the_cursor_file() {
        let (fixture, mut app) = app();
        // pin the editor through config so the test ignores $EDITOR
        app.config.editor.command = Some("vim".to_owned());
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('e'));
        let request = app.pending_editor.clone().expect("editor request");
        assert_eq!(
            request.purpose,
            crate::editor::EditorPurpose::OpenFile {
                path: "src/lib.rs".to_owned(),
            }
        );
        let absolute = fixture.root.join("src/lib.rs");
        assert_eq!(
            request.cmd,
            vec!["vim".to_owned(), absolute.to_string_lossy().into_owned()],
            "a file row opens at the top: no line argument"
        );
    }

    #[test]
    fn e_on_a_section_header_hints() {
        let (_fixture, mut app) = app();
        app.status.cursor = 0;
        app.handle(key('e'));
        assert_eq!(app.pending_editor, None);
        let message = app.message.expect("message");
        assert!(message.text.contains("no file under the cursor"));
    }

    #[test]
    fn e_on_an_expanded_diff_line_passes_the_line_number() {
        let (fixture, mut app) = app();
        app.config.editor.command = Some("vim".to_owned());
        // expand the unstaged file's inline diff
        cursor_to(&mut app, file_row_in(Section::Unstaged));
        app.handle(key('\t'));
        // find the first DiffLine and what line number it carries
        let rows = app.visible_rows();
        let (diff_line_pos, row) = rows
            .iter()
            .enumerate()
            .find(|(_, r)| matches!(r, Row::DiffLine { .. }))
            .expect("inline diff line present");
        let Row::DiffLine {
            section,
            file,
            hunk,
            line,
        } = *row
        else {
            unreachable!()
        };
        let expected_line_no = app
            .section_files(section)
            .get(file)
            .and_then(|f| f.hunks.get(hunk))
            .and_then(|h| h.lines.get(line))
            .and_then(|l| l.new_no.or(l.old_no))
            .expect("diff line has a line number");
        app.status.cursor = diff_line_pos;
        app.handle(key('e'));
        let request = app.pending_editor.clone().expect("editor request");
        let absolute = fixture.root.join("src/lib.rs");
        let expected_arg = format!("+{expected_line_no}");
        assert!(
            request.cmd.contains(&expected_arg),
            "expected {expected_arg} in {:?}",
            request.cmd
        );
        assert!(
            request
                .cmd
                .contains(&absolute.to_string_lossy().into_owned())
        );
    }

    #[test]
    fn refresh_picks_up_new_files() {
        let (fixture, mut app) = app();
        assert_eq!(app.section_files(Section::Untracked).len(), 1);
        fixture.write("another.md", "more\n");
        app.handle(ctrl_key('r'));
        app.settle_refresh();
        assert_eq!(app.section_files(Section::Untracked).len(), 2);
    }

    #[test]
    fn cursor_anchor_survives_refresh() {
        let (fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Staged));
        // a new untracked file shifts every row below the untracked section
        fixture.write("aaa.md", "first\n");
        app.handle(ctrl_key('r'));
        app.settle_refresh();
        let Some((section, file, _)) = app
            .visible_rows()
            .get(app.status.cursor)
            .cloned()
            .and_then(|row| app.row_file(&row))
            .map(|(s, f, h)| (s, f.path.clone(), h))
        else {
            panic!("cursor should still be on a file row");
        };
        assert_eq!(section, Section::Staged);
        assert_eq!(file, "ci.yml");
    }

    #[test]
    fn cursor_falls_back_to_the_section_when_its_file_leaves() {
        let (fixture, mut app) = app();
        cursor_to(&mut app, file_row_in(Section::Untracked));
        fixture.stage("todo.md");
        app.handle(ctrl_key('r'));
        app.settle_refresh();
        // todo.md moved to staged: the anchor follows the path there
        let row = app.visible_rows()[app.status.cursor].clone();
        let (section, file, _) = app.row_file(&row).expect("file row");
        assert_eq!(section, Section::Staged);
        assert_eq!(file.path, "todo.md");
    }

    #[test]
    fn recent_commits_are_cached_and_folded_by_default() {
        let (_fixture, mut app) = app();
        assert_eq!(app.status.recent.len(), 1);
        assert!(app.is_group_folded(Group::Recent));
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(key('\t'));
        assert!(!app.is_group_folded(Group::Recent));
        assert!(
            app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::Commit { .. }))
        );
    }

    /// One CI run against `oid`, the fixture every nesting test starts from.
    fn run_on(oid: &str) -> crate::ci::CiRun {
        use crate::ci::{CiRun, JobStatus, RunId};
        CiRun {
            id: RunId("1".into()),
            name: "build".into(),
            title: String::new(),
            branch: "main".into(),
            commit: oid.to_owned(),
            author: String::new(),
            created: None,
            status: JobStatus::Ok,
            url: None,
            remote: None,
        }
    }

    #[test]
    fn tab_on_a_commit_unfolds_its_runs_and_leaves_the_section_open() {
        let (_fixture, mut app) = app();
        let oid = app.status.recent[0].oid.clone();
        app.runs = vec![run_on(&oid)];
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(key('\t'));
        cursor_to(&mut app, |row| matches!(row, Row::Commit { index: 0 }));
        app.handle(key('\t'));

        assert!(app.status.unfolded_commits.contains(&oid));
        assert!(
            !app.is_group_folded(Group::Recent),
            "unfolding one commit leaves its section open"
        );
        assert!(
            app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::CiRun { nested: true, .. })),
            "the run shows under the commit"
        );

        app.handle(key('\t'));
        assert!(!app.status.unfolded_commits.contains(&oid));
    }

    #[test]
    fn tab_on_a_commit_without_runs_does_nothing() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(key('\t'));
        let before = app.visible_rows();
        cursor_to(&mut app, |row| matches!(row, Row::Commit { index: 0 }));
        app.handle(key('\t'));
        assert!(app.status.unfolded_commits.is_empty());
        assert_eq!(app.visible_rows(), before);
    }

    #[test]
    fn enter_on_a_nested_run_opens_its_graph() {
        let (_fixture, mut app) = app();
        let oid = app.status.recent[0].oid.clone();
        app.runs = vec![run_on(&oid)];
        app.status.group_folded[Group::Recent.index()] = false;
        app.status.unfolded_commits.insert(oid);
        cursor_to(&mut app, |row| matches!(row, Row::CiRun { .. }));
        app.handle(AppEvent::Key(KeyEvent::new(
            KeyCode::Enter,
            KeyModifiers::NONE,
        )));
        assert_eq!(app.screen(), crate::app::Screen::Graph);
        assert!(app.graph.is_some());
    }

    #[test]
    fn refresh_updates_the_recent_commit_cache() {
        let (fixture, mut app) = app();
        fixture.write("notes.txt", "alpha\nbeta\n");
        fixture.commit_all("second commit");
        app.handle(ctrl_key('r'));
        app.settle_refresh();
        assert_eq!(app.status.recent.len(), 2);
        assert_eq!(app.status.recent[0].subject, "second commit");
    }

    #[test]
    fn unpushed_section_lists_commits_ahead_of_the_upstream_before_recent() {
        let fixture = standard_fixture();
        fixture.track("main", "HEAD");
        fixture.write("shipped_one.rs", "pub fn one() {}\n");
        fixture.commit_all("first unpushed");
        fixture.write("shipped_two.rs", "pub fn two() {}\n");
        fixture.commit_all("second unpushed");
        let app = App::new(fixture.review(), LoadedConfig::default());
        assert_eq!(app.status.unpushed_commits().len(), 2);
        assert_eq!(app.status.unpushed_commits()[0].subject, "second unpushed");
        assert!(!app.is_group_folded(Group::Unpushed));
        let rows = app.visible_rows();
        let unpushed_at = rows
            .iter()
            .position(|r| matches!(r, Row::UnpushedHeader { count: 2, .. }))
            .expect("unpushed header with both commits");
        let recent_at = rows
            .iter()
            .position(|r| matches!(r, Row::RecentHeader { .. }))
            .expect("recent header");
        assert!(unpushed_at < recent_at, "unpushed comes before recent");
    }

    #[test]
    fn unpushed_section_is_absent_without_a_remote() {
        let (_fixture, app) = app();
        assert!(app.status.unpushed.is_none());
        assert!(
            !app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::UnpushedHeader { .. }))
        );
    }

    #[test]
    fn an_all_pushed_branch_still_shows_the_section_at_zero() {
        let fixture = standard_fixture();
        fixture.track("main", "HEAD");
        let app = App::new(fixture.review(), LoadedConfig::default());
        assert!(app.status.unpushed_commits().is_empty());
        assert!(
            app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::UnpushedHeader { count: 0, .. }))
        );
    }

    #[test]
    fn commits_a_remote_already_has_are_not_unpushed_through_a_local_upstream() {
        // tracking a local branch is legal git, and every commit past it is
        // still pushed when a remote ref contains it
        let fixture = standard_fixture();
        fixture.branch("older");
        fixture.write("shipped.rs", "pub fn shipped() {}\n");
        fixture.commit_all("shipped work");
        fixture.track("main", "HEAD");
        fixture
            .repo
            .find_branch("main", git2::BranchType::Local)
            .expect("main")
            .set_upstream(Some("older"))
            .expect("local upstream");
        fixture.write("local.rs", "pub fn local() {}\n");
        fixture.commit_all("only here");

        let app = App::new(fixture.review(), LoadedConfig::default());
        let subjects: Vec<_> = app
            .status
            .unpushed_commits()
            .iter()
            .map(|entry| entry.subject.as_str())
            .collect();
        assert_eq!(subjects, ["only here"]);
    }

    #[test]
    fn recent_commits_exclude_the_unpushed_oids() {
        let fixture = standard_fixture();
        fixture.track("main", "HEAD");
        fixture.write("shipped.rs", "pub fn shipped() {}\n");
        fixture.commit_all("only here");
        let app = App::new(fixture.review(), LoadedConfig::default());
        assert_eq!(app.status.unpushed_commits().len(), 1);
        assert_eq!(app.status.unpushed_commits()[0].subject, "only here");
        assert!(
            app.status.recent.iter().all(|e| e.subject != "only here"),
            "recent excludes the unpushed commit: {:?}",
            app.status.recent
        );
        assert!(
            app.status
                .recent
                .iter()
                .any(|e| e.subject == "initial commit"),
            "recent still carries the already-pushed commit"
        );
    }

    #[test]
    fn pushing_moves_a_commit_from_unpushed_into_recent() {
        let fixture = standard_fixture();
        fixture.track("main", "HEAD");
        fixture.write("shipped.rs", "pub fn shipped() {}\n");
        fixture.commit_all("only here");
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        assert_eq!(app.status.unpushed_commits().len(), 1);
        fixture.track("main", "HEAD");
        app.handle(ctrl_key('r'));
        app.settle_refresh();
        assert!(app.status.unpushed_commits().is_empty());
        assert!(
            app.status.recent.iter().any(|e| e.subject == "only here"),
            "the pushed commit lands in recent rather than vanishing: {:?}",
            app.status.recent
        );
    }

    #[test]
    fn unpushed_commits_do_not_eat_into_the_recent_list() {
        let fixture = standard_fixture();
        fixture.track("main", "HEAD");
        let mut config = LoadedConfig::default();
        config.config.ui.recent_commits = 2;
        for n in 0..4 {
            fixture.write(&format!("ahead{n}.rs"), "pub fn ahead() {}\n");
            fixture.commit_all(&format!("unpushed {n}"));
        }
        let app = App::new(fixture.review(), config);
        assert_eq!(app.status.unpushed_commits().len(), 4);
        assert_eq!(
            app.status.recent.len(),
            1,
            "the fixture has one pushed commit and all of it should survive: {:?}",
            app.status.recent
        );
        assert_eq!(app.status.recent[0].subject, "initial commit");
    }

    #[test]
    fn branches_are_sorted_newest_tip_first() {
        let fixture = standard_fixture();
        // the fixture's commits share one fixed time (for stable oids), so
        // "old" is branched off before main is advanced with a later one
        fixture.branch("old");
        fixture.commit_all_at("advance main", 1_700_000_100);
        let app = App::new(fixture.review(), LoadedConfig::default());
        let names: Vec<&str> = app
            .status
            .branches
            .iter()
            .map(|b| b.name.as_str())
            .collect();
        assert_eq!(names, vec!["main", "old"], "newest tip first: {names:?}");
    }

    #[test]
    fn the_checked_out_branch_leads_even_when_its_tip_is_older() {
        let fixture = standard_fixture();
        fixture.branch("newer");
        fixture.checkout("newer");
        fixture.commit_all_at("advance newer", 1_700_000_100);
        fixture.checkout("main");
        let app = App::new(fixture.review(), LoadedConfig::default());
        let names: Vec<&str> = app
            .status
            .branches
            .iter()
            .map(|b| b.name.as_str())
            .collect();
        assert_eq!(
            names,
            vec!["main", "newer"],
            "head leads so the cap cannot drop your own branch: {names:?}"
        );
    }

    #[test]
    fn branches_are_capped_at_the_inline_limit() {
        let fixture = standard_fixture();
        for n in 0..(super::BRANCHES_INLINE_LIMIT + 2) {
            fixture.branch(&format!("b{n}"));
        }
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        let total = app.status.branches.len();
        assert!(total > super::BRANCHES_INLINE_LIMIT);
        app.status.group_folded[Group::Branches.index()] = false;
        let rows = app.visible_rows();
        assert!(
            rows.iter()
                .any(|row| matches!(row, Row::BranchesHeader { count } if *count == total)),
            "the header counts every branch, so a cut list never reads as the total: {rows:?}"
        );
        assert_eq!(
            rows.iter()
                .filter(|row| matches!(row, Row::Branch { .. }))
                .count(),
            super::BRANCHES_INLINE_LIMIT,
            "only the newest fit inline: {rows:?}"
        );
    }

    #[test]
    fn branches_group_is_folded_by_default_and_tab_unfolds_it() {
        let fixture = standard_fixture();
        fixture.branch("feat/topic");
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        assert!(app.is_group_folded(Group::Branches));
        assert!(
            !app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::Branch { .. }))
        );
        cursor_to(&mut app, |row| matches!(row, Row::BranchesHeader { .. }));
        app.handle(key('\t'));
        assert!(!app.is_group_folded(Group::Branches));
        assert!(
            app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::Branch { .. }))
        );
    }

    #[test]
    fn enter_on_a_branch_row_checks_it_out() {
        let fixture = standard_fixture();
        fixture.branch("feat/topic");
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        app.status.group_folded[Group::Branches.index()] = false;
        let target = app
            .status
            .branches
            .iter()
            .position(|b| b.name == "feat/topic")
            .expect("the new branch is loaded");
        cursor_to(
            &mut app,
            |row| matches!(row, Row::Branch { index } if *index == target),
        );
        app.handle(key('\n'));
        assert_eq!(
            app.review.vcs.head().expect("head").branch.as_deref(),
            Some("feat/topic")
        );

        // checking out re-sorts the list (head leads), and the cursor rides
        // the branch it was on
        app.settle_refresh();
        let Some(Row::Branch { index }) = app.visible_rows().get(app.status.cursor).cloned() else {
            panic!("the cursor stays on a branch row");
        };
        assert_eq!(
            app.status.branches.get(index).map(|b| b.name.as_str()),
            Some("feat/topic")
        );
    }

    #[test]
    fn checkout_on_the_current_branch_is_a_noop_info_message() {
        let (_fixture, mut app) = app();
        app.checkout_branch("main");
        let message = app.message.clone().expect("message");
        assert_eq!(message.severity, super::super::Severity::Info);
        assert!(message.text.contains("already on main"), "{message:?}");
    }

    #[test]
    fn j_and_k_skip_the_repo_divider() {
        let (_fixture, mut app) = app();
        let divider_at = app
            .visible_rows()
            .iter()
            .position(|row| matches!(row, Row::RepoDivider))
            .expect("the repo band is present with at least the current branch");
        app.status.cursor = divider_at - 1;
        app.handle(key('j'));
        assert_ne!(
            app.status.cursor, divider_at,
            "j never lands on the divider"
        );
        assert!(app.status.cursor > divider_at - 1);

        app.status.cursor = divider_at + 1;
        app.handle(key('k'));
        assert_ne!(
            app.status.cursor, divider_at,
            "k never lands on the divider"
        );
    }

    #[test]
    fn enter_on_the_branches_header_opens_the_full_picker() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| matches!(row, Row::BranchesHeader { .. }));
        app.handle(key('\n'));
        assert!(matches!(app.modal, Some(Modal::BranchList { .. })));
    }

    #[test]
    fn enter_on_the_recent_header_opens_the_log() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(key('\n'));
        assert_eq!(app.screen(), Screen::Log);
    }

    #[test]
    fn enter_on_the_ci_header_opens_the_runs_screen() {
        use crate::ci::{CiRun, JobStatus, RunId};
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![crate::app::CiRemote {
            name: "origin".to_owned(),
            detected: crate::ci::Detected {
                kind: crate::ci::ProviderKind::GitHub,
                host: None,
            },
            url: None,
        }];
        app.runs = vec![CiRun {
            id: RunId("1".into()),
            name: "CI".into(),
            title: String::new(),
            branch: "main".into(),
            commit: "abc".into(),
            author: String::new(),
            created: None,
            status: JobStatus::Ok,
            url: None,
            remote: None,
        }];
        cursor_to(&mut app, |row| matches!(row, Row::CiHeader { .. }));
        app.handle(key('\n'));
        assert_eq!(app.screen(), Screen::Runs);
    }

    /// A scheduled/cron run matches no visible commit, branch tip, or PR
    /// head: the trailing CI group is the only place it can show up at all.
    #[test]
    fn a_forge_group_stays_and_says_zero_rather_than_vanishing() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.on_prs_event(Vec::new());
        let rows = app.visible_rows();
        assert!(
            rows.iter()
                .any(|row| matches!(row, Row::PrsHeader { count: Some(0) })),
            "learning the answer is zero is an answer, not a reason to disappear: {rows:?}"
        );
        assert!(
            rows.iter()
                .any(|row| matches!(row, Row::CiHeader { count: 0 })),
            "same for runs: {rows:?}"
        );
    }

    #[test]
    fn a_repo_with_no_forge_has_no_forge_groups() {
        let (_fixture, app) = app();
        assert!(app.ci_remotes().is_empty(), "the fixture has no forge");
        let rows = app.visible_rows();
        assert!(
            !rows
                .iter()
                .any(|row| matches!(row, Row::PrsHeader { .. } | Row::CiHeader { .. })),
            "a repo that cannot have PRs or runs shows neither: {rows:?}"
        );
    }

    #[test]
    fn a_ci_poll_keeps_the_cursor_on_the_row_it_was_on() {
        use crate::ci::{CiRun, JobStatus, RunId};
        let run = |id: &str| CiRun {
            id: RunId(id.to_owned()),
            name: id.to_owned(),
            title: String::new(),
            branch: "main".into(),
            commit: "0".repeat(40),
            author: String::new(),
            created: None,
            status: JobStatus::Ok,
            url: None,
            remote: None,
        };
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.runs = vec![run("first"), run("second")];
        app.status.group_folded[Group::Ci.index()] = false;
        cursor_to(&mut app, |row| matches!(row, Row::CiRun { index: 1, .. }));
        // the run under the cursor is gone once the poll lands
        app.on_ci_runs(vec![run("first")]);
        assert!(
            matches!(app.cursor_row(), Some(Row::CiHeader { .. })),
            "a vanished run drops the cursor on its own header, not an unrelated row: {:?}",
            app.cursor_row()
        );
    }

    #[test]
    fn a_run_matching_no_commit_branch_or_pr_still_appears_in_the_trailing_group() {
        use crate::ci::{CiRun, JobStatus, RunId};
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.runs = vec![CiRun {
            id: RunId("nightly".into()),
            name: "nightly".into(),
            title: "scheduled build".into(),
            branch: "cron/nightly".into(),
            commit: "0000000000000000000000000000000000000000".into(),
            author: String::new(),
            created: None,
            status: JobStatus::Ok,
            url: None,
            remote: None,
        }];
        assert!(
            app.runs_for_commit(&app.runs[0].commit).contains(&0),
            "sanity: the run exists in App::runs"
        );
        assert!(
            app.status.branches.iter().all(|b| b.name != "cron/nightly"),
            "the run's branch matches no local branch"
        );
        app.status.group_folded[Group::Ci.index()] = false;
        let rows = app.visible_rows();
        assert!(
            rows.iter()
                .any(|row| matches!(row, Row::CiHeader { count: 1 })),
            "the header counts the run: {rows:?}"
        );
        assert!(
            rows.iter()
                .any(|row| matches!(row, Row::CiRun { index: 0, .. })),
            "the run itself renders in the flat trailing section: {rows:?}"
        );
    }

    #[test]
    fn pr_row_sits_in_the_branch_band_above_the_divider() {
        let (_fixture, mut app) = app();
        app.pr = Some(crate::ci::PullRequest {
            number: 7,
            title: "Add widgets".into(),
            url: None,
            head_ref: "feat/x".into(),
            author: String::new(),
            base_ref: "main".into(),
            head_oid: "abc".into(),
        });
        let rows = app.visible_rows();
        let pr_at = rows
            .iter()
            .position(|r| matches!(r, Row::Pr))
            .expect("pr row");
        let divider_at = rows
            .iter()
            .position(|r| matches!(r, Row::RepoDivider))
            .expect("divider row");
        assert!(
            pr_at < divider_at,
            "the PR row precedes the repo-band divider"
        );
    }

    #[test]
    fn a_pr_landing_late_leaves_the_cursor_on_the_same_row() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(AppEvent::CiPr(Some(crate::ci::PullRequest {
            number: 7,
            title: "Add widgets".into(),
            url: None,
            head_ref: "feat/x".into(),
            author: String::new(),
            base_ref: "main".into(),
            head_oid: "abc".into(),
        })));
        assert!(
            matches!(app.cursor_row(), Some(Row::RecentHeader { .. })),
            "the PR row inserts above the repo band without dragging the cursor: {:?}",
            app.cursor_row()
        );
    }

    fn github_remote() -> crate::app::CiRemote {
        crate::app::CiRemote {
            name: "origin".into(),
            detected: crate::ci::Detected {
                kind: crate::ci::ProviderKind::GitHub,
                host: None,
            },
            url: None,
        }
    }

    fn pull_request(number: u64) -> crate::ci::PullRequest {
        crate::ci::PullRequest {
            number,
            title: format!("pr {number}"),
            url: None,
            base_ref: "main".into(),
            head_ref: format!("feat/{number}"),
            head_oid: "0000000000000000000000000000000000000abc".into(),
            author: "reviewer".into(),
        }
    }

    /// Mirrors the production loop, which only ever calls the CI provider
    /// once a request is actually queued: ties the assertion to a real forge
    /// call, not just to `pending_ci` bookkeeping.
    async fn dispatch_if_queued(app: &App, provider: &crate::ci::GitHubProvider) {
        use crate::ci::ForgeProvider;
        if matches!(app.pending_ci, Some(CiRequest::Prs)) {
            let _ = provider.list_prs().await;
        }
    }

    #[tokio::test]
    async fn a_folded_prs_group_issues_no_forge_request_until_unfolded() {
        use crate::ci::GitHubProvider;
        use crate::ci::test_support::RecordingRunner;
        use std::sync::Arc;

        // the forge is on the repo before the app is built, so eagerly queueing
        // the list at construction is a reachable mistake this test can catch
        let fixture = standard_fixture();
        fixture.remote("origin", "https://github.com/acme/widgets.git");
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        assert!(!app.ci_remotes().is_empty(), "the forge is detected");
        assert!(
            !matches!(app.pending_ci, Some(CiRequest::Prs)),
            "startup asks the forge for nothing this group needs: {:?}",
            app.pending_ci
        );
        assert!(
            app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::PrsHeader { count: None })),
            "the group shows itself, countless, before any fetch"
        );

        let runner = Arc::new(RecordingRunner::new(&[("pr list", "[]")]));
        let provider = GitHubProvider::new(
            Box::new(runner.clone()),
            Vec::new(),
            None,
            crate::ci::YamlCache::default(),
            crate::ci::EtagCache::default(),
            None,
        );

        dispatch_if_queued(&app, &provider).await;
        assert!(
            runner.calls().is_empty(),
            "a folded group must not fetch: {:?}",
            runner.calls()
        );

        cursor_to(&mut app, |row| matches!(row, Row::PrsHeader { .. }));
        app.handle(key('\t'));
        assert!(matches!(app.pending_ci, Some(CiRequest::Prs)));
        dispatch_if_queued(&app, &provider).await;
        assert_eq!(
            runner.calls().len(),
            1,
            "unfolding for the first time triggers exactly one fetch: {:?}",
            runner.calls()
        );
    }

    #[test]
    fn the_bracket_keys_step_group_headers() {
        let (fixture, mut app) = app();
        fixture.write("extra.rs", "pub fn extra() {}\n");
        app.handle(ctrl_key('r'));
        app.settle_refresh();
        app.handle(key('g'));
        app.handle(key('g'));
        let mut seen = Vec::new();
        for _ in 0..4 {
            app.handle(key(']'));
            if let Some(row) = app.cursor_row() {
                seen.push(super::is_section_header(&row));
            }
        }
        assert!(
            seen.iter().all(|stop| *stop),
            "every bracket stop is a group header: {seen:?}"
        );
        let forward = app.status.cursor;
        app.handle(key('['));
        assert!(
            app.status.cursor < forward,
            "the pair walks both ways: {} then {}",
            forward,
            app.status.cursor
        );
    }

    #[tokio::test]
    async fn folding_and_reopening_mid_flight_does_not_ask_twice() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        cursor_to(&mut app, |row| matches!(row, Row::PrsHeader { .. }));
        app.handle(key('\t'));
        assert!(matches!(app.pending_ci, Some(CiRequest::Prs)));
        // the request is on the wire; its answer has not landed
        app.pending_ci = None;
        app.handle(key('\t'));
        app.handle(key('\t'));
        assert!(
            app.pending_ci.is_none(),
            "a second identical request never reaches the forge: {:?}",
            app.pending_ci
        );
        app.on_prs_event(Vec::new());
        assert!(app.status.prs_loaded);
        assert!(!app.status.prs_in_flight);
    }

    #[tokio::test]
    async fn an_unrelated_ci_failure_leaves_the_pr_fetch_in_flight() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        cursor_to(&mut app, |row| matches!(row, Row::PrsHeader { .. }));
        app.handle(key('\t'));
        app.pending_ci = None;

        app.handle(AppEvent::CiError("run list failed".into()));
        assert!(
            app.status.prs_in_flight,
            "a failed runs poll says nothing about the PR fetch still on the wire"
        );

        app.handle(AppEvent::CiPrsError("pr list failed".into()));
        assert!(
            !app.status.prs_in_flight,
            "its own failure frees the slot for a retry"
        );
    }

    #[test]
    fn the_branch_pr_leads_the_status_rows() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.pr = Some(pull_request(7));
        assert!(
            matches!(app.visible_rows().first(), Some(Row::Pr)),
            "the PR sits above the working-tree sections, not beside the commits"
        );
    }

    /// The repo's walkthroughs sit in the branch band under the pull
    /// request, folded like any other group until unfolded.
    #[test]
    fn the_walkthrough_header_follows_the_branch_pr_and_starts_folded() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.pr = Some(pull_request(7));
        seat_walkthrough(&mut app);
        let rows = app.visible_rows();
        assert!(matches!(rows.first(), Some(Row::Pr)), "{rows:?}");
        assert!(
            matches!(rows.get(1), Some(Row::WalkthroughHeader { count: 1 })),
            "{rows:?}"
        );
        assert!(
            !rows
                .iter()
                .any(|row| matches!(row, Row::Walkthrough { .. })),
            "folded by default: {rows:?}"
        );
    }

    /// A review with no walkthrough has no row: the band never says an agent
    /// wrote nothing.
    #[test]
    fn a_review_without_a_walkthrough_has_no_row() {
        let (_fixture, app) = app();
        assert!(
            !app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::WalkthroughHeader { .. }))
        );
    }

    /// `tab` on the header unfolds the walkthroughs, one row each.
    #[test]
    fn tab_on_the_walkthrough_header_unfolds_the_walkthroughs() {
        let (_fixture, mut app) = app();
        seat_walkthrough(&mut app);
        cursor_to(&mut app, |row| matches!(row, Row::WalkthroughHeader { .. }));
        app.dispatch_status(Action::ToggleFold);
        let rows = app.visible_rows();
        let ids: Vec<String> = rows
            .iter()
            .filter_map(|row| match row {
                Row::Walkthrough { id } => Some(id.clone()),
                _ => None,
            })
            .collect();
        assert_eq!(ids, vec!["w1".to_owned()], "{rows:?}");
    }

    /// Every walkthrough on disk lists as its own row, newest published first.
    #[test]
    fn two_walkthroughs_list_as_two_rows_newest_first() {
        let (_fixture, mut app) = app();
        seat_walkthrough(&mut app);
        let source = diffler_core::source::ReviewSource::walkthrough("w2");
        app.review.ensure_source(&source).expect("ensure");
        app.review.session_for_mut(&source).set_walkthrough(
            diffler_core::walkthrough::Walkthrough {
                id: "w2".to_owned(),
                title: "A second tour".to_owned(),
                author: "agent".to_owned(),
                at: 1_700_000_001,
                stops: Vec::new(),
                skipped: None,
                summary: None,
                rev: None,
                about: diffler_core::source::ReviewSource::WorkingTree,
            },
        );
        app.review.save_for(&source).expect("save w2");
        app.reload_walkthroughs();
        app.status.group_folded[Group::Walkthrough.index()] = false;
        let rows = app.visible_rows();
        assert!(
            matches!(rows[0], Row::WalkthroughHeader { count: 2 }),
            "{rows:?}"
        );
        let ids: Vec<&str> = rows
            .iter()
            .filter_map(|row| match row {
                Row::Walkthrough { id } => Some(id.as_str()),
                _ => None,
            })
            .collect();
        assert_eq!(ids, vec!["w2", "w1"], "the newest one leads");
    }

    /// `<cr>` on the header does nothing special, like every other header:
    /// only `tab` folds and unfolds it.
    #[test]
    fn enter_on_the_walkthrough_header_does_nothing_special() {
        let (_fixture, mut app) = app();
        seat_walkthrough(&mut app);
        cursor_to(&mut app, |row| matches!(row, Row::WalkthroughHeader { .. }));
        app.dispatch_status(Action::Open);
        assert_eq!(app.screen(), crate::app::Screen::Status);
    }

    /// `<cr>` on a walkthrough row opens its summary when it has one.
    #[test]
    fn enter_on_a_walkthrough_row_opens_its_summary() {
        let (_fixture, mut app) = app();
        seat_walkthrough(&mut app);
        crate::test_support::set_walkthrough_summary(&mut app, "w1", "the shape of the change");
        app.reload_walkthroughs();
        app.status.group_folded[Group::Walkthrough.index()] = false;
        cursor_to(&mut app, |row| matches!(row, Row::Walkthrough { .. }));

        app.dispatch_status(Action::Open);

        assert_eq!(app.screen(), crate::app::Screen::Diff);
        let diff = app.diff.as_ref().expect("diff view");
        assert_eq!(diff.layout, crate::config::FileLayout::Walkthrough);
        assert_eq!(diff.slide, Some(crate::app::diff::Slide::Summary));
        assert_eq!(diff.tree_cursor, 0);
    }

    /// `<cr>` on a walkthrough row with no summary opens straight on its
    /// first stop, since the sidebar has no leading row to land on.
    #[test]
    fn enter_on_a_walkthrough_row_with_no_summary_opens_the_first_stop() {
        let (_fixture, mut app) = app();
        seat_walkthrough(&mut app);
        app.status.group_folded[Group::Walkthrough.index()] = false;
        cursor_to(&mut app, |row| matches!(row, Row::Walkthrough { .. }));

        app.dispatch_status(Action::Open);

        assert_eq!(app.screen(), crate::app::Screen::Diff);
        let diff = app.diff.as_ref().expect("diff view");
        assert_eq!(diff.layout, crate::config::FileLayout::Walkthrough);
        assert_eq!(diff.slide, Some(crate::app::diff::Slide::Stop(0)));
    }

    #[test]
    fn y_on_a_walkthrough_row_copies_its_title() {
        let (_fixture, mut app) = app();
        seat_walkthrough(&mut app);
        app.status.group_folded[Group::Walkthrough.index()] = false;
        cursor_to(&mut app, |row| matches!(row, Row::Walkthrough { .. }));
        app.dispatch_status(Action::CopyUrl);
        assert_eq!(
            app.pending_clipboard.as_deref(),
            Some("How the answer moved")
        );
    }

    /// `y` on the header copies nothing, the same as any other header.
    #[test]
    fn y_on_the_walkthrough_header_copies_nothing() {
        let (_fixture, mut app) = app();
        seat_walkthrough(&mut app);
        cursor_to(&mut app, |row| matches!(row, Row::WalkthroughHeader { .. }));
        app.dispatch_status(Action::CopyUrl);
        assert_eq!(app.pending_clipboard, None);
    }

    /// `d` on a walkthrough row asks, then deletes its review file and every
    /// comment in it; `d` on the header still opens the diff transient, like
    /// every other header.
    #[test]
    fn d_on_a_walkthrough_row_asks_then_deletes_it_and_its_comments() {
        let (_fixture, mut app) = app();
        seat_walkthrough(&mut app);
        app.status.group_folded[Group::Walkthrough.index()] = false;

        cursor_to(&mut app, |row| matches!(row, Row::WalkthroughHeader { .. }));
        app.handle(key('d'));
        assert!(app.modal.is_none());
        assert!(
            app.transient.is_some(),
            "the header keeps the diff transient"
        );
        app.handle(esc());

        cursor_to(&mut app, |row| matches!(row, Row::Walkthrough { .. }));
        app.handle(key('d'));
        assert!(matches!(app.modal, Some(crate::app::Modal::Confirm { .. })));
        app.handle(key('n'));
        assert!(!app.status.walkthroughs.is_empty(), "declined: still there");

        app.handle(key('d'));
        app.handle(key('y'));

        assert!(
            app.status.walkthroughs.is_empty(),
            "the walkthrough is gone"
        );
        let source = diffler_core::source::ReviewSource::walkthrough("w1");
        assert!(
            app.review
                .all_reviews()
                .expect("all reviews")
                .into_iter()
                .all(|(s, _)| s != source),
            "its review file is gone too"
        );
    }

    fn seat_walkthrough(app: &mut App) {
        crate::test_support::seat_walkthrough(
            app,
            "How the answer moved",
            &[
                ("The answer", Some("src/lib.rs#answer"), "why 42"),
                ("The list", Some("todo.md:1"), "why a list"),
            ],
        );
    }

    #[test]
    fn y_on_the_branch_pr_copies_its_url() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        let mut pr = pull_request(7);
        pr.url = Some("https://example.invalid/acme/widgets/pull/7".to_owned());
        app.pr = Some(pr);
        cursor_to(&mut app, |row| matches!(row, Row::Pr));

        app.dispatch_status(Action::CopyUrl);
        assert_eq!(
            app.pending_clipboard.as_deref(),
            Some("https://example.invalid/acme/widgets/pull/7")
        );
    }

    #[test]
    fn y_on_a_commit_copies_its_full_sha() {
        let (_fixture, mut app) = app();
        app.status.group_folded[Group::Recent.index()] = false;
        let row = cursor_to(&mut app, |row| matches!(row, Row::Commit { .. }));
        let Row::Commit { index } = row else {
            panic!("a commit row");
        };
        let expected = app.status.recent[index].oid.clone();

        app.dispatch_status(Action::CopyUrl);
        assert_eq!(app.pending_clipboard.as_deref(), Some(expected.as_str()));
        assert_eq!(expected.len(), 40, "the full sha, not the abbreviation");
    }

    #[test]
    fn y_on_a_pr_without_a_url_copies_nothing_and_says_so() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.pr = Some(pull_request(7));
        cursor_to(&mut app, |row| matches!(row, Row::Pr));

        app.dispatch_status(Action::CopyUrl);
        assert!(app.pending_clipboard.is_none());
        assert!(
            app.message
                .as_ref()
                .is_some_and(|m| m.text.contains("no URL for #7")),
            "{:?}",
            app.message
        );
    }

    #[test]
    fn y_on_a_row_that_points_at_nothing_copies_nothing() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));

        app.dispatch_status(Action::CopyUrl);
        assert!(app.pending_clipboard.is_none());
    }

    #[test]
    fn y_on_a_file_copies_its_repo_relative_path() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| matches!(row, Row::File { .. }));
        app.dispatch_status(Action::CopyUrl);
        assert_eq!(app.pending_clipboard.as_deref(), Some("todo.md"));
    }

    /// A line inside an expanded file still addresses that file, the way the
    /// editor jump reads it.
    #[test]
    fn y_on_a_line_inside_an_expanded_file_copies_the_file() {
        let (_fixture, mut app) = app();
        cursor_to(&mut app, |row| matches!(row, Row::File { .. }));
        app.handle(key('\t'));
        cursor_to(&mut app, |row| matches!(row, Row::DiffLine { .. }));
        app.dispatch_status(Action::CopyUrl);
        assert_eq!(app.pending_clipboard.as_deref(), Some("todo.md"));
    }

    #[test]
    fn y_on_a_folder_copies_the_folder() {
        let fixture = crate::test_support::Fixture::new();
        fixture.write("src/deep/inner.rs", "pub fn b() {}\n");
        fixture.commit_all("base");
        fixture.write("src/deep/inner.rs", "pub fn b() -> u32 {\n    2\n}\n");
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        app.config.ui.status_file_layout = crate::config::FileLayout::Tree;
        app.handle(AppEvent::Tick);
        cursor_to(&mut app, |row| matches!(row, Row::Dir { .. }));
        app.dispatch_status(Action::CopyUrl);
        assert!(
            app.pending_clipboard
                .as_deref()
                .is_some_and(|path| path.starts_with("src")),
            "{:?}",
            app.pending_clipboard
        );
    }

    #[test]
    fn the_current_branch_own_pr_is_not_repeated_in_the_inline_list() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.pr = Some(pull_request(7));
        app.prs = vec![pull_request(7), pull_request(8)];
        app.status.prs_loaded = true;
        app.status.group_folded[Group::Prs.index()] = false;
        let numbers: Vec<u64> = app
            .visible_rows()
            .iter()
            .filter_map(|row| match row {
                Row::OpenPr { index } => app.other_prs().get(*index).map(|pr| pr.number),
                _ => None,
            })
            .collect();
        assert_eq!(
            numbers,
            vec![8],
            "the branch's own PR is shown once, above the divider"
        );
        assert!(
            app.visible_rows()
                .iter()
                .any(|row| matches!(row, Row::PrsHeader { count: Some(1) })),
            "the header counts what the list can show: {:?}",
            app.visible_rows()
        );
    }

    #[tokio::test]
    async fn a_folded_prs_group_stops_polling_the_forge() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        // the branch's own PR resolves first and owns the slot until it does
        app.pr_checked = true;
        app.status.prs_loaded = true;
        app.status.group_folded[Group::Prs.index()] = false;
        app.on_ci_runs(Vec::new());
        assert!(
            matches!(app.pending_ci, Some(CiRequest::Prs)),
            "an open group keeps its list fresh"
        );
        app.pending_ci = None;
        app.status.group_folded[Group::Prs.index()] = true;
        app.on_ci_runs(Vec::new());
        assert!(
            app.pending_ci.is_none(),
            "folding it away stops the poll: {:?}",
            app.pending_ci
        );
    }

    #[test]
    fn prs_group_header_counts_every_pr_while_rows_cap_at_the_limit() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.prs = (0..(super::PRS_INLINE_LIMIT as u64 + 2))
            .map(pull_request)
            .collect();
        app.status.prs_loaded = true;
        app.status.group_folded[Group::Prs.index()] = false;
        let rows = app.visible_rows();
        let total = app.prs.len();
        assert!(
            rows.iter()
                .any(|row| matches!(row, Row::PrsHeader { count: Some(c) } if *c == total)),
            "the header counts every open PR, not the capped number: {rows:?}"
        );
        assert_eq!(
            rows.iter()
                .filter(|row| matches!(row, Row::OpenPr { .. }))
                .count(),
            super::PRS_INLINE_LIMIT,
            "only the newest fit inline: {rows:?}"
        );
    }

    #[test]
    fn enter_on_the_prs_header_opens_the_full_prs_screen() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        cursor_to(&mut app, |row| matches!(row, Row::PrsHeader { .. }));
        app.handle(key('\n'));
        assert_eq!(app.screen(), Screen::Prs);
    }

    #[test]
    fn enter_on_an_open_pr_row_reviews_that_pr() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.prs = vec![pull_request(9)];
        app.status.prs_loaded = true;
        app.status.group_folded[Group::Prs.index()] = false;
        cursor_to(&mut app, |row| matches!(row, Row::OpenPr { .. }));
        app.handle(key('\n'));
        // the fixture never fetched PR 9's head, so the open queues a fetch
        // the same way the branch's own PR row does
        let git = app.pending_git.take().expect("fetch queued");
        assert!(
            git.argv.iter().any(|a| a == "refs/pull/9/head"),
            "{:?}",
            git.argv
        );
        assert_eq!(app.pending_pr_open.as_ref().map(|p| p.number), Some(9));
    }

    #[test]
    fn checkout_on_an_open_pr_row_checks_that_pr_out() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.prs = vec![pull_request(9)];
        app.status.prs_loaded = true;
        app.status.group_folded[Group::Prs.index()] = false;
        cursor_to(&mut app, |row| matches!(row, Row::OpenPr { .. }));
        app.dispatch_status(Action::BranchCheckout);
        let git = app.pending_git.take().expect("checkout queued");
        assert_eq!(git.argv, ["gh", "pr", "checkout", "9"], "{:?}", git.argv);
        assert!(app.modal.is_none(), "no branch picker over the checkout");
    }

    #[test]
    fn checkout_off_a_pr_row_still_opens_the_branch_picker() {
        let (_fixture, mut app) = app();
        app.dispatch_status(Action::BranchCheckout);
        assert!(app.pending_git.is_none(), "no checkout queued");
        assert!(
            matches!(app.modal, Some(crate::app::Modal::BranchList { .. })),
            "{:?}",
            app.modal
        );
    }

    #[test]
    fn a_pr_list_landing_late_leaves_the_cursor_on_the_same_row() {
        let (_fixture, mut app) = app();
        app.ci_remotes = vec![github_remote()];
        app.status.group_folded[Group::Prs.index()] = false;
        cursor_to(&mut app, |row| matches!(row, Row::RecentHeader { .. }));
        app.handle(AppEvent::CiPrs(vec![pull_request(141), pull_request(142)]));
        assert!(
            matches!(app.cursor_row(), Some(Row::RecentHeader { .. })),
            "the PR rows insert above the repo band without dragging the cursor: {:?}",
            app.cursor_row()
        );
    }

    #[test]
    fn group_fold_defaults_survive_the_group_refactor() {
        let (_fixture, app) = app();
        assert!(
            !app.is_group_folded(Group::Unpushed),
            "unpushed opens by default"
        );
        assert!(app.is_group_folded(Group::Prs));
        assert!(app.is_group_folded(Group::Branches));
        assert!(app.is_group_folded(Group::Recent));
        assert!(app.is_group_folded(Group::Ci));
    }

    fn type_query(app: &mut App, query: &str) {
        app.handle(key('/'));
        for c in query.chars() {
            app.handle(key(c));
        }
        app.handle(key('\n'));
    }

    #[test]
    fn slash_search_moves_the_cursor_to_a_matching_file_row() {
        let (_fixture, mut app) = app();
        type_query(&mut app, "lib");
        let row = app.visible_rows()[app.status.cursor].clone();
        let (_, file, _) = app.row_file(&row).expect("cursor on a file row");
        assert_eq!(file.path, "src/lib.rs");
    }

    #[test]
    fn search_next_and_prev_cycle_status_matches() {
        let (_fixture, mut app) = app();
        // "changes" hits the Unstaged and Staged section titles
        type_query(&mut app, "changes");
        let section_at = |app: &App| match app.visible_rows()[app.status.cursor] {
            Row::SectionHeader { section, .. } => section,
            ref other => panic!("expected a section header, got {other:?}"),
        };
        assert_eq!(section_at(&app), Section::Unstaged);
        app.handle(key('n'));
        assert_eq!(section_at(&app), Section::Staged);
        app.handle(key('n'));
        assert_eq!(section_at(&app), Section::Unstaged, "next wraps");
        app.handle(key('N'));
        assert_eq!(section_at(&app), Section::Staged, "prev wraps back");
    }

    #[test]
    fn escape_clears_a_committed_status_search() {
        let (_fixture, mut app) = app();
        type_query(&mut app, "lib");
        assert!(app.search.is_some());
        app.handle(esc());
        assert!(app.search.is_none());
    }

    #[test]
    fn partially_staged_file_expanded_in_both_sections_settles() {
        // the same path carries different content in Unstaged and Staged, so
        // each side needs its own cache entry: a path-keyed cache would make
        // the two sections evict each other and re-enrich forever
        let fixture = standard_fixture();
        fixture.stage("src/lib.rs");
        fixture.write("src/lib.rs", "pub fn answer() -> u32 {\n    43\n}\n");
        let mut app = App::new(fixture.review(), LoadedConfig::default());
        for section in [Section::Unstaged, Section::Staged] {
            app.status.expanded[section.index()].insert("src/lib.rs".to_owned());
        }

        app.queue_enrich_status_expanded();
        assert_eq!(app.pending_enrich.len(), 2, "one job per section");
        app.enrich_now();
        app.queue_enrich_status_expanded();
        assert!(
            app.pending_enrich.is_empty(),
            "both sections stay settled once their outcomes land"
        );
    }

    #[test]
    fn ci_rollup_picks_the_worst_status_for_a_shared_commit() {
        use crate::ci::{CiRun, JobStatus, RunId};
        let (_fixture, mut app) = app();
        let run = |status| CiRun {
            id: RunId("x".into()),
            name: "CI".into(),
            title: String::new(),
            branch: "main".into(),
            commit: "abc123".into(),
            author: String::new(),
            created: None,
            status,
            url: None,
            remote: None,
        };
        app.runs = vec![run(JobStatus::Ok), run(JobStatus::Failed)];
        let indices = app.runs_for_commit("abc123");
        assert_eq!(app.ci_rollup(&indices), Some(JobStatus::Failed));
    }

    /// Two runs of one workflow started together and the forge cancelled the
    /// duplicate: the one that finished decides, so the commit reads green.
    /// A workflow whose only run was cancelled still says so.
    #[test]
    fn ci_rollup_lets_a_decided_run_outvote_its_cancelled_twin() {
        use crate::ci::{CiRun, JobStatus, RunId};
        let (_fixture, mut app) = app();
        let run = |name: &str, status| CiRun {
            id: RunId("x".into()),
            name: name.into(),
            title: String::new(),
            branch: "main".into(),
            commit: "abc123".into(),
            author: String::new(),
            created: None,
            status,
            url: None,
            remote: None,
        };
        app.runs = vec![
            run("Deploy - Dev", JobStatus::Neutral),
            run("Preview", JobStatus::Ok),
            run("Deploy - Dev", JobStatus::Ok),
            run("Deploy - Dev", JobStatus::Ok),
        ];
        let indices = app.runs_for_commit("abc123");
        assert_eq!(app.ci_rollup(&indices), Some(JobStatus::Ok));

        app.runs.push(run("Lint", JobStatus::Neutral));
        let indices = app.runs_for_commit("abc123");
        assert_eq!(
            app.ci_rollup(&indices),
            Some(JobStatus::Neutral),
            "a workflow that only ever got cancelled still reads as cancelled"
        );

        app.runs.push(run("Tests", JobStatus::Failed));
        let indices = app.runs_for_commit("abc123");
        assert_eq!(app.ci_rollup(&indices), Some(JobStatus::Failed));
    }

    #[test]
    fn ci_rollup_is_none_when_nothing_matches() {
        let (_fixture, app) = app();
        assert_eq!(app.ci_rollup(&app.runs_for_commit("missing")), None);
    }
}