abcop 0.16.0

Blazing-fast, opinionated ABC complexity linter for humans and LLM agents
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

//! C# candidate-evaluation policy over the shared scope model: the
//! RHS purity whitelist and the straight-line veto/boundary lists.

use tree_sitter::Node;

use super::CSharpFile;
use crate::scope_model;

/// Ancestors that mark a write as conditional.
const VETO_KINDS: &[&str] = &[
    "if_statement",
    "for_statement",
    "foreach_statement",
    "while_statement",
    "do_statement",
    "switch_statement",
    "try_statement",
    "catch_clause",
];

/// Ancestors that end the straight-line check (unit boundaries).
const OWNER_KINDS: &[&str] = &["method_declaration", "constructor_declaration"];

static CSHARP_SEMANTICS: scope_model::Semantics = scope_model::Semantics {
    pure: pure,
    veto: VETO_KINDS,
    owners: OWNER_KINDS,
    include_root_scope: false,
};

pub fn used_once_offenses(fm: &CSharpFile) -> Vec<crate::used_once::UsedOnceOffense> {
    scope_model::used_once_offenses(
        fm.tree.root_node(),
        &|byte| fm.line_col(byte),
        &fm.scopes,
        &CSHARP_SEMANTICS,
    )
}

pub fn never_used_offenses(fm: &CSharpFile) -> Vec<crate::never_used::NeverUsedOffense> {
    scope_model::never_used_offenses(&|byte| fm.line_col(byte), &fm.scopes, &CSHARP_SEMANTICS)
}

/// Conservative RHS purity: literals, arrays of literals, and operator
/// compositions over them. Interpolated strings reference variables and
/// are rejected via their children.
fn pure(n: Node) -> bool {
    match n.kind() {
        "integer_literal" | "real_literal" | "character_literal" | "string_literal"
        | "boolean_literal" | "null_literal" => true,
        "parenthesized_expression"
        | "binary_expression"
        | "unary_expression"
        | "prefix_unary_expression"
        | "postfix_unary_expression" => children_pure(n),
        _ => false,
    }
}

fn children_pure(n: Node) -> bool {
    let mut c = n.walk();
    n.children(&mut c)
        .filter(|ch| ch.is_named())
        .all(|ch| pure(ch))
}