release_kit/

assess.rs

//! Classify a target repository before anything lands.
//!
//! The assessment is read-only evidence plus one classification computed
//! from it by an explicit rule: `greenfield` when the target carries no
//! release mechanism and no release history, `brownfield` when a release
//! mechanism is already in place — a tool's configuration, a landed
//! destination, a landed block — and `needs-decision` when the target
//! shows release activity that no recognized mechanism explains: tags
//! with no tool behind them, or a second long-lived branch. The rule
//! lives here so a routing skill reads a verdict it can cite instead of
//! judging "some release setup" by feel. The gathering spawns git and
//! reads the disk; the rule itself is pure and unit-tested.

use std::process::Command;

use camino::Utf8Path;
use serde::Serialize;

use crate::diagnostic::{Diagnostic, Reason};
use crate::error::RkError;
use crate::landing::{self, manifest};

// The trunk and the release-line prefix come from the target's own
// committed configuration; a target that states neither keeps the
// compiled defaults in `crate::config`.

/// Files that mark a release mechanism, whichever tool owns it, beside
/// the configuration names of semantic-release and `GoReleaser`.
///
/// release-kit's own destinations are judged separately, as collisions;
/// this list is what other tools leave behind: release-plz's dotted form,
/// the workflow names a hand-rolled publish commonly takes, and a
/// changelog. The configuration names of semantic-release and
/// `GoReleaser` come from the migration inventory, which owns them,
/// through [`marker_names`]. `package.json` joins the list only when it
/// carries the top-level `release` key semantic-release reads, judged in
/// [`gather`].
pub const RELEASE_MARKERS: [&str; 7] = [
    ".release-plz.toml",
    ".github/workflows/publish.yml",
    ".github/workflows/publish.yaml",
    ".github/workflows/release.yaml",
    ".github/workflows/release-drafter.yml",
    "CHANGELOG.md",
    "CHANGES.md",
];

/// Every file name that marks a release mechanism: [`RELEASE_MARKERS`]
/// and the configuration names of semantic-release and `GoReleaser` that
/// the migration inventory owns.
pub fn marker_names() -> impl Iterator<Item = &'static str> {
    use crate::migrate::inventory::{GORELEASER, SEMANTIC_RELEASE, release_configs};
    RELEASE_MARKERS
        .into_iter()
        .chain(release_configs(SEMANTIC_RELEASE))
        .chain(release_configs(GORELEASER))
}

/// Branch names that conventionally outlive a topic.
///
/// A second one beside the trunk is the retired two-branch flow, or a
/// trunk under another name, and either is a migration step. A
/// `release/<line>` branch — the convention's own long-lived form — is
/// recognized by its prefix.
pub const LONG_LIVED_BRANCHES: [&str; 11] = [
    "master",
    "main",
    "trunk",
    "develop",
    "development",
    "dev",
    "staging",
    "next",
    "release",
    "production",
    "prod",
];

/// What the target is, for routing: the corpus verdict, computed from the
/// repository's evidence alone.
#[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize)]
#[serde(rename_all = "kebab-case")]
pub enum Classification {
    /// No release mechanism and no release history: land the workflow.
    Greenfield,
    /// A release mechanism is in place: migrate, never land beside it.
    Brownfield,
    /// Release activity no mechanism explains: the operator decides.
    NeedsDecision,
}

impl Classification {
    /// The kebab-case verdict word, as the JSON serializes it.
    #[must_use]
    pub const fn as_str(self) -> &'static str {
        match self {
            Self::Greenfield => "greenfield",
            Self::Brownfield => "brownfield",
            Self::NeedsDecision => "needs-decision",
        }
    }
}

/// The landing record's presence, the one fact `rk status` owns that the
/// routing needs before it reads the full report.
#[derive(Debug, Serialize)]
pub struct Landing {
    /// Whether `.release-kit/manifest.json` exists and reads.
    pub recorded: bool,
    /// The release-kit version the record names, where one exists.
    #[serde(skip_serializing_if = "Option::is_none")]
    pub rk_version: Option<String>,
}

/// The evidence the classification is computed from.
#[derive(Debug, Serialize)]
pub struct Evidence {
    /// The landing record, present or not.
    pub landing: Landing,
    /// The technologies the version files name, zero or many.
    pub technologies: Vec<&'static str>,
    /// The forge the origin remote maps to, where one is recognized.
    #[serde(skip_serializing_if = "Option::is_none")]
    pub forge: Option<&'static str>,
    /// The project path from the origin remote, where one exists.
    #[serde(skip_serializing_if = "Option::is_none")]
    pub repo: Option<String>,
    /// Release-mechanism files of other tools found at the target.
    pub release_markers: Vec<String>,
    /// Landable destinations already present: a whole file that exists, or
    /// a block destination whose marked block is present.
    pub collisions: Vec<String>,
    /// Whether the target is a git repository the evidence below reads.
    pub git: bool,
    /// How many tags the repository holds.
    pub tags: usize,
    /// Long-lived branches found besides the trunk, local or remote.
    pub long_lived_branches: Vec<String>,
}

/// Compute the verdict from the evidence. Pure, so the rule is testable
/// without a repository.
#[must_use]
pub const fn classify(evidence: &Evidence) -> Classification {
    verdict(
        evidence.release_markers.as_slice(),
        evidence.collisions.as_slice(),
        evidence.tags,
        evidence.long_lived_branches.as_slice(),
    )
}

/// The one rule both readers apply: a mechanism in place outranks
/// unexplained activity, and nothing at all is a fresh start.
const fn verdict(
    release_markers: &[String],
    collisions: &[String],
    tags: usize,
    long_lived_branches: &[String],
) -> Classification {
    if !release_markers.is_empty() || !collisions.is_empty() {
        return Classification::Brownfield;
    }
    if tags > 0 || !long_lived_branches.is_empty() {
        return Classification::NeedsDecision;
    }
    Classification::Greenfield
}

/// What marks release-kit's own landing at a target that carries no
/// record: the evidence the adoption detector reads.
#[derive(Debug, Default, Serialize)]
pub struct OwnLanding {
    /// Block destinations whose release-kit marked region is present.
    pub regions: Vec<String>,
    /// Whole-file rendered destinations holding exactly the bytes this
    /// binary renders for them at this target.
    pub rendered: Vec<String>,
}

impl OwnLanding {
    /// Whether any of release-kit's own landing is present.
    #[must_use]
    pub const fn present(&self) -> bool {
        !self.regions.is_empty() || !self.rendered.is_empty()
    }
}

/// Where a target with no record stands, by the one rule above and the
/// adoption detector before it.
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub enum Unrecorded {
    /// release-kit's own landing is present and only the record is
    /// missing.
    Adopt,
    /// Everything else, by the assessment's verdict.
    Assessed(Classification),
}

/// Classify a target with no record.
///
/// Pure, so the precedence is testable without a repository:
/// release-kit's own landing outranks every other mechanism, because its
/// marked regions and its rendered files are what a landing made before
/// the record existed leaves behind, and they would otherwise read as
/// another tool's collisions.
#[must_use]
pub const fn classify_unrecorded(facts: &Facts, own: &OwnLanding) -> Unrecorded {
    if own.present() {
        return Unrecorded::Adopt;
    }
    Unrecorded::Assessed(verdict(
        facts.release_markers.as_slice(),
        facts.collisions.as_slice(),
        facts.tags,
        facts.long_lived_branches.as_slice(),
    ))
}

/// Where a target with no record stands, by the precedence `rk status`
/// reads it with.
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub enum Standing {
    /// A migration is open: its archive emptied the target on purpose,
    /// and this outranks every other reading until it closes.
    Migrating,
    /// No migration is open, so the adoption detector and the rule.
    Unrecorded(Unrecorded),
}

/// Read where a target with no record stands: an open migration first,
/// then the adoption detector and the assessment's rule.
///
/// # Errors
///
/// [`RkError::Io`] for a read of the target that fails for a reason
/// other than absence.
pub fn standing(
    target: &Utf8Path,
    config: Option<&crate::config::Config>,
) -> Result<Standing, RkError> {
    if config.is_some_and(|config| config.migration.is_some()) {
        return Ok(Standing::Migrating);
    }
    let facts = gather_facts(target)?;
    let own = own_landing(target, config)?;
    Ok(Standing::Unrecorded(classify_unrecorded(&facts, &own)))
}

/// Find release-kit's own landing at `target`, reading and never
/// writing.
///
/// A marked region carries release-kit's own name, so its presence
/// alone is the evidence. A whole file is release-kit's only where it
/// holds exactly the bytes this binary renders for it here, because a
/// file at the same path from another tool is a migration's business.
/// Where this binary cannot resolve the parameters a render needs, the
/// rendered half reads as empty rather than failing: the regions still
/// answer.
///
/// # Errors
///
/// [`RkError::Io`] for a read that fails for a reason other than
/// absence.
pub fn own_landing(
    target: &Utf8Path,
    config: Option<&crate::config::Config>,
) -> Result<OwnLanding, RkError> {
    let mut own = OwnLanding::default();
    for destination in landing::BLOCK_DESTINATIONS {
        if landing::read_recorded(target, destination)?.is_some() {
            own.regions.push(destination.to_owned());
        }
    }
    let params = landing::Params::resolve(
        target,
        &landing::Inputs::default(),
        config,
        None,
        landing::Purpose::Preview,
    );
    let Ok(params) = params else {
        return Ok(own);
    };
    let Ok(evidence) = crate::projection::TargetEvidence::gather(target, None) else {
        return Ok(own);
    };
    let Ok(projection) =
        crate::projection::Projection::compute(&crate::projection::ProjectionInput {
            params,
            evidence,
        })
    else {
        return Ok(own);
    };
    for candidate in &projection.candidates {
        // A skill copy is no evidence of a landing: an agent's own skill
        // installer can write the same bytes into a project.
        if candidate.kind != landing::Kind::Rendered
            || candidate.region.is_some()
            || crate::projection::skill_folder(&candidate.destination).is_some()
        {
            continue;
        }
        if landing::read_recorded(target, &candidate.destination)?.as_deref()
            == Some(candidate.bytes.as_slice())
        {
            own.rendered.push(candidate.destination.clone());
        }
    }
    own.rendered.sort();
    Ok(own)
}

/// Whether `target` sits in a non-bare Git work tree.
///
/// A directory Git reports as no repository, and a bare repository,
/// answer `false`: both are facts the setup skill stops on, because the
/// method lands files a work tree commits.
///
/// # Errors
///
/// [`RkError::Subprocess`] where git does not run or refuses for a
/// reason other than the target not being a repository.
pub fn git_work_tree(target: &Utf8Path) -> Result<bool, RkError> {
    match git_lines(target, &["rev-parse", "--is-inside-work-tree"]) {
        Ok(lines) => Ok(lines.first().is_some_and(|line| line == "true")),
        Err(GitFailure::NotARepository) => Ok(false),
        Err(GitFailure::Other(error)) => Err(error),
    }
}

/// The repository's facts alone, with no record read: what the planner
/// gathers beside its own read of the record.
#[derive(Debug)]
pub struct Facts {
    /// The technologies the version files name, zero or many.
    pub technologies: Vec<&'static str>,
    /// The forge the origin remote maps to, where one is recognized.
    pub forge: Option<&'static str>,
    /// The project path from the origin remote, where one exists.
    pub repo: Option<String>,
    /// Release-mechanism files of other tools found at the target.
    pub release_markers: Vec<String>,
    /// Landable destinations already present.
    pub collisions: Vec<String>,
    /// Whether the target is a git repository.
    pub git: bool,
    /// How many tags the repository holds.
    pub tags: usize,
    /// Long-lived branches found besides the trunk.
    pub long_lived_branches: Vec<String>,
}

/// Gather the evidence at `target`, reading and never writing.
///
/// # Errors
///
/// Returns the record's own failure taxonomy for an unreadable or unknown
/// landing record — a broken record must not silently classify —
/// [`RkError::Io`] for a disk read that fails for a reason other than
/// absence, and [`RkError::Subprocess`] where git runs but cannot answer
/// for a repository, because an observation that cannot be read is not a
/// pass and must never read as an absent release history.
pub fn gather(target: &Utf8Path) -> Result<Evidence, RkError> {
    let record = manifest::load(target)?;
    let landing = Landing {
        recorded: record.is_some(),
        rk_version: record.map(|manifest| manifest.rk_version),
    };
    let facts = gather_facts(target)?;
    Ok(Evidence {
        landing,
        technologies: facts.technologies,
        forge: facts.forge,
        repo: facts.repo,
        release_markers: facts.release_markers,
        collisions: facts.collisions,
        git: facts.git,
        tags: facts.tags,
        long_lived_branches: facts.long_lived_branches,
    })
}

/// Gather the repository's facts at `target`, the record aside.
///
/// # Errors
///
/// [`RkError::Io`] for a disk read that fails for a reason other than
/// absence, and [`RkError::Subprocess`] where git runs but cannot answer
/// for a repository.
pub fn gather_facts(target: &Utf8Path) -> Result<Facts, RkError> {
    let detected = crate::detect::detect(target.as_std_path());
    let mut release_markers: Vec<String> = marker_names()
        .filter(|marker| target.join(marker).is_file())
        .map(str::to_owned)
        .collect();
    if package_json_names_a_release(target)? {
        release_markers.push("package.json".to_owned());
    }
    release_markers.sort();
    let mut collisions = Vec::new();
    for destination in landing::destinations() {
        if landing::read_recorded(target, destination)?.is_some() {
            collisions.push(destination.to_owned());
        }
    }
    collisions.sort();
    let (git, tags, long_lived_branches) = git_evidence(target)?;
    Ok(Facts {
        technologies: crate::detect::technologies_of(target.as_std_path()),
        forge: detected.forge.map(crate::detect::Forge::as_str),
        repo: detected.repo,
        release_markers,
        collisions,
        git,
        tags,
        long_lived_branches,
    })
}

/// Whether `package.json` carries the top-level `release` key
/// semantic-release reads its configuration from. An ordinary Node
/// project's manifest is not a release marker; only that key is.
fn package_json_names_a_release(target: &Utf8Path) -> Result<bool, RkError> {
    let path = target.join("package.json");
    let bytes = match std::fs::read(&path) {
        Ok(bytes) => bytes,
        Err(e) if e.kind() == std::io::ErrorKind::NotFound => return Ok(false),
        Err(e) => return Err(RkError::Io(e)),
    };
    // A manifest that does not parse is not evidence of a release
    // mechanism; the tool that would read it fails on it too.
    Ok(serde_json::from_slice::<serde_json::Value>(&bytes)
        .ok()
        .and_then(|value| value.get("release").map(|_| ()))
        .is_some())
}

/// The git-borne evidence: whether the target is a repository, how many
/// tags it holds, and which long-lived branches stand beside the trunk.
///
/// A directory git positively reports as no repository answers `false`
/// and empty — an observation, never a failure, because a plain
/// directory is a legitimate greenfield. Every other refusal — a
/// corrupt repository, an ownership refusal, a git that does not run —
/// is an error, because an unreadable history must not read as none.
fn git_evidence(target: &Utf8Path) -> Result<(bool, usize, Vec<String>), RkError> {
    let trunk = crate::config::trunk_of(target.as_std_path())?;
    let line_prefix = crate::config::line_prefix_of(target.as_std_path())?;
    let (trunk, line_prefix) = (trunk.as_str(), line_prefix.as_str());
    match git_lines(target, &["rev-parse", "--git-dir"]) {
        Ok(_) => {}
        Err(GitFailure::NotARepository) => return Ok((false, 0, Vec::new())),
        Err(GitFailure::Other(error)) => return Err(error),
    }
    let tags = git_lines(target, &["tag", "--list"]).map_err(GitFailure::into_error)?;
    let refs = git_lines(
        target,
        &[
            "for-each-ref",
            "--format=%(refname)",
            "refs/heads",
            "refs/remotes",
        ],
    )
    .map_err(GitFailure::into_error)?;
    Ok((
        true,
        tags.len(),
        long_lived_among(&refs, trunk, line_prefix),
    ))
}

/// The long-lived branch names among `refs`, given as full ref names.
///
/// `refs/heads/<name>` keeps its whole name, `refs/remotes/<remote>/<name>`
/// drops the remote alone, and a remote `HEAD` pointer is skipped. A
/// name is long-lived when it is a catalog entry other than the trunk or
/// carries the release-line prefix; each appears once, sorted.
#[must_use]
pub fn long_lived_among(refs: &[String], trunk: &str, line_prefix: &str) -> Vec<String> {
    let mut names = std::collections::BTreeSet::new();
    for reference in refs {
        let name = if let Some(local) = reference.strip_prefix("refs/heads/") {
            local
        } else if let Some(remote) = reference.strip_prefix("refs/remotes/") {
            match remote.split_once('/') {
                Some((_, "HEAD")) | None => continue,
                Some((_, name)) => name,
            }
        } else {
            continue;
        };
        let catalogued = name != trunk && LONG_LIVED_BRANCHES.contains(&name);
        if catalogued || name.starts_with(line_prefix) {
            names.insert(name.to_owned());
        }
    }
    names.into_iter().collect()
}

/// Why one git call gave no answer.
enum GitFailure {
    /// Git ran and said the target is not a repository.
    NotARepository,
    /// Git did not run, or ran and refused for another reason.
    Other(RkError),
}

impl GitFailure {
    /// After the target is known to be a repository, every failure is
    /// the same kind: a history that cannot be read.
    fn into_error(self) -> RkError {
        match self {
            Self::NotARepository => RkError::subprocess(
                Diagnostic::new(
                    Reason::SubprocessFailed,
                    "git stopped answering for a repository it had just recognized",
                )
                .expected("a readable repository"),
            ),
            Self::Other(error) => error,
        }
    }
}

/// The non-empty stdout lines of one git call.
///
/// The call answers for the `-C` target alone: the variables a running
/// hook exports are scrubbed, so an inherited `GIT_DIR` cannot redirect
/// the probe at another repository, and the locale is pinned to `C`, so
/// the one diagnostic this module reads — git's own "not a git
/// repository" — arrives untranslated.
fn git_lines(target: &Utf8Path, args: &[&str]) -> Result<Vec<String>, GitFailure> {
    let mut command = Command::new(crate::probes::git_bin());
    for var in crate::maintenance::GIT_HOOK_VARS {
        command.env_remove(var);
    }
    let out = command
        .env("LC_ALL", "C")
        .env_remove("LANGUAGE")
        .arg("-C")
        .arg(target)
        .args(args)
        .output()
        .map_err(|error| {
            GitFailure::Other(RkError::subprocess(
                Diagnostic::new(
                    Reason::SubprocessSpawn,
                    format!("git could not be spawned: {error}"),
                )
                .expected("git on PATH, or RK_GIT_BIN naming it"),
            ))
        })?;
    if !out.status.success() {
        let stderr = String::from_utf8_lossy(&out.stderr);
        if stderr.contains("not a git repository") {
            return Err(GitFailure::NotARepository);
        }
        return Err(GitFailure::Other(RkError::subprocess(
            Diagnostic::new(
                Reason::SubprocessFailed,
                format!(
                    "git {} failed at {target}: {}",
                    args.join(" "),
                    stderr.trim()
                ),
            )
            .expected("git answering for the target, or a target that is not a repository")
            .action("an unreadable history is not an absent one; repair the repository or its ownership before classifying"),
        )));
    }
    Ok(String::from_utf8_lossy(&out.stdout)
        .lines()
        .map(str::trim)
        .filter(|line| !line.is_empty())
        .map(str::to_owned)
        .collect())
}

#[cfg(test)]
mod tests {
    use super::{
        Classification, Evidence, Landing, RELEASE_MARKERS, classify, long_lived_among,
        marker_names,
    };
    use crate::migrate::inventory::{GORELEASER, Kind, PREDECESSORS, SEMANTIC_RELEASE};

    /// Whether a name has the shape of a semantic-release configuration
    /// file, `.releaserc` with or without an extension, or
    /// `release.config.<ext>`.
    fn semantic_release_shaped(name: &str) -> bool {
        name == ".releaserc"
            || name.starts_with(".releaserc.")
            || name.starts_with("release.config.")
    }

    /// `rk assess` and `rk migrate` name the same semantic-release
    /// configuration files: the inventory owns them, the assess list holds
    /// no second copy, and the TypeScript names are among them.
    #[test]
    fn assess_and_the_inventory_name_the_same_semantic_release_files() {
        let inventory: std::collections::BTreeSet<&str> = PREDECESSORS
            .iter()
            .filter(|(path, tool, kind)| {
                (*tool == SEMANTIC_RELEASE && *kind == Kind::ReleaseConfig)
                    || semantic_release_shaped(path)
            })
            .map(|(path, _, _)| *path)
            .collect();
        let assess: std::collections::BTreeSet<&str> = marker_names()
            .filter(|name| semantic_release_shaped(name))
            .collect();
        assert_eq!(assess, inventory);
        assert!(
            !RELEASE_MARKERS
                .iter()
                .any(|name| semantic_release_shaped(name)),
            "the assess list holds a second copy of a semantic-release name"
        );
        for name in [".releaserc.ts", "release.config.ts"] {
            assert!(assess.contains(name), "{name}");
        }
    }

    /// `rk assess` and `rk migrate` name the same `GoReleaser`
    /// configuration files: the inventory owns them, and the assess list
    /// holds no second copy.
    #[test]
    fn assess_and_the_inventory_name_the_same_goreleaser_files() {
        let shaped = |name: &str| name.contains("goreleaser.y");
        let inventory: std::collections::BTreeSet<&str> = PREDECESSORS
            .iter()
            .filter(|(path, tool, kind)| {
                (*tool == GORELEASER && *kind == Kind::ReleaseConfig) || shaped(path)
            })
            .filter(|(path, _, _)| !path.starts_with(".github/"))
            .map(|(path, _, _)| *path)
            .collect();
        let assess: std::collections::BTreeSet<&str> =
            marker_names().filter(|name| shaped(name)).collect();
        assert_eq!(assess, inventory);
        assert_eq!(assess.len(), 6, "{assess:?}");
        assert!(
            !RELEASE_MARKERS.iter().any(|name| shaped(name)),
            "the assess list holds a second copy of a GoReleaser name"
        );
    }

    /// The trunk is never evidence against itself; only the remote
    /// segment is stripped, so a topic branch whose last segment is a
    /// catalog name stays a topic branch; a release line is recognized
    /// by its prefix; a remote HEAD pointer is skipped; each name once.
    #[test]
    fn long_lived_branches_are_read_from_the_full_ref_names() {
        let refs: Vec<String> = [
            "refs/heads/master",
            "refs/remotes/origin/master",
            "refs/remotes/origin/HEAD",
            "refs/heads/develop",
            "refs/remotes/origin/develop",
            "refs/heads/feat/x",
            "refs/heads/feat/develop",
            "refs/remotes/origin/main",
            "refs/heads/release/1.2",
            "refs/remotes/upstream/release/1.2",
        ]
        .iter()
        .map(|name| (*name).to_owned())
        .collect();
        assert_eq!(
            long_lived_among(&refs, "master", "release/"),
            vec!["develop", "main", "release/1.2"]
        );
        assert!(
            long_lived_among(&["refs/heads/master".to_owned()], "master", "release/").is_empty()
        );
        assert!(
            long_lived_among(
                &["refs/heads/feat/develop".to_owned()],
                "master",
                "release/"
            )
            .is_empty()
        );
    }

    #[test]
    fn the_verdict_words_are_the_wire_form() {
        for (classification, word) in [
            (Classification::Greenfield, "greenfield"),
            (Classification::Brownfield, "brownfield"),
            (Classification::NeedsDecision, "needs-decision"),
        ] {
            assert_eq!(classification.as_str(), word);
            assert_eq!(
                serde_json::to_string(&classification).expect("serializes"),
                format!("\"{word}\"")
            );
        }
    }

    fn evidence() -> Evidence {
        Evidence {
            landing: Landing {
                recorded: false,
                rk_version: None,
            },
            technologies: Vec::new(),
            forge: None,
            repo: None,
            release_markers: Vec::new(),
            collisions: Vec::new(),
            git: true,
            tags: 0,
            long_lived_branches: Vec::new(),
        }
    }

    #[test]
    fn nothing_is_greenfield() {
        assert_eq!(classify(&evidence()), Classification::Greenfield);
    }

    #[test]
    fn a_release_marker_or_a_collision_is_brownfield() {
        let with_marker = Evidence {
            release_markers: vec!["CHANGELOG.md".into()],
            ..evidence()
        };
        assert_eq!(classify(&with_marker), Classification::Brownfield);
        let with_collision = Evidence {
            collisions: vec!["release-plz.toml".into()],
            ..evidence()
        };
        assert_eq!(classify(&with_collision), Classification::Brownfield);
    }

    /// A mechanism outranks unexplained activity: tags beside a marker
    /// are a history the mechanism made, not a question.
    #[test]
    fn a_mechanism_beside_activity_is_still_brownfield() {
        let both = Evidence {
            release_markers: vec!["CHANGELOG.md".into()],
            tags: 7,
            long_lived_branches: vec!["develop".into()],
            ..evidence()
        };
        assert_eq!(classify(&both), Classification::Brownfield);
    }

    #[test]
    fn activity_with_no_mechanism_needs_a_decision() {
        let tagged = Evidence {
            tags: 1,
            ..evidence()
        };
        assert_eq!(classify(&tagged), Classification::NeedsDecision);
        let branched = Evidence {
            long_lived_branches: vec!["develop".into()],
            ..evidence()
        };
        assert_eq!(classify(&branched), Classification::NeedsDecision);
    }
}