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

use clap::Parser;
use commitbot::config::Config;
use commitbot::Cli;
use std::fs;
use std::path::PathBuf;
use std::time::{SystemTime, UNIX_EPOCH};

fn unique_config_path(name: &str) -> PathBuf {
    let nanos = SystemTime::now()
        .duration_since(UNIX_EPOCH)
        .expect("system time before unix epoch")
        .as_nanos();
    std::env::temp_dir().join(format!("commitbot_{name}_{nanos}.toml"))
}

fn write_temp_config(name: &str, contents: &str) -> PathBuf {
    let path = unique_config_path(name);
    fs::write(&path, contents).expect("write temp config");
    path
}

#[test]
fn missing_config_uses_cli_values() {
    let missing_path = unique_config_path("missing");
    let cli = Cli::parse_from([
        "commitbot",
        "--config",
        missing_path.to_str().expect("utf-8 path"),
        "--provider",
        "ollama",
        "--model",
        "qwen3-coder:30b",
        "--url",
        "http://localhost:11434",
    ]);

    let cfg = Config::from_sources(&cli).expect("config should resolve from cli");
    assert_eq!(cfg.provider, "ollama");
    assert_eq!(cfg.model, "qwen3-coder:30b");
    assert_eq!(cfg.base_url.as_deref(), Some("http://localhost:11434"));
    assert!(cfg.stream);
}

#[test]
fn reads_values_from_default_config_table() {
    let config_path = write_temp_config(
        "default_table",
        r#"
[default]
provider = "ollama"
model = "llama3.1:8b-instruct-q5_K_M"
url = "http://gpu.trigapi.com:11434"
max_concurrent_requests = 2
stream = false
"#,
    );

    let cli = Cli::parse_from([
        "commitbot",
        "--config",
        config_path.to_str().expect("utf-8 path"),
    ]);

    let cfg = Config::from_sources(&cli).expect("config should load default table");
    assert_eq!(cfg.provider, "ollama");
    assert_eq!(cfg.model, "llama3.1:8b-instruct-q5_K_M");
    assert_eq!(cfg.base_url.as_deref(), Some("http://gpu.trigapi.com:11434"));
    assert_eq!(cfg.max_concurrent_requests, 2);
    assert!(!cfg.stream);

    fs::remove_file(config_path).ok();
}

#[test]
fn cli_overrides_file_values() {
    let config_path = write_temp_config(
        "cli_override",
        r#"
[default]
provider = "ollama"
model = "qwen3-coder:30b"
url = "http://gpu.trigapi.com:11434"
max_concurrent_requests = 1
stream = true
"#,
    );

    let cli = Cli::parse_from([
        "commitbot",
        "--config",
        config_path.to_str().expect("utf-8 path"),
        "--model",
        "gpt-oss:20b",
        "--url",
        "http://localhost:11434",
        "--max",
        "4",
        "--no-stream",
    ]);

    let cfg = Config::from_sources(&cli).expect("cli values should override file");
    assert_eq!(cfg.provider, "ollama");
    assert_eq!(cfg.model, "gpt-oss:20b");
    assert_eq!(cfg.base_url.as_deref(), Some("http://localhost:11434"));
    assert_eq!(cfg.max_concurrent_requests, 4);
    assert!(!cfg.stream);

    fs::remove_file(config_path).ok();
}

#[test]
fn invalid_openai_config_returns_error() {
    let config_path = write_temp_config(
        "invalid_openai",
        r#"
[default]
provider = "openai"
model = "gpt-5-nano"
"#,
    );

    let cli = Cli::parse_from([
        "commitbot",
        "--config",
        config_path.to_str().expect("utf-8 path"),
    ]);

    let err = Config::from_sources(&cli).expect_err("openai config should require an api key");
    assert!(
        err.to_string()
            .contains("OPENAI_API_KEY must be set via CLI, env var, or config file")
    );

    fs::remove_file(config_path).ok();
}