tga 2.6.0

//! Tier 1.6: classify by JIRA project key mapping.
//!
//! Why: Some organizations dedicate JIRA projects to a particular kind of
//! work (e.g. `INFRA` for platform work, `DATA` for data-pipeline features,
//! `TQL` for an existing-product bug tracker). When a commit references a
//! ticket from such a project, the project key itself is a strong
//! classification signal that no amount of message parsing can reliably
//! reproduce. See issue #206.
//!
//! What: Extracts the JIRA project key from a commit message (the prefix
//! of the first `PROJ-123`-style identifier) and looks it up in a
//! user-configured `HashMap<String, String>`. On hit, returns a verdict
//! with confidence [`DEFAULT_PROJECT_MAPPING_CONFIDENCE`] (0.88) — or the
//! caller-supplied confidence via
//! [`JiraProjectTier::with_taxonomy_and_confidence`].
//!
//! Test: Configure mappings `{"INFRA": "platform"}`, classify
//! `"INFRA-42 fix nginx"`, and assert the verdict is `"platform"` with the
//! configured confidence; assert that an unmapped key (`"FOO-1"`) yields
//! `None`.

use std::collections::HashMap;

use std::sync::OnceLock;

use regex::Regex;

use crate::classify::taxonomy::{TaxonomyRegistry, TopLevelCategory};
use crate::classify::tiers::ClassificationResult;
use crate::core::models::ClassificationMethod;

/// JIRA-style key pattern: uppercase project prefix + numeric suffix.
///
/// Compiled once on first access via [`OnceLock`]. If the (fixed) pattern
/// fails to compile, lookups simply return `None` rather than panicking.
fn jira_key_re() -> Option<&'static Regex> {
    static RE: OnceLock<Option<Regex>> = OnceLock::new();
    RE.get_or_init(|| Regex::new(r"\b([A-Z][A-Z0-9]+)-\d+\b").ok())
        .as_ref()
}

/// Default confidence assigned to every verdict from this tier.
///
/// Why: the JIRA project mapping (issue #206) is a tiebreaker — strong
/// enough to outrank the generic regex `jira-ticket` rule (confidence
/// 0.7) but soft enough that Tier-0 manual overrides (1.0) and exact-
/// keyword conventional-commit prefixes (0.95) still win. The 0.88
/// default sits in this gap by design.
pub const DEFAULT_PROJECT_MAPPING_CONFIDENCE: f64 = 0.88;

/// Tier-1.6 JIRA project-key classifier.
pub struct JiraProjectTier {
    mappings: HashMap<String, String>,
    taxonomy: TaxonomyRegistry,
    confidence: f64,
}

impl JiraProjectTier {
    /// Construct a new tier with the given `project_key → work_type` map.
    ///
    /// Keys are normalized to uppercase on insert so callers don't have to
    /// pre-uppercase their config. Confidence defaults to
    /// [`DEFAULT_PROJECT_MAPPING_CONFIDENCE`].
    pub fn new(mappings: HashMap<String, String>) -> Self {
        Self::with_taxonomy(mappings, TaxonomyRegistry::with_builtins())
    }

    /// Construct with a custom taxonomy registry (lets user-defined
    /// subcategories resolve to a top-level parent).
    pub fn with_taxonomy(mappings: HashMap<String, String>, taxonomy: TaxonomyRegistry) -> Self {
        Self::with_taxonomy_and_confidence(mappings, taxonomy, DEFAULT_PROJECT_MAPPING_CONFIDENCE)
    }

    /// Construct with a custom taxonomy and an explicit per-verdict
    /// confidence score.
    ///
    /// Why: issue #206 specifies the confidence as configurable so users
    /// can tune how aggressively the JIRA mapping overrides downstream
    /// regex/fuzzy verdicts.
    /// What: stores `confidence` verbatim and emits it on every hit.
    /// Test: covered by `confidence_override_threads_through` below.
    pub fn with_taxonomy_and_confidence(
        mappings: HashMap<String, String>,
        taxonomy: TaxonomyRegistry,
        confidence: f64,
    ) -> Self {
        let normalized = mappings
            .into_iter()
            .map(|(k, v)| (k.to_uppercase(), v))
            .collect();
        Self {
            mappings: normalized,
            taxonomy,
            confidence,
        }
    }

    /// Borrow the underlying mappings (primarily for tests / diagnostics).
    pub fn mappings(&self) -> &HashMap<String, String> {
        &self.mappings
    }

    /// Returns `true` when no mappings are configured (cheap check used by
    /// the engine to skip this tier entirely).
    pub fn is_empty(&self) -> bool {
        self.mappings.is_empty()
    }

    /// Extract the first JIRA project key from `commit_message` and look it
    /// up in the configured mappings.
    ///
    /// Returns a verdict with confidence 0.95 on hit, or `None` if no
    /// ticket reference is present or its project key isn't mapped.
    pub fn classify(&self, commit_message: &str) -> Option<ClassificationResult> {
        if self.mappings.is_empty() {
            return None;
        }
        let re = jira_key_re()?;
        let caps = re.captures(commit_message)?;
        let project = caps.get(1)?.as_str().to_uppercase();
        let category = self.mappings.get(&project)?.clone();

        let ticket_id = caps.get(0).map(|m| m.as_str().to_string());
        let top_level = self
            .taxonomy
            .resolve(&category)
            .unwrap_or(TopLevelCategory::Unknown);

        Some(ClassificationResult {
            category,
            subcategory: None,
            top_level: Some(top_level),
            confidence: self.confidence,
            // Why: the verdict comes from an external JIRA project-key mapping
            // (operator-supplied semantic context), not from a regex pattern
            // in the commit message. Labelling it `RegexRule` caused all
            // JIRA-project-mapped commits to appear as `regex_rule` in the
            // `classifications.method` column, hiding the JIRA contribution
            // from analytics dashboards (issue #319 / tga 1.5.3 regression
            // fix). `ExternalSource` is the correct label — it covers all
            // paths where classification is driven by external ticket-system
            // metadata rather than message-text heuristics.
            method: ClassificationMethod::ExternalSource,
            ticket_id,
            complexity: None,
        })
    }
}

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

    fn fixture() -> JiraProjectTier {
        let mut m = HashMap::new();
        m.insert("INFRA".to_string(), "platform".to_string());
        m.insert("DATA".to_string(), "feature".to_string());
        JiraProjectTier::new(m)
    }

    #[test]
    fn hit_returns_mapped_category() {
        let t = fixture();
        let r = t.classify("INFRA-42 fix nginx config").expect("hit");
        assert_eq!(r.category, "platform");
        assert!(
            (r.confidence - DEFAULT_PROJECT_MAPPING_CONFIDENCE).abs() < 1e-9,
            "default confidence is {DEFAULT_PROJECT_MAPPING_CONFIDENCE}"
        );
        assert_eq!(r.ticket_id.as_deref(), Some("INFRA-42"));
    }

    /// Why: the confidence-override constructor is the issue-#206 hook
    /// for tuning how aggressively the JIRA mapping overrides downstream
    /// tiers; a regression where the value is silently dropped would
    /// break that operator knob.
    /// What: build a tier with confidence=0.5 and assert the verdict
    /// reflects it.
    /// Test: pure constructor exercise.
    #[test]
    fn confidence_override_threads_through() {
        let mut m = HashMap::new();
        m.insert("INFRA".to_string(), "platform".to_string());
        let t = JiraProjectTier::with_taxonomy_and_confidence(
            m,
            TaxonomyRegistry::with_builtins(),
            0.5,
        );
        let r = t.classify("INFRA-1 any").expect("hit");
        assert!((r.confidence - 0.5).abs() < 1e-9);
    }

    #[test]
    fn unmapped_project_returns_none() {
        let t = fixture();
        assert!(t.classify("FOO-1 some change").is_none());
    }

    #[test]
    fn no_ticket_returns_none() {
        let t = fixture();
        assert!(t.classify("fix: something without a ticket").is_none());
    }

    #[test]
    fn empty_mappings_short_circuits() {
        let t = JiraProjectTier::new(HashMap::new());
        assert!(t.is_empty());
        assert!(t.classify("INFRA-1 anything").is_none());
    }

    #[test]
    fn mappings_normalized_to_uppercase() {
        let mut m = HashMap::new();
        m.insert("infra".to_string(), "platform".to_string());
        let t = JiraProjectTier::new(m);
        let r = t.classify("INFRA-7 patch").expect("hit");
        assert_eq!(r.category, "platform");
    }
}