drep-ai 2.3.0

A local commit gate: runs the linters your repo configures, and sends changed code to an LLM for review
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
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146

//! `result::AnalysisResult` — criteria 5-7.

use std::collections::BTreeMap;
use std::path::PathBuf;

use crate::analysis::findings::{Finding, Severity};
use crate::analysis::result::{AnalysisResult, FailureReason};
use crate::llm::error::{BackendErrorKind, LlmError};

/// Criterion 5: `merge` unions `failed_files`. Two results that each name
/// the same file merge to a map of size 1, not 2.
#[test]
fn merge_unions_failed_files() {
    let mut a = AnalysisResult::default();
    a.failed_files.insert(
        PathBuf::from("src/lib.rs"),
        FailureReason::Transport {
            status: Some(500),
            message: "boom".to_owned(),
        },
    );
    let mut b = AnalysisResult::default();
    b.failed_files.insert(
        PathBuf::from("src/lib.rs"),
        FailureReason::Transport {
            status: Some(500),
            message: "boom".to_owned(),
        },
    );

    a.merge(b);

    assert_eq!(
        a.failed_files.len(),
        1,
        "the same file merged twice must collapse to one entry, got {:?}",
        a.failed_files
    );
    assert!(
        a.failed_files.contains_key(&PathBuf::from("src/lib.rs")),
        "the merged file must still be present, got {:?}",
        a.failed_files
    );
}

/// Criterion 6: `merge` concatenates findings and sums `dropped_out_of_range`.
#[test]
fn merge_concatenates_findings_and_sums_dropped_out_of_range() {
    let mut a = AnalysisResult::default();
    a.findings.push(Finding {
        kind: "k1".to_owned(),
        severity: Severity::Info,
        file_path: "f.py".to_owned(),
        line: 1,
        column: None,
        message: "m1".to_owned(),
        suggestion: None,
    });
    a.dropped_out_of_range = 2;

    let mut b = AnalysisResult::default();
    b.findings.push(Finding {
        kind: "k2".to_owned(),
        severity: Severity::Warning,
        file_path: "f.py".to_owned(),
        line: 2,
        column: None,
        message: "m2".to_owned(),
        suggestion: None,
    });
    b.dropped_out_of_range = 3;

    a.merge(b);

    assert_eq!(
        a.findings.len(),
        2,
        "findings must concatenate, got {:?}",
        a.findings
    );
    assert_eq!(
        a.findings[0].kind, "k1",
        "findings must concatenate in order, got {:?}",
        a.findings
    );
    assert_eq!(
        a.findings[1].kind, "k2",
        "findings must concatenate in order, got {:?}",
        a.findings
    );
    assert_eq!(
        a.dropped_out_of_range, 5,
        "dropped_out_of_range must sum, not take max or last"
    );
}

/// Criterion 7: `has_failures` is false on a default result and true once a
/// file is added.
#[test]
fn has_failures_reflects_failed_files() {
    let mut result = AnalysisResult::default();
    assert!(
        !result.has_failures(),
        "a default result has no failures, got true"
    );

    result.failed_files.insert(
        PathBuf::from("src/lib.rs"),
        FailureReason::Transport {
            status: None,
            message: "x".to_owned(),
        },
    );
    assert!(
        result.has_failures(),
        "adding a file must flip has_failures to true"
    );
}

/// The `failed_files` field is what makes merging safe; the type is
/// declared as `BTreeMap<PathBuf, FailureReason>` so the merge test above
/// is the contract. This pins that the field is not accidentally a `Vec`
/// or a `HashSet` — a `Vec` would silently break the union semantics and a
/// `HashSet` would survive but make test reproducibility harder.
#[test]
fn failed_files_is_a_btreemap() {
    let result = AnalysisResult::default();
    let _: BTreeMap<PathBuf, FailureReason> = result.failed_files;
}

#[test]
fn a_typed_backend_failure_is_not_misreported_as_transport() {
    let reason = crate::analysis::code_quality::into_failure_reason(LlmError::Backend {
        kind: BackendErrorKind::Contract,
        message: "Codex attempted a forbidden tool".to_owned(),
    });

    assert!(matches!(
        reason,
        FailureReason::Backend {
            kind: BackendErrorKind::Contract,
            ..
        }
    ));
    assert!(!reason.one_line().contains("transport"));
}