commitbot 0.6.2

A CLI assistant that generates commit and PR messages from your diffs using LLMs.
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

use commitbot::git::{
    find_first_pr_number, format_pr_commit_appendix_with_remote, parse_remote_repo,
    short_commit_hash, split_diff_by_file, PrItem, PrSummaryMode,
};

#[test]
fn parses_github_ssh_remote() {
    let remote = parse_remote_repo("git@github.com:owner/repo.git").unwrap();
    assert_eq!(remote.provider, commitbot::git::GitProvider::GitHub);
    assert_eq!(remote.repo_id().as_deref(), Some("owner/repo"));
    assert_eq!(
        remote.commit_url("abcdef123456").as_deref(),
        Some("https://github.com/owner/repo/commit/abcdef123456")
    );
}

#[test]
fn parses_gitlab_https_remote() {
    let remote = parse_remote_repo("https://gitlab.example.com/group/subgroup/repo.git").unwrap();
    assert_eq!(remote.provider, commitbot::git::GitProvider::GitLab);
    assert_eq!(remote.repo_id().as_deref(), Some("subgroup/repo"));
    assert_eq!(
        remote.commit_url("abcdef123456").as_deref(),
        Some("https://gitlab.example.com/group/subgroup/repo/-/commit/abcdef123456")
    );
}

#[test]
fn parses_azure_ssh_remote() {
    let remote = parse_remote_repo("git@ssh.dev.azure.com:v3/org/project/repo").unwrap();
    assert_eq!(remote.provider, commitbot::git::GitProvider::AzureDevOps);
    assert_eq!(remote.repo_id().as_deref(), Some("project/repo"));
    assert_eq!(
        remote.commit_url("abcdef123456").as_deref(),
        Some("https://dev.azure.com/org/project/_git/repo/commit/abcdef123456")
    );
}

#[test]
fn appendix_falls_back_to_hash_only_when_no_remote() {
    let items = vec![PrItem {
        commit_hash: "abcdef123456".to_string(),
        title: "Refine PR footer rendering".to_string(),
        body: String::new(),
        pr_number: None,
    }];

    let appendix = format_pr_commit_appendix_with_remote(&items, None);
    assert!(appendix.contains("Commits in this PR:"));
    assert!(appendix.contains("- `abcdef1` Refine PR footer rendering"));
}

#[test]
fn find_first_pr_number_in_title() {
    let result = find_first_pr_number("Fix bug in #123");
    assert_eq!(result, Some(123));
}

#[test]
fn find_first_pr_number_in_body() {
    let result = find_first_pr_number("Closes #456");
    assert_eq!(result, Some(456));
}

#[test]
fn find_first_pr_number_multiple_hashes() {
    let result = find_first_pr_number("Related to #100, fixes #200");
    assert_eq!(result, Some(100));
}

#[test]
fn find_first_pr_number_no_hash() {
    let result = find_first_pr_number("No PR here");
    assert_eq!(result, None);
}

#[test]
fn find_first_pr_number_empty() {
    let result = find_first_pr_number("");
    assert_eq!(result, None);
}

#[test]
fn split_diff_by_file_single_file() {
    let diff = r#"diff --git a/src/main.rs b/src/main.rs
index 1234567..89abcdef 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -1,3 +1,4 @@
+use std::io;
 fn main() {
     println!("Hello");
 }"#;
    let result = split_diff_by_file(diff);
    assert_eq!(result.len(), 1);
    assert_eq!(result[0].0, "src/main.rs");
    assert!(result[0].1.contains("diff --git"));
}

#[test]
fn split_diff_by_file_multiple_files() {
    let diff = r#"diff --git a/src/main.rs b/src/main.rs
index 1234567..89abcdef 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -1,3 +1,4 @@
+use std::io;
diff --git a/src/lib.rs b/src/lib.rs
index abcdefg..1234567 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -1,2 +1,3 @@
+pub fn helper() {}
 pub fn other() {}"#;
    let result = split_diff_by_file(diff);
    assert_eq!(result.len(), 2);
    assert_eq!(result[0].0, "src/main.rs");
    assert_eq!(result[1].0, "src/lib.rs");
}

#[test]
fn split_diff_by_file_empty() {
    let result = split_diff_by_file("");
    assert!(result.is_empty());
}

#[test]
fn short_commit_hash_works() {
    let result = short_commit_hash("abcdef123456");
    assert_eq!(result, "abcdef1");
}

#[test]
fn short_commit_hash_short_input() {
    let result = short_commit_hash("abc");
    assert_eq!(result, "abc");
}

#[test]
fn pr_summary_mode_as_str() {
    assert_eq!(PrSummaryMode::ByCommits.as_str(), "commits");
    assert_eq!(PrSummaryMode::ByPrs.as_str(), "prs");
}