speccoder_runtime/

status.rs

//! `speccoder report`: a feature's tasks and the evidence behind
//! each tick, read from what is durable and reviewable — `tasks.md` at the
//! tip of `speccoder/<feature>` and the trailers of its commits. Read-only:
//! no model, no worktree, no state directory.

use std::collections::BTreeMap;
use std::path::Path;

use speccoder_core::snapshot::{ParsedTasks, TaskState};
use speccoder_core::{
    CommandData, CommandResult, CommandStatus, ConvergenceRound, StatusReport, StatusTotals,
    TaskStatus, TraceRow,
};

use crate::git::Git;

/// Field and record separators for `git log` output.
const FS: char = '\u{1f}';
const RS: char = '\u{1e}';

/// What a speccoder commit says about a task.
#[derive(Debug, Clone)]
struct Evidence {
    commit: String,
    verdict: String,
    run: String,
    reason: String,
    /// The task's line, unticked, when the commit was made in place.
    task_line: String,
    /// The intent it was judged against, when made in place.
    intent: String,
    /// `Acceptance-Strength:`.
    strength: String,
    /// `Regression-Only:`.
    regression: String,
}

/// Reads the trailers speccoder writes (`Spec-Task`, `Verdict`, `Run`) and
/// its convergence commits from `log` (newest first): the latest commit per
/// task wins; rounds come back oldest first.
fn read_log(log: &str, feature: &str) -> (BTreeMap<String, Evidence>, Vec<ConvergenceRound>) {
    let mut evidence: BTreeMap<String, Evidence> = BTreeMap::new();
    let mut rounds = Vec::new();
    for record in log.split(RS) {
        let fields: Vec<&str> = record.trim_start_matches('\n').split(FS).collect();
        let [
            commit,
            subject,
            task,
            verdict,
            run,
            reason,
            task_line,
            intent,
            strength,
            regression,
            body,
        ] = fields[..]
        else {
            continue;
        };
        if let Some(task) = task.trim().strip_prefix(&format!("{feature}/")) {
            evidence.entry(task.to_owned()).or_insert(Evidence {
                commit: commit.to_owned(),
                verdict: verdict.trim().to_owned(),
                run: run.trim().to_owned(),
                reason: reason.trim().to_owned(),
                task_line: task_line.trim().to_owned(),
                intent: intent.trim().trim_start_matches("sha256:").to_owned(),
                strength: strength.trim().to_owned(),
                regression: regression.trim().to_owned(),
            });
        } else if let Some(n) = subject
            .strip_prefix("speccoder: converge round ")
            .and_then(|r| r.split(' ').next())
            .and_then(|n| n.parse::<u32>().ok())
        {
            rounds.push(ConvergenceRound {
                round: n,
                commit: commit.to_owned(),
                tasks: body
                    .lines()
                    .filter(|l| {
                        l.strip_prefix('T')
                            .is_some_and(|r| r.starts_with(|c: char| c.is_ascii_digit()))
                    })
                    .map(str::to_owned)
                    .collect(),
            });
        }
    }
    rounds.reverse();
    (evidence, rounds)
}

/// A regression-only tick's reason from its trailer; a tick without one
/// (made before reasons were recorded) reads `unrecorded`, never a guess.
#[must_use]
pub fn regression_of(trailer: &str) -> speccoder_core::regression::Regression {
    use speccoder_core::regression::{Regression, RegressionReason, parse_trailer};
    Regression::of(parse_trailer(trailer.trim()).unwrap_or(RegressionReason::Unrecorded))
}

/// Each task's state: evidence for ticked tasks, the schedule for open ones.
/// `lines` (the open-line hash per task, for local ticks) decides whether a
/// commit naming a `Task-Line` still matches the task as written.
fn statuses(
    parsed: &ParsedTasks,
    evidence: &BTreeMap<String, Evidence>,
    lines: &BTreeMap<String, String>,
    saved: &[crate::attempts::Saved],
    skipped: &BTreeMap<String, String>,
) -> Vec<TaskStatus> {
    let (schedule, _) = speccoder_core::graph::schedule(parsed);
    parsed
        .tasks
        .iter()
        .map(|t| {
            let scheduled = schedule.tasks.iter().find(|s| s.id == t.id);
            let found = evidence.get(&t.id).filter(|e| {
                e.task_line.is_empty() || lines.get(&t.id).is_none_or(|h| *h == e.task_line)
            });
            // In place: a commit for exactly this line, the tick not written.
            let untick = found.filter(|e| !t.done && !e.task_line.is_empty() && !lines.is_empty());
            let state = if untick.is_some() {
                "evidence_without_tick".to_owned()
            } else if t.done {
                match found.map(|e| e.verdict.as_str()) {
                    Some(v @ ("verified" | "regression_only" | "attested")) => v.to_owned(),
                    _ => "ticked_without_evidence".to_owned(),
                }
            } else if skipped.contains_key(&t.id) {
                "skipped".to_owned()
            } else if scheduled.is_some_and(|s| s.state == TaskState::Ready) {
                "ready".to_owned()
            } else {
                "blocked".to_owned()
            };
            TaskStatus {
                id: t.id.clone(),
                title: t.title.clone(),
                phase: t.phase.clone().unwrap_or_default(),
                blocked_by: if t.done {
                    Vec::new()
                } else {
                    scheduled.map(|s| s.blocked_by.clone()).unwrap_or_default()
                },
                commit: found
                    .filter(|_| t.done || untick.is_some())
                    .map(|e| e.commit.clone()),
                attestation: None,
                strength: found
                    .filter(|e| t.done && !e.strength.is_empty())
                    .map(|e| e.strength.clone()),
                drift: None,
                run_id: found
                    .filter(|e| t.done && !e.run.is_empty())
                    .map(|e| e.run.clone()),
                // Newest first: the first saved attempt of an open task.
                last_attempt: saved
                    .iter()
                    .find(|a| !t.done && a.task == t.id && !crate::controls::is_set_aside(a))
                    .map(|a| speccoder_core::LastAttempt {
                        outcome: a.outcome.clone(),
                        reason: a.reason.clone(),
                        run_id: a.run_id.clone(),
                        reference: a.reference.clone(),
                    }),
                set_aside: None,
                bug: None,
                regression: found
                    .filter(|e| t.done && e.verdict == "regression_only")
                    .map(|e| regression_of(&e.regression)),
                note: if t.done {
                    found.map(|e| e.reason.clone()).filter(|r| !r.is_empty())
                } else {
                    skipped.get(&t.id).cloned()
                },
                state,
            }
        })
        .collect()
}

fn totals(tasks: &[TaskStatus]) -> StatusTotals {
    let n = |s: &str| tasks.iter().filter(|t| t.state == s).count();
    StatusTotals {
        verified: n("verified"),
        regression_only: n("regression_only"),
        ticked_without_evidence: n("ticked_without_evidence"),
        evidence_without_tick: n("evidence_without_tick"),
        without_attestation: tasks
            .iter()
            .filter(|t| t.attestation.as_deref() == Some("missing"))
            .count(),
        attested: n("attested"),
        skipped: n("skipped"),
        ready: n("ready"),
        blocked: n("blocked"),
        stale: tasks
            .iter()
            .filter(|t| t.drift.as_ref().is_some_and(speccoder_core::Drift::stale))
            .count(),
    }
}

/// `speccoder report [--feature]`.
#[must_use]
pub fn report(path: &Path, feature: Option<&str>) -> CommandResult {
    let inspected = crate::inspect(path, feature);
    let Some(CommandData::Inspect(snapshot)) = inspected.data() else {
        return CommandResult::carried("report", &inspected);
    };
    let format = crate::format::of(snapshot);
    let Some(f) = snapshot.feature.as_ref() else {
        // A project with nothing planned yet is a beginning, not an error.
        if snapshot.features.is_empty()
            && !snapshot
                .diagnostics
                .iter()
                .any(|d| d.code == "select.not-found")
        {
            let unit = format.messages().unit;
            let create = (format.planning().creates.first())
                .or((format.planning().toward_tasks)(false).first())
                .map_or_else(
                    || "speccoder spec".to_owned(),
                    |s| format!("speccoder spec {s} \"…\""),
                );
            return CommandResult::new(
                "report",
                CommandStatus::Ok,
                &format!("no {unit}s yet — `{create}` writes the first one"),
            )
            .with_next(vec![speccoder_core::next::NextStep {
                command: create,
                why: format!("describe the {unit}; its spec is written for you to review"),
            }]);
        }
        // Several and none current, or a pointer to one that is gone: say
        // which exist; the code's fixes say how to pick.
        let unit = format.messages().unit;
        let names: Vec<&str> = snapshot.features.iter().map(|f| f.name.as_str()).collect();
        if let Some(d) = snapshot
            .diagnostics
            .iter()
            .find(|d| d.code == "select.not-found")
        {
            return CommandResult::new(
                "report",
                CommandStatus::SelectionNeeded,
                &match &d.hint {
                    Some(h) => format!("{}; {h}", d.message),
                    None => d.message.clone(),
                },
            )
            // Named here, or remembered by the format's pointer.
            .with_code(if feature.is_some() {
                "FEATURE-UNKNOWN"
            } else {
                "FEATURE-GONE"
            });
        }
        return CommandResult::new(
            "report",
            CommandStatus::SelectionNeeded,
            &format!(
                "{} {unit}s and none is current ({})",
                names.len(),
                names.join(", ")
            ),
        );
    };
    let artifacts: Vec<String> = [
        ("spec", f.artifacts.spec),
        ("plan", f.artifacts.plan),
        ("tasks", f.artifacts.tasks),
    ]
    .into_iter()
    .filter(|(_, on)| *on)
    .map(|(n, _)| n.to_owned())
    .collect();
    let uncommitted = crate::hosting::uncommitted(path, feature);
    // No task list yet: the feature is being planned — a stage, not an
    // error. The report says what exists and which planning skills are next.
    let Some(checkout_tasks) = f.tasks.as_ref() else {
        let planning: Vec<String> = (format.planning().toward_tasks)(f.artifacts.plan)
            .iter()
            .map(|s| (*s).to_owned())
            .collect();
        return CommandResult::new(
            "report",
            CommandStatus::Ok,
            &format!(
                "{} is being planned: {} written, no {} yet",
                f.feature.name,
                if artifacts.is_empty() {
                    "nothing".to_owned()
                } else {
                    speccoder_core::words::listed(&artifacts)
                },
                format.messages().tasks_file
            ),
        )
        .with_data(CommandData::Report(Box::new(StatusReport {
            review: crate::review::totals_of(
                Path::new(&snapshot.root),
                format,
                &f.feature.name,
                &f.feature.dir,
            ),
            issues: issues_of(path, Some(&f.feature.name)),
            feature: f.feature.name.clone(),
            branch: None,
            source: "checkout".to_owned(),
            selected_by: crate::current::reason_text(snapshot.selection.reason, format),
            journey: journey_of(&artifacts, &uncommitted, 0, 0),
            artifacts,
            planning,
            uncommitted,
            unmerged_commits: 0,
            tasks: Vec::new(),
            convergence: Vec::new(),
            trace: Vec::new(),
            totals: speccoder_core::StatusTotals::default(),
        })));
    };
    let name = f.feature.name.clone();
    let branch = format!("speccoder/{name}");
    let root = Path::new(&snapshot.root);
    let git = Git::at(root);
    let has_branch = git
        .succeeds(&[
            "rev-parse",
            "--verify",
            "-q",
            &format!("refs/heads/{branch}"),
        ])
        .unwrap_or(false);

    let mut prefix = String::new();
    let (parsed, source, log, unmerged) = if has_branch {
        // tasks.md as committed at the branch tip, by its repository path.
        let top = git.toplevel().ok().flatten();
        let canonical = |p: &Path| std::fs::canonicalize(p).unwrap_or_else(|_| p.to_path_buf());
        let rel = top
            .as_deref()
            .and_then(|t| {
                canonical(root)
                    .strip_prefix(canonical(t))
                    .ok()
                    .map(Path::to_path_buf)
            })
            .unwrap_or_default();
        prefix = rel.to_string_lossy().replace('\\', "/");
        if !prefix.is_empty() {
            prefix.push('/');
        }
        let repo_path = rel
            .join(&checkout_tasks.path)
            .to_string_lossy()
            .replace('\\', "/");
        let text = git
            .run(&["show", &format!("{branch}:{repo_path}")])
            .map(|b| String::from_utf8_lossy(&b).into_owned());
        // In place, an ignored task list is never committed: its ticks are
        // the checkout's, checked against the commits' `Task-Line`.
        let local = text.is_err()
            && !git
                .succeeds(&["ls-files", "--error-unmatch", "--", &repo_path])
                .unwrap_or(false);
        let parsed = match text {
            Ok(text) => format.parse_tasks(&checkout_tasks.path, &text).0,
            Err(_) if local => checkout_tasks.clone(),
            Err(_) => {
                return CommandResult::new(
                    "report",
                    CommandStatus::GitPrecondition,
                    &format!("{branch} has no {repo_path}"),
                );
            }
        };
        let format = format!(
            "--format=%H{FS}%s{FS}%(trailers:key=Spec-Task,valueonly){FS}%(trailers:key=Verdict,valueonly){FS}%(trailers:key=Run,valueonly){FS}%(trailers:key=Reason,valueonly){FS}%(trailers:key=Task-Line,valueonly){FS}%(trailers:key=Intent-Digest,valueonly){FS}%(trailers:key=Acceptance-Strength,valueonly){FS}%(trailers:key=Regression-Only,valueonly){FS}%b{RS}"
        );
        let log = git
            .run(&["log", &format, &branch])
            .map(|b| String::from_utf8_lossy(&b).into_owned())
            .unwrap_or_default();
        let unmerged = git
            .line(&["rev-list", "--count", &format!("HEAD..{branch}")])
            .ok()
            .and_then(|n| n.trim().parse().ok())
            .unwrap_or(0);
        (
            parsed,
            if local { "checkout" } else { "branch" },
            log,
            unmerged,
        )
    } else {
        (checkout_tasks.clone(), "checkout", String::new(), 0)
    };
    let local_ticks = has_branch && source == "checkout";
    let lines = if local_ticks {
        open_lines(&root.join(&checkout_tasks.path), &parsed)
    } else {
        BTreeMap::new()
    };
    let (evidence, convergence) = read_log(&log, &name);
    let saved = crate::attempts::list(root, &name);
    let skipped = crate::controls::skipped(root, &name);
    let mut tasks = statuses(&parsed, &evidence, &lines, &saved, &skipped);
    // Changes a run set aside, newest first: the person decides on them,
    // whatever became of the task.
    for a in saved.iter().filter(|a| crate::controls::is_set_aside(a)) {
        if let Some(t) = tasks
            .iter_mut()
            .find(|t| t.id == a.task && t.set_aside.is_none())
        {
            t.set_aside = Some(speccoder_core::SetAside {
                reference: a.reference.clone(),
                files: crate::controls::changed_files(root, &a.commit),
            });
        }
    }
    // What each committed tick's note says (signatures are `attest verify`'s).
    for t in tasks.iter_mut() {
        if let Some(commit) = t.commit.as_deref() {
            t.attestation = Some(crate::attest::note_state(root, commit));
        }
    }
    if !crate::drift::mark(root, format, &f.feature.dir, &prefix, &mut tasks) && local_ticks {
        // The intent is not in git (in place, ignored): the commits'
        // `Intent-Digest` says whether it changed, not where.
        let now = crate::in_place::intent_digest(
            root,
            &crate::tools::policy::Policy::new(&[], &[]).rules(format.rules()),
            format.messages().tasks_file,
        );
        for t in &mut tasks {
            let judged = evidence.get(&t.id).map(|e| e.intent.as_str());
            if t.commit.is_some() && judged.is_some_and(|i| !i.is_empty() && i != now) {
                t.drift = Some(speccoder_core::Drift {
                    requirements: Vec::new(),
                    files: Vec::new(),
                });
            }
        }
    }
    let trace = trace(&format.requirement_ids(root, &f.feature.dir), &tasks);
    let totals = totals(&tasks);
    let finished = totals.verified
        + totals.regression_only
        + totals.attested
        + totals.ticked_without_evidence
        + totals.skipped;
    let done =
        totals.verified + totals.regression_only + totals.attested + totals.ticked_without_evidence;
    // Only what there is: a first report says "0/11 done: 3 ready, 8
    // blocked", not four kinds of evidence nothing has yet.
    let parts: Vec<String> = [
        (totals.verified, "verified"),
        (totals.regression_only, "regression only"),
        (totals.attested, "attested"),
        (totals.ticked_without_evidence, "without evidence"),
    ]
    .iter()
    .filter(|(n, _)| *n > 0)
    .map(|(n, w)| format!("{n} {w}"))
    .collect();
    let open: Vec<String> = [
        (totals.ready, "ready"),
        (totals.blocked, "blocked"),
        (totals.skipped, "skipped"),
    ]
    .iter()
    .filter(|(n, _)| *n > 0)
    .map(|(n, w)| format!("{n} {w}"))
    .collect();
    // A task list without a task: nothing is done, the tasks are to write.
    let mut message = if tasks.is_empty() {
        format!(
            "no tasks yet: {} lists none; {}",
            format.messages().tasks_file,
            format.messages().make_tasks
        )
    } else {
        format!("{done}/{} done", tasks.len())
    };
    let planning: Vec<String> = if tasks.is_empty() {
        (format.planning().toward_tasks)(f.artifacts.plan)
            .iter()
            .map(|s| (*s).to_owned())
            .collect()
    } else {
        Vec::new()
    };
    if !parts.is_empty() {
        message.push_str(&format!(": {}", parts.join(", ")));
    }
    if !open.is_empty() {
        message.push_str(&format!(
            "{}{}",
            if parts.is_empty() { ": " } else { "; " },
            open.join(", ")
        ));
    }
    if !convergence.is_empty() {
        message.push_str(&format!("; {} convergence round(s)", convergence.len()));
    }
    let uncited = trace.iter().filter(|r| r.task.is_none()).count();
    if uncited > 0 {
        message.push_str(&format!("; {uncited} requirement(s) no task cites"));
    }
    if totals.without_attestation > 0 {
        message.push_str(&format!(
            "; {} tick(s) without an attestation",
            totals.without_attestation
        ));
    }
    if totals.evidence_without_tick > 0 {
        message.push_str(&format!(
            "; {} committed without their local tick (the next `run --in-place` ticks them)",
            totals.evidence_without_tick
        ));
    }
    if totals.stale > 0 {
        message.push_str(&format!(
            "; {} tick(s) proven against an earlier wording of their requirements",
            totals.stale
        ));
    }
    let drifted = tasks.iter().filter(|t| t.drift.is_some()).count() - totals.stale;
    if drifted > 0 {
        message.push_str(&format!(
            "; the spec changed since {drifted}{} tick(s), not in what they cite",
            if totals.stale > 0 { " other" } else { "" }
        ));
    }
    if local_ticks {
        message.push_str(&format!(
            "; ticks are local to this checkout ({} is not in git)",
            format.messages().tasks_file
        ));
    }
    if has_branch {
        if unmerged > 0 {
            message.push_str(&format!("; {unmerged} commit(s) on {branch} not merged"));
        }
    } else {
        message.push_str("; speccoder has not worked on this feature yet");
    }
    // A bug task's outcome, from what speccoder keeps of the bug.
    if let Some(state) = crate::worktree::state_dir() {
        let plane = crate::worktree::project_state_dir(&state, root);
        for t in &mut tasks {
            t.bug = crate::bug::state::load(&plane, &name, &t.id)
                .map(|b| b.outcome.as_str().to_owned());
        }
    }
    CommandResult::new("report", CommandStatus::Ok, &message).with_data(CommandData::Report(
        Box::new(StatusReport {
            review: crate::review::totals_of(root, format, &name, &f.feature.dir),
            issues: issues_of(root, Some(&name)),
            feature: name,
            branch: has_branch.then_some(branch),
            source: source.to_owned(),
            selected_by: crate::current::reason_text(snapshot.selection.reason, format),
            journey: journey_of(&artifacts, &uncommitted, tasks.len(), finished),
            artifacts,
            planning,
            uncommitted,
            unmerged_commits: unmerged,
            tasks,
            convergence,
            trace,
            totals,
        }),
    ))
}

/// `speccoder task show TASK`: the report narrowed to one task.
#[must_use]
pub fn show(path: &Path, feature: Option<&str>, task: &str) -> CommandResult {
    let full = report(path, feature);
    let Some(CommandData::Report(r)) = full.data() else {
        return CommandResult::carried("task show", &full);
    };
    let Some(t) = r.tasks.iter().find(|t| t.id == task) else {
        return CommandResult::new(
            "task show",
            CommandStatus::NotFound,
            &format!("{} has no task {task}", r.feature),
        )
        .with_code("NOT-FOUND")
        .with_fix(vec![format!("speccoder report --feature {}", r.feature)]);
    };
    let mut one = (**r).clone();
    one.tasks = vec![t.clone()];
    one.trace.retain(|row| row.task.as_deref() == Some(task));
    one.convergence.clear();
    one.totals = totals(&one.tasks);
    let message = format!("{task}: {}", t.state.replace('_', " "));
    CommandResult::ok("task show", &message).with_data(CommandData::Report(Box::new(one)))
}

/// One row per requirement and task citing it (by its identifier as a word
/// in the title); a requirement no task cites is a row of its own.
fn trace(requirements: &[(String, String)], tasks: &[TaskStatus]) -> Vec<TraceRow> {
    requirements
        .iter()
        .flat_map(|(id, _)| {
            let rows: Vec<TraceRow> = tasks
                .iter()
                .filter(|t| crate::attest::cites(&t.title, id))
                .map(|t| TraceRow {
                    requirement: id.clone(),
                    scenario: None,
                    test: None,
                    task: Some(t.id.clone()),
                    verdict: Some(t.state.clone()),
                    changed: t
                        .drift
                        .as_ref()
                        .is_some_and(|d| d.requirements.iter().any(|r| r.id == *id)),
                })
                .collect();
            if rows.is_empty() {
                vec![TraceRow {
                    requirement: id.clone(),
                    scenario: None,
                    test: None,
                    task: None,
                    verdict: None,
                    changed: false,
                }]
            } else {
                rows
            }
        })
        .collect()
}

/// Each task's line with its box read as open, hashed: what an in-place
/// commit's `Task-Line` names.
fn open_lines(file: &Path, parsed: &ParsedTasks) -> BTreeMap<String, String> {
    let text = std::fs::read_to_string(file).unwrap_or_default();
    let lines: Vec<&str> = text.lines().collect();
    parsed
        .tasks
        .iter()
        .filter_map(|t| {
            let line = lines.get(usize::try_from(t.line).ok()?.checked_sub(1)?)?;
            let open = crate::in_place::untick(line.as_bytes());
            Some((t.id.clone(), crate::integrity::sha256_hex(&open)))
        })
        .collect()
}

fn journey_of(
    artifacts: &[String],
    uncommitted: &[String],
    total: usize,
    finished: usize,
) -> Vec<speccoder_core::actions::JourneyStep> {
    let has = |a: &str| artifacts.iter().any(|x| x == a);
    speccoder_core::actions::journey(speccoder_core::actions::JourneyFacts {
        spec: has("spec"),
        plan: has("plan"),
        tasks: has("tasks"),
        uncommitted: !uncommitted.is_empty(),
        total,
        finished,
    })
}

/// The GitHub issue addresses the feature's documents name.
fn issues_of(root: &Path, feature: Option<&str>) -> Vec<String> {
    let mut out: Vec<String> = Vec::new();
    for (_, _, text) in crate::documents::sources(root, feature) {
        for url in crate::github::issue_urls(&text) {
            if !out.contains(&url) {
                out.push(url);
            }
        }
    }
    out
}

#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn a_requirement_no_task_cites_is_a_row_of_its_own() {
        let task = |id: &str, title: &str, state: &str| TaskStatus {
            id: id.to_owned(),
            title: title.to_owned(),
            phase: "Phase 3".to_owned(),
            state: state.to_owned(),
            drift: None,
            blocked_by: Vec::new(),
            commit: None,
            run_id: None,
            last_attempt: None,
            set_aside: None,
            bug: None,
            regression: None,
            note: None,
            attestation: None,
            strength: None,
        };
        let ids = |v: &[&str]| {
            v.iter()
                .map(|i| ((*i).to_owned(), String::new()))
                .collect::<Vec<_>>()
        };
        let tasks = [
            task("T001", "Parse the config (FR-001, FR-002)", "verified"),
            task("T002", "Render it per FR-002", "ready"),
            task("T003", "Unrelated FR-0010 work", "ready"),
        ];
        let rows = trace(&ids(&["FR-001", "FR-002", "FR-003"]), &tasks);
        let shown: Vec<_> = rows
            .iter()
            .map(|r| {
                (
                    r.requirement.as_str(),
                    r.task.as_deref(),
                    r.verdict.as_deref(),
                )
            })
            .collect();
        assert_eq!(
            shown,
            [
                ("FR-001", Some("T001"), Some("verified")),
                ("FR-002", Some("T001"), Some("verified")),
                ("FR-002", Some("T002"), Some("ready")),
                ("FR-003", None, None),
            ]
        );
    }

    #[test]
    fn trailers_and_rounds_are_read_newest_first() {
        let rec = |c: &str, s: &str, t: &str, v: &str, r: &str, b: &str| {
            format!("{c}{FS}{s}{FS}{t}\n{FS}{v}\n{FS}{r}\n{FS}{FS}{FS}{FS}{FS}{FS}{b}{RS}\n")
        };
        let log = [
            rec(
                "c4",
                "speccoder: T003 again",
                "f/T003",
                "verified",
                "run_b",
                "",
            ),
            rec(
                "c3",
                "speccoder: converge round 1 (1 task)",
                "",
                "",
                "",
                "T003 Fix it per FR-1 (missing)\n\nSpec-Feature: f\nRun: run_c\n",
            ),
            rec(
                "c2",
                "speccoder: T003 first",
                "f/T003",
                "regression_only",
                "run_a",
                "",
            ),
            rec("c1", "init", "", "", "", ""),
            rec("c0", "other feature", "g/T001", "verified", "x", ""),
        ]
        .concat();
        let (ev, rounds) = read_log(&log, "f");
        assert_eq!(ev.len(), 1);
        assert_eq!(
            (ev["T003"].commit.as_str(), ev["T003"].verdict.as_str()),
            ("c4", "verified")
        );
        assert_eq!(rounds.len(), 1);
        assert_eq!(rounds[0].tasks, ["T003 Fix it per FR-1 (missing)"]);
    }
}