cha-core 0.6.5

Core analysis engine for Cha — pluggable code smell detection
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66

use crate::{AnalysisContext, Finding, Location, Plugin, Severity, SmellCategory};

/// Configurable thresholds for cyclomatic complexity.
pub struct ComplexityAnalyzer {
    pub warn_threshold: usize,
    pub error_threshold: usize,
}

impl Default for ComplexityAnalyzer {
    fn default() -> Self {
        Self {
            warn_threshold: 10,
            error_threshold: 20,
        }
    }
}

impl Plugin for ComplexityAnalyzer {
    fn name(&self) -> &str {
        "complexity"
    }

    fn description(&self) -> &str {
        "Cyclomatic complexity exceeds threshold"
    }

    fn analyze(&self, ctx: &AnalysisContext) -> Vec<Finding> {
        ctx.model
            .functions
            .iter()
            .filter_map(|f| self.check_function(ctx, f))
            .collect()
    }
}

impl ComplexityAnalyzer {
    /// Build a finding for a single function if its complexity exceeds the threshold.
    fn check_function(&self, ctx: &AnalysisContext, f: &crate::FunctionInfo) -> Option<Finding> {
        if f.complexity < self.warn_threshold {
            return None;
        }
        Some(Finding {
            smell_name: "high_complexity".into(),
            category: SmellCategory::Bloaters,
            severity: if f.complexity >= self.error_threshold {
                Severity::Error
            } else {
                Severity::Warning
            },
            location: Location {
                path: ctx.file.path.clone(),
                start_line: f.start_line,
                end_line: f.end_line,
                name: Some(f.name.clone()),
            },
            message: format!(
                "Function `{}` has complexity {} (threshold: {})",
                f.name, f.complexity, self.warn_threshold
            ),
            suggested_refactorings: vec![
                "Extract Method".into(),
                "Replace Conditional with Polymorphism".into(),
            ],
        })
    }
}