issue_links.rs

use std::sync::LazyLock;

use regex::Regex;

use crate::{
    config::{IssueLinksCheckCommitsConfig, IssueLinksConfig},
    github::GithubCommit,
    handlers::check_commits::MERGE_IGNORE_LIST,
};

static LINKED_RE: LazyLock<Regex> =
    LazyLock::new(|| Regex::new(r"\B(?P<org>[a-zA-Z-_]+/[a-zA-Z-_]+)?(#[0-9]+)\b").unwrap());

pub(super) fn issue_links_in_commits(
    conf: &IssueLinksConfig,
    commits: &[GithubCommit],
) -> Option<String> {
    let does_match = match conf.check_commits {
        IssueLinksCheckCommitsConfig::Off => return None,
        IssueLinksCheckCommitsConfig::All => has_issue_link,
        IssueLinksCheckCommitsConfig::Uncanonicalized => has_uncanonicalized_issue_link,
    };

    let issue_links_commits = commits
        .iter()
        .filter(|c| {
            !MERGE_IGNORE_LIST
                .iter()
                .any(|i| c.commit.message.starts_with(i))
        })
        .filter(|c| does_match(&c.commit.message))
        .map(|c| format!("- {}\n", c.sha))
        .collect::<String>();

    if issue_links_commits.is_empty() {
        None
    } else if matches!(
        conf.check_commits,
        IssueLinksCheckCommitsConfig::Uncanonicalized
    ) {
        Some(format!(
            r"There are uncanonicalized issue links (such as `#123`) in the commit messages of the following commits.
*Please add the organization and repository before the issue number (like so `rust-lang/rust#123`) to avoid issues with subtree.*
{issue_links_commits}",
        ))
    } else {
        Some(format!(
            r"There are issue links (such as `#123`) in the commit messages of the following commits.
*Please move them to the PR description, to avoid spamming the issues with references to the commit, and so this bot can automatically canonicalize them to avoid issues with subtree.*
{issue_links_commits}",
        ))
    }
}

fn has_issue_link(text: &str) -> bool {
    LINKED_RE.is_match(text)
}

fn has_uncanonicalized_issue_link(text: &str) -> bool {
    let Some(caps) = LINKED_RE.captures(text) else {
        return false;
    };
    caps.name("org").is_none()
}

#[test]
fn test_mentions_in_commits() {
    use super::dummy_commit_from_body;

    let config = IssueLinksConfig {
        check_commits: IssueLinksCheckCommitsConfig::All,
    };

    let mut commits = vec![dummy_commit_from_body(
        "d1992a392617dfb10518c3e56446b6c9efae38b0",
        "This is simple without issue links!",
    )];

    assert_eq!(issue_links_in_commits(&config, &commits), None);

    commits.push(dummy_commit_from_body(
        "86176475acda9c775f844f5ad2470f05aebd4249",
        "Rollup merge of #123\n\nWe ignore the issue link for Rollup merge of",
    ));
    commits.push(dummy_commit_from_body(
        "8009423d53d30b56d8cf0fec08f9852329a1a9a4",
        "Auto merge of #123\n\nWe ignore the issue link for Auto merge of",
    ));
    commits.push(dummy_commit_from_body(
        "1eeacf822f6c11cd10713ddcb54a72352cacb2c2",
        "Merge pull request #2236 from rust-lang/rustc-pull",
    ));

    assert_eq!(issue_links_in_commits(&config, &commits), None);

    commits.push(dummy_commit_from_body(
        "d7daa17bc97df9377640b0d33cbd0bbeed703c3a",
        "This is a body with a issue link #123.",
    ));

    assert_eq!(
        issue_links_in_commits(&config, &commits),
        Some(
            r"There are issue links (such as `#123`) in the commit messages of the following commits.
*Please move them to the PR description, to avoid spamming the issues with references to the commit, and so this bot can automatically canonicalize them to avoid issues with subtree.*
- d7daa17bc97df9377640b0d33cbd0bbeed703c3a
".to_string()
        )
    );

    assert_eq!(
        issue_links_in_commits(
            &IssueLinksConfig {
                check_commits: IssueLinksCheckCommitsConfig::Off,
            },
            &commits
        ),
        None
    );

    commits.push(dummy_commit_from_body(
        "891f0916a07c215ae8173f782251422f1fea6acb",
        "This is a body with a issue link (rust-lang/rust#123).",
    ));

    assert_eq!(
        issue_links_in_commits(&config, &commits),
        Some(
            r"There are issue links (such as `#123`) in the commit messages of the following commits.
*Please move them to the PR description, to avoid spamming the issues with references to the commit, and so this bot can automatically canonicalize them to avoid issues with subtree.*
- d7daa17bc97df9377640b0d33cbd0bbeed703c3a
- 891f0916a07c215ae8173f782251422f1fea6acb
".to_string()
        )
    );
}

#[test]
fn uncanonicalized() {
    use super::dummy_commit_from_body;

    let config = IssueLinksConfig {
        check_commits: IssueLinksCheckCommitsConfig::Uncanonicalized,
    };

    let mut commits = vec![dummy_commit_from_body(
        "d1992a392617dfb10518c3e56446b6c9efae38b0",
        "This is simple without issue links!",
    )];

    assert_eq!(issue_links_in_commits(&config, &commits), None);

    commits.push(dummy_commit_from_body(
        "86176475acda9c775f844f5ad2470f05aebd4249",
        "Test for canonicalized rust-lang/rust#123",
    ));

    assert_eq!(issue_links_in_commits(&config, &commits), None);

    commits.push(dummy_commit_from_body(
        "fererfe5acda9c775f844f5ad2470f05aebd4249",
        "Test for uncanonicalized #123",
    ));

    assert_eq!(
        issue_links_in_commits(&config, &commits),
        Some(
            r"There are uncanonicalized issue links (such as `#123`) in the commit messages of the following commits.
*Please add the organization and repository before the issue number (like so `rust-lang/rust#123`) to avoid issues with subtree.*
- fererfe5acda9c775f844f5ad2470f05aebd4249
".to_string()
        )
    );
}