cha-core 1.9.0

Core analysis engine for Cha — pluggable code smell detection
Documentation 
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
67
68
69
70
71
72

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

/// Detect high coupling via excessive imports.
pub struct CouplingAnalyzer {
    pub max_imports: usize,
}

impl Default for CouplingAnalyzer {
    fn default() -> Self {
        Self { max_imports: 15 }
    }
}

impl Plugin for CouplingAnalyzer {
    fn name(&self) -> &str {
        "coupling"
    }

    fn smells(&self) -> Vec<String> {
        vec!["high_coupling".into()]
    }

    fn description(&self) -> &str {
        "Too many imports (high coupling)"
    }

    fn analyze(&self, ctx: &AnalysisContext) -> Vec<Finding> {
        let mut findings = Vec::new();

        // Skip Rust mod declarations — module organization, not coupling
        // Skip module declarations — module organization, not coupling
        let import_count = ctx
            .model
            .imports
            .iter()
            .filter(|i| !i.is_module_decl)
            .count();

        let first = ctx.model.imports.first().map(|i| i.line).unwrap_or(1);
        let first_col = ctx.model.imports.first().map(|i| i.col).unwrap_or(0);
        let last = ctx.model.imports.last().map(|i| i.line).unwrap_or(1);

        if import_count > self.max_imports {
            findings.push(Finding {
                smell_name: "high_coupling".into(),
                category: SmellCategory::Couplers,
                severity: if import_count > self.max_imports * 2 {
                    Severity::Error
                } else {
                    Severity::Warning
                },
                location: Location {
                    path: ctx.file.path.clone(),
                    start_line: first,
                    start_col: first_col,
                    end_line: last,
                    name: None,
                    ..Default::default()
                },
                message: format!(
                    "File has {} imports (threshold: {})",
                    import_count, self.max_imports
                ),
                suggested_refactorings: vec!["Move Method".into(), "Extract Class".into()],
                actual_value: Some(import_count as f64),
                threshold: Some(self.max_imports as f64),
            });
        }

        findings
    }
}