repopilot 0.15.0

Local-first CLI for repository audit, architecture risk detection, baseline tracking, and CI-friendly code 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

use super::*;
use crate::baseline::model::{BASELINE_SCHEMA_VERSION, BASELINE_TOOL, Baseline, BaselineFinding};
use crate::findings::types::{Evidence, Finding, Severity};
use crate::scan::types::{ScanArtifacts, ScanMetadata, ScanSummary};
use std::path::PathBuf;

fn make_finding(rule_id: &str, path: &str, line: usize) -> Finding {
    Finding {
        id: format!("{rule_id}-test"),
        rule_id: rule_id.to_string(),
        title: "Test".to_string(),
        severity: Severity::High,
        evidence: vec![Evidence {
            path: PathBuf::from(path),
            line_start: line,
            line_end: None,
            snippet: String::new(),
        }],
        ..Default::default()
    }
}

fn make_summary(findings: Vec<Finding>) -> ScanSummary {
    ScanSummary {
        metadata: ScanMetadata {
            root_path: PathBuf::from("/project"),
            ..Default::default()
        },
        artifacts: ScanArtifacts {
            findings,
            ..Default::default()
        },
        ..Default::default()
    }
}

fn baseline_with_keys(keys: Vec<String>) -> Baseline {
    Baseline {
        schema_version: BASELINE_SCHEMA_VERSION,
        tool: BASELINE_TOOL.to_string(),
        created_at: "2024-01-01T00:00:00Z".to_string(),
        root: "/project".to_string(),
        findings: keys
            .into_iter()
            .map(|key| BaselineFinding {
                key: key.clone(),
                rule_id: "test.rule".to_string(),
                severity: "HIGH".to_string(),
                path: "src/main.rs".to_string(),
                message: "Test".to_string(),
            })
            .collect(),
    }
}

#[test]
fn new_finding_not_in_baseline_is_marked_new() {
    let finding = make_finding("test.rule", "src/main.rs", 10);
    let summary = make_summary(vec![finding]);
    let baseline = baseline_with_keys(vec![]);
    let baseline_path = PathBuf::from(".repopilot/baseline.json");

    let report = diff_summary_against_baseline(summary, &baseline, baseline_path);

    assert_eq!(report.findings[0].status, BaselineStatus::New);
    assert_eq!(report.new_count(), 1);
    assert_eq!(report.existing_count(), 0);
}

#[test]
fn finding_in_baseline_is_marked_existing() {
    let finding = make_finding("test.rule", "src/main.rs", 10);
    let root = PathBuf::from("/project");
    let key = stable_finding_key(&finding, &root);
    let summary = make_summary(vec![finding]);
    let baseline = baseline_with_keys(vec![key]);
    let baseline_path = PathBuf::from(".repopilot/baseline.json");

    let report = diff_summary_against_baseline(summary, &baseline, baseline_path);

    assert_eq!(report.findings[0].status, BaselineStatus::Existing);
    assert_eq!(report.new_count(), 0);
    assert_eq!(report.existing_count(), 1);
}

#[test]
fn legacy_line_based_baseline_key_matches_after_line_shift() {
    let finding = make_finding("test.rule", "src/main.rs", 12);
    let summary = make_summary(vec![finding]);
    let baseline = baseline_with_keys(vec!["test.rule:src/main.rs:10".to_string()]);
    let baseline_path = PathBuf::from(".repopilot/baseline.json");

    let report = diff_summary_against_baseline(summary, &baseline, baseline_path);

    assert_eq!(report.findings[0].status, BaselineStatus::Existing);
}

#[test]
fn all_findings_new_marks_every_finding_as_new() {
    let summary = make_summary(vec![
        make_finding("rule.one", "src/a.rs", 1),
        make_finding("rule.two", "src/b.rs", 2),
    ]);

    let report = all_findings_new(summary);

    assert_eq!(report.new_count(), 2);
    assert_eq!(report.existing_count(), 0);
    assert!(report.baseline_path.is_none());
}

#[test]
fn mixed_new_and_existing_findings() {
    let new_finding = make_finding("rule.new", "src/new.rs", 5);
    let existing_finding = make_finding("rule.existing", "src/existing.rs", 10);
    let root = PathBuf::from("/project");
    let existing_key = stable_finding_key(&existing_finding, &root);

    let summary = make_summary(vec![new_finding, existing_finding]);
    let baseline = baseline_with_keys(vec![existing_key]);

    let report = diff_summary_against_baseline(
        summary,
        &baseline,
        PathBuf::from(".repopilot/baseline.json"),
    );

    assert_eq!(report.new_count(), 1);
    assert_eq!(report.existing_count(), 1);
}